Re: [oe][OE-core][Patch 0/1] Revert "bin_package.bbclass: Inhibit the default dependencies"

2023-10-06 Thread Peter Kjellerstedt
> -Original Message-
> From: openembedded-core@lists.openembedded.org 
>  On Behalf Of Peter Kjellerstedt
> Sent: den 31 augusti 2023 19:28
> To: Ryan Eatmon ; Max Krummenacher ; 
> openembedded-core@lists.openembedded.org
> Cc: Max Krummenacher ; Randolph Sapp 
> 
> Subject: Re: [oe][OE-core][Patch 0/1] Revert "bin_package.bbclass: Inhibit 
> the default dependencies"
> 
> > -Original Message-
> > From: Ryan Eatmon 
> > Sent: den 28 augusti 2023 19:10
> > To: Peter Kjellerstedt ; Max Krummenacher 
> > ; openembedded-core@lists.openembedded.org
> > Cc: Max Krummenacher ; Randolph Sapp 
> > 
> > Subject: Re: [oe][OE-core][Patch 0/1] Revert "bin_package.bbclass: Inhibit 
> > the default dependencies"
> >
> > On 8/27/2023 4:23 PM, Peter Kjellerstedt wrote:
> > >> -Original Message-
> > >> From: Max Krummenacher 
> > >> Sent: den 27 augusti 2023 10:10
> > >> To: openembedded-core@lists.openembedded.org; Peter Kjellerstedt
> > >> 
> > >> Cc: Max Krummenacher ; Randolph Sapp
> > >> 
> > >> Subject: [oe][OE-core][Patch 0/1] Revert "bin_package.bbclass: Inhibit 
> > >> the
> > >> default dependencies"
> > >>
> > >> From: Max Krummenacher 
> > >>
> > >> Hi
> > >>
> > >> With commit d1d09bd4d7 ("bin_package.bbclass: Inhibit the default
> > >> dependencies") applied I'm getting a lot of these errors, i.e. qa
> > >> does miss libc and compiler provided libs:
> > >>
> > >> ERROR: ti-img-rogue-umlibs-23.1.6404501-r2 do_package_qa: QA Issue:
> > >> /usr/lib/libusc.so.23.1.6404501 contained in package ti-img-rogue-umlibs
> > >> requires ld-linux-aarch64.so.1(GLIBC_2.17)(64bit), but no providers found
> > >> in RDEPENDS:ti-img-rogue-umlibs? [file-rdeps]
> > >> ERROR: ti-img-rogue-umlibs-23.1.6404501-r2 do_package_qa: QA Issue:
> > >> /usr/lib/libusc.so.23.1.6404501 contained in package ti-img-rogue-umlibs
> > >> requires libc.so.6(GLIBC_2.17)(64bit), but no providers found in
> > >> RDEPENDS:ti-img-rogue-umlibs? [file-rdeps]
> > >> ERROR: ti-img-rogue-umlibs-23.1.6404501-r2 do_package_qa: QA Issue:
> > >> /usr/lib/libufwriter.so.23.1.6404501 contained in package ti-img-rogue-
> > >> umlibs requires libstdc++.so.6(GLIBCXX_3.4.14)(64bit), but no providers
> > >> found in RDEPENDS:ti-img-rogue-umlibs? [file-rdeps]
> > >>
> > >> Reverting the commit makes the build pass, alternatively adding
> > >> to depends in the recipe which is using the bin_package class
> > >> fixes it too:
> > >>
> > >> DEPENDS += " virtual/${TARGET_PREFIX}compilerlibs virtual/libc"
> > >>
> > >> I'd prefer reverting removing the default dependencies over fixing
> > >> each of the recipes which do use the bin_package class to actually
> > >> install binaries running in the target user space.
> > >>
> > >> Any opinions?
> > >
> > > Bummer. I guess we will have to update our recipes individually
> > > instead. :(
> >
> > Was there some issue that your patch was seeking to solve?  There was
> > not much explanation in your patch or discussion about it on the mailing
> > list before it was accepted.
> >
> > Or did this just seem like something to do since this class doesn't
> > build anything?
> 
> It just seemed logical that since nothing is built, there should be
> no need for the compiler. I did, however, miss the potential need for
> the runtime libraries.
> 
> > Just looking for background.
> >
> > Your commit is also the source of another error with this the same
> > ti-img-rogue-umlibs recipe that I've been trying to track down all last
> > week.  Max just beat me to finding it.
> >
> > I'm voting to revert your patch unless there is compelling reason for
> > your patch.
> 
> If it was not obvious from my response above, I am in favor of reverting
> my change since the errors reported by Max obviously was not part of my
> use case and did not affect any of our recipes.
> 
> //Peter

Please apply Max' revert. I thought it was applied back in August when 
this was discussed, but it apparently never happened.

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188795): 
https://lists.openembedded.org/g/openembedded-core/message/188795
Mute This Topic: https://lists.openembedded.org/mt/100987453/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core][PATCH v2 1/3] rust-cross-canadian: set CARGO_TARGET__RUSTFLAGS

2023-10-06 Thread Alexandre Belloni via lists.openembedded.org
Hello,

I believe this series causes this kind of failures:

https://autobuilder.yoctoproject.org/typhoon/#/builders/73/builds/7857/steps/17/logs/stdio

  File 
"/home/pokybuild/yocto-worker/qemux86-64/build/meta/lib/oeqa/sdk/cases/rust.py",
 line 57, in test_cargo_build
self._run('cd %s/hello; cargo run --target %s-gnu' % (self.tc.sdk_dir, 
sdksys))
  File 
"/home/pokybuild/yocto-worker/qemux86-64/build/meta/lib/oeqa/sdk/case.py", line 
14, in _run
return subprocess.check_output(". %s > /dev/null; %s;" % \
   ^^^
  File 
"/home/pokybuild/yocto-worker/qemux86-64/build/buildtools/sysroots/x86_64-pokysdk-linux/usr/lib/python3.11/subprocess.py",
 line 466, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
   ^
  File 
"/home/pokybuild/yocto-worker/qemux86-64/build/buildtools/sysroots/x86_64-pokysdk-linux/usr/lib/python3.11/subprocess.py",
 line 571, in run
raise CalledProcessError(retcode, process.args,
oeqa.utils.subprocesstweak.OETestCalledProcessError: Command '. 
/home/pokybuild/yocto-worker/qemux86-64/build/build/tmp/work/qemux86_64-poky-linux/core-image-sato/1.0/testimage-sdk/environment-setup-core2-64-poky-linux
 > /dev/null; cd 
/home/pokybuild/yocto-worker/qemux86-64/build/build/tmp/work/qemux86_64-poky-linux/core-image-sato/1.0/testimage-sdk//hello;
 cargo run --target i686-pokysdk-linux-gnu;' returned non-zero exit status 101
Standard Output: Finished dev [unoptimized + debuginfo] target(s) in 0.03s
 Running 
`/home/pokybuild/yocto-worker/qemux86-64/build/build/tmp/work/qemux86_64-poky-linux/core-image-sato/1.0/testimage-sdk/sysroots/i686-pokysdk-linux/lib/ld-linux-i686.so.2
 target/i686-pokysdk-linux-gnu/debug/hello`
error: could not execute process 
`/home/pokybuild/yocto-worker/qemux86-64/build/build/tmp/work/qemux86_64-poky-linux/core-image-sato/1.0/testimage-sdk/sysroots/i686-pokysdk-linux/lib/ld-linux-i686.so.2
 target/i686-pokysdk-linux-gnu/debug/hello` (never executed)
Caused by:
  No such file or directory (os error 2)


On 03/10/2023 09:06:28+0200, Sean Nyekjaer wrote:
> Avoid setting sdk-wide RUSTFLAGS as these flags only are valid when
> building for target.
> This will enable building for different targets with different
> RUSTFLAGS.
> 
> Signed-off-by: Sean Nyekjaer 
> ---
>  meta/recipes-devtools/rust/rust-cross-canadian.inc | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-devtools/rust/rust-cross-canadian.inc 
> b/meta/recipes-devtools/rust/rust-cross-canadian.inc
> index d569ac59f5..d25646237c 100644
> --- a/meta/recipes-devtools/rust/rust-cross-canadian.inc
> +++ b/meta/recipes-devtools/rust/rust-cross-canadian.inc
> @@ -54,8 +54,10 @@ do_install () {
>  mkdir "${ENV_SETUP_DIR}"
>  RUST_ENV_SETUP_SH="${ENV_SETUP_DIR}/rust.sh"
>  
> +RUST_TARGET_TRIPLE=`echo ${RUST_TARGET_SYS} | tr '[:lower:]' '[:upper:]' 
> | sed 's/-/_/g'`
> +
>  cat <<- EOF > "${RUST_ENV_SETUP_SH}"
> - export RUSTFLAGS="--sysroot=\$OECORE_TARGET_SYSROOT/usr -C 
> link-arg=--sysroot=\$OECORE_TARGET_SYSROOT"
> + export 
> CARGO_TARGET_${RUST_TARGET_TRIPLE}_RUSTFLAGS="--sysroot=\$OECORE_TARGET_SYSROOT/usr
>  -C link-arg=--sysroot=\$OECORE_TARGET_SYSROOT"
>   export 
> RUST_TARGET_PATH="\$OECORE_NATIVE_SYSROOT/usr/lib/${TARGET_SYS}/rustlib"
>   EOF
>  
> -- 
> 2.42.0
> 

> 
> 
> 


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188794): 
https://lists.openembedded.org/g/openembedded-core/message/188794
Mute This Topic: https://lists.openembedded.org/mt/101729592/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2] systemd: sysusersd: change how the root home directory is set

2023-10-06 Thread Alexandre Belloni via lists.openembedded.org
Hello,

This causes the following warnings:

https://autobuilder.yoctoproject.org/typhoon/#/builders/106/builds/6888/steps/11/logs/warnings

On 05/10/2023 22:55:08+0200, Louis Rannou wrote:
> From: Louis Rannou 
> 
> Modifying the sources brings troubles in the devtool command. Therefore it is
> better to patch the source with a custom variable, and later replace it with 
> the
> correct root path.
> 
> Suggestion from :
> 
> 
> Signed-off-by: Louis Rannou 
> Suggested-by: Peter Kjellerstedt 
> Signed-off-by: Louis Rannou 
> ---
>  ...usersd-Change-the-user-root-s-home-direct.patch | 31 
> ++
>  meta/recipes-core/systemd/systemd_254.4.bb | 11 
>  2 files changed, 37 insertions(+), 5 deletions(-)
> 
> diff --git 
> a/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
>  
> b/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
> new file mode 100644
> index 00..dd0d96d11b
> --- /dev/null
> +++ 
> b/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
> @@ -0,0 +1,31 @@
> +From bf97001978cdefad644f7b4b909f281368e5a4dd Mon Sep 17 00:00:00 2001
> +From: Kas User 
> +Date: Thu, 27 Jul 2023 08:23:06 +
> +Subject: [PATCH] sysusers.d: Change the user root's home directory
> +
> +The default sysusers basic.conf.in file sets the root home directory to 
> `/root`
> +and does not permit its configuration. Change this to `:ROOT_HOME:` which 
> must
> +be set before the installation.
> +
> +The upstream considers the root home directory should not be changed
> +
> +
> +Upstream-Status: Denied
> +Signed-off-by: Louis Rannou 
> +---
> + sysusers.d/basic.conf.in | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/sysusers.d/basic.conf.in b/sysusers.d/basic.conf.in
> +index 0aec080a4c..f26c224341 100644
> +--- a/sysusers.d/basic.conf.in
>  b/sysusers.d/basic.conf.in
> +@@ -7,7 +7,7 @@
> + 
> + # The superuser
> + g root0   --
> +-u root0:0 "Super User" /root
> ++u root0:0 "Super User" :ROOT_HOME:
> + 
> + # The nobody user/group for NFS file systems
> + g {{NOBODY_GROUP_NAME}} 65534   --
> diff --git a/meta/recipes-core/systemd/systemd_254.4.bb 
> b/meta/recipes-core/systemd/systemd_254.4.bb
> index 77724eb822..3adf62935d 100644
> --- a/meta/recipes-core/systemd/systemd_254.4.bb
> +++ b/meta/recipes-core/systemd/systemd_254.4.bb
> @@ -21,7 +21,6 @@ REQUIRED_DISTRO_FEATURES += "systemd"
>  SRC_URI += " \
> file://touchscreen.rules \
> file://00-create-volatile.conf \
> -   file://basic.conf.in \
> ${@bb.utils.contains('PACKAGECONFIG', 
> 'polkit_hostnamed_fallback', 
> 'file://org.freedesktop.hostname1_no_polkit.conf', '', d)} \
> ${@bb.utils.contains('PACKAGECONFIG', 
> 'polkit_hostnamed_fallback', 'file://00-hostnamed-network-user.conf', '', d)} 
> \
> file://init \
> @@ -30,6 +29,7 @@ SRC_URI += " \
> 
> file://0002-binfmt-Don-t-install-dependency-links-at-install-tim.patch \
> file://0008-implment-systemd-sysv-install-for-OE.patch \
> 
> file://0004-Move-sysusers.d-sysctl.d-binfmt.d-modules-load.d-to-.patch \
> +   file://0029-sysusersd-Change-the-user-root-s-home-direct.patch \
> "
>  
>  # patches needed by musl
> @@ -260,10 +260,6 @@ EXTRA_OEMESON += "-Dkexec-path=${sbindir}/kexec \
>  # The 60 seconds is watchdog's default vaule.
>  WATCHDOG_TIMEOUT ??= "60"
>  
> -do_configure:prepend() {
> -  sed s@:ROOT_HOME:@${ROOT_HOME}@g ${WORKDIR}/basic.conf.in > 
> ${S}/sysusers.d/basic.conf.in
> -}
> -
>  do_install() {
>   meson_do_install
>   install -d ${D}/${base_sbindir}
> @@ -375,6 +371,11 @@ do_install() {
>  sed -i -e 
> 's/#RebootWatchdogSec=10min/RebootWatchdogSec=${WATCHDOG_TIMEOUT}/' \
>  ${D}/${sysconfdir}/systemd/system.conf
>  fi
> +
> + # The root home directory specified in sysusers.d/ was patched to
> + # `:ROOT_HOME:`. Change now to the correct path given by ${ROOT_HOME}.
> + [ ! -f ${D}${libdir}/sysusers.d/basic.conf ] ||
> + sed -i s@:ROOT_HOME:@${ROOT_HOME}@g 
> ${D}${libdir}/sysusers.d/basic.conf
>  }
>  
>  python populate_packages:prepend (){
> 
> ---
> base-commit: 095021ac61c1df357e5f1868badd38038004317f
> change-id: 20231005-sysusers3-f448703f294f
> 
> Best regards,
> -- 
> Louis Rannou 
> 

> 
> 
> 


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188793): 
https://lists.openembedded.org/g/openembedded-core/message/188793
Mute This Topic: 

Re: [OE-core] [PATCH V3] strace: skip so_peerpidfd test

2023-10-06 Thread Randy MacLeod via lists.openembedded.org

On 2023-10-06 5:27 p.m., Randy MacLeod via lists.openembedded.org wrote:

From: Randy MacLeod

The so_peerpidfd test is failing in every linux-yocto-6.5 ptest run
due to a difference of square brackets in part of the log file.

Skip the failing test until the difference is understood and fixed.

Signed-off-by: Randy MacLeod
---
  .../skip-test-so_peerpidfd.gen.test.patch | 25 +++
  meta/recipes-devtools/strace/strace_6.5.bb|  1 +
  2 files changed, 26 insertions(+)
  create mode 100644 
meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch

diff --git 
a/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch 
b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
new file mode 100644
index 00..76f93ab613
--- /dev/null
+++ b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
@@ -0,0 +1,25 @@
+From 002d9f2512245536dfc8d62db429d97e2216ec3a Mon Sep 17 00:00:00 2001
+From: Randy MacLeod
+Date: Fri, 6 Oct 2023 12:08:23 -0700
+Subject: [PATCH] skip tests/so_peerpidfd.gen.test
+
+Upstream-Status: Pending


I updated the commit log but forgot to git add the changes I made to the 
patch!

Sigh.


Richard took pity on me and said that he'd fixed the Upstream-Status 
before merging.


Happy Canadian Thanksgiving weekend everyone,

../Randy


+
+Signed-off-by: Randy MacLeod
+---
+ tests/so_peerpidfd.gen.test | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/tests/so_peerpidfd.gen.test b/tests/so_peerpidfd.gen.test
+index 64ad3a2..f89da9f 100755
+--- a/tests/so_peerpidfd.gen.test
 b/tests/so_peerpidfd.gen.test
+@@ -1,4 +1,5 @@
+ #!/bin/sh -efu
+ # Generated by ./tests/gen_tests.sh from ./tests/gen_tests.in (so_peerpidfd 
--trace=getsockopt -y); do not edit.
+ . "${srcdir=.}/init.sh"
++skip_ "Test fails due to apparently trivial log format differences"
+ run_strace_match_diff --trace=getsockopt -y
+--
+2.39.0
+
diff --git a/meta/recipes-devtools/strace/strace_6.5.bb 
b/meta/recipes-devtools/strace/strace_6.5.bb
index 8f90c35925..d6475e8db9 100644
--- a/meta/recipes-devtools/strace/strace_6.5.bb
+++ b/meta/recipes-devtools/strace/strace_6.5.bb
@@ -14,6 +14,7 @@ SRC_URI ="https://strace.io/files/${PV}/strace-${PV}.tar.xz \ 
file://skip-load.patch \ 
file://0001-configure-Use-autoconf-macro-to-detect-largefile-sup.patch 
\ file://0002-tests-Replace-off64_t-with-off_t.patch \ + 
file://skip-test-so_peerpidfd.gen.test.patch \ "

  SRC_URI[sha256sum] = 
"dfb051702389e1979a151892b5901afc9e93bbc1c70d84c906ade3224ca91980"
  






--
# Randy MacLeod
# Wind River Linux

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188792): 
https://lists.openembedded.org/g/openembedded-core/message/188792
Mute This Topic: https://lists.openembedded.org/mt/101807172/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH V3] strace: skip so_peerpidfd test

