bug#8909: tests/ls/stat-free-color: fails on OpenSuSE 11.4

2011-06-21 Thread Bernhard Voelker
Hi, `make check` of coreutils.git (latest commit is 18bd9ce40ae05d52543fc9c5a5fdf4f82d13068d) on an OpenSuSE 11.4 host fails (everything else was ok/skipped): FAIL: ls/stat-free-color (exit: 1) == ++ initial_cwd_=/home/berny/coreutils/tests ++ fail=0 +++ test

bug#8909: tests/ls/stat-free-color: fails on OpenSuSE 11.4

2011-06-21 Thread Bernhard Voelker
On 06/21/2011 03:41 PM, Jim Meyering wrote: Bernhard Voelker wrote: ... LS_COLORS='rs=0:di=01;34:ln=01;36:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=00:su=00:sg=00:ca=00:tw=00:ow=00:st=00:ex=00:mh=00:' ++ export LS_COLORS + strace -o log -e stat,lstat,stat64,lstat64 ls --co

bug#8909: tests/ls/stat-free-color: fails on OpenSuSE 11.4

2011-06-21 Thread Bernhard Voelker
On 06/21/2011 04:16 PM, Jim Meyering wrote: Bernhard Voelker wrote: On 06/21/2011 03:41 PM, Jim Meyering wrote: It might be useful to see the entire strace output, too. To get that, remove the "-e stat,lstat,stat64,lstat64" argument from the strace invocation, and rerun the test.

bug#10064: sort: sometimes segfault with certain limits (using prlimit)

2011-11-16 Thread Bernhard Voelker
For the new support of util-linux' prlimit to allow a command to be run http://article.gmane.org/gmane.linux.utilities.util-linux-ng/5039 I tried to find a useful example for the manpage. I found a nice, huge file to feed sort with - /dev/zero, and my first try was to let the command be killed af

bug#10064: sort: sometimes segfault with certain limits (using prlimit)

2011-11-17 Thread Bernhard Voelker
On 11/16/2011 07:32 PM, Paul Eggert wrote: On 11/16/11 09:25, Bernhard Voelker wrote: prlimit --stack=1000 --memlock=10 --rss=1000 sort -u /dev/zero Well, mostly (4/5 times) sort got correctly "Killed", but a segfault smells. Sorry, I don't have prlimit and don'

bug#10110: tests/id/gnu-zero-uids: skipping with "not running on GNU/Hurd" on Linux

2011-11-22 Thread Bernhard Voelker
Hi, this new tests is skipped on my OpenSuSE-11.4 system: ... + require_gnu_ ++ uname + test Linux = GNU + skip_ 'not running on GNU/Hurd' + warn_ 'gnu-zero-uids: skipped test: not running on GNU/Hurd' + case $IFS in + printf '%s\n' 'gnu-zero-uids: skipped test: not running on GNU/Hurd' gnu-zero-

bug#10545: make check failures on virtual machine at hoster 1und1.de

2012-01-18 Thread Bernhard Voelker
Running `make check` on a virtual machine at 1und1.de is failing for 5 tests: FAIL: misc/nproc-avail ++ nproc --all + all=4 ++ OMP_NUM_THREADS= ++ nproc + available=8 + test 8 -le 4 + fail=1 I added an strace for both runs. FAIL: tail-2/wait + test '!' -r unreadable + timeout 1 tail -s0.1 -F

bug#10686: mv: moving hardlink of a softlink to the softlink does nothing

2012-02-01 Thread Bernhard Voelker
Playing around with the latest mv checkin, I noticed another corner case: Create a file 'f', a symlink 'l' to it, and then a hardlink 's' to that symlink: $ touch f && ln -s f l && ln l s && ls -ogi total 0 6444 -rw-r--r-- 1 0 Feb 1 08:52 f 6462 lrwxrwxrwx 2 1 Feb 1 08:52 l -> f 6462

bug#10686: mv: moving hardlink of a softlink to the softlink does nothing

2012-02-01 Thread Bernhard Voelker
On 02/01/2012 09:21 AM, Bernhard Voelker wrote: > $ touch f && ln -s f l && ln l s && ls -ogi > total 0 > 6444 -rw-r--r-- 1 0 Feb 1 08:52 f > 6462 lrwxrwxrwx 2 1 Feb 1 08:52 l -> f > 6462 lrwxrwxrwx 2 1 Feb 1 08:52 s -> f > >

bug#10686: mv: moving hardlink of a softlink to the softlink does nothing

2012-02-01 Thread Bernhard Voelker
On 02/01/2012 01:47 PM, Jim Meyering wrote: > Bernhard Voelker wrote: >> Playing around with the latest mv checkin, >> I noticed another corner case: >> >> Create a file 'f', a symlink 'l' to it, and >> then a hardlink 's' to th

bug#10686: mv: moving hardlink of a softlink to the softlink does nothing

2012-02-01 Thread Bernhard Voelker
On 02/01/2012 03:02 PM, Jim Meyering wrote: > Bernhard Voelker wrote: > ... >> well, if it's not standardized (yet), then I agree with you that we >> should choose the simpler b) alternative. >> >> Do you think it's work thinking about the -f case?

bug#10686: mv: moving hardlink of a softlink to the softlink does nothing

