gnulib problem with mktime & ctime interaction

2009-07-26 Thread Ross McFarland
when using gnulib's mktime test any cpp file that #includes ctime will receive the following errors: In file included from time.cc:6: /usr/include/c++/4.3/ctime:73: error: ‘::mktime’ has not been declared time.cc: In function ‘int main()’: time.cc:11: error: ‘mktime’ was not declared in this scope

Re: [PATCH] fts: avoid false-positive cycle-detection

2009-07-26 Thread James Youngman
On Sun, Jul 19, 2009 at 9:35 PM, Jim Meyering wrote: > I'm considering a rewrite of rm that uses fts.c, > and see that the resulting rm fails coreutils' rm/cycle test > because it incorrectly reports a cycle for that unusual case. > > The following patch fixes fts so that it properly reinitializes

Re: [PATCH] My version of pipe-filter

2009-07-26 Thread Bruno Haible
Hi Paolo, > Bruno, what do you think? This implementation is one out of 4 possible ones. Thanks for having written it; I will complete the two remaining ones. > +extern struct filter * filter_create (const char *progname, > + const char *prog_path, > +

Re: overly zealous git hook

2009-07-26 Thread Bruno Haible
Hi Jim, > >> Since I prefer that the hook continue to apply to all files > >> for which I have a say, perhaps you'd like to itemize the files > >> for which you want to allow trailing blank lines? > > > > Yes. These should be: modules/**/* and **/ChangeLog*. I believe it > > My point was that I'd