!NhAsaYbbgmzHtXTPQJ:funklause.de

Nix NodeJS

196 Members
57 Servers

Load older messages


SenderMessageTime
30 Apr 2024
@lily:lily.flowersLily Foster
In reply to @matejam:matrix.org
Here's a working version: https://github.com/MatejaMaric/yota-laravel/blob/c114ebc3c725a67a19d148a95543ed307094c659/derivation.nix
Thanks a lot Lily! 🤗
(if you want, you can set npmBuildScript = "prod" and include npmBuildHook as well)
20:46:36
@lily:lily.flowersLily Foster
In reply to @matejam:matrix.org
Here's a working version: https://github.com/MatejaMaric/yota-laravel/blob/c114ebc3c725a67a19d148a95543ed307094c659/derivation.nix
Thanks a lot Lily! 🤗
* (if you want, you can set npmBuildScript = "prod" and include npmBuildHook as well and remove the postBuild)
20:46:41
@c0ba1t:matrix.orgCobaltIirc, the nodejs 22 LTS has some problems with node gyp. You might want to stick with 18 for a few more weeks until testing has been done20:46:49
@lily:lily.flowersLily Foster
In reply to @c0ba1t:matrix.org
Iirc, the nodejs 22 LTS has some problems with node gyp. You might want to stick with 18 for a few more weeks until testing has been done
well if they're using npmHooks and nodejs, it'll be 20 for a bit
20:47:13
@lily:lily.flowersLily Foster i don't think nixpkgs has set nodejs = nodejs_22 yet (just node_20) 20:47:27
@lily:lily.flowersLily Foster
In reply to @c0ba1t:matrix.org
Iirc, the nodejs 22 LTS has some problems with node gyp. You might want to stick with 18 for a few more weeks until testing has been done
(also what problems? that's concerning if their build tool didn't work on release....)
20:47:44
@lily:lily.flowersLily Foster (i guess it is nodejs admittedly) 20:47:54
@matejam:matrix.orgmatejam

(if you want, you can set npmBuildScript = "prod" and include npmBuildHook as well and remove the postBuild)

Hm, for some reason if I do that it doesn't generate files that npm run prod generates in $src/share/php/yota-laravel/public/ where I want them.

20:56:46
@c0ba1t:matrix.orgCobaltIirc, it is on 22 on unstable. I don't remember the specifics but an unstable bum recently made our builds fail until we returned to 18. Iirc, it was similar to https://github.com/nodejs/node-gyp/issues/302920:59:57
@lily:lily.flowersLily Foster
In reply to @matejam:matrix.org

(if you want, you can set npmBuildScript = "prod" and include npmBuildHook as well and remove the postBuild)

Hm, for some reason if I do that it doesn't generate files that npm run prod generates in $src/share/php/yota-laravel/public/ where I want them.

share derivation? (e.g. on a pastebin?) that seems weird
21:00:53
@lily:lily.flowersLily Fosteroh you know what21:01:00
@lily:lily.flowersLily Foster
In reply to @c0ba1t:matrix.org
Iirc, it is on 22 on unstable. I don't remember the specifics but an unstable bum recently made our builds fail until we returned to 18. Iirc, it was similar to https://github.com/nodejs/node-gyp/issues/3029
but it was 20 before 22, did 20 not work either?
21:01:19
@lily:lily.flowersLily Foster
In reply to @lily:lily.flowers
oh you know what
i wonder if it's because the composer stuff changes build to that dir first
21:02:06
@lily:lily.flowersLily Fosterif it's working now, leave it as-is then :)21:02:13
@matejam:matrix.orgmatejam

i wonder if it's because the composer stuff changes build to that dir first

Yeah, maybe even the other way around. In the logs I see first the npm runs and then composer>

if it's working now, leave it as-is then :)

sure 😄

share derivation? (e.g. on a pastebin?) that seems weird

If you are interested, I can make a new branch with the change. It's my open source project anyway

21:04:59
@matejam:matrix.orgmatejam *

i wonder if it's because the composer stuff changes build to that dir first

Yeah, maybe even the other way around. In the logs I see first the npm runs and then composer.

if it's working now, leave it as-is then :)

sure 😄

share derivation? (e.g. on a pastebin?) that seems weird

If you are interested, I can make a new branch with the change. It's my open source project anyway

21:05:17
@lily:lily.flowersLily Foster
In reply to @matejam:matrix.org

i wonder if it's because the composer stuff changes build to that dir first

Yeah, maybe even the other way around. In the logs I see first the npm runs and then composer.

if it's working now, leave it as-is then :)

sure 😄

share derivation? (e.g. on a pastebin?) that seems weird

If you are interested, I can make a new branch with the change. It's my open source project anyway

nah, i'm pretty sure it's because composer changes dir and you want npm to run after that. so you're doing it right enough and this looks pretty awesome actually :)
21:05:34
@lily:lily.flowersLily Fosterit feels neat that you are able to mix the npm hooks into another builder without too much fuss and have a multi-language-package-manager app build in a derivation like that21:06:10
@lily:lily.flowersLily Foster
In reply to @c0ba1t:matrix.org
Iirc, it is on 22 on unstable. I don't remember the specifics but an unstable bum recently made our builds fail until we returned to 18. Iirc, it was similar to https://github.com/nodejs/node-gyp/issues/3029
oooof. well marsam left and i'm recovering from surgery, so someone else will probably have to bump the nodejs derivation
21:09:32
@lily:lily.flowersLily Foster
In reply to @lily:lily.flowers
oooof. well marsam left and i'm recovering from surgery, so someone else will probably have to bump the nodejs derivation
(no one else is on nodejs team rn except winter who also hasn't had time for things....)
21:10:15
@matejam:matrix.orgmatejam Yeah, Composer also has it's own version of fetchNpmDeps (mkComposerRepository) but I couldn't really make it work. I don't know how it would handle 3 or more language app builds 😂 I'm glad that it works for 2 21:10:14
@c0ba1t:matrix.orgCobalt
In reply to @lily:lily.flowers
oooof. well marsam left and i'm recovering from surgery, so someone else will probably have to bump the nodejs derivation
It looks like the fix for the upstream issue was merged today. Though, by the comments on the MR, it will likely take a bit until a patch release will be published. So there's no need to hurry here.
21:27:35
@c0ba1t:matrix.orgCobaltI'll try to make a PR when 20.0.1 gets released.21:28:23
@lily:lily.flowersLily Foster
In reply to @c0ba1t:matrix.org
It looks like the fix for the upstream issue was merged today. Though, by the comments on the MR, it will likely take a bit until a patch release will be published. So there's no need to hurry here.
(well hurry or not, i'm just making clear i don't think anyone from nodejs team will be touching it any time soon)
21:46:15
@lily:lily.flowersLily Foster
In reply to @c0ba1t:matrix.org
I'll try to make a PR when 20.0.1 gets released.
i assume you mean 22.0.1, but thank you!
21:46:36
1 May 2024
@mjolnir:nixos.orgNixOS Moderation Botchanged room power levels.15:06:24
2 May 2024
@mac:private.coffeemac joined the room.03:33:06
@timekillertk:matrix.orgTimekillerTK joined the room.04:11:49
@winter:catgirl.cloudWinter joined the room.21:02:51
4 May 2024
@purepani:matrix.orgpurepani joined the room.01:59:39

Show newer messages


Back to Room ListRoom Version: 6