Re: binutils info files

["Valeriy E. Ushakov" <uwe%stderr.spb.ru@localhost>]

On Wed, Sep 23, 2009 at 17:34:20 +0100, Patrick Welche wrote:

> On Wed, Sep 23, 2009 at 04:21:59PM +0400, Valeriy E. Ushakov wrote:
> > On Wed, Sep 23, 2009 at 09:48:28 +0100, Patrick Welche wrote:
> > 
> > > I seem to need the enclosed patch to build gpl3/binutils on i386 - am I
> > > missing something?
> > 
> > Part of the mail where you describe what is wrong and why do you think
> > this patch is needed?
> 
> Fair enough:
> 
> #    create  gas/as.info
> /usr/src/obj.i386/tooldir.NetBSD-5.99.18-i386/bin/nbmakeinfo 
> -I/usr/src/external
> /gpl3/binutils/dist/gas/doc 
> -I/usr/src/external/gpl3/binutils/usr.bin/gas/obj.i3
> 86/../common -I/usr/src/external/gpl3/binutils/dist/libiberty --no-split 
> --no-ve
> rsion-header -o as.info 
> /usr/src/external/gpl3/binutils/dist/gas/doc/as.texinfo
> /usr/src/external/gpl3/binutils/dist/gas/doc/as.texinfo:25: @include 
> `bfdver.tex
> i': No such file or directory.
> /usr/src/external/gpl3/binutils/dist/gas/doc/as.texinfo:173: warning: 
> undefined 
> flag: VERSION.
> 
> after rm -rf /usr/obj/*; sh build.sh -E build
> and bfdver.texi lives in ${DIST}/bfd/doc

Hmm, I see.  But that's only part of the story, it seems.

In the as.info I see 

    This file is a user guide to the GNU assembler `as' version .

which looks broken, but I don't see any warning in my most recent
build logs.  Ktracing the makeinfo command shows that it actually
picks up

  ${OBJDIR}/external/gpl3/binutils/usr.bin/gas/../common/bfdver.texi"

That consists of single line with

  @set VERSION

(no value) and is created by:

dependall ===> external/gpl3/binutils/usr.bin/common
#    create  common/bfdver.texi
rm -f config.texi
(echo "@set VERSION ";  if [ -n "" ]; then  echo "@set VERSION_PACKAGE ";  fi) 
> bfdver.texi

So 1) I guess that there's a -j race that you see that causes the
warning; 2) even for non-parallel builds bfdver.texi in the objdir is
generated incorrectly.

-uwe