| 16 Aug 2025 |
|  Lyn joined the room. | 14:25:09 |
| Lyn left the room. | 14:30:02 |
| 29 Aug 2025 |
|  Philip Taron (UTC-8) joined the room. | 00:36:22 |
| Philip Taron (UTC-8) | 383256 | 00:36:35 |
| Philip Taron (UTC-8) | * https://github.com/NixOS/nixpkgs/pull/383256
I'd love to commit this
can i get a reviewer | 00:37:16 |
|  magic_rb changed their profile picture. | 19:29:31 |
| 30 Aug 2025 |
 Cobalt | Hey, does anyone know the recommended way to run patchShebangs on the node_modules of workspace members?
The current implementation of buildNpmPackage only runs it once on the top-level node_modules leaving any workspace-level node_modules unpatched. This will lead to build errors if, e.g., the workspace members contain deps with shebangs, like vite.
postInstall doesn't appear to work/ be passed through so I only could get it to work as a buildPhase or configurePhase adjustment. Both of them however seem to be not really applicable to this step. It also feels like this should be accounted for by buildNpmPackage, especially given one can explicitly specify workspace members.
| 20:26:54 |
| Cobalt | (for context, I'm packaging jaeger-ui and they only include vite and some other deps as devDependencies of the workspace memeber packages/jaeger-ui. The build fails on calling vite in the build script of the workspace memeber as it has a /usr/bin/env node shebang and is unpatched as a consequence of landing in packages/jaeger-ui/node_modules/vite/bin/vite.js.) | 20:29:27 |
| Cobalt | * Hey, does anyone know the recommended way to run patchShebangs on the node_modules of workspace members?
The current implementation of buildNpmPackage only runs it once on the top-level node_modules leaving any workspace-level node_modules unpatched. This will lead to build errors if, e.g., the workspace members contain deps with shebangs, like vite.
postInstall doesn't appear to work/ be passed through so I only could get it to work as a buildPhase or configurePhase adjustment. Both of them however feel like the wrong Phase for this step. It also feels like this should be accounted for by buildNpmPackage, especially given one can explicitly specify workspace members.
| 20:30:54 |
| Cobalt | * Hey, does anyone know the recommended way to run patchShebangs on the node_modules of workspace members?
The current implementation of buildNpmPackage only runs it once on the top-level node_modules leaving any workspace-level node_modules unpatched. This will lead to build errors if, e.g., the workspace members contain deps with shebangs, like vite.
postInstall doesn't appear to work/ be passed through so I only could get it to work as a buildPhase or configurePhase adjustment. Both of them however feel like the wrong Phase for this step. It also seems like this should be accounted for by buildNpmPackage, especially given one can explicitly specify workspace members.
| 20:31:06 |
| Cobalt | * (for context, I'm packaging jaeger-ui and they only include vite and some other deps as devDependencies of the workspace member packages/jaeger-ui. The build fails on calling vite in the build script of the workspace member as it has a /usr/bin/env node shebang and is unpatched as a consequence of landing in packages/jaeger-ui/node_modules/vite/bin/vite.js.) | 20:31:25 |
| 2 Sep 2025 |
|  n3tcat joined the room. | 18:18:22 |
| n3tcat changed their display name from Stephen D to n3tcat. | 18:28:47 |
| n3tcat | hai :3 I'm trying to package an npm package that depends on another npm package that itself downloads a binary. I am building that binary and want to put it into node_modules but I'm finding it's getting blown out. Here's my package.nix: https://github.com/scd31/tileserver-gl/blob/nix/nix/package.nix | 18:31:52 |
| n3tcat | Wondering if anyone knows the semantics of buildNpmPackage, i.e. where it would be getting removed | 18:32:14 |
 Marie | have you tried using preBuild instead of prePatch? | 18:35:20 |
| n3tcat | If I do, then it falls over in the patchPhase with tileserver-gl> npm error request to https://github.com/maplibre/maplibre-native/releases/download/node-v6.2.0/node-v127-linux-x64-Release.tar.gz failed, reason: getaddrinfo EAI_AGAIN github.com | 18:37:13 |
| Marie | try npmFlags = [ "--ignore-scripts" ]; | 18:40:58 |
| n3tcat | trying nw | 18:41:57 |
| n3tcat | * trying now | 18:41:59 |
| n3tcat | * trying now | 18:42:01 |
| n3tcat | oh shit it seems to be doing things | 18:43:06 |
| n3tcat | ayyy making progress! | 18:45:07 |
| n3tcat | tytyty ily marie | 18:45:09 |
| n3tcat | actually I think I was a big premature. I think that breaks node sqlite | 18:49:41 |
| n3tcat | okay so it's still running but what seems to work is npm rebuild --ignore-scripts follows by the ln -s thing followed by npm rebuild | 19:05:02 |
| n3tcat | omg is worked | 19:16:08 |
| n3tcat | meowwwwwwwwwwww | 19:16:12 |
|  Charles joined the room. | 19:35:04 |
| Charles | are general typescript/javascript + nix questions okay here or is this room strictly nodejs? | 19:40:15 |
