Coreutils 8.1 test failures

2009-11-21 Thread Ludovic Courtès
Hello, The test suite for 8.1 fails on the NixOS GNU/Linux distro: http://hydra.nixos.org/build/145079/log/raw It also fails on Darwin and Cygwin: http://hydra.nixos.org/job/nixpkgs/stdenv/coreutils/all Thanks, Ludo’.

Re: Coreutils 8.1 test failures

2009-11-21 Thread Ludovic Courtès
Hi Eric, Thanks for the quick reply. I’m actually more interested in the GNU/Linux port, but I’ll inform the people who take care of Nixpkgs on Cygwin. Ludo’.

Re: coreutils-8.2 bug-fix-only release coming soon

2009-12-04 Thread Ludovic Courtès
Hi Jim, Jim Meyering writes: > Ludovic Courtès wrote: >> FWIW the Hydra build farm at TU Deflt[*] now continuously builds >> Coreutils from Git using Gnulib from Git[+]: >> >> http://hydra.nixos.org/jobset/gnu/coreutils-master >> http://hydra.nixos.or

Re: coreutils-8.2 bug-fix-only release coming soon

2009-12-04 Thread Ludovic Courtès
Hi, Jim Meyering writes: > Ludovic Courtès wrote: >> The latest build has all the output: >> http://hydra.nixos.org/build/171993/log/raw . > > Thanks for investigating. > >> Regarding ‘chgrp/basic’, the test finds groups='3 65534'. However, >&g

Re: coreutils-8.2 bug-fix-only release coming soon

2009-12-06 Thread Ludovic Courtès
Hi Jim, Jim Meyering writes: > The trouble is that this code appears to be doing incomplete > "su" emulation, in that it does not set the supplementary groups. Actually the intent of this snippet was to mimic Nix’ build process: https://svn.nixos.org/repos/nix/nix/trunk/src/libstore/build.cc

Re: coreutils-8.2 bug-fix-only release coming soon