2023-10-06 Thread Peter Kjellerstedt
> -Original Message-
> From: openembedded-core@lists.openembedded.org 
>  On Behalf Of Randy MacLeod via
> lists.openembedded.org
> Sent: den 6 oktober 2023 23:28
> To: openembedded-core@lists.openembedded.org
> Subject: [OE-core] [PATCH V3] strace: skip so_peerpidfd test
> 
> From: Randy MacLeod 
> 
> The so_peerpidfd test is failing in every linux-yocto-6.5 ptest run
> due to a difference of square brackets in part of the log file.
> 
> Skip the failing test until the difference is understood and fixed.
> 
> Signed-off-by: Randy MacLeod 
> ---
>  .../skip-test-so_peerpidfd.gen.test.patch | 25 +++
>  meta/recipes-devtools/strace/strace_6.5.bb|  1 +
>  2 files changed, 26 insertions(+)
>  create mode 100644 
> meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
> 
> diff --git 
> a/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch 
> b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
> new file mode 100644
> index 00..76f93ab613
> --- /dev/null
> +++ 
> b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
> @@ -0,0 +1,25 @@
> +From 002d9f2512245536dfc8d62db429d97e2216ec3a Mon Sep 17 00:00:00 2001
> +From: Randy MacLeod 
> +Date: Fri, 6 Oct 2023 12:08:23 -0700
> +Subject: [PATCH] skip tests/so_peerpidfd.gen.test
> +
> +Upstream-Status: Pending

Wasn't that supposed to be Inappropriate? ;)

> +
> +Signed-off-by: Randy MacLeod 

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188791): 
https://lists.openembedded.org/g/openembedded-core/message/188791
Mute This Topic: https://lists.openembedded.org/mt/101807172/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] cmake: upgrade to v3.27.7

2023-10-06 Thread Sudip Mukherjee
Changes:
Update sha256sum for new version.

Signed-off-by: Sudip Mukherjee 
---
 .../cmake/{cmake-native_3.27.5.bb => cmake-native_3.27.7.bb}| 0
 meta/recipes-devtools/cmake/cmake.inc   | 2 +-
 .../recipes-devtools/cmake/{cmake_3.27.5.bb => cmake_3.27.7.bb} | 0
 3 files changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-devtools/cmake/{cmake-native_3.27.5.bb => 
cmake-native_3.27.7.bb} (100%)
 rename meta/recipes-devtools/cmake/{cmake_3.27.5.bb => cmake_3.27.7.bb} (100%)

diff --git a/meta/recipes-devtools/cmake/cmake-native_3.27.5.bb 
b/meta/recipes-devtools/cmake/cmake-native_3.27.7.bb
similarity index 100%
rename from meta/recipes-devtools/cmake/cmake-native_3.27.5.bb
rename to meta/recipes-devtools/cmake/cmake-native_3.27.7.bb
diff --git a/meta/recipes-devtools/cmake/cmake.inc 
b/meta/recipes-devtools/cmake/cmake.inc
index ef4eec5ab1..ecb0e487df 100644
--- a/meta/recipes-devtools/cmake/cmake.inc
+++ b/meta/recipes-devtools/cmake/cmake.inc
@@ -19,7 +19,7 @@ CMAKE_MAJOR_VERSION = 
"${@'.'.join(d.getVar('PV').split('.')[0:2])}"
 SRC_URI = "https://cmake.org/files/v${CMAKE_MAJOR_VERSION}/cmake-${PV}.tar.gz \
 "
 
-SRC_URI[sha256sum] = 
"5175e8fe1ca9b1dd09090130db7201968bcce1595971ff9e9998c2f0765004c9"
+SRC_URI[sha256sum] = 
"08f71a106036bf051f692760ef9558c0577c42ac39e96ba097e7662bd4158d8e"
 
 UPSTREAM_CHECK_REGEX = "cmake-(?P\d+(\.\d+)+)\.tar"
 
diff --git a/meta/recipes-devtools/cmake/cmake_3.27.5.bb 
b/meta/recipes-devtools/cmake/cmake_3.27.7.bb
similarity index 100%
rename from meta/recipes-devtools/cmake/cmake_3.27.5.bb
rename to meta/recipes-devtools/cmake/cmake_3.27.7.bb
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188790): 
https://lists.openembedded.org/g/openembedded-core/message/188790
Mute This Topic: https://lists.openembedded.org/mt/101807518/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2 2/4] package_deb: Revert versioned providers workaround

2023-10-06 Thread Peter Kjellerstedt
> -Original Message-
> From: openembedded-core@lists.openembedded.org 
>  On Behalf Of Pavel Zhukov
> Sent: den 6 oktober 2023 21:10
> To: openembedded-core@lists.openembedded.org
> Cc: Pavel Zhukov 
> Subject: [OE-core] [PATCH v2 2/4] package_deb: Revert versioned providers 
> workaround
> 
> dpkg and apt seem to handle versioned provides correctly now [1] so this
> workaround is not needed anymore.
> This fixes [Yocto #14995] for package_deb.
> 
> [1]

Was there supposed to be a reference here?

> Signed-off-by: Pavel Zhukov 
> ---
>  meta/classes-global/package_deb.bbclass | 9 +
>  1 file changed, 1 insertion(+), 8 deletions(-)
> 

//Peter

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188789): 
https://lists.openembedded.org/g/openembedded-core/message/188789
Mute This Topic: https://lists.openembedded.org/mt/101804515/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH V3] strace: skip so_peerpidfd test

2023-10-06 Thread Randy MacLeod via lists.openembedded.org
From: Randy MacLeod 

The so_peerpidfd test is failing in every linux-yocto-6.5 ptest run
due to a difference of square brackets in part of the log file.

Skip the failing test until the difference is understood and fixed.

Signed-off-by: Randy MacLeod 
---
 .../skip-test-so_peerpidfd.gen.test.patch | 25 +++
 meta/recipes-devtools/strace/strace_6.5.bb|  1 +
 2 files changed, 26 insertions(+)
 create mode 100644 
meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch

diff --git 
a/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch 
b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
new file mode 100644
index 00..76f93ab613
--- /dev/null
+++ b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
@@ -0,0 +1,25 @@
+From 002d9f2512245536dfc8d62db429d97e2216ec3a Mon Sep 17 00:00:00 2001
+From: Randy MacLeod 
+Date: Fri, 6 Oct 2023 12:08:23 -0700
+Subject: [PATCH] skip tests/so_peerpidfd.gen.test
+
+Upstream-Status: Pending
+
+Signed-off-by: Randy MacLeod 
+---
+ tests/so_peerpidfd.gen.test | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/tests/so_peerpidfd.gen.test b/tests/so_peerpidfd.gen.test
+index 64ad3a2..f89da9f 100755
+--- a/tests/so_peerpidfd.gen.test
 b/tests/so_peerpidfd.gen.test
+@@ -1,4 +1,5 @@
+ #!/bin/sh -efu
+ # Generated by ./tests/gen_tests.sh from ./tests/gen_tests.in (so_peerpidfd 
--trace=getsockopt -y); do not edit.
+ . "${srcdir=.}/init.sh"
++skip_ "Test fails due to apparently trivial log format differences"
+ run_strace_match_diff --trace=getsockopt -y
+-- 
+2.39.0
+
diff --git a/meta/recipes-devtools/strace/strace_6.5.bb 
b/meta/recipes-devtools/strace/strace_6.5.bb
index 8f90c35925..d6475e8db9 100644
--- a/meta/recipes-devtools/strace/strace_6.5.bb
+++ b/meta/recipes-devtools/strace/strace_6.5.bb
@@ -14,6 +14,7 @@ SRC_URI = "https://strace.io/files/${PV}/strace-${PV}.tar.xz \
file://skip-load.patch \

file://0001-configure-Use-autoconf-macro-to-detect-largefile-sup.patch \
file://0002-tests-Replace-off64_t-with-off_t.patch \
+   file://skip-test-so_peerpidfd.gen.test.patch \
"
 SRC_URI[sha256sum] = 
"dfb051702389e1979a151892b5901afc9e93bbc1c70d84c906ade3224ca91980"
 
-- 
2.39.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188788): 
https://lists.openembedded.org/g/openembedded-core/message/188788
Mute This Topic: https://lists.openembedded.org/mt/101807172/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH V2] strace: skip so_peerpidfd test

2023-10-06 Thread Randy MacLeod via lists.openembedded.org

On 2023-10-06 5:14 p.m., Randy MacLeod via lists.openembedded.org wrote:

From: Randy MacLeod

The so_peerpidfd test is failing due to a difference of square brackets
in part of the log file. Skip the failing test until the difference is
understood and fixed.

Signed-off-by: Randy MacLeod
---
  .../skip-test-so_peerpidfd.gen.test.patch | 25 +++
  meta/recipes-devtools/strace/strace_6.5.bb|  1 +
  2 files changed, 26 insertions(+)
  create mode 100644 
meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch

diff --git 
a/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch 
b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
new file mode 100644
index 00..76f93ab613
--- /dev/null
+++ b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
@@ -0,0 +1,25 @@
+From 002d9f2512245536dfc8d62db429d97e2216ec3a Mon Sep 17 00:00:00 2001
+From: Randy MacLeod
+Date: Fri, 6 Oct 2023 12:08:23 -0700
+Subject: [PATCH] skip tests/so_peerpidfd.gen.test
+
+Upstream-Status: Pending


Sigh, the status should be Inappropriate, v3 coming!

I do plan to fix this upstream but this work-around is inappropriate for 
upstream.


../Randy


+
+Signed-off-by: Randy MacLeod
+---
+ tests/so_peerpidfd.gen.test | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/tests/so_peerpidfd.gen.test b/tests/so_peerpidfd.gen.test
+index 64ad3a2..f89da9f 100755
+--- a/tests/so_peerpidfd.gen.test
 b/tests/so_peerpidfd.gen.test
+@@ -1,4 +1,5 @@
+ #!/bin/sh -efu
+ # Generated by ./tests/gen_tests.sh from ./tests/gen_tests.in (so_peerpidfd 
--trace=getsockopt -y); do not edit.
+ . "${srcdir=.}/init.sh"
++skip_ "Test fails due to apparently trivial log format differences"
+ run_strace_match_diff --trace=getsockopt -y
+--
+2.39.0
+
diff --git a/meta/recipes-devtools/strace/strace_6.5.bb 
b/meta/recipes-devtools/strace/strace_6.5.bb
index 8f90c35925..d6475e8db9 100644
--- a/meta/recipes-devtools/strace/strace_6.5.bb
+++ b/meta/recipes-devtools/strace/strace_6.5.bb
@@ -14,6 +14,7 @@ SRC_URI ="https://strace.io/files/${PV}/strace-${PV}.tar.xz \ 
file://skip-load.patch \ 
file://0001-configure-Use-autoconf-macro-to-detect-largefile-sup.patch 
\ file://0002-tests-Replace-off64_t-with-off_t.patch \ + 
file://skip-test-so_peerpidfd.gen.test.patch \ "

  SRC_URI[sha256sum] = 
"dfb051702389e1979a151892b5901afc9e93bbc1c70d84c906ade3224ca91980"
  






--
# Randy MacLeod
# Wind River Linux

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188787): 
https://lists.openembedded.org/g/openembedded-core/message/188787
Mute This Topic: https://lists.openembedded.org/mt/101806877/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH V2] strace: skip so_peerpidfd test

2023-10-06 Thread Randy MacLeod via lists.openembedded.org
From: Randy MacLeod 

The so_peerpidfd test is failing due to a difference of square brackets
in part of the log file. Skip the failing test until the difference is
understood and fixed.

Signed-off-by: Randy MacLeod 
---
 .../skip-test-so_peerpidfd.gen.test.patch | 25 +++
 meta/recipes-devtools/strace/strace_6.5.bb|  1 +
 2 files changed, 26 insertions(+)
 create mode 100644 
meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch

diff --git 
a/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch 
b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
new file mode 100644
index 00..76f93ab613
--- /dev/null
+++ b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
@@ -0,0 +1,25 @@
+From 002d9f2512245536dfc8d62db429d97e2216ec3a Mon Sep 17 00:00:00 2001
+From: Randy MacLeod 
+Date: Fri, 6 Oct 2023 12:08:23 -0700
+Subject: [PATCH] skip tests/so_peerpidfd.gen.test
+
+Upstream-Status: Pending
+
+Signed-off-by: Randy MacLeod 
+---
+ tests/so_peerpidfd.gen.test | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/tests/so_peerpidfd.gen.test b/tests/so_peerpidfd.gen.test
+index 64ad3a2..f89da9f 100755
+--- a/tests/so_peerpidfd.gen.test
 b/tests/so_peerpidfd.gen.test
+@@ -1,4 +1,5 @@
+ #!/bin/sh -efu
+ # Generated by ./tests/gen_tests.sh from ./tests/gen_tests.in (so_peerpidfd 
--trace=getsockopt -y); do not edit.
+ . "${srcdir=.}/init.sh"
++skip_ "Test fails due to apparently trivial log format differences"
+ run_strace_match_diff --trace=getsockopt -y
+-- 
+2.39.0
+
diff --git a/meta/recipes-devtools/strace/strace_6.5.bb 
b/meta/recipes-devtools/strace/strace_6.5.bb
index 8f90c35925..d6475e8db9 100644
--- a/meta/recipes-devtools/strace/strace_6.5.bb
+++ b/meta/recipes-devtools/strace/strace_6.5.bb
@@ -14,6 +14,7 @@ SRC_URI = "https://strace.io/files/${PV}/strace-${PV}.tar.xz \
file://skip-load.patch \

file://0001-configure-Use-autoconf-macro-to-detect-largefile-sup.patch \
file://0002-tests-Replace-off64_t-with-off_t.patch \
+   file://skip-test-so_peerpidfd.gen.test.patch \
"
 SRC_URI[sha256sum] = 
"dfb051702389e1979a151892b5901afc9e93bbc1c70d84c906ade3224ca91980"
 
-- 
2.39.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188786): 
https://lists.openembedded.org/g/openembedded-core/message/188786
Mute This Topic: https://lists.openembedded.org/mt/101806877/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH] strace: skip so_peerpidfd test

2023-10-06 Thread Randy MacLeod via lists.openembedded.org

On 2023-10-06 4:06 p.m., Randy MacLeod via lists.openembedded.org wrote:

From: Randy MacLeod

The so_peerpidfd test is failing due to a difference of square brackets
in part of the log file. Skip the failing test until the difference is
understood and fixed.

Signed-off-by: Randy MacLeod
---
  .../skip-test-so_peerpidfd.gen.test.patch | 25 +++
  meta/recipes-devtools/strace/strace_6.5.bb|  2 ++
  2 files changed, 27 insertions(+)
  create mode 100644 
meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch

diff --git 
a/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch 
b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
new file mode 100644
index 00..76f93ab613
--- /dev/null
+++ b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
@@ -0,0 +1,25 @@
+From 002d9f2512245536dfc8d62db429d97e2216ec3a Mon Sep 17 00:00:00 2001
+From: Randy MacLeod
+Date: Fri, 6 Oct 2023 12:08:23 -0700
+Subject: [PATCH] skip tests/so_peerpidfd.gen.test
+
+Upstream-Status: Pending
+
+Signed-off-by: Randy MacLeod
+---
+ tests/so_peerpidfd.gen.test | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/tests/so_peerpidfd.gen.test b/tests/so_peerpidfd.gen.test
+index 64ad3a2..f89da9f 100755
+--- a/tests/so_peerpidfd.gen.test
 b/tests/so_peerpidfd.gen.test
+@@ -1,4 +1,5 @@
+ #!/bin/sh -efu
+ # Generated by ./tests/gen_tests.sh from ./tests/gen_tests.in (so_peerpidfd 
--trace=getsockopt -y); do not edit.
+ . "${srcdir=.}/init.sh"
++skip_ "Test fails due to apparently trivial log format differences"
+ run_strace_match_diff --trace=getsockopt -y
+--
+2.39.0
+
diff --git a/meta/recipes-devtools/strace/strace_6.5.bb 
b/meta/recipes-devtools/strace/strace_6.5.bb
index 8f90c35925..81baae474f 100644
--- a/meta/recipes-devtools/strace/strace_6.5.bb
+++ b/meta/recipes-devtools/strace/strace_6.5.bb
@@ -14,6 +14,7 @@ SRC_URI ="https://strace.io/files/${PV}/strace-${PV}.tar.xz \ 
file://skip-load.patch \ 
file://0001-configure-Use-autoconf-macro-to-detect-largefile-sup.patch 
\ file://0002-tests-Replace-off64_t-with-off_t.patch \ + 
file://skip-test-so_peerpidfd.gen.test.patch \ "

  SRC_URI[sha256sum] = 
"dfb051702389e1979a151892b5901afc9e93bbc1c70d84c906ade3224ca91980"
  
@@ -47,6 +48,7 @@ do_install_ptest() {

install -m 755 ${S}/build-aux/test-driver ${D}${PTEST_PATH}/build-aux/
install -m 644 ${B}/src/config.h ${D}${PTEST_PATH}/src/
  sed -i -e '/^src/s/strace.*[0-9]/ptest/' 
${D}/${PTEST_PATH}/${TESTDIR}/Makefile
+sed -i '4 i skip_ "Skip due to trivial output format difference."' 
${D}${PTEST_PATH}/tests/so_peerpidfd.gen.test



Oops, that line is no longer needed. V2 with it removed coming up.

../Randy



  }
  
  RDEPENDS:${PN}-ptest += "make coreutils grep gawk sed"






--
# Randy MacLeod
# Wind River Linux

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188785): 
https://lists.openembedded.org/g/openembedded-core/message/188785
Mute This Topic: https://lists.openembedded.org/mt/101805563/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core][kirkstone][PATCH] glibc: Update to latest on stable 2.35 branch

2023-10-06 Thread Peter Marko via lists.openembedded.org
From: Peter Marko 

Adresses CVE-2023-4911.

Single commit bump:
* c84018a05ae tunables: Terminate if end of input is reached (CVE-2023-4911)

Signed-off-by: Peter Marko 
---
 meta/recipes-core/glibc/glibc-version.inc | 2 +-
 meta/recipes-core/glibc/glibc_2.35.bb | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/glibc/glibc-version.inc 
b/meta/recipes-core/glibc/glibc-version.inc
index c23a43576c..e0d47f283b 100644
--- a/meta/recipes-core/glibc/glibc-version.inc
+++ b/meta/recipes-core/glibc/glibc-version.inc
@@ -1,6 +1,6 @@
 SRCBRANCH ?= "release/2.35/master"
 PV = "2.35"
-SRCREV_glibc ?= "73d4ce728a59deb2fd18969e559769b3f590fac9"
+SRCREV_glibc ?= "c84018a05aec80f5ee6f682db0da1130b0196aef"
 SRCREV_localedef ?= "794da69788cbf9bf57b59a852f9f11307663fa87"
 
 GLIBC_GIT_URI ?= "git://sourceware.org/git/glibc.git"
diff --git a/meta/recipes-core/glibc/glibc_2.35.bb 
b/meta/recipes-core/glibc/glibc_2.35.bb
index b4bad5b7ac..271520f76b 100644
--- a/meta/recipes-core/glibc/glibc_2.35.bb
+++ b/meta/recipes-core/glibc/glibc_2.35.bb
@@ -17,7 +17,7 @@ CVE_CHECK_IGNORE += "CVE-2019-1010022 CVE-2019-1010023 
CVE-2019-1010024"
 CVE_CHECK_IGNORE += "CVE-2019-1010025"
 
 # To avoid these in cve-check reports since the recipe version did not change
