Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There is of course a 7 line warning explaining why it isn't writable but I guess it went over my head so I tried making it writable anyway, wouldn't using an
mktemp
directory already avoid the issues/differences between multi-user and single-user Nix anyway without making it unwritable?Making this writable resolves issues with
node-gyp
, which wants to re-download things from the internet during the build process if it can't find things in$HOME
, there's probably some weird env vars to work around this, but this seemed like the simplest path to take (and I have this done elsewhere too).npm_config_nodedir
is also something I've seen used previously (at least in pnpm2nix), which supposedly also helps with install scripts wanting to download things when they shouldn't.There should probably still a comment but I didn't write it myself since I think I don't quite understand the issue.