2009-12-20 Thread Ludovic Courtès
Hi, Jim Meyering writes: > Ludovic Courtès wrote: [...] >> Regarding ‘chgrp/basic’, the test finds groups='3 65534'. However, >> 65534 corresponds to ‘nogroup’. (The build process runs under a special >> user, typically: >> >> $ id nixbld

Re: new snapshot available: coreutils-8.2.52-4db2f

2010-01-05 Thread Ludovic Courtès
Hello, Surprisingly “make dist” has been failing on Hydra for some time[*]: --8<---cut here---start->8--- gcc -std=gnu99 -I. -g -O2 -c -o exclude.o exclude.c In file included from mbuiter.h:106, from exclude.c:38: mbchar.h: In function 'm

Re: coreutils-8.2 bug-fix-only release coming soon

2010-01-07 Thread Ludovic Courtès
Hi Jim, Jim Meyering writes: > Ludovic Courtès wrote: >> Jim Meyering writes: >> >>>> Regarding ‘chgrp/basic’, the test finds groups='3 65534'. However, >>>> 65534 corresponds to ‘nogroup’. (The build process runs under a special >>

Re: coreutils-8.2 bug-fix-only release coming soon

2010-01-07 Thread Ludovic Courtès
Hi, Jim Meyering writes: > Ludovic Courtès wrote: [...] >> Yes, by email. Hydra email notification has been improved recently: >> >> - Notifications are only sent when the package’s status changes from >> SUCCEEDED to FAILED and vice versa. >> >>

Re: new snapshot available: coreutils-8.2.52-4db2f

2010-01-12 Thread Ludovic Courtès
Hi Jim, Jim Meyering writes: > Ludovic Courtès wrote: >> Surprisingly “make dist” has been failing on Hydra for some time[*]: >> >> gcc -std=gnu99 -I. -g -O2 -c -o exclude.o exclude.c >> In file included from mbuiter.h:106, >> from exclu

bug#10021: [PATCH id] Add error-checking on GNU

2011-11-11 Thread Ludovic Courtès
Hello, On GNU, processes can have zero or more UIDs/GIDs. In the case of a process with zero UIDs, for instance, ‘getuid’ returns -1 and sets ERRNO [0] (as an extension to POSIX [1].) Currently ‘id’ would print (unsigned int) -1 as the UID in that case, whereas it should rather print an error.

bug#10021: [PATCH id] Add error-checking on GNU

2011-11-12 Thread Ludovic Courtès
Hi Jim, Thanks for the quick review! Jim Meyering skribis: > However, wouldn't that fail unnecessarily for a process without > one ID even though id is being asked to print some other(s)? > E.g., it'd fail for a process with no EUID even when id > is being asked to print only the real UID or GI

bug#10021: [PATCH id] Add error-checking on GNU

2011-11-13 Thread Ludovic Courtès
Hi Jim, Jim Meyering skribis: > Oh, I also had to make the new test script executable, > or else "make check" would fail. It works for me even if it’s not executable when I run “make check TESTS=id/gnu-zero-uids”, apparently because it’s run this way: execve("/bin/sh", ["/bin/sh", "./shell-o

bug#10021: [PATCH id] Add error-checking on GNU

2011-11-13 Thread Ludovic Courtès
Hi Paul, Paul Eggert skribis: > On 11/12/11 13:48, Ludovic Courtès wrote: >> +#ifdef __GNU__ >> + if (euid == -1 && !use_real >> + && !just_group && !just_group_list && !just_context) >> +error (EXIT_FAILURE, errn

bug#10021: [PATCH id] Add error-checking on GNU

2011-11-14 Thread Ludovic Courtès
Hi Jim, Jim Meyering skribis: > Ludovic Courtès wrote: > >> Hi Paul, >> >> Paul Eggert skribis: >> >>> On 11/12/11 13:48, Ludovic Courtès wrote: >>>> +#ifdef __GNU__ >>>> + if (euid == -1 && !use_real >>>>

bug#10021: [PATCH id] Add error-checking on GNU

2011-11-14 Thread Ludovic Courtès
Hi, "Alan Curry" skribis: > Ludovic =?UTF-8?Q?Court=C3=A8s?= writes: >> >> OTOH, on POSIX-conforming systems (which includes GNU/Linux, so it may >> be the majority of systems in use), -1 may well be a valid UID/GID. > > That's a bizarre statement. > > 3.428 User ID > > A non-negative integ

bug#10021: [PATCH id] Add error-checking on GNU

2011-11-14 Thread Ludovic Courtès
Hi, Eric Blake skribis: > On 11/14/2011 11:54 AM, Paul Eggert wrote: [...] >>euid = geteuid (); >> - if (GETID_MAY_FAIL && euid == -1 && !use_real >> + if (euid < 0 && !use_real > > That is, how can this work? On systems where uid_t is signed, it makes > sense, but on system

bug#10021: [PATCH id] Add error-checking on GNU

2011-11-15 Thread Ludovic Courtès
Hi Paul, Paul Eggert skribis: > - if (GETID_MAY_FAIL && euid == -1 && !use_real > + if (euid < 0 && !use_real >&& !just_group && !just_group_list && !just_context) > error (EXIT_FAILURE, errno, _("cannot get effective UID")); On GNU/Hurd, no error would ever be ra

bug#11436: Please disable test cp/parent-perm-race on hurd-i386

2012-05-08 Thread Ludovic Courtès
Hi, Samuel Thibault skribis: > Eric Blake, le Tue 08 May 2012 12:46:57 -0600, a écrit : >> http://pubs.opengroup.org/onlinepubs/9699919799/functions/open.html >> O_NOFOLLOW >> If path names a symbolic link, fail and set errno to [ELOOP]. >> >> I wouldn't object to yet another O_* flag that

bug#11671: Bootstrap randomly fails on Hydra

2012-06-11 Thread Ludovic Courtès
heckout Maintainer(s): | Jim Meyering , Pádraig Brady , Ludovic Courtès System:| x86_64-linux Derivation store path: | /nix/store/3ya9v10145ky17mcc62aabivhnkpsqfh-coreutils-tarball-0pre07595ed8567368ca7857df974e1d8209716e7ec1.drv Output store path: | /nix/store/1hqvk6chfz4q

bug#11671: Bootstrap randomly fails on Hydra

2012-06-12 Thread Ludovic Courtès
Hi Paul, Paul Eggert skribis: > On 06/11/2012 05:21 AM, Ludovic Courtès wrote: >> /nix/store/0h02f2l6qbah1wil9bpan9c2flv99xv9-git-export/gnulib-tool: line >> 4700: ./build-aux/announce-gen.tmp: Permission denied > > That's the first diagnostic so that's what I&

bug#11671: Bootstrap randomly fails on Hydra

2012-06-14 Thread Ludovic Courtès
Hi Paul, Paul Eggert skribis: > On 06/12/2012 01:14 PM, Ludovic Courtès wrote: >> ‘$top_srcdir/gnulib/build-aux’ was indeed >> r-xr-xr-x. > > Can you please try to reproduce that problem with build-aux, > if only artificially, along with the following patch? So

bug#19933: LC_TIME category in Coreutils

2015-02-23 Thread Ludovic Courtès
Mark H Weaver skribis: > I tried adding 'glibc-locales' to my system profile, and got this: > > ERROR: union-build: collision between file and directories ((files > ("/gnu/store/nda1ih2iy8likfipl62afiqs6jwdsfxi-glibc-locales-2.20/share/locale/zh_CN/LC_TIME")) > (dirs > ("/gnu/store/mgfqcbm31rh

bug#19933: LC_TIME category in Coreutils

2015-02-24 Thread Ludovic Courtès
Paul Eggert skribis: > On 02/23/2015 01:28 PM, Ludovic Courtès wrote: >> I suspect this isn’t needed. > > This is needed so that 'ls -l' can output a date format suitable for > the current locale. These formats aren't part of glibc and so are not > in glibc

bug#19933: LC_TIME category in Coreutils

2015-02-24 Thread Ludovic Courtès
Andreas Schwab skribis: > l...@gnu.org (Ludovic Courtès) writes: > >> Mark H Weaver skribis: >> >>> I tried adding 'glibc-locales' to my system profile, and got this: >>> >>> ERROR: union-build: collision between file and directories ((f

bug#21459: Race condition in tests/tail-2/assert.sh

2015-09-11 Thread Ludovic Courtès
Hello, We have observed intermittent failures of tests/tail-2/assert.sh (Coreutils 8.24, libc 2.22), especially showing up on relatively slow machines (armhf and mips64el.) The failure is with ‘tail --follow=name’, which, in inotify mode, would fail to report that file ‘foo’ has been deleted. Th

bug#21460: Race condition in tests/tail-2/assert.sh

2015-09-11 Thread Ludovic Courtès
Paul Eggert skribis: > Ludovic Courtès wrote: >> I think the problem happens when ‘tail’ opens ‘foo’ right in between of >> the two notifications: ‘foo’ is still there, and so ‘tail’ doesn’t >> report anything. >> >> Does that make sense? > > Yes, thoug

bug#26253: tests/misc/cut-huge-range.sh fails on arm-linux-gnueabihf

2017-03-26 Thread Ludovic Courtès
Hi Pádraig, We found that ‘tests/misc/cut-huge-range.sh’ would fail on our ARMv7 GNU/Linux machines: https://bugs.gnu.org/26253 (I noticed there’s a thinko in the patch I used for Guix: ‘getconf PAGESIZE’ returns bytes whereas ‘ulimit -v’ expects kibibytes). The reason for this failure is tha

bug#26253: tests/misc/cut-huge-range.sh fails on arm-linux-gnueabihf

2017-03-27 Thread Ludovic Courtès
Hi Pádraig, Pádraig Brady skribis: > On 26/03/17 06:41, Ludovic Courtès wrote: [...] >> Part of the reason, I think, is that ‘vm’ is computed by simply running >> ‘cut -b1’: >> >> vm=$(get_min_ulimit_v_ cut -b1 /dev/null) \ >> || skip_ "this shell

bug#26253: tests/misc/cut-huge-range.sh fails on arm-linux-gnueabihf

2017-03-28 Thread Ludovic Courtès
Pádraig Brady skribis: > Pushed in your name at: > http://git.sv.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=f542200 Thank you! Ludo'.