2012-02-01 Thread Bernhard Voelker
On 02/01/2012 04:45 PM, Jim Meyering wrote: > Thanks for the clarification and quotes, Eric. > > Bernhard, here's a better patch. > With it, mv simply unlinks the "s" in your example: > > diff --git a/src/copy.c b/src/copy.c > index 4810de8..73f5cc5 100644 > --- a/src/copy.c > +++ b/src/copy.c >

bug#11816: sort -o: error comes late if opening the outfile fails

2012-06-29 Thread Bernhard Voelker
If opening the output file for writing is not possible - e.g. because the user doesn't have sufficient privileges, then sort issues an error. The problem is that the whole - then useless - computation is already done. In the following little example, it took ~15s to sort the data, and then to real

bug#11809: document "So how do we just simply make a backup file?"

2012-06-29 Thread Bernhard Voelker
On 06/29/2012 10:48 AM, Jim Meyering wrote: > Here's the doc patch I suggested, but I'll hold off for now. > I'd like to hear if anyone thinks it's worth adding a new option, > which would obviate such a script. I think it's okay, that special backup case is described in the info page of cp twice

bug#11854: make syntax-check -j issue

2012-07-04 Thread Bernhard Voelker
After pulling to the lastest revision (v8.17-37-g74427c7) and a successful build (make -j), a subsequent "make syntax-check -j" failed: ... 8.47 vulnerable_makefile_CVE-2009-4029 8.78 copyright_check CC hostname.o CCLD arch CCLD arch CC hostname.o CC hostn

bug#11858: df -m undocumented, why no df -g

2012-07-04 Thread Bernhard Voelker
k at some point. Wouldn't it then be consequent to remove the long option --megabyte? (It's sister --kilobyte has been removed by v6.9-151-g1e07a21 in 2007.) Have a nice day, Berny >From 664bf8a103585915896f3be9f3a01f11ce17ddc6 Mon Sep 17 00:00:00 2001 From: Bernhard Voelker Date:

bug#11858: df -m undocumented, why no df -g

2012-07-05 Thread Bernhard Voelker
On 07/05/2012 02:35 PM, Jim Meyering wrote: > However, I'm tempted to remove it directly this time, since it's been > undocumented for a while: > > 5 years in df.1 and df --help: COREUTILS-6_9-151-g1e07a21 > 11 years in coreutils.texi: FILEUTILS-4_1_4-28-gf5bf6fe > > What do you think? I ag

bug#11858: df -m undocumented, why no df -g

2012-07-11 Thread Bernhard Voelker
On 07/05/2012 05:08 PM, Jim Meyering wrote: > Bernhard Voelker wrote: >> On 07/05/2012 02:35 PM, Jim Meyering wrote: >>> However, I'm tempted to remove it directly this time, since it's been >>> undocumented for a while: >>> >>> 5

bug#11956: misc/tty-eof: sometimes failure under heavy load

2012-07-17 Thread Bernhard Voelker
FAIL: misc/tty-eof (exit: 1) F: 1: YSBiCg== F: 1: a b F: 1: 780509149 4 F: 1: a b F: 1: a b F: 1: a b F: 1: a b F: 1: a b F: 1: 7557d2f3a6ad1a3a8ebd23a94ab0c642 - F: 1: 1a b F: 1: 000 020141 005142 F: 1: a b F: 1: F: 1:

bug#10915: 8.13: df -- overly long output lines are very hard to read

2012-07-26 Thread Bernhard Voelker
On 07/26/2012 04:23 PM, Pádraig Brady wrote: > On 03/01/2012 10:56 AM, Voelker, Bernhard wrote: >> >> What about a more general --fmt (or --format) option to >> just get the columns you want in the order you want? >> E.g. >> >> df --format=size,free%,mnt,fs >> or >> df --format=size-h,mnt # -h

bug#10915: 8.13: df -- overly long output lines are very hard to read

2012-07-27 Thread Bernhard Voelker
On 07/26/2012 06:11 PM, Pádraig Brady wrote: > On 07/26/2012 04:43 PM, Bernhard Voelker wrote: >> On 07/26/2012 04:23 PM, Pádraig Brady wrote: >>> So a full --output list supported by df could be >>> FSTYPE,SOURCE,TARGET,SIZE,USED,AVAIL,FREEPCT Today, there's no

bug#10915: 8.13: df -- overly long output lines are very hard to read

2012-07-27 Thread Bernhard Voelker
On 07/27/2012 02:04 PM, Jim Meyering wrote: > However, it is hard to judge without seeing the actual changes. > If you feel strongly about it, go ahead and implement your preference. > Maybe the resulting code will be small and clean enough that the added > flexibility will be an obvious net gain.

bug#9780: sort -u throws out non-duplicates

2012-08-16 Thread Bernhard Voelker
On 08/16/2012 10:09 AM, Jim Meyering wrote: >> FYI, here's the required test: >> > >> > (yes 7|head -10; echo 1)|sed 's/^/1 /'|sort -k2,2 --p=1 -S32b -u >> > >> > Without the if (key) { ... } part of my patch, it would fail. >> > I had to tweak the number of '7's (s/11/10) in the input to make

bug#12212: [patch] dd's help text contains superfluous word

2012-08-16 Thread Bernhard Voelker
Re: [PATCH 1/2] dd: the word BLOCKS no longer occurs in the help text On 08/16/2012 06:45 PM, Benno Schulenberg wrote: > > Hi, > > The help text of dd now mentions that the word BLOCKS can > be replaced with something. But the word BLOCKS does not > occur any longer in the entire help text. At

bug#9780: sort -u data loss deserves new release ASAP [Re: bug#9780: sort -u...

