[Buildroot] [RFC PATCH] autotools: do not overwrite first include path
Arnout Vandecappelle
arnout at mind.be
Tue Jan 14 23:27:36 UTC 2020
On 14/01/2020 11:52, Michael Walle wrote:
> Hi Arnout,
>
> Am 2020-01-14 00:37, schrieb Arnout Vandecappelle:
[snip]
>> That said, I wonder: why don't we simply export ACLOCAL_PATH? I think with
>> that, we don't need the -I options in autoheader and autoconf any more, right?
>
> You mean something like that:
>
> ACLOCAL = $(HOST_DIR)/bin/aclocal -I $(ACLOCAL_DIR)
> ACLOCAL_PATH = $(ACLOCAL_DIR):$(ACLOCAL_HOST_DIR)
>
> AUTORECONF = $(HOST_CONFIGURE_OPTS) ACLOCAL_PATH="$(ACLOCAL_PATH)"
> ACLOCAL="$(ACLOCAL)" AUTOCONF="$(AUTOCONF)" AUTOHEADER="$(AUTOHEADER)"
> AUTOMAKE="$(AUTOMAKE)" AUTOPOINT=/bin/true $(HOST_DIR)/bin/autoreconf -f -i
>
> That won't work if $(ACLOCAL) is used alone. But from what I see that isn't
> used for now. So we could give that a try.
>
> Or rather something like that:
> ACLOCAL = $(HOST_DIR)/bin/aclocal -I $(ACLOCAL_DIR)
> ACLOCAL_PATH = $(ACLOCAL_DIR):$(ACLOCAL_HOST_DIR)
> export ACLOCAL_PATH
>
> AUTORECONF = $(HOST_CONFIGURE_OPTS) ACLOCAL="$(ACLOCAL)" AUTOCONF="$(AUTOCONF)"
> AUTOHEADER="$(AUTOHEADER)" AUTOMAKE="$(AUTOMAKE)" AUTOPOINT=/bin/true
> $(HOST_DIR)/bin/autoreconf -f -i
This second one is what I meant. But of course, this only works if autoconf and
automake don't need the explicit -I options.
> But that would clutter the variable ACLOCAL_PATH
That can actually be considered a good thing. If the user has set ACLOCAL_PATH,
we really do want to override it (we don't want to rely on anything from the host).
> and no other package exports variables.
Packages per se, indeed no, but this stuff can actually be considered part of
the infrastructure. They could just as well be defined in package/Makefile.in or
package/pkg-autotools.mk.
A bunch of files are already exporting variables - particularly
package/pkg-download.mk.
Regards,
Arnout
>
> -michael
More information about the buildroot
mailing list