-CVE_CHECK_IGNORE += "CVE-2023-4813 CVE-2023-4806 CVE-2023-5156"
+CVE_CHECK_IGNORE += "CVE-2023-4813 CVE-2023-4806 CVE-2023-4911 CVE-2023-5156"
 
 DEPENDS += "gperf-native bison-native"
 
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188784): 
https://lists.openembedded.org/g/openembedded-core/message/188784
Mute This Topic: https://lists.openembedded.org/mt/101805676/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] strace: skip so_peerpidfd test

2023-10-06 Thread Randy MacLeod via lists.openembedded.org
From: Randy MacLeod 

The so_peerpidfd test is failing due to a difference of square brackets
in part of the log file. Skip the failing test until the difference is
understood and fixed.

Signed-off-by: Randy MacLeod 
---
 .../skip-test-so_peerpidfd.gen.test.patch | 25 +++
 meta/recipes-devtools/strace/strace_6.5.bb|  2 ++
 2 files changed, 27 insertions(+)
 create mode 100644 
meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch

diff --git 
a/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch 
b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
new file mode 100644
index 00..76f93ab613
--- /dev/null
+++ b/meta/recipes-devtools/strace/strace/skip-test-so_peerpidfd.gen.test.patch
@@ -0,0 +1,25 @@
+From 002d9f2512245536dfc8d62db429d97e2216ec3a Mon Sep 17 00:00:00 2001
+From: Randy MacLeod 
+Date: Fri, 6 Oct 2023 12:08:23 -0700
+Subject: [PATCH] skip tests/so_peerpidfd.gen.test
+
+Upstream-Status: Pending
+
+Signed-off-by: Randy MacLeod 
+---
+ tests/so_peerpidfd.gen.test | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/tests/so_peerpidfd.gen.test b/tests/so_peerpidfd.gen.test
+index 64ad3a2..f89da9f 100755
+--- a/tests/so_peerpidfd.gen.test
 b/tests/so_peerpidfd.gen.test
+@@ -1,4 +1,5 @@
+ #!/bin/sh -efu
+ # Generated by ./tests/gen_tests.sh from ./tests/gen_tests.in (so_peerpidfd 
--trace=getsockopt -y); do not edit.
+ . "${srcdir=.}/init.sh"
++skip_ "Test fails due to apparently trivial log format differences"
+ run_strace_match_diff --trace=getsockopt -y
+-- 
+2.39.0
+
diff --git a/meta/recipes-devtools/strace/strace_6.5.bb 
b/meta/recipes-devtools/strace/strace_6.5.bb
index 8f90c35925..81baae474f 100644
--- a/meta/recipes-devtools/strace/strace_6.5.bb
+++ b/meta/recipes-devtools/strace/strace_6.5.bb
@@ -14,6 +14,7 @@ SRC_URI = "https://strace.io/files/${PV}/strace-${PV}.tar.xz \
file://skip-load.patch \

file://0001-configure-Use-autoconf-macro-to-detect-largefile-sup.patch \
file://0002-tests-Replace-off64_t-with-off_t.patch \
+   file://skip-test-so_peerpidfd.gen.test.patch \
"
 SRC_URI[sha256sum] = 
"dfb051702389e1979a151892b5901afc9e93bbc1c70d84c906ade3224ca91980"
 
@@ -47,6 +48,7 @@ do_install_ptest() {
install -m 755 ${S}/build-aux/test-driver ${D}${PTEST_PATH}/build-aux/
install -m 644 ${B}/src/config.h ${D}${PTEST_PATH}/src/
 sed -i -e '/^src/s/strace.*[0-9]/ptest/' 
${D}/${PTEST_PATH}/${TESTDIR}/Makefile
+sed -i '4 i skip_ "Skip due to trivial output format difference."' 
${D}${PTEST_PATH}/tests/so_peerpidfd.gen.test
 }
 
 RDEPENDS:${PN}-ptest += "make coreutils grep gawk sed"
-- 
2.39.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188783): 
https://lists.openembedded.org/g/openembedded-core/message/188783
Mute This Topic: https://lists.openembedded.org/mt/101805563/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[[OE-core][dunfell][PATCH] libpcre2 : Follow up fix CVE-2022-1586

2023-10-06 Thread Shinu Chandran via lists.openembedded.org
CVE-2022-1586 was originally fixed by OE commit
https://github.com/openembedded/openembedded-core/commit/7f4daf88b71f
through libpcre2 commit
https://github.com/PCRE2Project/pcre2/commit/50a51cb7e672

The follow up patch is required to resolve a bug in the initial 
fix[50a51cb7e672]
https://github.com/PCRE2Project/pcre2/commit/d4fa336fbcc3

Reference:
https://nvd.nist.gov/vuln/detail/CVE-2022-1586
https://security-tracker.debian.org/tracker/CVE-2022-1586

Signed-off-by: Shinu Chandran 
---
 .../libpcre2/CVE-2022-1586-regression.patch   | 30 +++
 .../recipes-support/libpcre/libpcre2_10.34.bb |  1 +
 2 files changed, 31 insertions(+)
 create mode 100644 
meta/recipes-support/libpcre/libpcre2/CVE-2022-1586-regression.patch

diff --git 
a/meta/recipes-support/libpcre/libpcre2/CVE-2022-1586-regression.patch 
b/meta/recipes-support/libpcre/libpcre2/CVE-2022-1586-regression.patch
new file mode 100644
index 00..42ee417fe7
--- /dev/null
+++ b/meta/recipes-support/libpcre/libpcre2/CVE-2022-1586-regression.patch
@@ -0,0 +1,30 @@
+From 5d1e62b0155292b994aa1c96d4ed8ce4346ef4c2 Mon Sep 17 00:00:00 2001
+From: Zoltan Herczeg 
+Date: Thu, 24 Mar 2022 05:34:42 +
+Subject: [PATCH] Fix incorrect value reading in JIT.
+
+CVE: CVE-2022-1586
+Upstream-Status: Backport 
[https://github.com/PCRE2Project/pcre2/commit/d4fa336fbcc3]
+
+(cherry picked from commit d4fa336fbcc388f89095b184ba6d99422cfc676c)
+Signed-off-by: Shinu Chandran 
+---
+ src/pcre2_jit_compile.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/pcre2_jit_compile.c b/src/pcre2_jit_compile.c
+index 493c96d..fa57942 100644
+--- a/src/pcre2_jit_compile.c
 b/src/pcre2_jit_compile.c
+@@ -7188,7 +7188,7 @@ while (*cc != XCL_END)
+ {
+ SLJIT_ASSERT(*cc == XCL_PROP || *cc == XCL_NOTPROP);
+ cc++;
+-if (*cc == PT_CLIST && *cc == XCL_PROP)
++if (*cc == PT_CLIST && cc[-1] == XCL_PROP)
+   {
+   other_cases = PRIV(ucd_caseless_sets) + cc[1];
+   while (*other_cases != NOTACHAR)
+-- 
+2.25.1
+
diff --git a/meta/recipes-support/libpcre/libpcre2_10.34.bb 
b/meta/recipes-support/libpcre/libpcre2_10.34.bb
index 3e1b001c32..53277270d2 100644
--- a/meta/recipes-support/libpcre/libpcre2_10.34.bb
+++ b/meta/recipes-support/libpcre/libpcre2_10.34.bb
@@ -13,6 +13,7 @@ LIC_FILES_CHKSUM = 
"file://LICENCE;md5=b1588d3bb4cb0e1f5a597d908f8c5b37"
 SRC_URI = 
"http://downloads.yoctoproject.org/mirror/sources/pcre2-${PV}.tar.bz2 \
file://pcre-cross.patch \
file://CVE-2022-1586.patch \
+   file://CVE-2022-1586-regression.patch \
file://CVE-2022-1587.patch \
file://CVE-2022-41409.patch \
 "
-- 
2.28.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188782): 
https://lists.openembedded.org/g/openembedded-core/message/188782
Mute This Topic: https://lists.openembedded.org/mt/101804527/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH v2 4/4] selftest: Add tests for populate_sdk task

2023-10-06 Thread Pavel Zhukov
Two tests to cover both installing package with IMAGE_INSTALL as
well as installing versioned dependencies of the package (using perl (>=
5.XX).

Related: [Yocto #13338] [Yocto #14995] [Yocto #14066]

Signed-off-by: Pavel Zhukov 
---
 .../testsdk-perldepends.bb| 16 
 meta/lib/oeqa/selftest/cases/sdk.py   | 79 +++
 2 files changed, 95 insertions(+)
 create mode 100644 
meta-selftest/recipes-test/testsdk-perldepends/testsdk-perldepends.bb
 create mode 100644 meta/lib/oeqa/selftest/cases/sdk.py

diff --git 
a/meta-selftest/recipes-test/testsdk-perldepends/testsdk-perldepends.bb 
b/meta-selftest/recipes-test/testsdk-perldepends/testsdk-perldepends.bb
new file mode 100644
index 00..428e1f52e2
--- /dev/null
+++ b/meta-selftest/recipes-test/testsdk-perldepends/testsdk-perldepends.bb
@@ -0,0 +1,16 @@
+SUMMARY = "Test recipe for sdk populate test case"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = 
"file://${COMMON_LICENSE_DIR}/MIT;md5=0835ade698e0bcf8506ecda2f7b4f302"
+
+INHIBIT_DEFAULT_DEPS = "1"
+
+RDEPENDS:${PN}="perl (>= 5.12)"
+
+do_install(){
+   mkdir -p ${D}/var/lib/
+   touch ${D}/var/lib/testname 
+   echo "${PN}" > ${D}/var/lib/testname
+}
+
+
+
diff --git a/meta/lib/oeqa/selftest/cases/sdk.py 
b/meta/lib/oeqa/selftest/cases/sdk.py
new file mode 100644
index 00..35734d8a7a
--- /dev/null
+++ b/meta/lib/oeqa/selftest/cases/sdk.py
@@ -0,0 +1,79 @@
+#
+# Copyright OpenEmbedded Contributors
+#
+# SPDX-License-Identifier: MIT
+#
+
+import os
+
+from oeqa.selftest.case import OESelftestTestCase
+from oeqa.utils.commands import get_bb_var, bitbake
+
+class PopulateSDK(OESelftestTestCase):
+'''Tests for populate SDK task(s)'''
+
+@classmethod
+def setUpClass(self):
+
+super(PopulateSDK, self).setUpClass()
+self.buildtarget = 'core-image-minimal'
+self.classname = 'PopulateSDK'
+
+def sdk_build(self):
+"""
+Test if package added via IMAGE_INSTALL
+conflicts with dummy SDK providers
+"""
+SDKTASK = '-c populate_sdk'
+bbargs = "{} {}".format(SDKTASK, self.buildtarget)
+self.logger.info("{}: doing bitbake {} ".format(self.classname, 
bbargs))
+return bitbake(bbargs).status
+
+def set_config(self, image_install, package_classes):
+config = 'IMAGE_INSTALL:append = " {}"\n'.format(image_install)
+config += 'PACKAGE_CLASSES = " package_{}"\n'.format(package_classes)
+return config
+
+def do_test_build(self, image_install, package_manager):
+self.write_config(self.set_config(image_install, package_manager))
+res = self.sdk_build()
+self.assertEqual(0, res, "Failed to populate SDK with {} in 
IMAGE_INSTALL and {} package manager"\
+.format(image_install, package_manager))
+
+def test_image_install_ipk(self):
+"""
+Regression test for [Yocto #13338] 
+"""
+self.do_test_build("bash", "ipk")
+
+def test_image_install_rpm(self):
+"""
+Regression test for [Yocto #13338] 
+"""
+self.do_test_build("bash", "rpm")
+
+def test_image_install_deb(self):
+"""
+Regression test for [Yocto #13338] 
+"""
+self.do_test_build("bash", "deb")
+
+
+def test_image_install_confictdeps_ipk(self):
+"""
+Regression test for [Yocto #14995]
+"""
+self.skipTest("Disabled until [Yocto #14995] is fixed")
+self.do_test_build("testsdk-perldepends", "ipk")
+
+def test_image_install_confictdeps_rpm(self):
+"""
+Regression test for [Yocto #14995]
+"""
+self.do_test_build("testsdk-perldepends", "rpm")
+
+def test_image_install_confictdeps_deb(self):
+"""
+Regression test for [Yocto #14995]
+"""
+self.do_test_build("testsdk-perldepends", "deb")
-- 
2.41.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188781): 
https://lists.openembedded.org/g/openembedded-core/message/188781
Mute This Topic: https://lists.openembedded.org/mt/101804518/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH v2 3/4] dummy-sdk-package.inc: Specify providers version

2023-10-06 Thread Pavel Zhukov
Some of the packages require versioned providers
(DEPENDS: perl (>= 5.38) is an example and for such packages
do_populate_sdk fails because dummy packages provided unversioned
packages (PROVIDES: perl) which doesn't meet the version requirement.
Specify 999.9-r9 version for such provides to work this around

Fixes [Yocto #14995]

Signed-off-by: Pavel Zhukov 
---
 meta/recipes-core/meta/dummy-sdk-package.inc | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/meta/dummy-sdk-package.inc 
b/meta/recipes-core/meta/dummy-sdk-package.inc
index 99863b86f1..b1571a2cc5 100644
--- a/meta/recipes-core/meta/dummy-sdk-package.inc
+++ b/meta/recipes-core/meta/dummy-sdk-package.inc
@@ -18,14 +18,17 @@ ALLOW_EMPTY:${PN} = "1"
 PR[vardeps] += "DUMMYPROVIDES"
 PR[vardeps] += "DUMMYPROVIDES_PACKAGES"
 
+DUMMYMAXVER ??= "999.9-r9"
 DUMMYPROVIDES_PACKAGES ??= ""
 DUMMYPROVIDES_PACKAGES_FILTERED = "${@' '.join([pkg for pkg in 
d.getVar('DUMMYPROVIDES_PACKAGES').split() if d.getVar('IMAGE_INSTALL') is None 
or pkg not in d.getVar('IMAGE_INSTALL').split()])}"
 DUMMYPROVIDES_PACKAGES_MULTILIB = "${@' '.join([multilib_pkg_extend(d, pkg) 
for pkg in d.getVar('DUMMYPROVIDES_PACKAGES_FILTERED').split()])}"
+DUMMYPROVIDES_MAXVER = "${@' '.join(['{} (= {})'.format(pkg, 
d.getVar('DUMMYMAXVER')) for pkg in 
d.getVar('DUMMYPROVIDES_PACKAGES_MULTILIB').split()]) + 
d.getVar('DUMMYPROVIDES')}"
+
 DUMMYPROVIDES += "${DUMMYPROVIDES_PACKAGES_MULTILIB}"
 
 python populate_packages:prepend() {
 p = d.getVar("PN")
-d.appendVar("RPROVIDES:%s" % p, "${DUMMYPROVIDES}")
+d.appendVar("RPROVIDES:%s" % p, "${DUMMYPROVIDES_MAXVER}")
 d.appendVar("RCONFLICTS:%s" % p, "${DUMMYPROVIDES_PACKAGES_MULTILIB}")
 d.appendVar("RREPLACES:%s" % p, "${DUMMYPROVIDES_PACKAGES_MULTILIB}")
 }
-- 
2.41.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188780): 
https://lists.openembedded.org/g/openembedded-core/message/188780
Mute This Topic: https://lists.openembedded.org/mt/101804517/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH v2 2/4] package_deb: Revert versioned providers workaround

2023-10-06 Thread Pavel Zhukov
dpkg and apt seem to handle versioned provides correctly now [1] so this
workaround is not needed anymore.
This fixes [Yocto #14995] for package_deb.

[1]
Signed-off-by: Pavel Zhukov 
---
 meta/classes-global/package_deb.bbclass | 9 +
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/meta/classes-global/package_deb.bbclass 
b/meta/classes-global/package_deb.bbclass
index 5aba3837bd..d7bbfae3fb 100644
--- a/meta/classes-global/package_deb.bbclass
+++ b/meta/classes-global/package_deb.bbclass
@@ -184,8 +184,6 @@ def deb_write_pkg(pkg, d):
 #   '<' = less or equal
 #   '>' = greater or equal
 # adjust these to the '<<' and '>>' equivalents
-# Also, "=" specifiers only work if they have the PR in, so 1.2.3 
!= 1.2.3-r0
-# so to avoid issues, map this to ">= 1.2.3 << 1.2.3.0"
 for dep in list(var.keys()):
 if '(' in dep or '/' in dep:
 newdep = re.sub(r'[(:)/]', '__', dep)
@@ -200,10 +198,6 @@ def deb_write_pkg(pkg, d):
 var[dep][i] = var[dep][i].replace("< ", "<< ")
 elif (v or "").startswith("> "):
 var[dep][i] = var[dep][i].replace("> ", ">> ")
-elif (v or "").startswith("= ") and "-r" not in v:
-ver = var[dep][i].replace("= ", "")
-var[dep][i] = var[dep][i].replace("= ", ">= ")
-var[dep].append("<< " + ver + ".0")
 
 rdepends = bb.utils.explode_dep_versions2(localdata.getVar("RDEPENDS") 
or "")
 debian_cmp_remap(rdepends)
@@ -220,8 +214,7 @@ def deb_write_pkg(pkg, d):
 del rrecommends[dep]
 rsuggests = 
bb.utils.explode_dep_versions2(localdata.getVar("RSUGGESTS") or "")
 debian_cmp_remap(rsuggests)
-# Deliberately drop version information here, not wanted/supported by 
deb
-rprovides = 
dict.fromkeys(bb.utils.explode_dep_versions2(localdata.getVar("RPROVIDES") or 
""), [])
+rprovides = 
bb.utils.explode_dep_versions2(localdata.getVar("RPROVIDES") or "")
 # Remove file paths if any from rprovides, debian does not support 
custom providers
 for key in list(rprovides.keys()):
 if key.startswith('/'):
-- 
2.41.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188778): 
https://lists.openembedded.org/g/openembedded-core/message/188778
Mute This Topic: https://lists.openembedded.org/mt/101804515/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH v2 1/4] dummy-sdk-package.inc: Filter packages which are marked for installation

