[Buildroot] [PATCH 1/1] Declare SED Makefile instead of package/Makefile.in so it exists globally.
Arnout Vandecappelle
arnout at mind.be
Tue Feb 16 22:35:07 UTC 2016
Hi Alvaro,
In addition to Thomas's comments, please use your full name in the
Signed-off-by line.
On 16-02-16 21:35, Thomas Petazzoni wrote:
> Hello,
>
> The title is too long, it should be:
>
> Makefile: declare SED globally
>
> or something like that.
>
> On Tue, 16 Feb 2016 16:15:26 +0100, Alvaro G. M wrote:
>> In particular, this allows the use of buildroot-submodule and in general,
>> any use of buildroot from out of tree.
>
> This commit log should be improved, because:
>
> 1/ Why should we care about buildroot-submodule, and if we care, what
> is the actual problem ?
>
> 2/ What do you mean by "use Buildroot from out of tree" ? I do out of
> tree builds every day and it works just fine. So this aspect needs
> to be detailed.
>
> I have nothing against the change, which is fairly simple, but it needs
> to be justified properly.
I suspect we're not going to like the justification :-)
Regards,
Arnout
--
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: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
More information about the buildroot
mailing list