[Buildroot] [PATCH] Complain loudly if BR2_GLOBAL_PATCH_DIR does not exit

Michal Sojka sojka at merica.cz
Wed Oct 22 11:43:00 UTC 2014


On Wed, Oct 22 2014, Maxime Hadjinlian wrote:
> Hi Michal, Yann, all
>
> On Tue, Oct 21, 2014 at 6:58 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
>> Michal, All,
>>
>> On 2014-10-21 10:42 +0200, Michal Sojka spake thusly:
>>> If the BR2_GLOBAL_PATCH_DIR is misspelled in the .config or if somebody
>>> renamed the existing directory with patches, buildroot happily builds
>>> everything but without the patches. As this can lead to surprising
>>> results,
>>> ---
>>>  package/pkg-generic.mk | 5 +++++
>>>  1 file changed, 5 insertions(+)
>>>
>>> diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
>>> index 259ee02..46d92aa 100644
>>> --- a/package/pkg-generic.mk
>>> +++ b/package/pkg-generic.mk
>>> @@ -162,6 +162,11 @@ $(BUILD_DIR)/%/.stamp_patched:
>>>       $(Q)touch $@
>>>       @$(call step_end,patch)
>>>
>>> +# Check that all directories specified in BR2_GLOBAL_PATCH_DIR exist.
>>> +$(foreach dir,$(call qstrip,$(BR2_GLOBAL_PATCH_DIR)),\
>>> +     $(if $(wildcard $(dir)),,\
>>> +             $(error BR2_GLOBAL_PATCH_DIR contains nonexistent directory $(dir))))
>>> +
> On a small note, since we already do a '[...] test -d $${D} [...]',
> why don't you simply add an else condition with the error message ?

Because $(D) is in fact $(BR2_GLOBAL_PATCH_DIR)/$(RAWNAME), which is not
required to exist for every package.

Best regards,
-Michal



More information about the buildroot mailing list