2023-10-06 Thread Pavel Zhukov
if packages is provided by dummysdk and in the same time marked for
installation with IMAGE_INSTALL it causes conflict in apt because virtual 
providers are
not taken into account if package is asked to be installed explicitly.
Filter such packages from provides/conflicts to workaround this problem.
This workaround brakes RPM usecase because of file conlicts with
DUMMYPROVIDES, use DUMMYPROVIDES_PACKAGES_MULTILIB instead (which
doesn't include file based conflicts).
While this is needed for the case of package_deb only adding it for all
package managers to not complicate the code.

Fixes: [Yocto #13338] [Yocto #14066]

Fixes:
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 target-sdk-provides-dummy : Conflicts: bash
E: Unable to correct problems, you have held broken packages.

Signed-off-by: Pavel Zhukov 
---
 meta/recipes-core/meta/dummy-sdk-package.inc | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/meta/dummy-sdk-package.inc 
b/meta/recipes-core/meta/dummy-sdk-package.inc
index bd26e39ad3..99863b86f1 100644
--- a/meta/recipes-core/meta/dummy-sdk-package.inc
+++ b/meta/recipes-core/meta/dummy-sdk-package.inc
@@ -19,13 +19,14 @@ PR[vardeps] += "DUMMYPROVIDES"
 PR[vardeps] += "DUMMYPROVIDES_PACKAGES"
 
 DUMMYPROVIDES_PACKAGES ??= ""
-DUMMYPROVIDES_PACKAGES_MULTILIB = "${@' '.join([multilib_pkg_extend(d, pkg) 
for pkg in d.getVar('DUMMYPROVIDES_PACKAGES').split()])}"
+DUMMYPROVIDES_PACKAGES_FILTERED = "${@' '.join([pkg for pkg in 
d.getVar('DUMMYPROVIDES_PACKAGES').split() if d.getVar('IMAGE_INSTALL') is None 
or pkg not in d.getVar('IMAGE_INSTALL').split()])}"
+DUMMYPROVIDES_PACKAGES_MULTILIB = "${@' '.join([multilib_pkg_extend(d, pkg) 
for pkg in d.getVar('DUMMYPROVIDES_PACKAGES_FILTERED').split()])}"
 DUMMYPROVIDES += "${DUMMYPROVIDES_PACKAGES_MULTILIB}"
 
 python populate_packages:prepend() {
 p = d.getVar("PN")
 d.appendVar("RPROVIDES:%s" % p, "${DUMMYPROVIDES}")
-d.appendVar("RCONFLICTS:%s" % p, "${DUMMYPROVIDES}")
+d.appendVar("RCONFLICTS:%s" % p, "${DUMMYPROVIDES_PACKAGES_MULTILIB}")
 d.appendVar("RREPLACES:%s" % p, "${DUMMYPROVIDES_PACKAGES_MULTILIB}")
 }
 
-- 
2.41.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188779): 
https://lists.openembedded.org/g/openembedded-core/message/188779
Mute This Topic: https://lists.openembedded.org/mt/101804516/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core][PATCH] python3-ptest: skip test_storlines

2023-10-06 Thread Trevor Gamblin
[YOCTO #14933]

test_storlines is yet another Python ptest that fails intermittently on
the Yocto AB, so disable it during ptests for now.

Signed-off-by: Trevor Gamblin 
---
 ...orlines-skip-due-to-load-variability.patch | 32 +++
 .../recipes-devtools/python/python3_3.11.5.bb |  1 +
 2 files changed, 33 insertions(+)
 create mode 100644 
meta/recipes-devtools/python/python3/0001-test_storlines-skip-due-to-load-variability.patch

diff --git 
a/meta/recipes-devtools/python/python3/0001-test_storlines-skip-due-to-load-variability.patch
 
b/meta/recipes-devtools/python/python3/0001-test_storlines-skip-due-to-load-variability.patch
new file mode 100644
index 000..199031d42aa
--- /dev/null
+++ 
b/meta/recipes-devtools/python/python3/0001-test_storlines-skip-due-to-load-variability.patch
@@ -0,0 +1,32 @@
+From 013ff01fdf2aa6ca69a7c80a2a2996630877e4ea Mon Sep 17 00:00:00 2001
+From: Trevor Gamblin 
+Date: Fri, 6 Oct 2023 10:59:44 -0400
+Subject: [PATCH] test_storlines: skip due to load variability
+
+This is yet another test that intermittently fails on the Yocto AB when
+a worker is under heavy load, so skip it during testing.
+
+Upstream-Status: Inappropriate [OE-Specific]
+
+[YOCTO #14933]
+
+Signed-off-by: Trevor Gamblin 
+---
+ Lib/test/test_ftplib.py | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/Lib/test/test_ftplib.py b/Lib/test/test_ftplib.py
+index 082a90d46b..508814d56a 100644
+--- a/Lib/test/test_ftplib.py
 b/Lib/test/test_ftplib.py
+@@ -629,6 +629,7 @@ def test_storbinary_rest(self):
+ self.client.storbinary('stor', f, rest=r)
+ self.assertEqual(self.server.handler_instance.rest, str(r))
+ 
++@unittest.skip('timing related test, dependent on load')
+ def test_storlines(self):
+ data = RETR_DATA.replace('\r\n', '\n').encode(self.client.encoding)
+ f = io.BytesIO(data)
+-- 
+2.41.0
+
diff --git a/meta/recipes-devtools/python/python3_3.11.5.bb 
b/meta/recipes-devtools/python/python3_3.11.5.bb
index 8e023c7dfb8..d375de4b19c 100644
--- a/meta/recipes-devtools/python/python3_3.11.5.bb
+++ b/meta/recipes-devtools/python/python3_3.11.5.bb
@@ -33,6 +33,7 @@ SRC_URI = 
"http://www.python.org/ftp/python/${PV}/Python-${PV}.tar.xz \
file://0001-Avoid-shebang-overflow-on-python-config.py.patch \
file://0001-Update-test_sysconfig-for-posix_user-purelib.patch \

file://0001-skip-no_stdout_fileno-test-due-to-load-variability.patch \
+   file://0001-test_storlines-skip-due-to-load-variability.patch \
"
 
 SRC_URI:append:class-native = " \
-- 
2.41.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188777): 
https://lists.openembedded.org/g/openembedded-core/message/188777
Mute This Topic: https://lists.openembedded.org/mt/101800998/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH] oeqa/selftest/context: Rely on bitbake-getvar --value to only return value

2023-10-06 Thread Peter Kjellerstedt
> -Original Message-
> From: openembedded-core@lists.openembedded.org 
>  On Behalf Of Peter Kjellerstedt
> Sent: den 23 september 2023 06:06
> To: openembedded-core@lists.openembedded.org
> Subject: [OE-core] [PATCH] oeqa/selftest/context: Rely on bitbake-getvar 
> --value to only return value
> 
> Before, "bitbake-getvar --value " would include log output together
> with the value. This was handled by piping the output to "tail -1".
> Now, "bitbake-getvar --value" will no longer output any logs so the
> piping to "tail" is no longer needed.
> 
> Signed-off-by: Peter Kjellerstedt 
> ---
> 
> This of course relies on the corresponding patch for bitbake-getvar to
> have been applied first.
> 
>  meta/lib/oeqa/selftest/context.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/lib/oeqa/selftest/context.py 
> b/meta/lib/oeqa/selftest/context.py
> index 5a09aeedff..c148aa5aab 100644
> --- a/meta/lib/oeqa/selftest/context.py
> +++ b/meta/lib/oeqa/selftest/context.py
> @@ -111,7 +111,7 @@ class OESelftestTestContext(OETestContext):
> 
>  # Relative paths in BBLAYERS only works when the new build dir share 
> the same ascending node
>  if self.newbuilddir:
> -bblayers = subprocess.check_output("bitbake-getvar --value 
> BBLAYERS | tail -1", cwd=builddir, shell=True, text=True)
> +bblayers = subprocess.check_output("bitbake-getvar --value 
> BBLAYERS", cwd=builddir, shell=True, text=True)
>  if '..' in bblayers:
>  bblayers_abspath = [os.path.abspath(path) for path in 
> bblayers.split()]
>  with open("%s/conf/bblayers.conf" % newbuilddir, "a") as f:

Since the fixes to bitbake-getvalue have merged, it should be possible 
to merge this now.

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188776): 
https://lists.openembedded.org/g/openembedded-core/message/188776
Mute This Topic: https://lists.openembedded.org/mt/101535023/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] openssh: drop sudo from ptest dependencies

2023-10-06 Thread Mikko Rapeli
The tests don't actually need sudo on core-image-ptest-openssh.
Based on logs seen in
https://bugzilla.yoctoproject.org/show_bug.cgi?id=15178 it seems
that socket errors from sudo are creeping into stderr which are failing
the banner ptest from openssh. Removing sudo should help removing
the stderr messages and possibly cure the banner test failures.

Signed-off-by: Mikko Rapeli 
---
 meta/recipes-connectivity/openssh/openssh/run-ptest | 2 +-
 meta/recipes-connectivity/openssh/openssh_9.4p1.bb  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-connectivity/openssh/openssh/run-ptest 
b/meta/recipes-connectivity/openssh/openssh/run-ptest
index 1e6eec5799..b2244d725a 100755
--- a/meta/recipes-connectivity/openssh/openssh/run-ptest
+++ b/meta/recipes-connectivity/openssh/openssh/run-ptest
@@ -19,7 +19,7 @@ output_failed_logs() {
 trap output_failed_logs 0
 
 sed -i "/\t\tagent-ptrace /d" Makefile
-make -k BUILDDIR=`pwd`/.. .OBJDIR=`pwd` .CURDIR=`pwd` SUDO="sudo" tests \
+make -k BUILDDIR=`pwd`/.. .OBJDIR=`pwd` .CURDIR=`pwd` SUDO="" tests \
 | sed -u -e 's/^skipped/SKIP: /g' -e 's/^ok /PASS: /g' -e 
's/^failed/FAIL: /g'
 
 SSHAGENT=`which ssh-agent`
diff --git a/meta/recipes-connectivity/openssh/openssh_9.4p1.bb 
b/meta/recipes-connectivity/openssh/openssh_9.4p1.bb
index a38d9c2b81..e2508aa63a 100644
--- a/meta/recipes-connectivity/openssh/openssh_9.4p1.bb
+++ b/meta/recipes-connectivity/openssh/openssh_9.4p1.bb
@@ -158,7 +158,7 @@ FILES:${PN}-keygen = "${bindir}/ssh-keygen"
 RDEPENDS:${PN} += "${PN}-scp ${PN}-ssh ${PN}-sshd ${PN}-keygen 
${PN}-sftp-server"
 RDEPENDS:${PN}-sshd += "${PN}-keygen ${@bb.utils.contains('DISTRO_FEATURES', 
'pam', 'pam-plugin-keyinit pam-plugin-loginuid', '', d)}"
 # gdb would make attach-ptrace test pass rather than skip but not worth the 
build dependencies
-RDEPENDS:${PN}-ptest += "${PN}-sftp ${PN}-misc ${PN}-sftp-server make sed sudo 
coreutils openssl-bin"
+RDEPENDS:${PN}-ptest += "${PN}-sftp ${PN}-misc ${PN}-sftp-server make sed 
coreutils openssl-bin"
 
 RPROVIDES:${PN}-ssh = "ssh"
 RPROVIDES:${PN}-sshd = "sshd"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188775): 
https://lists.openembedded.org/g/openembedded-core/message/188775
Mute This Topic: https://lists.openembedded.org/mt/101799351/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] Useful for me to send more Python 3.12 patches?

2023-10-06 Thread Chris Laplante via lists.openembedded.org
Hi Richard,

Thanks for accepting my patch from a few days ago :). Just wanted to check if 
you would like me to send more Python 3.12 compatibility patches. If there are 
bigger fish to fry I will hold off, otherwise I'm happy to help with making 
everything compatible.

P.S. I ask because I didn't realize that Python 3.12 released on the same day 
as when I sent the patch. The tool I use to maintain Python envs (rye) 
dutifully updated to the latest Python version on the day I started some work 
on poky.

Thanks,
Chris

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188774): 
https://lists.openembedded.org/g/openembedded-core/message/188774
Mute This Topic: https://lists.openembedded.org/mt/101799331/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH v3] systemd: sysusersd: change how the root home directory is set

2023-10-06 Thread Louis Rannou
From: Louis Rannou 

Modifying the sources brings troubles in the devtool command. Therefore it is
better to patch the source with a custom variable, and later replace it with the
correct root path.

Suggestion from :


Signed-off-by: Louis Rannou 
Suggested-by: Peter Kjellerstedt 
Signed-off-by: Louis Rannou 
---
Changes in v3:
- remove file basic.conf.in
- fix patch 'From'  line
- Link to v2: 
https://lore.kernel.org/r/20231005-sysusers3-v2-1-938c9cca7...@syslinbit.com
---
 ...usersd-Change-the-user-root-s-home-direct.patch | 31 +
 meta/recipes-core/systemd/systemd/basic.conf.in| 40 --
 meta/recipes-core/systemd/systemd_254.4.bb | 11 +++---
 3 files changed, 37 insertions(+), 45 deletions(-)

diff --git 
a/meta/recipes-core/systemd/systemd/0030-sysusersd-Change-the-user-root-s-home-direct.patch
 
b/meta/recipes-core/systemd/systemd/0030-sysusersd-Change-the-user-root-s-home-direct.patch
new file mode 100644
index 00..30d800df72
--- /dev/null
+++ 
b/meta/recipes-core/systemd/systemd/0030-sysusersd-Change-the-user-root-s-home-direct.patch
@@ -0,0 +1,31 @@
+From bf97001978cdefad644f7b4b909f281368e5a4dd Mon Sep 17 00:00:00 2001
+From: Louis Rannou 
+Date: Thu, 27 Jul 2023 08:23:06 +
+Subject: [PATCH] sysusers.d: Change the user root's home directory
+
+The default sysusers basic.conf.in file sets the root home directory to `/root`
+and does not permit its configuration. Change this to `:ROOT_HOME:` which must
+be set before the installation.
+
+The upstream considers the root home directory should not be changed
+
+
+Upstream-Status: Denied
+Signed-off-by: Louis Rannou 
+---
+ sysusers.d/basic.conf.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/sysusers.d/basic.conf.in b/sysusers.d/basic.conf.in
+index 0aec080a4c..f26c224341 100644
+--- a/sysusers.d/basic.conf.in
 b/sysusers.d/basic.conf.in
+@@ -7,7 +7,7 @@
+ 
+ # The superuser
+ g root0   --
+-u root0:0 "Super User" /root
++u root0:0 "Super User" :ROOT_HOME:
+ 
+ # The nobody user/group for NFS file systems
+ g {{NOBODY_GROUP_NAME}} 65534   --
diff --git a/meta/recipes-core/systemd/systemd/basic.conf.in 
b/meta/recipes-core/systemd/systemd/basic.conf.in
deleted file mode 100644
index fac288f7fa..00
--- a/meta/recipes-core/systemd/systemd/basic.conf.in
+++ /dev/null
@@ -1,40 +0,0 @@
-#  This file is part of systemd.
-#
-#  systemd is free software; you can redistribute it and/or modify it
-#  under the terms of the GNU Lesser General Public License as published by
-#  the Free Software Foundation; either version 2.1 of the License, or
-#  (at your option) any later version.
-
-# The superuser
-u root0 "root" :ROOT_HOME:
-
-# The nobody user/group for NFS file systems
-g {{NOBODY_GROUP_NAME}} 65534   --
-u {{NOBODY_USER_NAME }} 65534:65534 "Nobody" -
-
-# Administrator group: can *see* more than normal users
-g adm {{ADM_GID}} --
-
-# Administrator group: can *do* more than normal users
-g wheel   {{WHEEL_GID  }} --
-
-# Access to shared database of users on the system
-g utmp{{UTMP_GID   }} --
-
-# Physical and virtual hardware access groups
-g audio   {{AUDIO_GID  }} --
-g cdrom   {{CDROM_GID  }} --
-g dialout {{DIALOUT_GID}} --
-g disk{{DISK_GID   }} --
-g input   {{INPUT_GID  }} --
-g kmem{{KMEM_GID   }} --
-g kvm {{KVM_GID}} --
-g lp  {{LP_GID }} --
-g render  {{RENDER_GID }} --
-g sgx {{SGX_GID}} --
-g tape{{TAPE_GID   }} --
-g tty {{TTY_GID}} --
-g video   {{VIDEO_GID  }} --
-
-# Default group for normal users
-g users   {{USERS_GID  }} --
diff --git a/meta/recipes-core/systemd/systemd_254.4.bb 
b/meta/recipes-core/systemd/systemd_254.4.bb
index 77724eb822..c408035696 100644
--- a/meta/recipes-core/systemd/systemd_254.4.bb
+++ b/meta/recipes-core/systemd/systemd_254.4.bb
@@ -21,7 +21,6 @@ REQUIRED_DISTRO_FEATURES += "systemd"
 SRC_URI += " \
file://touchscreen.rules \
file://00-create-volatile.conf \
-   file://basic.conf.in \
${@bb.utils.contains('PACKAGECONFIG', 'polkit_hostnamed_fallback', 
'file://org.freedesktop.hostname1_no_polkit.conf', '', d)} \
${@bb.utils.contains('PACKAGECONFIG', 'polkit_hostnamed_fallback', 
'file://00-hostnamed-network-user.conf', '', d)} \
file://init \
@@ -30,6 +29,7 @@ SRC_URI += " \

file://0002-binfmt-Don-t-install-dependency-links-at-install-tim.patch \

[OE-core] [PATCH] zstd: fix LICENSE statement

2023-10-06 Thread Massimiliano Minella
From: Massimiliano Minella 

zstd is dual-licensed under BSD _OR_ GPLv2. License wording in the
README for v1.5.5 is misleading, but license headers in the code clearly
state that there is a choice between the two licenses.

Signed-off-by: Massimiliano Minella 
---
 meta/recipes-extended/zstd/zstd_1.5.5.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/zstd/zstd_1.5.5.bb 
b/meta/recipes-extended/zstd/zstd_1.5.5.bb
index 5c5fb5e734..2d72af50a4 100644
--- a/meta/recipes-extended/zstd/zstd_1.5.5.bb
+++ b/meta/recipes-extended/zstd/zstd_1.5.5.bb
@@ -5,7 +5,7 @@ It's backed by a very fast entropy stage, provided by Huff0 and 
FSE library."
 HOMEPAGE = "http://www.zstd.net/;
 SECTION = "console/utils"
 
-LICENSE = "BSD-3-Clause & GPL-2.0-only"
+LICENSE = "BSD-3-Clause | GPL-2.0-only"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=0822a32f7acdbe013606746641746ee8 \
 file://COPYING;md5=39bba7d2cf0ba1036f2a6e2be52fe3f0 \
 "
-- 
2.42.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188772): 
https://lists.openembedded.org/g/openembedded-core/message/188772
Mute This Topic: https://lists.openembedded.org/mt/101798771/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2] systemd: sysusersd: change how the root home directory is set

2023-10-06 Thread Louis Rannou



On 06/10/2023 15:16, Julien Stephan wrote:

Le jeu. 5 oct. 2023 à 22:55, Louis Rannou  a écrit :


From: Louis Rannou 

Modifying the sources brings troubles in the devtool command. Therefore it is
better to patch the source with a custom variable, and later replace it with the
correct root path.

Suggestion from :


Signed-off-by: Louis Rannou 
Suggested-by: Peter Kjellerstedt 
Signed-off-by: Louis Rannou 
---
  ...usersd-Change-the-user-root-s-home-direct.patch | 31 ++
  meta/recipes-core/systemd/systemd_254.4.bb | 11 
  2 files changed, 37 insertions(+), 5 deletions(-)

diff --git 
a/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
 
b/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
new file mode 100644
index 00..dd0d96d11b
--- /dev/null
+++ 
b/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
@@ -0,0 +1,31 @@
+From bf97001978cdefad644f7b4b909f281368e5a4dd Mon Sep 17 00:00:00 2001
+From: Kas User 


Hi Louis,

Is this intended to have kas user and email address instead of yours?


No, I thought I changed that.



+Date: Thu, 27 Jul 2023 08:23:06 +
+Subject: [PATCH] sysusers.d: Change the user root's home directory
+
+The default sysusers basic.conf.in file sets the root home directory to `/root`
+and does not permit its configuration. Change this to `:ROOT_HOME:` which must
+be set before the installation.
+
+The upstream considers the root home directory should not be changed
+
+
+Upstream-Status: Denied
+Signed-off-by: Louis Rannou 
+---
+ sysusers.d/basic.conf.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/sysusers.d/basic.conf.in b/sysusers.d/basic.conf.in
+index 0aec080a4c..f26c224341 100644
+--- a/sysusers.d/basic.conf.in
 b/sysusers.d/basic.conf.in
+@@ -7,7 +7,7 @@
+
+ # The superuser
+ g root0   --
+-u root0:0 "Super User" /root
++u root0:0 "Super User" :ROOT_HOME:
+
+ # The nobody user/group for NFS file systems
+ g {{NOBODY_GROUP_NAME}} 65534   --
diff --git a/meta/recipes-core/systemd/systemd_254.4.bb 
b/meta/recipes-core/systemd/systemd_254.4.bb
index 77724eb822..3adf62935d 100644
--- a/meta/recipes-core/systemd/systemd_254.4.bb
+++ b/meta/recipes-core/systemd/systemd_254.4.bb
@@ -21,7 +21,6 @@ REQUIRED_DISTRO_FEATURES += "systemd"
  SRC_URI += " \
 file://touchscreen.rules \
 file://00-create-volatile.conf \
-   file://basic.conf.in \


If this file is not needed anymore you should probably delete it with
this commit


Good catch. Thanks
Louis



Cheers
Julien


 ${@bb.utils.contains('PACKAGECONFIG', 'polkit_hostnamed_fallback', 
'file://org.freedesktop.hostname1_no_polkit.conf', '', d)} \
 ${@bb.utils.contains('PACKAGECONFIG', 'polkit_hostnamed_fallback', 
'file://00-hostnamed-network-user.conf', '', d)} \
 file://init \
@@ -30,6 +29,7 @@ SRC_URI += " \
 
file://0002-binfmt-Don-t-install-dependency-links-at-install-tim.patch \
 file://0008-implment-systemd-sysv-install-for-OE.patch \
 
file://0004-Move-sysusers.d-sysctl.d-binfmt.d-modules-load.d-to-.patch \
+   file://0029-sysusersd-Change-the-user-root-s-home-direct.patch \
 "

  # patches needed by musl
@@ -260,10 +260,6 @@ EXTRA_OEMESON += "-Dkexec-path=${sbindir}/kexec \
  # The 60 seconds is watchdog's default vaule.
  WATCHDOG_TIMEOUT ??= "60"

-do_configure:prepend() {
-  sed s@:ROOT_HOME:@${ROOT_HOME}@g ${WORKDIR}/basic.conf.in > 
${S}/sysusers.d/basic.conf.in
-}
-
  do_install() {
 meson_do_install
 install -d ${D}/${base_sbindir}
@@ -375,6 +371,11 @@ do_install() {
  sed -i -e 
's/#RebootWatchdogSec=10min/RebootWatchdogSec=${WATCHDOG_TIMEOUT}/' \
  ${D}/${sysconfdir}/systemd/system.conf
  fi
+
+   # The root home directory specified in sysusers.d/ was patched to
+   # `:ROOT_HOME:`. Change now to the correct path given by ${ROOT_HOME}.
+   [ ! -f ${D}${libdir}/sysusers.d/basic.conf ] ||
+   sed -i s@:ROOT_HOME:@${ROOT_HOME}@g 
${D}${libdir}/sysusers.d/basic.conf
  }

  python populate_packages:prepend (){

---
base-commit: 095021ac61c1df357e5f1868badd38038004317f
change-id: 20231005-sysusers3-f448703f294f

Best regards,
--
Louis Rannou 





-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188771): 
https://lists.openembedded.org/g/openembedded-core/message/188771
Mute This Topic: https://lists.openembedded.org/mt/101785446/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]

[OE-core] [PATCH] scripts/resulttool: do not try to parse metadata as tests results

2023-10-06 Thread Alexis Lothoré via lists . openembedded . org
From: Alexis Lothoré 

When regression report is computed during a CI build, a lot of errors
often appears regarding missing test status:

ERROR: Failed to retrieved base test case status: ptestresult.sections
ERROR: Failed to retrieved base test case status: ptestresult.sections
ERROR: Failed to retrieved base test case status: reproducible
ERROR: Failed to retrieved base test case status: reproducible.rawlogs
[...]

Those errors are caused by entries in test results which are not exactly
test results (i.e. an entry with a relevant "status" field containing value 
such as
"PASSED", "FAILED, "SKIPPED", etc) but additional data, which depends on
the log parser associated to the test, or tests which store results in a
different way. For example, the ptestresult.sections entry is generated by
the ptest log parser and can contain additional info about ptest such as
"begin", "end", "duration", "exitcode" or "timeout". Another example is a
"reproducible" section, which does not have a "status" field but rather
contains a big "files" entry containing lists of identical, missing, or
different files between two builds.

Remove those errors by adding a list of known entries which do not hold
test results as expected by resulttool, and by ignoring those keys when
encountered during test results comparison. I could also have completely
removed the warning about missing test case status, but that would
silently hide any real future issue with relevant test results

Signed-off-by: Alexis Lothoré 
---
 scripts/lib/resulttool/regression.py | 14 ++
 1 file changed, 14 insertions(+)

diff --git a/scripts/lib/resulttool/regression.py 
b/scripts/lib/resulttool/regression.py
index 3d64b8f4af7c..e15a268c0206 100644
--- a/scripts/lib/resulttool/regression.py
+++ b/scripts/lib/resulttool/regression.py
@@ -78,6 +78,16 @@ STATUS_STRINGS = {
 "None": "No matching test result"
 }
 
+TEST_KEY_WHITELIST = [
+"ltpposixresult.rawlogs",
+"ltpposixresult.sections",
+"ltpresult.rawlogs",
+"ltpresult.sections",
+"ptestresult.sections",
+"reproducible",
+"reproducible.rawlogs"
+]
+
 def test_has_at_least_one_matching_tag(test, tag_list):
 return "oetags" in test and any(oetag in tag_list for oetag in 
test["oetags"])
 
@@ -189,6 +199,10 @@ def compare_result(logger, base_name, target_name, 
base_result, target_result):
 
 if base_result and target_result:
 for k in base_result:
+# Some entries present in test results are known not to be test
+# results but metadata about tests
+if k in TEST_KEY_WHITELIST:
+continue
 base_testcase = base_result[k]
 base_status = base_testcase.get('status')
 if base_status:
-- 
2.42.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188770): 
https://lists.openembedded.org/g/openembedded-core/message/188770
Mute This Topic: https://lists.openembedded.org/mt/101797859/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH 2/3] cmake.bbclass: Add *.cmake to CONFIGURE_FILES

2023-10-06 Thread Peter Kjellerstedt
> -Original Message-
> From: openembedded-core@lists.openembedded.org 
>  On Behalf Of Peter Kjellerstedt
> Sent: den 3 oktober 2023 03:05
> To: openembedded-core@lists.openembedded.org
> Subject: [OE-core] [PATCH 2/3] cmake.bbclass: Add *.cmake to CONFIGURE_FILES
> 
> This makes do_configure depend on changes to any cmake file when
> externalsrc is active.
> 
> Signed-off-by: Peter Kjellerstedt 
> ---
>  meta/classes-recipe/cmake.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes-recipe/cmake.bbclass 
> b/meta/classes-recipe/cmake.bbclass
> index 41748b0820..d978b88944 100644
> --- a/meta/classes-recipe/cmake.bbclass
> +++ b/meta/classes-recipe/cmake.bbclass
> @@ -156,7 +156,7 @@ EOF
> 
>  addtask generate_toolchain_file after do_patch before do_configure
> 
> -CONFIGURE_FILES = "CMakeLists.txt"
> +CONFIGURE_FILES = "CMakeLists.txt *.cmake"
> 
>  do_configure[cleandirs] = "${@d.getVar('B') if d.getVar('S') != 
> d.getVar('B') else ''}"
> 

Any reason this was not integrated today together with the other two 
patches in this series? 

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188769): 
https://lists.openembedded.org/g/openembedded-core/message/188769
Mute This Topic: https://lists.openembedded.org/mt/101797468/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 1/2] oeqa/selftest/devtool: abort if a local workspace already exist

2023-10-06 Thread Julien Stephan
if user run devtool selftests with a local workspacelayer
the tests fail with various error such as:

- devtool.DevtoolAddTests.test_devtool_add just hangs
- devtool.DevtoolModifyTests.* fail with the following error:

 ERROR: Found duplicated BBFILE_COLLECTIONS 'workspacelayer', check 
bblayers.conf or layer.conf to fix it.
 Found duplicated BBFILE_COLLECTIONS 'workspacelayer', check bblayers.conf or 
layer.conf to fix it.

Check if a workspacelayer exists, warn the user and abort the tests

Signed-off-by: Julien Stephan 
---
 meta/lib/oeqa/selftest/cases/devtool.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/lib/oeqa/selftest/cases/devtool.py 
b/meta/lib/oeqa/selftest/cases/devtool.py
index b577f6d62a1..c36b1efa78b 100644
--- a/meta/lib/oeqa/selftest/cases/devtool.py
+++ b/meta/lib/oeqa/selftest/cases/devtool.py
@@ -27,6 +27,9 @@ def setUpModule():
 corecopydir = os.path.join(templayerdir, 'core-copy')
 bblayers_conf = os.path.join(os.environ['BUILDDIR'], 'conf', 
'bblayers.conf')
 edited_layers = []
+# make sure user doesn't have a local workspace
+result = runCmd('bitbake-layers show-layers')
+assert "workspacelayer" not in result.output, "Devtool test suite cannot 
be run with a local workspace directory"
 
 # We need to take a copy of the meta layer so we can modify it and not
 # have any races against other tests that might be running in parallel
-- 
2.41.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188767): 
https://lists.openembedded.org/g/openembedded-core/message/188767
Mute This Topic: https://lists.openembedded.org/mt/101797308/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 2/2] oeqa/selftest/devtool: remove spaces on empty line

2023-10-06 Thread Julien Stephan
Signed-off-by: Julien Stephan 
---
 meta/lib/oeqa/selftest/cases/devtool.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/selftest/cases/devtool.py 
b/meta/lib/oeqa/selftest/cases/devtool.py
index c36b1efa78b..b5c488be8e8 100644
--- a/meta/lib/oeqa/selftest/cases/devtool.py
+++ b/meta/lib/oeqa/selftest/cases/devtool.py
@@ -575,7 +575,7 @@ class DevtoolAddTests(DevtoolBase):
 checkvars['S'] = '${WORKDIR}/MarkupSafe-%s' % testver
 checkvars['SRC_URI'] = url
 self._test_recipe_contents(recipefile, checkvars, [])
- 
+
 def test_devtool_add_fetch_git(self):
 tempdir = tempfile.mkdtemp(prefix='devtoolqa')
 self.track_for_cleanup(tempdir)
-- 
2.41.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188768): 
https://lists.openembedded.org/g/openembedded-core/message/188768
Mute This Topic: https://lists.openembedded.org/mt/101797309/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2] systemd: sysusersd: change how the root home directory is set

2023-10-06 Thread Julien Stephan
Le jeu. 5 oct. 2023 à 22:55, Louis Rannou  a écrit :
>
> From: Louis Rannou 
>
> Modifying the sources brings troubles in the devtool command. Therefore it is
> better to patch the source with a custom variable, and later replace it with 
> the
> correct root path.
>
> Suggestion from :
> 
>
> Signed-off-by: Louis Rannou 
> Suggested-by: Peter Kjellerstedt 
> Signed-off-by: Louis Rannou 
> ---
>  ...usersd-Change-the-user-root-s-home-direct.patch | 31 
> ++
>  meta/recipes-core/systemd/systemd_254.4.bb | 11 
>  2 files changed, 37 insertions(+), 5 deletions(-)
>
> diff --git 
> a/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
>  
> b/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
> new file mode 100644
> index 00..dd0d96d11b
> --- /dev/null
> +++ 
> b/meta/recipes-core/systemd/systemd/0029-sysusersd-Change-the-user-root-s-home-direct.patch
> @@ -0,0 +1,31 @@
> +From bf97001978cdefad644f7b4b909f281368e5a4dd Mon Sep 17 00:00:00 2001
> +From: Kas User 

Hi Louis,

Is this intended to have kas user and email address instead of yours?

> +Date: Thu, 27 Jul 2023 08:23:06 +
> +Subject: [PATCH] sysusers.d: Change the user root's home directory
> +
> +The default sysusers basic.conf.in file sets the root home directory to 
> `/root`
> +and does not permit its configuration. Change this to `:ROOT_HOME:` which 
> must
> +be set before the installation.
> +
> +The upstream considers the root home directory should not be changed
> +
> +
> +Upstream-Status: Denied
> +Signed-off-by: Louis Rannou 
> +---
> + sysusers.d/basic.conf.in | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/sysusers.d/basic.conf.in b/sysusers.d/basic.conf.in
> +index 0aec080a4c..f26c224341 100644
> +--- a/sysusers.d/basic.conf.in
>  b/sysusers.d/basic.conf.in
> +@@ -7,7 +7,7 @@
> +
> + # The superuser
> + g root0   --
> +-u root0:0 "Super User" /root
> ++u root0:0 "Super User" :ROOT_HOME:
> +
> + # The nobody user/group for NFS file systems
> + g {{NOBODY_GROUP_NAME}} 65534   --
> diff --git a/meta/recipes-core/systemd/systemd_254.4.bb 
> b/meta/recipes-core/systemd/systemd_254.4.bb
> index 77724eb822..3adf62935d 100644
> --- a/meta/recipes-core/systemd/systemd_254.4.bb
> +++ b/meta/recipes-core/systemd/systemd_254.4.bb
> @@ -21,7 +21,6 @@ REQUIRED_DISTRO_FEATURES += "systemd"
>  SRC_URI += " \
> file://touchscreen.rules \
> file://00-create-volatile.conf \
> -   file://basic.conf.in \

If this file is not needed anymore you should probably delete it with
this commit

Cheers
Julien

> ${@bb.utils.contains('PACKAGECONFIG', 
> 'polkit_hostnamed_fallback', 
> 'file://org.freedesktop.hostname1_no_polkit.conf', '', d)} \
> ${@bb.utils.contains('PACKAGECONFIG', 
> 'polkit_hostnamed_fallback', 'file://00-hostnamed-network-user.conf', '', d)} 
> \
> file://init \
> @@ -30,6 +29,7 @@ SRC_URI += " \
> 
> file://0002-binfmt-Don-t-install-dependency-links-at-install-tim.patch \
> file://0008-implment-systemd-sysv-install-for-OE.patch \
> 
> file://0004-Move-sysusers.d-sysctl.d-binfmt.d-modules-load.d-to-.patch \
> +   file://0029-sysusersd-Change-the-user-root-s-home-direct.patch \
> "
>
>  # patches needed by musl
> @@ -260,10 +260,6 @@ EXTRA_OEMESON += "-Dkexec-path=${sbindir}/kexec \
>  # The 60 seconds is watchdog's default vaule.
>  WATCHDOG_TIMEOUT ??= "60"
>
> -do_configure:prepend() {
> -  sed s@:ROOT_HOME:@${ROOT_HOME}@g ${WORKDIR}/basic.conf.in > 
> ${S}/sysusers.d/basic.conf.in
> -}
> -
>  do_install() {
> meson_do_install
> install -d ${D}/${base_sbindir}
> @@ -375,6 +371,11 @@ do_install() {
>  sed -i -e 
> 's/#RebootWatchdogSec=10min/RebootWatchdogSec=${WATCHDOG_TIMEOUT}/' \
>  ${D}/${sysconfdir}/systemd/system.conf
>  fi
> +
> +   # The root home directory specified in sysusers.d/ was patched to
> +   # `:ROOT_HOME:`. Change now to the correct path given by ${ROOT_HOME}.
> +   [ ! -f ${D}${libdir}/sysusers.d/basic.conf ] ||
> +   sed -i s@:ROOT_HOME:@${ROOT_HOME}@g 
> ${D}${libdir}/sysusers.d/basic.conf
>  }
>
>  python populate_packages:prepend (){
>
> ---
> base-commit: 095021ac61c1df357e5f1868badd38038004317f
> change-id: 20231005-sysusers3-f448703f294f
>
> Best regards,
> --
> Louis Rannou 
>
>
> 
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188766): 
https://lists.openembedded.org/g/openembedded-core/message/188766
Mute This Topic: https://lists.openembedded.org/mt/101785446/21656
Group Owner: 

Re: [OE-core] [PATCH] devtool: modify: correctly handle multiple sources in SRC_URI

2023-10-06 Thread Julien Stephan
Le ven. 6 oct. 2023 à 12:38, Richard Purdie
 a écrit :
>
> On Fri, 2023-10-06 at 12:34 +0200, Julien Stephan wrote:
> > Le sam. 30 sept. 2023 à 12:01, Julien Stephan via
> > lists.openembedded.org 
> > a écrit :
> > >
> > > Le mer. 27 sept. 2023 à 22:36, Richard Purdie
> > >  a écrit :
> > > >
> > > > On Wed, 2023-09-27 at 18:50 +0200, Julien Stephan wrote:
> > > > > [YOCTO #15162]
> > > > >
> > > > > When recipes such as bzip2 have several sources in SRC_URI, all the
> > > > > sources are correctly extracted into devtool temporary workdir, but
> > > > > currently, we only get back the primary source and all files declared
> > > > > using "files://", then the devtool temporary workdir is deleted
> > > > >
> > > > > Fix this by moving all remaining files from devtool temporary workdir
> > > > > into actual WORKDIR (except "source-date-epoch", "recipe-sysroot",
> > > > > and "recipe-sysroot-native") before deleting it
> > > > >
> > > > > Here is the stack trace from devtool modify/build bzip2:
> > > > >
> > > > > NOTE: bzip2: compiling from external source tree 
> > > > > <...>/build/workspace/sources/bzip2
> > > > > ERROR: bzip2-1.0.8-r0 do_install_ptest_base: 
> > > > > ExecutionError('<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368',
> > > > >  1, None, None)
> > > > > ERROR: Logfile of failure stored in: 
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/log.do_install_ptest_base.3368
> > > > > Log data follows:
> > > > > > DEBUG: Executing shell function do_install_ptest_base
> > > > > > NOTE: make -j 16 
> > > > > > DESTDIR=<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest
> > > > > >  install-ptest
> > > > > > sed  -n '/^runtest:/,/^install-ptest:/{/^install-ptest:/!p}' \
> > > > > >../../../../../../workspace/sources/bzip2/Makefile.am
> > > > > >   > 
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/Makefile
> > > > > > cp ../../../../../../workspace/sources/bzip2/sample1.ref  
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > > cp ../../../../../../workspace/sources/bzip2/sample2.ref  
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > > cp ../../../../../../workspace/sources/bzip2/sample3.ref  
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > > cp ../../../../../../workspace/sources/bzip2/sample1.bz2  
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > > cp ../../../../../../workspace/sources/bzip2/sample2.bz2  
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > > cp ../../../../../../workspace/sources/bzip2/sample3.bz2  
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > > ln -s /usr/bin/bzip2  
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/bzip2
> > > > > > cp: cannot stat 
> > > > > > '<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/git/commons-compress':
> > > > > >  No such file or directory
> > > > > > WARNING: 
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368:189
> > > > > >  exit 1 from 'cp -r <...>/build/tmp/work/core2-64-poky-linux/bzip2/
> > > > > 1.0.8/git/commons-compress 
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/bzip2-tests/commons-compress'
> > > > > > WARNING: Backtrace (BB generated script):
> > > > > >   #1: do_install_ptest, 
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > > > >  line 189
> > > > > >   #2: do_install_ptest_base, 
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > > > >  line 158
> > > > > >   #3: main, 
> > > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > > > >  line 226
> > > > > ERROR: Task 
> > > > > (<...>/poky/meta/recipes-extended/bzip2/bzip2_1.0.8.bb:do_install_ptest_base)
> > > > >  failed with exit code '1'
> > > > > NOTE: Tasks Summary: Attempted 776 tasks of which 765 didn't need to 
> > > > > be rerun and 1 failed.
> > > > >
> > > > > Summary: 1 task failed:
> > > > >   
> > > > > <...>/poky/meta/recipes-extended/bzip2/bzip2_1.0.8.bb:do_install_ptest_base
> > > > >
> > > > > Signed-off-by: Julien Stephan 
> > > > > ---
> > > > >  scripts/lib/devtool/standard.py | 8 
> > > > >  1 file changed, 8 insertions(+)
> > > > >
> > > > > diff --git a/scripts/lib/devtool/standard.py 
> > > > > b/scripts/lib/devtool/standard.py
> > > > > index d53fb810071..6c8ad7a6c8c 100644
> > > > > --- a/scripts/lib/devtool/standard.py
> > > > > +++ 

Re: [OE-core] [PATCH v5 0/5] Add bblock helper scripts

2023-10-06 Thread Julien Stephan
Le ven. 6 oct. 2023 à 12:59, Richard Purdie
 a écrit :
>
> On Fri, 2023-10-06 at 12:56 +0200, Alexandre Belloni wrote:
> > I started an oe-selftest that passed, I'm going to carry that in my
> > branch to see whether there are other issues.
>
> I've had a couple of runs in my branch and not seen any issue from that
> as yet FWIW.
>
> I am leaning towards merging this at this point.
>
> Cheers,
>
> Richard

Hi Richard and Alexandre,

That is very good news :)

Cheers

Julien

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188764): 
https://lists.openembedded.org/g/openembedded-core/message/188764
Mute This Topic: https://lists.openembedded.org/mt/101570279/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core][dunfell][PATCH] openssl: Upgrade 1.1.1v -> 1.1.1w

