Re: [PATCH] local.conf.sample: Add DISTRO_FEATURES_INITMAN

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

On 01/23/2013 08:22 AM, Darren Hart wrote:
>
>
> On 01/23/2013 12:12 AM, Radu Moisan wrote:
>>
>> On 01/23/2013 01:05 AM, Darren Hart wrote:
>>>
>>> On 01/22/2013 09:01 AM, Richard Purdie wrote:
>>>> On Tue, 2013-01-22 at 17:29 +0200, Radu Moisan wrote:
>>>>> Default option is sysvinit
>>>>>
>>>>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
>>>>> ---
>>>>>    meta-yocto/conf/local.conf.sample |    3 +++
>>>>>    1 file changed, 3 insertions(+)
>>>>>
>>>>> diff --git a/meta-yocto/conf/local.conf.sample b/meta-yocto/conf/local.conf.sample
>>>>> index 200b4af..273e0eb 100644
>>>>> --- a/meta-yocto/conf/local.conf.sample
>>>>> +++ b/meta-yocto/conf/local.conf.sample
>>>>> @@ -243,3 +243,6 @@ BB_DISKMON_DIRS = "\
>>>>>    # track the version of this file when it was generated. This can safely be ignored if
>>>>>    # this doesn't mean anything to you.
>>>>>    CONF_VERSION = "1"
>>>>> +
>>>>> +# set default init manager: sysvinit or systemd
>>>>> +DISTRO_FEATURES_INITMAN ?= "sysvinit"
>>> Is tiny-init also an appropriate value here?
>>
>> No it is not. We could consider it though.
>>
>>>
>>> How does this differ from VIRTUAL_RUNTIME_init_manager?
>>>
>>
>> It's just a naming style issue, I used VIRTUAL_RUNTIME_init_manager when
>> I started and eventually Richard suggested the new variant.
>
> Is VIRTUAL_RUNTIME_init_manager deprecated? Does it need to be replaced
> in all the distro configs?
>
I will have a poky-tiny patch set here soon as it's more that just this, 
also poky-tiny will simply over ride this with a direct setting of 
DISTRO_FEATURE_INITMAN.

It looks like VIRTUAL_RUNTIME_init_manager gets set from 
DISTRO_FEATURE_INITMAN, so it might be duplication, not sure if RP 
intended that or if there is another round of cleanup that needs to 
happen here.

Sau!