2012-08-17 Thread Bernhard Voelker
On 08/17/2012 12:00 PM, Jim Meyering wrote: > I want to make a release with this fix as soon as possible. > Since I'm making this a mostly-bug-fix release, the du and md5 --tag > changes will have to wait for 8.20. > However, I'll be happy to apply documentation-correcting changes > if someone woul

bug#12301: tail -F & ZFS

2012-08-29 Thread Bernhard Voelker
On 08/29/2012 09:43 AM, Jim Meyering wrote: > This change enables tail -f to use inotify and lets > stat -f --format=%T report the file system type name, "zfs". > * src/stat.c (human_fstype): Add a case: zfs, 0x2fc12fc1. Hi Jim, I tried to find documentation about that magic number. There seem t

bug#12301: tail -F & ZFS

2012-08-29 Thread Bernhard Voelker
On 08/29/2012 11:12 AM, Jim Meyering wrote: >> Can we be sure that 0x2fc12fc1 is used for all ZFS >> implementations? > > If there end up being two or more magic numbers for the same file > system (or ZFS variants going by new names), we'll adapt. Ok, that sounds good. Thanks. Have a nice day, B

bug#12322: coreutils-8.1{8, 9} with /etc/mtab as real file, df/total-unprocessed fail, df/no-mtab-status is skipped

2012-09-02 Thread Bernhard Voelker
On 09/01/2012 05:49 PM, g@free.fr wrote: > Maybe both tests could check if /etc/mtab is a symlink and adjust df > parameter to not fail? > > I made this change on the tests to see if that work > cd $(DIR_APP) && sed -i "s/ '\.' / /" tests/df/total-unprocessed > cd $(DIR_APP) && se

bug#12339: Bug: rm -fr . doesn't dir depth first deletion yet it is documented to do so.

2012-09-03 Thread Bernhard Voelker
On 09/03/2012 10:04 PM, Paul Eggert wrote: > I was about to suggest the following, on GNU/Linux: > > # Don't do this unless you know what you're doing! > rm -fr /proc/self/cwd/ > > Except it doesn't work! Not even if I append '.': > > $ mkdir /tmp/victim > $ cd /tmp/victim > $ touch f

bug#12339: Bug: rm -fr . doesn't dir depth first deletion yet it is documented to do so.

2012-09-04 Thread Bernhard Voelker
On 09/04/2012 12:42 PM, Jim Meyering wrote: > I have mixed feelings, but am leaning towards the > Solaris 11/FreeBSD behavior. They must have fixed it for Solaris, here's an older one: $ uname -sr ; mkdir d; ln -s d s; /bin/rm -r s/; ls SunOS 5.9 d Have a nice day, Berny

bug#12339: Bug: rm -fr . doesn't dir depth first deletion yet it is documented to do so.

2012-09-04 Thread Bernhard Voelker
On 09/04/2012 02:55 PM, Jim Meyering wrote: > With the following patch, I see new behavior. > It's an improvement, but we're still not there: > > $ mkdir -p d/e/f; ln -s d s; rm -r s/ > rm: cannot remove 's/': Not a directory > [Exit 1] > $ find > . > ./s > ./d > > N

bug#12350: Composites identified as primes in factor.c (when HAVE_GMP)

2012-09-04 Thread Bernhard Voelker
On 09/04/2012 04:46 PM, Jim Meyering wrote: > incvolving 3 or more prime factors.) When s = 2, we get the 3/4 factor. s/incvolving/involving/ Have a nice day, Berny

bug#12339: [PATCH] rm: avoid bogus diagnostic for a slash-decorated symlink-to-dir

2012-09-04 Thread Bernhard Voelker
On 09/05/2012 08:25 AM, Paul Eggert wrote: > On 09/04/2012 06:55 PM, Linda A. Walsh wrote: > >> So how do delete all files in the directory without wild cards? > > Why tie your hands behind your back? Use wild cards: > > cd DIRECTORY && rm -fr * .[!.] .??* > > If you do this a lot, put it i

bug#12339: Bug: rm -fr . doesn't dir depth first deletion yet it is documented to do so.

2012-09-06 Thread Bernhard Voelker
On September 6, 2012 at 12:56 PM Linda Walsh wrote: > Jim Meyering wrote: > > Thanks for the patch, but it would be pretty rotten for GNU rm to make > > it so "rm -rf ." deletes everything under ".", while all other vendor > > rm programs diagnose the POSIX-mandated error. People would curse

bug#12366: [gnu-prog-discuss] Writing unwritable files

2012-09-06 Thread Bernhard Voelker
On September 6, 2012 at 2:12 PM Paolo Bonzini wrote: > [For bug-coreutils: the context here is that sed -i, just like perl -i, > breaks hard links and thus destroys the content of files with 0400 > permission]. I consider this being 2 different cases: * 'sed -i' breaks hard links: That's bec

bug#12365: Should cp -n return 0, when DEST exists?

2012-09-06 Thread Bernhard Voelker
On September 6, 2012 at 5:24 PM Jim Meyering wrote: > Bernhard Voelker wrote: > > Maybe it's worth adding a line about the exist status > > when using -n or -i (together with answering 'n')? > > Yes, please. That would be an improvement. (my fi

bug#12366: [gnu-prog-discuss] Writing unwritable files

2012-09-06 Thread Bernhard Voelker
On September 6, 2012 at 7:23 PM Paul Eggert wrote: > On 09/06/2012 10:12 AM, Bob Proulx wrote: > > The file replacement is atomic. The reading of the file is not. > > Sure, but the point is that from the end user's > point of view, 'sed -i' is not atomic, and can't > be expected to be atomic. >

bug#12445: ls --color does not color symlinks when not resolving them

2012-09-16 Thread Bernhard Voelker
On 09/16/2012 05:46 AM, Pádraig Brady wrote: > +++ b/tests/misc/ls-misc.pl > @@ -263,6 +263,21 @@ my @Tests = >{POST => sub {unlink 's' or die "s: $!\n"; > restore_ls_colors; }}, > ], > + # The code related to the above fixes introduced a regression, > +

bug#12445: ls --color does not color symlinks when not resolving them

2012-09-16 Thread Bernhard Voelker
On 09/16/2012 11:03 AM, Jim Meyering wrote: > Bernhard Voelker wrote: > >> On 09/16/2012 05:46 AM, Pádraig Brady wrote: >> >>> + # The code related to the above fixes introduced a regression, >>> + # that was fixed after coreutils-8.19. >> &

bug#12212: [patch] dd's help text contains superfluous word

2012-09-18 Thread Bernhard Voelker
On 08/17/2012 08:30 AM, Bernhard Voelker wrote: > Re: [PATCH 1/2] dd: the word BLOCKS no longer occurs in the help text > > On 08/16/2012 06:45 PM, Benno Schulenberg wrote: >> >> Hi, >> >> The help text of dd now mentions that the word BLOCKS can >> be

bug#12212: [patch] dd's help text contains superfluous word

2012-09-18 Thread Bernhard Voelker
On 09/18/2012 02:27 PM, Jim Meyering wrote: > Bernhard Voelker wrote: >> Anyone catching up on Benno's patch with my addendum? >> (I'm not sure about the second patch though.) > > Hi Berny, > I was waiting for a complete patch ;-) Hi Jim, actually, me too

bug#12212: [patch] dd's help text contains superfluous word

2012-09-18 Thread Bernhard Voelker
On 09/18/2012 02:54 PM, Bernhard Voelker wrote: > Here's a try. oops, sorry. I removed the wrong word in the texi file. Here's a better one. >From 39ea828e94e8232660f83c62e1eff85026070b4c Mon Sep 17 00:00:00 2001 From: Benno Schulenberg Date: Tue, 18 Sep 2012 14:58:42 +0200 Sub

bug#12212: [patch] dd's help text contains superfluous word

2012-09-18 Thread Bernhard Voelker
On 09/18/2012 03:20 PM, Jim Meyering wrote: > Applied with the above log tweaks. > Will push shortly. Thanks!

bug#12542: [PATCH] du: avoid abort on systems for which ->me_type is not malloc'd

2012-09-30 Thread Bernhard Voelker
On 09/30/2012 02:43 PM, Jim Meyering wrote: > A better one should come soon, in which I add > a function in mountlist.c (declared in the .h file) > by which to encapsulate this mount-entry freeing process. Good idea - also df doesn't free the mount_list: valgrind --leak-check=full --show-reacha

bug#12656: WG: Re[4]: bug#12656: cp since 8.11 corrupts files

2012-10-21 Thread Bernhard Voelker
On 10/17/2012 10:44 AM, Jim Meyering wrote: > From a5365003c88f4fce6293827c13f90acd0b5bd0cc Mon Sep 17 00:00:00 2001 > From: Jim Meyering > Date: Tue, 16 Oct 2012 17:43:49 +0200 > Subject: [PATCH] cp: avoid data-corrupting free-memory-read > > * src/extent-scan.c (extent_scan_read): Reset our las

bug#12715: [PATCH] build: do not require help2man at build-from-tarball time

2012-10-25 Thread Bernhard Voelker
On 10/25/2012 04:08 PM, Jim Meyering wrote: > Bernhard Voelker wrote: >> Isn't it "Due to lack on the build system, ..."? >> >> I.e. coreutils could be built on system A lacking perl, >> and then be installed on system B which has perl. The above >>

bug#12794: Bug in dd: it sends wrong messages to stderr

2012-11-08 Thread Bernhard Voelker
Dear Linda, On 11/07/2012 10:06 AM, Linda Walsh wrote: > That something appears "open" to > contributing, means nothing by itself. yes, it does (see below). > I've seen more than one open > source projects that claimed to be "open", only to see patches to > correct design problems, persistent

bug#12841: Make for AIX 8.20 fails with undefined symbol .rpl_malloc

2012-11-12 Thread Bernhard Voelker
On 11/13/2012 06:19 AM, Paul Eggert wrote: > I pushed the following further change, to fix it. Something seems to be not quite correct: $ make clean all [...] CC src/factor.o src/factor.c:679:1: error: 'WIDE_UINT_BITS' undeclared here (not in a function) src/factor.c:679:1: err

bug#12841: Make for AIX 8.20 fails with undefined symbol .rpl_malloc

2012-11-13 Thread Bernhard Voelker
On 11/13/2012 04:46 PM, Paul Eggert wrote: > On 11/12/2012 11:00 PM, Bernhard Voelker wrote: >> I'm not sure, but shouldn't the prime list src/primes.h depend >> on src/make-prime-list? > > I'd rather not do that, since that would mean every builder > wo

bug#12903: "cut -b 0-" is supposed to be invalid

2012-11-16 Thread Bernhard Voelker
ange starting with 0 (-f 0-2) >as invalid". >Can you confirm this is an incomplete bugfix made on 2007-05-22? I'd say yes. Here's a proposed fix. Have a nice day, Berny >From 26c6ab75498c4ec6710828799f5452c8243feda1 Mon Sep 17 00:00:00 2001 From: Bernhard Voelker D

bug#12903: "cut -b 0-" is supposed to be invalid

2012-11-18 Thread Bernhard Voelker
On 11/16/2012 11:23 PM, Jim Meyering wrote: > Bernhard Voelker wrote: > >> Hi Marcel, >> >> On 11/16/2012 07:35 AM, Marcel Böhme wrote: >>>Hi, >>>The command "echo 12345 | cut -b 0-" prints an empty line while it >>>

bug#12903: "cut -b 0-" is supposed to be invalid

2012-11-19 Thread Bernhard Voelker
On 11/19/2012 02:05 AM, Jim Meyering wrote: > You're welcome to push it. Thanks, pushed: http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=commit;h=1482f730b47d1abad3d75199dde5237d9bf16a4a Marking as done. Have a nice day, Berny

bug#12212: [patch] dd's help text contains superfluous word

2012-11-19 Thread Bernhard Voelker
On 09/18/2012 04:08 PM, Bernhard Voelker wrote: > On 09/18/2012 03:20 PM, Jim Meyering wrote: >> Applied with the above log tweaks. >> Will push shortly. > > Thanks! That was commit v8.19-136-g6274128 back on Sep 18th. 2012. Marked as done. Have a nice day, Berny

bug#12940: Deprecated "nl --page-increment" to be removed

2012-11-20 Thread Bernhard Voelker
-up. Here's a patch. BTW: Interestingly, there was no test-case for the -i option. Well, tests/misc/nl.sh doesn't cover too much cases anyway. Have a nice day, Berny >From 31c86a3fe100ac68bf7fc34800bdd615c9e06e88 Mon Sep 17 00:00:00 2001 From: Bernhard Voelker Date: Tue, 20 Nov 201

bug#12940: Deprecated "nl --page-increment" to be removed

2012-11-20 Thread Bernhard Voelker
On 11/20/2012 11:04 AM, Pádraig Brady wrote: > On 11/20/2012 09:58 AM, Bernhard Voelker wrote: >> diff --git a/NEWS b/NEWS >> index e4a284c..df4b2e4 100644 >> --- a/NEWS >> +++ b/NEWS >> @@ -25,6 +25,11 @@ GNU coreutils NEWS-*-

bug#12947: [brl...@debian.org: Bug#598018: install: temporary insecure file permissions]

2012-11-21 Thread Bernhard Voelker
On November 21, 2012 at 3:12 AM Paul Eggert wrote: > I pushed this [...] This is more of a question, and I may be wrong, but isn't here still a race afterwards? execve("src/ginstall", ["src/ginstall", "-g", "video", "-m", "664", "src/ginstall", "/tmp/g"], ...) = 0 ... stat("src/gi

bug#12959: "seq -w -1e-3 9" misaligned

2012-11-23 Thread Bernhard Voelker
On 11/23/2012 10:04 AM, Pádraig Brady wrote: > > The attached should fix this. > * src/seq.c (scan_arg): Calculate the width more accurately > for numbers specified using scientific notation. > * tests/misc/seq.pl: Add test cases for cases that were mishandled s/$/./ > * NEWS: Mention the fix.

bug#12959: "seq -w -1e-3 9" misaligned

2012-11-23 Thread Bernhard Voelker
On 11/23/2012 11:13 AM, Pádraig Brady wrote: > Thanks for the review! > Pádraig. No worries, you're welcome. For what it's worth, you may also add an integer test case like "seq -w -1e3 1" which is/was also affected by this bug: $ seq -w -1e3 1 | head -n 2 -1000 -999 Have a nice day, Be

bug#12966: cut: Problems with overlapping, open-ended ranges

2012-11-24 Thread Bernhard Voelker
On 11/24/2012 08:11 AM, Jim Meyering wrote: > Subject: [PATCH] cut: interpret "-b3-,2-" like "-b2-", not like "-b3-" Isn't it the other way round? I.e. "-b3-,2-" has already been working correctly. - Subject: [PATCH] cut: interpret "-b3-,2-" like "-b2-", not like "-b3-" + Subject: [PATCH] cut: i

bug#13030: factor: infinite loop on Linux/powerpc

2012-12-05 Thread Bernhard Voelker
On 11/29/2012 08:34 PM, Pádraig Brady wrote: > On 11/29/2012 02:28 PM, Colin Watson wrote: > The fix looks good thanks. > I'll commit [...] Now, "make syntax-check" fails: Colin Watson maint.mk: remove the above names from THANKS.in make: *** [sc_THANKS_in_duplicates] Error 1 Before remov

bug#13030: factor: infinite loop on Linux/powerpc

