-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mongosh: 1.6.0 -> 1.6.1 #203918
mongosh: 1.6.0 -> 1.6.1 #203918
Conversation
@winterqt Sorry to bother. Really need help here. I fail to build mongosh with
|
@aaronjheng Does this persist if you remove the Node 16 override? (Did another issue prompt the addition of that override? It shouldn't be needed.) |
Yes.
Nope. Upstream requires Node.js v16.x. |
Looks like a Git dependency is... trying to install something in an install script? Maybe? Can you add |
|
Sorry, to be clear, I mean full build logs (the entire |
See mongosh.log |
It's an npm thing, sigh (ref). For now, you can patch out this dependency, and try again. I need to mull over how to properly fix this, but patching that out should get it working with no effects to the behavior of the package in the end. Let me know if anything else comes up -- happy to help. |
Thanks for the help. I tried, but not working. |
Okay, proper fix it is then 😅 Should have something to test later today. |
Hi! Apologies for taking so long to get back to you on this. The fix for this issue is #204877, but notice that it's not really... a fix. The reality is that this situation is very annoying to deal with, and it doesn't seem like this is common enough within the packages we ship (at least, not yet...) to come up with a convoluted solution, when, like in this case, workarounds exist. Patching out |
I appreciate your help very much. After patching out |
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes