[OE-core] [PATCH] rootfs-postcommands: update systemd_create_users

[Vyacheslav Yurkov]

Process all systemd-sysusers configuration files in order to create
users/groups at build time. systemd-sysusers would try to create them at
run-time, but for read-only rootfs that's not possible and results in
warnings from different services:

systemd-udevd[166]: /lib/udev/rules.d/50-udev-default.rules:39 Unknown group 
'render', ignoring
systemd-udevd[166]: /lib/udev/rules.d/50-udev-default.rules:40 Unknown group 
'render', ignoring
systemd-udevd[166]: /lib/udev/rules.d/50-udev-default.rules:42 Unknown group 
'sgx', ignoring

Signed-off-by: Vyacheslav Yurkov 
---
 meta/classes/rootfs-postcommands.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/rootfs-postcommands.bbclass 
b/meta/classes/rootfs-postcommands.bbclass
index 7fe9e3d8c8..a3f96ef7ed 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -62,7 +62,7 @@ python () {
 }
 
 systemd_create_users () {
-   for conffile in ${IMAGE_ROOTFS}/usr/lib/sysusers.d/systemd.conf 
${IMAGE_ROOTFS}/usr/lib/sysusers.d/systemd-remote.conf; do
+   for conffile in ${IMAGE_ROOTFS}/usr/lib/sysusers.d/*.conf; do
[ -e $conffile ] || continue
grep -v "^#" $conffile | sed -e '/^$/d' | while read type name 
id comment; do
if [ "$type" = "u" ]; then
-- 
2.28.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158527): 
https://lists.openembedded.org/g/openembedded-core/message/158527
Mute This Topic: https://lists.openembedded.org/mt/87178494/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] classes/meson: Add optional rust definitions

[Joshua Watt]

Adds the rust tools to the cross and native files if present so that
projects that use both rust and meson can build

Signed-off-by: Joshua Watt 
---

V2: Fixes so that cross-compiling works correctly by passing --target
and RUSTFLAGS


 meta/classes/meson.bbclass | 11 ++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/meta/classes/meson.bbclass b/meta/classes/meson.bbclass
index 4ba70de3dc..a7981e481f 100644
--- a/meta/classes/meson.bbclass
+++ b/meta/classes/meson.bbclass
@@ -36,8 +36,15 @@ MESON_CROSS_FILE = ""
 MESON_CROSS_FILE:class-target = "--cross-file ${WORKDIR}/meson.cross"
 MESON_CROSS_FILE:class-nativesdk = "--cross-file ${WORKDIR}/meson.cross"
 
+def rust_tool(d, target_var):
+rustc = d.getVar('RUSTC')
+if not rustc:
+return ""
+cmd = [rustc, "--target", d.getVar(target_var)] + 
d.getVar("RUSTFLAGS").split()
+return "rust = %s" % repr(cmd)
+
 addtask write_config before do_configure
-do_write_config[vardeps] += "CC CXX LD AR NM STRIP READELF CFLAGS CXXFLAGS 
LDFLAGS"
+do_write_config[vardeps] += "CC CXX LD AR NM STRIP READELF CFLAGS CXXFLAGS 
LDFLAGS RUSTC RUSTFLAGS"
 do_write_config() {
 # This needs to be Py to split the args into single-element lists
 cat >${WORKDIR}/meson.cross <
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158526): 
https://lists.openembedded.org/g/openembedded-core/message/158526
Mute This Topic: https://lists.openembedded.org/mt/87178130/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] pls comment on weston non-PAM patch (link inside)

[Tom Hochstein]

Thanks Alex. @Otavio pushed the 
hardest for the use case and I don't really have anything to add. Given that 
Pekka has suggested that there is a better approach without weston-launch, it 
seems that removing the patch is fine. In the worst case we can backport the 
patch to meta-freescale until we have a new solution.

Tom

From: Alexander Kanavin 
Sent: Friday, November 19, 2021 11:06 AM
To: OE-core ; Tom Hochstein 
; Denys Dmytriyenko 
Cc: ppaala...@gmail.com
Subject: Re: pls comment on weston non-PAM patch (link inside)

Hello guys,

reminding you to speak up in the patch submission; if you don't, upstream will 
close it without merging, and I will start working towards removing the patch 
from Yocto as well.

Alex

On Fri, 12 Nov 2021 at 10:48, Alexander Kanavin 
mailto:alex.kana...@gmail.com>> wrote:
Hello folks,

I sent the weston non-PAM patch upstream and they have raised questions about 
its validity, can you please comment on it?
https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/725

Thanks,
Alex




-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158525): 
https://lists.openembedded.org/g/openembedded-core/message/158525
Mute This Topic: https://lists.openembedded.org/mt/87003161/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] pls comment on weston non-PAM patch (link inside)

[Alexander Kanavin]

Hello guys,

reminding you to speak up in the patch submission; if you don't, upstream
will close it without merging, and I will start working towards removing
the patch from Yocto as well.

Alex

On Fri, 12 Nov 2021 at 10:48, Alexander Kanavin 
wrote:

> Hello folks,
>
> I sent the weston non-PAM patch upstream and they have raised questions
> about its validity, can you please comment on it?
> https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/725
>
> Thanks,
> Alex
>
>
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158524): 
https://lists.openembedded.org/g/openembedded-core/message/158524
Mute This Topic: https://lists.openembedded.org/mt/87003161/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] dev-manual: how to purge duplicate sstate cache files

[Michael Opdenacker]

Signed-off-by: Michael Opdenacker 
---
 documentation/dev-manual/common-tasks.rst | 25 ++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/documentation/dev-manual/common-tasks.rst 
b/documentation/dev-manual/common-tasks.rst
index 3eead147a3..37612c84b7 100644
--- a/documentation/dev-manual/common-tasks.rst
+++ b/documentation/dev-manual/common-tasks.rst
@@ -6242,8 +6242,11 @@ Changing the listed common targets is as easy as editing 
your version of
 ``conf-notes.txt`` in your custom template configuration directory and
 making sure you have ``TEMPLATECONF`` set to your directory.
 
+Conserving Disk Space
+=
+
 Conserving Disk Space During Builds
-===
+---
 
 To help conserve disk space during builds, you can add the following
 statement to your project's ``local.conf`` configuration file found in
@@ -6257,6 +6260,26 @@ building a recipe once the recipe is built. For more 
information on
 :ref:`rm_work ` class in the
 Yocto Project Reference Manual.
 
+Purging Duplicate Shared State Cache Files
+---
+
+After multiple build iterations, the Shared State (sstate) cache can contain
+duplicate cache files for a given package, while only the most recent one
+is likely to be reusable. The following command purges all but the
+newest sstate cache file for each package::
+
+   sstate-cache-management.sh --remove-duplicated 
--cache-dir=build/sstate-cache
+
+This command will ask you to confirm the deletions it identifies.
+
+Note::
+
+   The duplicated sstate cache files of one package must have the same
+   architecture, which means that sstate cache files with multiple
+   architectures are not considered as duplicate.
+
+Run ``sstate-cache-management.sh`` for more details about this script.
+
 Working with Packages
 =
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158523): 
https://lists.openembedded.org/g/openembedded-core/message/158523
Mute This Topic: https://lists.openembedded.org/mt/87171189/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] ncurses: disable big core only for target

[Alibek Omarov]

Building some terminal emulators(foot, for example) may require
tic to be built with enabled big core.

It's highly likely that tic will be used as native or in SDK,
so we can lift memory restriction.

Signed-off-by: Alibek Omarov 
---
 meta/recipes-core/ncurses/ncurses.inc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/ncurses/ncurses.inc 
b/meta/recipes-core/ncurses/ncurses.inc
index a0ecd8a80b8..793056c8901 100644
--- a/meta/recipes-core/ncurses/ncurses.inc
+++ b/meta/recipes-core/ncurses/ncurses.inc
@@ -38,7 +38,6 @@ base_libdir:class-native = "${libdir}"
 # Display corruption occurs on 64 bit hosts without these settings
 # This was derrived from the upstream debian ncurses which uses
 # these settings for 32 and 64 bit hosts.
-EXCONFIG_ARGS = ""
 EXCONFIG_ARGS:class-native = " \
--disable-lp64 \
--with-chtype='long' \
@@ -47,6 +46,8 @@ EXCONFIG_ARGS:class-nativesdk = " \
--disable-lp64 \
--with-chtype='long' \
--with-mmask-t='long'"
+# Assume target system has little memory but native has enough
+EXCONFIG_ARGS = "--disable-big-core"
 
 PACKAGES_DYNAMIC = "^${PN}-lib.*"
 
@@ -80,7 +81,6 @@ ncurses_configure() {

--with-termpath='${sysconfdir}/termcap:${datadir}/misc/termcap${EX_TERMCAP}' \

--with-terminfo-dirs='${sysconfdir}/terminfo:${datadir}/terminfo${EX_TERMINFO}' 
\
--with-shared \
-   --disable-big-core \
--program-prefix= \
--with-ticlib \
--with-termlib=${EX_TERMLIB} \
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158522): 
https://lists.openembedded.org/g/openembedded-core/message/158522
Mute This Topic: https://lists.openembedded.org/mt/87169500/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] oeqa: fix warnings for append operators combined with +=

[Yi Zhao]

Signed-off-by: Yi Zhao 
---
 meta/lib/oeqa/runtime/cases/ksample.py| 2 +-
 meta/lib/oeqa/selftest/cases/imagefeatures.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oeqa/runtime/cases/ksample.py 
b/meta/lib/oeqa/runtime/cases/ksample.py
index 4d12d1d07d..c69e3fe4ac 100644
--- a/meta/lib/oeqa/runtime/cases/ksample.py
+++ b/meta/lib/oeqa/runtime/cases/ksample.py
@@ -10,7 +10,7 @@ from oeqa.core.decorator.depends import OETestDepends
 from oeqa.core.decorator.data import skipIfNotFeature
 
 # need some kernel fragments
-# echo "KERNEL_FEATURES:append += \" 
features\/kernel\-sample\/kernel\-sample.scc\"" >> local.conf
+# echo "KERNEL_FEATURES:append = \" 
features\/kernel\-sample\/kernel\-sample.scc\"" >> local.conf
 class KSample(OERuntimeTestCase):
 def cmd_and_check(self, cmd='', match_string=''):
 status, output = self.target.run(cmd)
diff --git a/meta/lib/oeqa/selftest/cases/imagefeatures.py 
b/meta/lib/oeqa/selftest/cases/imagefeatures.py
index 12902add94..18f37c6d7d 100644
--- a/meta/lib/oeqa/selftest/cases/imagefeatures.py
+++ b/meta/lib/oeqa/selftest/cases/imagefeatures.py
@@ -229,7 +229,7 @@ USERADD_GID_TABLES += "files/static-group"
 def test_no_busybox_base_utils(self):
 config = """
 # Enable wayland
-DISTRO_FEATURES:append += "pam opengl wayland"
+DISTRO_FEATURES:append = " pam opengl wayland"
 
 # Switch to systemd
 DISTRO_FEATURES += "systemd"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158521): 
https://lists.openembedded.org/g/openembedded-core/message/158521
Mute This Topic: https://lists.openembedded.org/mt/87168837/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 02/36] groff: mark patch as non-upstreamable

[Alexander Kanavin]

On Fri, 19 Nov 2021 at 13:29, Richard Purdie <
richard.pur...@linuxfoundation.org> wrote:

> On Fri, 2021-11-19 at 13:08 +0100, Alexander Kanavin wrote:
> > I checked; the patches are the same, except the upstream one misses one
> file,
> > which is fixed in a different upstream patch. I'd say we can keep things
> as
> > they are.
>
> Can we mark as a backport of those two commits then?
>

Right, I will add the upstream references to the patch.

Alex

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158520): 
https://lists.openembedded.org/g/openembedded-core/message/158520
Mute This Topic: https://lists.openembedded.org/mt/87121973/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 02/36] groff: mark patch as non-upstreamable

[Richard Purdie]

On Fri, 2021-11-19 at 13:08 +0100, Alexander Kanavin wrote:
> I checked; the patches are the same, except the upstream one misses one file,
> which is fixed in a different upstream patch. I'd say we can keep things as
> they are.

Can we mark as a backport of those two commits then?

Cheers,

Richard




-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158519): 
https://lists.openembedded.org/g/openembedded-core/message/158519
Mute This Topic: https://lists.openembedded.org/mt/87121973/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 02/36] groff: mark patch as non-upstreamable

[Alexander Kanavin]

I checked; the patches are the same, except the upstream one misses one
file, which is fixed in a different upstream patch. I'd say we can keep
things as they are.

Alex

On Fri, 19 Nov 2021 at 12:34, Alexander Kanavin via lists.openembedded.org
 wrote:

> This is the upstream fix:
>
> https://git.savannah.gnu.org/cgit/groff.git/commit/?id=979f3f4266151c7681a68a40d2c4913842a7271d
>
> I can backport that, rebase the existing patch on top and see what's left,
> sure.
>
> Alex
> On Fri, 19 Nov 2021 at 12:15, Richard Purdie <
> richard.pur...@linuxfoundation.org> wrote:
>
>> On Wed, 2021-11-17 at 16:34 +0100, Alexander Kanavin wrote:
>> > Signed-off-by: Alexander Kanavin 
>> > ---
>> >  meta/recipes-extended/groff/files/0001-Include-config.h.patch | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git
>> a/meta/recipes-extended/groff/files/0001-Include-config.h.patch
>> b/meta/recipes-extended/groff/files/0001-Include-config.h.patch
>> > index 46065bc513..34fca1eb2f 100644
>> > --- a/meta/recipes-extended/groff/files/0001-Include-config.h.patch
>> > +++ b/meta/recipes-extended/groff/files/0001-Include-config.h.patch
>> > @@ -20,7 +20,7 @@ In file included from
>> TOPDIR/build/tmp/work/aarch64-yoe-linux-musl/groff/1.22.4-
>> >  We delete eqn.cpp and qen.hpp in do_configure
>> >  to ensure they're regenerated and deterministic.
>> >
>> > -Upstream-Status: Pending
>> > +Upstream-Status: Inappropriate [issue fixed upstream with a similar
>> patch]
>> >  Signed-off-by: Khem Raj 
>> >  ---
>> >   src/libs/libgroff/assert.cpp  |   4 +
>>
>>
>> Should we replace this with the upstream patch? That way we'd have a clean
>> backport which would help at upgrade time?
>>
>> Is the upstream patch exactly functionally equivalent?
>>
>> Sometimes it can be good to test the upstream version so we know come
>> upgrade
>> time we're ok and we don't need to ask upstream to make any further
>> tweaks.
>>
>
>
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158518): 
https://lists.openembedded.org/g/openembedded-core/message/158518
Mute This Topic: https://lists.openembedded.org/mt/87121973/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 7/7] systemd: add nativesdk

[Luca Bocassi]

On Fri, 2021-11-19 at 12:39 +0100, Alexander Kanavin wrote:
> Once again, I do not want this accepted in oe-core, not until there's
> equivalent native functionality.
> 
> Alex

As already mentioned, that's completely unrelated, and I do not have
the time nor the interest to invest weeks of my time working on that.
But arbitrarily rejecting patches is your prerogative of course, so
I'll just keep these changes out-of-tree then.

> On Fri, 19 Nov 2021 at 12:35, Luca Bocassi 
> wrote:
> > From: Luca Boccassi 
> > 
> > Allows using tools like systemd-analyze at build time and in
> > CI systems
> > 
> > Signed-off-by: Luca Boccassi 
> > ---
> >  meta/recipes-core/systemd/systemd-conf_1.0.bb    | 2 ++
> >  meta/recipes-core/systemd/systemd-serialgetty.bb | 2 ++
> >  meta/recipes-core/systemd/systemd_249.5.bb       | 2 ++
> >  3 files changed, 6 insertions(+)
> > 
> > diff --git a/meta/recipes-core/systemd/systemd-conf_1.0.bb
> > b/meta/recipes-core/systemd/systemd-conf_1.0.bb
> > index 61ce7939d3..84de4d17cf 100644
> > --- a/meta/recipes-core/systemd/systemd-conf_1.0.bb
> > +++ b/meta/recipes-core/systemd/systemd-conf_1.0.bb
> > @@ -41,3 +41,5 @@ FILES:${PN} = "\
> >      ${systemd_unitdir}/system.conf.d/ \
> >      ${systemd_unitdir}/network/ \
> >  "
> > +
> > +BBCLASSEXTEND = "nativesdk"
> > diff --git a/meta/recipes-core/systemd/systemd-serialgetty.bb
> > b/meta/recipes-core/systemd/systemd-serialgetty.bb
> > index 7ca951cdbd..f877e6ad18 100644
> > --- a/meta/recipes-core/systemd/systemd-serialgetty.bb
> > +++ b/meta/recipes-core/systemd/systemd-serialgetty.bb
> > @@ -50,3 +50,5 @@ FILES:${PN} =
> > "${systemd_system_unitdir}/*.service ${sysconfdir}"
> >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> > 
> >  ALLOW_EMPTY:${PN} = "1"
> > +
> > +BBCLASSEXTEND = "nativesdk"
> > diff --git a/meta/recipes-core/systemd/systemd_249.5.bb
> > b/meta/recipes-core/systemd/systemd_249.5.bb
> > index 2928a85c93..445a50b1e3 100644
> > --- a/meta/recipes-core/systemd/systemd_249.5.bb
> > +++ b/meta/recipes-core/systemd/systemd_249.5.bb
> > @@ -773,6 +773,8 @@ ALTERNATIVE_TARGET[runlevel] =
> > "${base_bindir}/systemctl"
> >  ALTERNATIVE_LINK_NAME[runlevel] = "${base_sbindir}/runlevel"
> >  ALTERNATIVE_PRIORITY[runlevel] ?= "300"
> > 
> > +BBCLASSEXTEND = "nativesdk"
> > +
> >  # Nothing picks up /var in the nativesdk case
> >  do_install_append_class-nativesdk () {
> >         rm -rf ${D}/var



signature.asc
Description: This is a digitally signed message part

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158517): 
https://lists.openembedded.org/g/openembedded-core/message/158517
Mute This Topic: https://lists.openembedded.org/mt/87165499/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 7/7] systemd: add nativesdk

[Alexander Kanavin]

Once again, I do not want this accepted in oe-core, not until there's
equivalent native functionality.

Alex

On Fri, 19 Nov 2021 at 12:35, Luca Bocassi  wrote:

> From: Luca Boccassi 
>
> Allows using tools like systemd-analyze at build time and in
> CI systems
>
> Signed-off-by: Luca Boccassi 
> ---
>  meta/recipes-core/systemd/systemd-conf_1.0.bb| 2 ++
>  meta/recipes-core/systemd/systemd-serialgetty.bb | 2 ++
>  meta/recipes-core/systemd/systemd_249.5.bb   | 2 ++
>  3 files changed, 6 insertions(+)
>
> diff --git a/meta/recipes-core/systemd/systemd-conf_1.0.bb
> b/meta/recipes-core/systemd/systemd-conf_1.0.bb
> index 61ce7939d3..84de4d17cf 100644
> --- a/meta/recipes-core/systemd/systemd-conf_1.0.bb
> +++ b/meta/recipes-core/systemd/systemd-conf_1.0.bb
> @@ -41,3 +41,5 @@ FILES:${PN} = "\
>  ${systemd_unitdir}/system.conf.d/ \
>  ${systemd_unitdir}/network/ \
>  "
> +
> +BBCLASSEXTEND = "nativesdk"
> diff --git a/meta/recipes-core/systemd/systemd-serialgetty.bb
> b/meta/recipes-core/systemd/systemd-serialgetty.bb
> index 7ca951cdbd..f877e6ad18 100644
> --- a/meta/recipes-core/systemd/systemd-serialgetty.bb
> +++ b/meta/recipes-core/systemd/systemd-serialgetty.bb
> @@ -50,3 +50,5 @@ FILES:${PN} = "${systemd_system_unitdir}/*.service
> ${sysconfdir}"
>  PACKAGE_ARCH = "${MACHINE_ARCH}"
>
>  ALLOW_EMPTY:${PN} = "1"
> +
> +BBCLASSEXTEND = "nativesdk"
> diff --git a/meta/recipes-core/systemd/systemd_249.5.bb
> b/meta/recipes-core/systemd/systemd_249.5.bb
> index 2928a85c93..445a50b1e3 100644
> --- a/meta/recipes-core/systemd/systemd_249.5.bb
> +++ b/meta/recipes-core/systemd/systemd_249.5.bb
> @@ -773,6 +773,8 @@ ALTERNATIVE_TARGET[runlevel] =
> "${base_bindir}/systemctl"
>  ALTERNATIVE_LINK_NAME[runlevel] = "${base_sbindir}/runlevel"
>  ALTERNATIVE_PRIORITY[runlevel] ?= "300"
>
> +BBCLASSEXTEND = "nativesdk"
> +
>  # Nothing picks up /var in the nativesdk case
>  do_install_append_class-nativesdk () {
> rm -rf ${D}/var
> --
> 2.30.2
>
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158516): 
https://lists.openembedded.org/g/openembedded-core/message/158516
Mute This Topic: https://lists.openembedded.org/mt/87165499/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 5/8] volatile-binds: add nativesdk

[Luca Bocassi]

On Wed, 2021-11-17 at 17:42 +, Luca Boccassi wrote:
> On Wed, 2021-11-17 at 17:40 +, Richard Purdie wrote:
> > On Wed, 2021-11-17 at 12:31 +, Luca Bocassi wrote:
> > > From: Luca Boccassi 
> > > 
> > > Signed-off-by: Luca Boccassi 
> > > ---
> > >  meta/recipes-core/volatile-binds/volatile-binds.bb | 2 ++
> > >  1 file changed, 2 insertions(+)
> > > 
> > > diff --git a/meta/recipes-core/volatile-binds/volatile-binds.bb 
> > > b/meta/recipes-core/volatile-binds/volatile-binds.bb
> > > index 66e28f4fc9..0f27353031 100644
> > > --- a/meta/recipes-core/volatile-binds/volatile-binds.bb
> > > +++ b/meta/recipes-core/volatile-binds/volatile-binds.bb
> > > @@ -84,3 +84,5 @@ do_install[dirs] = "${WORKDIR}"
> > >  do_install_append_class-nativesdk () {
> > >  rm -rf ${D}/var
> > >  }
> > > +
> > > +BBCLASSEXTEND = "nativesdk"
> > 
> > Is there any useful component left in volatile-binds in the SDK case? I 
> > don't
> > think we should be extending this as the recipe makes no sense. Rather 
> > nativesdk
> > variants probably shouldn't be depending upon it?
> 
> It's pulled in automatically. I don't know enough about yocto to be
> able to tell whether it's intentional and needed or not.

Looks like it's working without, so changed in v2 to remove the
dependency for nativesdk instead.

-- 
Kind regards,
Luca Boccassi


signature.asc
Description: This is a digitally signed message part

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158515): 
https://lists.openembedded.org/g/openembedded-core/message/158515
Mute This Topic: https://lists.openembedded.org/mt/87118021/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/8] systemd: skip chown when building for nativesdk

[Luca Bocassi]

On Wed, 2021-11-17 at 18:32 +, Luca Boccassi wrote:
> On Wed, 2021-11-17 at 18:14 +, Peter Kjellerstedt wrote:
> > > -Original Message-
> > > From: openembedded-core@lists.openembedded.org
> > >  On Behalf Of Luca
> > > Bocassi
> > > Sent: den 17 november 2021 13:32
> > > To: openembedded-core@lists.openembedded.org
> > > Cc: paul.eggle...@microsoft.com
> > > Subject: [OE-core] [PATCH 1/8] systemd: skip chown when building
> > > for nativesdk
> > > 
> > > From: Luca Boccassi 
> > > 
> > > The useradd class is a no-op in the nativesdk case, so chown will
> > > fail.
> > > Skip them.
> > > 
> > > Signed-off-by: Luca Boccassi 
> > > ---
> > >  meta/recipes-core/systemd/systemd_249.5.bb | 10 --
> > >  1 file changed, 8 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/meta/recipes-core/systemd/systemd_249.5.bb
> > > b/meta/recipes-core/systemd/systemd_249.5.bb
> > > index 8bdc0ca028..2f019a4b1f 100644
> > > --- a/meta/recipes-core/systemd/systemd_249.5.bb
> > > +++ b/meta/recipes-core/systemd/systemd_249.5.bb
> > > @@ -275,7 +275,10 @@ do_install() {
> > >   # which is expected to be empty.
> > >   rm -rf ${D}${localstatedir}/log
> > >   else
> > > - chown root:systemd-journal
> > > ${D}${localstatedir}/log/journal
> > > + # The useradd class is a no-op in the nativesdk
> > > case, so chown will fail
> > > + if [ "${PN}" != "nativesdk-systemd" ]; then
> > 
> > If you change that if statement here and below to:
> > 
> > if [ "${PN}" = "${BPN}" ]; then
> > 
> > then it will apply just as well if anyone introduces systemd-native.
> 
> Thanks, will test that and send a v2 tomorrow if it is all green.

This works, thanks, applied in v2.

-- 
Kind regards,
Luca Boccassi


signature.asc
Description: This is a digitally signed message part

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158514): 
https://lists.openembedded.org/g/openembedded-core/message/158514
Mute This Topic: https://lists.openembedded.org/mt/87118014/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 7/7] systemd: add nativesdk

[Luca Bocassi]

From: Luca Boccassi 

Allows using tools like systemd-analyze at build time and in
CI systems

Signed-off-by: Luca Boccassi 
---
 meta/recipes-core/systemd/systemd-conf_1.0.bb| 2 ++
 meta/recipes-core/systemd/systemd-serialgetty.bb | 2 ++
 meta/recipes-core/systemd/systemd_249.5.bb   | 2 ++
 3 files changed, 6 insertions(+)

diff --git a/meta/recipes-core/systemd/systemd-conf_1.0.bb 
b/meta/recipes-core/systemd/systemd-conf_1.0.bb
index 61ce7939d3..84de4d17cf 100644
--- a/meta/recipes-core/systemd/systemd-conf_1.0.bb
+++ b/meta/recipes-core/systemd/systemd-conf_1.0.bb
@@ -41,3 +41,5 @@ FILES:${PN} = "\
 ${systemd_unitdir}/system.conf.d/ \
 ${systemd_unitdir}/network/ \
 "
+
+BBCLASSEXTEND = "nativesdk"
diff --git a/meta/recipes-core/systemd/systemd-serialgetty.bb 
b/meta/recipes-core/systemd/systemd-serialgetty.bb
index 7ca951cdbd..f877e6ad18 100644
--- a/meta/recipes-core/systemd/systemd-serialgetty.bb
+++ b/meta/recipes-core/systemd/systemd-serialgetty.bb
@@ -50,3 +50,5 @@ FILES:${PN} = "${systemd_system_unitdir}/*.service 
${sysconfdir}"
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 
 ALLOW_EMPTY:${PN} = "1"
+
+BBCLASSEXTEND = "nativesdk"
diff --git a/meta/recipes-core/systemd/systemd_249.5.bb 
b/meta/recipes-core/systemd/systemd_249.5.bb
index 2928a85c93..445a50b1e3 100644
--- a/meta/recipes-core/systemd/systemd_249.5.bb
+++ b/meta/recipes-core/systemd/systemd_249.5.bb
@@ -773,6 +773,8 @@ ALTERNATIVE_TARGET[runlevel] = "${base_bindir}/systemctl"
 ALTERNATIVE_LINK_NAME[runlevel] = "${base_sbindir}/runlevel"
 ALTERNATIVE_PRIORITY[runlevel] ?= "300"
 
+BBCLASSEXTEND = "nativesdk"
+
 # Nothing picks up /var in the nativesdk case
 do_install_append_class-nativesdk () {
rm -rf ${D}/var
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158513): 
https://lists.openembedded.org/g/openembedded-core/message/158513
Mute This Topic: https://lists.openembedded.org/mt/87165499/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 6/7] os-release: add nativesdk

[Luca Bocassi]

From: Luca Boccassi 

Required to build systemd tools

Signed-off-by: Luca Boccassi 
---
 meta/recipes-core/os-release/os-release.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-core/os-release/os-release.bb 
b/meta/recipes-core/os-release/os-release.bb
index 8847fe30c3..2f2aa66c46 100644
--- a/meta/recipes-core/os-release/os-release.bb
+++ b/meta/recipes-core/os-release/os-release.bb
@@ -51,3 +51,5 @@ do_install () {
 }
 
 FILES:${PN} += "${nonarch_libdir}/os-release"
+
+BBCLASSEXTEND = "nativesdk"
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158512): 
https://lists.openembedded.org/g/openembedded-core/message/158512
Mute This Topic: https://lists.openembedded.org/mt/87165498/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 5/7] kbd: add nativesdk

[Luca Bocassi]

From: Luca Boccassi 

Required to build systemd tools

Signed-off-by: Luca Boccassi 
---
 meta/recipes-core/kbd/kbd_2.4.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/kbd/kbd_2.4.0.bb 
b/meta/recipes-core/kbd/kbd_2.4.0.bb
index 7cb45eff77..7efc7af312 100644
--- a/meta/recipes-core/kbd/kbd_2.4.0.bb
+++ b/meta/recipes-core/kbd/kbd_2.4.0.bb
@@ -43,4 +43,4 @@ ALTERNATIVE:${PN} = "chvt deallocvt fgconsole openvt showkey \
  ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 
'vlock','', d)}"
 ALTERNATIVE_PRIORITY = "100"
 
-BBCLASSEXTEND = "native"
+BBCLASSEXTEND = "native nativesdk"
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158511): 
https://lists.openembedded.org/g/openembedded-core/message/158511
Mute This Topic: https://lists.openembedded.org/mt/87165495/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/7] systemd: remove volatile-binds RDEPENDS for nativesdk

[Luca Bocassi]

From: Luca Boccassi 

Not needed for SDK binaries

Signed-off-by: Luca Boccassi 
---
v2: remove dependency instead of adding nativesdk to volatile-binds

 meta/recipes-core/systemd/systemd_249.5.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/systemd/systemd_249.5.bb 
b/meta/recipes-core/systemd/systemd_249.5.bb
index 9993036aac..2928a85c93 100644
--- a/meta/recipes-core/systemd/systemd_249.5.bb
+++ b/meta/recipes-core/systemd/systemd_249.5.bb
@@ -644,6 +644,7 @@ FILES:${PN}-dev += "${base_libdir}/security/*.la 
${datadir}/dbus-1/interfaces/ $
 RDEPENDS:${PN} += "kmod dbus util-linux-mount util-linux-umount udev (= 
${EXTENDPKGV}) systemd-udev-rules util-linux-agetty util-linux-fsck"
 RDEPENDS:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 
'serial-getty-generator', '', 'systemd-serialgetty', d)}"
 RDEPENDS:${PN} += "volatile-binds"
+RDEPENDS_${PN}_remove_class-nativesdk = "volatile-binds"
 
 RRECOMMENDS:${PN} += "systemd-extra-utils \
   udev-hwdb \
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158510): 
https://lists.openembedded.org/g/openembedded-core/message/158510
Mute This Topic: https://lists.openembedded.org/mt/87165494/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/7] systemd: remove /var in nativesdk builds

[Luca Bocassi]

From: Luca Boccassi 

Nothing picks it up, so it fails QA checks

Signed-off-by: Luca Boccassi 
---
 meta/recipes-core/systemd/systemd_249.5.bb | 5 +
 1 file changed, 5 insertions(+)

diff --git a/meta/recipes-core/systemd/systemd_249.5.bb 
b/meta/recipes-core/systemd/systemd_249.5.bb
index ec9c350297..9993036aac 100644
--- a/meta/recipes-core/systemd/systemd_249.5.bb
+++ b/meta/recipes-core/systemd/systemd_249.5.bb
@@ -772,6 +772,11 @@ ALTERNATIVE_TARGET[runlevel] = "${base_bindir}/systemctl"
 ALTERNATIVE_LINK_NAME[runlevel] = "${base_sbindir}/runlevel"
 ALTERNATIVE_PRIORITY[runlevel] ?= "300"
 
+# Nothing picks up /var in the nativesdk case
+do_install_append_class-nativesdk () {
+   rm -rf ${D}/var
+}
+
 pkg_postinst:${PN}:libc-glibc () {
if [ "${PN}" != "${BPN}" ]; then
exit 0
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158509): 
https://lists.openembedded.org/g/openembedded-core/message/158509
Mute This Topic: https://lists.openembedded.org/mt/87165493/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/7] systemd: skip postinst in nativesdk builds

[Luca Bocassi]

From: Luca Boccassi 

They will fail, as there's nowhere to run them

Signed-off-by: Luca Boccassi 
---
v2: use "${PN}" = "${BPN}" as suggested by reviewers

 meta/recipes-core/systemd/systemd_249.5.bb | 12 
 1 file changed, 12 insertions(+)

diff --git a/meta/recipes-core/systemd/systemd_249.5.bb 
b/meta/recipes-core/systemd/systemd_249.5.bb
index 2df2de0cf3..ec9c350297 100644
--- a/meta/recipes-core/systemd/systemd_249.5.bb
+++ b/meta/recipes-core/systemd/systemd_249.5.bb
@@ -773,12 +773,18 @@ ALTERNATIVE_LINK_NAME[runlevel] = 
"${base_sbindir}/runlevel"
 ALTERNATIVE_PRIORITY[runlevel] ?= "300"
 
 pkg_postinst:${PN}:libc-glibc () {
+   if [ "${PN}" != "${BPN}" ]; then
+   exit 0
+   fi
sed -e '/^hosts:/s/\s*\//' \
-e 's/\(^hosts:.*\)\(\\)\(.*\)\(\\)\(.*\)/\1\2 
myhostname \3\4\5/' \
-i $D${sysconfdir}/nsswitch.conf
 }
 
 pkg_prerm:${PN}:libc-glibc () {
+   if [ "${PN}" != "${BPN}" ]; then
+   exit 0
+   fi
sed -e '/^hosts:/s/\s*\//' \
-e '/^hosts:/s/\s*myhostname//' \
-i $D${sysconfdir}/nsswitch.conf
@@ -786,6 +792,9 @@ pkg_prerm:${PN}:libc-glibc () {
 
 PACKAGE_WRITE_DEPS += "qemu-native"
 pkg_postinst:udev-hwdb () {
+   if [ "${PN}" != "${BPN}" ]; then
+   exit 0
+   fi
if test -n "$D"; then
$INTERCEPT_DIR/postinst_intercept update_udev_hwdb ${PKG} 
mlprefix=${MLPREFIX} binprefix=${MLPREFIX} rootlibexecdir="${rootlibexecdir}" 
PREFERRED_PROVIDER_udev="${PREFERRED_PROVIDER_udev}"
else
@@ -794,5 +803,8 @@ pkg_postinst:udev-hwdb () {
 }
 
 pkg_prerm:udev-hwdb () {
+   if [ "${PN}" != "${BPN}" ]; then
+   exit 0
+   fi
rm -f $D${sysconfdir}/udev/hwdb.bin
 }
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158508): 
https://lists.openembedded.org/g/openembedded-core/message/158508
Mute This Topic: https://lists.openembedded.org/mt/87165492/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/7] systemd: skip chown when building for nativesdk

[Luca Bocassi]

From: Luca Boccassi 

The useradd class is a no-op in the nativesdk case, so chown will fail.
Skip them.

Signed-off-by: Luca Boccassi 
---
v2: use "${PN}" = "${BPN}" as suggested by reviewers

 meta/recipes-core/systemd/systemd_249.5.bb | 10 --
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/systemd/systemd_249.5.bb 
b/meta/recipes-core/systemd/systemd_249.5.bb
index 8bdc0ca028..2df2de0cf3 100644
--- a/meta/recipes-core/systemd/systemd_249.5.bb
+++ b/meta/recipes-core/systemd/systemd_249.5.bb
@@ -275,7 +275,10 @@ do_install() {
# which is expected to be empty.
rm -rf ${D}${localstatedir}/log
else
-   chown root:systemd-journal ${D}${localstatedir}/log/journal
+   # The useradd class is a no-op in the nativesdk case, so chown 
will fail
+   if [ "${PN}" = "${BPN}" ]; then
+   chown root:systemd-journal 
${D}${localstatedir}/log/journal
+   fi
 
# journal-remote creates this at start
rm -rf ${D}${localstatedir}/log/journal/remote
@@ -319,7 +322,10 @@ do_install() {
if ${@bb.utils.contains('PACKAGECONFIG', 'polkit', 'true', 'false', 
d)}; then
if [ -d ${D}${datadir}/polkit-1/rules.d ]; then
chmod 700 ${D}${datadir}/polkit-1/rules.d
-   chown polkitd:root ${D}${datadir}/polkit-1/rules.d
+   # The useradd class is a no-op in the nativesdk case, 
so chown will fail
+   if [ "${PN}" = "${BPN}" ]; then
+   chown polkitd:root 
${D}${datadir}/polkit-1/rules.d
+   fi
fi
fi
 
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158507): 
https://lists.openembedded.org/g/openembedded-core/message/158507
Mute This Topic: https://lists.openembedded.org/mt/87165491/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 02/36] groff: mark patch as non-upstreamable

[Alexander Kanavin]

This is the upstream fix:
https://git.savannah.gnu.org/cgit/groff.git/commit/?id=979f3f4266151c7681a68a40d2c4913842a7271d

I can backport that, rebase the existing patch on top and see what's left,
sure.

Alex
On Fri, 19 Nov 2021 at 12:15, Richard Purdie <
richard.pur...@linuxfoundation.org> wrote:

> On Wed, 2021-11-17 at 16:34 +0100, Alexander Kanavin wrote:
> > Signed-off-by: Alexander Kanavin 
> > ---
> >  meta/recipes-extended/groff/files/0001-Include-config.h.patch | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git
> a/meta/recipes-extended/groff/files/0001-Include-config.h.patch
> b/meta/recipes-extended/groff/files/0001-Include-config.h.patch
> > index 46065bc513..34fca1eb2f 100644
> > --- a/meta/recipes-extended/groff/files/0001-Include-config.h.patch
> > +++ b/meta/recipes-extended/groff/files/0001-Include-config.h.patch
> > @@ -20,7 +20,7 @@ In file included from
> TOPDIR/build/tmp/work/aarch64-yoe-linux-musl/groff/1.22.4-
> >  We delete eqn.cpp and qen.hpp in do_configure
> >  to ensure they're regenerated and deterministic.
> >
> > -Upstream-Status: Pending
> > +Upstream-Status: Inappropriate [issue fixed upstream with a similar
> patch]
> >  Signed-off-by: Khem Raj 
> >  ---
> >   src/libs/libgroff/assert.cpp  |   4 +
>
>
> Should we replace this with the upstream patch? That way we'd have a clean
> backport which would help at upgrade time?
>
> Is the upstream patch exactly functionally equivalent?
>
> Sometimes it can be good to test the upstream version so we know come
> upgrade
> time we're ok and we don't need to ask upstream to make any further tweaks.
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158506): 
https://lists.openembedded.org/g/openembedded-core/message/158506
Mute This Topic: https://lists.openembedded.org/mt/87121973/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 0/8] systemd: allow building for nativesdk

[Luca Bocassi]

On Fri, 2021-11-19 at 09:43 +0800, ChenQi wrote:
> Hi Luca,
> 
> If the real useful part is only about systemd-analyze in case of
> nativesdk/native, I'd suggest adding systemd-analyze-native or
> nativesdk-systemd-analyze instead of extending the current systemd
> recipe.
> This is because systemd has a whole bunch of dependencies which
> basically make no sense in case of native/nativesdk. Looking at the
> src/analyze/meson.build file, I guess it could be built
> independently.
> 
> Regards,
> Qi

Hi,

Building individual binaries like that is not supported, so it cannot
work.

> On 11/17/21 8:31 PM, Luca Bocassi wrote:
> From: Luca Boccassi 
> 
> systemd-analyze in the next release will gain the ability to
> run the 'security' and 'verify' verbs offline, on local units.
> This is very useful at build time and/or in CI systems to detect
> errors and issues before changes are deployed, as a static
> analyzer tool.
> This series enables building the systemd recipe for the nativesdk
> targets, so that systemd-analyze can be included in the SDK and
> called by users/build systems/CI systems.
> 
> Luca Boccassi (8):
>   systemd: skip chown when building for nativesdk
>   systemd: skip postinst in nativesdk builds
>   systemd: remove /var in nativesdk builds
>   volatile-binds: remove /var in nativesdk builds
>   volatile-binds: add nativesdk
>   kbd: add nativesdk
>   os-release: add nativesdk
>   systemd: add nativesdk
> 
>  meta/recipes-core/kbd/kbd_2.4.0.bb|  2 +-
>  meta/recipes-core/os-release/os-release.bb|  2 ++
>  meta/recipes-core/systemd/systemd-conf_1.0.bb |  2 ++
>  .../systemd/systemd-serialgetty.bb|  2 ++
>  meta/recipes-core/systemd/systemd_249.5.bb| 29
> +--
>  .../volatile-binds/volatile-binds.bb  |  7 +
>  6 files changed, 41 insertions(+), 3 deletions(-)
> 
>  
>  
> 
> 
> 
>  



signature.asc
Description: This is a digitally signed message part

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158505): 
https://lists.openembedded.org/g/openembedded-core/message/158505
Mute This Topic: https://lists.openembedded.org/mt/87118013/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 02/36] groff: mark patch as non-upstreamable

[Richard Purdie]

On Wed, 2021-11-17 at 16:34 +0100, Alexander Kanavin wrote:
> Signed-off-by: Alexander Kanavin 
> ---
>  meta/recipes-extended/groff/files/0001-Include-config.h.patch | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-extended/groff/files/0001-Include-config.h.patch 
> b/meta/recipes-extended/groff/files/0001-Include-config.h.patch
> index 46065bc513..34fca1eb2f 100644
> --- a/meta/recipes-extended/groff/files/0001-Include-config.h.patch
> +++ b/meta/recipes-extended/groff/files/0001-Include-config.h.patch
> @@ -20,7 +20,7 @@ In file included from 
> TOPDIR/build/tmp/work/aarch64-yoe-linux-musl/groff/1.22.4-
>  We delete eqn.cpp and qen.hpp in do_configure
>  to ensure they're regenerated and deterministic.
>  
> -Upstream-Status: Pending
> +Upstream-Status: Inappropriate [issue fixed upstream with a similar patch]
>  Signed-off-by: Khem Raj 
>  ---
>   src/libs/libgroff/assert.cpp  |   4 +


Should we replace this with the upstream patch? That way we'd have a clean
backport which would help at upgrade time? 

Is the upstream patch exactly functionally equivalent?

Sometimes it can be good to test the upstream version so we know come upgrade
time we're ok and we don't need to ask upstream to make any further tweaks.

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158504): 
https://lists.openembedded.org/g/openembedded-core/message/158504
Mute This Topic: https://lists.openembedded.org/mt/87121973/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] gcc: Tweak Upstream-Status formatting

[Richard Purdie]

Signed-off-by: Richard Purdie 
---
 meta/recipes-devtools/gcc/gcc/0001-CVE-2021-35465.patch | 2 +-
 meta/recipes-devtools/gcc/gcc/0002-CVE-2021-35465.patch | 2 +-
 meta/recipes-devtools/gcc/gcc/0003-CVE-2021-35465.patch | 2 +-
 meta/recipes-devtools/gcc/gcc/0004-CVE-2021-35465.patch | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-devtools/gcc/gcc/0001-CVE-2021-35465.patch 
b/meta/recipes-devtools/gcc/gcc/0001-CVE-2021-35465.patch
index 6b1d4e3fce4..e4aee10e370 100644
--- a/meta/recipes-devtools/gcc/gcc/0001-CVE-2021-35465.patch
+++ b/meta/recipes-devtools/gcc/gcc/0001-CVE-2021-35465.patch
@@ -20,7 +20,7 @@ gcc:
the command line.
 
 CVE: CVE-2021-35465
-Upstream-Status: 
Backport[https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=3929bca9ca95de9d35e82ae8828b188029e3eb70]
+Upstream-Status: Backport 
[https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=3929bca9ca95de9d35e82ae8828b188029e3eb70]
 Signed-off-by: Pgowda 
 
 ---
diff --git a/meta/recipes-devtools/gcc/gcc/0002-CVE-2021-35465.patch 
b/meta/recipes-devtools/gcc/gcc/0002-CVE-2021-35465.patch
index 98841e6d7c1..e09818fecf7 100644
--- a/meta/recipes-devtools/gcc/gcc/0002-CVE-2021-35465.patch
+++ b/meta/recipes-devtools/gcc/gcc/0002-CVE-2021-35465.patch
@@ -15,7 +15,7 @@ libgcc:
Add vlldm erratum work-around.
 
 CVE: CVE-2021-35465
-Upstream-Status: 
Backport[https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=574e7950bd6b34e9e2cacce18c802b45505d1d0a]
+Upstream-Status: Backport 
[https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=574e7950bd6b34e9e2cacce18c802b45505d1d0a]
 Signed-off-by: Pgowda 
 
 ---
diff --git a/meta/recipes-devtools/gcc/gcc/0003-CVE-2021-35465.patch 
b/meta/recipes-devtools/gcc/gcc/0003-CVE-2021-35465.patch
index d87be198669..c7a7c76bf87 100644
--- a/meta/recipes-devtools/gcc/gcc/0003-CVE-2021-35465.patch
+++ b/meta/recipes-devtools/gcc/gcc/0003-CVE-2021-35465.patch
@@ -16,7 +16,7 @@ gcc:
use when erratum mitigation is needed.
 
 CVE: CVE-2021-35465
-Upstream-Status: 
Backport[https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=30461cf8dba3d3adb15a125e4da48800eb2b9b8f]
+Upstream-Status: Backport 
[https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=30461cf8dba3d3adb15a125e4da48800eb2b9b8f]
 Signed-off-by: Pgowda 
 
 ---
diff --git a/meta/recipes-devtools/gcc/gcc/0004-CVE-2021-35465.patch 
b/meta/recipes-devtools/gcc/gcc/0004-CVE-2021-35465.patch
index 12dfe682fa7..9dd6a313c2d 100644
--- a/meta/recipes-devtools/gcc/gcc/0004-CVE-2021-35465.patch
+++ b/meta/recipes-devtools/gcc/gcc/0004-CVE-2021-35465.patch
@@ -17,7 +17,7 @@ gcc/testsuite:
* gcc.target/arm/cmse/mainline/8_1m/softfp/cmse-8a.c: Likewise.
 
 CVE: CVE-2021-35465
-Upstream-Status: 
Backport[https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=809330ab8450261e05919b472783bf15e4b000f7]
+Upstream-Status: Backport 
[https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=809330ab8450261e05919b472783bf15e4b000f7]
 Signed-off-by: Pgowda 
 
 ---
-- 
2.32.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158503): 
https://lists.openembedded.org/g/openembedded-core/message/158503
Mute This Topic: https://lists.openembedded.org/mt/87165128/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] buildhistory.bbclass: fix regression from FILES_INFO changes

[Richard Purdie]

On Tue, 2021-11-16 at 17:06 -0800, S. Lockwood-Childs wrote:
> Started getting a stack trace during packagedata task for a
> recipe that built fine in hardknott
> 
>  *** 0002:buildhistory_emit_pkghistory(d)
>  0003:
> File: 'meta/classes/buildhistory.bbclass', lineno: 313, function: 
> buildhistory_emit_pkghistory
>  0309:pkginfo.filevars[filevar] = localdata.getVar(filevar) 
> or ""
>  0310:
>  0311:# Gather information about packaged files
>  0312:val = localdata.getVar('FILES_INFO') or ''
>  *** 0313:dictval = json.loads(val)
>  0314:filelist = list(dictval.keys())
>  0315:filelist.sort()
>  0316:pkginfo.filelist = " ".join([shlex.quote(x) for x in 
> filelist])
>  ...
>  Exception: json.decoder.JSONDecodeError: Expecting value: line 1 column 1 
> (char 0)
> 
> which turns out to mean FILES_INFO is undefined for the main package built by
> this recipe (FILES_INFO lookup for other packages such as -dev seemed to work 
> fine).
> 
> Tracked the regression down to this commit:
> package/scripts: Fix FILES_INFO handling
> OE-Core rev: a1190903e0a61a12c9854c96af918ae8d12c6327
> 
> If buildhistory.bbclass uses the same per-package syntax to read
> FILES_INFO as package.bbclass uses now when setting it, then
> this regression goes away.
> 
> Signed-off-by: S. Lockwood-Childs 
> ---
>  meta/classes/buildhistory.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes/buildhistory.bbclass 
> b/meta/classes/buildhistory.bbclass
> index 7c44fec2d1..99300ab431 100644
> --- a/meta/classes/buildhistory.bbclass
> +++ b/meta/classes/buildhistory.bbclass
> @@ -309,7 +309,7 @@ python buildhistory_emit_pkghistory() {
>  pkginfo.filevars[filevar] = localdata.getVar(filevar) or ""
>  
>  # Gather information about packaged files
> -val = localdata.getVar('FILES_INFO') or ''
> +val = localdata.getVar('FILES_INFO:%s' % pkg) or ''
>  dictval = json.loads(val)
>  filelist = list(dictval.keys())
>  filelist.sort()

Which revision of the project are you seeing that on? This shouldn't be needed
since a number of lines above there it says:

localdata.setVar('OVERRIDES', d.getVar("OVERRIDES", False) + ":" + pkg)

i.e. pkg is in OVERRIDES and therefore FILES_INFO should be FILES_INFO:pkg if
that is set.

I think this is masking some other issue and isn't the correct fix.

Cheers,

Richard





-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158502): 
https://lists.openembedded.org/g/openembedded-core/message/158502
Mute This Topic: https://lists.openembedded.org/mt/87110164/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] scripts/lnr: remove

[Ross Burton]

On Wed, 17 Nov 2021 at 18:24, Quentin Schulz  wrote:
> We could have a simple wrapper instead which would be a simple shell script:
>
> #!/bin/sh
>
> ln --relative --symbolic $1 $2
>
> To not require a change in layers?
> Not sure it's that important considering that one could do a simple sed 
> across all layers? Anyway just thought about that so shared it.

The point was requiring the change in layers so we're using standard
tools and not pointless scripts.  I sent patches for some larger
layers already so the fallout shouldn't be *too* bad.

Ross

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158501): 
https://lists.openembedded.org/g/openembedded-core/message/158501
Mute This Topic: https://lists.openembedded.org/mt/87124801/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 34/36] busybox: drop 0001-Use-CC-when-linking-instead-of-LD-and-use-CFLAGS-and.patch

[Andre McCurdy]

On Fri, Nov 19, 2021 at 12:17 AM Alexander Kanavin
 wrote:
>
> On Fri, 19 Nov 2021 at 00:14, Andre McCurdy  wrote:
>>
>> > So everything that is supposed to be there is actually present, and the 
>> > patch is indeed unneeded.
>>
>> Looks good. If you have the full strace logs for both cases then it
>> might still be useful to check which (if any) command lines are
>> changed by removing the patch, but if not then removing the whole
>> patch does seem OK.
>
> There is no tooling to compare strace files, they have process ids, and 
> probably other unrelated files that needs to be stripped out or matched.

I guess looking at the two logs in a graphical tool such as meld would
highlight if command line options completely disappear, even with the
noise of differences in process IDs etc.

> Just to reiterate: this is for the master branch, and I do insist on "if AB 
> is green then it works as intended" - if issues are found later, then the 
> right way to deal with them is to improve the AB tests, not ask maintainers 
> do even more manual work.

Yes, and I'm agreeing with you!

> Can you help me review the remaining 400 "upstream-status: pending" patches?

No. I made the decision some time ago to stop contributing to OE. Not
planning on restarting now.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#158500): 
https://lists.openembedded.org/g/openembedded-core/message/158500
Mute This Topic: https://lists.openembedded.org/mt/87122011/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 34/36] busybox: drop 0001-Use-CC-when-linking-instead-of-LD-and-use-CFLAGS-and.patch

[Alexander Kanavin]

On Fri, 19 Nov 2021 at 00:14, Andre McCurdy  wrote:

> > So everything that is supposed to be there is actually present, and the
> patch is indeed unneeded.
>
> Looks good. If you have the full strace logs for both cases then it
> might still be useful to check which (if any) command lines are
> changed by removing the patch, but if not then removing the whole
> patch does seem OK.
>

There is no tooling to compare strace files, they have process ids, and
probably other unrelated files that needs to be stripped out or matched.

Just to reiterate: this is for the master branch, and I do insist on "if AB
is green then it works as intended" - if issues are found later, then the
right way to deal with them is to improve the AB tests, not ask maintainers
do even more manual work. Can you help me review the remaining 400
"upstream-status: pending" patches?

Alex

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