[Buildroot] [PATCH v2 01/15] package/pkg-autotools.mk: Factorize hooks.
Arnout Vandecappelle
arnout at mind.be
Tue Nov 11 13:33:04 UTC 2014
On 11/11/14 14:17, Yann E. MORIN wrote:
> Arnout, All,
>
> On 2014-11-11 14:03 +0100, Arnout Vandecappelle spake thusly:
>> I'm mostly going to negate Yann's comments :-)
>
> Meh... :-)
>
>>>> $(2)_DEPENDENCIES += host-automake host-autoconf host-libtool
>>>> +else
>>>> +# default values are not evaluated yet, so don't rely on this defaulting to YES
>>>> +ifneq ($$($(2)_LIBTOOL_PATCH),NO)
>>>> +$(2)_POST_PATCH_HOOKS += LIBTOOL_PATCH_HOOK
>>>> +endif
>>>> endif
>>>
>>> I think you got the order wrong here: we want the libtool patch to be
>>> applied _before_ we autoreconf.
>>
>> No, we don't. We want it to be done after the autoreconf, because autoreconf
>> generates the libtool scripts.
>>
>> In the original, there was a shell condition that evaluates that AUTORECONF is
>> not YES in the LIBTOOL_PATCH_HOOK. So you would get the message that libtool is
>> patched before the autoreconf, but the actual patching only happens after the
>> reconf.
>
> Well, both of us are partialy wrong, I think. LIBTOOL_PATCH_HOOK is a
> post-patch hook, while GETTEXTIZE_HOOK and AUTORECONF_HOOK are pre-configure
> hooks. So, the order at which they are defined is irrelevant, as it is done
> right now.
>
> But I agree with Arnout, we need to rework this.
>
> The issue I can see is that, in case we're not autoreconfiguring, we can
> only apply the libtool patch once, and that has to be as a post-patch.
> But if we do autoreconf, it only makes sense to apply it after the
> autoreconf.
>
> One obvious thing to do would be to always apply it at post-patch,
> whether we autoreconf or not, and if we do autoreconf, also apply it
> after the autoreconf. After all, patching is not something that takes a
> lot of time, is it?
>
> That way, the code path is a bit more obvious.
>
> And move the 13-or-so lines that do the conditional patching out to a
> single function, so we can share it more easily.
All this makes it pretty clear that reworking the call should be done in a
separate patch :-)
Regards,
Arnout
>
> Sigh, I need some more coffee...
>
> Regards,
> Yann E. MORIN.
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list