This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix binutils html target.


On Fri, Feb 17, 2006 at 01:24:54PM +0000, Joseph S. Myers wrote:
> On Fri, 17 Feb 2006, Nick Clifton wrote:
> 
> > Hi Carlos,
> > 
> > > These patches enable building the binutils html target.
> > 
> > These seem to be OK, apart from one thing:
> > 
> > -TEXI2HTML = texi2html
> > +TEXI2HTML = `if [ -f ../texinfo/makeinfo/makeinfo ]; \
> > +	then echo "../texinfo/makeinfo/makeinfo --html"; \
> > +	else echo "makeinfo --html"; fi`
> > +
> > 
> > Why is the use of texi2html being abandoned ?  If the makeinfo program is not
> > installed or present in the build tree, shouldn't the texi2html program be
> > used instead ?
> 
> These programs are not compatible in their command-line options.  In 
> general texi2html cannot be relied upon to support the required Texinfo 
> language features, whereas a makeinfo version check ensures that it does 
> support the required features (and the in-tree copy of texinfo.tex should 
> be of a suitable version to support them - a more recent version might 
> need to be imported - and be used by -I options on all texi2dvi uses).  
> The GNU Coding Standards suggest "makeinfo --no-split --html" as the 
> proper TEXI2HTML definition.

I would respond but Joseph has done a great job.

I think it is reasonable to require makeinfo when generating html targets.

Cheers,
Carlos.
-- 
Carlos O'Donell
CodeSourcery
carlos@codesourcery.com
(650) 331-3385 x716


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]