[PATCH 1/1] fix cyclic dep on crossbuild-essential-riscv64

[PATCH 1/1] fix cyclic dep on crossbuild-essential-riscv64

[Felix Moessbauer]

This patch fixes a cyclic dependency on the crossbuild-essentials,
introduced in d90f75bd and made visible by recent chroot optimizations.
The crossbuild-essential-riscv64 package is a dependency when building
with sbuild for the non-native architecture. As this (meta) package is
arch=all anyways, we simply build it using the native architecture
(e.g. --host-arch amd64 on x64) to not depend on the cross-toolchain.

Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
---
 .../crossbuild-essential-riscv64_0.2-isar.bb                   | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-essential-riscv64_0.2-isar.bb b/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-essential-riscv64_0.2-isar.bb
index 9554f1f6..c5092146 100644
--- a/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-essential-riscv64_0.2-isar.bb
+++ b/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-essential-riscv64_0.2-isar.bb
@@ -7,6 +7,9 @@
 inherit dpkg-raw
 
 DPKG_ARCH = "all"
+# build this package using the host toolchain
+# to break cyclic dependency in the cross chroot
+PACKAGE_ARCH = "${HOST_ARCH}"
 
 DEBIAN_DEPENDS = " \
     gcc-riscv64-linux-gnu, \
-- 
2.34.1

Re: [PATCH 1/1] fix cyclic dep on crossbuild-essential-riscv64

[Jan Kiszka]

On 25.01.23 11:01, Felix Moessbauer wrote:
> This patch fixes a cyclic dependency on the crossbuild-essentials,
> introduced in d90f75bd and made visible by recent chroot optimizations.
> The crossbuild-essential-riscv64 package is a dependency when building
> with sbuild for the non-native architecture. As this (meta) package is
> arch=all anyways, we simply build it using the native architecture
> (e.g. --host-arch amd64 on x64) to not depend on the cross-toolchain.
> 
> Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
> ---
>  .../crossbuild-essential-riscv64_0.2-isar.bb                   | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-essential-riscv64_0.2-isar.bb b/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-essential-riscv64_0.2-isar.bb
> index 9554f1f6..c5092146 100644
> --- a/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-essential-riscv64_0.2-isar.bb
> +++ b/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-essential-riscv64_0.2-isar.bb
> @@ -7,6 +7,9 @@
>  inherit dpkg-raw
>  
>  DPKG_ARCH = "all"
> +# build this package using the host toolchain
> +# to break cyclic dependency in the cross chroot
> +PACKAGE_ARCH = "${HOST_ARCH}"
>  

...or revert 30e1c1cc51f4 again. But this one is faster. Ack.

Jan

>  DEBIAN_DEPENDS = " \
>      gcc-riscv64-linux-gnu, \

-- 
Siemens AG, Technology
Competence Center Embedded Linux

Re: [PATCH 1/1] fix cyclic dep on crossbuild-essential-riscv64

[Moessbauer, Felix]

Any news on this?

Riscv64 is still broken in next, this patch fixes it, got an Ack and
some time has passed for others to comment.

Felix

On Wed, 2023-01-25 at 10:01 +0000, Felix Moessbauer wrote:
> This patch fixes a cyclic dependency on the crossbuild-essentials,
> introduced in d90f75bd and made visible by recent chroot
> optimizations.
> The crossbuild-essential-riscv64 package is a dependency when
> building
> with sbuild for the non-native architecture. As this (meta) package
> is
> arch=all anyways, we simply build it using the native architecture
> (e.g. --host-arch amd64 on x64) to not depend on the cross-toolchain.
> 
> Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
> ---
>  .../crossbuild-essential-riscv64_0.2-isar.bb                   | 3
> +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/meta/recipes-devtools/crossbuild-essential-
> riscv64/crossbuild-essential-riscv64_0.2-isar.bb b/meta/recipes-
> devtools/crossbuild-essential-riscv64/crossbuild-essential-
> riscv64_0.2-isar.bb
> index 9554f1f6..c5092146 100644
> --- a/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-
> essential-riscv64_0.2-isar.bb
> +++ b/meta/recipes-devtools/crossbuild-essential-riscv64/crossbuild-
> essential-riscv64_0.2-isar.bb
> @@ -7,6 +7,9 @@
>  inherit dpkg-raw
>  
>  DPKG_ARCH = "all"
> +# build this package using the host toolchain
> +# to break cyclic dependency in the cross chroot
> +PACKAGE_ARCH = "${HOST_ARCH}"
>  
>  DEBIAN_DEPENDS = " \
>      gcc-riscv64-linux-gnu, \

Re: [PATCH 1/1] fix cyclic dep on crossbuild-essential-riscv64

[Uladzimir Bely]

In the email from среда, 25 января 2023 г. 13:01:57 +03 user Felix Moessbauer 
wrote:
> This patch fixes a cyclic dependency on the crossbuild-essentials,
> introduced in d90f75bd and made visible by recent chroot optimizations.
> The crossbuild-essential-riscv64 package is a dependency when building
> with sbuild for the non-native architecture. As this (meta) package is
> arch=all anyways, we simply build it using the native architecture
> (e.g. --host-arch amd64 on x64) to not depend on the cross-toolchain.
> 
> Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
> ---
>  .../crossbuild-essential-riscv64_0.2-isar.bb                   | 3 +++
>  1 file changed, 3 insertions(+)
> 

Applied to next, thanks.