!NhAsaYbbgmzHtXTPQJ:funklause.de

Nix NodeJS

209 Members
60 Servers

Load older messages


SenderMessageTime
24 May 2023
@lily:lily.flowersLily Foster
In reply to @raitobezarius:matrix.org
yes this

You're just wanting something like this on buildNpmPackage, right?

diff --git a/pkgs/build-support/node/build-npm-package/default.nix b/pkgs/build-support/node/build-npm-package/default.nix
index 1c3fb6a74ef..f4cb7d763ed 100644
--- a/pkgs/build-support/node/build-npm-package/default.nix
+++ b/pkgs/build-support/node/build-npm-package/default.nix
@@ -53,7 +53,7 @@ stdenv.mkDerivation (args // {
   # Stripping takes way too long with the amount of files required by a typical Node.js project.
   dontStrip = args.dontStrip or true;
 
-  passthru = { inherit npmDeps; } // (args.passthru or { });
+  passthru = { inherit nodejs npmDeps; } // (args.passthru or { });
 
   meta = (args.meta or { }) // { platforms = args.meta.platforms or nodejs.meta.platforms; };
 })
00:21:33
@lily:lily.flowersLily Foster Wait are you wanting (buildNpmPackage { ... }).nodejs to be available or something just like buildNpmPackage.nodejs? 00:22:23
@lily:lily.flowersLily Foster
In reply to @robert:funklause.de
Definitely! Thank you very much
Opened as https://github.com/NixOS/nixpkgs/pull/233804 and tested against your second commit in your bitwarden-cli PR :)
11:18:28
@lily:lily.flowersLily Foster Oh I just noticed your comment about npm prune --workspace being non-ideal. That seems bizarre to me, and remove the --workspace on the prune command didn't seem to reduce output size 🤔 11:23:52
@lily:lily.flowersLily Foster * Oh I just noticed your comment about npm prune --workspace being non-ideal. That seems bizarre to me, and removing the --workspace on the prune command didn't seem to reduce output size 🤔 11:23:59
@lily:lily.flowersLily FosterI'll investigate later today hopefully11:24:08
25 May 2023
@raitobezarius:matrix.orgraitobezarius changed their display name from raitobezarius to disko in NixOS 23.11 when.13:32:24
@raitobezarius:matrix.orgraitobezarius changed their display name from disko in NixOS 23.11 when to raitobezarius.13:37:28
27 May 2023
@mjolnir:nixos.orgNixOS Moderation Botchanged room power levels.16:41:01
28 May 2023
@hellwolf:matrix.orghellwolf joined the room.13:29:10
@hellwolf:matrix.orghellwolf Hey y'all! Let's say I pick a rfew nodePakcages e.g "nodePackages.three|mathjax", what would a one liner say using nix-shell where then I can run node and require("three")? 13:34:44
@lily:lily.flowersLily Foster I think we would need setup hooks that set NODE_PATH. Which actually would probably be a good idea to add to the nodejs derivation 13:41:51
@lily:lily.flowersLily FosterHmmm if I remember later I'll experiment with adding that13:42:18
@hellwolf:matrix.orghellwolf

Okay. Something similar to haskellPackages.ghcWithPackages would be handy, say:

nix-shell -p 'nodejsPackages.nodeWithPackages { pkgs: [ pkgs.three ] } ?

14:06:45
@lily:lily.flowersLily Foster Well it would be something like nodejs.withPackages probably but also nodePackages is gonna go away and/or be redone before that's viable... 14:08:49
@hellwolf:matrix.orghellwolfI see. I don't think it's too big a stretch to proposing the Nth package manager for NodeJS ecosytem, after now pnpm.14:10:06
@hellwolf:matrix.orghellwolfI think Nix has the potential to be the right one :)14:10:19
@hellwolf:matrix.orghellwolfBut still need that Nth nix node builder though too.14:10:54
@hellwolf:matrix.orghellwolfOr is buildNpmPackage targeting that?14:11:25
@hellwolf:matrix.orghellwolf * Or is buildNpmPackage targeting that role? 14:11:27
@lily:lily.flowersLily Foster buildNpmPackage is really more for leaf-like application-level npm packages 14:12:03
@lily:lily.flowersLily Foster At some point we may end up trying to build out a real Nix-based node package set, but npm dependencies go really deep. Unlike how other package sets like python generally are 14:14:08
@lily:lily.flowersLily FosterSo I doubt it'll really be worth it tbh14:14:23
@lily:lily.flowersLily FosterWe'll probably aim for more like how the rust stuff works14:14:35
@lily:lily.flowersLily Foster I want to have something like importCargoLock but for package-lock.json but it's far down on the backburner rn 14:15:12
@hellwolf:matrix.orghellwolf

npm dependencies go really deep.

Indeed. I wonder if it's inherit to node, or just how npm/yarn works.

14:21:15
@lily:lily.flowersLily Foster
In reply to @hellwolf:matrix.org

npm dependencies go really deep.

Indeed. I wonder if it's inherit to node, or just how npm/yarn works.

Tbh just how any similar system works. See Mix/Elixir, Rust/Cargo, Go, etc. Those don't go as deep as npm but they do get large
14:23:00
@hellwolf:matrix.orghellwolfI see.14:45:41
@hellwolf:matrix.orghellwolfOne more question, when using buildNpmPackage packages. do packages still share the build outputs of their dependencies if possible?14:46:12
@lily:lily.flowersLily Foster
In reply to @hellwolf:matrix.org
One more question, when using buildNpmPackage packages. do packages still share the build outputs of their dependencies if possible?
How do you mean exactly?
15:01:56

Show newer messages


Back to Room ListRoom Version: 6