[binutils-gdb] bfd: fix generation of bfd.texi in out-of-tree builds

Jose E. Marchesi jemarch@sourceware.org
Thu Jan 23 21:06:39 GMT 2025


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=19125d93aa693be8bd541470d65b7ea635b76af4

commit 19125d93aa693be8bd541470d65b7ea635b76af4
Author: Jose E. Marchesi <jose.marchesi@oracle.com>
Date:   Thu Jan 23 20:16:09 2025 +0100

    bfd: fix generation of bfd.texi in out-of-tree builds
    
    [In the sequel TS means $(top_srcdir) and TB means $(top_builddir)]
    
    The Texinfo file TS/bfd/doc/bfd.texi @includes many other .texi files
    such as:
    
       bfdt.texi
       bfdio.texi
       section.texi
       ...
    
    These .texi files are generated from the bfd/*.c source files, by a
    program called `chew' that is distributed along with BFD, via some
    default rules and macro magic in TS/bfd/doc/local.mk.  Important
    point: the .texi files are generated in TB/bfd/doc/, not TS/bfd/doc.
    
    Now, AM_MAKEINFOFLAGS in local.mk is defined as:
    
      AM_MAKEINFOFLAGS = --no-split -I "$(srcdir)/%D%" -I %D%
    
    Where %D% is 'doc/' in this case.  Now, it looks like the directory
    containing the .texi file is automatically inserted in the @include
    search path, so the -I %D% above places TB/bfd/doc _after_ TS/bfd/doc.
    
    Since currently TS/bfd/doc/bfdt.texi is outdated and is missing some
    nodes, the error above happens.
    
    This patch changes bfd/doc/local.mk to use -P to prepend the current
    build directory to the @include search path, rather than -I, which
    appends it.
    
    bfd/ChangeLog:
    
    2025-01-23  Jose E. Marchesi  <jose.marchesi@oracle.com>
    
            * doc/local.mk (AM_MAKEINFOFLAGS): Prepend the build directory to
            the @include search path.
            * Makefile.in: Regenerate.

Diff:
---
 bfd/ChangeLog    | 6 ++++++
 bfd/Makefile.in  | 2 +-
 bfd/doc/local.mk | 2 +-
 3 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index da27f8c3f31..d8facc241c9 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2025-01-23  Jose E. Marchesi  <jose.marchesi@oracle.com>
+
+	* doc/local.mk (AM_MAKEINFOFLAGS): Prepend the build directory to
+	the @include search path.
+	* Makefile.in: Regenerate.
+
 2025-01-19  Nick Clifton  <nickc@redhat.com>
 
 	* 2.44 Branch point.
diff --git a/bfd/Makefile.in b/bfd/Makefile.in
index ee674a36c5b..1543af7984a 100644
--- a/bfd/Makefile.in
+++ b/bfd/Makefile.in
@@ -1301,7 +1301,7 @@ SRCIPROT = $(srcdir)/cache.c $(srcdir)/libbfd.c \
 TEXIDIR = $(srcdir)/../texinfo/fsf
 info_TEXINFOS = doc/bfd.texi
 doc_bfd_TEXINFOS = $(DOCFILES) doc/bfdsumm.texi
-AM_MAKEINFOFLAGS = --no-split -I "$(srcdir)/doc" -I doc
+AM_MAKEINFOFLAGS = --no-split -I "$(srcdir)/doc" -P doc
 TEXI2DVI = texi2dvi -I "$(srcdir)/doc" -I doc
 MKDOC = doc/chew$(EXEEXT_FOR_BUILD)
 
diff --git a/bfd/doc/local.mk b/bfd/doc/local.mk
index 9b75402387c..515efbf72c9 100644
--- a/bfd/doc/local.mk
+++ b/bfd/doc/local.mk
@@ -72,7 +72,7 @@ TEXIDIR = $(srcdir)/../texinfo/fsf
 info_TEXINFOS = %D%/bfd.texi
 %C%_bfd_TEXINFOS = $(DOCFILES) %D%/bfdsumm.texi
 
-AM_MAKEINFOFLAGS = --no-split -I "$(srcdir)/%D%" -I %D%
+AM_MAKEINFOFLAGS = --no-split -I "$(srcdir)/%D%" -P %D%
 TEXI2DVI = texi2dvi -I "$(srcdir)/%D%" -I %D%
 
 MKDOC = %D%/chew$(EXEEXT_FOR_BUILD)


More information about the Binutils-cvs mailing list