[oe-core][PATCH 2/2] packagegroup-core-boot: recommend ifupdown

2024-04-10 Thread Joe Slater via lists.openembedded.org
From: Joe Slater 

If the distro features sysvinit and pni-names are
enabled, RRECOMMENDS ifupdown because busybox ifupdown
will not initialize the renamed interfaces.

Signed-off-by: Joe Slater 
---
 .../init-ifupdown/init-ifupdown-1.0/qemuall/interfaces   | 5 -
 meta/recipes-core/packagegroups/packagegroup-core-boot.bb| 4 +++-
 2 files changed, 3 insertions(+), 6 deletions(-)
 delete mode 100644 
meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces

diff --git 
a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces 
b/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces
deleted file mode 100644
index 16967763e5..00
--- a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces
+++ /dev/null
@@ -1,5 +0,0 @@
-# /etc/network/interfaces -- configuration file for ifup(8), ifdown(8)
- 
-# The loopback interface
-auto lo
-iface lo inet loopback
diff --git a/meta/recipes-core/packagegroups/packagegroup-core-boot.bb 
b/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
index d96d2f5fff..fecc3334ea 100644
--- a/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
@@ -37,4 +37,6 @@ RDEPENDS:${PN} = "\
 RRECOMMENDS:${PN} = "\
 ${VIRTUAL-RUNTIME_base-utils-syslog} \
 ${MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS} \
-${@bb.utils.contains("DISTRO_FEATURES", "sysvinit", "init-ifupdown", "", 
d)}"
+${@bb.utils.contains("DISTRO_FEATURES", "sysvinit", "init-ifupdown", "", 
d)} \
+${@bb.utils.contains("DISTRO_FEATURES", "sysvinit pni-names", "ifupdown", 
"", d)} \
+"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198114): 
https://lists.openembedded.org/g/openembedded-core/message/198114
Mute This Topic: https://lists.openembedded.org/mt/105454861/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] init-ifupdown: modify interfaces for busybox

2024-04-10 Thread Joe Slater via lists.openembedded.org
From: Joe Slater 

Busybox ifupdown does not recognize /xxx names, so we
use eth0 instead of /eth0.  If we want to find "predictable name"
interfaces starting with en..., we will have to use the
real ifupdown.

Signed-off-by: Joe Slater 
---
 meta/recipes-core/init-ifupdown/init-ifupdown-1.0/interfaces | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/interfaces 
b/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/interfaces
index e617fcf69b..3d0f0c6768 100644
--- a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/interfaces
+++ b/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/interfaces
@@ -14,10 +14,11 @@ iface wlan0 inet dhcp
 iface atml0 inet dhcp
 
 # Wired or wireless interfaces including predictable names
-auto /eth0
+auto eth0
 iface eth0 inet dhcp
 iface eth1 inet dhcp
 
+# Busybox ifupdown won't process /en* correctly
 auto /en*=eth
 iface eth inet dhcp
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198113): 
https://lists.openembedded.org/g/openembedded-core/message/198113
Mute This Topic: https://lists.openembedded.org/mt/105454860/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] bash: Fix file-substitution error-handling bug

2024-04-10 Thread Zev Weiss
This is part of a patch that's been upstream for a while but hasn't yet
been released.  The bug is causing some downstream difficulties, so a
local patch to tide us over until the next release makes things a bit
easier.

Signed-off-by: Zev Weiss 
---
Changes in v2:
 - Added s-o-b line to patch file [patchtest]

 .../bash/bash/fix-filesubst-errexit.patch | 34 +++
 meta/recipes-extended/bash/bash_5.2.21.bb |  1 +
 2 files changed, 35 insertions(+)
 create mode 100644 meta/recipes-extended/bash/bash/fix-filesubst-errexit.patch

diff --git a/meta/recipes-extended/bash/bash/fix-filesubst-errexit.patch 
b/meta/recipes-extended/bash/bash/fix-filesubst-errexit.patch
new file mode 100644
index ..60f1852316e8
--- /dev/null
+++ b/meta/recipes-extended/bash/bash/fix-filesubst-errexit.patch
@@ -0,0 +1,34 @@
+From 59ddfda14e3c9aa6286bb4c4c0748f7c1324a65a Mon Sep 17 00:00:00 2001
+From: Chet Ramey 
+Date: Fri, 7 Apr 2023 00:28:46 -0700
+Subject: [PATCH] $(https://git.savannah.gnu.org/cgit/bash.git/commit/?h=devel=ec9447ce9392a0f93d96789c3741285fede8a150
+
+Upstream-Status: Backport
+
+Signed-off-by: Zev Weiss 
+---
+ builtins/evalstring.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/builtins/evalstring.c b/builtins/evalstring.c
+index df3dd68e2a7e..6612081cd646 100644
+--- a/builtins/evalstring.c
 b/builtins/evalstring.c
+@@ -753,7 +753,7 @@ open_redir_file (r, fnp)
+   fd = open(fn, O_RDONLY);
+   if (fd < 0)
+ {
+-  file_error (fn);
++  internal_error ("%s: %s", fn, strerror (errno));
+   free (fn);
+   if (fnp)
+   *fnp = 0;
+-- 
+2.40.0
+
diff --git a/meta/recipes-extended/bash/bash_5.2.21.bb 
b/meta/recipes-extended/bash/bash_5.2.21.bb
index 46d921bbe6c6..532adf4c1a0d 100644
--- a/meta/recipes-extended/bash/bash_5.2.21.bb
+++ b/meta/recipes-extended/bash/bash_5.2.21.bb
@@ -13,6 +13,7 @@ SRC_URI = "${GNU_MIRROR}/bash/${BP}.tar.gz;name=tarball \
file://fix-run-builtins.patch \
file://use_aclocal.patch \

file://0001-changes-to-SIGINT-handler-while-waiting-for-a-child-.patch \
+   file://fix-filesubst-errexit.patch \
"
 
 SRC_URI[tarball.sha256sum] = 
"c8e31bdc59b69aaffc5b36509905ba3e5cbb12747091d27b4b977f078560d5b8"
-- 
2.34.1


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



Patchtest results for [OE-core][PATCH] bash: Fix file-substitution error-handling bug

2024-04-10 Thread Patchtest
Thank you for your submission. Patchtest identified one
or more issues with the patch. Please see the log below for
more information:

---
Testing patch 
/home/patchtest/share/mboxes/bash-Fix-file-substitution-error-handling-bug.patch

FAIL: test Signed-off-by presence: A patch file has been added without a 
Signed-off-by tag: 'fix-filesubst-errexit.patch' 
(test_patch.TestPatch.test_signed_off_by_presence)

PASS: pretest src uri left files 
(test_metadata.TestMetadata.pretest_src_uri_left_files)
PASS: test CVE check ignore (test_metadata.TestMetadata.test_cve_check_ignore)
PASS: test CVE tag format (test_patch.TestPatch.test_cve_tag_format)
PASS: test Signed-off-by presence 
(test_mbox.TestMbox.test_signed_off_by_presence)
PASS: test Upstream-Status presence 
(test_patch.TestPatch.test_upstream_status_presence_format)
PASS: test author valid (test_mbox.TestMbox.test_author_valid)
PASS: test commit message presence 
(test_mbox.TestMbox.test_commit_message_presence)
PASS: test lic files chksum modified not mentioned 
(test_metadata.TestMetadata.test_lic_files_chksum_modified_not_mentioned)
PASS: test max line length (test_metadata.TestMetadata.test_max_line_length)
PASS: test mbox format (test_mbox.TestMbox.test_mbox_format)
PASS: test non-AUH upgrade (test_mbox.TestMbox.test_non_auh_upgrade)
PASS: test shortlog format (test_mbox.TestMbox.test_shortlog_format)
PASS: test shortlog length (test_mbox.TestMbox.test_shortlog_length)
PASS: test src uri left files 
(test_metadata.TestMetadata.test_src_uri_left_files)

SKIP: pretest pylint: No python related patches, skipping test 
(test_python_pylint.PyLint.pretest_pylint)
SKIP: test bugzilla entry format: No bug ID found 
(test_mbox.TestMbox.test_bugzilla_entry_format)
SKIP: test lic files chksum presence: No added recipes, skipping test 
(test_metadata.TestMetadata.test_lic_files_chksum_presence)
SKIP: test license presence: No added recipes, skipping test 
(test_metadata.TestMetadata.test_license_presence)
SKIP: test pylint: No python related patches, skipping test 
(test_python_pylint.PyLint.test_pylint)
SKIP: test series merge on head: Merge test is disabled for now 
(test_mbox.TestMbox.test_series_merge_on_head)
SKIP: test summary presence: No added recipes, skipping test 
(test_metadata.TestMetadata.test_summary_presence)
SKIP: test target mailing list: Series merged, no reason to check other mailing 
lists (test_mbox.TestMbox.test_target_mailing_list)

---

Please address the issues identified and
submit a new revision of the patch, or alternatively, reply to this
email with an explanation of why the patch should be accepted. If you
believe these results are due to an error in patchtest, please submit a
bug at https://bugzilla.yoctoproject.org/ (use the 'Patchtest' category
under 'Yocto Project Subprojects'). For more information on specific
failures, see: https://wiki.yoctoproject.org/wiki/Patchtest. Thank
you!

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198111): 
https://lists.openembedded.org/g/openembedded-core/message/198111
Mute This Topic: https://lists.openembedded.org/mt/105454274/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] bash: Fix file-substitution error-handling bug

2024-04-10 Thread Zev Weiss
This is part of a patch that's been upstream for a while but hasn't yet
been released.  The bug is causing some downstream difficulties, so a
local patch to tide us over until the next release makes things a bit
easier.

Signed-off-by: Zev Weiss 
---
 .../bash/bash/fix-filesubst-errexit.patch | 33 +++
 meta/recipes-extended/bash/bash_5.2.21.bb |  1 +
 2 files changed, 34 insertions(+)
 create mode 100644 meta/recipes-extended/bash/bash/fix-filesubst-errexit.patch

diff --git a/meta/recipes-extended/bash/bash/fix-filesubst-errexit.patch 
b/meta/recipes-extended/bash/bash/fix-filesubst-errexit.patch
new file mode 100644
index ..9ef1394e74c4
--- /dev/null
+++ b/meta/recipes-extended/bash/bash/fix-filesubst-errexit.patch
@@ -0,0 +1,33 @@
+From 59ddfda14e3c9aa6286bb4c4c0748f7c1324a65a Mon Sep 17 00:00:00 2001
+From: Chet Ramey 
+Date: Fri, 7 Apr 2023 00:28:46 -0700
+Subject: [PATCH] $(https://git.savannah.gnu.org/cgit/bash.git/commit/?h=devel=ec9447ce9392a0f93d96789c3741285fede8a150
+
+Upstream-Status: Backport
+
+---
+ builtins/evalstring.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/builtins/evalstring.c b/builtins/evalstring.c
+index df3dd68e2a7e..6612081cd646 100644
+--- a/builtins/evalstring.c
 b/builtins/evalstring.c
+@@ -753,7 +753,7 @@ open_redir_file (r, fnp)
+   fd = open(fn, O_RDONLY);
+   if (fd < 0)
+ {
+-  file_error (fn);
++  internal_error ("%s: %s", fn, strerror (errno));
+   free (fn);
+   if (fnp)
+   *fnp = 0;
+-- 
+2.40.0
+
diff --git a/meta/recipes-extended/bash/bash_5.2.21.bb 
b/meta/recipes-extended/bash/bash_5.2.21.bb
index 46d921bbe6c6..532adf4c1a0d 100644
--- a/meta/recipes-extended/bash/bash_5.2.21.bb
+++ b/meta/recipes-extended/bash/bash_5.2.21.bb
@@ -13,6 +13,7 @@ SRC_URI = "${GNU_MIRROR}/bash/${BP}.tar.gz;name=tarball \
file://fix-run-builtins.patch \
file://use_aclocal.patch \

file://0001-changes-to-SIGINT-handler-while-waiting-for-a-child-.patch \
+   file://fix-filesubst-errexit.patch \
"
 
 SRC_URI[tarball.sha256sum] = 
"c8e31bdc59b69aaffc5b36509905ba3e5cbb12747091d27b4b977f078560d5b8"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198110): 
https://lists.openembedded.org/g/openembedded-core/message/198110
Mute This Topic: https://lists.openembedded.org/mt/105454172/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] linux-yocto: Enable nft modules for ptest images

2024-04-10 Thread Khem Raj
nftables ptests need it

Signed-off-by: Khem Raj 
---
 meta/recipes-kernel/linux/linux-yocto_6.6.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto_6.6.bb 
b/meta/recipes-kernel/linux/linux-yocto_6.6.bb
index 1c03e22c3cb..e8e84f8f788 100644
--- a/meta/recipes-kernel/linux/linux-yocto_6.6.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_6.6.bb
@@ -64,7 +64,7 @@ KERNEL_FEATURES:append:qemuall=" cfg/virtio.scc 
features/drm-bochs/drm-bochs.scc
 KERNEL_FEATURES:append:qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
 KERNEL_FEATURES:append:qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
 KERNEL_FEATURES:append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " 
cfg/x32.scc", "", d)}"
-KERNEL_FEATURES:append = " ${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
features/scsi/scsi-debug.scc", "", d)}"
+KERNEL_FEATURES:append = " ${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
features/scsi/scsi-debug.scc features/nf_tables/nft_test.scc", "", d)}"
 KERNEL_FEATURES:append = " ${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
features/gpio/mockup.scc features/gpio/sim.scc", "", d)}"
 KERNEL_FEATURES:append:powerpc =" arch/powerpc/powerpc-debug.scc"
 KERNEL_FEATURES:append:powerpc64 =" arch/powerpc/powerpc-debug.scc"
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198109): 
https://lists.openembedded.org/g/openembedded-core/message/198109
Mute This Topic: https://lists.openembedded.org/mt/105453044/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/1] init-ifupdown: add predictable interface names

