| Message ID | 20260804122335.3094065-1-richard.purdie@linuxfoundation.org |
|---|---|
| State | Accepted, archived |
| Commit | 45d9c3f1a060d6d89a40837fb860b39ba44415ff |
| Headers | show |
| Series | [1/9] libva-utils: 2.23.0 -> 2.24.0 | expand |
Thank you for your submission. Patchtest identified one or more issues with the patch. Please see the log below for more information: --- Testing patch /home/patchtest/share/mboxes/1-9-libva-utils-2.23.0---2.24.0.patch FAIL: test commit message presence: Please include a commit message on your patch explaining the change (test_mbox.TestMbox.test_commit_message_presence) PASS: pretest src uri left files (test_metadata.TestMetadata.pretest_src_uri_left_files) PASS: test CVE check ignore (test_metadata.TestMetadata.test_cve_check_ignore) PASS: test Signed-off-by presence (test_mbox.TestMbox.test_signed_off_by_presence) PASS: test auh changelog truncation notice (test_mbox.TestMbox.test_auh_changelog_truncation_notice) PASS: test author valid (test_mbox.TestMbox.test_author_valid) PASS: test commit message user tags (test_mbox.TestMbox.test_commit_message_user_tags) PASS: test lic files chksum modified not mentioned (test_metadata.TestMetadata.test_lic_files_chksum_modified_not_mentioned) PASS: test max line length (test_metadata.TestMetadata.test_max_line_length) PASS: test mbox format (test_mbox.TestMbox.test_mbox_format) PASS: test non-AUH upgrade (test_mbox.TestMbox.test_non_auh_upgrade) PASS: test shortlog format (test_mbox.TestMbox.test_shortlog_format) PASS: test shortlog length (test_mbox.TestMbox.test_shortlog_length) PASS: test src uri left files (test_metadata.TestMetadata.test_src_uri_left_files) PASS: test target mailing list (test_mbox.TestMbox.test_target_mailing_list) SKIP: pretest pylint: No python related patches, skipping test (test_python_pylint.PyLint.pretest_pylint) SKIP: test CVE tag format: No new source patches introduced (test_patch.TestPatch.test_cve_tag_format) SKIP: test Signed-off-by presence: No new source patches introduced (test_patch.TestPatch.test_signed_off_by_presence) SKIP: test Upstream-Status presence: No new source patches introduced (test_patch.TestPatch.test_upstream_status_presence_format) SKIP: test bugzilla entry format: No bug ID found (test_mbox.TestMbox.test_bugzilla_entry_format) SKIP: test lic files chksum presence: No added recipes, skipping test (test_metadata.TestMetadata.test_lic_files_chksum_presence) SKIP: test license presence: No added recipes, skipping test (test_metadata.TestMetadata.test_license_presence) SKIP: test pylint: No python related patches, skipping test (test_python_pylint.PyLint.test_pylint) SKIP: test series merge on head: Merge test is disabled for now (test_mbox.TestMbox.test_series_merge_on_head) SKIP: test summary presence: No added recipes, skipping test (test_metadata.TestMetadata.test_summary_presence) --- Please address the issues identified and submit a new revision of the patch, or alternatively, reply to this email with an explanation of why the patch should be accepted. If you believe these results are due to an error in patchtest, please submit a bug at https://bugzilla.yoctoproject.org/ (use the 'Patchtest' category under 'Yocto Project Subprojects'). For more information on specific failures, see: https://wiki.yoctoproject.org/wiki/Patchtest. Thank you!
On Tue, 2026-08-04 at 13:23 +0100, Richard Purdie via lists.openembedded.org wrote: > Drop one patch applied upstream, refresh another. > Switch to xz compressed source tarball. > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > --- > .../{libtool-2.5.4.inc => libtool-2.6.2.inc} | 5 +-- > ...-cross_2.5.4.bb => libtool-cross_2.6.2.bb} | 0 > ...ative_2.5.4.bb => libtool-native_2.6.2.bb} | 0 > ...3-libtool-remove-help2man-dependency.patch | 17 ++++---- > ...r-static-libs-for-internal-compiler-.patch | 39 ------------------- > .../{libtool_2.5.4.bb => libtool_2.6.2.bb} | 0 > ...ol_2.5.4.bb => nativesdk-libtool_2.6.2.bb} | 0 > 7 files changed, 10 insertions(+), 51 deletions(-) > rename meta/recipes-devtools/libtool/{libtool-2.5.4.inc => libtool-2.6.2.inc} (85%) > rename meta/recipes-devtools/libtool/{libtool-cross_2.5.4.bb => libtool-cross_2.6.2.bb} (100%) > rename meta/recipes-devtools/libtool/{libtool-native_2.5.4.bb => libtool-native_2.6.2.bb} (100%) > delete mode 100644 meta/recipes-devtools/libtool/libtool/0005-libtool-Check-for-static-libs-for-internal-compiler-.patch > rename meta/recipes-devtools/libtool/{libtool_2.5.4.bb => libtool_2.6.2.bb} (100%) > rename meta/recipes-devtools/libtool/{nativesdk-libtool_2.5.4.bb => nativesdk-libtool_2.6.2.bb} (100%) This breaks libunwind on qemuriscv64: https://autobuilder.yoctoproject.org/valkyrie/#/builders/45/builds/2034 Cheers, Richard
On Tue, 2026-08-04 at 18:00 +0100, Richard Purdie via lists.openembedded.org wrote: > On Tue, 2026-08-04 at 13:23 +0100, Richard Purdie via > lists.openembedded.org wrote: > > Drop one patch applied upstream, refresh another. > > Switch to xz compressed source tarball. > > > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > > --- > > .../{libtool-2.5.4.inc => libtool-2.6.2.inc} | 5 +-- > > ...-cross_2.5.4.bb => libtool-cross_2.6.2.bb} | 0 > > ...ative_2.5.4.bb => libtool-native_2.6.2.bb} | 0 > > ...3-libtool-remove-help2man-dependency.patch | 17 ++++---- > > ...r-static-libs-for-internal-compiler-.patch | 39 --------------- > > ---- > > .../{libtool_2.5.4.bb => libtool_2.6.2.bb} | 0 > > ...ol_2.5.4.bb => nativesdk-libtool_2.6.2.bb} | 0 > > 7 files changed, 10 insertions(+), 51 deletions(-) > > rename meta/recipes-devtools/libtool/{libtool-2.5.4.inc => > > libtool-2.6.2.inc} (85%) > > rename meta/recipes-devtools/libtool/{libtool-cross_2.5.4.bb => > > libtool-cross_2.6.2.bb} (100%) > > rename meta/recipes-devtools/libtool/{libtool-native_2.5.4.bb => > > libtool-native_2.6.2.bb} (100%) > > delete mode 100644 meta/recipes-devtools/libtool/libtool/0005- > > libtool-Check-for-static-libs-for-internal-compiler-.patch > > rename meta/recipes-devtools/libtool/{libtool_2.5.4.bb => > > libtool_2.6.2.bb} (100%) > > rename meta/recipes-devtools/libtool/{nativesdk-libtool_2.5.4.bb > > => nativesdk-libtool_2.6.2.bb} (100%) > > This breaks libunwind on qemuriscv64: > > https://autobuilder.yoctoproject.org/valkyrie/#/builders/45/builds/2034 There are two possible fixes I found, we can either s/WCClinker/XCClinker/ on configure.ac or disable cxx exceptions (which would match aarch64/x86): diff --git a/meta/recipes-support/libunwind/libunwind_1.8.3.bb b/meta/recipes-support/libunwind/libunwind_1.8.3.bb index bc1a108e42..1a3a816d78 100644 --- a/meta/recipes-support/libunwind/libunwind_1.8.3.bb +++ b/meta/recipes-support/libunwind/libunwind_1.8.3.bb @@ -27,6 +27,8 @@ PACKAGECONFIG[zlib] = "--enable-zlibdebuginfo,--disable-zlibdebuginfo,zlib" PACKAGECONFIG[latexdocs] = "--enable-documentation, --disable-documentation, latex2man-native" EXTRA_OECONF = "--enable-static --disable-tests" +EXTRA_OECONF:append:riscv64 = " --disable-cxx-exceptions" + # http://errors.yoctoproject.org/Errors/Details/20487/ ARM_INSTRUCTION_SET:armv4 = "arm" not sure which is "right"... Cheers, Richard
Hi Richard, Actually , I also sent a patch for lib tool upgrade on master https://lists.openembedded.org/g/openembedded-core/message/242734 I reproduced it locally and dug into why. The root cause is that -WCClinker was never a valid libtool flag (only -XCClinker is). It worked in 2.5.4 by accident, but 2.6.2 widened the passthrough pattern from -Wa,*|-Werror|-Werror=* to -W* (for LTO warnings), so now -WCClinker gets passed directly to gcc which rejects it. I'd go with --disable-cxx-exceptions for riscv64 — consistent with all other arches. The -WCClinker typo can be reported upstream to libunwind separately. Regards, Jaipaul From: openembedded-core@lists.openembedded.org <openembedded-core@lists.openembedded.org> on behalf of Richard Purdie via lists.openembedded.org <richard.purdie=linuxfoundation.org@lists.openembedded.org> Date: Tuesday, 4 August 2026 at 22:13 To: openembedded-core@lists.openembedded.org <openembedded-core@lists.openembedded.org> Cc: Khem Raj <raj.khem@gmail.com> Subject: Re: [OE-core] [PATCH 5/9] libtool: 2.5.4 -> 2.6.2 On Tue, 2026-08-04 at 18:00 +0100, Richard Purdie via lists.openembedded.org wrote: > On Tue, 2026-08-04 at 13:23 +0100, Richard Purdie via > lists.openembedded.org wrote: > > Drop one patch applied upstream, refresh another. > > Switch to xz compressed source tarball. > > > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > > --- > > .../{libtool-2.5.4.inc => libtool-2.6.2.inc} | 5 +-- > > ...-cross_2.5.4.bb => libtool-cross_2.6.2.bb} | 0 > > ...ative_2.5.4.bb => libtool-native_2.6.2.bb} | 0 > > ...3-libtool-remove-help2man-dependency.patch | 17 ++++---- > > ...r-static-libs-for-internal-compiler-.patch | 39 --------------- > > ---- > > .../{libtool_2.5.4.bb => libtool_2.6.2.bb} | 0 > > ...ol_2.5.4.bb => nativesdk-libtool_2.6.2.bb} | 0 > > 7 files changed, 10 insertions(+), 51 deletions(-) > > rename meta/recipes-devtools/libtool/{libtool-2.5.4.inc => > > libtool-2.6.2.inc} (85%) > > rename meta/recipes-devtools/libtool/{libtool-cross_2.5.4.bb => > > libtool-cross_2.6.2.bb} (100%) > > rename meta/recipes-devtools/libtool/{libtool-native_2.5.4.bb => > > libtool-native_2.6.2.bb} (100%) > > delete mode 100644 meta/recipes-devtools/libtool/libtool/0005- > > libtool-Check-for-static-libs-for-internal-compiler-.patch > > rename meta/recipes-devtools/libtool/{libtool_2.5.4.bb => > > libtool_2.6.2.bb} (100%) > > rename meta/recipes-devtools/libtool/{nativesdk-libtool_2.5.4.bb > > => nativesdk-libtool_2.6.2.bb} (100%) > > This breaks libunwind on qemuriscv64: > > https://autobuilder.yoctoproject.org/valkyrie/#/builders/45/builds/2034 There are two possible fixes I found, we can either s/WCClinker/XCClinker/ on configure.ac or disable cxx exceptions (which would match aarch64/x86): diff --git a/meta/recipes-support/libunwind/libunwind_1.8.3.bb b/meta/recipes-support/libunwind/libunwind_1.8.3.bb index bc1a108e42..1a3a816d78 100644 --- a/meta/recipes-support/libunwind/libunwind_1.8.3.bb +++ b/meta/recipes-support/libunwind/libunwind_1.8.3.bb @@ -27,6 +27,8 @@ PACKAGECONFIG[zlib] = "--enable-zlibdebuginfo,--disable-zlibdebuginfo,zlib" PACKAGECONFIG[latexdocs] = "--enable-documentation, --disable-documentation, latex2man-native" EXTRA_OECONF = "--enable-static --disable-tests" +EXTRA_OECONF:append:riscv64 = " --disable-cxx-exceptions" + # http://errors.yoctoproject.org/Errors/Details/20487/ ARM_INSTRUCTION_SET:armv4 = "arm" not sure which is "right"... Cheers, Richard
On Tue, Aug 4, 2026 at 1:13 PM Richard Purdie via lists.openembedded.org <richard.purdie=linuxfoundation.org@lists.openembedded.org> wrote: > On Tue, 2026-08-04 at 18:00 +0100, Richard Purdie via > lists.openembedded.org wrote: > > On Tue, 2026-08-04 at 13:23 +0100, Richard Purdie via > > lists.openembedded.org wrote: > > > Drop one patch applied upstream, refresh another. > > > Switch to xz compressed source tarball. > > > > > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > > > --- > > > .../{libtool-2.5.4.inc => libtool-2.6.2.inc} | 5 +-- > > > ...-cross_2.5.4.bb => libtool-cross_2.6.2.bb} | 0 > > > ...ative_2.5.4.bb => libtool-native_2.6.2.bb} | 0 > > > ...3-libtool-remove-help2man-dependency.patch | 17 ++++---- > > > ...r-static-libs-for-internal-compiler-.patch | 39 --------------- > > > ---- > > > .../{libtool_2.5.4.bb => libtool_2.6.2.bb} | 0 > > > ...ol_2.5.4.bb => nativesdk-libtool_2.6.2.bb} | 0 > > > 7 files changed, 10 insertions(+), 51 deletions(-) > > > rename meta/recipes-devtools/libtool/{libtool-2.5.4.inc => > > > libtool-2.6.2.inc} (85%) > > > rename meta/recipes-devtools/libtool/{libtool-cross_2.5.4.bb => > > > libtool-cross_2.6.2.bb} (100%) > > > rename meta/recipes-devtools/libtool/{libtool-native_2.5.4.bb => > > > libtool-native_2.6.2.bb} (100%) > > > delete mode 100644 meta/recipes-devtools/libtool/libtool/0005- > > > libtool-Check-for-static-libs-for-internal-compiler-.patch > > > rename meta/recipes-devtools/libtool/{libtool_2.5.4.bb => > > > libtool_2.6.2.bb} (100%) > > > rename meta/recipes-devtools/libtool/{nativesdk-libtool_2.5.4.bb > > > => nativesdk-libtool_2.6.2.bb} (100%) > > > > This breaks libunwind on qemuriscv64: > > > > https://autobuilder.yoctoproject.org/valkyrie/#/builders/45/builds/2034 > > There are two possible fixes I found, we can either > s/WCClinker/XCClinker/ on configure.ac or disable cxx exceptions (which > would match aarch64/x86): > Lets disable cxx exceptions. > > diff --git a/meta/recipes-support/libunwind/libunwind_1.8.3.bb > b/meta/recipes-support/libunwind/libunwind_1.8.3.bb > index bc1a108e42..1a3a816d78 100644 > --- a/meta/recipes-support/libunwind/libunwind_1.8.3.bb > +++ b/meta/recipes-support/libunwind/libunwind_1.8.3.bb > @@ -27,6 +27,8 @@ PACKAGECONFIG[zlib] = > "--enable-zlibdebuginfo,--disable-zlibdebuginfo,zlib" > PACKAGECONFIG[latexdocs] = "--enable-documentation, > --disable-documentation, latex2man-native" > > EXTRA_OECONF = "--enable-static --disable-tests" > +EXTRA_OECONF:append:riscv64 = " --disable-cxx-exceptions" > + > > # http://errors.yoctoproject.org/Errors/Details/20487/ > ARM_INSTRUCTION_SET:armv4 = "arm" > > not sure which is "right"... > > Cheers, > > Richard > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#242802): > https://lists.openembedded.org/g/openembedded-core/message/242802 > Mute This Topic: https://lists.openembedded.org/mt/120600439/1997914 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [ > raj.khem@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- > >
On Tue, 2026-08-04 at 21:36 +0000, Jaipaul Cheernam wrote: > Actually , I also sent a patch for lib tool upgrade on > masterhttps://lists.openembedded.org/g/openembedded-core/message/2427 > 34 > > I reproduced it locally and dug into why. The root cause is that - > WCClinker was never a valid libtool flag (only -XCClinker is). > It worked in 2.5.4 by accident, but 2.6.2 widened the passthrough > pattern from -Wa,*|-Werror|-Werror=* to -W* (for LTO warnings), > so now -WCClinker gets passed directly to gcc which rejects it. It was fixed upstream here: https://github.com/libunwind/libunwind/commit/8e6a348b94a1b8e77ec2a086ff27ea9b78357d71 > I'd go with --disable-cxx-exceptions for riscv64 — consistent with > all other arches. The -WCClinker typo can be reported upstream to > libunwind separately. > and upstream do unconditionally disable it now: https://github.com/libunwind/libunwind/commit/bda7a2267b8385b869b4eef6a2fcdd696b887ed8 :) Cheers, Richard
diff --git a/meta/recipes-graphics/libva/libva-utils_2.23.0.bb b/meta/recipes-graphics/libva/libva-utils_2.24.0.bb similarity index 90% rename from meta/recipes-graphics/libva/libva-utils_2.23.0.bb rename to meta/recipes-graphics/libva/libva-utils_2.24.0.bb index 970ff50ac95..b3fd521ff13 100644 --- a/meta/recipes-graphics/libva/libva-utils_2.23.0.bb +++ b/meta/recipes-graphics/libva/libva-utils_2.24.0.bb @@ -14,8 +14,8 @@ SECTION = "x11" LICENSE = "MIT" LIC_FILES_CHKSUM = "file://COPYING;md5=b148fc8adf19dc9aec17cf9cd29a9a5e" -SRC_URI = "git://github.com/intel/libva-utils.git;branch=v2.23-branch;protocol=https;tag=${PV}" -SRCREV = "5896b6779d36f83bf483aff62da8aae30ba8d664" +SRC_URI = "git://github.com/intel/libva-utils.git;branch=v2.24-branch;protocol=https;tag=${PV}" +SRCREV = "c3ba9e07e61841f1bc2c9932d957b44e5827a852" UPSTREAM_CHECK_GITTAGREGEX = "(?P<pver>(\d+(\.\d+)+))$"
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> --- .../libva/{libva-utils_2.23.0.bb => libva-utils_2.24.0.bb} | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) rename meta/recipes-graphics/libva/{libva-utils_2.23.0.bb => libva-utils_2.24.0.bb} (90%)