[Buildroot] [PATCH 1/2] fs: Add common rootfs dependencies to PACKAGES
Yann E. MORIN
yann.morin.1998 at free.fr
Mon Nov 12 16:46:10 UTC 2018
Andreas, All,
On 2018-11-12 13:53 +0100, Andreas Naumann spake thusly:
> Am 09.11.18 um 23:11 schrieb Arnout Vandecappelle:
> >On 08/11/2018 14:04, Andreas Naumann wrote:
> ...
> >>+PACKAGES += $(ROOTFS_COMMON_DEPENDENCIES)
> >
> > I don't really like this, for the simple reason that it will enable the
> >dependencies even if they are not needed because we're not building any rootfs.
>
> good point, I dont like that either. I'd propose to change/move the patch
> into something like
> PACKAGES += $$(filter-out $$(PACKAGES),$$(ROOTFS_COMMON_DEPENDENCIES))
> inside the rootfs-conditional clause at the bottom of fs/common.mk. Actually
> I'm not sure the filter-out stuff is really needed since duplicate targets
> may accumulate in PACKAGES anyway.
Yes, that would be OK to moce them to the per-filesystem macro, but we
already have:
ROOTFS_$(2)_DEPENDENCIES += rootfs-common
So maybe just do:
ROOTFS_$(2)_DEPENDENCIES += \
rootfs-common \
$$(ROOTFS_COMMON_DEPENDENCIES)
Regards,
Yann E. MORIN.
> >
> > It's a small thing though, and working around it is a lot more complicated (you
> >could e.g. select a new Config.in symbol that is selected by all the rootfses).
> >And when we ever get the Config.in.host for all host packages, it's solved
> >already. So
> >
> >Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
> >
> > Regards,
> > Arnout
> >
> >> $(ROOTFS_COMMON_TAR): ROOTFS=COMMON
> >> $(ROOTFS_COMMON_TAR): FAKEROOT_SCRIPT=$(FS_DIR)/fakeroot.fs
> >>
> >_______________________________________________
> >buildroot mailing list
> >buildroot at busybox.net
> >http://lists.busybox.net/mailman/listinfo/buildroot
> >
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list