2024-04-10 Thread Joe Slater via lists.openembedded.org


> -Original Message-
> From: openembedded-core@lists.openembedded.org  c...@lists.openembedded.org> On Behalf Of Joe Slater via
> lists.openembedded.org
> Sent: Wednesday, April 10, 2024 12:30 PM
> To: Richard Purdie ; Ross Burton
> 
> Cc: openembedded-core@lists.openembedded.org; MacLeod, Randy
> 
> Subject: Re: [oe-core][PATCH 1/1] init-ifupdown: add predictable interface 
> names
> 
> The easiest thing to do is to change "auto /eth0" back to "auto eth0".  This 
> lets
> bitbake's ifupdown initialize eth0 and also allows the real ifupdown to 
> initialize
> en* interfaces if they exist.

Oops, that's busybox, not bitbake.  Since we would need ifupdown to process 
renamed interfaces, I'm preparing a patch for packagegroup-core-boot  to 
RRECOMMENDS it into the rootfs if sysvinit and pni-names are distro features.

Joe

> 
> Joe
> 
> > -Original Message-
> > From: Richard Purdie 
> > Sent: Wednesday, April 10, 2024 10:29 AM
> > To: Ross Burton ; Slater, Joseph
> > 
> > Cc: openembedded-core@lists.openembedded.org; MacLeod, Randy
> > 
> > Subject: Re: [oe-core][PATCH 1/1] init-ifupdown: add predictable
> > interface names
> >
> > On Wed, 2024-04-10 at 13:41 +, Ross Burton wrote:
> > > On 10 Jan 2024, at 21:10, Joe Slater via lists.openembedded.org
> > >  wrote:
> > > > -# Wired or wireless interfaces
> > > > -auto eth0
> > > > +# Wired or wireless interfaces including predictable names auto
> > > > +/eth0
> > >
> > > Obviously we’re days away from release but I think we need to revert
> > > or rewrite this.
> > >
> > > The /name syntax is specific to “proper” ifupdown, so any system
> > > which uses busybox’s ifupdown will fail to have functional
> > > networking.  We don’t see this on the AB because there is a separate
> > > interfaces file for the qemuall override, this file is only used in
> > > non-qemu situations.
> >
> > I'm worried about the number of other changes over the last few months
> > that are related to this so a simple revert might not get us to a good 
> > place.
> >
> > I'd like to see a patch proposal on what we should do. Time is short
> > as testing changes takes time and we're now a bit behind already :/.
> >
> > So specific proposals welcome.
> >
> > Cheers,
> >
> > Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198108): 
https://lists.openembedded.org/g/openembedded-core/message/198108
Mute This Topic: https://lists.openembedded.org/mt/103649977/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/1] init-ifupdown: add predictable interface names

2024-04-10 Thread Joe Slater via lists.openembedded.org
The easiest thing to do is to change "auto /eth0" back to "auto eth0".  This 
lets bitbake's ifupdown initialize eth0 and also allows the real ifupdown to 
initialize en* interfaces if they exist.

Joe

> -Original Message-
> From: Richard Purdie 
> Sent: Wednesday, April 10, 2024 10:29 AM
> To: Ross Burton ; Slater, Joseph
> 
> Cc: openembedded-core@lists.openembedded.org; MacLeod, Randy
> 
> Subject: Re: [oe-core][PATCH 1/1] init-ifupdown: add predictable interface 
> names
> 
> On Wed, 2024-04-10 at 13:41 +, Ross Burton wrote:
> > On 10 Jan 2024, at 21:10, Joe Slater via lists.openembedded.org
> >  wrote:
> > > -# Wired or wireless interfaces
> > > -auto eth0
> > > +# Wired or wireless interfaces including predictable names auto
> > > +/eth0
> >
> > Obviously we’re days away from release but I think we need to revert
> > or rewrite this.
> >
> > The /name syntax is specific to “proper” ifupdown, so any system which
> > uses busybox’s ifupdown will fail to have functional networking.  We
> > don’t see this on the AB because there is a separate interfaces file
> > for the qemuall override, this file is only used in non-qemu
> > situations.
> 
> I'm worried about the number of other changes over the last few months that 
> are
> related to this so a simple revert might not get us to a good place.
> 
> I'd like to see a patch proposal on what we should do. Time is short as 
> testing
> changes takes time and we're now a bit behind already :/.
> 
> So specific proposals welcome.
> 
> Cheers,
> 
> Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198107): 
https://lists.openembedded.org/g/openembedded-core/message/198107
Mute This Topic: https://lists.openembedded.org/mt/103649977/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/1] init-ifupdown: add predictable interface names

2024-04-10 Thread Alexander Kanavin
On Wed, 10 Apr 2024 at 19:29, Richard Purdie
 wrote:
> I'd like to see a patch proposal on what we should do. Time is short as
> testing changes takes time and we're now a bit behind already :/.
>
> So specific proposals welcome.

If no one comes out with a better proposal, I'd suggest reverting the
top five commts, all of them:
https://git.yoctoproject.org/poky/log/?qt=author=slater

Alex

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198106): 
https://lists.openembedded.org/g/openembedded-core/message/198106
Mute This Topic: https://lists.openembedded.org/mt/103649977/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] systemd: make predictable name mac policy opt-out

2024-04-10 Thread Peter Marko via lists.openembedded.org

-Original Message-
From: Ross Burton  
Sent: Wednesday, April 10, 2024 18:18
To: Marko, Peter (ADV D EU SK BFS1) 
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core][PATCH] systemd: make predictable name mac policy opt-out

> On 8 Apr 2024, at 08:04, Peter Marko via lists.openembedded.org 
>  wrote:
> > 
> > From: Peter Marko 
> > 
> > Even the patch says it's inappropriate for upstream, and it's also 
> > inappropriate for some downstream projects, too.
> > So make it possible to opt-out on it.
>
> I’m looking at these patches because of the fallout from the use of matches 
> in the interfaces file.   Presumably you want to make this opt-out for 
> concrete reasons, can you explain what broke?
>
> Ross

Basically, we have networkmanager and firewalld configuration matching 
interface names.
In addition, also our applications are hardcoding the interface names to be 
able to configure interfaces on demand.
Switching to dynamic names is not realistic.

After upgrading from 5.0_M3 to 5.0_M4 our wlan0 interface gets renamed by udev 
and thus networking breaks.
Unlike our ethernet ports with names defined in device tree, wifi chip uses 
external vendor kernel module so I'm not sure if I'm able to configure a stable 
kernel name for it.

Peter

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198105): 
https://lists.openembedded.org/g/openembedded-core/message/198105
Mute This Topic: https://lists.openembedded.org/mt/105396950/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/1] init-ifupdown: add predictable interface names

2024-04-10 Thread Richard Purdie
On Wed, 2024-04-10 at 13:41 +, Ross Burton wrote:
> On 10 Jan 2024, at 21:10, Joe Slater via lists.openembedded.org
>  wrote:
> > -# Wired or wireless interfaces
> > -auto eth0
> > +# Wired or wireless interfaces including predictable names
> > +auto /eth0
> 
> Obviously we’re days away from release but I think we need to revert
> or rewrite this.
> 
> The /name syntax is specific to “proper” ifupdown, so any system
> which uses busybox’s ifupdown will fail to have functional
> networking.  We don’t see this on the AB because there is a separate
> interfaces file for the qemuall override, this file is only used in
> non-qemu situations.

I'm worried about the number of other changes over the last few months
that are related to this so a simple revert might not get us to a good
place.

I'd like to see a patch proposal on what we should do. Time is short as
testing changes takes time and we're now a bit behind already :/.

So specific proposals welcome.

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198104): 
https://lists.openembedded.org/g/openembedded-core/message/198104
Mute This Topic: https://lists.openembedded.org/mt/103649977/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][dunfell] linux-yocto/5.4: configuration warning fixes

2024-04-10 Thread Bruce Ashfield
From: Bruce Ashfield 

Integrating the following commit(s) to linux-yocto/.:

1/2 [
Author: Bruce Ashfield
Email: bruce.ashfi...@gmail.com
Subject: net/cfg: remove CONFIG_NET_SCH_CBQ
Date: Thu, 6 Jul 2023 12:39:20 -0400

commit 051d442098421c28c7 [net/sched: Retire CBQ qdisc] removes
CONFIG_NET_SCH_CBQ from the tree, so we drop it from our fragments.

Signed-off-by: Bruce Ashfield 
]

2/2 [
Author: Bruce Ashfield
Email: bruce.ashfi...@gmail.com
Subject: cfg/net: remove CONFIG_NET_SCH_DSMARK
Date: Thu, 6 Jul 2023 12:42:07 -0400

commit bbe77c14ee6185a61b [net/sched: Retire dsmark qdisc] upstream
has removed CONFIG_NET_SCH_DSMARK so we drop it from our fragments
as well.

Signed-off-by: Bruce Ashfield 
]

Signed-off-by: Bruce Ashfield 
---
 meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb   | 2 +-
 meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb | 2 +-
 meta/recipes-kernel/linux/linux-yocto_5.4.bb  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
index 89b8d2b8258..f912304858a 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
@@ -12,7 +12,7 @@ python () {
 }
 
 SRCREV_machine ?= "c93e75bc334ba00df2d66411a0d79c4378cf4af8"
-SRCREV_meta ?= "c841eec84cf56e6b837f12a359c35c5dfb26da5f"
+SRCREV_meta ?= "ecd382f3477fae022ad1881e4c39e810cdc3c760"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \

git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-5.4;destsuffix=${KMETA}"
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
index 6e049b03f97..2f947824716 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
@@ -17,7 +17,7 @@ KCONF_BSP_AUDIT_LEVEL = "2"
 
 SRCREV_machine_qemuarm ?= "d29f3f3a932319053ad24d84b087b0a57908c1bc"
 SRCREV_machine ?= "b6480d09d84d09e7560daa5c1d73917292ae30c0"
-SRCREV_meta ?= "c841eec84cf56e6b837f12a359c35c5dfb26da5f"
+SRCREV_meta ?= "ecd382f3477fae022ad1881e4c39e810cdc3c760"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
 
diff --git a/meta/recipes-kernel/linux/linux-yocto_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
index 2a659e32fd6..108043bd981 100644
--- a/meta/recipes-kernel/linux/linux-yocto_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
@@ -22,7 +22,7 @@ SRCREV_machine_qemux86 ?= 
"fe901e2f4b156e9cf7ddb03f479f7339d28e398b"
 SRCREV_machine_qemux86-64 ?= "fe901e2f4b156e9cf7ddb03f479f7339d28e398b"
 SRCREV_machine_qemumips64 ?= "f59947f338319b1741db5dfac34f08399561ab25"
 SRCREV_machine ?= "fe901e2f4b156e9cf7ddb03f479f7339d28e398b"
-SRCREV_meta ?= "c841eec84cf56e6b837f12a359c35c5dfb26da5f"
+SRCREV_meta ?= "ecd382f3477fae022ad1881e4c39e810cdc3c760"
 
 # remap qemuarm to qemuarma15 for the 5.4 kernel
 # KMACHINE_qemuarm ?= "qemuarma15"
-- 
2.39.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198103): 
https://lists.openembedded.org/g/openembedded-core/message/198103
Mute This Topic: https://lists.openembedded.org/mt/105445325/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] systemd: make predictable name mac policy opt-out

2024-04-10 Thread Ross Burton
On 8 Apr 2024, at 08:04, Peter Marko via lists.openembedded.org 
 wrote:
> 
> From: Peter Marko 
> 
> Even the patch says it's inappropriate for upstream,
> and it's also inappropriate for some downstream projects, too.
> So make it possible to opt-out on it.

I’m looking at these patches because of the fallout from the use of matches in 
the interfaces file.   Presumably you want to make this opt-out for concrete 
reasons, can you explain what broke?

Ross
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198102): 
https://lists.openembedded.org/g/openembedded-core/message/198102
Mute This Topic: https://lists.openembedded.org/mt/105396950/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] systemd: make predictable name mac policy opt-out

2024-04-10 Thread Ross Burton
On 8 Apr 2024, at 08:04, Peter Marko via lists.openembedded.org 
 wrote:
> +   ${@bb.utils.contains('PACKAGECONFIG', 'predictable-if-mac', 
> 'file://0001-NamePolicy.patch', '', d)} \

There’s a few other places which use the pni-names DISTRO_FEATURE (which 
probably needs to be documented), this should respect that I guess.

Ross


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198101): 
https://lists.openembedded.org/g/openembedded-core/message/198101
Mute This Topic: https://lists.openembedded.org/mt/105396950/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] [yocto-patches] [PATCH 1/2] README: Switch to yocto-patches mailing list

2024-04-10 Thread Richard Purdie
On Wed, 2024-04-10 at 14:30 +0200, Quentin Schulz via lists.yoctoproject.org 
wrote:
> Hi Richard,
> 
> On 4/10/24 11:29, Richard Purdie via lists.yoctoproject.org wrote:
> > We're trying to align the various components so tweak pseudo to match our
> > other projects a little more and use the yocto-patches mailing list for
> > submissions.
> > 
> > Signed-off-by: Richard Purdie 
> > ---
> >   README | 17 -
> >   1 file changed, 4 insertions(+), 13 deletions(-)
> > 
> > diff --git a/README b/README
> > index 468cf66..425ee6e 100644
> > --- a/README
> > +++ b/README
> > @@ -1,17 +1,9 @@
> >   pseudo -- an analogue to sudo
> >   
> > -IMPORTANT NOTE:
> > +The official home for pseudo's git repository is:
> >   
> > -As of this writing, the official home for pseudo's git repository has
> > -changed to:
> >     git://git.yoctoproject.org/pseudo
> >   
> 
> Maybe we could use https here?
> 
> https://git.yoctoproject.org/pseudo
>
> for example?

We could, yes.

> > -The old site at:
> > -   https://github.com/wrpseudo/pseudo
> > -
> > -is no longer the "real" master, although I'll probably try to keep it in
> > -sync.
> > -
> >   OVERVIEW:
> >   
> >   The pseudo utility offers a way to run commands in a virtualized "root"
> > @@ -88,7 +80,6 @@ available for other people to look at.
> >   
> >   CONTACT:
> >   
> > -Discussions and patches should be directed at the openembedded-core mailing
> > -list at openembedded-core at lists.openembedded.org. More information at
> > -https://www.openembedded.org/wiki/Mailing_lists. Bugs should be filed with
> > -the Yocto project at https://bugzilla.yoctoproject.org/.
> > +Patches should be directed at the yocto-patches mailing list at 
> > yocto-patches
> > +at lists.yoctoproject.org.  Bugs should be filed with the Yocto project
> > +at https://bugzilla.yoctoproject.org/.
> 
> Maybe we want to add a prefix here too so we can identify easily for 
> which project this is for?

It is implied, but yes, we should mention that specifically, agreed.

Cheers,

Richard




-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198100): 
https://lists.openembedded.org/g/openembedded-core/message/198100
Mute This Topic: https://lists.openembedded.org/mt/105440467/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/1] init-ifupdown: add predictable interface names

2024-04-10 Thread Ross Burton
On 10 Jan 2024, at 21:10, Joe Slater via lists.openembedded.org 
 wrote:
> -# Wired or wireless interfaces
> -auto eth0
> +# Wired or wireless interfaces including predictable names
> +auto /eth0

Obviously we’re days away from release but I think we need to revert or rewrite 
this.

The /name syntax is specific to “proper” ifupdown, so any system which uses 
busybox’s ifupdown will fail to have functional networking.  We don’t see this 
on the AB because there is a separate interfaces file for the qemuall override, 
this file is only used in non-qemu situations.

Ross
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198099): 
https://lists.openembedded.org/g/openembedded-core/message/198099
Mute This Topic: https://lists.openembedded.org/mt/103649977/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 5/5] licenses.conf: Add missing LGPLv2.0+ license

2024-04-10 Thread Steve Sakoman
From: Colin McAllister 

Adds LGPLv2.0+ license variation to match LGPLv2+.

Signed-off-by: Colin McAllister 
Signed-off-by: Steve Sakoman 
---
 meta/conf/licenses.conf | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/conf/licenses.conf b/meta/conf/licenses.conf
index d14c365977..c78823e847 100644
--- a/meta/conf/licenses.conf
+++ b/meta/conf/licenses.conf
@@ -37,6 +37,7 @@ SPDXLICENSEMAP[GPL-3.0-only] = "GPL-3.0"
 SPDXLICENSEMAP[LGPLv2] = "LGPL-2.0"
 SPDXLICENSEMAP[LGPLv2+] = "LGPL-2.0+"
 SPDXLICENSEMAP[LGPLv2.0] = "LGPL-2.0"
+SPDXLICENSEMAP[LGPLv2.0+] = "LGPL-2.0+"
 SPDXLICENSEMAP[LGPL-2.0-only] = "LGPL-2.0"
 SPDXLICENSEMAP[LGPL2.1] = "LGPL-2.1"
 SPDXLICENSEMAP[LGPLv2.1] = "LGPL-2.1"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198098): 
https://lists.openembedded.org/g/openembedded-core/message/198098
Mute This Topic: https://lists.openembedded.org/mt/105441138/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 4/5] xserver-xorg: Backport fix for CVE-2024-31080

2024-04-10 Thread Steve Sakoman
From: Ashish Sharma 

Upstream-Status: Backport 
[https://gitlab.freedesktop.org/xorg/xserver/-/commit/96798fc1967491c80a4d0c8d9e0a80586cb2152b]
Signed-off-by: Ashish Sharma 
Signed-off-by: Steve Sakoman 
---
 .../xserver-xorg/CVE-2024-31080.patch | 49 +++
 .../xorg-xserver/xserver-xorg_1.20.14.bb  |  1 +
 2 files changed, 50 insertions(+)
 create mode 100644 
meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31080.patch

diff --git 
a/meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31080.patch 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31080.patch
new file mode 100644
index 00..da735efb2b
--- /dev/null
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31080.patch
@@ -0,0 +1,49 @@
+From 96798fc1967491c80a4d0c8d9e0a80586cb2152b Mon Sep 17 00:00:00 2001
+From: Alan Coopersmith 
+Date: Fri, 22 Mar 2024 18:51:45 -0700
+Subject: [PATCH] Xi: ProcXIGetSelectedEvents needs to use unswapped length to
+ send reply
+
+CVE-2024-31080
+
+Reported-by: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=69762
+Fixes: 53e821ab4 ("Xi: add request processing for XIGetSelectedEvents.")
+Signed-off-by: Alan Coopersmith 
+Part-of: 
+
+Upstream-Status: Backport 
[https://gitlab.freedesktop.org/xorg/xserver/-/commit/96798fc1967491c80a4d0c8d9e0a80586cb2152b]
+CVE: CVE-2024-31080
+Signed-off-by: Ashish Sharma 
+
+ Xi/xiselectev.c | 5 -
+ 1 file changed, 4 insertions(+), 1 deletion(-)
+
+diff --git a/Xi/xiselectev.c b/Xi/xiselectev.c
+index edcb8a0d36..ac14949871 100644
+--- a/Xi/xiselectev.c
 b/Xi/xiselectev.c
+@@ -349,6 +349,7 @@ ProcXIGetSelectedEvents(ClientPtr client)
+ InputClientsPtr others = NULL;
+ xXIEventMask *evmask = NULL;
+ DeviceIntPtr dev;
++uint32_t length;
+ 
+ REQUEST(xXIGetSelectedEventsReq);
+ REQUEST_SIZE_MATCH(xXIGetSelectedEventsReq);
+@@ -418,10 +419,12 @@ ProcXIGetSelectedEvents(ClientPtr client)
+ }
+ }
+ 
++/* save the value before SRepXIGetSelectedEvents swaps it */
++length = reply.length;
+ WriteReplyToClient(client, sizeof(xXIGetSelectedEventsReply), );
+ 
+ if (reply.num_masks)
+-WriteToClient(client, reply.length * 4, buffer);
++WriteToClient(client, length * 4, buffer);
+ 
+ free(buffer);
+ return Success;
+-- 
+GitLab
+
diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.20.14.bb 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.20.14.bb
index ade250542f..04a6e734ef 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.20.14.bb
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.20.14.bb
@@ -31,6 +31,7 @@ SRC_URI += 
"file://0001-xf86pciBus.c-use-Intel-ddx-only-for-pre-gen4-hardwar.pat
file://CVE-2024-0408.patch \
file://CVE-2024-0409.patch \
file://CVE-2024-31081.patch \
+   file://CVE-2024-31080.patch \
 "
 SRC_URI[md5sum] = "453fc86aac8c629b3a5b77e8dcca30bf"
 SRC_URI[sha256sum] = 
"54b199c9280ff8bf0f73a54a759645bd0eeeda7255d1c99310d5b7595f3ac066"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198097): 
https://lists.openembedded.org/g/openembedded-core/message/198097
Mute This Topic: https://lists.openembedded.org/mt/105441137/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 3/5] xserver-xorg: Backport fix for CVE-2024-31081

2024-04-10 Thread Steve Sakoman
From: Ashish Sharma 

