Re: [PATCH] bootstrap: remove the need for a sorted .gitignore

2013-01-27 Thread Gary V. Vaughan
Hi Padraig, On 26 Jan 2013, at 21:39, Pádraig Brady p...@draigbrady.com wrote: Thanks for the review and tips Gary. Welcome, and likewise for yours :) While I noticed some of those I didn't fix because those styles/issues were already used (elsewhere) in bootstrap. Unfortunately so :(

Fwd: [patch #6107] Patch to add transparent decompression

2013-01-27 Thread Reuben Thomas
As per the following, the (currently non-existent) ig flavor of pipe-filter would enable a simple implementation of a useful grep patch. Perhaps it would be worth recording this fact in pipe-filter.h as a motivator? -- Forwarded message -- From: Reuben Thomas

Re: Build errors on Solaris 2.6 7

2013-01-27 Thread Paul Eggert
On 01/12/2013 09:51 PM, Paul Eggert wrote: + * modules/futimens-tests (test_futimens_LDADD): + * modules/utimens-tests (test_utimens_LDADD): Add @LIBINTL@. This introduced a problem when running these tests individually, which I fixed as follows: futimens-tests, utimens-tests: Depend

Re: Fwd: [patch #6107] Patch to add transparent decompression

2013-01-27 Thread Paul Eggert
On 01/27/2013 10:13 AM, Reuben Thomas wrote: Perhaps it would be worth recording this fact in pipe-filter.h as a motivator? I'm not sure, since I don't recall the issue. Perhaps if you proposed a patch -- not too wordy I hope -- that would clarify things.

Re: [PATCH] bootstrap: remove the need for a sorted .gitignore

2013-01-27 Thread Jim Meyering
Stefano Lattarini wrote: Throwing in my two cents here ... ... Thanks. $(...) is not supported by even some modern systems, including solaris 10. Actually, only by /bin/sh. Solaris 10 has a working Korn shell in /bin/ksh and a working POSIX shell in /usr/xpg4/bin/sh. Requiring a

Re: Fwd: [patch #6107] Patch to add transparent decompression

2013-01-27 Thread Reuben Thomas
On 27 January 2013 19:47, Paul Eggert egg...@cs.ucla.edu wrote: On 01/27/2013 10:13 AM, Reuben Thomas wrote: Perhaps it would be worth recording this fact in pipe-filter.h as a motivator? I'm not sure, since I don't recall the issue. If you don't recall it, that sounds more like a yes

Re: Build errors on Solaris 2.6 7

2013-01-27 Thread Paul Eggert
On 01/22/2013 11:40 AM, Tom G. Christensen wrote: It makes it worse, now it fails for Solaris 10. OK, let's try a different tack. If memory serves Bruno Haible said a while ago that we'd eventually have to do this, so let's give it a try. Attached is a patch against the current gnulib -- can

Re: You may need to add #include directives for the following .h files

2013-01-27 Thread Bruce Korb
On 01/27/13 05:35, Gary V. Vaughan wrote: On 27 Jan 2013, at 03:27, Bruce Korb bruce.k...@gmail.com wrote: Hi Gary, Hey Bruce! Speaking of bootstrap wishes, Hi Gary, Follow up: worked like a charm! Thank you. I use *two* bootstrap.conf files: one by that name and another:

Re: [PATCH] bootstrap: remove the need for a sorted .gitignore

2013-01-27 Thread Gary V. Vaughan
Hi Jim, Stefano, Thanks for your comments. On 28 Jan 2013, at 03:07, Jim Meyering j...@meyering.net wrote: Stefano Lattarini wrote: Throwing in my two cents here ... ... Stefano, I didn't get a copy of your message for some reason, only Jim's reply :-o $(...) is not supported by even some

Re: bug#13516: tests/rm/unread3 fails on Mac OS X 10.8

2013-01-27 Thread Paul Eggert
On 01/24/2013 12:35 PM, Assaf Gordon wrote: With this patch, the recursive loop is gone, but it leads to another bug: descending into the directory (due to -r) doesn't work. Thanks, can you please try this patch instead? It's a bit more drastic, but I hope it fixes the loop without

Re: [PATCH] bootstrap: remove the need for a sorted .gitignore

2013-01-27 Thread Jim Meyering
Gary V. Vaughan wrote: ... so it would be safer to probe for and use a backslash safe echo here Much better would be to just use printf. Safer, simpler and more portable! Yes. For uses like this, echo has been deprecated for well over a decade. And here's another one :) What users (if

Re: [PATCH] bootstrap: remove the need for a sorted .gitignore

2013-01-27 Thread Gary V. Vaughan
Hi Jim, On 28 Jan 2013, at 09:52, Jim Meyering j...@meyering.net wrote: Gary V. Vaughan wrote: ... so it would be safer to probe for and use a backslash safe echo here Much better would be to just use printf. Safer, simpler and more portable! Yes. For uses like this, echo has been

Re: You may need to add #include directives for the following .h files

2013-01-27 Thread Gary V. Vaughan
Hi Bruce, On 28 Jan 2013, at 08:32, Bruce Korb bruce.k...@gmail.com wrote: Also, I don't know how you use $debug_cmd, It's all in the comments :) # Allow overriding, eg assuming that you follow the convention of # putting '$debug_cmd' at the start of all your functions, you can get # bash to