[Buildroot] [PATCH 1/2] package: pkg-autotools: Add option to run autogen.sh
thomas.petazzoni at bootlin.com
Mon Dec 24 09:59:11 UTC 2018
On Mon, 24 Dec 2018 10:21:59 +0100, Yann E. MORIN wrote:
> On 2018-12-24 00:37 +0200, Vadim Kochan spake thusly:
> > Add $(PKG)_AUTOGEN variable which allows to execute autogen.sh script
> > on pre-configure stage.
> I'm not sure we want to have this in the infra. IIRC, we already spoke
> about it in the past, more than once, and the conclusion had always been
> that this was not demed useful.
> First, we only have three autotools packages that need that. This is a
> bit on the short lead to turn it into the infra.
> Second, we don't want to make it easy to use autogen.sh. We prefer
> developpers to use AUTORECONF=YES instead. Using autogen.sh should be
> a last-resort option.
> Thirdly, and lastly, not all packages have autogen.sh, some have it as
> bootstrap.sh, for example Asterisk:
> (Note that we don't need to call Asterisk's bootstrap.sh in Buildroot,
> but that is just an example that such scripts are not always named
> So, my opinion is that we should not have it in the infra.
Overall, I agree with you. If we wanted to make this more flexible than
what we have today, I would suggest to allow customizing the command
that it used for autoreconfiguring.
Something like this:
$(2)_AUTORECONF_CMD ?= $(AUTORECONF)
$(Q)cd $($(PKG)_SRCDIR) && $($(PKG)_AUTORECONF_ENV) $($(PKG)_AUTORECONF_CMD) $($(PKG)_AUTORECONF_OPTS)
One drawback is that having a CMD variable doesn't really match our
typical CMDS pattern.
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
More information about the buildroot