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 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 show a function call

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

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 o

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 introdu

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 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 modern syst

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 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: bootstrap.std wherein I

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: Fwd: [patch #6107] Patch to add transparent decompression

2013-01-27 Thread Reuben Thomas
On 27 January 2013 19:47, Paul Eggert 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 than a no! >

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 d

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

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 Date: 22 August 201

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 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 :( > Addressing them are

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

2013-01-27 Thread Gary V. Vaughan
On 27 Jan 2013, at 03:27, Bruce Korb wrote: > Hi Gary, Hey Bruce! > Speaking of bootstrap wishes, how about a hack that captures the > "You may need to include this" list? As the various modules > change dependencies and add new headers, I'd as soon glue the > "you may need to add" cruft to the