2012-12-05 Thread Bernhard Voelker
On 12/05/2012 11:08 AM, Colin Watson wrote: > On Wed, Dec 05, 2012 at 09:58:11AM +0100, Bernhard Voelker wrote: >> Now, "make syntax-check" fails: >> >> Colin Watson >> maint.mk: remove the above names from THANKS.in >> make: *** [sc_THANKS_in_dup

bug#13099: possible test bug in "tests/df/total-verify.sh"

2012-12-06 Thread Bernhard Voelker
quot;Mounted on") field. Well, it seems that none of us had a vfat file system mounted while running the tests yet. I wrapped your fix in the following patch. As there's no official coreutils release yet with that bug, I didn't add a NEWS entry. Have a nice day, Berny >From d59716

bug#13099: possible test bug in "tests/df/total-verify.sh"

2012-12-06 Thread Bernhard Voelker
On 12/06/2012 10:55 AM, Pádraig Brady wrote: > That still passes here with a "standard" file system at least. > Please push. I could reproduce the failure with a loop-mounted vfat image. Thanks for the review, pushed: http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=43a6ccf0

bug#13119: cp --no-preserve=mode exits 1 since coreutils 8.20

2012-12-08 Thread Bernhard Voelker
lse if (x->explicit_no_preserve_mode) +{ + set_acl (dst_name, dest_desc, 0666 & ~cached_umask ()); + return_val = false; +} And furthermore, the test preserve-mode.sh which would have detected this error, don't check cp's exit code ;-( Here's a proposed patch

bug#13119: cp --no-preserve=mode exits 1 since coreutils 8.20

2012-12-09 Thread Bernhard Voelker
On 12/09/2012 01:48 PM, Pádraig Brady wrote: > The patch looks perfect. Thanks for the review. Pushed: http://git.sv.gnu.org/cgit/coreutils.git/commit/?id=62543570 Marking the bug as "done". Have a nice day, Berny

bug#13172: tail: unrecognized file system type

2012-12-14 Thread Bernhard Voelker
quot;remote" file system, i.e. not supporting inotify. Conrad, would you mind giving us your full name, so that we can mention it correctly in THANKS.in? Here comes the patch. Have a nice day, Berny >From 5997134899c9abd5efd635a446fb1a3975483a82 Mon Sep 17 00:00:00 2001 From: Bernhard Voel

bug#13172: tail: unrecognized file system type

2012-12-14 Thread Bernhard Voelker
On 12/14/2012 10:28 AM, Pádraig Brady wrote: > On 12/14/2012 09:18 AM, Bernhard Voelker wrote: >> +case S_MAGIC_CEPH: /* 0x00c36400 remote */ >> + return "ceph"; > > s/00c/00C/ for consistency > > thanks for handling this. > Pádraig. Thanks for t

bug#13227: seq -s broken in coretutils 8.20?

2012-12-19 Thread Bernhard Voelker
On 12/20/2012 01:33 AM, Pádraig Brady wrote: > The attached should fix it. The patch looks good, thanks. Have a nice day, Berny

bug#13243: closed (Re: bug#13243: [PATCH] enhancement: modify md5sum to allow piping)

2012-12-22 Thread Bernhard Voelker
On 12/21/2012 12:12 AM, Daniel Santos wrote: > On 12/20/2012 11:49 PM, Eric Blake wrote: >> dd if=/dev/sda3 | pbzip2 -c2 | tee >(md5sum > /tmp/sda3.dat.bzip2.md5) | >> netcat 192.168.1.123 45678 > Wow! It was worth writing a patch just to discover the >(list) and > <(list) constructs. I knew abo

bug#13280: od 8.19: inappropriate formatting of the man pages

2012-12-27 Thread Bernhard Voelker
On December 27, 2012 at 12:42 PM "Pádraig Brady" wrote: > On 12/26/2012 07:43 AM, Akim Demaille wrote: > I've adjusted things a bit in the attached patch, Looks good. As it was reported as a bug, shouldn't we mention it in the commit message: - Reported by Akim Demaille. + Reported by Akim D

bug#13389: echo command redhat linux OS

2013-01-08 Thread Bernhard Voelker
tag 13389 notabug thanks On 01/08/2013 06:57 AM, Mohanad Azzam wrote: > Dears > > Could we print values to be as three column ,each column present the values > of each file. > > More explanation : > I have three files ,each file include a queue of values .I need to print all > the values by on

bug#13391: dd silently ignores lseek error

2013-01-08 Thread Bernhard Voelker
On 01/09/2013 02:14 AM, Pádraig Brady wrote: > I had a look around for a tool to verify > that a file/device supports the seek operation > and couldn't find one. > So this seems like useful functionality. > Worth applying the attached? > * cfg.mk (sc_dd_O_FLAGS): Add O_SEEKABLE to the list of pr

bug#13394: Misalignment for "seq -w"

2013-01-09 Thread Bernhard Voelker
On 01/09/2013 11:14 AM, Marcel Böhme wrote: > Dear all, > > There are the following problems with the -w parameter of the seq tool: > > $seq -w -1 1.0 0 > -1.0 > 0.0 > 1.0 > > But it should print: > -1.0 > 00.0 > 01.0 Hmm, according to the TEXI manual, the FIRST number should also use a fixed p

bug#13394: Misalignment for "seq -w"

2013-01-09 Thread Bernhard Voelker
On 01/09/2013 01:05 PM, Pádraig Brady wrote: > Looks like a bug. I'll fix with: > > diff --git a/src/seq.c b/src/seq.c > index e1b467c..3eb53f8 100644 > --- a/src/seq.c > +++ b/src/seq.c > @@ -332,6 +332,8 @@ get_default_format (operand first, operand step, operand > last) > last_wi

bug#13447: ln "" foo gives misleading error message

2013-01-15 Thread Bernhard Voelker
On 01/15/2013 09:23 AM, Ken Irving wrote: > (Previously sent in error to the bug-gnu-utils list.) > > I've been using symbolic links in a non-file-related way, e.g., to store > arbitrary string values, but find that if I try to create a symlink with > an empty 'target' name, e.g., as 'ln -s "" foo

bug#13447: ln "" foo gives misleading error message

2013-01-15 Thread Bernhard Voelker
On 01/15/2013 10:44 AM, Bob Proulx wrote: > Jim Meyering wrote: >> Pádraig Brady wrote: >>> Interestingly I notice that solaris for example allows a NULL old_path. >> >> That Solaris behavior is contrary to POSIX 2008 >> http://pubs.opengroup.org/onlinepubs/9699919799/functions/symlink.html > > Wh

bug#13453: comm

2013-01-16 Thread Bernhard Voelker
On 01/16/2013 05:53 PM, Dennis Miller wrote: > My version doesn't have those options. I'm running > GNU coreutils 5.97-34.el5_8.1. What version has these options? The option --check-order has been added in coreutils-7.0 back in 2008. Have a nice day, Berny

bug#13525: 8.20 - [seq 1 3 1] treated as [seq 1 3]

2013-01-22 Thread Bernhard Voelker
On 01/22/2013 01:17 PM, Pádraig Brady wrote: > Updated patch attached. That one is looking good ... but while we're at it: Anyone tried this, i.e. a Zero as INCREMENT? $ seq 1 0 2 This is equal to `yes 0`. Well, this is probably a (not documented) feature, but in the following examples, the "

bug#13525: 8.20 - [seq 1 3 1] treated as [seq 1 3]

2013-01-22 Thread Bernhard Voelker
On 01/22/2013 02:24 PM, Pádraig Brady wrote: > Yes I was wondering that myself. > > Though I suppose that `seq 0 0 1` prints endlessly, > means that it's consistent that as long as start <= end > and step == 0, then start is printed endlessly. Yes, from a mathematical point of view, seq is right.

bug#13530: head: memory exhausted when printing all from stdin but last P/E bytes

2013-01-23 Thread Bernhard Voelker
On 01/23/2013 01:34 PM, Pádraig Brady wrote: > There is the argument that we _should_ allocate > everything up front to indicate immediately > that the system can't (currently) support the requested operation, > but given the 'currently' caveat above I guess it's > more general to fail when we actu

bug#13530: head: memory exhausted when printing all from stdin but last P/E bytes

2013-01-23 Thread Bernhard Voelker
On 01/23/2013 02:03 PM, Pádraig Brady wrote: > On 01/23/2013 12:53 PM, Bernhard Voelker wrote: >> head doesn't "allocate everything up front" - instead, it only >> allocates the pointer array which would hold the actual data. > > Sure. I was wondering whet

bug#13538: sort wrong

2013-01-24 Thread Bernhard Voelker
tag 13538 + notabug close 13538 stop On 01/24/2013 07:16 AM, XU Jian H wrote: > $ cat log > /VOBS/MediaGW/gw/src/cha > /VOBS/MediaGW/gw/src/res > /VOBS/MediaGW/gw/src/res > /VOBS/MediaGW/gw/src/cha > /VOBS/MediaGW/gw/src/cha > /VOBS/MediaGW/gw/src/cha > /VOBS/MediaGW/gw/src@@/m > /VOBS/MediaGW/gw/

bug#13535: timeout command not working when SIGALRM blocked

2013-01-24 Thread Bernhard Voelker
On 01/25/2013 04:12 AM, Pádraig Brady wrote: > I agree with your patch, and have attached an updated version. +1 if you mention http://bugs.gnu.org/13535. Have a nice day, Berny

bug#13582: [PATCH] stat: add ext4 to the ext2/ext3 list

2013-01-28 Thread Bernhard Voelker
tag 13582 + notabug close 13582 stop On 01/29/2013 07:13 AM, Mike Frysinger wrote: > Since ext4 returns the same info as ext2/ext3, add it to the list. > This fixes the output of running `stat -f / -c %T` on my system that > has an ext4 rootfs. Thanks for the patch, however, you submitted it to t

bug#13582: [PATCH] stat: add ext4 to the ext2/ext3 list

2013-01-29 Thread Bernhard Voelker
On 01/29/2013 07:39 PM, Mike Frysinger wrote:> On Tuesday 29 January 2013 01:57:42 Bernhard Voelker wrote: >> On 01/29/2013 07:13 AM, Mike Frysinger wrote: >> Thanks for the patch, however, you submitted it to the bug-coreutils >> mailing list which automatically opened a

bug#13582: [PATCH] stat: add ext4 to the ext2/ext3 list

2013-01-30 Thread Bernhard Voelker
On 01/30/2013 10:27 PM, Bob Proulx wrote: > I am just adding references here to tie in the previous discussions. Thanks for that excellent summary. Reading (again) through it, it seems to me that we have 2 arguments: a) basically, it would be good to be able to distinguish ext2/ext3 from ext4 wh

bug#13582: [PATCH] stat: add ext4 to the ext2/ext3 list

2013-01-31 Thread Bernhard Voelker
On 01/30/2013 11:44 PM, Bob Proulx wrote: > Bernhard Voelker wrote: >> b) just adding "ext4" to "ext2/ext3" as it has already been proposed >> by several people could break existing scripts that use code like this: >> >>> test "$(stat

bug#13627: cut: Commit 06aeeec reintroduced SEG_FAULT

2013-02-04 Thread Bernhard Voelker
On 02/04/2013 12:52 PM, Pádraig Brady wrote: > On 02/04/2013 09:22 AM, Marcel Böhme wrote: >> The current version of cut (after 6.12.2012) exposes a SEG_FAULT: >> $echo 123 | cut --output-del="." -b-1,9- > Nice one! > The attached should fix it. Hi Padraig, thanks. The fix look

bug#13665: Question about the translation of double_to_human

2013-02-10 Thread Bernhard Voelker
On 02/10/2013 05:28 PM, Pádraig Brady wrote: > I've cleaned this up in the attached so that both use the same > method to output the messages, and both now enable these messages > with the the same ---debug option. > Also translations are removed from these messages, > and a syntax check added to s

bug#13681: [patches] numfmt: standardize synopsis and docstring, and slice up helptext

2013-02-10 Thread Bernhard Voelker
On 02/10/2013 09:34 PM, Benno Schulenberg wrote: > > On Sun, Feb 10, 2013, at 21:01, Pádraig Brady wrote: >> I'll apply this for the upcoming release. > > Thanks. Just to be sure: this will require a pre3, if translations > are meant (where possible) to be complete. > > Oh, and one small overlo

