Re: [Buildroot] [PATCH v5 4/8] boot/optee-os: bump to version 3.16.0

From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: "Clément Léger" <clement.leger@bootlin.com>
Cc: Etienne Carriere <etienne.carriere@linaro.org>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v5 4/8] boot/optee-os: bump to version 3.16.0
Date: Fri, 18 Mar 2022 23:40:31 +0100	[thread overview]
Message-ID: <20220318224031.GV283544@scaer> (raw)
In-Reply-To: <20220318132427.48737-5-clement.leger@bootlin.com>

Clément, All,

On 2022-03-18 14:24 +0100, Clément Léger spake thusly:
> Bump OP-TEE OS package version to OP-TEE release 3.16.0 and set
> BR2_TARGET_OPTEE_OS_LATEST to select
> BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_CRYPTOGRAPHY since
> python-cryptography is now needed to build optee-os.
> 
> Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
> Signed-off-by: Clément Léger <clement.leger@bootlin.com>
> ---
>  boot/optee-os/Config.in     | 5 +++--
>  boot/optee-os/optee-os.hash | 4 ++--
>  2 files changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/boot/optee-os/Config.in b/boot/optee-os/Config.in
> index 02b5d7c86f..811937e025 100644
> --- a/boot/optee-os/Config.in
> +++ b/boot/optee-os/Config.in
> @@ -19,7 +19,8 @@ choice
>  	  Select the version of OP-TEE OS you want to use
>  
>  config BR2_TARGET_OPTEE_OS_LATEST
> -	bool "3.15.0"
> +	select BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_CRYPTOGRAPHY
> +	bool "3.16.0"

3.16.0 now needs the dependency on BR2_PACKAGE_HOST_RUSTC_ARCH_SUPPORTS,
so if that is not met, there will only be the custom git as a fallback.

This is not so nice that the latest has to, well, be the latest, and
that we don;t keep 3.15.0 around for those that can't have a host rustc
compiler.

But then, who are we kidding? Builds on non-x86 are pretty rare yet;
even if some people might be on PPC or AArch64, they still are to be
no much more than two-handful (1023).

Also, check-package whines;

    $ make check-package
    boot/optee-os/Config.in:22: attributes order: type, default, depends
    on, select, help (http://nightly.buildroot.org/#_config_files)

And a hidden one:

    boot/optee-os/optee-os.hash:4: separation does not match expectation
    (http://nightly.buildroot.org/#adding-packages-hash)

Applied to master with those fixed, thanks.

Regards,
Yann E. MORIN.

>  	help
>  	  Use the latest release tag from the OP-TEE OS official Git
>  	  repository.
> @@ -51,7 +52,7 @@ endif
>  
>  config BR2_TARGET_OPTEE_OS_VERSION
>  	string
> -	default "3.15.0"	if BR2_TARGET_OPTEE_OS_LATEST
> +	default "3.16.0"	if BR2_TARGET_OPTEE_OS_LATEST
>  	default BR2_TARGET_OPTEE_OS_CUSTOM_REPO_VERSION \
>  				if BR2_TARGET_OPTEE_OS_CUSTOM_GIT
>  
> diff --git a/boot/optee-os/optee-os.hash b/boot/optee-os/optee-os.hash
> index 3c32e25b36..b9b0e5b43f 100644
> --- a/boot/optee-os/optee-os.hash
> +++ b/boot/optee-os/optee-os.hash
> @@ -1,4 +1,4 @@
> -# From https://github.com/OP-TEE/optee_os/archive/3.15.0/optee-os-3.15.0.tar.gz
> -sha256 e5bb3d9eedaf7785af091602addac5b52118f4cdc108af9cd6f6c96b21503ab8  optee-os-3.15.0.tar.gz
> +# From https://github.com/OP-TEE/optee_os/archive/3.16.0/optee-os-3.16.0.tar.gz
> +sha256 ebc8e18ad2039ee97c34f74a7546de9119e26f04c368b6c7fd0c55f93d33d2d6  optee-os-3.16.0.tar.gz
>  # Locally computed
>  sha256 1247ee90858f4037b6cac63cbffddfed435d0d73c631b37d78c1e6e6ab3e5d1a  LICENSE
> -- 
> 2.34.1
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot