Re: [PATCH] avoid warnings about initialization of automatic aggregates

2008-12-01 Thread Jim Meyering
Pádraig Brady <[EMAIL PROTECTED]> wrote: > Jim Meyering wrote: >> Finally getting serious about avoiding warnings, >> I've just eliminated this batch: >> >>>From c58b5daa337b16416be50adfeb3e99e3c009c891 Mon Sep 17 00:00:00 2001 >> From: Jim Meyering <[EMAIL PROTECTED]> >> Date: Sun, 30 Nov 2008 22:

Re: fix du and wc wrt --files0-from=F

2008-12-01 Thread Jim Meyering
Pádraig Brady <[EMAIL PROTECTED]> wrote: > Jim Meyering wrote: >>>From 622f12ed8fd799ad83546233a60655a6a0d2a4b6 Mon Sep 17 00:00:00 2001 >> From: Jim Meyering <[EMAIL PROTECTED]> >> Date: Tue, 25 Nov 2008 18:38:26 +0100 >> Subject: [PATCH 2/2] wc: read and process --files0-from= input a name at a

Re: [PATCH] seq: plug a leak

2008-12-01 Thread Jim Meyering
Pádraig Brady <[EMAIL PROTECTED]> wrote: > Jim Meyering wrote: >> I've just pushed this: >> >>>From 9461c4d8e58f080a6b27caa923820a4ac8eea6d5 Mon Sep 17 00:00:00 2001 >> From: Jim Meyering <[EMAIL PROTECTED]> >> Date: Mon, 1 Dec 2008 22:39:58 +0100 >> Subject: [PATCH] seq: plug a leak >> >> * src/fa

Re: [PATCH] avoid warnings about initialization of automatic aggregates

2008-12-01 Thread Pádraig Brady
Jim Meyering wrote: > Finally getting serious about avoiding warnings, > I've just eliminated this batch: > >>From c58b5daa337b16416be50adfeb3e99e3c009c891 Mon Sep 17 00:00:00 2001 > From: Jim Meyering <[EMAIL PROTECTED]> > Date: Sun, 30 Nov 2008 22:37:42 +0100 > Subject: [PATCH] avoid warnings ab

Re: fix du and wc wrt --files0-from=F

2008-12-01 Thread Pádraig Brady
Jim Meyering wrote: > >>From 622f12ed8fd799ad83546233a60655a6a0d2a4b6 Mon Sep 17 00:00:00 2001 > From: Jim Meyering <[EMAIL PROTECTED]> > Date: Tue, 25 Nov 2008 18:38:26 +0100 > Subject: [PATCH 2/2] wc: read and process --files0-from= input a name at a > time, > > when the file name list is not

Re: [PATCH] seq: plug a leak

2008-12-01 Thread Pádraig Brady
Jim Meyering wrote: > I've just pushed this: > >>From 9461c4d8e58f080a6b27caa923820a4ac8eea6d5 Mon Sep 17 00:00:00 2001 > From: Jim Meyering <[EMAIL PROTECTED]> > Date: Mon, 1 Dec 2008 22:39:58 +0100 > Subject: [PATCH] seq: plug a leak > > * src/factor.c (emit_ul_factor): Call mpz_clear. I presu

[PATCH] seq: plug a leak

2008-12-01 Thread Jim Meyering
I've just pushed this: >From 9461c4d8e58f080a6b27caa923820a4ac8eea6d5 Mon Sep 17 00:00:00 2001 From: Jim Meyering <[EMAIL PROTECTED]> Date: Mon, 1 Dec 2008 22:39:58 +0100 Subject: [PATCH] seq: plug a leak * src/factor.c (emit_ul_factor): Call mpz_clear. --- src/factor.c |1 + 1 files changed

Re: [bug #24974] Document that comm's option "-1", "-2" and "-3" can be combined

2008-12-01 Thread Frank Lin PIAT
On Mon, 2008-12-01 at 22:00 +0100, Andreas Schwab wrote: > Pádraig Brady <[EMAIL PROTECTED]> writes: > > > # set operations on sorted files > > > > comm file1 file2 | sed 's/^\t*//' #Union > > That only works when no data line begins with a TAB. sort -m does not > have this problem. Actuall

Re: [bug #24974] Document that comm's option "-1", "-2" and "-3" can be combined

2008-12-01 Thread Andreas Schwab
Pádraig Brady <[EMAIL PROTECTED]> writes: > # set operations on sorted files > > comm file1 file2 | sed 's/^\t*//' #Union That only works when no data line begins with a TAB. sort -m does not have this problem. > # set operations on unsorted files > > sort file1 file2 | uniq #

[PATCH] avoid warnings about initialization of automatic aggregates

2008-12-01 Thread Jim Meyering
Finally getting serious about avoiding warnings, I've just eliminated this batch: >From c58b5daa337b16416be50adfeb3e99e3c009c891 Mon Sep 17 00:00:00 2001 From: Jim Meyering <[EMAIL PROTECTED]> Date: Sun, 30 Nov 2008 22:37:42 +0100 Subject: [PATCH] avoid warnings about initialization of automatic a

Re: [bug #24974] Document that comm's option "-1", "-2" and "-3" can be combined

2008-12-01 Thread Frank Lin PIAT
On Mon, 2008-12-01 at 00:25 +, Pádraig Brady wrote: > anonymous wrote: > > URL: > > > > > > Hello, > > > > Could you explicitly document that "-1", "-2" and "-3" options > > of "comm" can be combined. > > This isn't obvious, and may lead some people [me

Re: [bug #24974] Document that comm's option "-1", "-2" and "-3" can be combined

2008-12-01 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Pádraig Brady on 12/1/2008 10:15 AM: > Bo Borgerson wrote: >> Pádraig Brady wrote: >>> p.s. Those new --check-order --nocheck-order options confuse me. >>> When they were added I only took a quick look at the implementation >>> rather than

fix du and wc wrt --files0-from=F

2008-12-01 Thread Jim Meyering
Here are the two patches I expect to push: [as I write this I realize that while there was already very good test coverage of most of the modified code, I should write a test that exercises the bug in question, at least for du: read a large list from a file and ensure that it no longer fails in a m

Re: [bug #24974] Document that comm's option "-1", "-2" and "-3" can be combined

2008-12-01 Thread Pádraig Brady
Bo Borgerson wrote: > Pádraig Brady wrote: >> p.s. Those new --check-order --nocheck-order options confuse me. >> When they were added I only took a quick look at the implementation >> rather than the interface (which Bo Borgerson kindly sped up for us). >> Perhaps something like this would be clea

avoid a warning in hostname.c

2008-12-01 Thread Jim Meyering
I also pushed another little change: http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=21eb87e6eafee9111ac2da32cfbfb9ee34a4b465 commit 21eb87e6eafee9111ac2da32cfbfb9ee34a4b465 Author: Jim Meyering <[EMAIL PROTECTED]> Date: Mon Dec 1 07:27:55 2008 +0100 * src/hostname.c (se

FYI: argv-iter: new module

2008-12-01 Thread Jim Meyering
I've just pushed the argv-iter module: (with a few tweaks since the initial discussion), http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=ca738e4414 along with a test module: http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=d3bf10 and code to make cor

Re: [bug #24974] Document that comm's option "-1", "-2" and "-3" can be combined

2008-12-01 Thread Bo Borgerson
Pádraig Brady wrote: > > p.s. Those new --check-order --nocheck-order options confuse me. > When they were added I only took a quick look at the implementation > rather than the interface (which Bo Borgerson kindly sped up for us). > Perhaps something like this would be clearer: > > --check-ord

[bug #1212] wishlist: ls sort case insensitive option

2008-12-01 Thread alez
Follow-up Comment #8, bug #1212 (project coreutils): http://sexintheshower.sensualwriter.com/ sex in the shower http://sexintheshower.sensualwriter.com/2008/12/01/teen-girl-fingering-and-masturbating-in-the-shower-video/ Teen girl fingering and masturbating in the shower video _

Re: du --files-from feature request

2008-12-01 Thread Pádraig Brady
Pádraig Brady wrote: > Jim Meyering wrote: >> No problem. >> BTW, I've already fixed it, but forgot to mention it in the log text. > > Noticed that, thanks. > > On a related note, would the attached patch be useful, > as I found inconsistent descriptions of ASCII NUL > throughout the info doc. O

Failure to compile latest coreutils-6.12

2008-12-01 Thread EA Games
Hello guys, I am using 2.6.21.5 kernel (gcc 4.3.2, ld 2.17.50.0.17*). I've tried compiling the latest coreutils that I downloaded from the official website and I keep getting the same error message. mv unistd.h-t unistd.h make all-am make[3]: Entering directory `/mnt/lfs/tmp/coreutils-6.12/lib'

Re: [bug #24949] coreutils pwd not implementing latest POSIX features

2008-12-01 Thread Jim Meyering
Paul Smith <[EMAIL PROTECTED]> wrote: > On Mon, 2008-12-01 at 10:33 +, James Youngman wrote: >> If you can rely enough on the platform being POSIX-conforming for -P >> to work, then why not just use Perl's POSIX module? > > On some embedded systems where the rootfs is in a ramdisk I use, for >

Re: [bug #24949] coreutils pwd not implementing latest POSIX features

2008-12-01 Thread Paul Smith
On Mon, 2008-12-01 at 10:33 +, James Youngman wrote: > If you can rely enough on the platform being POSIX-conforming for -P > to work, then why not just use Perl's POSIX module? On some embedded systems where the rootfs is in a ramdisk I use, for example, microperl. This is an amazingly flexi

Re: du --files-from feature request

2008-12-01 Thread Pádraig Brady
Jim Meyering wrote: > > No problem. > BTW, I've already fixed it, but forgot to mention it in the log text. Noticed that, thanks. On a related note, would the attached patch be useful, as I found inconsistent descriptions of ASCII NUL throughout the info doc. cheers, Pádraig. >From 3557a0e286fe

Re: [bug #24949] coreutils pwd not implementing latest POSIX features

2008-12-01 Thread James Youngman
On Mon, Dec 1, 2008 at 2:04 AM, Paul D. Smith <[EMAIL PROTECTED]> wrote: > > Follow-up Comment #2, bug #24949 (project coreutils): > > The problem is that without -P I can't invoke pwd from things like Perl > portably. If I use "my $pwd = `pwd`;" and it runs a shell and uses the shell > builtin ve

Re: du --files-from feature request

2008-12-01 Thread Andreas Schwab
Jim Meyering <[EMAIL PROTECTED]> writes: > @@ -2424,11 +2424,11 @@ pr invocation > Optional argument @var{number-separator} is the character appended to > the line number to separate it from the text followed. The default > separator is the TAB character. In a strict sense a TAB is always > -

Re: du --files-from feature request

2008-12-01 Thread Jim Meyering
Pádraig Brady <[EMAIL PROTECTED]> wrote: > Jim Meyering wrote: >> Pádraig Brady <[EMAIL PROTECTED]> wrote: >>> I also did a non user visible change to the info docs while I was >>> at it to standardize on using lower-case @var{file}: >>> >>> http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=co

Re: du --files-from feature request

2008-12-01 Thread Pádraig Brady
Jim Meyering wrote: > Pádraig Brady <[EMAIL PROTECTED]> wrote: >> I also did a non user visible change to the info docs while I was >> at it to standardize on using lower-case @var{file}: >> >> http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=1ce9d86c308d7daa6271a2e87629984b153ee9

Re: du --files-from feature request

2008-12-01 Thread Jim Meyering
Pádraig Brady <[EMAIL PROTECTED]> wrote: > I also did a non user visible change to the info docs while I was > at it to standardize on using lower-case @var{file}: > > http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=1ce9d86c308d7daa6271a2e87629984b153ee9da Thanks! For a general