2023-10-06 Thread Sourav Kumar Pramanik
From: Sourav Pramanik 

https://www.openssl.org/news/openssl-1.1.1-notes.html

Major changes between OpenSSL 1.1.1v and OpenSSL 1.1.1w [11 Sep 2023]

* Fix POLY1305 MAC implementation corrupting XMM registers on Windows 
(CVE-2023-4807)

Signed-off-by: Sourav Kumar Pramanik 
---
 .../openssl/{openssl_1.1.1v.bb => openssl_1.1.1w.bb}| 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-connectivity/openssl/{openssl_1.1.1v.bb => 
openssl_1.1.1w.bb} (98%)

diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1v.bb 
b/meta/recipes-connectivity/openssl/openssl_1.1.1w.bb
similarity index 98%
rename from meta/recipes-connectivity/openssl/openssl_1.1.1v.bb
rename to meta/recipes-connectivity/openssl/openssl_1.1.1w.bb
index d1222dc470..8a53b06862 100644
--- a/meta/recipes-connectivity/openssl/openssl_1.1.1v.bb
+++ b/meta/recipes-connectivity/openssl/openssl_1.1.1w.bb
@@ -26,7 +26,7 @@ SRC_URI_append_class-nativesdk = " \
file://environment.d-openssl.sh \
"
 
-SRC_URI[sha256sum] = 
"d6697e2871e77238460402e9362d47d18382b15ef9f246aba6c7bd780d38a6b0"
+SRC_URI[sha256sum] = 
"cf3098950cb4d853ad95c0841f1f9c6d3dc102dccfcacd521d93925208b76ac8"
 
 inherit lib_package multilib_header multilib_script ptest
 MULTILIB_SCRIPTS = "${PN}-bin:${bindir}/c_rehash"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188763): 
https://lists.openembedded.org/g/openembedded-core/message/188763
Mute This Topic: https://lists.openembedded.org/mt/101796386/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 1/2] cryptodev: Update to latest git for 6.5 kernel fixes

2023-10-06 Thread Richard Purdie
Update to pull in recent git changes which allow cryptodev to work
with 6.5 kernels.

Signed-off-by: Richard Purdie 
---
 meta/recipes-kernel/cryptodev/cryptodev.inc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/cryptodev/cryptodev.inc 
b/meta/recipes-kernel/cryptodev/cryptodev.inc
index 61c688f1d0e..64a9c2926b8 100644
--- a/meta/recipes-kernel/cryptodev/cryptodev.inc
+++ b/meta/recipes-kernel/cryptodev/cryptodev.inc
@@ -10,7 +10,8 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
 
 SRC_URI = 
"git://github.com/cryptodev-linux/cryptodev-linux;branch=master;protocol=https \
"
-SRCREV = "795f5fda1485d447cdbad342e8304be317d4f7e8"
+SRCREV = "bb8bc7cf60d2c0b097c8b3b0e807f805b577a53f"
+PV = "1.13+git${SRCPV}"
 
 S = "${WORKDIR}/git"
 
-- 
2.39.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188761): 
https://lists.openembedded.org/g/openembedded-core/message/188761
Mute This Topic: https://lists.openembedded.org/mt/101795561/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 2/2] qemux86/qemuarm: Drop kernel version overrides

2023-10-06 Thread Richard Purdie
Drop the version overrides for the kernel for the x86 and arm machines
so we can go back to following the distro versions. The reasons for
these versions is mostly historical at this point as the issues were
resolved.

Signed-off-by: Richard Purdie 
---
 meta/conf/machine/include/x86/x86-base.inc | 1 -
 meta/conf/machine/qemuarm.conf | 4 
 meta/conf/machine/qemuarmv5.conf   | 1 -
 3 files changed, 6 deletions(-)

diff --git a/meta/conf/machine/include/x86/x86-base.inc 
b/meta/conf/machine/include/x86/x86-base.inc
index 050c20af70c..fc6c39148db 100644
--- a/meta/conf/machine/include/x86/x86-base.inc
+++ b/meta/conf/machine/include/x86/x86-base.inc
@@ -18,7 +18,6 @@ SERIAL_CONSOLES ?= "115200;ttyS0"
 # kernel-related variables
 #
 PREFERRED_PROVIDER_virtual/kernel ??= "linux-yocto"
-PREFERRED_VERSION_linux-yocto ??= "6.4%"
 
 #
 # XSERVER subcomponents, used to build the XSERVER variable
diff --git a/meta/conf/machine/qemuarm.conf b/meta/conf/machine/qemuarm.conf
index 04216cdc2e0..881733eb2da 100644
--- a/meta/conf/machine/qemuarm.conf
+++ b/meta/conf/machine/qemuarm.conf
@@ -28,8 +28,4 @@ QB_ROOTFS_OPT = "-drive 
id=disk0,file=@ROOTFS@,if=none,format=raw -device virtio
 QB_SERIAL_OPT = "-device virtio-serial-device -chardev null,id=virtcon -device 
virtconsole,chardev=virtcon"
 QB_TCPSERIAL_OPT = "-device virtio-serial-device -chardev 
socket,id=virtcon,port=@PORT@,host=127.0.0.1 -device 
virtconsole,chardev=virtcon"
 
-# 6.4 causes Xorg to fail to read the virtio framebuffer so stick with 6.1 
until
-# that is resolved.
-PREFERRED_VERSION_linux-yocto ??= "6.1%"
-
 KMACHINE:qemuarm = "qemuarma15"
diff --git a/meta/conf/machine/qemuarmv5.conf b/meta/conf/machine/qemuarmv5.conf
index deb4869c0aa..f0a86f1e4c7 100644
--- a/meta/conf/machine/qemuarmv5.conf
+++ b/meta/conf/machine/qemuarmv5.conf
@@ -16,5 +16,4 @@ QB_GRAPHICS = "-device virtio-gpu-pci"
 QB_OPT_APPEND = "-device qemu-xhci -device usb-tablet -device usb-kbd"
 QB_DTB = "${@oe.utils.version_less_or_equal('PREFERRED_VERSION_linux-yocto', 
'4.7', '', 'zImage-versatile-pb.dtb', d)}"
 
-PREFERRED_VERSION_linux-yocto ??= "6.4%"
 KMACHINE:qemuarmv5 = "arm-versatile-926ejs"
-- 
2.39.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188762): 
https://lists.openembedded.org/g/openembedded-core/message/188762
Mute This Topic: https://lists.openembedded.org/mt/101795562/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 4/4] busybox: use ttyrun to run getty only if the terminal exists

2023-10-06 Thread Ross Burton
From: Ross Burton 

Wrap calls to start_getty with ttyrun, so that getty isn't started if
the device doesn't exist.  As we know start_getty is only called when
the device exists we can remove the partial workaround for this problem
in that scripts too.

This neatly obsoletes SERIAL_CONSOLES_CHECK, whose sole purpose was to
check what terminals are present at boot and rewrite inittab.  Notably,
this meant that SERIAL_CONSOLES_CHECK made using a read-only rootfs
impossible.

(From OE-Core rev: 950ecaabc04836efc346be0ac7e0331e2378872b)

Signed-off-by: Ross Burton 
Signed-off-by: Richard Purdie 
---
 .../busybox/busybox-inittab_1.36.1.bb | 46 +--
 1 file changed, 12 insertions(+), 34 deletions(-)

diff --git a/meta/recipes-core/busybox/busybox-inittab_1.36.1.bb 
b/meta/recipes-core/busybox/busybox-inittab_1.36.1.bb
index 868d7a230f6..462c9bb9c5f 100644
--- a/meta/recipes-core/busybox/busybox-inittab_1.36.1.bb
+++ b/meta/recipes-core/busybox/busybox-inittab_1.36.1.bb
@@ -15,14 +15,17 @@ do_compile() {
 do_install() {
install -d ${D}${sysconfdir}
install -D -m 0644 ${WORKDIR}/inittab ${D}${sysconfdir}/inittab
-   tmp="${SERIAL_CONSOLES}"
-   [ -n "$tmp" ] && echo >> ${D}${sysconfdir}/inittab
-   for i in $tmp
-   do
-   j=`echo ${i} | sed s/\;/\ /g`
-   id=`echo ${i} | sed -e 's/^.*;//' -e 's/;.*//'`
-   echo "$id::respawn:${base_sbindir}/getty ${j}" >> 
${D}${sysconfdir}/inittab
-   done
+
+CONSOLES="${SERIAL_CONSOLES}"
+for s in $CONSOLES
+do
+speed=$(echo $s | cut -d\; -f 1)
+device=$(echo $s | cut -d\; -f 2)
+label=$(echo $device | sed -e 's/tty//' | tail --bytes=5)
+
+echo "$label::respawn:${sbindir}/ttyrun $device 
${base_bindir}/start_getty $speed $device vt102" >> ${D}${sysconfdir}/inittab
+done
+
if [ "${USE_VT}" = "1" ]; then
cat <>${D}${sysconfdir}/inittab
 # ${base_sbindir}/getty invocations for the runlevels.
@@ -45,32 +48,6 @@ EOF
 
 }
 
-pkg_postinst:${PN} () {
-# run this on host and on target
-if [ "${SERIAL_CONSOLES_CHECK}" = "" ]; then
-   exit 0
-fi
-}
-
-pkg_postinst_ontarget:${PN} () {
-# run this on the target
-if [ -e /proc/consoles ]; then
-tmp="${SERIAL_CONSOLES_CHECK}"
-for i in $tmp
-do
-j=`echo ${i} | sed -e s/^.*\;//g -e s/\:.*//g`
-k=`echo ${i} | sed s/^.*\://g`
-if [ -z "`grep ${j} /proc/consoles`" ]; then
-if [ -z "${k}" ] || [ -z "`grep ${k} /proc/consoles`" 
] || [ ! -e /dev/${j} ]; then
-sed -i -e /^.*${j}\ /d -e /^.*${j}$/d 
/etc/inittab
-fi
-fi
-done
-kill -HUP 1
-else
-exit 1
-fi
-}
 
 # SERIAL_CONSOLES is generally defined by the MACHINE .conf.
 # Set PACKAGE_ARCH appropriately.
@@ -79,6 +56,7 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
 FILES:${PN} = "${sysconfdir}/inittab"
 CONFFILES:${PN} = "${sysconfdir}/inittab"
 
+RDEPENDS:${PN} = "ttyrun"
 RCONFLICTS:${PN} = "sysvinit-inittab"
 
 USE_VT ?= "1"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188760): 
https://lists.openembedded.org/g/openembedded-core/message/188760
Mute This Topic: https://lists.openembedded.org/mt/101795373/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 2/4] ttyrun: add new recipe

2023-10-06 Thread Ross Burton
From: Ross Burton 

ttyrun is a small tool from IBM's s390-tools package to run a command
if the specified TTY is present, and hang if not.

This is useful so that you can list all of the potential terminals in
inittab and not have getty quiting instantly when a device isn't
present, resulting in the "respawning too fast" errors.

Note that DISTRELEASE has to be set as otherwise "build$(DATE)" is used,
which is non-reproducible.

Signed-off-by: Ross Burton 
---
 meta/conf/distro/include/maintainers.inc  |  1 +
 meta/recipes-core/ttyrun/ttyrun_2.29.0.bb | 33 +++
 2 files changed, 34 insertions(+)
 create mode 100644 meta/recipes-core/ttyrun/ttyrun_2.29.0.bb

diff --git a/meta/conf/distro/include/maintainers.inc 
b/meta/conf/distro/include/maintainers.inc
index dcc507bb982..b2579fc44c3 100644
--- a/meta/conf/distro/include/maintainers.inc
+++ b/meta/conf/distro/include/maintainers.inc
@@ -810,6 +810,7 @@ RECIPE_MAINTAINER:pn-texinfo-dummy-native = "Anuj Mittal 

 RECIPE_MAINTAINER:pn-tiff = "Alexander Kanavin "
 RECIPE_MAINTAINER:pn-time = "Robert Yang "
 RECIPE_MAINTAINER:pn-ttf-bitstream-vera = "Anuj Mittal "
+RECIPE_MAINTAINER:pn-ttyrun = "Ross Burton "
 RECIPE_MAINTAINER:pn-tzcode-native = "Unassigned "
 RECIPE_MAINTAINER:pn-tzdata = "Unassigned "
 RECIPE_MAINTAINER:pn-u-boot = "Marek Vasut "
diff --git a/meta/recipes-core/ttyrun/ttyrun_2.29.0.bb 
b/meta/recipes-core/ttyrun/ttyrun_2.29.0.bb
new file mode 100644
index 000..e0956b9ac61
--- /dev/null
+++ b/meta/recipes-core/ttyrun/ttyrun_2.29.0.bb
@@ -0,0 +1,33 @@
+SUMMARY = "Start the program if the specified terminal device is available."
+DESCRIPTION = "ttyrun is typically used to prevent a respawn through the \
+init(8) program when a terminal is not available."
+HOMEPAGE = "https://github.com/ibm-s390-linux/s390-tools;
+
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=f5118f167b055bfd7c3450803f1847af"
+
+SRC_URI = 
"git://github.com/ibm-s390-linux/s390-tools;protocol=https;branch=master"
+SRCREV = "d9ce54dee3ac3827e76624352293a83eb05c727e"
+
+S = "${WORKDIR}/git"
+
+EXTRA_OEMAKE = "\
+V=1 \
+CC="${CC}" \
+DISTRELEASE=${PR} \
+"
+
+# We just want ttyrun and not the rest of s390-utils
+
+do_configure() {
+oe_runmake -C ${S}/iucvterm/src clean
+}
+
+do_compile() {
+oe_runmake -C ${S}/iucvterm/src ttyrun
+}
+
+do_install() {
+install -d ${D}${sbindir}
+install ${S}/iucvterm/src/ttyrun ${D}${sbindir}
+}
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188758): 
https://lists.openembedded.org/g/openembedded-core/message/188758
Mute This Topic: https://lists.openembedded.org/mt/101795370/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 3/4] sysvinit-inittab: use ttyrun to run getty only if the terminal exists

2023-10-06 Thread Ross Burton
From: Ross Burton 

Wrap calls to start_getty with ttyrun, so that getty isn't started if
the device doesn't exist.  As we know start_getty is only called when
the device exists we can remove the partial workaround for this problem
in that scripts too.

This neatly obsoletes SERIAL_CONSOLES_CHECK, whose sole purpose was to
check what terminals are present at boot and rewrite inittab.  Notably,
this meant that SERIAL_CONSOLES_CHECK made using a read-only rootfs
impossible.

(From OE-Core rev: f4fd17d5a5e4eaa31995d3ca52c871cfbdc0df68)

Signed-off-by: Ross Burton 
Signed-off-by: Richard Purdie 
---
 .../sysvinit/sysvinit-inittab/start_getty | 11 ---
 .../sysvinit/sysvinit-inittab_2.88dsf.bb  | 30 ++-
 2 files changed, 7 insertions(+), 34 deletions(-)

diff --git a/meta/recipes-core/sysvinit/sysvinit-inittab/start_getty 
b/meta/recipes-core/sysvinit/sysvinit-inittab/start_getty
index f60409eae3a..f5671ee53d2 100644
--- a/meta/recipes-core/sysvinit/sysvinit-inittab/start_getty
+++ b/meta/recipes-core/sysvinit/sysvinit-inittab/start_getty
@@ -1,5 +1,9 @@
 #!/bin/sh
 