Upstream-Status: Backport 
[https://gitlab.freedesktop.org/xorg/xserver/-/commit/3e77295f888c67fc7645db5d0c00926a29ffecee]
Signed-off-by: Ashish Sharma 
Signed-off-by: Steve Sakoman 
---
 .../xserver-xorg/CVE-2024-31081.patch | 47 +++
 .../xorg-xserver/xserver-xorg_1.20.14.bb  |  1 +
 2 files changed, 48 insertions(+)
 create mode 100644 
meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31081.patch

diff --git 
a/meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31081.patch 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31081.patch
new file mode 100644
index 00..d2c551a0e5
--- /dev/null
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31081.patch
@@ -0,0 +1,47 @@
+From 3e77295f888c67fc7645db5d0c00926a29ffecee Mon Sep 17 00:00:00 2001
+From: Alan Coopersmith 
+Date: Fri, 22 Mar 2024 18:56:27 -0700
+Subject: [PATCH] Xi: ProcXIPassiveGrabDevice needs to use unswapped length to
+ send reply
+
+CVE-2024-31081
+
+Fixes: d220d6907 ("Xi: add GrabButton and GrabKeysym code.")
+Signed-off-by: Alan Coopersmith 
+Part-of: 
+
+Upstream-Status: Backport 
[https://gitlab.freedesktop.org/xorg/xserver/-/commit/3e77295f888c67fc7645db5d0c00926a29ffecee]
+CVE: CVE-2024-31081
+Signed-off-by: Ashish Sharma 
+
+ Xi/xipassivegrab.c | 5 -
+ 1 file changed, 4 insertions(+), 1 deletion(-)
+
+diff --git a/Xi/xipassivegrab.c b/Xi/xipassivegrab.c
+index c9ac2f8553..896233bec2 100644
+--- a/Xi/xipassivegrab.c
 b/Xi/xipassivegrab.c
+@@ -93,6 +93,7 @@ ProcXIPassiveGrabDevice(ClientPtr client)
+ GrabParameters param;
+ void *tmp;
+ int mask_len;
++uint32_t length;
+ 
+ REQUEST(xXIPassiveGrabDeviceReq);
+ REQUEST_FIXED_SIZE(xXIPassiveGrabDeviceReq,
+@@ -247,9 +248,11 @@ ProcXIPassiveGrabDevice(ClientPtr client)
+ }
+ }
+ 
++/* save the value before SRepXIPassiveGrabDevice swaps it */
++length = rep.length;
+ WriteReplyToClient(client, sizeof(rep), );
+ if (rep.num_modifiers)
+-WriteToClient(client, rep.length * 4, modifiers_failed);
++WriteToClient(client, length * 4, modifiers_failed);
+ 
+  out:
+ free(modifiers_failed);
+-- 
+GitLab
+
diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.20.14.bb 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.20.14.bb
index d6c6c5bd45..ade250542f 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.20.14.bb
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.20.14.bb
@@ -30,6 +30,7 @@ SRC_URI += 
"file://0001-xf86pciBus.c-use-Intel-ddx-only-for-pre-gen4-hardwar.pat
file://CVE-2024-21886-2.patch \
file://CVE-2024-0408.patch \
file://CVE-2024-0409.patch \
+   file://CVE-2024-31081.patch \
 "
 SRC_URI[md5sum] = "453fc86aac8c629b3a5b77e8dcca30bf"
 SRC_URI[sha256sum] = 
"54b199c9280ff8bf0f73a54a759645bd0eeeda7255d1c99310d5b7595f3ac066"
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198096): 
https://lists.openembedded.org/g/openembedded-core/message/198096
Mute This Topic: https://lists.openembedded.org/mt/105441136/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 2/5] go: Backport fix for CVE-2024-24784

2024-04-10 Thread Steve Sakoman
From: Ashish Sharma 

