Re: AC_PATH_PROGS_FEATURE_CHECK [was: Bug in gnulib-tools prevents bison from bootstrapping]

[Eric Blake-1]

> The parts related to the above look ok, but
...
> this patch has many peripheral whitespace changes and other cleanups.

Oops.  I'll separate those out and commit separately;
the whitespace changes were introduced by emacs
whitespace-cleanup mode.

> > address@hidden AC_CHECK_PROG (@var{variable}, @var{prog-to-check-for},
> @var{value-if-found}, @ovar{value-if-not-found}, @ovar{path}, 
> @ovar{reject})
> > address@hidden AC_CHECK_PROG (@var{variable}, @var{prog-to-check-for}, @
> > +  @var{value-if-found}, @ovar{value-if-not-found}, @
> > +  @dvar{path, $PATH}, @ovar{reject})
> 
> Why the @\n?  We don't do that elsewhere in the manual.

The texinfo manual documents that inside @def*, @\n
serves as a line-continuation marker and can be used to
make the .texi source file fit 80-column editing; it
has no effect on the final rendering in either info or dvi.
I'm not particularly fond of long lines when they can be
avoided, so I plan to ultimately convert the entire
manual in that manner, as a matter of style (I already
did it for M4's .texinfo source file).  But in the
meantime, I agree with your complaint that it should be
an independent patch.

-- 
Eric Blake

-- 
View this message in context: 
http://www.nabble.com/Re%3A-Bug-in-gnulib-tools-prevents-bison-from-bootstrapping-tf4360008.html#a12510544
Sent from the Gnu - Autoconf - Patches mailing list archive at Nabble.com.