From 175da48170dc8998f4223a66160232c1fddbcf73 Mon Sep 17 00:00:00 2001
From: John Ogle <john@ogle.fyi>
Date: Mon, 22 Dec 2025 12:17:54 -0800
Subject: [PATCH] Fix jellyfinScaleFactor implementation in kodi role

The original implementation had several issues that prevented it from
building:
- Used buildInputs instead of nativeBuildInputs for makeWrapper
- Referenced wrong executable name (jellyfinmediaplayer vs jellyfin-desktop)
- Used wrapProgram which doesn't work with symlinks from symlinkJoin

Fixed by using makeWrapper directly with the correct executable path
after removing the symlink.

Also enabled jellyfinScaleFactor = 1.5 on boxy for UI scaling.
---
 machines/boxy/configuration.nix | 1 +
 roles/kodi/default.nix          | 8 +++++---
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/machines/boxy/configuration.nix b/machines/boxy/configuration.nix
index 8e898cf..fefda4c 100644
--- a/machines/boxy/configuration.nix
+++ b/machines/boxy/configuration.nix
@@ -26,6 +26,7 @@ with lib;
       enable = true;
       autologin = true;
       wayland = true;
+      jellyfinScaleFactor = 1.5;
     };
     users.enable = true;
   };
diff --git a/roles/kodi/default.nix b/roles/kodi/default.nix
index b65a7b2..e44431a 100644
--- a/roles/kodi/default.nix
+++ b/roles/kodi/default.nix
@@ -48,10 +48,12 @@ in
       then pkgs.symlinkJoin {
         name = "jellyfin-media-player-scaled";
         paths = [ pkgs.jellyfin-media-player ];
-        buildInputs = [ pkgs.makeWrapper ];
+        nativeBuildInputs = [ pkgs.makeWrapper ];
         postBuild = ''
-          wrapProgram $out/bin/jellyfinmediaplayer \
-            --add-flags "--scale-factor ${toString cfg.jellyfinScaleFactor}"
+          mkdir -p $out/bin
+          rm -f $out/bin/jellyfin-desktop
+          makeWrapper ${pkgs.jellyfin-media-player}/bin/jellyfin-desktop $out/bin/jellyfin-desktop \
+            --add-flags "--tv --scale-factor ${toString cfg.jellyfinScaleFactor}"
         '';
       }
       else pkgs.jellyfin-media-player;