Re: [PATCH 3/3] populate_sdk_ext: Don't put nativesdk and crosssdk items in ext sdk

[Paul Eggleton <paul.eggleton@linux.intel.com>]

On Thu, 07 Jan 2016 11:24:27 Randy Witt wrote:
> The nativesdk and crosssdk targets were only getting pulled into the
> extensible sdk due to buildtools dependencies being in BB_TASKDEPDATA.
> However the nativesdk and crosssdk targets are not needed for the
> extensible sdk to be used.
> 
> This patch therefore removes them since it reduces the size of the
> extensible sdks that include sstate by ~300MB.
> 
> Signed-off-by: Randy Witt <randy.e.witt@linux.intel.com>
> ---
>  meta/classes/populate_sdk_ext.bbclass | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/meta/classes/populate_sdk_ext.bbclass
> b/meta/classes/populate_sdk_ext.bbclass index 5c99bd0..ef014b8 100644
> --- a/meta/classes/populate_sdk_ext.bbclass
> +++ b/meta/classes/populate_sdk_ext.bbclass
> @@ -173,6 +173,12 @@ python copy_buildsystem () {
>      # Filter the locked signatures file to just the sstate tasks we are
> interested in allowed_tasks = ['do_populate_lic', 'do_populate_sysroot',
> 'do_packagedata', 'do_package_write_ipk', 'do_package_write_rpm',
> 'do_package_write_deb', 'do_package_qa', 'do_deploy'] excluded_targets =
> d.getVar('SDK_TARGETS', True)
> +
> +    # Since buildtools is included in the sdk, the nativesdk items and
> +    # crosssdk end up in the tasks in locked-sigs.inc. However, the
> nativesdk 
> +    # and crosssdk items aren't needed for the extensible sdk to
> function so 
> +    # remove them from the list.
> +    excluded_targets += 'nativesdk crosssdk'

You need a leading space here to make this work (or alternatively split the 
excluded_targets value in the previous patch when you first set it, and add the 
items as list items here).

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre