bug#52835: [PATCH 0/2] Fix spawning a child not setting standard fds properly

2022-12-23 Thread Ludovic Courtès
Hi Josselin, Josselin Poiret skribis: > Here is hopefully the last reroll of this patchset. First of all, I did not > include the gnulib patch again because it still applies cleanly and it is > extremely large, but it should be applied before those 3 patches. Yay! > The first two patches shou

bug#52835: [PATCH 0/2] Fix spawning a child not setting standard fds properly

2022-12-23 Thread Bug reports for GUILE, GNU's Ubiquitous Extension Language
Hi Ludo, thanks for the quick review and fixes. > I pushed it to ‘wip-posix-spawn’ along with fixups I’m proposing, mostly > along the lines of what I suggested in > : Nice but also see below. > I also added Andrew Whatson’s patch from >

bug#52835: [PATCH v7 2/2] Make system* and piped-process internally use spawn.

2022-12-23 Thread Bug reports for GUILE, GNU's Ubiquitous Extension Language
* libguile/posix.c (scm_system_star, scm_piped_process): Use do_spawn. (start_child): Remove function. Co-authored-by: Ludovic Courtès --- libguile/posix.c | 233 --- 1 file changed, 78 insertions(+), 155 deletions(-) diff --git a/libguile/posix.c b/l

bug#52835: [PATCH v7 1/2] Add spawn* and spawn.

2022-12-23 Thread Bug reports for GUILE, GNU's Ubiquitous Extension Language
* libguile/posix.c: Include spawn.h from Gnulib. (do_spawn, scm_spawn_process): New functions. * module/ice-9/spawn.scm: New file (spawn): New procedure. --- libguile/posix.c | 82 ++ libguile/posix.h | 2 ++ module/ice-9/spawn.scm | 54