[coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Jim Meyering
Nelson H. F. Beebe wrote: [...there appears to be a hang in "make check" on FreeBSD...] > > PASS: misc/id-groups > SKIP: misc/md5sum > SKIP: misc/md5sum-newline > PASS: misc/md5sum-parallel > PASS: misc/mknod Hi Nelson, [Cc'ing the list] Thanks for the heads up and for all the testing. The test

[coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Andreas Schwab
Jim Meyering writes: > For the record, I reduced it to this, > where /bin/sh is /usr/local/bin/bash, which is 4.1.7: > > cat <<\EOF > init.cfg > f() > { > case $IFS in '') exit 91;; esac > local s=$IFS > case $s in '') exit 97;; esac > } > f > EOF > $ /bin/sh -c '. init.cfg' > [Exit 97] Pe

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Jim Meyering
Andreas Schwab wrote: > Jim Meyering writes: > >> For the record, I reduced it to this, >> where /bin/sh is /usr/local/bin/bash, which is 4.1.7: Actually, /bin/sh is *not* bash on that particular system. Rather, it's simply FreeBSD 8.1's /bin/sh. That requires a more general work-around... >> ca

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Jim Meyering
Jim Meyering wrote: > Andreas Schwab wrote: >> Jim Meyering writes: >> >>> For the record, I reduced it to this, >>> where /bin/sh is /usr/local/bin/bash, which is 4.1.7: > > Actually, /bin/sh is *not* bash on that particular system. > Rather, it's simply FreeBSD 8.1's /bin/sh. > That requires a

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Andreas Schwab
Jim Meyering writes: > Here's a better test case: > > $ /bin/sh -c 'f(){ local s=$IFS; test -n "$s"; }; f' > [Exit 1] > > And based on that, here's a patch to gnulib's tests/init.sh > that solves the problem more generally: > (but this is not enough for coreutils, since some > failing tes

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Jim Meyering
Andreas Schwab wrote: > Jim Meyering writes: > >> Here's a better test case: >> >> $ /bin/sh -c 'f(){ local s=$IFS; test -n "$s"; }; f' >> [Exit 1] >> >> And based on that, here's a patch to gnulib's tests/init.sh >> that solves the problem more generally: >> (but this is not enough for c

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Andreas Schwab
Jim Meyering writes: > IMHO, the fact that VAR=$OTHER_VAL never needs quotes > should imply that `local VAR=$OTHER_VAL' also requires no quotes. I wonder where POSIX requires or even allows that export VAR=$foo does not perform word splitting (although the second export example would be unsafe w

[coreutils] new snapshot available: coreutils-8.6.24-6c058

2010-11-06 Thread Jim Meyering
With Paul's big stat change, I'm hoping we've reached closure on that issue. This snapshot also includes plenty of fixes in gnulib by Eric Blake: fixes for icc-related problems as well as for Solaris 10, so if you can build this with icc, or on Solaris, please do so and let us know how it goes. c

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Eric Blake
On 11/06/2010 08:19 AM, Jim Meyering wrote: > Andreas Schwab wrote: > >> Jim Meyering writes: >> >>> Here's a better test case: >>> >>> $ /bin/sh -c 'f(){ local s=$IFS; test -n "$s"; }; f' >>> [Exit 1] >>> >>> And based on that, here's a patch to gnulib's tests/init.sh >>> that solves the

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Jim Meyering
Eric Blake wrote: > On 11/06/2010 08:19 AM, Jim Meyering wrote: >> Andreas Schwab wrote: >> >>> Jim Meyering writes: >>> Here's a better test case: $ /bin/sh -c 'f(){ local s=$IFS; test -n "$s"; }; f' [Exit 1] And based on that, here's a patch to gnulib's test

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Jim Meyering
Andreas Schwab wrote: > Jim Meyering writes: > >> IMHO, the fact that VAR=$OTHER_VAL never needs quotes >> should imply that `local VAR=$OTHER_VAL' also requires no quotes. > > I wonder where POSIX requires or even allows that export VAR=$foo does > not perform word splitting (although the second

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Ralf Wildenhues
Hello, * Jim Meyering wrote on Sat, Nov 06, 2010 at 02:55:47PM CET: > Jim Meyering wrote: > > Andreas Schwab wrote: > >> Jim Meyering writes: > >>> cat <<\EOF > init.cfg > >>> f() > >>> { > >>> case $IFS in '') exit 91;; esac > >>> local s=$IFS > >>> case $s in '') exit 97;; esac > >>> } > >

[coreutils] [PATCH] tests: avoid portability problem with dash vs. local v=$splittable_val

2010-11-06 Thread Jim Meyering
Hi Pádraig, When running make check SHELL=dash, I got one failure. It was in misc/stat-birthtime, due to its use retry_delay_: + num_sleeps=16 + test 5 -le 5 + gawk -v n=16 -v s=.07 BEGIN { for (i=0;iFrom bda912213e8605121226ce2e07951f1b682ef595 Mon Sep 17 00:00:00 2001 From: Jim Meyering

Re: [coreutils] Re: Builds of coreutils-8.6.19-1369f

2010-11-06 Thread Jim Meyering
Ralf Wildenhues wrote: > Hello, > > * Jim Meyering wrote on Sat, Nov 06, 2010 at 02:55:47PM CET: >> Jim Meyering wrote: >> > Andreas Schwab wrote: >> >> Jim Meyering writes: >> >>> cat <<\EOF > init.cfg >> >>> f() >> >>> { >> >>> case $IFS in '') exit 91;; esac >> >>> local s=$IFS >> >>> cas

Re: [coreutils] [PATCH] tests: avoid portability problem with dash vs. local v=$splittable_val

2010-11-06 Thread Pádraig Brady
On 06/11/10 17:08, Jim Meyering wrote: > Hi Pádraig, > > When running make check SHELL=dash, I got one failure. > It was in misc/stat-birthtime, due to its use retry_delay_: > > + num_sleeps=16 > + test 5 -le 5 > + gawk -v n=16 -v s=.07 BEGIN { for (i=0;i + local delay=.07 .07 .07 .07 .07

[coreutils] [PATCH] split: fail immediately if impossible to create a large file

2010-11-06 Thread Pádraig Brady
For example, if splitting to 4G files (max on VFAT for example), and `split` is built without large file support, then we would have only failed after about 2G was written. If the input is currently less than that, we won't know about the issue until later. I'm not planning to apply for this relea