bump elm node packages#304198
Conversation
The marked as broken are not a new thing and were discussed in some past PR. The failure in elm-land and elm-pages might be a blocker though. cc @zupo @jali-clarke |
|
Yeah those should work - any idea why linux packages were removed? |
|
I did not try to investigate it but my first guess from: is that npm install (or post-install or so) tries to download the tgz from the internet which fails in the sandbox. I'm not sure why it needs to download it or what initiates that download. Maybe it could avoided. If not I think we can try to fetch this using nix and put them into the node_modules in patchPhase which might be enough for npm to skip trying to fetch these cowboy-style. Likely these are invisible to node2nix since npm allows any sort of user defined effect in the install phase. |
|
We should try again |
Current elm-land in master is actually broken. It does build, but it doesn't run. This commit rewrites it from node2nix to buildNpmPackage and adds a test to confirm it runs. Previus work: * NixOS#344866 * NixOS#304198 * NixOS#267725 Refs Thaigersprint/thaigersprint-2025#1
Description of changes
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.