Upstream-Status: Backport 
[https://github.com/golang/go/commit/5330cd225ba54c7dc78c1b46dcdf61a4671a632c]
Signed-off-by: Ashish Sharma 
Signed-off-by: Steve Sakoman 
---
 meta/recipes-devtools/go/go-1.14.inc  |   1 +
 .../go/go-1.14/CVE-2024-24784.patch   | 205 ++
 2 files changed, 206 insertions(+)
 create mode 100644 meta/recipes-devtools/go/go-1.14/CVE-2024-24784.patch

diff --git a/meta/recipes-devtools/go/go-1.14.inc 
b/meta/recipes-devtools/go/go-1.14.inc
index 69b65f3eb2..9c7ceda891 100644
--- a/meta/recipes-devtools/go/go-1.14.inc
+++ b/meta/recipes-devtools/go/go-1.14.inc
@@ -91,6 +91,7 @@ SRC_URI += "\
 file://CVE-2023-45289.patch \
 file://CVE-2023-45290.patch \
 file://CVE-2024-24785.patch \
+file://CVE-2024-24784.patch \
 "
 
 SRC_URI_append_libc-musl = " 
file://0009-ld-replace-glibc-dynamic-linker-with-musl.patch"
diff --git a/meta/recipes-devtools/go/go-1.14/CVE-2024-24784.patch 
b/meta/recipes-devtools/go/go-1.14/CVE-2024-24784.patch
new file mode 100644
index 00..e9d9d972b9
--- /dev/null
+++ b/meta/recipes-devtools/go/go-1.14/CVE-2024-24784.patch
@@ -0,0 +1,205 @@
+From 5330cd225ba54c7dc78c1b46dcdf61a4671a632c Mon Sep 17 00:00:00 2001
+From: Roland Shoemaker 
+Date: Wed, 10 Jan 2024 11:02:14 -0800
+Subject: [PATCH] [release-branch.go1.22] net/mail: properly handle special
+ characters in phrase and obs-phrase
+
+Fixes a couple of misalignments with RFC 5322 which introduce
+significant diffs between (mostly) conformant parsers.
+
+This change reverts the changes made in CL50911, which allowed certain
+special RFC 5322 characters to appear unquoted in the "phrase" syntax.
+It is unclear why this change was made in the first place, and created
+a divergence from comformant parsers. In particular this resulted in
+treating comments in display names incorrectly.
+
+Additionally properly handle trailing malformed comments in the group
+syntax.
+
+For #65083
+Fixed #65849
+
+Change-Id: I00dddc044c6ae3381154e43236632604c390f672
+Reviewed-on: https://go-review.googlesource.com/c/go/+/96
+Reviewed-by: Damien Neil 
+LUCI-TryBot-Result: Go LUCI 

+Reviewed-on: https://go-review.googlesource.com/c/go/+/566215
+Reviewed-by: Carlos Amedee 
+
+Upstream-Status: Backport 
[https://github.com/golang/go/commit/5330cd225ba54c7dc78c1b46dcdf61a4671a632c]
+CVE: CVE-2024-24784
+Signed-off-by: Ashish Sharma 
+
+ src/net/mail/message.go  | 30 +++
+ src/net/mail/message_test.go | 40 ++--
+ 2 files changed, 46 insertions(+), 24 deletions(-)
+
+diff --git a/src/net/mail/message.go b/src/net/mail/message.go
+index af516fc30f470..fc2a9e46f811b 100644
+--- a/src/net/mail/message.go
 b/src/net/mail/message.go
+@@ -280,7 +280,7 @@ func (a *Address) String() string {
+   // Add quotes if needed
+   quoteLocal := false
+   for i, r := range local {
+-  if isAtext(r, false, false) {
++  if isAtext(r, false) {
+   continue
+   }
+   if r == '.' {
+@@ -444,7 +444,7 @@ func (p *addrParser) parseAddress(handleGroup bool) 
([]*Address, error) {
+   if !p.consume('<') {
+   atext := true
+   for _, r := range displayName {
+-  if !isAtext(r, true, false) {
++  if !isAtext(r, true) {
+   atext = false
+   break
+   }
+@@ -479,7 +479,9 @@ func (p *addrParser) consumeGroupList() ([]*Address, 
error) {
+   // handle empty group.
+   p.skipSpace()
+   if p.consume(';') {
+-  p.skipCFWS()
++  if !p.skipCFWS() {
++  return nil, errors.New("mail: misformatted 
parenthetical comment")
++  }
+   return group, nil
+   }
+ 
+@@ -496,7 +498,9 @@ func (p *addrParser) consumeGroupList() ([]*Address, 
error) {
+   return nil, errors.New("mail: misformatted 
parenthetical comment")
+   }
+   if p.consume(';') {
+-  p.skipCFWS()
++  if !p.skipCFWS() {
++  return nil, errors.New("mail: misformatted 
parenthetical comment")
++  }
+   break
+   }
+   if !p.consume(',') {
+@@ -566,6 +570,12 @@ func (p *addrParser) consumePhrase() (phrase string, err 
error) {
+   var words []string
+   var isPrevEncoded bool
+   for {
++  // obs-phrase allows CFWS after one word
++  if len(words) > 0 {
++  if !p.skipCFWS() {
++  return "", errors.New("mail: misformatted 
parenthetical comment")
++  }
++  }
+   // word = atom / quoted-string
+   var word string
+   p.skipSpace()
+@@ -661,7 +671,6 @@ 

[OE-core][dunfell 1/5] ncurses: Backport fix for CVE-2023-50495

2024-04-10 Thread Steve Sakoman
From: Vijay Anusuri 

Upstream-Status: Backport from 
https://github.com/ThomasDickey/ncurses-snapshots/commit/efe9674ee14b14b788f9618941f97d31742f0adc

Reference: 
https://invisible-island.net/archives/ncurses/6.4/ncurses-6.4-20230424.patch.gz

Signed-off-by: Vijay Anusuri 
Signed-off-by: Steve Sakoman 
---
 .../ncurses/files/CVE-2023-50495.patch| 79 +++
 meta/recipes-core/ncurses/ncurses_6.2.bb  |  1 +
 2 files changed, 80 insertions(+)
 create mode 100644 meta/recipes-core/ncurses/files/CVE-2023-50495.patch

diff --git a/meta/recipes-core/ncurses/files/CVE-2023-50495.patch 
b/meta/recipes-core/ncurses/files/CVE-2023-50495.patch
new file mode 100644
index 00..58c23866d1
--- /dev/null
+++ b/meta/recipes-core/ncurses/files/CVE-2023-50495.patch
@@ -0,0 +1,79 @@
+Fix for CVE-2023-50495 from upstream:
+https://github.com/ThomasDickey/ncurses-snapshots/commit/efe9674ee14b14b788f9618941f97d31742f0adc
+
+Reference:
+https://invisible-island.net/archives/ncurses/6.4/ncurses-6.4-20230424.patch.gz
+
+Upstream-Status: Backport [import from suse 
ftp.pbone.net/mirror/ftp.opensuse.org/update/leap-micro/5.3/sle/src/ncurses-6.1-15.5.20.1.src.rpm
+Upstream commit 
https://github.com/ThomasDickey/ncurses-snapshots/commit/efe9674ee14b14b788f9618941f97d31742f0adc]
+CVE: CVE-2023-50495
+Signed-off-by: Vijay Anusuri 
+---
+ ncurses/tinfo/parse_entry.c | 23 ---
+ 1 file changed, 16 insertions(+), 7 deletions(-)
+
+diff --git a/ncurses/tinfo/parse_entry.c b/ncurses/tinfo/parse_entry.c
+index 23574b66..56ba9ae6 100644
+--- a/ncurses/tinfo/parse_entry.c
 b/ncurses/tinfo/parse_entry.c
+@@ -110,7 +110,7 @@ _nc_extend_names(ENTRY * entryp, const char *name, int 
token_type)
+   /* Well, we are given a cancel for a name that we don't recognize */
+   return _nc_extend_names(entryp, name, STRING);
+ default:
+-  return 0;
++  return NULL;
+ }
+ 
+ /* Adjust the 'offset' (insertion-point) to keep the lists of extended
+@@ -142,6 +142,11 @@ _nc_extend_names(ENTRY * entryp, const char *name, int 
token_type)
+   for (last = (unsigned) (max - 1); last > tindex; last--)
+ 
+ if (!found) {
++  char *saved;
++
++  if ((saved = _nc_save_str(name)) == NULL)
++  return NULL;
++
+   switch (token_type) {
+   case BOOLEAN:
+   tp->ext_Booleans++;
+@@ -169,7 +174,7 @@ _nc_extend_names(ENTRY * entryp, const char *name, int 
token_type)
+   TYPE_REALLOC(char *, actual, tp->ext_Names);
+   while (--actual > offset)
+   tp->ext_Names[actual] = tp->ext_Names[actual - 1];
+-  tp->ext_Names[offset] = _nc_save_str(name);
++  tp->ext_Names[offset] = saved;
+ }
+ 
+ temp.nte_name = tp->ext_Names[offset];
+@@ -337,6 +342,8 @@ _nc_parse_entry(ENTRY * entryp, int literal, bool silent)
+   bool is_use = (strcmp(_nc_curr_token.tk_name, "use") == 0);
+   bool is_tc = !is_use && (strcmp(_nc_curr_token.tk_name, "tc") == 0);
+   if (is_use || is_tc) {
++  char *saved;
++
+   if (!VALID_STRING(_nc_curr_token.tk_valstring)
+   || _nc_curr_token.tk_valstring[0] == '\0') {
+   _nc_warning("missing name for use-clause");
+@@ -350,11 +357,13 @@ _nc_parse_entry(ENTRY * entryp, int literal, bool silent)
+   _nc_curr_token.tk_valstring);
+   continue;
+   }
+-  entryp->uses[entryp->nuses].name = 
_nc_save_str(_nc_curr_token.tk_valstring);
+-  entryp->uses[entryp->nuses].line = _nc_curr_line;
+-  entryp->nuses++;
+-  if (entryp->nuses > 1 && is_tc) {
+-  BAD_TC_USAGE
++  if ((saved = _nc_save_str(_nc_curr_token.tk_valstring)) != NULL) {
++  entryp->uses[entryp->nuses].name = saved;
++  entryp->uses[entryp->nuses].line = _nc_curr_line;
++  entryp->nuses++;
++  if (entryp->nuses > 1 && is_tc) {
++  BAD_TC_USAGE
++  }
+   }
+   } else {
+   /* normal token lookup */
+-- 
+2.25.1
+
diff --git a/meta/recipes-core/ncurses/ncurses_6.2.bb 
b/meta/recipes-core/ncurses/ncurses_6.2.bb
index 33285bcb5b..dbff149f55 100644
--- a/meta/recipes-core/ncurses/ncurses_6.2.bb
+++ b/meta/recipes-core/ncurses/ncurses_6.2.bb
@@ -6,6 +6,7 @@ SRC_URI += "file://0001-tic-hang.patch \
file://CVE-2021-39537.patch \
file://CVE-2022-29458.patch \
file://CVE-2023-29491.patch \
+   file://CVE-2023-50495.patch \
"
 # commit id corresponds to the revision in package version
 SRCREV = "a669013cd5e9d6434e5301348ea51baf306c93c4"
-- 
2.34.1


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

[OE-core][dunfell 0/5] Patch review

2024-04-10 Thread Steve Sakoman
Please review this set of changes for dunfell and have comments back by
end of day Friday, April 12

Passed a-full on autobuilder:

https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/6774

The following changes since commit 47ce772102b45db14dc21112367534ea1c37e33c:

  perf: bump PR to deal with sstate corruption on autobuilder (2024-04-02 
05:46:56 -1000)

are available in the Git repository at:

  https://git.openembedded.org/openembedded-core-contrib stable/dunfell-nut
  
http://cgit.openembedded.org/openembedded-core-contrib/log/?h=stable/dunfell-nut

Ashish Sharma (3):
  go: Backport fix for CVE-2024-24784
  xserver-xorg: Backport fix for CVE-2024-31081
  xserver-xorg: Backport fix for CVE-2024-31080

Colin McAllister (1):
  licenses.conf: Add missing LGPLv2.0+ license

Vijay Anusuri (1):
  ncurses: Backport fix for CVE-2023-50495

 meta/conf/licenses.conf   |   1 +
 .../ncurses/files/CVE-2023-50495.patch|  79 +++
 meta/recipes-core/ncurses/ncurses_6.2.bb  |   1 +
 meta/recipes-devtools/go/go-1.14.inc  |   1 +
 .../go/go-1.14/CVE-2024-24784.patch   | 205 ++
 .../xserver-xorg/CVE-2024-31080.patch |  49 +
 .../xserver-xorg/CVE-2024-31081.patch |  47 
 .../xorg-xserver/xserver-xorg_1.20.14.bb  |   2 +
 8 files changed, 385 insertions(+)
 create mode 100644 meta/recipes-core/ncurses/files/CVE-2023-50495.patch
 create mode 100644 meta/recipes-devtools/go/go-1.14/CVE-2024-24784.patch
 create mode 100644 
meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31080.patch
 create mode 100644 
meta/recipes-graphics/xorg-xserver/xserver-xorg/CVE-2024-31081.patch

-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198093): 
https://lists.openembedded.org/g/openembedded-core/message/198093
Mute This Topic: https://lists.openembedded.org/mt/105441132/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][kirkstone][PATCH v2 1/1] expat: fix CVE-2023-52425

2024-04-10 Thread Steve Sakoman
Unfortunately this change is causing ptest failures:

{'expat': ['test_accounting_precision',
   'test_return_ns_triplet',
   'test_column_number_after_parse',
   'test_default_current',
   'test_external_entity_values']}

Could you investigate and see if you can provide a fix?  Otherwise
I'll need to revert this prior to the next kirkstone release.

Steve

On Fri, Mar 29, 2024 at 4:12 AM Meenali Gupta via
lists.openembedded.org
 wrote:
>
> From: Meenali Gupta 
>
> libexpat through 2.5.0 allows a denial of service (resource consumption) 
> because
> many full reparsings are required in the case of a large token for which 
> multiple
> buffer fills are needed.
>
> References:
> https://nvd.nist.gov/vuln/detail/CVE-2023-52425
>
> Changes related to test directory are not included as most of the files are 
> not present
> and are introduced in the later version.
>
> Signed-off-by: Meenali Gupta 
> ---
>  .../expat/expat/CVE-2023-52425-0001.patch |  40 
>  .../expat/expat/CVE-2023-52425-0002.patch |  87 +++
>  .../expat/expat/CVE-2023-52425-0003.patch | 222 ++
>  .../expat/expat/CVE-2023-52425-0004.patch |  42 
>  .../expat/expat/CVE-2023-52425-0005.patch |  69 ++
>  .../expat/expat/CVE-2023-52425-0006.patch |  67 ++
>  .../expat/expat/CVE-2023-52425-0007.patch | 159 +
>  .../expat/expat/CVE-2023-52425-0008.patch |  95 
>  .../expat/expat/CVE-2023-52425-0009.patch |  52 
>  .../expat/expat/CVE-2023-52425-0010.patch | 111 +
>  .../expat/expat/CVE-2023-52425-0011.patch |  89 +++
>  .../expat/expat/CVE-2023-52425-0012.patch |  87 +++
>  meta/recipes-core/expat/expat_2.5.0.bb|  12 +
>  13 files changed, 1132 insertions(+)
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0001.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0002.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0003.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0004.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0005.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0006.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0007.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0008.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0009.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0010.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0011.patch
>  create mode 100644 meta/recipes-core/expat/expat/CVE-2023-52425-0012.patch
>
> diff --git a/meta/recipes-core/expat/expat/CVE-2023-52425-0001.patch 
> b/meta/recipes-core/expat/expat/CVE-2023-52425-0001.patch
> new file mode 100644
> index 00..4e21ade018
> --- /dev/null
> +++ b/meta/recipes-core/expat/expat/CVE-2023-52425-0001.patch
> @@ -0,0 +1,40 @@
> +From d5b02e96ab95d2a7ae0aea72d00054b9d036d76d Mon Sep 17 00:00:00 2001
> +From: Sebastian Pipping 
> +Date: Thu, 9 Nov 2023 19:28:05 +0100
> +Subject: [PATCH] xmlwf: Document argument "-q"
> +
> +Rebased-and-adapted-by: Snild Dolkow 
> +
> +CVE: CVE-2023-52425
> +
> +Upstream-Status: Backport 
> [https://github.com/libexpat/libexpat/commit/d5b02e96ab95d2a7ae0aea72d00054b9d036d76d]
> +
> +Signed-off-by: Meenali Gupta 
> +---
> + doc/xmlwf.xml | 10 ++
> + 1 file changed, 10 insertions(+)
> +
> +diff --git a/doc/xmlwf.xml b/doc/xmlwf.xml
> +index 9603abf..3d35393 100644
> +--- a/doc/xmlwf.xml
>  b/doc/xmlwf.xml
> +@@ -313,6 +313,16 @@ supports both.
> + 
> +   
> +
> ++  
> ++-q
> ++
> ++  
> ++Disable reparse deferral, and allow quadratic parse runtime
> ++on large tokens (default: reparse deferral enabled).
> ++  
> ++
> ++  
> ++
> +   
> + -r
> + 
> +--
> +2.40.0
> +
> diff --git a/meta/recipes-core/expat/expat/CVE-2023-52425-0002.patch 
> b/meta/recipes-core/expat/expat/CVE-2023-52425-0002.patch
> new file mode 100644
> index 00..8376727778
> --- /dev/null
> +++ b/meta/recipes-core/expat/expat/CVE-2023-52425-0002.patch
> @@ -0,0 +1,87 @@
> +From 09fdf998e7cf3f8f9327e6602077791095aedd4d Mon Sep 17 00:00:00 2001
> +From: Sebastian Pipping 
> +Date: Thu, 9 Nov 2023 19:14:14 +0100
> +Subject: [PATCH] xmlwf: Support disabling reparse deferral
> +
> +Rebased-and-adapted-by: Snild Dolkow 
> +
> +CVE: CVE-2023-52425
> +
> +Upstream-Status: Backport 
> [https://github.com/libexpat/libexpat/commit/09fdf998e7cf3f8f9327e6602077791095aedd4d]
> +
> +Signed-off-by: Meenali Gupta 
> +---
> + xmlwf/xmlwf.c  | 20 
> + xmlwf/xmlwf_helpgen.py |  4 
> + 2 files changed, 24 insertions(+)
> +
> +diff --git a/xmlwf/xmlwf.c b/xmlwf/xmlwf.c
> +index dd023a9..9a5441c 100644
> +--- a/xmlwf/xmlwf.c
>  

Re: [OE-core] [yocto-patches] [PATCH 1/2] README: Switch to yocto-patches mailing list

2024-04-10 Thread Quentin Schulz via lists.openembedded.org

Hi Richard,

On 4/10/24 11:29, Richard Purdie via lists.yoctoproject.org wrote:

We're trying to align the various components so tweak pseudo to match our
other projects a little more and use the yocto-patches mailing list for
submissions.

Signed-off-by: Richard Purdie 
---
  README | 17 -
  1 file changed, 4 insertions(+), 13 deletions(-)

diff --git a/README b/README
index 468cf66..425ee6e 100644
--- a/README
+++ b/README
@@ -1,17 +1,9 @@
  pseudo -- an analogue to sudo
  
-IMPORTANT NOTE:

+The official home for pseudo's git repository is:
  
-As of this writing, the official home for pseudo's git repository has

-changed to:
git://git.yoctoproject.org/pseudo
  


Maybe we could use https here?

https://git.yoctoproject.org/pseudo

for example?


-The old site at:
-   https://github.com/wrpseudo/pseudo
-
-is no longer the "real" master, although I'll probably try to keep it in
-sync.
-
  OVERVIEW:
  
  The pseudo utility offers a way to run commands in a virtualized "root"

@@ -88,7 +80,6 @@ available for other people to look at.
  
  CONTACT:
  
-Discussions and patches should be directed at the openembedded-core mailing

-list at openembedded-core at lists.openembedded.org. More information at
-https://www.openembedded.org/wiki/Mailing_lists. Bugs should be filed with
-the Yocto project at https://bugzilla.yoctoproject.org/.
+Patches should be directed at the yocto-patches mailing list at yocto-patches
+at lists.yoctoproject.org.  Bugs should be filed with the Yocto project
+at https://bugzilla.yoctoproject.org/.


Maybe we want to add a prefix here too so we can identify easily for 
which project this is for?


e.g. for meta-rockchip we have:

"""
git config format.subjectprefix "meta-rockchip] [PATCH"
"""

maybe we want

"""
git config format.subjectprefix "pseudo] [PATCH"
"""

?

Cheers,
Quentin

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198091): 
https://lists.openembedded.org/g/openembedded-core/message/198091
Mute This Topic: https://lists.openembedded.org/mt/105440467/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] rust: set CVE_STATUS for CVE-2024-24576

2024-04-10 Thread Sadineni, Harish via lists.openembedded.org
From: Harish Sadineni 

CVE-2024-24576 only applies when invoking batch files (with the `bat` and `cmd` 
extensions) on Windows & No other platform or use is affected.
More details about CVE is here: https://nvd.nist.gov/vuln/detail/CVE-2024-24576

Signed-off-by: Harish Sadineni 
---
 meta/recipes-devtools/rust/rust-source.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-devtools/rust/rust-source.inc 
b/meta/recipes-devtools/rust/rust-source.inc
index 6bef99039d..b14221b6cb 100644
--- a/meta/recipes-devtools/rust/rust-source.inc
+++ b/meta/recipes-devtools/rust/rust-source.inc
@@ -20,3 +20,5 @@ RUSTSRC = "${WORKDIR}/rustc-${RUST_VERSION}-src"
 
 UPSTREAM_CHECK_URI = 
"https://forge.rust-lang.org/infra/other-installation-methods.html;
 UPSTREAM_CHECK_REGEX = "rustc-(?P\d+(\.\d+)+)-src"
+
+CVE_STATUS[CVE-2024-24576] = "not-applicable-platform: Issue only applies on 
Windows"
-- 
2.43.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198090): 
https://lists.openembedded.org/g/openembedded-core/message/198090
Mute This Topic: https://lists.openembedded.org/mt/105439811/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] devtool: standard: throws appropriate error if source is in detached HEAD

2024-04-10 Thread Peter Kjellerstedt
> -Original Message-
> From: openembedded-core@lists.openembedded.org 
>  On Behalf Of Julien Stephan
> Sent: den 10 april 2024 09:07
> To: openembedded-core@lists.openembedded.org
> Cc: Julien Stephan 
> Subject: [OE-core] [PATCH v2] devtool: standard: throws appropriate error if 
> source is in detached HEAD
> 
> If source is in detached HEAD, we get the following error when using
> detvool finish command:
> 
> [...]
> File "<...>/poky/scripts/lib/devtool/standard.py", line 1938, in 
> _update_recipe
>   bb.process.run('git checkout %s' % startbranch, cwd=srctree)
> File "<...>/poky/bitbake/lib/bb/process.py", line 189, in run
>   raise ExecutionError(cmd, pipe.returncode, stdout, stderr)
>   bb.process.ExecutionError: Execution of 'git checkout (HEAD detached at 
> 9bbf87e)' failed with exit code 2:
>   /bin/sh: -c: line 1: syntax error near unexpected token `('
>   /bin/sh: -c: line 1: `git checkout (HEAD detached at 9bbf87e)'
> 
> Check this and throws an appropriate error in this case
> 
> Signed-off-by: Julien Stephan 
> 
> ---
> 
> Changes in v2:
> 
> - update error message to avoid confusion with a potentially non
>   existing "main" branch

As you noted in an earlier mail, the same error message is used some 
lines later. Please change that one too.

> ---
>  scripts/lib/devtool/standard.py | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/scripts/lib/devtool/standard.py
> b/scripts/lib/devtool/standard.py
> index 2c174927ddb..51e5794a0a7 100644
> --- a/scripts/lib/devtool/standard.py
> +++ b/scripts/lib/devtool/standard.py
> @@ -1888,6 +1888,8 @@ def _update_recipe(recipename, workspace, rd, mode,
> appendlayerdir, wildcard_ver
>  for line in stdout.splitlines():
>  branchname = line[2:]
>  if line.startswith('* '):
> +if 'HEAD' in line:
> +raise DevtoolError('Detached HEAD - please check out a 
> branch, e.g., "devtool"')
>  startbranch = branchname
>  if branchname.startswith(override_branch_prefix):
>  override_branches.append(branchname)
> --
> 2.44.0

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198089): 
https://lists.openembedded.org/g/openembedded-core/message/198089
Mute This Topic: https://lists.openembedded.org/mt/105437662/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] fchmodat: Ignore symlinks, don't return an error

2024-04-10 Thread Richard Purdie
Ugrading to coreutils 9.5 pulls in:
https://git.savannah.gnu.org/cgit/coreutils.git/commit/src/chmod.c?id=07a69fc3ba717b879218592db685a1c79869cb28
which calls fchmodat with AT_SYMLINK_NOFOLLOW.

All the docs I can find say that symlinks don't have modes and are
always readable. With the error in place, we see things like:

chmod: changing permissions of 
'XXX/build-appliance-image/15.0.0/rootfs/home/builder/poky/scripts/esdk-tools/runqemu-gen-tapdevs':
 Function not implemented

when building build-appliance-image, which uses tools from coreutils-native.
It is only a matter of time before distros start shipping those newer tools.

I believe the correct thing to do is just return success here rather
than failure. This does fix the build failure.

Signed-off-by: Richard Purdie 
---
 ports/unix/guts/fchmodat.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/ports/unix/guts/fchmodat.c b/ports/unix/guts/fchmodat.c
index d1ac7bb..5a31151 100644
--- a/ports/unix/guts/fchmodat.c
+++ b/ports/unix/guts/fchmodat.c
@@ -28,9 +28,8 @@
return rc;
}
if (S_ISLNK(buf.st_mode)) {
-   /* we don't really support chmod of a symlink */
-   errno = ENOSYS;
-   return -1;
+   /* according to docs, "chmod on a symbolic link always succeeds 
and has no effect" */
+   return 0;
}
save_errno = errno;
 
-- 
2.40.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198088): 
https://lists.openembedded.org/g/openembedded-core/message/198088
Mute This Topic: https://lists.openembedded.org/mt/105438610/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] README: Switch to yocto-patches mailing list

2024-04-10 Thread Richard Purdie
We're trying to align the various components so tweak pseudo to match our
other projects a little more and use the yocto-patches mailing list for
submissions.

Signed-off-by: Richard Purdie 
---
 README | 17 -
 1 file changed, 4 insertions(+), 13 deletions(-)

diff --git a/README b/README
index 468cf66..425ee6e 100644
--- a/README
+++ b/README
@@ -1,17 +1,9 @@
 pseudo -- an analogue to sudo
 
-IMPORTANT NOTE:
+The official home for pseudo's git repository is:
 
-As of this writing, the official home for pseudo's git repository has
-changed to:
git://git.yoctoproject.org/pseudo
 
-The old site at:
-   https://github.com/wrpseudo/pseudo
-
-is no longer the "real" master, although I'll probably try to keep it in
-sync.
-
 OVERVIEW:
 
 The pseudo utility offers a way to run commands in a virtualized "root"
@@ -88,7 +80,6 @@ available for other people to look at.
 
 CONTACT:
 
-Discussions and patches should be directed at the openembedded-core mailing
-list at openembedded-core at lists.openembedded.org. More information at
-https://www.openembedded.org/wiki/Mailing_lists. Bugs should be filed with
-the Yocto project at https://bugzilla.yoctoproject.org/.
+Patches should be directed at the yocto-patches mailing list at yocto-patches
+at lists.yoctoproject.org.  Bugs should be filed with the Yocto project
+at https://bugzilla.yoctoproject.org/.
-- 
2.40.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198087): 
https://lists.openembedded.org/g/openembedded-core/message/198087
Mute This Topic: https://lists.openembedded.org/mt/105438609/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] Yocto Project Status 9 April 2024 (WW15)

2024-04-10 Thread Alexander Kanavin
On Tue, 9 Apr 2024 at 16:55, Stephen Jolley  wrote:
> We received some disappointing feedback from a key organization in the RISC-V 
> ecosystem that the Yocto Project was not important and not worth funding. We 
> will take this feedback into account.

Can anything else be disclosed about this?

Alex

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198086): 
https://lists.openembedded.org/g/openembedded-core/message/198086
Mute This Topic: https://lists.openembedded.org/mt/105423109/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] gtk4: update 4.14.1 -> 4.14.2

2024-04-10 Thread Markus Volk
Overview of Changes in 4.14.2, 03-04-2024
=

* GtkScale:
 - Improve positioning of values in some cases

* Theme:
 - Make progress in entries visible

* Accessibility:
 - Fix text insertion handling

* GDK:
 - dnd: Use the default cursor durion motion
 - dnd: Use a better cursor for indicating the move action

* GSK:
 - gl: Handle offloads in offscreen context better
 - Fix text rendering problems with some fonts

* Wayland:
 - Tighten up some protocol version checks
 - Use the presentation time protocol
 - Fix a crash with subsurfaces
 - Improve settings portal handling

* macOS:
 - Fix up the app menu support

* Windows:
 - Fix problems with minimization
 - Fix build without fontconfig

* Debugging:
 - Add font settings in the inspector

* Demos:
 - Clean up the application demo
 - Update cursor images for the cursor demo

* Translation updates:
 Catalan
 Czech
 French
 Georgian
 Hebrew
 Persian
 Slovenian
 Turkish
 Ukrainian

Signed-off-by: Markus Volk 
---
 meta/recipes-gnome/gtk+/{gtk4_4.14.1.bb => gtk4_4.14.2.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-gnome/gtk+/{gtk4_4.14.1.bb => gtk4_4.14.2.bb} (98%)

diff --git a/meta/recipes-gnome/gtk+/gtk4_4.14.1.bb 
b/meta/recipes-gnome/gtk+/gtk4_4.14.2.bb
similarity index 98%
rename from meta/recipes-gnome/gtk+/gtk4_4.14.1.bb
rename to meta/recipes-gnome/gtk+/gtk4_4.14.2.bb
index ce733769a5..282e81f048 100644
--- a/meta/recipes-gnome/gtk+/gtk4_4.14.1.bb
+++ b/meta/recipes-gnome/gtk+/gtk4_4.14.2.bb
@@ -37,7 +37,7 @@ MAJ_VER = "${@oe.utils.trim_version("${PV}", 2)}"
 UPSTREAM_CHECK_REGEX = "gtk-(?P\d+\.(\d*[02468])+(\.\d+)+)\.tar.xz"
 
 SRC_URI = 
"http://ftp.gnome.org/pub/gnome/sources/gtk/${MAJ_VER}/gtk-${PV}.tar.xz;
-SRC_URI[sha256sum] = 
"fcefb3f132f8cc4711a9efa5b353c9ae9bb5eeff0246fa74dbc2f2f839b9e308"
+SRC_URI[sha256sum] = 
"22604cef2898a79e5f2143bb7aee2b7d1fa2eb946989a9d1338ecf9c8ae0e072"
 
 S = "${WORKDIR}/gtk-${PV}"
 
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198085): 
https://lists.openembedded.org/g/openembedded-core/message/198085
Mute This Topic: https://lists.openembedded.org/mt/105438163/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] update-rc.d: add +git to PV

2024-04-10 Thread Peter Marko via lists.openembedded.org
From: Peter Marko 

This hash is ahead of the tag, so adapt PV accordingly.

Signed-off-by: Peter Marko 
---
 meta/recipes-core/update-rc.d/update-rc.d_0.8.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/update-rc.d/update-rc.d_0.8.bb 
b/meta/recipes-core/update-rc.d/update-rc.d_0.8.bb
index ba622fe716..27723c88ef 100644
--- a/meta/recipes-core/update-rc.d/update-rc.d_0.8.bb
+++ b/meta/recipes-core/update-rc.d/update-rc.d_0.8.bb
@@ -8,6 +8,7 @@ LIC_FILES_CHKSUM = 
"file://update-rc.d;beginline=5;endline=15;md5=d40a07c27f5354
 
 SRC_URI = "git://git.yoctoproject.org/update-rc.d;branch=master;protocol=https"
 SRCREV = "b8f950105010270a768aa12245d6abf166346015"
+PV .= "+git"
 
 UPSTREAM_CHECK_COMMITS = "1"
 
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198084): 
https://lists.openembedded.org/g/openembedded-core/message/198084
Mute This Topic: https://lists.openembedded.org/mt/105437813/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 v5 1/3] image.bbclass/rootfs: archive and deploy package database

2024-04-10 Thread Johannes Schneider via lists.openembedded.org
archive the package database after the rootfs has been put together as
*rootfs-pkdbfs.tar.gz, and put it into the deploy folder.

This creates a snapshot of the package mangers state at the point in
time when all dependencies have been resolved and installed; which
could be used by "extension images" to built upon.

Signed-off-by: Johannes Schneider 
---
 meta/classes-recipe/image.bbclass | 44 +++-
 meta/classes-recipe/image_types.bbclass   |  1 +
 meta/conf/documentation.conf  |  1 +
 meta/lib/oe/package_manager/deb/rootfs.py |  1 +
 meta/lib/oe/package_manager/ipk/rootfs.py |  1 +
 meta/lib/oe/package_manager/rpm/rootfs.py |  1 +
 meta/lib/oe/rootfs.py | 20 
 meta/lib/oeqa/selftest/cases/imagefeatures.py | 51 +++
 8 files changed, 119 insertions(+), 1 deletion(-)

diff --git a/meta/classes-recipe/image.bbclass 
b/meta/classes-recipe/image.bbclass
index 28be6c6362..3ccaaa17b8 100644
--- a/meta/classes-recipe/image.bbclass
+++ b/meta/classes-recipe/image.bbclass
@@ -42,6 +42,9 @@ IMAGE_FEATURES ?= ""
 IMAGE_FEATURES[type] = "list"
 IMAGE_FEATURES[validitems] += "debug-tweaks read-only-rootfs 
read-only-rootfs-delayed-postinsts stateless-rootfs empty-root-password 
allow-empty-password allow-root-login serial-autologin-root 
post-install-logging overlayfs-etc"
 
+# Generate snapshot of the package database?
+IMAGE_GEN_PKGDBFS ?= "0"
+
 # Generate companion debugfs?
 IMAGE_GEN_DEBUGFS ?= "0"
 
@@ -131,7 +134,8 @@ def rootfs_variables(d):
  
'IMAGE_ROOTFS_MAXSIZE','IMAGE_NAME','IMAGE_LINK_NAME','IMAGE_MANIFEST','DEPLOY_DIR_IMAGE','IMAGE_FSTYPES','IMAGE_INSTALL_COMPLEMENTARY','IMAGE_LINGUAS',
 'IMAGE_LINGUAS_COMPLEMENTARY', 'IMAGE_LOCALES_ARCHIVE',
  
'MULTILIBRE_ALLOW_REP','MULTILIB_TEMP_ROOTFS','MULTILIB_VARIANTS','MULTILIBS','ALL_MULTILIB_PACKAGE_ARCHS','MULTILIB_GLOBAL_VARIANTS','BAD_RECOMMENDATIONS','NO_RECOMMENDATIONS',
  
'PACKAGE_ARCHS','PACKAGE_CLASSES','TARGET_VENDOR','TARGET_ARCH','TARGET_OS','OVERRIDES','BBEXTENDVARIANT','FEED_DEPLOYDIR_BASE_URI','INTERCEPT_DIR','USE_DEVFS',
- 'CONVERSIONTYPES', 'IMAGE_GEN_DEBUGFS', 'ROOTFS_RO_UNNEEDED', 
'IMGDEPLOYDIR', 'PACKAGE_EXCLUDE_COMPLEMENTARY', 
'REPRODUCIBLE_TIMESTAMP_ROOTFS', 'IMAGE_INSTALL_DEBUGFS']
+ 'CONVERSIONTYPES', 'IMAGE_GEN_PKGDBFS', 'IMAGE_GEN_DEBUGFS', 
'ROOTFS_RO_UNNEEDED', 'IMGDEPLOYDIR', 'PACKAGE_EXCLUDE_COMPLEMENTARY', 
'REPRODUCIBLE_TIMESTAMP_ROOTFS',
+ 'IMAGE_INSTALL_DEBUGFS']
 variables.extend(rootfs_command_variables(d))
 variables.extend(variable_depends(d))
 return " ".join(variables)
@@ -337,6 +341,17 @@ python do_image_qa_setscene () {
 }
 addtask do_image_qa_setscene
 
+def setup_pkgdbfs_variables(d):
+d.appendVar('IMAGE_ROOTFS', '-pkgdb')
+if d.getVar('IMAGE_LINK_NAME'):
+d.appendVar('IMAGE_LINK_NAME', '-pkgdb')
+d.appendVar('IMAGE_NAME','-pkgdb')
+d.setVar('IMAGE_FSTYPES', 'tar.gz')
+
+python setup_pkgdbfs () {
+setup_pkgdbfs_variables(d)
+}
+
 def setup_debugfs_variables(d):
 d.appendVar('IMAGE_ROOTFS', '-dbg')
 if d.getVar('IMAGE_LINK_NAME'):
@@ -381,6 +396,11 @@ python () {
 alltypes = d.getVar('IMAGE_FSTYPES').split()
 typedeps = {}
 
+if d.getVar('IMAGE_GEN_PKGDBFS') == "1":
+pkgdbfs_fstypes = ['tar.gz']
+for t in pkgdbfs_fstypes:
+alltypes.append("pkgdbfs_" + t)
+
 if d.getVar('IMAGE_GEN_DEBUGFS') == "1":
 debugfs_fstypes = d.getVar('IMAGE_FSTYPES_DEBUGFS').split()
 for t in debugfs_fstypes:
@@ -393,6 +413,10 @@ python () {
 basetypes[baset]= []
 if t not in basetypes[baset]:
 basetypes[baset].append(t)
+pkgdb = ""
+if t.startswith("pkgdbfs_"):
+t = t[8:]
+pkgdb = "pkgdbfs_"
 debug = ""
 if t.startswith("debugfs_"):
 t = t[8:]
@@ -401,6 +425,13 @@ python () {
 vardeps.add('IMAGE_TYPEDEP:' + t)
 if baset not in typedeps:
 typedeps[baset] = set()
+deps = [pkgdb + dep for dep in deps]
+for dep in deps:
+if dep not in alltypes:
+alltypes.append(dep)
+_add_type(dep)
+basedep = _image_base_type(dep)
+typedeps[baset].add(basedep)
 deps = [debug + dep for dep in deps]
 for dep in deps:
 if dep not in alltypes:
@@ -419,6 +450,7 @@ python () {
 
 maskedtypes = (d.getVar('IMAGE_TYPES_MASKED') or "").split()
 maskedtypes = [dbg + t for t in maskedtypes for dbg in ("", "debugfs_")]
+maskedtypes = [pkgdb + t for t in maskedtypes for pkgdb in ("", 
"pkgdbfs_")]
 
 for t in basetypes:
 vardeps = set()
@@ -430,6 +462,11 @@ python () {
 continue
 
 localdata = bb.data.createCopy(d)
+pkgdb = ""
+if t.startswith("pkgdbfs_"):
+

[OE-core] [PATCH v5 3/3] classes: add a systemd-sysext image class

2024-04-10 Thread Johannes Schneider via lists.openembedded.org
systemd-sysext can load a raw-image containing usr/ and opt/ folders
to mount them as RO overlay over the rootfs, to "extend" the systems.

This class provides the necessary changes/additions to the enclosed
file-system so that systemd-sysext accepts the extension for "merge"
into the rootfs.

With such an created image, placed into the correct folder (see [1]),
`systemd-sysext list` should be able to list the "extension" and
`systemd-sysext merge` should enable the overlay. On both commands a
preceding "SYSTEMD_LOG_LEVEL=debug" can aide in figuring out what is
amiss.

The strict name checking systemd-sysext does against the name of
extension-release.NAME file, is disabled, as there is only one such in
the resulting image. This is done to allow a user to freely rename the
resulting image file.
Note that for e.g. squashfs, the kernel needs CONFIG_SQUASHFS_XATTR=y

Link: 
https://www.freedesktop.org/software/systemd/man/latest/systemd-sysext.html
Link: 
https://0pointer.net/blog/testing-my-system-code-in-usr-without-modifying-usr.html
Signed-off-by: Johannes Schneider 
---
 meta/classes-recipe/image-sysext.bbclass | 43 
 1 file changed, 43 insertions(+)
 create mode 100644 meta/classes-recipe/image-sysext.bbclass

diff --git a/meta/classes-recipe/image-sysext.bbclass 
b/meta/classes-recipe/image-sysext.bbclass
new file mode 100644
index 00..bc3e4d52b5
--- /dev/null
+++ b/meta/classes-recipe/image-sysext.bbclass
@@ -0,0 +1,43 @@
+# SPDX-License-Identifier: MIT
+#
+# Copyright Leica Geosystems AG
+#
+
+# systemd-sysext [1] has a simple mechanism for version compatibility:
+# the extension to be loaded has to contain a
+# /usr/lib/extension-release.d/extension-release.NAME
+# with "NAME" *exactly* matching the filename of the extensions
+# raw-device filename/
+#
+# from the extension-release file the "ID" and "VERSION_ID" fields are
+# matched against the etc/os-release and the extension is only "merged"
+# if no mismatches between NAME, ID, and VERSION_ID.
+#
+# Link: 
https://www.freedesktop.org/software/systemd/man/latest/systemd-sysext.html
+
+inherit image
+
+IMAGE_NAME_SUFFIX = ".sysext"
+EXTENSION_NAME = "${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.${IMAGE_FSTYPES}"
+IMAGE_LINK_NAME:append = ".sysext"
+
+DEPENDS += " os-release"
+
+sysext_image_mangle_rootfs() {
+R=${IMAGE_ROOTFS}
+
+# pull a copy of the rootfs version information, which systemd-sysext 
matches against
+cp -av ${RECIPE_SYSROOT}/${nonarch_libdir}/os-release 
${WORKDIR}/extension-release.base
+
+echo 'EXTENSION_RELOAD_MANAGER=1' >> ${WORKDIR}/extension-release.base
+
+install -d $R${nonarch_libdir}/extension-release.d
+install -m 0644 ${WORKDIR}/extension-release.base \
+
$R${nonarch_libdir}/extension-release.d/extension-release.${EXTENSION_NAME}
+
+# disable systemd-sysext's strict name checking, so that the image file 
can be renamed, while still being 'merge'-able
+setfattr -n user.extension-release.strict -v false \
+
$R${nonarch_libdir}/extension-release.d/extension-release.${EXTENSION_NAME}
+}
+
+ROOTFS_POSTPROCESS_COMMAND += " sysext_image_mangle_rootfs; "
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198083): 
https://lists.openembedded.org/g/openembedded-core/message/198083
Mute This Topic: https://lists.openembedded.org/mt/105437787/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 v5 2/3] image.bbclass/rootfs: set and unpack package-database

2024-04-10 Thread Johannes Schneider via lists.openembedded.org
set the package-database of a "lower image" to unpack and build upon
when installing packages for the current image. This way a lean image
will be created, which only holds the packages that are not already
present in the lower image.

An image build such could then be used with overlayfs or systemd-
sysext to extend the "lower image" on demand; for development purposes
on a device running the "lower image" in RO mode for example.

A configuration could look as follows:
  some-core-image.bb
inherit image
IMAGE_GEN_PKGDBFS = "1"

  extending-image.bb
inherit image
IMAGE_BASE_PKGDB = "some-core-image"

Signed-off-by: Johannes Schneider 
---
 meta/classes-recipe/image.bbclass | 23 --
 meta/conf/documentation.conf  |  3 +-
 meta/lib/oe/package_manager/deb/rootfs.py |  2 +
 meta/lib/oe/package_manager/ipk/rootfs.py |  6 ++-
 meta/lib/oe/package_manager/rpm/rootfs.py |  7 ++-
 meta/lib/oe/rootfs.py | 18 
 meta/lib/oeqa/selftest/cases/imagefeatures.py | 46 +++
 7 files changed, 97 insertions(+), 8 deletions(-)

diff --git a/meta/classes-recipe/image.bbclass 
b/meta/classes-recipe/image.bbclass
index 3ccaaa17b8..c573c37cd8 100644
--- a/meta/classes-recipe/image.bbclass
+++ b/meta/classes-recipe/image.bbclass
@@ -42,8 +42,16 @@ IMAGE_FEATURES ?= ""
 IMAGE_FEATURES[type] = "list"
 IMAGE_FEATURES[validitems] += "debug-tweaks read-only-rootfs 
read-only-rootfs-delayed-postinsts stateless-rootfs empty-root-password 
allow-empty-password allow-root-login serial-autologin-root 
post-install-logging overlayfs-etc"
 
-# Generate snapshot of the package database?
+# Image layering:
+# a "base image" would create a snapshot of the package-database after the
+# installation of all packages into the rootfs is done. The next/other image
+# "layered on-top" of the former would then import that database and install
+# further packages; without reinstalling packages/dependencies that are already
+# installed in the layer below.
+# Set to '1' in a "base image" recipe, to preserve a snapshot of the package 
database.
 IMAGE_GEN_PKGDBFS ?= "0"
+# "PN" of a "base image", upon which the current image is to be built upon.
+IMAGE_BASE_PKGDB ?= ""
 
 # Generate companion debugfs?
 IMAGE_GEN_DEBUGFS ?= "0"
@@ -118,6 +126,15 @@ do_rootfs[depends] += " \
 "
 do_rootfs[recrdeptask] += "do_packagedata"
 
+python () {
+# make sure that the 'base image' has been queued in before this
+# image wants to unpack and build upon the formers pgkdb
+base_image = d.getVar('IMAGE_BASE_PKGDB')
+pn = d.getVar('PN')
+if base_image and base_image != pn:
+d.appendVarFlag("do_rootfs", 'depends', ' '+ base_image + 
':do_image_complete')
+}
+
 def rootfs_command_variables(d):
 return 
['ROOTFS_POSTPROCESS_COMMAND','ROOTFS_PREPROCESS_COMMAND','ROOTFS_POSTINSTALL_COMMAND','ROOTFS_POSTUNINSTALL_COMMAND','OPKG_PREPROCESS_COMMANDS','OPKG_POSTPROCESS_COMMANDS','IMAGE_POSTPROCESS_COMMAND',
 
'IMAGE_PREPROCESS_COMMAND','RPM_PREPROCESS_COMMANDS','RPM_POSTPROCESS_COMMANDS','DEB_PREPROCESS_COMMANDS','DEB_POSTPROCESS_COMMANDS']
@@ -134,8 +151,8 @@ def rootfs_variables(d):
  
'IMAGE_ROOTFS_MAXSIZE','IMAGE_NAME','IMAGE_LINK_NAME','IMAGE_MANIFEST','DEPLOY_DIR_IMAGE','IMAGE_FSTYPES','IMAGE_INSTALL_COMPLEMENTARY','IMAGE_LINGUAS',
 'IMAGE_LINGUAS_COMPLEMENTARY', 'IMAGE_LOCALES_ARCHIVE',
  
'MULTILIBRE_ALLOW_REP','MULTILIB_TEMP_ROOTFS','MULTILIB_VARIANTS','MULTILIBS','ALL_MULTILIB_PACKAGE_ARCHS','MULTILIB_GLOBAL_VARIANTS','BAD_RECOMMENDATIONS','NO_RECOMMENDATIONS',
  
'PACKAGE_ARCHS','PACKAGE_CLASSES','TARGET_VENDOR','TARGET_ARCH','TARGET_OS','OVERRIDES','BBEXTENDVARIANT','FEED_DEPLOYDIR_BASE_URI','INTERCEPT_DIR','USE_DEVFS',
- 'CONVERSIONTYPES', 'IMAGE_GEN_PKGDBFS', 'IMAGE_GEN_DEBUGFS', 
'ROOTFS_RO_UNNEEDED', 'IMGDEPLOYDIR', 'PACKAGE_EXCLUDE_COMPLEMENTARY', 
'REPRODUCIBLE_TIMESTAMP_ROOTFS',
- 'IMAGE_INSTALL_DEBUGFS']
+ 'CONVERSIONTYPES', 'IMAGE_GEN_PKGDBFS', 'IMAGE_BASE_PKGDB', 
'IMAGE_GEN_DEBUGFS', 'ROOTFS_RO_UNNEEDED', 'IMGDEPLOYDIR', 
'PACKAGE_EXCLUDE_COMPLEMENTARY',
+ 'REPRODUCIBLE_TIMESTAMP_ROOTFS', 'IMAGE_INSTALL_DEBUGFS']
 variables.extend(rootfs_command_variables(d))
 variables.extend(variable_depends(d))
 return " ".join(variables)
diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
index 36aebb59ab..9f493cfe96 100644
--- a/meta/conf/documentation.conf
+++ b/meta/conf/documentation.conf
@@ -208,6 +208,7 @@ ICECC_PATH[doc] = "The location of the icecc binary."
 ICECC_CLASS_DISABLE[doc] = "Identifies user classes that you do not want the 
Icecream distributed compile support to consider."
 ICECC_RECIPE_DISABLE[doc] = "Identifies user recipes that you do not want the 
Icecream distributed compile support to consider."
 ICECC_RECIPE_ENABLE[doc] = "Identifies user recipes that use 

[OE-core] [PATCH v5 0/3] pkg-database and systemd-sysext image

2024-04-10 Thread Johannes Schneider via lists.openembedded.org
systemd-sysext allows to overlay another image (or multiple) ontop of
a "base-image" = the current rootfs, via the use of overlayfs; to add
tools and features meant for development purposes.

To quote the documentation on systemd-sysext:
" ...addition in order to make debugging/development easier). System
extension images should not be misunderstood as a generic software
packaging framework, ..."

To build a lean image, that only holds packages that are not already
part of the base-image, a snapshot of the package-database is taken
after the installation of the base-rootfs is done, and picked up again
when collecting the rootfs of such a extension image.

with all this in place an example usage could look like this:
some-core-image.bb
  inherit core-image
  IMAGE_GEN_PKGDBFS = "1"

extending-image.bb
  inherit image-sysext
  IMAGE_FSTYPES = "squashfs"
  IMAGE_BASE_PKGDB = "some-core-image"
  # the above pointing at a package-db similar to:
  # 
build/deploy/images/$MACHINE/some-core-image-$MACHINE-20240210172305-pkgdb.rootfs.tar.gz

then on the device, running some-core-image, with the extension image placed at 
FN:
$> ln -s "$FN" /run/extensions/$(basename $FN).raw
$> systemd-sysext list
$> SYSTEMD_LOG_LEVEL=debug systemd-sysext merge

As long as the VERSION_ID of the extension image matches the os-release
in the base image, the above commands return sucessfully;
for details on the compativility check see the docs for systemd-sysext.

=

changes with v2:
rebase from 'kirkstone' onto 'master'

changes with v3:
incorporate review suggestions for simplification
add task dependency handling
add oe-selftest for the pkgdb handling
add variable documentation and
some more comments, and examples in the commit-msg

changes with v4:
rebase onto 'master' => no functional changes
fixed patchtest findings

changes with v5:
rebase onto 'master'
add '.sysext' to the deployed symlink name
sidenote on the tests and autobuilder failure: run locally they 
succeed, e.g.:
  #> oe-selftest --verbose -r 
imagefeatures.ImageFeatures.test_image_gen_pkgdbfs

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198080): 
https://lists.openembedded.org/g/openembedded-core/message/198080
Mute This Topic: https://lists.openembedded.org/mt/105437784/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] devtool: standard: throws appropriate error if source is in detached HEAD

2024-04-10 Thread Julien Stephan
If source is in detached HEAD, we get the following error when using
detvool finish command:

[...]
File "<...>/poky/scripts/lib/devtool/standard.py", line 1938, in 
_update_recipe
  bb.process.run('git checkout %s' % startbranch, cwd=srctree)
File "<...>/poky/bitbake/lib/bb/process.py", line 189, in run
  raise ExecutionError(cmd, pipe.returncode, stdout, stderr)
  bb.process.ExecutionError: Execution of 'git checkout (HEAD detached at 
9bbf87e)' failed with exit code 2:
  /bin/sh: -c: line 1: syntax error near unexpected token `('
  /bin/sh: -c: line 1: `git checkout (HEAD detached at 9bbf87e)'

Check this and throws an appropriate error in this case

Signed-off-by: Julien Stephan 

---

Changes in v2:

- update error message to avoid confusion with a potentially non
  existing "main" branch
---
 scripts/lib/devtool/standard.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py
index 2c174927ddb..51e5794a0a7 100644
--- a/scripts/lib/devtool/standard.py
+++ b/scripts/lib/devtool/standard.py
@@ -1888,6 +1888,8 @@ def _update_recipe(recipename, workspace, rd, mode, 
appendlayerdir, wildcard_ver
 for line in stdout.splitlines():
 branchname = line[2:]
 if line.startswith('* '):
+if 'HEAD' in line:
+raise DevtoolError('Detached HEAD - please check out a 
branch, e.g., "devtool"')
 startbranch = branchname
 if branchname.startswith(override_branch_prefix):
 override_branches.append(branchname)
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198079): 
https://lists.openembedded.org/g/openembedded-core/message/198079
Mute This Topic: https://lists.openembedded.org/mt/105437662/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: standard: throws appropriate error if source is in detached HEAD

2024-04-10 Thread Julien Stephan
Le mer. 10 avr. 2024 à 01:11, Peter Kjellerstedt
 a écrit :
>
> > -Original Message-
> > From: openembedded-core@lists.openembedded.org 
> >  On Behalf Of Julien Stephan
> > Sent: den 9 april 2024 09:19
> > To: openembedded-core@lists.openembedded.org
> > Cc: Julien Stephan 
> > Subject: [OE-core] [PATCH] devtool: standard: throws appropriate error if 
> > source is in detached HEAD
> >
> > If source is in detached HEAD, we get the following error when using
> > detvool finish command:
> >
> > [...]
> > File "<...>/poky/scripts/lib/devtool/standard.py", line 1938, in 
> > _update_recipe
> >   bb.process.run('git checkout %s' % startbranch, cwd=srctree)
> > File "<...>/poky/bitbake/lib/bb/process.py", line 189, in run
> >   raise ExecutionError(cmd, pipe.returncode, stdout, stderr)
> >   bb.process.ExecutionError: Execution of 'git checkout (HEAD detached at 
> > 9bbf87e)' failed with exit code 2:
> >   /bin/sh: -c: line 1: syntax error near unexpected token `('
> >   /bin/sh: -c: line 1: `git checkout (HEAD detached at 9bbf87e)'
> >
> > Check this and throws an appropriate error in this case
> >
> > Signed-off-by: Julien Stephan 
> > ---
> >  scripts/lib/devtool/standard.py | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/scripts/lib/devtool/standard.py
> > b/scripts/lib/devtool/standard.py
> > index 7972b4f8223..7f32b0999a4 100644
> > --- a/scripts/lib/devtool/standard.py
> > +++ b/scripts/lib/devtool/standard.py
> > @@ -1885,6 +1885,8 @@ def _update_recipe(recipename, workspace, rd, mode, 
> > appendlayerdir, wildcard_ver
> >  for line in stdout.splitlines():
> >  branchname = line[2:]
> >  if line.startswith('* '):
> > +if 'HEAD' in line:
> > +raise DevtoolError('Detached HEAD - please check out 
> > the main branch in source tree first')
>
> Using "main branch" here might lead to confusion since it is unclear
> if the error is referring to an actual "main" branch, which not all
> repositories have, or to whatever branch is primarily used.

Hi Peter,

I used the same error message as the one already existing a few lines below.
To be honest I read it as "the primary branch" not "the branch named
main" but I understand it can be confusing.

> Additionally, when using devtool, the branch is typically "devtool".
>
> I would change it to:
>
> raise DevtoolError('Detached HEAD - please check out a 
> branch, e.g., "devtool"')

makes sense, let me send a v2.
Thank you!

>
> >  startbranch = branchname
> >  if branchname.startswith(override_branch_prefix):
> >  override_branches.append(branchname)
> > --
> > 2.44.0
>
> //Peter
>

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