Re: [PATCH v5 2/3] refactor linux-custom.inc to use ISAR's DEB_BUILD_PROFILES support

[Jan Kiszka <jan.kiszka@siemens.com>]

On 12.01.22 15:39, Felix Moessbauer wrote:
> This patch replaces the manual setup of the DEB_BUILD_PROFILES
> environment variable in the linux-custom.inc recipe.
> Instead, the recently introduced DEB_BUILD_PROFILES infrastructure
> of ISAR is used.
> 
> Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
> ---
>   meta/recipes-kernel/linux/linux-custom.inc | 13 ++++++++-----
>   1 file changed, 8 insertions(+), 5 deletions(-)
> 
> diff --git a/meta/recipes-kernel/linux/linux-custom.inc b/meta/recipes-kernel/linux/linux-custom.inc
> index ed89aa09..59d42c84 100644
> --- a/meta/recipes-kernel/linux/linux-custom.inc
> +++ b/meta/recipes-kernel/linux/linux-custom.inc
> @@ -117,6 +117,14 @@ def config_fragments(d):
>               fragments.append(local)
>       return fragments
>   
> +def get_additional_build_profiles(d):
> +    profiles = d.getVar('BASE_DISTRO', True)
> +    if d.getVar('KERNEL_LIBC_DEV_DEPLOY', True) != '1':
> +        profiles += ' nolibcdev'
> +    return profiles
> +
> +DEB_BUILD_PROFILES += "${@get_additional_build_profiles(d)}"
> +
>   do_prepare_build_prepend() {
>   	# copy meta-data over to source tree
>   	rm -rf ${S}/debian
> @@ -176,10 +184,5 @@ dpkg_configure_kernel() {
>   }
>   
>   dpkg_runbuild_prepend() {
> -	profiles="${BASE_DISTRO}"
> -	if [ "${KERNEL_LIBC_DEV_DEPLOY}" != "1" ]; then
> -		profiles="${profiles} nolibcdev"
> -	fi
> -	export DEB_BUILD_PROFILES="${profiles}"
>   	dpkg_configure_kernel
>   }

After local experiments, it looks like this patch must be folded into 
patch 1 in order to avoid that linux-libc-dev gets built accidentally 
when it shouldn't.

Jan

-- 
Siemens AG, Technology
Competence Center Embedded Linux