Re: [PATCH v2 3/4] meta: u-boot: Prepare for newer versions

[Gylstorff Quirin <quirin.gylstorff@siemens.com>]

Hi,

On 11/22/21 4:26 PM, Vijai Kumar K wrote:
> Newer versions of u-boot require libssl-dev:native for compilation.
> It also needs libssl-dev of the host architecture for cross compilation
> of tools.
> 
> Signed-off-by: Vijai Kumar K <Vijaikumar_Kanagarajan@mentor.com>
> ---
>   meta/recipes-bsp/u-boot/u-boot-custom.inc | 4 +++-
>   1 file changed, 3 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 5198809..cfae0e2 100644
> --- a/meta/recipes-bsp/u-boot/u-boot-custom.inc
> +++ b/meta/recipes-bsp/u-boot/u-boot-custom.inc
> @@ -41,7 +41,9 @@ python() {
>               d.setVar('U_BOOT_BUILD_PROFILES_remove', "config")
>   }
>   
> -DEBIAN_BUILD_DEPENDS ?= "bc, bison, flex, device-tree-compiler, git"
> +DEBIAN_BUILD_DEPENDS ?= "bc, bison, flex, device-tree-compiler, git, libssl-dev:native"
> +DEBIAN_BUILD_DEPENDS += "${@', libssl-dev' \
> +	if bb.utils.contains('U_BOOT_BUILD_PROFILES', 'tools', 1, 0, d) else ''}"

Is there are reason why you didn't use `Build-Depends syntax extension` 
from[1]?


[1]: https://wiki.debian.org/BuildProfileSpec


Quirin
>   
>   TEMPLATE_FILES = "debian/control.tmpl"
>   TEMPLATE_VARS += "MACHINE DEBIAN_BUILD_DEPENDS"
>