diff mbox series

rpm-sequoia-crypto-policy: clean up dependencies

Message ID 20250224050848.2216918-1-zboszor@gmail.com
State Accepted, archived
Commit 34fc028a8a3a6628b11c6ee67676532012f55f47
Headers show
Series rpm-sequoia-crypto-policy: clean up dependencies | expand

Commit Message

Böszörményi Zoltán Feb. 24, 2025, 5:08 a.m. UTC
Rely on host-provided python3, coreutils and make.

openssl-native is not needed either, as the build scripts
use crypto functionality via python.

Signed-off-by: Zoltán Böszörményi <zboszor@gmail.com>
---
 .../rpm-sequoia/rpm-sequoia-crypto-policy_git.bb              | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Richard Purdie Feb. 25, 2025, 10:36 p.m. UTC | #1
On Mon, 2025-02-24 at 06:08 +0100, Zoltan Boszormenyi via lists.openembedded.org wrote:
> Rely on host-provided python3, coreutils and make.
> 
> openssl-native is not needed either, as the build scripts
> use crypto functionality via python.
> 
> Signed-off-by: Zoltán Böszörményi <zboszor@gmail.com>
> ---
>  .../rpm-sequoia/rpm-sequoia-crypto-policy_git.bb              | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/meta/recipes-devtools/rpm-sequoia/rpm-sequoia-crypto-policy_git.bb b/meta/recipes-devtools/rpm-sequoia/rpm-sequoia-crypto-policy_git.bb
> index 098881e3da..dcc0c7264c 100644
> --- a/meta/recipes-devtools/rpm-sequoia/rpm-sequoia-crypto-policy_git.bb
> +++ b/meta/recipes-devtools/rpm-sequoia/rpm-sequoia-crypto-policy_git.bb
> @@ -5,9 +5,7 @@ LICENSE = "LGPL-2.1-or-later"
>  
>  LIC_FILES_CHKSUM = "file://COPYING.LESSER;md5=a6f89e2100d9b6cdffcea4f398e37343"
>  
> -DEPENDS = "coreutils-native openssl-native make-native"
> -
> -inherit allarch python3native
> +inherit allarch
>  
>  SRC_URI = "git://gitlab.com/redhat-crypto/fedora-crypto-policies.git;protocol=https;branch=master"
>  


It looks like the python script in there needs python 3.11 for tomllib;

https://autobuilder.yoctoproject.org/valkyrie/#/builders/3/builds/1100

so the other pieces are good but we probably need the python dependency
unfortunately. Could you tweak the patch please?

It may be worth leaving a comment that we can use python from the host
system when we have 3.11 as the min version.

Cheers,

Richard
diff mbox series

Patch

diff --git a/meta/recipes-devtools/rpm-sequoia/rpm-sequoia-crypto-policy_git.bb b/meta/recipes-devtools/rpm-sequoia/rpm-sequoia-crypto-policy_git.bb
index 098881e3da..dcc0c7264c 100644
--- a/meta/recipes-devtools/rpm-sequoia/rpm-sequoia-crypto-policy_git.bb
+++ b/meta/recipes-devtools/rpm-sequoia/rpm-sequoia-crypto-policy_git.bb
@@ -5,9 +5,7 @@  LICENSE = "LGPL-2.1-or-later"
 
 LIC_FILES_CHKSUM = "file://COPYING.LESSER;md5=a6f89e2100d9b6cdffcea4f398e37343"
 
-DEPENDS = "coreutils-native openssl-native make-native"
-
-inherit allarch python3native
+inherit allarch
 
 SRC_URI = "git://gitlab.com/redhat-crypto/fedora-crypto-policies.git;protocol=https;branch=master"