bug#13681: [patches] numfmt: standardize synopsis and docstring, and slice up helptext

2013-02-10 Thread Bernhard Voelker
On 02/11/2013 02:42 AM, Pádraig Brady wrote: > On 02/10/2013 10:11 PM, Bernhard Voelker wrote: >> I'll push tomorrow unless I receive further comments. > > +1 Thanks, pushed: http://git.sv.gnu.org/cgit/coreutils.git/commit/?id=b5f45b64 Marking as done. Have a nice day, Berny

bug#13681: [patches] numfmt: standardize synopsis and docstring, and slice up helptext

2013-02-11 Thread Bernhard Voelker
On 02/11/2013 10:31 AM, Pádraig Brady wrote: > Actually I noticed a small issue with man page formatting, > which I addressed with the attached. Good catch, thanks! +1 Have a nice day, Berny

bug#13666: Touch: Cannot use a symbolic link as a reference time

2013-02-13 Thread Bernhard Voelker
tag 13666 notabug thanks On 02/09/2013 08:06 PM, Michael_google gmail_Gersten wrote: > I can set the timestamp of a symbolic link with "-h". > > But I cannot use "-r -h" to use the timestamp on a symbolic link as > the reference time. Hello Michael, thanks for the bug report, however I doubt th

bug#13737: Add -h option to 'users'

2013-02-18 Thread Bernhard Voelker
On 02/18/2013 09:31 PM, anatoly techtonik wrote: > On Mon, Feb 18, 2013 at 8:21 PM, Eric Blake wrote: > Your phrase about GNU users preference can not be backed up by any proof > links, so it can not serve as an argument. That's why I proposed a poll. > Nowadays GNU users are also Python and Ruby

bug#13911: make check errors for coreutils-8.21 on Solaris 10

2013-03-10 Thread Bernhard Voelker
On 03/09/2013 03:15 PM, Marc Girod wrote: Hi Marc, thanks for reporting the test failures. > FAIL: tests/tail-2/F-vs-missing.sh (exit: 1) This one is a failure during the cleanup after the actual test ... > + cleanup_ > + : > + cd /proj/vobadm100/tmp/coreutils-8.21 > + chmod -R u+rwx /proj/vob

bug#14024: Test failure in coreutils 8.13

2013-03-27 Thread Bernhard Voelker
On 03/27/2013 02:13 PM, Pádraig Brady wrote: > On 03/27/2013 11:47 AM, Ellis N. Thomas wrote: >> Results of grep FAIL: tests/test-suite.log >> FAIL: tests/cp/nfs-removal-race.sh (exit: 99) >> FAIL: tests/ls/getxattr-speedup.sh (exit: 99) > > Both of those are caused by: > >> + gcc -std=gn

bug#14108: Bug in dd

2013-04-02 Thread Bernhard Voelker
> On April 1, 2013 at 6:58 AM Bob Proulx wrote: > Haneef Mubarak wrote: > > dd doesn't have a quiet option, ie: dd --quiet > > > > Having a quiet option would allow for an easier use of piping with dd > > The dd manual documents this as "status=none": Additional note: this has been added in coreu

bug#14116: [PATCH] ln: allow to overwrite relative symlink

2013-04-03 Thread Bernhard Voelker
On 04/03/2013 07:43 PM, Pádraig Brady wrote: > Subject: [PATCH 1/2] ln: --relative: fix updating of existing symlinks > > Don't dereference an existing symlink being replaced. > I.E. generate the symlink relative to the symlink's containing dir, > rather than to some arbitrary place it points to.

bug#14174: BUG REP: tee takes an annoyingly long time in some system.

2013-04-11 Thread Bernhard Voelker
tag 14174 + moreinfo close 14174 thanks On 04/10/2013 11:48 AM, Gao, Jie (Kyrie, HPIT-DS-CDC) wrote: > Hello sir, > > Not sure if it is tee's problem. Tee works well in my workstation, but it > will take > a long time on cluster. See the example below. Could you if possible show me > the > reas

bug#14189: ls -d bug ??

2013-04-12 Thread Bernhard Voelker
On 04/12/2013 08:06 AM, Bob Proulx wrote: > Some local wordsmithing turned out the following as a better > improvement. It lists what it does in the positive first. And > removes the negative which was seen as being too confusing. > > `-d' > `--directory' > List only the name of directo

  1   2   3   4   5   6   7   >