* [Buildroot] [PATCH] system: Fix warning when selecting systemd
@ 2015-11-06 18:23 Maxime Hadjinlian
2015-11-06 18:29 ` Yann E. MORIN
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Maxime Hadjinlian @ 2015-11-06 18:23 UTC (permalink / raw)
To: buildroot
With systemd and a custom skeleton, you end up with the following
warning:
warning: (BR2_INIT_SYSTEMD) selects BR2_ROOTFS_MERGED_USR which has
unmet direct dependencies (BR2_ROOTFS_SKELETON_DEFAULT)
Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
---
system/Config.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/system/Config.in b/system/Config.in
index 4d07010..92712a8 100644
--- a/system/Config.in
+++ b/system/Config.in
@@ -92,7 +92,7 @@ config BR2_INIT_SYSTEMD
depends on BR2_USE_MMU
depends on !BR2_STATIC_LIBS
depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_10
- select BR2_ROOTFS_MERGED_USR
+ select BR2_ROOTFS_MERGED_USR if BR2_ROOTFS_SKELETON_DEFAULT
select BR2_PACKAGE_SYSTEMD
comment "systemd needs (e)glibc toolchain, headers >= 3.10"
--
2.6.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* [Buildroot] [PATCH] system: Fix warning when selecting systemd
2015-11-06 18:23 [Buildroot] [PATCH] system: Fix warning when selecting systemd Maxime Hadjinlian
@ 2015-11-06 18:29 ` Yann E. MORIN
2015-11-08 13:58 ` Thomas Petazzoni
2015-11-09 20:31 ` Peter Korsgaard
2 siblings, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2015-11-06 18:29 UTC (permalink / raw)
To: buildroot
Maxime, All,
On 2015-11-06 19:23 +0100, Maxime Hadjinlian spake thusly:
> With systemd and a custom skeleton, you end up with the following
> warning:
> warning: (BR2_INIT_SYSTEMD) selects BR2_ROOTFS_MERGED_USR which has
> unmet direct dependencies (BR2_ROOTFS_SKELETON_DEFAULT)
>
> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> ---
> system/Config.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/system/Config.in b/system/Config.in
> index 4d07010..92712a8 100644
> --- a/system/Config.in
> +++ b/system/Config.in
> @@ -92,7 +92,7 @@ config BR2_INIT_SYSTEMD
> depends on BR2_USE_MMU
> depends on !BR2_STATIC_LIBS
> depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_10
> - select BR2_ROOTFS_MERGED_USR
> + select BR2_ROOTFS_MERGED_USR if BR2_ROOTFS_SKELETON_DEFAULT
> select BR2_PACKAGE_SYSTEMD
>
> comment "systemd needs (e)glibc toolchain, headers >= 3.10"
> --
> 2.6.1
>
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] system: Fix warning when selecting systemd
2015-11-06 18:23 [Buildroot] [PATCH] system: Fix warning when selecting systemd Maxime Hadjinlian
2015-11-06 18:29 ` Yann E. MORIN
@ 2015-11-08 13:58 ` Thomas Petazzoni
2015-11-09 20:31 ` Peter Korsgaard
2 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2015-11-08 13:58 UTC (permalink / raw)
To: buildroot
Dear Maxime Hadjinlian,
On Fri, 6 Nov 2015 19:23:45 +0100, Maxime Hadjinlian wrote:
> With systemd and a custom skeleton, you end up with the following
> warning:
> warning: (BR2_INIT_SYSTEMD) selects BR2_ROOTFS_MERGED_USR which has
> unmet direct dependencies (BR2_ROOTFS_SKELETON_DEFAULT)
>
> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
> ---
> system/Config.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] system: Fix warning when selecting systemd
2015-11-06 18:23 [Buildroot] [PATCH] system: Fix warning when selecting systemd Maxime Hadjinlian
2015-11-06 18:29 ` Yann E. MORIN
2015-11-08 13:58 ` Thomas Petazzoni
@ 2015-11-09 20:31 ` Peter Korsgaard
2015-11-09 20:46 ` Yann E. MORIN
2 siblings, 1 reply; 5+ messages in thread
From: Peter Korsgaard @ 2015-11-09 20:31 UTC (permalink / raw)
To: buildroot
>>>>> "Maxime" == Maxime Hadjinlian <maxime.hadjinlian@gmail.com> writes:
> With systemd and a custom skeleton, you end up with the following
> warning:
> warning: (BR2_INIT_SYSTEMD) selects BR2_ROOTFS_MERGED_USR which has
> unmet direct dependencies (BR2_ROOTFS_SKELETON_DEFAULT)
> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
> ---
> system/Config.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> diff --git a/system/Config.in b/system/Config.in
> index 4d07010..92712a8 100644
> --- a/system/Config.in
> +++ b/system/Config.in
> @@ -92,7 +92,7 @@ config BR2_INIT_SYSTEMD
> depends on BR2_USE_MMU
> depends on !BR2_STATIC_LIBS
> depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_10
> - select BR2_ROOTFS_MERGED_USR
> + select BR2_ROOTFS_MERGED_USR if BR2_ROOTFS_SKELETON_DEFAULT
Hmm, we have a number of workarounds for merged /usr under package/, so
those packages will break with a custom skeleton (that contains a merged
/usr as required by systemd) as those workarounds are not performed.
Perhaps we should instead introduce a hidden BR2_ROOTFS_MERGED_USR
kconfig option for the !BR2_ROOTFS_SKELETON_DEFAULT case?
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] system: Fix warning when selecting systemd
2015-11-09 20:31 ` Peter Korsgaard
@ 2015-11-09 20:46 ` Yann E. MORIN
0 siblings, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2015-11-09 20:46 UTC (permalink / raw)
To: buildroot
Peter, All,
On 2015-11-09 21:31 +0100, Peter Korsgaard spake thusly:
> >>>>> "Maxime" == Maxime Hadjinlian <maxime.hadjinlian@gmail.com> writes:
>
> > With systemd and a custom skeleton, you end up with the following
> > warning:
> > warning: (BR2_INIT_SYSTEMD) selects BR2_ROOTFS_MERGED_USR which has
> > unmet direct dependencies (BR2_ROOTFS_SKELETON_DEFAULT)
>
> > Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
> > ---
> > system/Config.in | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> > diff --git a/system/Config.in b/system/Config.in
> > index 4d07010..92712a8 100644
> > --- a/system/Config.in
> > +++ b/system/Config.in
> > @@ -92,7 +92,7 @@ config BR2_INIT_SYSTEMD
> > depends on BR2_USE_MMU
> > depends on !BR2_STATIC_LIBS
> > depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_10
> > - select BR2_ROOTFS_MERGED_USR
> > + select BR2_ROOTFS_MERGED_USR if BR2_ROOTFS_SKELETON_DEFAULT
>
> Hmm, we have a number of workarounds for merged /usr under package/, so
> those packages will break with a custom skeleton (that contains a merged
> /usr as required by systemd) as those workarounds are not performed.
>
> Perhaps we should instead introduce a hidden BR2_ROOTFS_MERGED_USR
> kconfig option for the !BR2_ROOTFS_SKELETON_DEFAULT case?
Hmm... Good point. I'll see to work on it...
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-11-09 20:46 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-06 18:23 [Buildroot] [PATCH] system: Fix warning when selecting systemd Maxime Hadjinlian
2015-11-06 18:29 ` Yann E. MORIN
2015-11-08 13:58 ` Thomas Petazzoni
2015-11-09 20:31 ` Peter Korsgaard
2015-11-09 20:46 ` Yann E. MORIN
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox