[PATCH] Revert "kernel: restore scripts in the sysroot"

[PATCH] Revert "kernel: restore scripts in the sysroot"

[Richard Purdie]

This reverts commit 6a6735cb98d529c8c5376aedb5c951339f54fe96.

The module class already ensures the scripts are rebuilt correctly. Running
this at sstate installation time is problematic since it can require the
cross compiler. Adding such a dependency would cause issues of its own.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 4974507..24ac778 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -290,17 +290,6 @@ kernel_do_install() {
 }
 do_install[prefuncs] += "package_get_auto_pr"
 
-
-SSTATEPOSTINSTFUNCS += "kernelscripts_sstate_postinst"
-kernelscripts_sstate_postinst () {
-	if [ "${BB_CURRENTTASK}" = "populate_sysroot" -o "${BB_CURRENTTASK}" = "populate_sysroot_setscene" ]; then
-		( 
-		  cd ${STAGING_KERNEL_DIR}
-		  oe_runmake scripts
-		)
-	fi
-}
-
 python sysroot_stage_all () {
     oe.path.copyhardlinktree(d.expand("${D}${KERNEL_SRC_PATH}"), d.expand("${SYSROOT_DESTDIR}${KERNEL_SRC_PATH}"))
 }

Re: [PATCH] Revert "kernel: restore scripts in the sysroot"

[Bruce Ashfield]

On 13-11-27 08:44 AM, Richard Purdie wrote:
> This reverts commit 6a6735cb98d529c8c5376aedb5c951339f54fe96.
>
> The module class already ensures the scripts are rebuilt correctly. Running
> this at sstate installation time is problematic since it can require the
> cross compiler. Adding such a dependency would cause issues of its own.

I was waiting for the dust to clear and have been running with the
same thing here, saves me from sending it and I've removed it from
my queue.

Acked-by: Bruce Ashfield <bruce.ashfield@windriver.com>

>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
> index 4974507..24ac778 100644
> --- a/meta/classes/kernel.bbclass
> +++ b/meta/classes/kernel.bbclass
> @@ -290,17 +290,6 @@ kernel_do_install() {
>   }
>   do_install[prefuncs] += "package_get_auto_pr"
>
> -
> -SSTATEPOSTINSTFUNCS += "kernelscripts_sstate_postinst"
> -kernelscripts_sstate_postinst () {
> -	if [ "${BB_CURRENTTASK}" = "populate_sysroot" -o "${BB_CURRENTTASK}" = "populate_sysroot_setscene" ]; then
> -		(
> -		  cd ${STAGING_KERNEL_DIR}
> -		  oe_runmake scripts
> -		)
> -	fi
> -}
> -
>   python sysroot_stage_all () {
>       oe.path.copyhardlinktree(d.expand("${D}${KERNEL_SRC_PATH}"), d.expand("${SYSROOT_DESTDIR}${KERNEL_SRC_PATH}"))
>   }
>
>

Re: [PATCH] Revert "kernel: restore scripts in the sysroot"

[Khem Raj]

[-- Attachment #1: Type: text/plain, Size: 1614 bytes --]

On Wednesday, November 27, 2013, Bruce Ashfield wrote:

> On 13-11-27 08:44 AM, Richard Purdie wrote:
>
>> This reverts commit 6a6735cb98d529c8c5376aedb5c951339f54fe96.
>>
>> The module class already ensures the scripts are rebuilt correctly.
>> Running
>> this at sstate installation time is problematic since it can require the
>> cross compiler. Adding such a dependency would cause issues of its own.
>>
>
> I was waiting for the dust to clear and have been running with the
> same thing here, saves me from sending it and I've removed it from
> my queue.
>
> Acked-by: Bruce Ashfield <bruce.ashfield@windriver.com>
>
>
Looks ok to me


>
>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> ---
>> diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
>> index 4974507..24ac778 100644
>> --- a/meta/classes/kernel.bbclass
>> +++ b/meta/classes/kernel.bbclass
>> @@ -290,17 +290,6 @@ kernel_do_install() {
>>   }
>>   do_install[prefuncs] += "package_get_auto_pr"
>>
>> -
>> -SSTATEPOSTINSTFUNCS += "kernelscripts_sstate_postinst"
>> -kernelscripts_sstate_postinst () {
>> -       if [ "${BB_CURRENTTASK}" = "populate_sysroot" -o
>> "${BB_CURRENTTASK}" = "populate_sysroot_setscene" ]; then
>> -               (
>> -                 cd ${STAGING_KERNEL_DIR}
>> -                 oe_runmake scripts
>> -               )
>> -       fi
>> -}
>> -
>>   python sysroot_stage_all () {
>>       oe.path.copyhardlinktree(d.expand("${D}${KERNEL_SRC_PATH}"),
>> d.expand("${SYSROOT_DESTDIR}${KERNEL_SRC_PATH}"))
>>   }
>>
>>
>>
>

[-- Attachment #2: Type: text/html, Size: 2348 bytes --]