In reply to @denbrahe:matrix.org
Lily Foster: were you able to work on a solution for retrying package downloads with buildNpmPackage and prefetch-npm-deps already?

Yes I've worked on it. I actually found that switching out the ureq library that prefetch-npm-deps is using with something else (e.g. minreq) caused the issues to stop occuring for me

We were wanting to have it use isahc instead, but that library was giving us issues with certs for some reason that I haven't had the time to dive into yet

I'll let you know when we have more, because the random request failures on poor internet connections thing is annoying

In reply to @raitobezarius:matrix.org
yeah you have some stabilization days ahead of you if needed :)

In reply to @raitobezarius:matrix.org
I have been dreaming those last days about starting the yarn backend for buildNpmPackage

In reply to @hexa:lossy.network
should we merge botamusique and navidrome with the npmConfigHook reschedule hack?

In reply to @hexa:lossy.network
Did the npmRoot change work for botamusique? Because it caused problems with navidrome.

In reply to @hexa:lossy.network
https://github.com/NixOS/nixpkgs/pull/230991#issuecomment-1552133508

In reply to @hexa:lossy.network
https://github.com/NixOS/nixpkgs/pull/230991#issuecomment-1552133508

I cannot replicate this by cherry-picking your commit from #229953 on top of #230991 and adding this diff:

diff --git a/pkgs/servers/misc/navidrome/default.nix b/pkgs/servers/misc/navidrome/default.nix
index 7c121e82210..309601e059f 100644
--- a/pkgs/servers/misc/navidrome/default.nix
+++ b/pkgs/servers/misc/navidrome/default.nix
@@ -28,22 +28,14 @@ buildGoModule rec {
 
   vendorHash = "sha256-C8w/qCts8VqNDTQVXtykjmSbo5uDrvS9NOu3SHpAlDE=";
 
+  npmRoot = "ui";
+
   npmDeps = fetchNpmDeps {
     inherit src;
     sourceRoot = "source/ui";
     hash = "sha256-qxwTiXLmZnTnmTSBmWPjeFCP7qzvTFN0xXp5lFkWFog=";
   };
 
-  prePatch = ''
-    postPatchHooks=("''${postPatchHooks[@]/npmConfigHook}")
-  '';
-
-  postPatch = ''
-    pushd ui
-    npmConfigHook
-    popd
-  '';
-
   nativeBuildInputs = [
     makeWrapper
     nodejs