[Buildroot] [PATCH v4 3/3] fs/common.mk: make sure that static devices from packages are created
Arnout Vandecappelle
arnout at mind.be
Wed Dec 5 09:18:44 UTC 2018
On 05/12/2018 08:53, Thomas Petazzoni wrote:
> Hello,
>
> On Tue, 4 Dec 2018 22:05:19 +0100, Arnout Vandecappelle wrote:
>
>>> diff --git a/fs/common.mk b/fs/common.mk
>>> index 077ce8903e..a560417c6c 100644
>>> --- a/fs/common.mk
>>> +++ b/fs/common.mk
>>> @@ -61,10 +61,10 @@ endif
>>> $(call PRINTF,$(PACKAGES_PERMISSIONS_TABLE)) > $(ROOTFS_FULL_DEVICES_TABLE)
>>> ifneq ($(ROOTFS_DEVICE_TABLES),)
>>> cat $(ROOTFS_DEVICE_TABLES) >> $(ROOTFS_FULL_DEVICES_TABLE)
>>> +endif
>>> ifeq ($(BR2_ROOTFS_DEVICE_CREATION_STATIC),y)
>>> $(call PRINTF,$(PACKAGES_DEVICES_TABLE)) >> $(ROOTFS_FULL_DEVICES_TABLE)
>>> endif
>>
>> This is still conflicting with the purpose of patch 2/3: the package device
>> tables are still overriding the user-specified global device table.
>
> Fair point. Should we do:
>
> $(call PRINTF,$(PACKAGES_PERMISSIONS_TABLE)) > $(ROOTFS_FULL_DEVICES_TABLE)
> ifeq ($(BR2_ROOTFS_DEVICE_CREATION_STATIC),y)
> $(call PRINTF,$(PACKAGES_DEVICES_TABLE)) >> $(ROOTFS_FULL_DEVICES_TABLE)
> endif
> ifneq ($(ROOTFS_DEVICE_TABLES),)
> cat $(ROOTFS_DEVICE_TABLES) >> $(ROOTFS_FULL_DEVICES_TABLE)
> endif
>
> instead ?
Yep, that's what I meant. I guess.
Regards,
Arnout
More information about the buildroot
mailing list