[Buildroot] [PATCH v3] infra: Add PRE_*_HOOKS for every step
Luca Ceresoli
luca at lucaceresoli.net
Mon Apr 7 21:48:39 UTC 2014
Hi Thomas,
Thomas Petazzoni wrote:
> Hello,
>
> On Fri, 28 Mar 2014 19:29:32 +0100, Maxime Hadjinlian wrote:
>
>> +* +LIBFOO_PRE_LEGAL_INFO_HOOKS+
>> * +LIBFOO_POST_LEGAL_INFO_HOOKS+
>
> I'm generally fine with the patch, but I'm wondering what's the
> usefulness of the legal-info hooks. I know you're merely adding a
> pre-hook to match an existing post-hook, but I'm wondering what would
> be the usage of a post-legal-info hook actually.
>
> Luca?
Maybe some mysterious package wants to save some additional files in the
sources directory, or anything even more strange...?
No, I can't really think of anything sound.
OTOH having pre- and post-hooks for all steps /except one/ does not look
very polished, and I think the effort to maintain an unused hook is
minimal.
So I would keep Maxime's patch as-is, unless there is a solid
reason to do otherwise.
--
Luca
More information about the buildroot
mailing list