[Buildroot] [PATCH v2 1/1] linux: Make dtc install step more reliable
Andreas Naumann
dev at andin.de
Mon Nov 26 07:50:35 UTC 2018
Hi Thomas,
Am 23.11.18 um 22:29 schrieb Thomas Petazzoni:
> Hello,
>
> On Tue, 13 Nov 2018 16:50:16 +0100, Andreas Naumann wrote:
>> Checking for the existence of the dtc binary built by the
>> non-dependent dtc package may cause instable behaviour when giving more
>> freedom on the order of how the packages are built (parallelization).
>>
>> In addidion, when moving to per-package host/target method, the check
>> would always trigger in the isolated host, leading to linux-dtc always
>> being installed as dtc.
>> This in turn may lead to undesired overwriting of the real host-dtc binary
>> when finally assembling the global host dir.
>>
>> Thus rework the linux-dtc install condition to be defined by configuration
>> rather than compile time order.
>>
>> Signed-off-by: Andreas Naumann <anaumann at ultratronik.de>
>> ---
>> Changes v1 -> v2:
>> - simplified by direct usage of make conditional almost as suggested by
>> Arnout (test revealed that an additinal comma was needed for correct logic)
>> ---
>> linux/linux.mk | 4 +---
>> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> I hesitated a bit, but in the end decided to apply this to master.
> Indeed, with the current code, it's really a matter of luck whether
> host-dtc gets built before/after linux that determines if the dtc ->
> linux-dtc symlink will be created or not. With this patch, regardless
> of whether host-dtc is built before or after linux, we'll have the same
> result. To me, this was a sufficient motivation to apply this patch to
> master.
>
Thanks for applying it. May I ask why you hesitated, was the commit
message clear enough? Or just too small of an issue?
regards,
Andreas
> Thanks!
>
> Thomas
>
More information about the buildroot
mailing list