bug#20437: ls links too many dynamic libraries

2015-04-27 Thread Pádraig Brady
tag 20437 notabug close 20437 stop On 27/04/15 06:30, Paul Eggert wrote: Currently GNU 'ls' dynamically links a whole bunch of libraries, libraries like libpcre and liblzma. Can we figure out some way to remove the runtime dependencies on these libraries? It's better if a core utility

bug#20437: ls links too many dynamic libraries

2015-04-27 Thread Pádraig Brady
On 27/04/15 14:12, Pádraig Brady wrote: tag 20437 notabug close 20437 stop On 27/04/15 06:30, Paul Eggert wrote: Currently GNU 'ls' dynamically links a whole bunch of libraries, libraries like libpcre and liblzma. Can we figure out some way to remove the runtime dependencies

bug#20442: bug+patch: du output misaligned on different terminals

2015-04-27 Thread Pádraig Brady
tag 20442 wontfix close 20442 stop On 27/04/15 20:11, L. A. Walsh wrote: This is a fix/work-around for (RFE#19849 (bug#19849) which was about addingg options to expand tabs and/or set a tabsize for output from 'du' so output would line up as intended. Without that enhancement, the

bug#20442: bug+patch: du output misaligned on different terminals

2015-04-27 Thread Pádraig Brady
On 28/04/15 01:13, Linda Walsh wrote: reopen 20442 thanks === Your more general case doesn't work: du -sh /tmp/t*|numfmt --format %10f numfmt: rejecting suffix in input: ‘4.0K’ (consider using --from) du -sh --time /tmp/t*|numfmt --format %10f numfmt: rejecting suffix in input:

bug#20438: coreutils 8.23 on OS X : Calling gcp -al fails on symbolic links

2015-04-27 Thread Pádraig Brady
On 27/04/15 12:34, Thomas Baigneres wrote: Hello, Consider the following directory hierarchy: $ ls -lR total 0 drwxr-xr-x 4 user group 136 Apr 24 13:51 source ./source: total 16 -rw-r--r-- 1 user group 0 Apr 24 13:50 file.txt lrwxr-xr-x

bug#20310: [COREUTILS 2/2] ls: Don't treat lack of acl support as an error

2015-04-20 Thread Pádraig Brady
On 12/04/15 15:37, Andreas Gruenbacher wrote: * src/ls.c (file_has_acl_cache): When a file system doesn't support acls, fail with errno set to ENOTSUP. (gobble_file): Don't treat lack of acl support as an error. --- src/ls.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-)

bug#20366: Strange behaviour of df (GNU coreutils) 8.23

2015-04-19 Thread Pádraig Brady
On 18/04/15 11:44, Benjamin Beier wrote: Hello, I am running multiple Gentoo/Funtoo servers and yesterday I noticed the output of 'df -a' on the Funtoo systems does not show any information about the root partition, although this works fine on all my Gentoo systems. I compared the

bug#20354: [feature request] ln with command line arguments in reverse order

2015-04-18 Thread Pádraig Brady
tag 20354 wontfix close 20354 stop On 18/04/15 07:09, Bernhard Voelker wrote: On 04/17/2015 04:52 PM, Erik Auerswald wrote: On Fri, Apr 17, 2015 at 01:45:02PM +0100, Pádraig Brady wrote: How I think about it is: cp [OPTION] EXISTING NEW mv [OPTION] EXISTING NEW ln [OPTIONS

bug#20354: [feature request] ln with command line arguments in reverse order

2015-04-17 Thread Pádraig Brady
On 17/04/15 12:45, Erik Auerswald wrote: On Fri, Apr 17, 2015 at 01:12:01PM +0200, Bernhard Voelker wrote: On 04/17/2015 10:39 AM, Ma Jiehong wrote: Currently, 'cp', 'mv' and 'ln' share the same basic syntax, that is to say the following: cp [OPTION] SOURCE DEST mv [OPTION] SOURCE DEST

bug#18748: cp doesn't behaves as mkdir and touch when a default acl exists

2015-04-13 Thread Pádraig Brady
On 13/04/15 00:10, Andreas Grünbacher wrote: When a file is copied with cp, the default is to create the new file in the target directory, with the file mode of the original file as the create mode. This default can be overridden with cp's -p or --preserve=mode options. This has the

bug#19760: [bug] tail -f with inotify fails to follow a file after a rename()

2015-03-31 Thread Pádraig Brady
On 31/03/15 07:30, Bernhard Voelker wrote: On 03/31/2015 05:15 AM, Pádraig Brady wrote: + tail -f continues to follow changes to a file even after it's renamed. + [bug introduced in coreutils-7.5] + It is not 100% clear to me by this sentence what was the actual change; maybe a little

bug#19760: [bug] tail -f with inotify fails to follow a file after a rename()

2015-03-31 Thread Pádraig Brady
BTW given that -f was broken for so long (6 years) it lends more weight to making -f behave like -F by default. Note POSIX allows (and even implies this), and openBSD -f behaves like -F for example. Not something appropriate for coreutils 8.x, and I'd be 60:40 against changing in a later major

bug#20238: Can these Errors be fixed? Thank you.

2015-03-31 Thread Pádraig Brady
On 31/03/15 21:29, Dake Zhang wrote: FAIL: tests/misc/ls-time ++ ls -ut a b c + set b c a + test 'b c a' = 'c b a' + fail=1 + test 1 = 1 + ls -l --full-time --time=access a b c -rw-r--r-- 1 dakez BCM\Domain Users 0 1998-01-14 11:00:00.0 + a -rw-r--r-- 1 dakez BCM\Domain Users

bug#19760: [bug] tail -f with inotify fails to follow a file after a rename()

2015-03-30 Thread Pádraig Brady
On 03/02/15 23:30, Pádraig Brady wrote: On 03/02/15 22:04, Stephane Chazelas wrote: Hello, On Linux, when inotify is used, tail -f file follows a file only until it's renamed. After it is renamed, the inotify watch is removed, which means tail sits there doing nothing and any further

bug#20210: tests/df/skip-duplicates fails on Debian-kFreeBSD due to calling 'strstr(NULL, )'

2015-03-27 Thread Pádraig Brady
On 27/03/15 00:28, Assaf Gordon wrote: Hello, A somewhat exotic test failure: On Debian/kFreeBSD 'tests/df/skip-duplicates' fails with 'df' segfaulting like so: ... ./tests/df/skip-duplicates.sh: line 113: 7741 Segmentation fault LD_PRELOAD=./k.so df ...

bug#20199: Enhancement request for date's -d option: different epochs

2015-03-25 Thread Pádraig Brady
On 25/03/15 12:26, Ulrich Windl wrote: Hi! I'm not subscribed to this list, and I hope this is the right place to report an enhancement request as there seems to be no bugzilla for that. Anyway: When downloading the current leap seconds list for out NTP server I realized that the dates

bug#20199: Enhancement request for date's -d option: different epochs

2015-03-25 Thread Pádraig Brady
On 25/03/15 13:10, Pádraig Brady wrote: On 25/03/15 12:26, Ulrich Windl wrote: Hi! I'm not subscribed to this list, and I hope this is the right place to report an enhancement request as there seems to be no bugzilla for that. Anyway: When downloading the current leap seconds list for out

bug#20203: add example to date man page

2015-03-25 Thread Pádraig Brady
On 25/03/15 23:58, 積丹尼 Dan Jacobson wrote: Please change -I[TIMESPEC], --iso-8601[=TIMESPEC] output date/time in ISO 8601 format. TIMESPEC='date' for date only (the default), 'hours', 'minutes', 'seconds', or 'ns' for date and time to

bug#20172: make each ls sort option say what is first on the man page

2015-03-23 Thread Pádraig Brady
On 23/03/15 00:37, 積丹尼 Dan Jacobson wrote: Please make $ man ls|grep -e -[St].*sort -S sort by file size -t sort by modification time, newest first say $ man ls|grep -e -[St].*sort -S sort by file size, largest first -t sort by modification time,

bug#20166: LC_ALL=C

2015-03-22 Thread Pádraig Brady
tag 20166 notabug close 20166 stop On 22/03/15 15:52, Ralf Richter wrote: LC_ALL=C Dear guys, I use Debian 7.8.0. ls -g does not work. I have to write g out. ls --group-directories-first works fine. Please be so kind and fix that. -g is not the same as --g If there is some other

bug#20120: wc output padding differs when - is in the file list

2015-03-19 Thread Pádraig Brady
On 18/03/15 17:54, Bernhard Voelker wrote: On 03/16/2015 06:42 AM, Eric Mrak wrote: It seems that whenever STDIN is involved the results padding reverts to the BSD-style 7/8 padding. When files are given as input (excluding STDIN) the padding reflects the width of the largest count. When

bug#20130: GNU test behaviour

2015-03-18 Thread Pádraig Brady
tag 20130 notabug close 20130 stop On 17/03/15 22:27, Paul Eggert wrote: On 03/17/2015 02:23 PM, Robson Júnior wrote: `test -e` with no filename being passed to. It returns 0, although it should be 1. No, 'test -e' should exit with status 0, because '-e' is a nonempty string. In

bug#20127: bug in stty

2015-03-18 Thread Pádraig Brady
tag 20127 notabug close 20127 stop On 17/03/15 19:33, Morris Keesan wrote: stty --version: stty (GNU coreutils) 8.13 uname -a: Linux ice4 3.2.0-75-generic #110-Ubuntu SMP Tue Dec 16 19:11:55 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux

bug#20114: tr does not support multibyte characters in the first argument

2015-03-16 Thread Pádraig Brady
On 16/03/15 02:30, Bruno Haible wrote: POSIX [1] specifies that the recognition of characters in 'tr' depends on the environment variables LANG, etc. But trying to replace a multibyte character by another character does not work: $ echo $LANG de_DE.UTF-8 $ enspace=`printf '\u2002'` $

bug#20094: cp --dry-run

2015-03-12 Thread Pádraig Brady
On 12/03/15 02:27, 積丹尼 Dan Jacobson wrote: Proposal: add a new cp --dry-run Reason: we want to know what the verbose output of e.g., $ cp -i -av /mnt/usb/thumb/ tmp/androidBackup/SDCard will be without actually running the command first. We are about to commit to a megabyte copy and we want

bug#20091: mv command

2015-03-11 Thread Pádraig Brady
On 11/03/15 21:02, Rogers, Charles (MAN-Corporate-CON) wrote: Is it ever possible for the mv command ( without using the –u option ) to leave the file(s) in the source directory, while also copying to the destination directory? We were experiencing this under zsh and

bug#20029: 'yes' surprisingly slow

2015-03-10 Thread Pádraig Brady
On 10/03/15 21:18, Ole Tange wrote: On Tue, Mar 10, 2015 at 1:31 AM, Giuseppe Scrivano gscriv...@gnu.org wrote: $ time src/yes `echo {1..2000}` | head -c 2000M | md5sum 55c293324aa6ecce14f0bf30da5a4686 - real0m7.994s user0m11.093s sys 0m2.953s versus (with the patch): $

bug#20029: 'yes' surprisingly slow

2015-03-09 Thread Pádraig Brady
On 07/03/15 12:10, Pádraig Brady wrote: On 07/03/15 11:49, Ole Tange wrote: These two commands give the same output: $ yes `echo {1..1000}` | head -c 2300M | md5sum a0241f2247e9a37db60e7def3e4f7038 - $ yes `echo {1..1000}` | head -c 2300M | md5sum a0241f2247e9a37db60e7def3e4f7038

bug#20029: 'yes' surprisingly slow

2015-03-09 Thread Pádraig Brady
On 09/03/15 20:02, Eric Blake wrote: On 03/09/2015 01:47 PM, Pádraig Brady wrote: Note this change also ensures that yes will only write complete lines for lines softer than BUFSIZ. s/softer/smaller/ * src/yes.c (main): Build up a BUFSIZ buffer of lines, and output that, rather than

bug#20029: 'yes' surprisingly slow

2015-03-09 Thread Pádraig Brady
On 10/03/15 00:31, Giuseppe Scrivano wrote: Pádraig Brady p...@draigbrady.com writes: The attached should make things more efficient here. thanks, Pádraig. From 7959bbf19307705e98f08cfa32a9dcf67672590c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?P=C3=A1draig=20Brady?= p...@draigbrady.com

bug#20060: dummy-man broken

2015-03-08 Thread Pádraig Brady
On 08/03/15 10:38, Marcus Brinkmann wrote: Hi, just a short notice that dummy-man does not work anymore (found this while trying to bootstrap a new architecture): ./man/dummy-man: too many non-option arguments Was this with v8.23 or with git? There is this related recent fix already in

bug#20035: GNU coreutils 8.4 October 2014 TR(1)

2015-03-08 Thread Pádraig Brady
On 08/03/15 03:11, James Geller wrote: Hi, This is not a bug report. I just wanted to say, it would be SO nice if you could include 3 or 45 examples in the man pages. It would have made my life easier. It took me a while and a Google search to get it to work. Please write

bug#20029: 'yes' surprisingly slow

2015-03-07 Thread Pádraig Brady
On 07/03/15 11:49, Ole Tange wrote: These two commands give the same output: $ yes `echo {1..1000}` | head -c 2300M | md5sum a0241f2247e9a37db60e7def3e4f7038 - $ yes `echo {1..1000}` | head -c 2300M | md5sum a0241f2247e9a37db60e7def3e4f7038 - But the time to run is quite different:

bug#19992: Small mistake in comment in source code of ls

2015-03-04 Thread Pádraig Brady
On 03/03/15 21:40, Jarosław Gruca wrote: In the source code of ls (file src/ls.c): { 0, NULL }, /* ec: End color (replaces lc+no+rc) */ there is a small mistake in the comment. In the place of 'lc+no+rc' should be 'lc+rs+rc' ('rs' instead of 'no'). Each file is written as

bug#19969: problem: wc -c doesn't read actual # of bytes in file

2015-03-02 Thread Pádraig Brady
On 02/03/15 21:29, Linda Walsh wrote: Jim Meyering wrote: As root: # cd /proc # find -H [^0-9]* -name self -prune -o -name thread-self -prune -o -type f ! -name kmsg ! -name kcore ! -name kpagecount ! -name kpageflags -print0|wc -c --files0-from=- |sort -n Thanks for the report.

bug#19951: tail: unrecognized file system type 0x013111a8 for `xxx.yyy'. Reverting to polling.

2015-03-01 Thread Pádraig Brady
On 27/02/15 18:02, Bernhard Voelker wrote: On 02/26/2015 07:33 AM, Seymour, Shane M wrote: case S_MAGIC_IBRIX: /* 0x013111A8 remote */ return ibrix; Thanks for the report. As you already found the proper place to insert the code, and as it's a trivial change not requiring

bug#19956: Solved (Re: df fails to show all mounted file systems)

2015-03-01 Thread Pádraig Brady
On 26/02/15 15:41, Vesa-Matti J Kari wrote: Hello, On Thu, 26 Feb 2015, Vesa-Matti J Kari wrote: [...] starting from coreutils-8.21 (i.e. the bug exist in 8.22 and 8.23 too), lots of NFS mounts are missing: vmkari@cedi:/var/tmp/vmk$ coreutils-8.21/src/df Filesystem

bug#13183: tail -f ignores SIGPIPE

2015-02-15 Thread Pádraig Brady
On 14/12/12 14:33, Pádraig Brady wrote: tag 13183 + notabug close 13183 stop On 12/14/2012 02:04 PM, Ruediger Meier wrote: Hi, I want to use tail and grep to follow a file until a particular pattern appears. But tail does not exit when grep is finished. $ echo xxx /tmp/blabla $ tail

bug#19857: BUG with head -n-0

2015-02-13 Thread Pádraig Brady
unarchive 16329 forcemerge 19857 16329 stop On 13/02/15 16:21, matshyeq wrote: Hi All, I think I've found an issue when head is called with -n-0 parameter It should return whole file but only seem to work for files with newline character at the end. For those that don't have it - it

bug#19765: tail -F stops watching when read permissions are removed

2015-02-05 Thread Pádraig Brady
On 04/02/15 10:22, Stephane Chazelas wrote: When watchnig a file by name with tail -F, if read permissions are removed, tail stops watching even though it has a file descriptor open on the file. With inotify: $ : file $ tail -F file [1] 20796 $ exec 3 file $ echo 1 3 1 $ chmod 0

bug#19784: build fails on make-prime-list when asan is enabled

2015-02-05 Thread Pádraig Brady
On 05/02/15 15:21, Yury Usishchev wrote: Hello! We tried to build coreutils with address sanitizer enabled and encountered an error: GEN src/primes.h ==12657== ERROR: AddressSanitizer: heap-buffer-overflow This can be reproduced on git master using gcc-4.8 or gcc-4.9 by git

bug#19760: [bug] tail -f with inotify fails to follow a file after a rename()

2015-02-03 Thread Pádraig Brady
On 03/02/15 22:04, Stephane Chazelas wrote: Hello, On Linux, when inotify is used, tail -f file follows a file only until it's renamed. After it is renamed, the inotify watch is removed, which means tail sits there doing nothing and any further modifications to the file are ignored.

bug#19747: Testsuite summary for GNU coreutils 8.23 - FAIL: 1

2015-02-02 Thread Pádraig Brady
On 02/02/15 20:53, Miras wrote: Hi, When testing the package coreutils-8.23.tar.xz appeared one error. The package is has built correctly. I was building a Linux system using the LFS Linux from scratch. FAIL: tests/df/skip-duplicates == + df --local

bug#19725: Correction for md5sum manual page

2015-01-29 Thread Pádraig Brady
On 29/01/15 16:11, Terry Hoye wrote: Regarding GNU coreutils 8.12.197-032bbSeptember 2011, man md5sum states in part: [quote] The sums are computed as described in RFC 1321. When checking, the input should be a former output of this program. The default mode is to print a line

bug#19681: [PATCH] sync: use syncfs(2) if any argument is specified

2015-01-28 Thread Pádraig Brady
On 28/01/15 08:17, Bernhard Voelker wrote: On 01/27/2015 03:58 PM, Pádraig Brady wrote: From 12c6f0fd7f44133a2af8950c69b2bfa46ea5d3a4 Mon Sep 17 00:00:00 2001 From: Giuseppe Scrivano gscriv...@gnu.org Date: Sun, 25 Jan 2015 01:33:45 +0100 Subject: [PATCH] sync: support syncing specified

bug#19681: [PATCH] sync: use syncfs(2) if any argument is specified

2015-01-27 Thread Pádraig Brady
On 27/01/15 10:48, Giuseppe Scrivano wrote: Pádraig Brady p...@draigbrady.com writes: thanks! Pádraig. Thanks for the review, I've amended the changes you suggested: There were a few problems: - Compile failure without HAVE_SYNCFS (defined) - Various errors() used errno where undefined

bug#19705: unrecognized file system type

2015-01-27 Thread Pádraig Brady
On 27/01/15 16:50, Nicolas St-Pierre wrote: tail: unrecognized file system type 0xf2f52010 for â.logâ. please report this to bug-coreutils@gnu.org mailto:bug-coreutils@gnu.org. reverting to polling Already fixed with: http://git.sv.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=4c49dc82

bug#19681: [PATCH] sync: use syncfs(2) if any argument is specified

2015-01-26 Thread Pádraig Brady
On 26/01/15 08:36, Giuseppe Scrivano wrote: Pádraig Brady p...@draigbrady.com writes: On 25/01/15 18:05, Bernhard Voelker wrote: On 01/25/2015 06:41 PM, Pádraig Brady wrote: So we have: fdatasync fsync syncfs sync referring to:: file data, file data + metadata, file system, all file

bug#19681: [PATCH] sync: use syncfs(2) if any argument is specified

2015-01-26 Thread Pádraig Brady
On 26/01/15 21:27, Giuseppe Scrivano wrote: Pádraig Brady p...@draigbrady.com writes: On 26/01/15 08:36, Giuseppe Scrivano wrote: Pádraig Brady p...@draigbrady.com writes: On 25/01/15 18:05, Bernhard Voelker wrote: On 01/25/2015 06:41 PM, Pádraig Brady wrote: So we have: fdatasync fsync

bug#7420: [Feature request]: add option to dd to fsync|fdatasync after each block written

2015-01-25 Thread Pádraig Brady
unarchive 7420 tag 7420 notabug close 7420 stop On 17/11/10 10:19, Марк Коренберг wrote: [Feature request]: add option to dd to fsync|fdatasync after each block written Suppose I want to show progress with: pv image.img | dd bs=16M of=/dev/sdc it will not work, as dd will write to sdc

bug#19681: [PATCH] sync: use syncfs(2) if any argument is specified

2015-01-25 Thread Pádraig Brady
On 25/01/15 18:05, Bernhard Voelker wrote: On 01/25/2015 06:41 PM, Pádraig Brady wrote: So we have: fdatasync fsync syncfs sync referring to:: file data, file data + metadata, file system, all file systems [...] I'd be incline to go with the _what_ interface above. Either way, I

bug#19681: [PATCH] sync: use syncfs(2) if any argument is specified

2015-01-25 Thread Pádraig Brady
On 25/01/15 15:38, Paul Eggert wrote: If we're adding this sort of option, shouldn't we also give users the ability to invoke fsync and fdatasync on a single file, as opposed to syncfs on an entire file system? Yes good point on also integrating per file syncing. Per file syncing is

bug#7421: [Feature request]: add option to dd to issue ioctl(BLKFLSBUF) on output descriptior after each write or at the end

2015-01-25 Thread Pádraig Brady
unarchive 7421 tag 7421 wontfix close 7421 stop On 17/11/10 10:41, Марк Коренберг wrote: [Feature request]: add option to dd to issue ioctl(BLKFLSBUF) on output descriptior after each write or at the end I already has sent a message about fsync/fdatasync after each write. It seems that

bug#19681: [PATCH] sync: use syncfs(2) if any argument is specified

2015-01-24 Thread Pádraig Brady
thanks I like it! Tweaks below... On 25/01/15 00:48, Giuseppe Scrivano wrote: * configure.ac: Check if syncfs(2) is available. * NEWS: Mention the new feature. * doc/coreutils.texi (sync invocation): Document the new feature. * src/sync.c (usage): Describe that arguments are now accepted.

bug#19654: tr command odd behavior on Linux Platform

2015-01-22 Thread Pádraig Brady
tag 19654 notabug close 19654 stop On 22/01/15 09:52, Kousik Mandal wrote: Hi Team, I am observing an unexpected behavior with tr command. In a directory if there exist one file named a and executing following tr command. This is becoming a FAQ:

bug#19609: coreutils tests on illumos

2015-01-19 Thread Pádraig Brady
tag 19609 notabug close 19609 stop On 15/01/15 18:05, Alexander Pyhalov wrote: Hello. I was looking at coreutils test suite on illumos. There are currently about 5 failing tests. First are 2 df tests. They assume that struct mntent *getmntent (FILE *fp) is used by df. But it's not true on

bug#19605: cp -v vs LC_ALL vs. quote marks

2015-01-15 Thread Pádraig Brady
On 15/01/15 16:01, 積丹尼 Dan Jacobson wrote: I'm saying please don't force me to need LC_ALL=C to make the quotes U+0027 APOSTROPHE always. Long ago there were no quotes. Then somebody thought quotes looked pretty, so they added U+0027 APOSTROPHE always. Then somebody else thought `'

bug#19605: cp -v vs LC_ALL vs. quote marks

2015-01-15 Thread Pádraig Brady
On 15/01/15 17:28, 積丹尼 Dan Jacobson wrote: All I know is in xterm I click three times and all of '...' including the quotes gets copied, which is fine with me. Just keep it all 0x27. Ah right that's an xterm specific feature. See XTerm*on3Clicks here:

bug#19604: echo --help does not work

2015-01-15 Thread Pádraig Brady
tag 19604 notabug close 19604 stop On 15/01/15 06:20, prateek goyal wrote: Hi, when I try to use --help option with echo command, it does not print help contents, but prints --help. prateek@prateek-pc:~/Documents/awk$ echo --help --help You're actually using your shell's echo

bug#19605: cp -v vs LC_ALL vs. quote marks

2015-01-15 Thread Pádraig Brady
On 15/01/15 11:38, 積丹尼 Dan Jacobson wrote: I am glad that these days plain ' is being used instead of goofy `' $ LC_ALL=C cp -v /dev/null /tmp/$RANDOM 21 '/dev/null' - '/tmp/29920' That way one can not worry about copy and pasting them with the mouse. The problem is, if I don't use

bug#19578: Memory leaks in coreutils/lib/locale_charset.c

2015-01-13 Thread Pádraig Brady
tag 19578 notabug close 19578 stop On 13/01/15 09:35, Daiki Ueno wrote: Zhaopeng Li z...@ustc.edu.cn writes: At line 534 of coreutils/lib/locale_charset.c, var ‘aliases' points to a buffer which is allocated using malloc() . This buffer is not freed when codeset is still an empty string

bug#19580: Memory Leak in coreutils/lib/localcharset.c

2015-01-13 Thread Pádraig Brady
forcemerge 19580 19578 stop On 13/01/15 10:31, Zhaopeng Li wrote: At line 221, the assignment (old_res_ptr = res_ptr) will lead to memory leak when iteration of corresponding loop is greater than 3. Same non issue really. We don't want to free() here. I'm not sure how to avoid the warning

bug#19570: bug: df and bind mounts

2015-01-12 Thread Pádraig Brady
On 12/01/15 21:27, Bernhard Voelker wrote: On 01/12/2015 02:31 AM, Pádraig Brady wrote: On 11/01/15 23:36, Vladimir A. Pavlov wrote: run /run tmpfs rw,noatime,nodiratime,nodev,noexec,mode=0755,size=1m 0 0 /run/cgs/httpd /usr/cgs/httpd/run none rw,bind 0 0 Thanks for the analysis and patch

bug#19570: bug: df and bind mounts

2015-01-11 Thread Pádraig Brady
On 11/01/15 23:36, Vladimir A. Pavlov wrote: Hello, I have an issue with df (both in version 8.23 and in master branch). I have tmpfs mounted as /run . There is /run/cgs/httpd subdirectory in /run (just a subdirectory, not a tmpfs or another mount). This /run/cgs/httpd is bind-mounted to

bug#19544: RFE: please fix limited dd output control (want xfer stats, but not blocks).

2015-01-09 Thread Pádraig Brady
On 09/01/15 11:18, Linda Walsh wrote: The blocks are a bit uninteresting: 7+0 records in 7+0 records out 6+0 records in 11+0 records out 8+0 records in 8+0 records out 2+0 records in ... 2+0 records out 15+0 records in 15+0 records out --- Tells me nothing -- not size

bug#19530: Solaris 10 df and zfs

2015-01-07 Thread Pádraig Brady
On 07/01/15 22:13, Ted Carr wrote: Pádraig, Here is what I get: # ./stat -f / File: / ID: 4010002 Namelen: 255 Type: zfs Block size: 131072 Fundamental block size: 512 Blocks: Total: 106248371 Free: 83645114 Available: 83645114 Inodes: Total: 83922472 Free:

bug#19530: Solaris 10 df and zfs

2015-01-07 Thread Pádraig Brady
On 07/01/15 17:00, Ted Carr wrote: Hello All, I have a requirement for the latest version of coreutils on Solaris 10 SPARC and everything is working as expected with the exception of ‘df’ on ZFS based filesystems… It is having an issue correctly displaying the file system mounted

bug#19520: shuf: extra operand handling with -i

2015-01-05 Thread Pádraig Brady
On 06/01/15 03:42, Paul Eggert wrote: Thanks for reporting that. I installed the attached patch, which has a test for the bug. It's probably worth mentioning the crash in NEWS and augmenting the test to detect crashes in other situations, which I've done in the attached. thanks, Pádraig.

bug#19503: most translations of proper names aren't being used

2015-01-04 Thread Pádraig Brady
On 04/01/15 16:50, Jim Meyering wrote: On Sun, Jan 4, 2015 at 7:53 AM, Pádraig Brady p...@draigbrady.com wrote: ... Also there is the more general point about how correct it is to attribute a program to author(s) in any case, as that tracked to a much more accurate level of detail by git

bug#19228: Challenging output from help2man split

2015-01-01 Thread Pádraig Brady
On 30/11/14 17:33, Pádraig Brady wrote: On 30/11/14 17:06, Kevin O'Gorman wrote: This is sent to the bug addresses for both help2man and split because I'm not sure where the fault lies. Both are GNU projects, so I hope you can cooperate and figure it out. In any event, the output

bug#19476: Poor output from help2man split

2014-12-31 Thread Pádraig Brady
forcemerge 19228 19476 stop

bug#19228: bug#19476: Poor output from help2man split

2014-12-31 Thread Pádraig Brady
On 31/12/14 15:32, Kevin O'Gorman wrote: On Wed, Dec 31, 2014 at 4:06 AM, Pádraig Brady p...@draigbrady.com mailto:p...@draigbrady.com wrote: forcemerge 19228 19476 stop I have no clue what that means. Just merging with the same issue you raised a month ago. I'll have

bug#19456: GNU coreutils - touch / add -v, --verbose option

2014-12-28 Thread Pádraig Brady
On 28/12/14 08:33, Jari Aalto wrote: It would be nice to see progress of touched files. Please add option[1]: -v, --verbose Jari [1] Not included in touch(1), GNU coreutils 8.23 Maybe. What's your use case exactly. In other tools that have --verbose output, there is the opportunity

bug#19447: chmod - problem

2014-12-26 Thread Pádraig Brady
tag 19447 notabug close 19447 stop On 26/12/14 18:28, Tom wrote: Hi chmod does not work recursively. The command chmod --recursive --verbose a-x ./*.txt only has effects in the actual working directory, but not in the subdirectories. You're passing only .txt files to chmod here so

bug#19375: closed (Re: bug#19377: bug#19378: [PATCH 3/4] cat, chcon, chgrp, chmod, chown, cp, du, head: support wildcards on OS/2)

2014-12-18 Thread Pádraig Brady
reopen 19377 stop On 19/12/14 01:13, KO Myung-Hun wrote: GNU bug Tracking System wrote: Your bug report #19377: [PATCH 1/4] doc: add $(EXEEXT) suffix to the executables which was filed against the coreutils package, has been closed. The explanation is attached below, along with your

bug#19377: bug#19378: [PATCH 3/4] cat, chcon, chgrp, chmod, chown, cp, du, head: support wildcards on OS/2

2014-12-15 Thread Pádraig Brady
On 15/12/14 01:15, KO Myung-Hun wrote: Pádraig Brady wrote: forcemerge 19378 19377 stop On 14/12/14 03:47, KO Myung-Hun wrote: And ln,ls,mv,rm,tail. * src/cat.c (main): Expand wildcards on OS/2. * src/chcon.c (main): Likewise. * src/chgrp.c (main): Likewise. * src/chmod.c (main

bug#19375: [PATCH 1/4] doc: add $(EXEEXT) suffix to the executables

2014-12-14 Thread Pádraig Brady
foremerge 19375 19377 stop On 14/12/14 03:47, KO Myung-Hun wrote: * man/local.mk: Add $(EXEEXT) suffix to the executables. LGTM. thanks, Pádraig.

bug#19374: [PATCH 2/4] build: configure.ac: support a response file on OS/2

2014-12-14 Thread Pádraig Brady
forcemerge 19374 19377 stop On 14/12/14 03:47, KO Myung-Hun wrote: * configure.ac (LDFLAGS): Add -Zargs-resp on os2*. This imparts no information. Please comment as to the why rather than the what. --- configure.ac | 3 +++ 1 file changed, 3 insertions(+) diff --git a/configure.ac

bug#19377: bug#19378: [PATCH 3/4] cat, chcon, chgrp, chmod, chown, cp, du, head: support wildcards on OS/2

2014-12-14 Thread Pádraig Brady
forcemerge 19378 19377 stop On 14/12/14 03:47, KO Myung-Hun wrote: And ln,ls,mv,rm,tail. * src/cat.c (main): Expand wildcards on OS/2. * src/chcon.c (main): Likewise. * src/chgrp.c (main): Likewise. * src/chmod.c (main): Likewise. * src/chown.c (main): Likewise. * src/cp.c (main):

bug#19377: bug#19376: [PATCH 4/4] build: use -pi.bak instead of -pi

2014-12-14 Thread Pádraig Brady
forcemerge 19376 19377 stop On 14/12/14 03:47, KO Myung-Hun wrote: This fixes the following error. - Can't do inplace edit without backup. - * Makefile.am (dist-hook): Use -pi.bak instead of -pi. * bootstrap.conf (bootstrap_epilogue): Likewise. --- Makefile.am| 2 +-

bug#19305: Integrate dd wrapper progress-dd with coreutils

2014-12-08 Thread Pádraig Brady
On 08/12/14 14:29, Sebastian Pipping wrote: Hi there, I would like to see dd wrapper progress-dd into coreutils. All it does is to keep sending USR1 to a child dd process to increase usability of that dd feature. It's up here:

bug#19238: Fully fix du circular warning on bind mounts

2014-12-01 Thread Pádraig Brady
On 01/12/14 08:54, Boris Ranto wrote: The du circular warning can still be hit even though a file system is in good condition. All we need to do is to get the message is to begin traversing the file system between the bind mount source and bind mount target directories, i.e this short script

bug#19184: coreutils-8.23 Compile Error on Solaris 8 - Sun Studio 11

2014-12-01 Thread Pádraig Brady
On 01/12/14 13:58, Ted Carr wrote: Hi All, Sorry for the delay... Requested output: bash-2.03$ grep gl_cv_func_printf_sizes_c99 config.log gl_cv_func_printf_sizes_c99=no Thanks for the confirmation. Your issue should be address in the upcoming 8.24 release that contains these changes:

bug#19240: cut 8.22 adds newline

2014-12-01 Thread Pádraig Brady
On 01/12/14 21:18, Eric Blake wrote: [re-adding the bug, with permission] On 12/01/2014 01:10 PM, John Kendall wrote: Thanks, Eric. My only, admittedly weak, rebuttal is that the behavior of sort might not be the best behavior to imitate. It's understandable why POSIX defines how sort

bug#19240: cut 8.22 adds newline

2014-12-01 Thread Pádraig Brady
On 01/12/14 23:06, Eric Blake wrote: On 12/01/2014 03:06 PM, Pádraig Brady wrote: BTW the argument that it's not a text file is a bit beside the point as POSIX also says text files can't contain NUL chars, but we process this just fine: $ printf 'a\000b' | cut -c3 b The fact

bug#19228: Challenging output from help2man split

2014-11-30 Thread Pádraig Brady
On 30/11/14 17:06, Kevin O'Gorman wrote: This is sent to the bug addresses for both help2man and split because I'm not sure where the fault lies. Both are GNU projects, so I hope you can cooperate and figure it out. In any event, the output of help2man split loses some line breaks and

bug#18499: Possible mv race for hardlinks (rhbz #1141368 )

2014-11-29 Thread Pádraig Brady
On 21/11/14 03:30, Pádraig Brady wrote: We want to leave the logic in place for cp and install though, and I've adjusted your patch accordingly. I've also adjusted the tests to pass and augmented the tests to cover one of the cases missed in the previous patch. I'll push this tomorrow

bug#19218: Inconsistent spacing of output of ls --full-time [file argument]

2014-11-29 Thread Pádraig Brady
tag 19218 notabug close 19218 stop On 29/11/14 21:14, Paul Eggert wrote: I don't see a bug in the cases you mention. First, 'ls' dynamically adjusts column widths to fit the data, and this is considered to be a feature. Right. This is a limitation of cut, rather than anything wrong with ls.

bug#19184: coreutils-8.23 Compile Error on Solaris 8 - Sun Studio 11

2014-11-27 Thread Pádraig Brady
On 27/11/14 02:35, Paul Eggert wrote: Pádraig Brady wrote: I did that in the attached. I also see uses of %z in dd.c, od.c, and split.c No other uses with fprintf() though. The od one goes through printf, but that's only debugging. The error() ones are fine as they go through any printf

bug#19184: coreutils-8.23 Compile Error on Solaris 8 - Sun Studio 11

2014-11-26 Thread Pádraig Brady
On 26/11/14 19:21, Paul Eggert wrote: On 11/26/2014 06:19 AM, Ted Carr wrote: I get some errors when running 'make check'. You have any time to help with those or should I just send in another email? Can't hurt to email them to bug-coreutils@gnu.org. Can't promise anything: Solaris 8

bug#19184: coreutils-8.23 Compile Error on Solaris 8 - Sun Studio 11

2014-11-26 Thread Pádraig Brady
On 26/11/14 20:37, Pádraig Brady wrote: The other issue is incorrect prompts output by rm due to its use of %zu Ted could you give the output of: grep gl_cv_func_printf_sizes_c99 config.log Now coreutils doesn't make use of the fprintf-posix module to act on the above test. Jim

bug#19154: [PATCH] Extend file size support in paste.

2014-11-23 Thread Pádraig Brady
On 23/11/14 18:50, Tobias Stoeckmann wrote: The function paste_parallel just has to remember if there was a character in a line at all, not how many. Changing size_t to a simple boolean statement removes a possible overflow situation with 4 GB files on 32 bit systems. --- src/paste.c | 6

bug#18499: Possible mv race for hardlinks (rhbz #1141368 )

2014-11-21 Thread Pádraig Brady
On 21/11/14 08:29, Boris Ranto wrote: On Fri, 2014-11-21 at 03:30 +, Pádraig Brady wrote: We want to leave the logic in place for cp and install though, and I've adjusted your patch accordingly. I've also adjusted the tests to pass and augmented the tests to cover one of the cases missed

bug#18827: [Feature request] no CR for yes

2014-11-21 Thread Pádraig Brady
tag 18827 wontfix close 18827 stop On 25/10/14 18:59, Pádraig Brady wrote: On 10/25/2014 03:53 PM, George Shuklin wrote: Yes is very nice to generate large repeating patterns, but it always adds \n at the end. It's OK for the string data but sometimes mess with binary. Any way to disable

bug#18499: Possible mv race for hardlinks (rhbz #1141368 )

2014-11-21 Thread Pádraig Brady
On 21/11/14 11:53, Pádraig Brady wrote: On 21/11/14 08:29, Boris Ranto wrote: On Fri, 2014-11-21 at 03:30 +, Pádraig Brady wrote: We want to leave the logic in place for cp and install though, and I've adjusted your patch accordingly. I've also adjusted the tests to pass and augmented

bug#19148: ls --inode --sort=inode

2014-11-21 Thread Pádraig Brady
On 21/11/14 23:57, 積丹尼 Dan Jacobson wrote: $ man ls --sort=WORD sort by WORD instead of name: none (-U), size (-S), time (-t), version (-v), extension (-X) Perhaps add new functionality: inode (-i) Yes maybe, especially when combined with -R. Do you

bug#18499: Possible mv race for hardlinks (rhbz #1141368 )

2014-11-21 Thread Pádraig Brady
I just realised that this change avoids a more problematic issue. The loss of files on case insensitive file systems supporting hardlinks. Consider hfs... $ truncate -s1G hfs.img $ mkfs.hfsplus hfs.img $ mkdir hfs $ sudo mount hfs.img hfs $ sudo su # cd hfs # touch foo # ln foo

bug#18499: Possible mv race for hardlinks (rhbz #1141368 )

2014-11-20 Thread Pádraig Brady
On 18/11/14 19:28, Boris Ranto wrote: On Tue, 2014-11-18 at 16:46 +, Pádraig Brady wrote: On 18/11/14 16:29, Boris Ranto wrote: On Mon, 2014-11-17 at 00:28 +, Pádraig Brady wrote: On 16/11/14 16:35, Paul Eggert wrote: Pádraig Brady wrote: If we change this, it's much more likely

bug#18499: Possible mv race for hardlinks (rhbz #1141368 )

2014-11-18 Thread Pádraig Brady
On 18/11/14 16:29, Boris Ranto wrote: On Mon, 2014-11-17 at 00:28 +, Pádraig Brady wrote: On 16/11/14 16:35, Paul Eggert wrote: Pádraig Brady wrote: If we change this, it's much more likely that people will start complaining about their non overlapping mv instances failing. I'd far

bug#19072: cp does not work with davfs2 (fuse) under GNU/Linux; coreutils-8.21

2014-11-17 Thread Pádraig Brady
On 17/11/14 08:23, Klaus-Jürgen Wolf wrote: Pádraig, wow super, cp --sparse=never works though the file I tried with was in no way sparse. I checked (after umount mount again) the checksums, and it's a real success. The error (before) was absolutely repeatable, it happened always, without

<    6   7   8   9   10   11   12   13   14   15   >