Re: [PATCH] parse-duration: Lessen the GPL

2012-06-10 Thread Bruce Korb
On 06/10/12 16:19, Bruno Haible wrote: Hi Bruce, Paul, Jim, Bruno, and Ralf as the set of all contributors besides yourself (although some of those contributions were trivial) before making this change. They were all trivial, actually. Well, I think 3-line comments as in the 2008-12-17 chan

Re: gnulib portability issues

2012-06-10 Thread Rich Felker
On Sun, Jun 10, 2012 at 07:02:32PM -0700, Paul Eggert wrote: > On 06/10/2012 06:10 AM, Rich Felker wrote: > > As such, the first fix I would recommend is simply removing the check > > for freadahead(stdin)>0 from closein.c > > But that will hurt performance on GNU/Linux and most other > POSIXish h

Re: gnulib portability issues

2012-06-10 Thread Paul Eggert
On 06/10/2012 05:43 AM, Rich Felker wrote: > If the latter is acceptable though, the stdio_ext.h function > __freading might suffice as an implementation. On musl doesn't __freading actually return a count? Perhaps we could just use that.

Re: gnulib portability issues

2012-06-10 Thread Paul Eggert
On 06/10/2012 06:10 AM, Rich Felker wrote: > As such, the first fix I would recommend is simply removing the check > for freadahead(stdin)>0 from closein.c But that will hurt performance on GNU/Linux and most other POSIXish hosts, as it will cause them to issue lseek system calls and whatnot that

Re: Why require SLOW_BUT_NO_HACKS for stubs?

2012-06-10 Thread Paul Eggert
On 06/09/2012 11:05 PM, Isaac Dunham wrote: > Is there any reason not to merge Performance, surely. But if there's consensus that performance does not matter that much with musl, perhaps we should default to the slow version with musl. Is there any simple way to tell at compile-time, or at confi

Re: [PATCH] maint.mk: prohibit common grammar error: "all these"

2012-06-10 Thread Paul Eggert
On 06/10/2012 12:56 PM, Jim Meyering wrote: > Can anyone think of a common way to use "all these" that is > not in error? Yes, lots: Tired with all these, for restful death I cry -- Shakespeare, Sonnet 66, line 1 But Mary kept all these things, and pondered them in their heart. --

Re: [PATCH] install-reloc support for installing multiple programs

2012-06-10 Thread Bruno Haible
Sylvain wrote: > This is a request to commit to patched version to the gnulib > repository. Thanks for the reminder. I am in the process of writing a unit test for it, and want to commit the unit test no later than the big change. Bruno

Re: gnupload --replace

2012-06-10 Thread Bruno Haible
Karl Berry wrote: > diff --git a/build-aux/ylwrap b/build-aux/ylwrap > old mode 100644 > new mode 100755 This should be a separate commit. Cf. also http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=commitdiff;h=b8c5e24996c9626f03890cd986fc1fb7b17fa610

Re: [PATCH] parse-duration: Lessen the GPL

2012-06-10 Thread Bruno Haible
Hi Bruce, > > Paul, Jim, Bruno, and Ralf as the set of all contributors > > besides yourself (although some of those contributions were trivial) > > before making this change. > > They were all trivial, actually. Well, I think 3-line comments as in the 2008-12-17 change [1] are copyrightable, be

Re: GNU sed version 4.2.1: on OS X, C locale gets aliased to UTF-8

2012-06-10 Thread Max Horn
Hi again, Am 07.06.2012 um 14:07 schrieb Bruno Haible: [...] > >> But this is dangerous, because now UTF-8 is set but MB_CUR_MAX is 1 >> and various parts of sed interpret "RĂ©mi Leblond" as an invalid >> character sequence for a UTF-8 character set. > > Indeed, I can see how this inconsistenc

[PATCH] maint.mk: prohibit common grammar error: "all these"

2012-06-10 Thread Jim Meyering
FYI, I've pushed the change below. Can anyone think of a common way to use "all these" that is not in error? So far, all of the uses I've seen are wrong, and instead should be written "all of these". See below for a list of the violations in gnulib: I won't go changing FDL text (obviously), but

Re: gnulib portability issues

2012-06-10 Thread Rich Felker
On Sat, Jun 09, 2012 at 10:01:38PM -0700, Paul Eggert wrote: > > it might be better to make sense of WHY it's needed/wanted > > Yes, quite true, but that information wasn't communicated to the > gnulib mailing list. For GNU m4, it seems freadahead is not used directly. Instead, it's pulled in by

Re: gnulib portability issues

2012-06-10 Thread Rich Felker
On Sat, Jun 09, 2012 at 10:01:38PM -0700, Paul Eggert wrote: > On 06/09/2012 09:25 PM, Rich Felker wrote: > > For seekable input, comparing ftello(f) and the underlying fd offset > > from lseek(fileno(f),0,SEEK_CUR) would work, but that also will fail > > for non-seekable input. > > Thanks, that's

Why require SLOW_BUT_NO_HACKS for stubs?

2012-06-10 Thread Isaac Dunham
Hello, I'm using musl as a libc, and have run into a number of times that gnulib stopped build. By defining SLOW_BUT_NO_HACKS, the software ended up working. This is the documented behavior, but it doesn't seem like the right one: if a stub is usable enough to allow using it, why shouldn't it be av

[PATCH] install-reloc support for installing multiple programs

2012-06-10 Thread Sylvain
Hi, We discussed at : http://lists.gnu.org/archive/html/bug-gnulib/2012-05/msg00027.html that a fix is needed in 'install-reloc'. Bruno Haible provided a patched version, which was successfully tested. This is a request to commit to patched version to the gnulib repository. Cheers! Sylvain >Fro