Make replacing xdg-open with handlr less expensive

[UserSv4]

Would it be better to use this instead of https://github.com/Philipp-M/nixos-config/blob/d69854eaf39df91ee49ea7053abe461c1397f4f1/configuration.nix#L23C8-L23C8 ?
system.replaceRuntimeDependencies = [ { original = pkgs.xdg-utils; replacement = pkgs.xdg-utils.overrideAttrs (oldAttrs: { postInstall = oldAttrs.postInstall + '' # "overwrite" xdg-open with handlr cp ${pkgs.writeShellScriptBin "xdg-open" "${pkgs.handlr}/bin/handlr open \"$@\""}/bin/xdg-open $out/bin/xdg-open ''; }); } ];

[Philipp-M]

Thanks for the suggestion.

Hmm, I tried that but then the nixos-rebuild fails because builtins.storePath is used in system.replaceRuntimeDependencies, which doesn't seem to be allowed in pure evaluation mode, see also NixOS/nixpkgs#199162

Also since, I'm e.g. enabling cudaSupport for everything (which is also quite expensive) a lot needs to be recompiled anyway. I'm currently updating all the flake inputs roughly once every 1-2 months and let the system compile overnight, so that everything relevant is cached and upload all the artifacts to my server (via nix-serve) for my other machines.

But I certainly like the idea. I've also thought about patching a few packages such as chromium, which doesn't really need xdg-open while compiling, so I think it would be better to wrap it afterwards (generally a good idea if possible for a lot of packages to avoid cache invalidation because of changing dependencies).

[Philipp-M]

FWIW, I have created a PR for chromium: NixOS/nixpkgs#246038