Re: [PATCH 1/4] update-rc.d/useradd: Add additional dependecies

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

On 06/10/2014 04:57 AM, Phil Blundell wrote:
> On Thu, 2014-06-05 at 17:09 -0700, Saul Wold wrote:
>> These dependcies are needed to ensure that thier packages are created
>> correctly since these classes have runtime dependiences in their packages
>> but they are not actually created yet at rootfs time.
>
> Can you be more specific about why these
> dependecies/dependcies/dependiences are required?  I can't, offhand,
> think of any reason why update-rc.d requires initscripts for example.
>
It's more about having initscripts-funtions package built so that it's 
available for the dynamic addition to the RDEPENDS later in 
populate_pacakges_updatercd().  If initscripts is not built and we add 
initscripts-functions to the RDEPENDS, the final rootfs will fail with 
an unsatisfied dependency.

> Also, what's going on with the PACKAGESPLITFUNCS_remove_class-nativesdk?
>
We don't need to run that funtion to create the nativesdk sysroot, so 
don't run it.

Sau!

> thanks
>
> p.
>
>>
>> [YOCTO #6072]
>>
>> Signed-off-by: Saul Wold <sgw@linux.intel.com>
>> ---
>>   meta/classes/update-rc.d.bbclass | 2 ++
>>   meta/classes/useradd.bbclass     | 2 +-
>>   2 files changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/classes/update-rc.d.bbclass b/meta/classes/update-rc.d.bbclass
>> index 56eef4e..ca9486b 100644
>> --- a/meta/classes/update-rc.d.bbclass
>> +++ b/meta/classes/update-rc.d.bbclass
>> @@ -1,6 +1,7 @@
>>   UPDATERCPN ?= "${PN}"
>>
>>   DEPENDS_append = " update-rc.d-native"
>> +DEPENDS_append_class-target = " initscripts"
>>   UPDATERCD = "update-rc.d"
>>   UPDATERCD_class-cross = ""
>>   UPDATERCD_class-native = ""
>> @@ -67,6 +68,7 @@ python __anonymous() {
>>   }
>>
>>   PACKAGESPLITFUNCS_prepend = "populate_packages_updatercd "
>> +PACKAGESPLITFUNCS_remove_class-nativesdk = "populate_packages_updatercd "
>>
>>   populate_packages_updatercd[vardeps] += "updatercd_prerm updatercd_postrm updatercd_preinst updatercd_postinst"
>
>
>
>