[PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain

public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed

* [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain
@ 2025-11-03 15:42 'Andreas Naumann' via isar-users
  2025-11-03 15:42 ` [PATCH 1/2] rootfs: " 'Andreas Naumann' via isar-users
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: 'Andreas Naumann' via isar-users @ 2025-11-03 15:42 UTC (permalink / raw)
  To: isar-users; +Cc: Andreas Naumann

This is a stripped down version of my previous 
[RFC 0/5] Improving multiarch support for arch-incompatible packages

For now, I've dropped all patches except the one which covers the problem
where targets in the dependency chain are built for both the host and the
target architecture while only one is needed.

The second patch is a way to reproduce this. It's quite hacky, and
would need serious rework before integration into the testsuite.

Andreas Naumann (2):
  rootfs: Prevent adding unneeded package variants to recursive
    dependency chain
  DONOTMERGE: testcase for transient native dependency

 meta-isar/recipes-app/adduser/adduser.bb         | 6 ++++++
 meta-isar/recipes-app/gss/gss.bb                 | 5 +++++
 meta-isar/recipes-core/images/isar-image-base.bb | 2 ++
 meta/classes/rootfs.bbclass                      | 3 ++-
 4 files changed, 15 insertions(+), 1 deletion(-)
 create mode 100644 meta-isar/recipes-app/adduser/adduser.bb
 create mode 100644 meta-isar/recipes-app/gss/gss.bb

-- 
2.43.0

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/20251103154223.985951-1-anaumann%40emlix.com.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] rootfs: Prevent adding unneeded package variants to recursive dependency chain
  2025-11-03 15:42 [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain 'Andreas Naumann' via isar-users
@ 2025-11-03 15:42 ` 'Andreas Naumann' via isar-users
  2025-11-03 15:42 ` [PATCH 2/2] DONOTMERGE: testcase for transient native dependency 'Andreas Naumann' via isar-users
  2025-11-07 12:00 ` [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain Zhihang Wei
  2 siblings, 0 replies; 6+ messages in thread
From: 'Andreas Naumann' via isar-users @ 2025-11-03 15:42 UTC (permalink / raw)
  To: isar-users; +Cc: Andreas Naumann

Using recrdeptask has the side effect that every package of which a task
appears in the dependency chain, will be fully built and deployed, even
if just a subset of tasks was needed to fulfil an inter-task dependency.

For coupled packages like the native/compat ones, which can share part of
their tasks, see 2ca3a7e  dpkg-source: Build source package only once,
this leads to a full build of the base package, while e.g. only the native
part needed to be produced.

Refrain from doing so by going back to using deptask which only adds the
deploy task of the direct dependencies and rely on correct cache and
inter-task settings of the used classes/recipes.
Add rdeptask handling to allow for runtime dependency settings.

This is a partial revert of 7c7628e  rootfs: recursively depend on packages.

This fixes build failures with custom packages which dont support
crosscompiling (-native only or "all"), improves performance as no unneeded
compiling is being done, and avoids ambiguity when both packages produce the
same artifact (deb-file).

Signed-off-by: Andreas Naumann <anaumann@emlix.com>
---
 meta/classes/rootfs.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/rootfs.bbclass b/meta/classes/rootfs.bbclass
index cd8fd1188d..082a723996 100644
--- a/meta/classes/rootfs.bbclass
+++ b/meta/classes/rootfs.bbclass
@@ -234,7 +234,8 @@ do_rootfs_install[root_cleandirs] = "${ROOTFSDIR}"
 do_rootfs_install[vardeps] += "${ROOTFS_CONFIGURE_COMMAND} ${ROOTFS_INSTALL_COMMAND}"
 do_rootfs_install[vardepsexclude] += "IMAGE_ROOTFS"
 do_rootfs_install[depends] = "bootstrap-${@'target' if d.getVar('ROOTFS_ARCH') == d.getVar('DISTRO_ARCH') else 'host'}:do_build"
-do_rootfs_install[recrdeptask] = "do_deploy_deb"
+do_rootfs_install[deptask] = "do_deploy_deb"
+do_rootfs_install[rdeptask] = "do_deploy_deb"
 do_rootfs_install[network] = "${TASK_USE_SUDO}"
 python do_rootfs_install() {
     configure_cmds = (d.getVar("ROOTFS_CONFIGURE_COMMAND") or "").split()
-- 
2.43.0

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/20251103154223.985951-2-anaumann%40emlix.com.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 2/2] DONOTMERGE: testcase for transient native dependency
  2025-11-03 15:42 [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain 'Andreas Naumann' via isar-users
  2025-11-03 15:42 ` [PATCH 1/2] rootfs: " 'Andreas Naumann' via isar-users
@ 2025-11-03 15:42 ` 'Andreas Naumann' via isar-users
  2025-11-03 15:52   ` 'MOESSBAUER, Felix' via isar-users
  2025-11-07 12:00 ` [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain Zhihang Wei
  2 siblings, 1 reply; 6+ messages in thread
From: 'Andreas Naumann' via isar-users @ 2025-11-03 15:42 UTC (permalink / raw)
  To: isar-users; +Cc: Andreas Naumann

Create a dependency chain as follows:
image -> native all package (adduser) -> native any package (gss)
Since gss is broken for crosscompile atm, this will trigger a build failure
if the -native variant is not handed down the dependency chain properly and
somehow ends up building gss for the target.

Signed-off-by: Andreas Naumann <anaumann@emlix.com>
---
 meta-isar/recipes-app/adduser/adduser.bb         | 6 ++++++
 meta-isar/recipes-app/gss/gss.bb                 | 5 +++++
 meta-isar/recipes-core/images/isar-image-base.bb | 2 ++
 3 files changed, 13 insertions(+)
 create mode 100644 meta-isar/recipes-app/adduser/adduser.bb
 create mode 100644 meta-isar/recipes-app/gss/gss.bb

diff --git a/meta-isar/recipes-app/adduser/adduser.bb b/meta-isar/recipes-app/adduser/adduser.bb
new file mode 100644
index 0000000000..d513274710
--- /dev/null
+++ b/meta-isar/recipes-app/adduser/adduser.bb
@@ -0,0 +1,6 @@
+inherit dpkg
+
+DPKG_ARCH = "all"
+SRC_URI = "apt://${PN}"
+
+DEPENDS:append = "gss"
diff --git a/meta-isar/recipes-app/gss/gss.bb b/meta-isar/recipes-app/gss/gss.bb
new file mode 100644
index 0000000000..8de670406c
--- /dev/null
+++ b/meta-isar/recipes-app/gss/gss.bb
@@ -0,0 +1,5 @@
+inherit dpkg
+
+SRC_URI = "apt://${PN}"
+
+DEB_BUILD_OPTIONS += "${@ 'nocheck' if bb.utils.to_boolean(d.getVar('ISAR_CROSS_COMPILE')) else '' }"
diff --git a/meta-isar/recipes-core/images/isar-image-base.bb b/meta-isar/recipes-core/images/isar-image-base.bb
index b381d85931..d3d78e6d58 100644
--- a/meta-isar/recipes-core/images/isar-image-base.bb
+++ b/meta-isar/recipes-core/images/isar-image-base.bb
@@ -11,3 +11,5 @@ LIC_FILES_CHKSUM = "file://${LAYERDIR_core}/licenses/COPYING.GPLv2;md5=751419260
 PV = "1.0"
 
 inherit image
+
+DEPENDS += "adduser-native"
-- 
2.43.0

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/20251103154223.985951-3-anaumann%40emlix.com.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/2] DONOTMERGE: testcase for transient native dependency
  2025-11-03 15:42 ` [PATCH 2/2] DONOTMERGE: testcase for transient native dependency 'Andreas Naumann' via isar-users
@ 2025-11-03 15:52   ` 'MOESSBAUER, Felix' via isar-users
  2025-11-03 17:00     ` 'Andreas Naumann' via isar-users
  0 siblings, 1 reply; 6+ messages in thread
From: 'MOESSBAUER, Felix' via isar-users @ 2025-11-03 15:52 UTC (permalink / raw)
  To: isar-users, anaumann

On Mon, 2025-11-03 at 16:42 +0100, 'Andreas Naumann' via isar-users
wrote:
> Create a dependency chain as follows:
> image -> native all package (adduser) -> native any package (gss)
> Since gss is broken for crosscompile atm, this will trigger a build failure
> if the -native variant is not handed down the dependency chain properly and
> somehow ends up building gss for the target.

Many thanks for this testcase. With that, I was able to cross-check if
my fixes of "handle DPKG_ARCH=all case for transitive deps" properly
work (I will send them soon). Feel free to add this test to the meta-
test layer so we can use it in the CI as well.

PS: This patch depends on "This sets proxy environment variables to be
passed to schroot."

Best regards,
Felix

> 
> Signed-off-by: Andreas Naumann <anaumann@emlix.com>
> ---
>  meta-isar/recipes-app/adduser/adduser.bb         | 6 ++++++
>  meta-isar/recipes-app/gss/gss.bb                 | 5 +++++
>  meta-isar/recipes-core/images/isar-image-base.bb | 2 ++
>  3 files changed, 13 insertions(+)
>  create mode 100644 meta-isar/recipes-app/adduser/adduser.bb
>  create mode 100644 meta-isar/recipes-app/gss/gss.bb
> 
> diff --git a/meta-isar/recipes-app/adduser/adduser.bb b/meta-isar/recipes-app/adduser/adduser.bb
> new file mode 100644
> index 0000000000..d513274710
> --- /dev/null
> +++ b/meta-isar/recipes-app/adduser/adduser.bb
> @@ -0,0 +1,6 @@
> +inherit dpkg
> +
> +DPKG_ARCH = "all"
> +SRC_URI = "apt://${PN}"
> +
> +DEPENDS:append = "gss"
> diff --git a/meta-isar/recipes-app/gss/gss.bb b/meta-isar/recipes-app/gss/gss.bb
> new file mode 100644
> index 0000000000..8de670406c
> --- /dev/null
> +++ b/meta-isar/recipes-app/gss/gss.bb
> @@ -0,0 +1,5 @@
> +inherit dpkg
> +
> +SRC_URI = "apt://${PN}"
> +
> +DEB_BUILD_OPTIONS += "${@ 'nocheck' if bb.utils.to_boolean(d.getVar('ISAR_CROSS_COMPILE')) else '' }"
> diff --git a/meta-isar/recipes-core/images/isar-image-base.bb b/meta-isar/recipes-core/images/isar-image-base.bb
> index b381d85931..d3d78e6d58 100644
> --- a/meta-isar/recipes-core/images/isar-image-base.bb
> +++ b/meta-isar/recipes-core/images/isar-image-base.bb
> @@ -11,3 +11,5 @@ LIC_FILES_CHKSUM = "file://${LAYERDIR_core}/licenses/COPYING.GPLv2;md5=751419260
>  PV = "1.0"
>  
>  inherit image
> +
> +DEPENDS += "adduser-native"
> -- 
> 2.43.0
> 
> -- 
> You received this message because you are subscribed to the Google Groups "isar-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
> To view this discussion visit https://groups.google.com/d/msgid/isar-users/20251103154223.985951-3-anaumann%40emlix.com.

-- 
Siemens AG
Linux Expert Center
Friedrich-Ludwig-Bauer-Str. 3
85748 Garching, Germany

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/f54fb041035f57dfb84328589acc06200b99a0fb.camel%40siemens.com.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/2] DONOTMERGE: testcase for transient native dependency
  2025-11-03 15:52   ` 'MOESSBAUER, Felix' via isar-users
@ 2025-11-03 17:00     ` 'Andreas Naumann' via isar-users
  0 siblings, 0 replies; 6+ messages in thread
From: 'Andreas Naumann' via isar-users @ 2025-11-03 17:00 UTC (permalink / raw)
  To: MOESSBAUER, Felix, isar-users

Am 03.11.25 um 16:52 schrieb MOESSBAUER, Felix:
> On Mon, 2025-11-03 at 16:42 +0100, 'Andreas Naumann' via isar-users
> wrote:
>> Create a dependency chain as follows:
>> image -> native all package (adduser) -> native any package (gss)
>> Since gss is broken for crosscompile atm, this will trigger a build failure
>> if the -native variant is not handed down the dependency chain properly and
>> somehow ends up building gss for the target.
> Many thanks for this testcase. With that, I was able to cross-check if
> my fixes of "handle DPKG_ARCH=all case for transitive deps" properly
> work (I will send them soon). Feel free to add this test to the meta-
> test layer so we can use it in the CI as well.

Thanks, but once gss crosscompile gets fixed, the test is broken. Might be
better to supply a stable broken package all by ourselves.

>
> PS: This patch depends on "This sets proxy environment variables to be
> passed to schroot."

I didnt have that applied.

regards,
Andreas

>
> Best regards,
> Felix
>
>> Signed-off-by: Andreas Naumann <anaumann@emlix.com>
>> ---
>>   meta-isar/recipes-app/adduser/adduser.bb         | 6 ++++++
>>   meta-isar/recipes-app/gss/gss.bb                 | 5 +++++
>>   meta-isar/recipes-core/images/isar-image-base.bb | 2 ++
>>   3 files changed, 13 insertions(+)
>>   create mode 100644 meta-isar/recipes-app/adduser/adduser.bb
>>   create mode 100644 meta-isar/recipes-app/gss/gss.bb
>>
>> diff --git a/meta-isar/recipes-app/adduser/adduser.bb b/meta-isar/recipes-app/adduser/adduser.bb
>> new file mode 100644
>> index 0000000000..d513274710
>> --- /dev/null
>> +++ b/meta-isar/recipes-app/adduser/adduser.bb
>> @@ -0,0 +1,6 @@
>> +inherit dpkg
>> +
>> +DPKG_ARCH = "all"
>> +SRC_URI = "apt://${PN}"
>> +
>> +DEPENDS:append = "gss"
>> diff --git a/meta-isar/recipes-app/gss/gss.bb b/meta-isar/recipes-app/gss/gss.bb
>> new file mode 100644
>> index 0000000000..8de670406c
>> --- /dev/null
>> +++ b/meta-isar/recipes-app/gss/gss.bb
>> @@ -0,0 +1,5 @@
>> +inherit dpkg
>> +
>> +SRC_URI = "apt://${PN}"
>> +
>> +DEB_BUILD_OPTIONS += "${@ 'nocheck' if bb.utils.to_boolean(d.getVar('ISAR_CROSS_COMPILE')) else '' }"
>> diff --git a/meta-isar/recipes-core/images/isar-image-base.bb b/meta-isar/recipes-core/images/isar-image-base.bb
>> index b381d85931..d3d78e6d58 100644
>> --- a/meta-isar/recipes-core/images/isar-image-base.bb
>> +++ b/meta-isar/recipes-core/images/isar-image-base.bb
>> @@ -11,3 +11,5 @@ LIC_FILES_CHKSUM = "file://${LAYERDIR_core}/licenses/COPYING.GPLv2;md5=751419260
>>   PV = "1.0"
>>   
>>   inherit image
>> +
>> +DEPENDS += "adduser-native"
>> -- 
>> 2.43.0
>>
>> -- 
>> You received this message because you are subscribed to the Google Groups "isar-users" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
>> To view this discussion visit https://groups.google.com/d/msgid/isar-users/20251103154223.985951-3-anaumann%40emlix.com.

-- 
Andreas Naumann

emlix GmbH
Headquarters: Berliner Str. 12, 37073 Goettingen, Germany
Phone +49 (0)551 30664-0, e-mail info@emlix.com
District Court of Goettingen, Registry Number HR B 3160
Managing Directors: Heike Jordan, Dr. Uwe Kracke
VAT ID No. DE 205 198 055
Office Berlin: Panoramastr. 1, 10178 Berlin, Germany
Office Bonn: Bachstr. 6, 53115 Bonn, Germany
http://www.emlix.com

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/419cf849-6f72-4cbd-ae7b-c4de6f120b61%40emlix.com.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain
  2025-11-03 15:42 [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain 'Andreas Naumann' via isar-users
  2025-11-03 15:42 ` [PATCH 1/2] rootfs: " 'Andreas Naumann' via isar-users
  2025-11-03 15:42 ` [PATCH 2/2] DONOTMERGE: testcase for transient native dependency 'Andreas Naumann' via isar-users
@ 2025-11-07 12:00 ` Zhihang Wei
  2 siblings, 0 replies; 6+ messages in thread
From: Zhihang Wei @ 2025-11-07 12:00 UTC (permalink / raw)
  To: Andreas Naumann, isar-users

Hi,

the patch caused "citest.py:ReproTest.test_repro_unsigned" to fail on 
full CI,
when building target 'mc:qemuarm-bookworm:isar-image-base'.

Log follows:

11:39:04  (1/1) 
/build/isar_wzh_devel_8/13/testsuite/citest.py:ReproTest.test_repro_unsigned: 
STARTED
11:58:27  (1/1) 
/build/isar_wzh_devel_8/13/testsuite/citest.py:ReproTest.test_repro_unsigned: 
FAIL: Bitbake failed (1162.57 s)

[stdlog] 2025-11-07 10:56:41,807 avocado.test cibuilder L0320 INFO | 
NOTE: Running task 199 of 216 
(mc:qemuarm-bookworm:/build/isar_wzh_devel_8/13/meta-isar/recipes-app/hello/hello.bb:do_deploy_deb)
[stdlog] 2025-11-07 10:56:41,838 avocado.test cibuilder L0320 INFO | 
NOTE: recipe hello-1.0-r0: task do_deploy_deb: Started
[stdlog] 2025-11-07 10:56:41,882 avocado.test cibuilder L0320 INFO | 
NOTE: recipe hello-1.0-r0: task do_deploy_deb: Succeeded
[stdlog] 2025-11-07 10:57:25,345 avocado.app cibuilder L0322 ERROR| 
ERROR: mc:qemuarm-bookworm:gss-1.0-r0 do_dpkg_build: 
ExecutionError('/build/isar_wzh_devel_8/13/build/tmp/work/debian-bookworm-armhf/gss/1.0-r0/temp/run.dpkg_runbuild.1741456', 
2, None, None)
[stdlog] 2025-11-07 10:57:25,346 avocado.app cibuilder L0322 ERROR| 
ERROR: Logfile of failure stored in: 
/build/isar_wzh_devel_8/13/build/tmp/work/debian-bookworm-armhf/gss/1.0-r0/temp/log.do_dpkg_build.1741456
[stdlog] 2025-11-07 10:57:52,019 avocado.app cibuilder L0322 ERROR| 
ERROR: Task 
(mc:qemuarm-bookworm:/build/isar_wzh_devel_8/13/meta-isar/recipes-app/gss/gss.bb:do_dpkg_build) 
failed with exit code '1'

In log.do_dpkg_build.1741456:
Making check in tests
make[3]: Entering directory '/<<PKGBUILDDIR>>/build/tests'
make  basic saslname krb5context \
   threadsafety.sh
make[4]: Entering directory '/<<PKGBUILDDIR>>/build/tests'
gcc -DHAVE_CONFIG_H -I. -I../../tests -I..  -I../lib/headers 
-I../../lib/headers -DLOCALEDIR=\"/usr/share/locale\" -Wdate-time 
-D_FORTIFY_SOURCE=2   -g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. 
-fstack-protector-strong -Wformat -Werror=format-security -c -o basic.o 
../../tests/basic.c
gcc -DHAVE_CONFIG_H -I. -I../../tests -I..  -I../lib/headers 
-I../../lib/headers -DLOCALEDIR=\"/usr/share/locale\" -Wdate-time 
-D_FORTIFY_SOURCE=2   -g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. 
-fstack-protector-strong -Wformat -Werror=format-security -c -o 
saslname.o ../../tests/saslname.c
gcc -DHAVE_CONFIG_H -I. -I../../tests -I..  -I../lib/headers 
-I../../lib/headers -DLOCALEDIR=\"/usr/share/locale\" -Wdate-time 
-D_FORTIFY_SOURCE=2   -g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. 
-fstack-protector-strong -Wformat -Werror=format-security -c -o 
krb5context.o ../../tests/krb5context.c
make[4]: Nothing to be done for '../../tests/threadsafety.sh'.
/bin/bash ../libtool  --tag=CC   --mode=link gcc   -g -O2 
-ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -Wformat 
-Werror=format-security -no-install -Wl,-z,relro -Wl,-z,now -o saslname 
saslname.o ../lib/libgss.la
/bin/bash ../libtool  --tag=CC   --mode=link gcc   -g -O2 
-ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -Wformat 
-Werror=format-security -no-install -Wl,-z,relro -Wl,-z,now -o basic 
basic.o ../lib/libgss.la
/bin/bash ../libtool  --tag=CC   --mode=link gcc   -g -O2 
-ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -Wformat 
-Werror=format-security -no-install -Wl,-z,relro -Wl,-z,now -o 
krb5context krb5context.o ../lib/libgss.la  
-L/usr/lib/arm-linux-gnueabihf -lshishi
libtool: link: gcc -g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. 
-fstack-protector-strong -Wformat -Werror=format-security -Wl,-z 
-Wl,relro -Wl,-z -Wl,now -o saslname saslname.o ../lib/.libs/libgss.so 
-Wl,-rpath -Wl,/<<PKGBUILDDIR>>/build/lib/.libs
libtool: link: gcc -g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. 
-fstack-protector-strong -Wformat -Werror=format-security -Wl,-z 
-Wl,relro -Wl,-z -Wl,now -o basic basic.o  ../lib/.libs/libgss.so 
-Wl,-rpath -Wl,/<<PKGBUILDDIR>>/build/lib/.libs
libtool: link: gcc -g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. 
-fstack-protector-strong -Wformat -Werror=format-security -Wl,-z 
-Wl,relro -Wl,-z -Wl,now -o krb5context krb5context.o 
../lib/.libs/libgss.so -L/usr/lib/arm-linux-gnueabihf -lshishi 
-Wl,-rpath -Wl,/<<PKGBUILDDIR>>/build/lib/.libs
make[4]: Leaving directory '/<<PKGBUILDDIR>>/build/tests'
make  check-TESTS
make[4]: Entering directory '/<<PKGBUILDDIR>>/build/tests'
make[5]: Entering directory '/<<PKGBUILDDIR>>/build/tests'
PASS: threadsafety.sh
PASS: saslname
PASS: basic
FAIL: krb5context
==============================================================
    GNU Generic Security Service 1.0.4: tests/test-suite.log
==============================================================

# TOTAL: 4
# PASS:  3
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

.. contents:: :depth: 2

FAIL: krb5context
=================

loop 0 init failure
gss_accept_sec_context failure
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x108C2C: main (krb5context.c:225)
==1811217==
==1811217== Use of uninitialised value of size 4
==1811217==    at 0x492AD0C: _itoa_word (_itoa.c:177)
==1811217==    by 0x4932851: __vfprintf_internal 
(vfprintf-process-arg.c:164)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108C39: main (krb5context.c:229)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x492AD12: _itoa_word (_itoa.c:177)
==1811217==    by 0x4932851: __vfprintf_internal 
(vfprintf-process-arg.c:164)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108C39: main (krb5context.c:229)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x49327B8: __vfprintf_internal 
(vfprintf-process-arg.c:174)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108C39: main (krb5context.c:229)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x49324D4: __vfprintf_internal 
(vfprintf-process-arg.c:184)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108C39: main (krb5context.c:229)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x4932500: __vfprintf_internal 
(vfprintf-process-arg.c:204)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108C39: main (krb5context.c:229)
==1811217==
loop 0 accept flag failure (32)
gss_accept_sec_context failure
gss_init_sec_context failure (2)
client gss_wrap failure
server gss_unwrap failure
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x108CDA: main (krb5context.c:344)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x49327B8: __vfprintf_internal 
(vfprintf-process-arg.c:174)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x49324D4: __vfprintf_internal 
(vfprintf-process-arg.c:184)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x49324F2: __vfprintf_internal 
(vfprintf-process-arg.c:188)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x4932500: __vfprintf_internal 
(vfprintf-process-arg.c:204)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x493250C: pad_func (vfprintf-internal.c:192)
==1811217==    by 0x493250C: __vfprintf_internal 
(vfprintf-process-arg.c:211)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x4932182: __vfprintf_internal 
(vfprintf-process-arg.c:368)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x49322A4: __vfprintf_internal 
(vfprintf-process-arg.c:392)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x486BECC: strnlen (in 
/usr/libexec/valgrind/vgpreload_memcheck-arm-linux.so)
==1811217==
==1811217== Use of uninitialised value of size 4
==1811217==    at 0x486BED8: strnlen (in 
/usr/libexec/valgrind/vgpreload_memcheck-arm-linux.so)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x486BEE4: strnlen (in 
/usr/libexec/valgrind/vgpreload_memcheck-arm-linux.so)
==1811217==
==1811217== Use of uninitialised value of size 4
==1811217==    at 0x49496E4: _IO_default_xsputn (genops.c:394)
==1811217==    by 0x49496E4: _IO_default_xsputn (genops.c:370)
==1811217==    by 0x4932DBB: outstring_func (vfprintf-internal.c:239)
==1811217==    by 0x4932DBB: __vfprintf_internal 
(vfprintf-process-arg.c:421)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x49496EE: _IO_default_xsputn (genops.c:393)
==1811217==    by 0x49496EE: _IO_default_xsputn (genops.c:370)
==1811217==    by 0x4932DBB: outstring_func (vfprintf-internal.c:239)
==1811217==    by 0x4932DBB: __vfprintf_internal 
(vfprintf-process-arg.c:421)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
==1811217== Use of uninitialised value of size 4
==1811217==    at 0x49496EE: _IO_default_xsputn (genops.c:393)
==1811217==    by 0x49496EE: _IO_default_xsputn (genops.c:370)
==1811217==    by 0x4932DBB: outstring_func (vfprintf-internal.c:239)
==1811217==    by 0x4932DBB: __vfprintf_internal 
(vfprintf-process-arg.c:421)
==1811217==    by 0x49331B3: buffered_vfprintf (vfprintf-internal.c:1748)
==1811217==    by 0x1093E1: vfprintf (stdio2.h:109)
==1811217==    by 0x1093E1: fail (utils.c:43)
==1811217==    by 0x108CFD: main (krb5context.c:346)
==1811217==
wrap+unwrap failed (4, 1065348400, GNUTLS)
==1811217== Conditional jump or move depends on uninitialised value(s)
==1811217==    at 0x4867A54: free (in 
/usr/libexec/valgrind/vgpreload_memcheck-arm-linux.so)
==1811217==
==1811217== Invalid free() / delete / delete[] / realloc()
==1811217==    at 0x4867AA4: free (in 
/usr/libexec/valgrind/vgpreload_memcheck-arm-linux.so)
==1811217==  Address 0x401e630 is 1520 bytes inside data symbol 
"_rtld_local"
==1811217==
client gss_delete_sec_context failure
server gss_delete_sec_context failure
loop 0 init failure
gss_accept_sec_context failure
gss_accept_sec_context failure
loop 1 ret_flags failure (0)
gss_init_sec_context failure (2)
client gss_wrap failure
server gss_unwrap failure
wrap+unwrap failed (4, 0, )
client gss_delete_sec_context failure
server gss_delete_sec_context failure
loop 2 ret_flags failure (0)
loop 1 init failure
gss_accept_sec_context failure
gss_accept_sec_context failure
gss_init_sec_context failure (2)
client gss_wrap failure
server gss_unwrap failure
wrap+unwrap failed (4, 0, )
client gss_delete_sec_context failure
server gss_delete_sec_context failure
GSS-API error accept_sec_context (major): No credentials were supplied, 
or the credentials were unavailable or inaccessible
GSS-API error accept_sec_context (minor): Principal in credential cache 
does not match desired name
GSS-API error accept_sec_context (major): A token was invalid
GSS-API error accept_sec_context (minor): No krb5 error
GSS-API error init_sec_context (major): No credentials were supplied, or 
the credentials were unavailable or inaccessible
GSS-API error init_sec_context (minor): Principal in credential cache 
does not match desired name
GSS-API error client wrap (major): No context has been established
GSS-API error client wrap (minor): No krb5 error
GSS-API error client wrap (major): No context has been established
GSS-API error client wrap (minor): No krb5 error
GSS-API error client delete_sec_context (major): No context has been 
established
GSS-API error client delete_sec_context (major): A parameter was malformed
GSS-API error client delete_sec_context (minor): No krb5 error
GSS-API error server delete_sec_context (major): No context has been 
established
GSS-API error server delete_sec_context (major): A parameter was malformed
GSS-API error server delete_sec_context (minor): No krb5 error
GSS-API error accept_sec_context (major): No credentials were supplied, 
or the credentials were unavailable or inaccessible
GSS-API error accept_sec_context (minor): Principal in credential cache 
does not match desired name
GSS-API error accept_sec_context (major): A token was invalid
GSS-API error accept_sec_context (minor): No krb5 error
GSS-API error init_sec_context (major): No credentials were supplied, or 
the credentials were unavailable or inaccessible
GSS-API error init_sec_context (minor): Principal in credential cache 
does not match desired name
GSS-API error client wrap (major): No context has been established
GSS-API error client wrap (minor): No krb5 error
GSS-API error client wrap (major): No context has been established
GSS-API error client wrap (minor): No krb5 error
GSS-API error client delete_sec_context (major): No context has been 
established
GSS-API error client delete_sec_context (major): A parameter was malformed
GSS-API error client delete_sec_context (minor): No krb5 error
GSS-API error server delete_sec_context (major): No context has been 
established
GSS-API error server delete_sec_context (major): A parameter was malformed
GSS-API error server delete_sec_context (minor): No krb5 error
GSS-API error accept_sec_context (major): No credentials were supplied, 
or the credentials were unavailable or inaccessible
GSS-API error accept_sec_context (minor): Principal in credential cache 
does not match desired name
GSS-API error accept_sec_context (major): A token was invalid
GSS-API error accept_sec_context (minor): No krb5 error
GSS-API error init_sec_context (major): A token was invalid
GSS-API error init_sec_context (minor): No krb5 error
GSS-API error client wrap (major): No context has been established
GSS-API error client wrap (minor): No krb5 error
GSS-API error client wrap (major): No context has been established
GSS-API error client wrap (minor): No krb5 error
GSS-API error client delete_sec_context (major): No context has been 
established
GSS-API error client delete_sec_context (major): A parameter was malformed
GSS-API error client delete_sec_context (minor): No krb5 error
GSS-API error server delete_sec_context (major): No context has been 
established
GSS-API error server delete_sec_context (major): A parameter was malformed
GSS-API error server delete_sec_context (minor): No krb5 error
FAIL krb5context (exit status: 1)

============================================================================
Testsuite summary for GNU Generic Security Service 1.0.4
============================================================================
# TOTAL: 4
# PASS:  3
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0
============================================================================
See tests/test-suite.log
Please report to bug-gss@gnu.org
============================================================================
make[5]: *** [Makefile:1839: test-suite.log] Error 1
make[5]: Leaving directory '/<<PKGBUILDDIR>>/build/tests'
make[4]: *** [Makefile:1947: check-TESTS] Error 2
make[4]: Leaving directory '/<<PKGBUILDDIR>>/build/tests'
make[3]: *** [Makefile:2034: check-am] Error 2
make[3]: Leaving directory '/<<PKGBUILDDIR>>/build/tests'
make[2]: *** [Makefile:1557: check-recursive] Error 1
make[2]: Leaving directory '/<<PKGBUILDDIR>>/build'
make[1]: *** [Makefile:1857: check] Error 2
make[1]: Leaving directory '/<<PKGBUILDDIR>>/build'
dh_auto_test: error: cd build && make -j32 check "TESTSUITEFLAGS=-j32 
--verbose" VERBOSE=1 returned exit code 2
make: *** [debian/rules:9: binary] Error 25
dpkg-buildpackage: error: debian/rules binary subprocess returned exit 
status 2



To redo the test using avocado:
1. Have a clean clone of isar, checkout to branch next and apply your 
patches:
$ git clone -b next https://github.com/ilbers/isar.git
$ cd isar
$ git am /path-to/0001-my-contribution-to-isar.patch
2. Disable unrelated targets to make error appears faster, by applying
the following diff to "testsuite/citest.py":

diff --git a/testsuite/citest.py b/testsuite/citest.py
index a1214e9c..f81f46a3 100755
--- a/testsuite/citest.py
+++ b/testsuite/citest.py
@@ -135,7 +135,6 @@ class ReproTest(CIBaseTest):

      def test_repro_unsigned(self):
          targets = [
-            'mc:qemuamd64-bookworm:isar-image-base',
              'mc:qemuarm-bookworm:isar-image-base',
          ]

3.Run kas shell, setup CI prerequisites (avocado, qemu) and cleanup:
$ ./kas/kas-container shell kas/isar.yaml --command \
     "rm -rf /work/build/conf && /work/scripts/ci_setup.sh"
4.Run the failed test in fast:
$ cd /work/testsuite
$ avocado run citest.py:ReproTest.test_repro_unsigned$

Best regards,
Zhihang

On 11/3/25 16:42, 'Andreas Naumann' via isar-users wrote:
> This is a stripped down version of my previous
> [RFC 0/5] Improving multiarch support for arch-incompatible packages
>
> For now, I've dropped all patches except the one which covers the problem
> where targets in the dependency chain are built for both the host and the
> target architecture while only one is needed.
>
> The second patch is a way to reproduce this. It's quite hacky, and
> would need serious rework before integration into the testsuite.
>
> Andreas Naumann (2):
>    rootfs: Prevent adding unneeded package variants to recursive
>      dependency chain
>    DONOTMERGE: testcase for transient native dependency
>
>   meta-isar/recipes-app/adduser/adduser.bb         | 6 ++++++
>   meta-isar/recipes-app/gss/gss.bb                 | 5 +++++
>   meta-isar/recipes-core/images/isar-image-base.bb | 2 ++
>   meta/classes/rootfs.bbclass                      | 3 ++-
>   4 files changed, 15 insertions(+), 1 deletion(-)
>   create mode 100644 meta-isar/recipes-app/adduser/adduser.bb
>   create mode 100644 meta-isar/recipes-app/gss/gss.bb
>

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/e235ab70-7a44-4a92-b91a-71d398ad4560%40ilbers.de.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2025-11-07 12:00 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-11-03 15:42 [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain 'Andreas Naumann' via isar-users
2025-11-03 15:42 ` [PATCH 1/2] rootfs: " 'Andreas Naumann' via isar-users
2025-11-03 15:42 ` [PATCH 2/2] DONOTMERGE: testcase for transient native dependency 'Andreas Naumann' via isar-users
2025-11-03 15:52   ` 'MOESSBAUER, Felix' via isar-users
2025-11-03 17:00     ` 'Andreas Naumann' via isar-users
2025-11-07 12:00 ` [PATCH 0/2] Prevent adding unneeded package variants to recursive dependency chain Zhihang Wei

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox