Conversation
|
@soda0289 thoughts? |
|
Thanks for pushing a fix. I haven't touched this change in a while, been busy with work. Give me some time later today to test this out. |
|
I've been thinking more about this and I think we might be able to avoid the serialization requirement (which breaks configs that include functions) by using the fork() syscall. Turns out that sycalls are a little bit difficult to call from node (child_process.fork() doesn't actually call fork :/), but here's a short snippet where I managed to get them working: Note that I took the number 57 from the linux kernel code, so it might not work correctly on Mac (actually I'm not sure if there's any non-backwards-compatible changes on that API that could break this code on older versions of the linux kernel). |
The changes made in your PR break the watch command, which is used by serverless-offline. This PR fixes it.
Thanks for the PR btw, it helped a lot :D