Fix CI builds and seek-to-resume desync

- CMakeLists.txt: gate -Wl,--gc-sections behind NOT APPLE since macOS
  ld64 doesn't accept it.
- tools/build-linux-release.sh: pass -Db_lto=false explicitly so a
  cached meson build dir (in CI) can't keep LTO on; ar/ranlib without
  the lto plugin silently dropped wayland interface symbols.
- PlayerController.cpp: drop the rebase in playbackPositionFromMpvTime
  that added m_resumeStartSeconds to mpv's reported time when below the
  resume point. It made click-to-seek to a position before the resume
  point bounce back to the resume point on both Starfish and desktop.
- PlayerController.cpp: use absolute+exact for click-committed seeks so
  they land on the requested frame instead of a distant keyframe; the
  scrub path still uses keyframes for low-latency drag feedback.

Author: sachk

CMakeLists.txt

@@ -246,14 +246,18 @@ if(NOT BUILD_SHARED_LIBS)
             Qt6::QJpegPlugin 
             Qt6::QWebpPlugin
     )
-    # Add linker GC to strip unused code from static libs
-    target_link_options(jellyfin-native PRIVATE
-        "-Wl,--gc-sections"
-    )
-    target_compile_options(jellyfin-native PRIVATE
-        "-ffunction-sections"
-        "-fdata-sections"
-    )
+    # Add linker GC to strip unused code from static libs.
+    # macOS ld64 doesn't accept --gc-sections (uses -dead_strip instead, on by
+    # default at -O), so only apply this on non-Apple platforms.
+    if(NOT APPLE)
+        target_link_options(jellyfin-native PRIVATE
+            "-Wl,--gc-sections"
+        )
+        target_compile_options(jellyfin-native PRIVATE
+            "-ffunction-sections"
+            "-fdata-sections"
+        )
+    endif()
 endif()
 
 set_target_properties(jellyfin-native PROPERTIES

src/player/PlayerController.cpp

@@ -464,10 +464,13 @@ void PlayerController::seek(double seconds) {
   const double clampedSeconds =
       m_durationSeconds > 0.0 ? qBound(0.0, seconds, m_durationSeconds)
                               : qMax(0.0, seconds);
+  // Use absolute+exact so a committed click lands on the requested frame
+  // instead of a possibly-distant keyframe. The optimistic-scrub path uses
+  // keyframes for low latency during drag; commit needs precision.
   const QByteArray command =
       QByteArray("no-osd seek ") + QByteArray::number(clampedSeconds, 'f', 3) +
-      QByteArray(" absolute+keyframes");
-  qInfo() << "player: absolute keyframe seek" << clampedSeconds;
+      QByteArray(" absolute+exact");
+  qInfo() << "player: absolute exact seek" << clampedSeconds;
   beginSeekCommand(command, clampedSeconds);
 }
 
@@ -894,13 +897,6 @@ void PlayerController::setPositionSeconds(double seconds) {
 double PlayerController::playbackPositionFromMpvTime(double seconds) const {
   if (!std::isfinite(seconds))
     return m_positionSeconds;
-
-  // Starfish can expose a timeline rebased to 0 after mpv starts a resumed
-  // stream. Keep the app-facing clock on Jellyfin's absolute media timeline so
-  // relative seeks and progress reports stay anchored to the resume point.
-  if (m_resumeStartSeconds > 0.0 && seconds + 5.0 < m_resumeStartSeconds)
-    return m_resumeStartSeconds + seconds;
-
   return seconds;
 }
 

tools/build-linux-release.sh

@@ -22,6 +22,7 @@ MPV_SETUP_ARGS=(
   --prefix "$MPV_PREFIX"
   --buildtype release
   --default-library shared
+  -Db_lto=false
   -Dbuild-date=false
   -Dlibmpv=true
   -Dcplayer=true