Re: [RFC PATCH 2/2] meta/u-boot: Include libssl-dev only when building tools

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

On 15.11.21 17:06, Vijai Kumar K wrote:
> libssl-dev of the host architecture is needed only when you
> cross compile u-boot-tools.

Did you mean "host" in the Gnu/Debian sense, i.e. the target (in Isar
sense)? Or "build" - but that would mean ":native"?

> 
> Signed-off-by: Vijai Kumar K <Vijaikumar_Kanagarajan@mentor.com>
> ---
>  meta/recipes-bsp/u-boot/u-boot-custom.inc | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-bsp/u-boot/u-boot-custom.inc b/meta/recipes-bsp/u-boot/u-boot-custom.inc
> index bc92552..0887757 100644
> --- a/meta/recipes-bsp/u-boot/u-boot-custom.inc
> +++ b/meta/recipes-bsp/u-boot/u-boot-custom.inc
> @@ -25,7 +25,8 @@ python() {
>  }
>  
>  DEBIAN_BUILD_DEPENDS ?= "bc, bison, flex, device-tree-compiler, git, libssl-dev:native"
> -DEBIAN_BUILD_DEPENDS += "${@', libssl-dev' if d.getVar('ISAR_CROSS_COMPILE') == '1' else ''}"
> +DEBIAN_BUILD_DEPENDS += "${@', libssl-dev' if (d.getVar('ISAR_CROSS_COMPILE') == '1' and \
> +			bb.utils.contains('U_BOOT_BUILD_PROFILES', 'tools', 1, 0, d)) else ''}"
>  
>  
>  TEMPLATE_FILES = "debian/control.tmpl"
> 

Code makes sense - I assume to tested with a very recent U-Boot and all
profiles on.

Jan

-- 
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux