[Buildroot] [PATCH 02/30] toolchain-external: pass CC and CFLAGS as arguments
Arnout Vandecappelle
arnout at mind.be
Tue Oct 25 12:46:01 UTC 2016
On 25-10-16 13:44, Thomas Petazzoni wrote:
> Hello,
>
> On Tue, 25 Oct 2016 13:07:01 +0200, Arnout Vandecappelle wrote:
>
>> This is exactly the same commit message as the previous commit. However, what
>> this patch actually does is
>>
>> 1. Pass up CC and CFLAGS one level more
>> -> I think this should be squashed with the previous commit.
>>
>> 2. pass LIBS as arguments
>> -> This should be a separate commit, with the appropriate commit message.
>
> I believe Romain's idea is to split this per *function* (or family
> of functions) being changed, not per arguments being passed. So indeed,
> the wording of his commit titles and logs is not very good. It should
> have been:
>
> toolchain-external: TOOLCHAIN_EXTERNAL_CREATE_*_LIB_SYMLINK: use arguments instead of global variables
> toolchain-external: TOOLCHAIN_EXTERNAL_INSTALL_*_LIBS: use arguments instead of global variables
> toolchain-external: TOOLCHAIN_EXTERNAL_INSTALL_WRAPPER: use arguments instead of global variables
> toolchain-external: TOOLCHAIN_EXTERNAL_INSTALL_TARGET_GDBSERVER: use arguments instead of global variables
>
> I find this split a lot more logical than a split by argument being
> added, as the latter would mean a given function gets modified in two
> different patches.
Sure, but then it would have been 4 patches instead of 2. The way it was split
up now was completely illogical IMHO.
To make things easy, however, I'd just squash these two in a single commit.
Regards,
Arnout
>
> Best regards,
>
> Thomas
>
--
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