diff mbox

system: Fix warning when selecting systemd

Message ID 1446834225-19878-1-git-send-email-maxime.hadjinlian@gmail.com
State Accepted
Headers show

Commit Message

Maxime Hadjinlian Nov. 6, 2015, 6:23 p.m. UTC
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(-)

Comments

Yann E. MORIN Nov. 6, 2015, 6:29 p.m. UTC | #1
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
>
Thomas Petazzoni Nov. 8, 2015, 1:58 p.m. UTC | #2
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
Peter Korsgaard Nov. 9, 2015, 8:31 p.m. UTC | #3
>>>>> "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?
Yann E. MORIN Nov. 9, 2015, 8:46 p.m. UTC | #4
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.
diff mbox

Patch

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"