autotools: do not overwrite first include path
authorMichael Walle <michael@walle.cc>
Tue, 4 Feb 2020 12:40:12 +0000 (13:40 +0100)
committerYann E. MORIN <yann.morin.1998@free.fr>
Tue, 4 Feb 2020 13:02:38 +0000 (14:02 +0100)
commitd255b67972b4b7f27572581fe0c8c8aa03d850c8
treea9e8a15d962895814e5e45d210c5a5290344bc70
parent44b2fc55fa3cc161fd48327bcb51b7902d4b9a82
autotools: do not overwrite first include path

The first include path is special in aclocal. For example it is the path
for the --install option. Also, the first include is treated in a
special way if it doesn't exists. This might be the case if there is the
following construct:

  configure.ac: AC_CONFIG_MACRO_DIR([m4])
  Makefile.am: ACLOCAL_AMFLAGS="-I m4"

If the package doesn't have local macros, the m4/ directory might not
exist. aclocal will then just issue a warning instead of aborting the
execution with a fatal error. See discussion here:
  https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=565663

Don't use the "-I" option in aclocal. Instead use ACLOCAL_PATH to pass
the system-wide include dirs.

As a side effect this should fix the use of $(ACLOCAL) alone. Up until
now, $(ACLOCAL) didn't include the ACLOCAL_HOST_DIR system include path.

autoreconf will pass the "-I" options to every tool it runs, of which
aclocal, which, as seen above, we don't want. So move the argument down
to each individual tool, except for aclocal.

Signed-off-by: Michael Walle <michael@walle.cc>
Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
[yann.morin.1998@free.fr: slight rewording of the commit log]
Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
package/autoconf/autoconf.mk
package/automake/automake.mk