+SPEED=$1
+DEVICE=$2
+TERM=$3
+
 # busybox' getty does this itself, util-linux' agetty needs extra help
 getty="/sbin/getty"
 case $(readlink -f "${getty}") in
@@ -13,9 +17,4 @@ case $(readlink -f "${getty}") in
 ;;
 esac
 
-if [ -e /sys/class/tty/$2 -a -c /dev/$2 ]; then
-   ${setsid:-} ${getty} ${options:-} -L $1 $2 $3
-else
-# Prevent respawning to fast error if /dev entry does not exist
-sleep 1000
-fi
+${setsid:-} ${getty} ${options:-} -L $SPEED $DEVICE $TERM
diff --git a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb 
b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
index e70dc705c05..6bbe517df1b 100644
--- a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
+++ b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
@@ -28,7 +28,7 @@ do_install() {
 device=$(echo $s | cut -d\; -f 2)
 label=$(echo $device | sed -e 's/tty//' | tail --bytes=5)
 
-echo "$label:12345:respawn:${base_bindir}/start_getty $speed $device 
vt102" >> ${D}${sysconfdir}/inittab
+echo "$label:12345:respawn:${sbindir}/ttyrun $device 
${base_bindir}/start_getty $speed $device vt102" >> ${D}${sysconfdir}/inittab
 done
 
 if [ "${USE_VT}" = "1" ]; then
@@ -52,33 +52,6 @@ EOF
 fi
 }
 
-pkg_postinst:${PN} () {
-# run this on host and on target
-if [ "${SERIAL_CONSOLES_CHECK}" = "" ]; then
-   exit 0
-fi
-}
-
-pkg_postinst_ontarget:${PN} () {
-# run this on the target
-if [ -e /proc/consoles ]; then
-   tmp="${SERIAL_CONSOLES_CHECK}"
-   for i in $tmp
-   do
-   j=`echo ${i} | sed -e s/^.*\;//g -e s/\:.*//g`
-   k=`echo ${i} | sed s/^.*\://g`
-   if [ -z "`grep ${j} /proc/consoles`" ]; then
-   if [ -z "${k}" ] || [ -z "`grep ${k} /proc/consoles`" ] 
|| [ ! -e /dev/${j} ]; then
-   sed -i -e /^.*${j}\ /d -e /^.*${j}$/d 
/etc/inittab
-   fi
-   fi
-   done
-   kill -HUP 1
-else
-   exit 1
-fi
-}
-
 # USE_VT and SERIAL_CONSOLES are generally defined by the MACHINE .conf.
 # Set PACKAGE_ARCH appropriately.
 PACKAGE_ARCH = "${MACHINE_ARCH}"
@@ -89,4 +62,5 @@ CONFFILES:${PN} = "${sysconfdir}/inittab"
 USE_VT ?= "1"
 SYSVINIT_ENABLED_GETTYS ?= "1"
 
+RDEPENDS:${PN} = "ttyrun"
 RCONFLICTS:${PN} = "busybox-inittab"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188759): 
https://lists.openembedded.org/g/openembedded-core/message/188759
Mute This Topic: https://lists.openembedded.org/mt/101795371/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 1/4] sysvinit-initab: rewrite loop to generate inittab

2023-10-06 Thread Ross Burton
From: Ross Burton 

Instead of one-letter variables and arcane sed expressions, use English
words and 'cut'.

(From OE-Core rev: de9833792dd0cfac6f17d5c733c263b976e89cf6)

Signed-off-by: Ross Burton 
Signed-off-by: Richard Purdie 
---
 .../sysvinit/sysvinit-inittab_2.88dsf.bb   | 14 +++---
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb 
b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
index b1f9cb4caa6..e70dc705c05 100644
--- a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
+++ b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
@@ -21,14 +21,14 @@ do_install() {
 install -m 0755 ${WORKDIR}/start_getty ${D}${base_bindir}/start_getty
 sed -e 's,/usr/bin,${bindir},g' -i ${D}${base_bindir}/start_getty
 
-set -x
-tmp="${SERIAL_CONSOLES}"
-for i in $tmp
+CONSOLES="${SERIAL_CONSOLES}"
+for s in $CONSOLES
 do
-   j=`echo ${i} | sed s/\;/\ /g`
-   l=`echo ${i} | sed -e 's/tty//' -e 's/^.*;//' -e 's/;.*//'`
-   label=`echo $l | sed 's/.*\(\)/\1/'`
-   echo "$label:12345:respawn:${base_bindir}/start_getty ${j} vt102" >> 
${D}${sysconfdir}/inittab
+speed=$(echo $s | cut -d\; -f 1)
+device=$(echo $s | cut -d\; -f 2)
+label=$(echo $device | sed -e 's/tty//' | tail --bytes=5)
+
+echo "$label:12345:respawn:${base_bindir}/start_getty $speed $device 
vt102" >> ${D}${sysconfdir}/inittab
 done
 
 if [ "${USE_VT}" = "1" ]; then
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188757): 
https://lists.openembedded.org/g/openembedded-core/message/188757
Mute This Topic: https://lists.openembedded.org/mt/101795368/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] qemuboot: reduce default size of software I/O translation buffer

2023-10-06 Thread Ross Burton
From: Ross Burton 

With 6.5+ (specifically, if DMA_BOUNCE_UNALIGNED_KMALLOC is set) the
SWIOTLB is used, and it defaults to 64MB. This is too much when there's
only 256MB of RAM, so request 0 slabs and lets the kernel round up to
the appropriate minimum (1MB on aarch64, typically). In virtual hardware
there's very little need for these bounce buffers, so the 64MB would be
mostly wasted.

Signed-off-by: Ross Burton 
---
 meta/classes-recipe/qemuboot.bbclass | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/meta/classes-recipe/qemuboot.bbclass 
b/meta/classes-recipe/qemuboot.bbclass
index e30b380c3dc..5c4bbd67371 100644
--- a/meta/classes-recipe/qemuboot.bbclass
+++ b/meta/classes-recipe/qemuboot.bbclass
@@ -107,6 +107,13 @@ QB_ROOTFS_EXTRA_OPT ?= ""
 QB_GRAPHICS ?= ""
 QB_NFSROOTFS_EXTRA_OPT ?= ""
 
+# With 6.5+ (specifically, if DMA_BOUNCE_UNALIGNED_KMALLOC is set) the SW IO 
TLB
+# is used, and it defaults to 64MB. This is too much when there's only 256MB of
+# RAM, so request 0 slabs and lets the kernel round up to the appropriate 
minimum
+# (1MB, typically). In virtual hardware there's very little need for these 
bounce
+# buffers, so the 64MB would be mostly wasted.
+QB_KERNEL_CMDLINE_APPEND:append = " swiotlb=0"
+
 # This should be kept align with ROOT_VM
 QB_DRIVE_TYPE ?= "/dev/sd"
 
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188756): 
https://lists.openembedded.org/g/openembedded-core/message/188756
Mute This Topic: https://lists.openembedded.org/mt/101795308/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] qemuboot: reduce default size of software I/O translation buffer

2023-10-06 Thread Ross Burton
From: Ross Burton 

With 6.5+ (specifically, if DMA_BOUNCE_UNALIGNED_KMALLOC is set) the
SWIOTLB is used, and it defaults to 64MB. This is too much when there's
only 256MB of RAM, so request 0 slabs and lets the kernel round up to
the appropriate minimum (1MB on aarch64, typically). In virtual hardware
there's very little need for these bounce buffers, so the 64MB would be
mostly wasted.

Signed-off-by: Ross Burton 
---
 meta/classes-recipe/qemuboot.bbclass | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/meta/classes-recipe/qemuboot.bbclass 
b/meta/classes-recipe/qemuboot.bbclass
index e30b380c3dc..5c4bbd67371 100644
--- a/meta/classes-recipe/qemuboot.bbclass
+++ b/meta/classes-recipe/qemuboot.bbclass
@@ -107,6 +107,13 @@ QB_ROOTFS_EXTRA_OPT ?= ""
 QB_GRAPHICS ?= ""
 QB_NFSROOTFS_EXTRA_OPT ?= ""
 
+# With 6.5+ (specifically, if DMA_BOUNCE_UNALIGNED_KMALLOC is set) the SW IO 
TLB
+# is used, and it defaults to 64MB. This is too much when there's only 256MB of
+# RAM, so request 0 slabs and lets the kernel round up to the appropriate 
minimum
+# (1MB, typically). In virtual hardware there's very little need for these 
bounce
+# buffers, so the 64MB would be mostly wasted.
+QB_KERNEL_CMDLINE_APPEND:append = " swiotlb=0"
+
 # This should be kept align with ROOT_VM
 QB_DRIVE_TYPE ?= "/dev/sd"
 
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188755): 
https://lists.openembedded.org/g/openembedded-core/message/188755
Mute This Topic: https://lists.openembedded.org/mt/101795308/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v5 0/5] Add bblock helper scripts

2023-10-06 Thread Richard Purdie
On Fri, 2023-10-06 at 12:56 +0200, Alexandre Belloni wrote:
> I started an oe-selftest that passed, I'm going to carry that in my
> branch to see whether there are other issues.

I've had a couple of runs in my branch and not seen any issue from that
as yet FWIW.

I am leaning towards merging this at this point.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188754): 
https://lists.openembedded.org/g/openembedded-core/message/188754
Mute This Topic: https://lists.openembedded.org/mt/101570279/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v5 0/5] Add bblock helper scripts

2023-10-06 Thread Alexandre Belloni via lists.openembedded.org
I started an oe-selftest that passed, I'm going to carry that in my
branch to see whether there are other issues.

On 05/10/2023 10:02:22+0200, Julien Stephan wrote:
> Hi Jose,
> 
> I just pushed my branch on poky-contrib
> (https://git.yoctoproject.org/poky-contrib/log/?h=jstephan/bblock)
> with the fix you just suggested.
> 
> I do not reproduce the issue locally, so I am not sure this fix the problem..
> Richard, Alexandre what do you think?  Can you restart a build on autobuidler?
> 
> Cheers
> Julien
> 
> Le lun. 2 oct. 2023 à 12:25, Jose Quaresma  a écrit :
> >
> > Hi Julien,
> >
> > Julien Stephan  escreveu no dia sábado, 30/09/2023 
> > à(s) 15:13:
> >>
> >> Le mer. 27 sept. 2023 à 23:17, Richard Purdie
> >>  a écrit :
> >> >
> >> > On Wed, 2023-09-27 at 14:47 +0200, Julien Stephan wrote:
> >> > > Le mer. 27 sept. 2023 à 12:16, Alexandre Belloni
> >> > >  a écrit :
> >> > > >
> >> > > > Hello,
> >> > > >
> >> > > > I had a build were all the tests passed, then I had this failure:
> >> > > >
> >> > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/86/builds/5812/steps/14/logs/stdio
> >> > > >
> >> > >
> >> > > Hi Alexandre,
> >> > > I only see
> >> > >
> >> > >bblock.BBLock.test_lock_single_recipe_all_tasks: FAILED
> >> > >
> >> > > but I can't see the actual error :( Is it possible to have it 
> >> > > somewhere?
> >> >
> >> > I am a bit worried this will be hard to reproduce/debug so I had a look
> >> > at things as I do really want to get things resolved.
> >> >
> >> > The error log says:
> >> >
> >> > WARNING: The quilt:do_build sig is computed to be 
> >> > 255536826e64d835fcc6b5a277e9a3dacaae75318e608fa2df289bdfa23df40e, but 
> >> > the sig is locked to 
> >> > 0f48875f011499948d53edc6abbe9e4efd617d5ef9233055e4b61411cf01cb87 in 
> >> > SIGGEN_LOCKEDSIGS_core2-64
> >> >
> >> > and I couldn't reproduce those stamps in a local build or on the
> >> > builder where it failed (once I got access which took a while).
> >> >
> >> > The "while" is important as I think in the meantime things changed
> >> > which effected the results.
> >> >
> >> > On the autobuilder, "bitbake quilt -S none" generated stamp files for
> >> > quilt. I did this for "build" and "build-st-3046552", the latter being
> >> > the one where the test failed. There were two different do_build stamps
> >> > generated. The diff:
> >> >
> >> > bitbake-diffsigs 
> >> > tmp/stamps/core2-64-poky-linux/quilt/0.67.do_build.sigdata.b502a759bb6bceb6c8ccd76b57db2612501bb66bd9c46ece5f99f8bf16d7ac4a
> >> >   
> >> > ../build/tmp/stamps/core2-64-poky-linux/quilt/0.67.do_build.sigdata.7cf16a1f87d5058e134f3f6fc02612515e28784625413c87d3545e93ae0ab407
> >> > NOTE: Reconnecting to bitbake server...
> >> > Hash for task dependency diffstat:do_package_write_rpm changed from 
> >> > 41171f7938a2ddbe244372c1cc4300d91b7dca425eb8b214c19262e4949a7f18 to 
> >> > 2211ac20553e9dddc710970cf1a663412ece5b5a2071e47c5471b3ee01efa2ce
> >> > Hash for task dependency diffstat:do_package changed from 
> >> > f3fe486831eede23d2ebbc1efe2e2510f49f5df6d5d8a8ceb7aa2ddc6e11b1ec to 
> >> > e5ff0ed8a81d8022b0981162e9b4e286886d9e281adee81175725072c3d413ec
> >> > Unable to find matching sigdata for diffstat:do_package with hash 
> >> > e5ff0ed8a81d8022b0981162e9b4e286886d9e281adee81175725072c3d413ec
> >> >
> >> > i.e. a couple of dependencies were different.
> >> >
> >> > bitbake-diffsigs 
> >> > tmp/stamps/core2-64-poky-linux/diffstat/1.65.do_package.sigdata.f3fe486831eede23d2ebbc1efe2e2510f49f5df6d5d8a8ceb7aa2ddc6e11b1ec
> >> >   
> >> > ../build/tmp/stamps/core2-64-poky-linux/diffstat/1.65.do_package.sigdata.af8ebe96c5cf424e1094c1599400a56096302e16d0f691daaeec380b963f37b2
> >> >
> >> > showed no output, i.e. they appear to be the same but different.
> >> >
> >> > My best guess at what is going on here is hash equivalence causing
> >> > problems.
> >> >
> >> > So in summary, I think the best solution will be to disable hash
> >> > equivalence for these tests.
> >> >
> >> >
> >>
> >> Hi Richard,
> >>
> >> Thank you a lot for digging into this!
> >> How can I disable the hash equivalence for these tests?
> >
> >
> > Can you try with BB_SIGNATURE_HANDLER using the bibake default config
> >
> > self.write_config('BB_SIGNATURE_HANDLER = "OEBasicHash"')
> >
> > Jose
> >
> >>
> >> Should I use BB_BASEHASH_IGNORE_VARS for that?
> >>
> >> I tried the following:
> >> self.write_config("BB_BASEHASH_IGNORE_VARS += \"TOPDIR\"")
> >>
> >> But this doesn't work, I get a lot of errors like the following:
> >> ERROR: When reparsing
> >> /home/jstephan/upstream/yocto/poky/meta/recipes-devtools/quilt/quilt_0.67.bb:do_packagedata,
> >> the basehash value changed from
> >> 8b6adb69d1f05eddb3e86143bf9a51060f4fac153742964bb4a3d53bee93d377 to
> >> 9dee5dada20d7f8ccfc89ed3114c428af8c96d665234fead22c25fb118f9f62b. The
> >> metadata is not deterministic and this needs to be fixed.
> >> ERROR: The following commands may help:
> >> ERROR: $ bitbake quilt -cdo_packagedata -Snone
> >> 

[OE-core] Yocto Project Summit 2023.11 registrations are open now!

2023-10-06 Thread Josef Holzmayr
Registration are open now! Join us for the Yocto Project Summit 2023.11 to
not only hear about the latest and greatest, but also challenges and
solutions from the whole community.

https://summit.yoctoproject.org/yocto-project-summit-2023-11/

Josef,
on behalf of the organization committee

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188752): 
https://lists.openembedded.org/g/openembedded-core/message/188752
Mute This Topic: https://lists.openembedded.org/mt/101794917/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH] devtool: modify: correctly handle multiple sources in SRC_URI

