Hi, Removing bug-gnulib from CC.
Pádraig Brady <[email protected]> writes: > On 07/11/2025 12:22, Bruno Haible wrote: >> I wrote: >>>> $ ./configure gl_cv_func_posix_spawnp_secure_exec=no --quiet >>> >>> I reproduce it. Smaller reproducer: >>> $ cd src >>> $ mkdir subdir >>> $ ./ginstall -s -c -m 555 dd subdir >>> >>> It crashes if and only if $PATH is long enough: >> I can force the crash using the attached testcase.sh, that sets >> a PATH of size ca. 4 KB, on >> - Linux/SPARC >> - Linux/ppc64, Linux/ppc64le >> - Solaris 10 and 11 / SPARC >> I'm committing this fix, which fixes the problem on all these >> platforms. > > Excellent. > The code looks good to me. > > I'll add the following to our test script: > > +# Ensure there are no issues with posix_spawnp() and large $PATHs > +# which we saw when initially changing from execvp() to posix_spawnp(). > +PATH=$(printf '%4001s' '' | sed 's/\(.\{79\}\)./\1:/g'):$PATH \ > + ginstall $strip -c -m 555 $dd $dir || fail=1 > > thanks for fixing this! Great, thank you both for the help with this. I pushed my renameatu changes in Gnulib for GNU/Hurd [1] and updated the submodule in coreutils [2]. That fixes tests/mv/backup-dir and tests/mv/trailing-slash on GNU/Hurd. There following 6 tests still fail: tests/cp/parent-perm-race tests/cp/sparse-to-pipe tests/cp/special-f tests/factor/factor-parallel tests/nice/nice tests/tail/tail-c This is less than what failed for the last release, so I don't think it is a blocker. But it would still be nice to fix them. I'll attach the test-suite.log here. Collin [1] https://github.com/coreutils/gnulib/commit/862a81c0e15448adde6a6e7473ec47e3a4bd91a6 [2] https://github.com/coreutils/coreutils/commit/9abf15047744f6d678d3f4360f4f8fadb7304145
test-suite.log.xz
Description: application/xz
