childprocess kill() results in -4048 EPERM error on windows since update to v21.6.2 about node

[santigimeno]

This is fixed in libuv, so it'll be fixed once a new version reaches node.

[mcollina]

@santigimeno could you take a look?

[targos]

@nodejs/libuv

[Uzlopak]

@targos

Should I open another issue in libuv?

[targos]

Maybe? I don't know if that would be relevant without a repro using libuv directly.

FWIW this commit changed the implementation of the uv_kill function on Windows: libuv/libuv@129362f

@vtjnash

[santigimeno]

So, I think there's a regression in libuv which might be solved in libuv/libuv#4341. Regardless, just curious why the need to call child.kill() in those tests as the child processes should exit gracefully.

[Uzlopak]

@santigimeno

It spins up the server and then kills it. Was this way for quiete a while. i could have patched it, but then I could not have report this bug ;)

[Uzlopak]

great job, thanks