2023-10-06 Thread Richard Purdie
On Fri, 2023-10-06 at 12:34 +0200, Julien Stephan wrote:
> Le sam. 30 sept. 2023 à 12:01, Julien Stephan via
> lists.openembedded.org 
> a écrit :
> > 
> > Le mer. 27 sept. 2023 à 22:36, Richard Purdie
> >  a écrit :
> > > 
> > > On Wed, 2023-09-27 at 18:50 +0200, Julien Stephan wrote:
> > > > [YOCTO #15162]
> > > > 
> > > > When recipes such as bzip2 have several sources in SRC_URI, all the
> > > > sources are correctly extracted into devtool temporary workdir, but
> > > > currently, we only get back the primary source and all files declared
> > > > using "files://", then the devtool temporary workdir is deleted
> > > > 
> > > > Fix this by moving all remaining files from devtool temporary workdir
> > > > into actual WORKDIR (except "source-date-epoch", "recipe-sysroot",
> > > > and "recipe-sysroot-native") before deleting it
> > > > 
> > > > Here is the stack trace from devtool modify/build bzip2:
> > > > 
> > > > NOTE: bzip2: compiling from external source tree 
> > > > <...>/build/workspace/sources/bzip2
> > > > ERROR: bzip2-1.0.8-r0 do_install_ptest_base: 
> > > > ExecutionError('<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368',
> > > >  1, None, None)
> > > > ERROR: Logfile of failure stored in: 
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/log.do_install_ptest_base.3368
> > > > Log data follows:
> > > > > DEBUG: Executing shell function do_install_ptest_base
> > > > > NOTE: make -j 16 
> > > > > DESTDIR=<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest
> > > > >  install-ptest
> > > > > sed  -n '/^runtest:/,/^install-ptest:/{/^install-ptest:/!p}' \
> > > > >../../../../../../workspace/sources/bzip2/Makefile.am  
> > > > > > 
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/Makefile
> > > > > cp ../../../../../../workspace/sources/bzip2/sample1.ref  
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > cp ../../../../../../workspace/sources/bzip2/sample2.ref  
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > cp ../../../../../../workspace/sources/bzip2/sample3.ref  
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > cp ../../../../../../workspace/sources/bzip2/sample1.bz2  
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > cp ../../../../../../workspace/sources/bzip2/sample2.bz2  
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > cp ../../../../../../workspace/sources/bzip2/sample3.bz2  
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > > ln -s /usr/bin/bzip2  
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/bzip2
> > > > > cp: cannot stat 
> > > > > '<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/git/commons-compress':
> > > > >  No such file or directory
> > > > > WARNING: 
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368:189
> > > > >  exit 1 from 'cp -r <...>/build/tmp/work/core2-64-poky-linux/bzip2/
> > > > 1.0.8/git/commons-compress 
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/bzip2-tests/commons-compress'
> > > > > WARNING: Backtrace (BB generated script):
> > > > >   #1: do_install_ptest, 
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > > >  line 189
> > > > >   #2: do_install_ptest_base, 
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > > >  line 158
> > > > >   #3: main, 
> > > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > > >  line 226
> > > > ERROR: Task 
> > > > (<...>/poky/meta/recipes-extended/bzip2/bzip2_1.0.8.bb:do_install_ptest_base)
> > > >  failed with exit code '1'
> > > > NOTE: Tasks Summary: Attempted 776 tasks of which 765 didn't need to be 
> > > > rerun and 1 failed.
> > > > 
> > > > Summary: 1 task failed:
> > > >   
> > > > <...>/poky/meta/recipes-extended/bzip2/bzip2_1.0.8.bb:do_install_ptest_base
> > > > 
> > > > Signed-off-by: Julien Stephan 
> > > > ---
> > > >  scripts/lib/devtool/standard.py | 8 
> > > >  1 file changed, 8 insertions(+)
> > > > 
> > > > diff --git a/scripts/lib/devtool/standard.py 
> > > > b/scripts/lib/devtool/standard.py
> > > > index d53fb810071..6c8ad7a6c8c 100644
> > > > --- a/scripts/lib/devtool/standard.py
> > > > +++ b/scripts/lib/devtool/standard.py
> > > > @@ -684,6 +684,14 @@ def _extract_source(srctree, keep_temp, devbranch, 
> > > > sync, config, basepath, works
> > > >  shutil.move(srcsubdir, srctree)
> > > >  

Re: [OE-core] [PATCH] devtool: modify: correctly handle multiple sources in SRC_URI

2023-10-06 Thread Julien Stephan
Le sam. 30 sept. 2023 à 12:01, Julien Stephan via
lists.openembedded.org 
a écrit :
>
> Le mer. 27 sept. 2023 à 22:36, Richard Purdie
>  a écrit :
> >
> > On Wed, 2023-09-27 at 18:50 +0200, Julien Stephan wrote:
> > > [YOCTO #15162]
> > >
> > > When recipes such as bzip2 have several sources in SRC_URI, all the
> > > sources are correctly extracted into devtool temporary workdir, but
> > > currently, we only get back the primary source and all files declared
> > > using "files://", then the devtool temporary workdir is deleted
> > >
> > > Fix this by moving all remaining files from devtool temporary workdir
> > > into actual WORKDIR (except "source-date-epoch", "recipe-sysroot",
> > > and "recipe-sysroot-native") before deleting it
> > >
> > > Here is the stack trace from devtool modify/build bzip2:
> > >
> > > NOTE: bzip2: compiling from external source tree 
> > > <...>/build/workspace/sources/bzip2
> > > ERROR: bzip2-1.0.8-r0 do_install_ptest_base: 
> > > ExecutionError('<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368',
> > >  1, None, None)
> > > ERROR: Logfile of failure stored in: 
> > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/log.do_install_ptest_base.3368
> > > Log data follows:
> > > > DEBUG: Executing shell function do_install_ptest_base
> > > > NOTE: make -j 16 
> > > > DESTDIR=<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest
> > > >  install-ptest
> > > > sed  -n '/^runtest:/,/^install-ptest:/{/^install-ptest:/!p}' \
> > > >../../../../../../workspace/sources/bzip2/Makefile.am  > 
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/Makefile
> > > > cp ../../../../../../workspace/sources/bzip2/sample1.ref  
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > cp ../../../../../../workspace/sources/bzip2/sample2.ref  
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > cp ../../../../../../workspace/sources/bzip2/sample3.ref  
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > cp ../../../../../../workspace/sources/bzip2/sample1.bz2  
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > cp ../../../../../../workspace/sources/bzip2/sample2.bz2  
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > cp ../../../../../../workspace/sources/bzip2/sample3.bz2  
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/
> > > > ln -s /usr/bin/bzip2  
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/bzip2
> > > > cp: cannot stat 
> > > > '<...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/git/commons-compress':
> > > >  No such file or directory
> > > > WARNING: 
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368:189
> > > >  exit 1 from 'cp -r <...>/build/tmp/work/core2-64-poky-linux/bzip2/
> > > 1.0.8/git/commons-compress 
> > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/image/usr/lib/bzip2/ptest/bzip2-tests/commons-compress'
> > > > WARNING: Backtrace (BB generated script):
> > > >   #1: do_install_ptest, 
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > >  line 189
> > > >   #2: do_install_ptest_base, 
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > >  line 158
> > > >   #3: main, 
> > > > <...>/build/tmp/work/core2-64-poky-linux/bzip2/1.0.8/temp/run.do_install_ptest_base.3368,
> > > >  line 226
> > > ERROR: Task 
> > > (<...>/poky/meta/recipes-extended/bzip2/bzip2_1.0.8.bb:do_install_ptest_base)
> > >  failed with exit code '1'
> > > NOTE: Tasks Summary: Attempted 776 tasks of which 765 didn't need to be 
> > > rerun and 1 failed.
> > >
> > > Summary: 1 task failed:
> > >   
> > > <...>/poky/meta/recipes-extended/bzip2/bzip2_1.0.8.bb:do_install_ptest_base
> > >
> > > Signed-off-by: Julien Stephan 
> > > ---
> > >  scripts/lib/devtool/standard.py | 8 
> > >  1 file changed, 8 insertions(+)
> > >
> > > diff --git a/scripts/lib/devtool/standard.py 
> > > b/scripts/lib/devtool/standard.py
> > > index d53fb810071..6c8ad7a6c8c 100644
> > > --- a/scripts/lib/devtool/standard.py
> > > +++ b/scripts/lib/devtool/standard.py
> > > @@ -684,6 +684,14 @@ def _extract_source(srctree, keep_temp, devbranch, 
> > > sync, config, basepath, works
> > >  shutil.move(srcsubdir, srctree)
> > >  symlink_oelocal_files_srctree(d,srctree)
> > >
> > > +# In case SRC_URI contains multiple sources
> > > +# they are extracted into the devtool temporary workdir
> > > +# We need to move them into WORKDIR, otherwise they will be 
> > > lost

Re: [OE-core] [PATCH 5/5] qemurunner: Show both the login console log and all logging upon failure

2023-10-06 Thread Mikko Rapeli
Hi,

On Thu, Oct 05, 2023 at 04:47:16PM +0100, Richard Purdie wrote:
> On Thu, 2023-10-05 at 18:27 +0300, Mikko Rapeli wrote:
> > Hi,
> > 
> > On Wed, Oct 04, 2023 at 09:35:20PM +0100, Richard Purdie wrote:
> > > On Wed, 2023-10-04 at 22:34 +0300, Mikko Rapeli wrote:
> > > > Hi,
> > > > 
> > > > On Wed, Oct 04, 2023 at 02:29:40PM +0100, Richard Purdie wrote:
> > > > > On Wed, 2023-10-04 at 16:07 +0300, Mikko Rapeli wrote:
> > > > > > Hi,
> > > > > > 
> > > > > > Acked-by: Mikko Rapeli 
> > > > > > 
> > > > > > for the full series. I hope our discussion over #yocto irc channel
> > > > > > helped resolve these and the test failures seen with master-next.
> > > > > 
> > > > > Yes, thanks! It was useful to have someone to talk the issues through
> > > > > with and much appreciated.
> > > > > 
> > > > > Sadly I don't think we've found all the issues yet.
> > > > > 
> > > > > On the build N-1 we have a selftest failure on debian12-ty-1:
> > > > > 
> > > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/80/builds/5803/steps/14/logs/stdio
> > > > 
> > > > Can I get from this test run stdio to the more detailed task, 
> > > > qemurunner or qemu boot logs somehow?
> > > > Or do you have to dig into the system manually to capture them, like in 
> > > > the example below?
> > > > 
> > > > I would like to check into the detailed logs from both passing and 
> > > > failing test runs.
> > > > Of course kernel dmesg would need to be there so maybe after your patch 
> > > > to the runqemu
> > > > side at least.
> > > > 
> > > > If there is a regression in kernel to the serial device detection and 
> > > > numbering,
> > > > then we need some way to see e.g. 8250 uart/tty0/ttyAMA0 related dmesg 
> > > > messages.
> > > 
> > > I can't capture everything in time in each case but I did grab:
> > > 
> > > https://autobuilder.yocto.io/pub/failed-builds-data/6.5%20kernel/e/
> > > 
> > > which is from
> > > 
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/76/builds/7863/steps/21/logs/stdio
> > > 
> > > which has this threads patches applied.
> > > 
> > > I've put two sets of logs there from the same builds, one success, one
> > > is missing the getty on one log.
> > > 
> > > 
> > > I also have the logs from:
> > > 
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/86/builds/5848/steps/14/logs/stdio
> > > 
> > > which I've saved here:
> > > 
> > > https://autobuilder.yocto.io/pub/failed-builds-data/6.5%20kernel/f/
> > > 
> > > noting that: 
> > > 
> > > https://autobuilder.yocto.io/pub/failed-builds-data/6.5%20kernel/f/qemu_boot_log.20231004134427
> > > 
> > > is 3 bytes in size.
> > 
> > All this is very confusing indeed. I sent another version of the detect 
> > login prompt without
> > utf-8 conversion, with some luck that could help. At least on some of the 
> > logs the login prompt
> > is there but with lots of garbage around it. Serial port core got 
> > refactored in 6.5 and diff to
> > 6.4 is quite large so there could be bugs there, and those could be 
> > triggering new
> > errors visible to userspace too.
> 
> With the changes I've made to the logging and the raw log files we have
> without any utf-8 conversion, I'm now 99% certain this isn't any utf
> conversion issue since the login prompts are not in the raw binary
> files and those are written with no conversion. It is much more likely
> the getty isn't running or isn't communicating somehow.
> 
> I'm trying to work out how to add some further debug/testing when this
> fails in a way it will help us debug.

Alright, I was confused by logs from working login detection. I presume
qemu is configured with two serial ports to avoid mixing kernel messages with
the login prompt. It would be easy to grep for the login prompt also from the 
first
serial output though it would not explain the root cause for these issues
but could allow testing to move forward to the ssh connection. In all of the 
logs
I saw, the first serial port had a login prompt.

The serial ports are not used in testing, AFAIK. Oh now there are the debug
commands in master-next. Nice, I was hoping for this kind of data in logs when
things go horribly wrong.

Cheers,

-Mikko

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188749): 
https://lists.openembedded.org/g/openembedded-core/message/188749
Mute This Topic: https://lists.openembedded.org/mt/101753837/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2 0/4] insane: Add unimplemented-ptest checks

2023-10-06 Thread Yoann Congal
Le ven. 6 oct. 2023 à 09:20, Richard Purdie <
richard.pur...@linuxfoundation.org> a écrit :

> On Wed, 2023-10-04 at 14:57 +0200, Yoann Congal wrote:
> > To increase ptest coverage we can check if the sources of a recipe looks
> like
> > it contains unittest and warn the user that a test may be implemented
> there.
> >
> > This series provide the check infrastructure as a package QA check and
> some checks for :
> > python pytest, perl Test::, meson, cmake, autotools-based tests... as
> well as
> > the naive check of "Is there a test/ directory in the sources?" which
> work
> > surprisingly well.
> >
> > Currently, this check find:
> > * 309 unimplemented ptest in oe-core/meta-poky/meta-yocto-bsp
> > * 827 unimplemented ptest in meta-openembedded
> >
> > Full list :
> https://gist.githubusercontent.com/ycongal-smile/0969e92aec1ca37e61b6cdbcaf1f6885/raw/b1be8a5b576e144b6563f5481d31b32de9b22368/gistfile1.txt
> >
> > v1->v2:
> > * Corrected: shortlog tag
> > * Fixed infinite loop on source package containing symlink loops (e.g.
> md5deep)
> > * Added test result
> >
> > Jérémy Rosen (4):
> >   insane: Add unimplemented-ptest infrastructure
> >   insane: Detect python and perl based tests
> >   insane: Detect build-system test harnesses
> >   insane: Add a naive heuristic to detect test subdirectories
> >
> >  meta/classes-global/insane.bbclass | 50 ++
> >  1 file changed, 50 insertions(+)
>
> Somehow this series causes:
>
> oe-selftest -r
> sstatetests.SStateHashSameSigs2.test_sstate_allarch_samesigs
>
> to fail and the other test failures in:
>
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/80/builds/5816/steps/14/logs/stdio
>
> I've not looked into why, just narrowed it to this series.
>

Ok, thanks! We'll look into this.
-- 
Yoann Congal
Smile ECS - Tech expert

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188748): 
https://lists.openembedded.org/g/openembedded-core/message/188748
Mute This Topic: https://lists.openembedded.org/mt/101754134/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2 0/4] insane: Add unimplemented-ptest checks

2023-10-06 Thread Richard Purdie
On Wed, 2023-10-04 at 14:57 +0200, Yoann Congal wrote:
> To increase ptest coverage we can check if the sources of a recipe looks like
> it contains unittest and warn the user that a test may be implemented there.
> 
> This series provide the check infrastructure as a package QA check and some 
> checks for :
> python pytest, perl Test::, meson, cmake, autotools-based tests... as well as
> the naive check of "Is there a test/ directory in the sources?" which work
> surprisingly well.
> 
> Currently, this check find:
> * 309 unimplemented ptest in oe-core/meta-poky/meta-yocto-bsp
> * 827 unimplemented ptest in meta-openembedded
> 
> Full list : 
> https://gist.githubusercontent.com/ycongal-smile/0969e92aec1ca37e61b6cdbcaf1f6885/raw/b1be8a5b576e144b6563f5481d31b32de9b22368/gistfile1.txt
> 
> v1->v2:
> * Corrected: shortlog tag
> * Fixed infinite loop on source package containing symlink loops (e.g. 
> md5deep)
> * Added test result
> 
> Jérémy Rosen (4):
>   insane: Add unimplemented-ptest infrastructure
>   insane: Detect python and perl based tests
>   insane: Detect build-system test harnesses
>   insane: Add a naive heuristic to detect test subdirectories
> 
>  meta/classes-global/insane.bbclass | 50 ++
>  1 file changed, 50 insertions(+)

Somehow this series causes:

oe-selftest -r
sstatetests.SStateHashSameSigs2.test_sstate_allarch_samesigs

to fail and the other test failures in:

https://autobuilder.yoctoproject.org/typhoon/#/builders/80/builds/5816/steps/14/logs/stdio

I've not looked into why, just narrowed it to this series.

Cheers,

Richard



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188747): 
https://lists.openembedded.org/g/openembedded-core/message/188747
Mute This Topic: https://lists.openembedded.org/mt/101754134/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH 1/3] dummy-sdk-package.inc: Filter packages which are marked for installation

2023-10-06 Thread Pavel Zhukov

Hi Alex,

Thank you for heads up. Interesting breakage... Reproduced locally.

-- 
Pavel

Alexandre Belloni  writes:

> Hello,
>
> This series causes the following failure:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/20/builds/8312/steps/12/logs/stdio
>
> | Can't locate Text/Tabs.pm in @INC (you may need to install the
> | Text::Tabs module) (@INC entries checked:
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/bitbake-build-2946y_hb/tmp/work/x86_64-linux/automake-native/1.16.5/build/lib/
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/bitbake-build-2946y_hb/tmp/work/x86_64-linux/automake-native/1.16.5/build/../automake-1.16.5/lib/
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/site_perl/5.38.0/x86_64-linux
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/site_perl/5.38.0
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/vendor_perl/5.38.0
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/5.38.0/x86_64-linux
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/5.38.0
> | 
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/perl5/site_perl/5.38.0/x86_64-linux
> | 
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/perl5/site_perl/5.38.0
> | 
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/perl5/vendor_perl/5.38.0/x86_64-linux
> | 
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/perl5/vendor_perl/5.38.0
> | 
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/perl5/5.38.0/x86_64-linux
> | 
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/perl5/5.38.0)
> | at ../automake-1.16.5/doc/help2man line 28.
> | BEGIN failed--compilation aborted at ../automake-1.16.5/doc/help2man line 
> 28.
> | make: *** [Makefile:3741: doc/aclocal-1.16.1] Error 2
> | make: *** Waiting for unfinished jobs
> | if LC_ALL=C grep '@[a-zA-Z0-9_][a-zA-Z0-9_]*@' runtest-t; then echo
> | "runtest contains unexpanded substitution (see lines above)"; exit
> | 1; fi; chmod a-w runtest-t && mv -f runtest-t runtest
> | Can't locate Text/Tabs.pm in @INC (you may need to install the
> | Text::Tabs module) (@INC entries checked:
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/bitbake-build-2946y_hb/tmp/work/x86_64-linux/automake-native/1.16.5/build/lib/
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/bitbake-build-2946y_hb/tmp/work/x86_64-linux/automake-native/1.16.5/build/../automake-1.16.5/lib/
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/site_perl/5.38.0/x86_64-linux
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/site_perl/5.38.0
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/vendor_perl/5.38.0
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/5.38.0/x86_64-linux
> | 
> /home/pokybuild/yocto-worker/buildtools/build/build/tmp/work/x86_64-nativesdk-pokysdk-linux/buildtools-extended-tarball/1.0/testimage-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/../..//usr/lib/perl5/5.38.0
> | 
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/perl5/site_perl/5.38.0/x86_64-linux
> | 
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/usr/lib/perl5/site_perl/5.38.0
> | 
> 

[OE-core] [PATCH] devtool/upgrade: check all git config locations

2023-10-06 Thread Marcus Flyckt
Fixes [YOCTO #15222]

In some situations its more practical to keep git configuration
at `/etc/gitconfig` instead of `$HOME/.gitconfig` (e.g., when mounting
git configuration into a docker container).

This change makes `devtool upgrade` consider any available
git configuration instead of only checking `--global`.

Signed-off-by: Marcus Flyckt 
---
 scripts/lib/devtool/upgrade.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/lib/devtool/upgrade.py b/scripts/lib/devtool/upgrade.py
index 487308912f..9cd50be3a2 100644
--- a/scripts/lib/devtool/upgrade.py
+++ b/scripts/lib/devtool/upgrade.py
@@ -442,7 +442,7 @@ def _create_new_recipe(newpv, md5, sha256, srcrev, 
srcbranch, srcsubdir_old, src
 def _check_git_config():
 def getconfig(name):
 try:
-value = bb.process.run('git config --global %s' % name)[0].strip()
+value = bb.process.run('git config %s' % name)[0].strip()
 except bb.process.ExecutionError as e:
 if e.exitcode == 1:
 value = None
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#188745): 
https://lists.openembedded.org/g/openembedded-core/message/188745
Mute This Topic: https://lists.openembedded.org/mt/101792562/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-