-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
RFC: remove node packages #14532
Comments
+1 We should take the same approach as ruby where default patches are provided for common packages, mostly for adding the missing buildInputs. Then developers use npm2nix (bundix with ruby) to generate the dependency list for each program that is going to be added to nixpkgs (or their own programs). Each program has their own dependency set but that's fine. Building a single plane of all the node packages, all working together is not a tenable solution as it's exponential in complexity and probably even not possible. Note that if each npm package is in it's own derivation then it's still possible to share code between projects even if the dependency set is declared independently. Hopefully in the future we will have some tool that is capable to run over all of our node projects and tries to nudge versions together to reduce the overall closure size. |
+1 |
So, merge it? |
+1 i think this should also be done for python packages. any thoughts on this @domenkozar @FRidh of-course not jumping into it but to gradually going toward that. |
Just to give another alternative, what I've done for my applications is generate An implementation of the builder I'm using can be found here: https://gist.github.com/bobvanderlinden/4d02b7fdc6f3c52eda188f95667e3915 The downside is that this only works for dependencies without compiled binaries, as the resulting |
Is it okay to close this? At least this seems like a step in the wrong direction to me. I think the root problem is that we just don't have anyone willing to put in the work to keep it updated. I think this can be solved by automating the process similar to how @r-ryantm works. If anyone is interested in this, I have opened an issue to look into it: nix-community/nixpkgs-update#12. |
Thank you for your contributions.
|
I guess this is no longer an issue by now. If no one responds in a week then I'll go ahead and close this issue. Ofcourse always feel free to reopen or create a new one at any time! (I'm going through old issues marked as stale to see what can be triaged.) |
Closed per above discussion. |
I just had a look at
node-packages-generated.nix
. While I didn't go through every single package basically any package that I picked out was utterly outdated. I sure hope nobody is relying on those packages and so I currently see little value in having those.In #13103 (comment) @svanderburg comments:
I would thus suggest to do exactly that: remove most of
node-packages.json
leaving only what falls into one of the two categories above.It is too bad that interfacing with npm is such a flakey business so the integration cannot be automated further to allow for up to date packages with less manual work. Which is exactly the reason why the scope of included node packages should be limited.
I'm happy to provide a PR if this OK with everyone.
/cc @zimbatm @lo1tuma
The text was updated successfully, but these errors were encountered: