Avoid nixpkgs.config.packageOverrides

Apparently packageOverrides are deprecated by overlays. Overlays
apparently can "increase eval time". If intel-vaapi-driver is a "leaf"
package then we can use the overridden package instance without
replacing it in nixpkgs. It seems like the only place this package
should be referenced is from hardware.graphics.extraPackages, and not as
a dependency of other packages.

See https://discourse.nixos.org/t/overriding-a-package/59565

1 files changed, 4 insertions, 6 deletions

diff --git a/hostnix/elmo/media.nix b/hostnix/elmo/media.nix
index 53fa0d2..5adf9a1 100644
--- a/hostnix/elmo/media.nix
+++ b/hostnix/elmo/media.nix
@@ -12,17 +12,15 @@
 #     <DetectNetworkChange>false</DetectNetworkChange>
 
 {
-  nixpkgs.config.packageOverrides = pkgs: {
-    # Apparently adds some hardware acceleration.
-    intel-vaapi-driver = pkgs.intel-vaapi-driver.override { enableHybridCodec = true; };
-  };
-
   hardware.graphics = {
     enable = true;
     # Haswell seems too old to be supported by intel-media-driver (iHD). While
     # QSV is apparently implemented for intel-vaapi-driver (i965) by
     # intel-media-sdk, Jellyfin seems to only support QSV on iHD.
-    extraPackages = with pkgs; [ intel-vaapi-driver ];
+    extraPackages = [
+      # Apparently adds some hardware acceleration.
+      (pkgs.intel-vaapi-driver.override { enableHybridCodec = true; })
+    ];
   };
 
   services.jellyfin.enable = true;