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 +++

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 --color=always

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. The output

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

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't know what it does. Can

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'

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 Trying to mv the hardlink over the symlink seems to succeed

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 that symlink: $ touch f ln -s f l ln l s ls -ogi total 0 6444 -rw-r--r

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? $ ~/git/coreutils/src/mv -f s l /home/berny/git

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 @@ -1229,7

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

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

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

2012-07-05 Thread Bernhard Voelker
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 m...@bernhard-voelker.de Date: Thu, 5 Jul 2012 08:44:22 +0200 Subject: [PATCH] df: Remove

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 agree,

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 years in df.1 and df --help: COREUTILS-6_9-151-g1e07a21 11 years

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 # column name-h or

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 FREEPCT, but USEDPCT. I'd leave

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 it trigger.

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

2012-08-17 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.

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 would

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 to

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, Berny

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) sed -i s/df

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 foo $

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 Notice how

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-05 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 into a shell

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 coreut...@tlinx.org 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

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

2012-09-06 Thread Bernhard Voelker
On September 6, 2012 at 2:12 PM Paolo Bonzini bonz...@gnu.org 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

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 j...@meyering.net 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 first patch created on cygwin) From

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

2012-09-06 Thread Bernhard Voelker
On September 6, 2012 at 7:23 PM Paul Eggert egg...@cs.ucla.edu 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

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, + # that

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. What did you want to say by this? I understood

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 ;-) I've never written a patch in other

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 bensb...@justemail.net Date: Tue, 18 Sep 2012 14:58:42 +0200

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

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 j...@meyering.net 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):

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, persistently

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:

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 would have to build src/primes.h, and I'd rather

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

2012-11-16 Thread Bernhard Voelker
. 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 m...@bernhard-voelker.de Date: Fri, 16 Nov 2012 21:12:16 +0100 Subject: [PATCH

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 should fail with fields and positions are numbered from 1 according

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
: 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 m...@bernhard-voelker.de Date: Tue, 20 Nov 2012 10:39:19

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-*- outline -*- field can be in any column

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 egg...@cs.ucla.edu 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 ...

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, Berny

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: interpret

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 removing

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_duplicates] Error 1 Before removing that entry: Are you

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

2012-12-06 Thread Bernhard Voelker
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 d597168b25d58dbd29cf04222608eef262a7d08a Mon Sep 17 00:00:00 2001 From: Bernhard Voelker m...@bernhard-voelker.de Date: Thu, 6 Dec 2012 10:11:42 +0100 Subject: [PATCH

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
; +} 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. Have a nice day, Berny From 62543570d72b756a3b04ca9d1ebec6f4dd2eea4b Mon Sep 17 00:00:00 2001 From: Bernhard Voelker m...@bernhard-voelker.de Date

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
, 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 Voelker m...@bernhard-voelker.de Date: Fri, 14 Dec 2012 10:11:51 +0100 Subject: [PATCH] tail,stat: improve

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 the review. Konrad also confirmed that it works

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 about tee,

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 p...@draigbrady.com 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. +

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 one

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 private

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 point decimal

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_width--;

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', the

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 Where does it

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 1

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 actually

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 whether that should change to allocate everything up

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 the

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 new ticket. Therefore, I'm

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

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 -f -c %T .) = ext2/ext3 ... I'm wondering how often users check

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 looks okay,

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 stop

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 overlooked

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#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 ebl...@redhat.com 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

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

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=gnu99 -shared

bug#14108: Bug in dd

2013-04-02 Thread Bernhard Voelker
On April 1, 2013 at 6:58 AM Bob Proulx b...@proulx.com 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

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 reason why

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 directories,

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

2013-04-12 Thread Bernhard Voelker
On 04/12/2013 09:06 PM, Jim Meyering wrote: Pádraig Brady wrote: ... I expect to push soon, the attached more complete fix to realloc the array. Thanks! That change looks fine and passed tests here, too. +1 Have a nice day, Berny

bug#14251: coreutils-8.15: tail.c : Need support for StorNext file system as distributed file system

2013-04-25 Thread Bernhard Voelker
On April 25, 2013 at 12:08 AM Pádraig Brady p...@draigbrady.com wrote: While I'm slightly hesitant above adding support for this closed source file system, there is precedence, and I see the clients are GPL. So I propose to support this with the attached. Hi Padraig, I also searched a bit

bug#13127: [PATCH] cut: use only one data strucutre

2013-05-08 Thread Bernhard Voelker
On May 7, 2013 at 2:10 PM Pádraig Brady p...@draigbrady.com wrote: It's always best to have separate changes. I've split the fix out (attached) with an associated test. The patch looks fine, thanks. Have a nice day, Berny

bug#13127: [PATCH] cut: use only one data strucutre

2013-05-08 Thread Bernhard Voelker
On May 7, 2013 at 3:51 PM Pádraig Brady p...@draigbrady.com wrote: I'll apply the attached in a little while. +1 Have a nice day, Berny

  1   2   3   4   5   6   >