| 6 Jan 2023 |
|  Lily Foster joined the room. | 01:14:07 |
|  zebrag joined the room. | 01:24:04 |
| 7 Jan 2023 |
|  aktaboot changed their profile picture. | 20:27:20 |
| 9 Jan 2023 |
|  @adis:blad.is left the room. | 06:27:07 |
| 12 Jan 2023 |
|  stroem - sourcegraph joined the room. | 20:43:46 |
| stroem - sourcegraph | How do I set the nodejs used by a package in nixpkgs.nodePackages? Ive tried .override { nodejs = pkgs.nodejs_16-x; } but thats not doing the job | 20:55:28 |
| stroem - sourcegraph | nodejs-16_x.pkgs.pnpm doesnt add pnpm to path like nodePackages.pnpm does | 21:01:34 |
| stroem - sourcegraph | ive also tried an overlay to set global nodejs = nodejs-16_x; to no avail, as well as the approach from here https://discourse.nixos.org/t/how-to-use-pnpm-with-recent-nodejs/21867/2, which adds nodejs16 to the PATH of /nix/store/s01asphglgvimr64ymbddc0lf8ln7rs1-pnpm-7.24.2/bin/pnpm, but that script invokes /nix/store/s01asphglgvimr64ymbddc0lf8ln7rs1-pnpm-7.24.2/bin/.pnpm-wrapped which has node 14 in its shebang | 21:12:19 |
| Lily Foster | In reply to @strum355:matrix.org
nodejs-16_x.pkgs.pnpm doesnt add pnpm to path like nodePackages.pnpm does (oops I tried to fix as much of that as I could in the node2nix update, but I was more doing that as a quick/hacky stopgap until we got better tooling so I guess some packages stopped working) | 21:13:31 |
| stroem - sourcegraph | is there a quick-fix i can do in my flake? 😬 | 21:14:07 |
| Lily Foster | In reply to @strum355:matrix.org
ive also tried an overlay to set global nodejs = nodejs-16_x; to no avail, as well as the approach from here https://discourse.nixos.org/t/how-to-use-pnpm-with-recent-nodejs/21867/2, which adds nodejs16 to the PATH of /nix/store/s01asphglgvimr64ymbddc0lf8ln7rs1-pnpm-7.24.2/bin/pnpm, but that script invokes /nix/store/s01asphglgvimr64ymbddc0lf8ln7rs1-pnpm-7.24.2/bin/.pnpm-wrapped which has node 14 in its shebang So wait you want it to build with nodejs-14_x but then run with 16? | 21:14:13 |
| stroem - sourcegraph | i want nothing to do with node 14 | 21:14:25 |
| stroem - sourcegraph | im not sure where or how node 14 is getting dragged into it | 21:14:42 |
| Lily Foster | Ohh it works in 18 but not 16. Well that's bizarre | 21:14:43 |
| Lily Foster | In reply to @strum355:matrix.org
im not sure where or how node 14 is getting dragged into it (thought it was my sins from that node2nix PR I made to unblock nodePackages coming to haunt me -- that's when I had nightmares about npm not copying bins right and just rewrote it to copy manually) | 21:15:21 |
| Lily Foster | In reply to @strum355:matrix.org
is there a quick-fix i can do in my flake? 😬 Easiest fix is probably to just make a wrapper derivation to manually add the bin on top of nodejs-16_x.pkgs.pnpm | 21:16:35 |
| Lily Foster | I'm not sure why it wouldn't be getting copied though | 21:16:42 |
| Lily Foster | In reply to @strum355:matrix.org
nodejs-16_x.pkgs.pnpm doesnt add pnpm to path like nodePackages.pnpm does Are you on stable or unstable? I get a bin/pnpm that seems to work when building nodejs-16_x.pkgs.pnpm on my system? | 21:19:07 |
| stroem - sourcegraph | pinned to 6f38b43c8c84c800f93465b2241156419fd4fd52 | 21:35:56 |
 Winter (she/her) | In reply to @lily:lily.flowers
Ohh it works in 18 but not 16. Well that's bizarre oh god... | 21:36:10 |
| Lily Foster | In reply to @strum355:matrix.org
pinned to 6f38b43c8c84c800f93465b2241156419fd4fd52 Oh that's from a while ago. Can you try a newer nixpkgs revision? | 21:37:18 |
| Winter (she/her) | Oh that's ancient in terms of Node fixes | 21:48:26 |
| Winter (she/her) | In reply to @lily:lily.flowers
Ohh it works in 18 but not 16. Well that's bizarre (This isn't expected with old node2nix even... huh.) | 21:49:20 |
| Winter (she/her) | It would only work on 14, not 16, and certainly not 18. | 21:49:32 |
| Lily Foster | In reply to @winterqt:nixos.dev
It would only work on 14, not 16, and certainly not 18. Unless that's old enough their nodePackages was still v14. I forget when exactly that landed | 21:53:26 |
| Winter (she/her) | In reply to @lily:lily.flowers
Unless that's old enough their nodePackages was still v14. I forget when exactly that landed After that commit. | 21:53:58 |
| Winter (she/her) | That's for sure. | 21:54:03 |
| Winter (she/her) | I think. | 21:54:05 |
| Lily Foster | Oh yeah it was over a month after that commit | 21:54:16 |
| Winter (she/her) | Which is why I don't understand why it works on 14 and 18, but not 16. | 21:54:21 |
