| 12 Jan 2023 |
 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 |
| Winter (she/her) | That does not make sense | 21:54:25 |
| Winter (she/her) | Unless they're mixing Nixpkgs revisions | 21:54:31 |
| Lily Foster | In reply to @winterqt:nixos.dev
Which is why I don't understand why it works on 14 and 18, but not 16. I don't think it was 18. I assume they were on a recent enough commit that nodePackages was on 18 (because they mentioned not knowing where 14 came from) | 21:55:06 |
| Lily Foster | (So pretty sure it was only working on 14 for them, to clarify) | 21:55:29 |
| Winter (she/her) | In reply to @lily:lily.flowers
I don't think it was 18. I assume they were on a recent enough commit that nodePackages was on 18 (because they mentioned not knowing where 14 came from) ...so where does 14 come from, if their nodePackages is on 18? You're confusing me 😅 | 21:56:40 |
| Lily Foster | In reply to @winterqt:nixos.dev
...so where does 14 come from, if their nodePackages is on 18? You're confusing me 😅 It's not on 18 if they're on a commit that old (I briefly mistakenly thought it was but it's definitely not) | 21:57:37 |
| Winter (she/her) | So did it only work on 14 and they misspoke (or you misunderstood)? Because you said it's weird that it only didn't work on 16... | 21:58:19 |
| Lily Foster | (Sry for confusion) | 21:58:32 |
| Lily Foster | In reply to @winterqt:nixos.dev
So did it only work on 14 and they misspoke (or you misunderstood)? Because you said it's weird that it only didn't work on 16... Yes, I misunderstood | 21:59:03 |
| Lily Foster | * Yes, I misunderstood and it was only 14 | 21:59:19 |
| 13 Jan 2023 |
| stroem - sourcegraph | 22.11 pin seems to use 16.18.1, which i think is exactly what I need. Bit unfortunate I couldnt overwrite the nodejs version on the older pin though 🤔 I dont understand that exactly, has that issue been fixed since or is that still outstanding? | 12:47:20 |
| Lily Foster | In reply to @strum355:matrix.org
22.11 pin seems to use 16.18.1, which i think is exactly what I need. Bit unfortunate I couldnt overwrite the nodejs version on the older pin though 🤔 I dont understand that exactly, has that issue been fixed since or is that still outstanding? You would have needed to also use the new node-env.nix though, which is why just overriding nodejs didn't do what you needed | 12:49:03 |
| Lily Foster | In reply to @strum355:matrix.org
22.11 pin seems to use 16.18.1, which i think is exactly what I need. Bit unfortunate I couldnt overwrite the nodejs version on the older pin though 🤔 I dont understand that exactly, has that issue been fixed since or is that still outstanding? You can do it with any nixpkgs pin after like mid-October, when all the dust settled on the nodePackages fixes | 12:49:38 |
| stroem - sourcegraph | nodejs always makes me sad, thanks for the help 🙂 | 12:52:11 |
