Re: [PATCH 19/22] packagegroup-core-boot: revert to specifying sysvinit as default init manager

[Saul Wold <sgw@linux.intel.com>]

On 03/22/2013 10:38 AM, Ross Burton wrote:
> Don't follow DISTRO_FEATURES_INITMAN as that makes the packagegroups rebuild if
> you switch init manager.
>
This change still seems to include a reference to 
DISTRO_FEATURES_INITMAN for the systemd-compat-units line, is that correct?

  ${@base_contains('DISTRO_FEATURES_INITMAN', 'systemd', 
'systemd-compat-units', '',d)} \


Sau!



> As in hybrid situations there's generally a clear primary and minimal init
> manager choice, so change VIRTUAL-RUNTIME_init_manager to set the primary init
> manager, and roll your own groups/images for the secondary.
>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>   meta/recipes-core/packagegroups/packagegroup-core-boot.bb |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-core/packagegroups/packagegroup-core-boot.bb b/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
> index e73b46e..7f278f6 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
> @@ -26,8 +26,8 @@ RCONFLICTS_${PN} = "task-core-boot"
>   # Distro can override the following VIRTUAL-RUNTIME providers:
>   VIRTUAL-RUNTIME_dev_manager ?= "udev"
>   VIRTUAL-RUNTIME_login_manager ?= "tinylogin"
> -VIRTUAL-RUNTIME_init_manager ?= "${DISTRO_FEATURES_INITMAN}"
> -VIRTUAL-RUNTIME_initscripts ?= "${@base_contains('DISTRO_FEATURES_INITMAN', 'sysvinit', 'initscripts', '',d)}"
> +VIRTUAL-RUNTIME_init_manager ?= "sysvinit"
> +VIRTUAL-RUNTIME_initscripts ?= "initscripts"
>   VIRTUAL-RUNTIME_keymaps ?= "keymaps"
>
>   RDEPENDS_${PN} = "\
>