config/debuginfod: do not include pkg.m4 directly
authorMike Frysinger <vapier@gentoo.org>
Sun, 7 Feb 2021 00:24:29 +0000 (19:24 -0500)
committerMike Frysinger <vapier@gentoo.org>
Sat, 13 Feb 2021 05:15:11 +0000 (00:15 -0500)
Any code using AC_DEBUGINFOD from this dir is using -I../config when
running aclocal, so an explicit include on pkg.m4 is unnecessary:
aclocal will find the pkg.m4 in this dir just as easily.  This is
seen in the only two dirs that use AC_DEBUGINFOD (binutils & gdb)
as their aclocal.m4 already has m4_include on config m4 files.

The include as written only works if aclocal is run on a dir that is
at the same level of config/.  Any other depth will fail.
./
|-- config/
|-- binutils/    # works
|-- gdb/         # works
`-- sim/         # works
    `-- <port>/  # fails

It fails even if AC_DEBUGINFOD itself isn't used:
sim/bfin/ $ aclocal -I../../config
aclocal-1.15: error: ../../config/debuginfod.m4:8: file '../config/pkg.m4' does not exist

binutils/ChangeLog
binutils/aclocal.m4
config/ChangeLog
config/debuginfod.m4
gdb/ChangeLog
gdb/aclocal.m4

index a717099426534ebb887bae0cefa58b449570ebbb..5b16bb2fc117d3c5941458cf30560fcdc84e955c 100644 (file)
@@ -1,3 +1,7 @@
+2021-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * aclocal.m4: Regenerate.
+
 2021-02-12  Nick Clifton  <nickc@redhat.com>
 
        * configure.ac (follow-debug-links): Add option to enable or
index 8eca8cf94c73b4c1f160fed5b1f5b01c9376a5eb..e6b490ac4c6c727c5d397fdb107dab0cccb9429b 100644 (file)
@@ -1201,6 +1201,7 @@ m4_include([../config/lib-link.m4])
 m4_include([../config/lib-prefix.m4])
 m4_include([../config/nls.m4])
 m4_include([../config/override.m4])
+m4_include([../config/pkg.m4])
 m4_include([../config/plugins.m4])
 m4_include([../config/po.m4])
 m4_include([../config/progtest.m4])
index 847fc0656fff5667eaab46ac35a11b34427985eb..95dcc85b42b91e086d7cc380219f784dfb3b5e16 100644 (file)
@@ -1,3 +1,7 @@
+2021-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * debuginfod.m4: Delete m4_include(../config/pkg.m4).
+
 2021-01-12  H.J. Lu  <hongjiu.lu@intel.com>
 
        PR binutils/26792
@@ -2293,4 +2297,3 @@ Thu Jan 30 16:17:30 1992  Stu Grossman  (grossman at cygnus.com)
 Tue Dec 10 00:10:55 1991  K. Richard Pixley  (rich at rtl.cygnus.com)
 
        * ChangeLog: fresh changelog.
-
index f9eb0988e3547411968cbfa627a655e49d5ffcd4..2c1bfbdb544bc9f34c76b2349912707a6c5ee159 100644 (file)
@@ -5,8 +5,6 @@ dnl Public License, this file may be distributed as part of a program
 dnl that contains a configuration script generated by Autoconf, under
 dnl the same distribution terms as the rest of that program.
 
-m4_include([../config/pkg.m4])
-
 AC_DEFUN([AC_DEBUGINFOD],
 [
 # Handle optional debuginfod support
index e79b9535f707833e97f5dc537f568743e6f156f9..204bb40b2c9b313cee56efea68aa3ad983bbbeff 100644 (file)
@@ -1,3 +1,7 @@
+2021-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * aclocal.m4: Regenerate.
+
 2021-02-12  Tom de Vries  <tdevries@suse.de>
 
        PR threads/26228
index 110b416e61561930fa4e4a01eb23873cc6339d85..cbea739c0e51662a4ef6249e7db69428de40dea0 100644 (file)
@@ -199,4 +199,5 @@ AC_DEFUN([_AM_SUBST_NOTMAKE])
 # Public sister of _AM_SUBST_NOTMAKE.
 AC_DEFUN([AM_SUBST_NOTMAKE], [_AM_SUBST_NOTMAKE($@)])
 
+m4_include([../config/pkg.m4])
 m4_include([acinclude.m4])