Re: [OE-core] [PATCH v2 8/8] gstreamer1.0-plugins-bad: remove meson option for plugins with dependencies always present
Hi Richard, Thanks for overview of the merge process. Jose Richard Purdie escreveu no dia terça, 15/12/2020 à(s) 23:02: > Hi Jose, > > On Fri, 2020-12-11 at 21:13 +, Jose Quaresma wrote: > > I have think about this patch and probably is better to drop it. > > It is now on master-next > > > > Without this we have meson options for all the gstreamer plugins that > > have external dependencies. > > > > Probably its better to have it explicity enabled than don't have this > > options at all and falback to meson auto discovery dependencies. > > FWIW, I agree. I put things in master-next to run through the > autobuilder in parallel with other review so just because something > makes it there, it doesn't mean its definitely going to be merged. I've > dropped this one, thanks for highlighting! > > Cheers, > > Richard > > -- best regards, José Quaresma -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145810): https://lists.openembedded.org/g/openembedded-core/message/145810 Mute This Topic: https://lists.openembedded.org/mt/78889254/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] ltp: fix rpc build error
Add support to tirpc tests build.
Export incdir and libdir to have the right path.
Signed-off-by: Kory Maincent
---
meta/recipes-extended/ltp/ltp_20200930.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-extended/ltp/ltp_20200930.bb
b/meta/recipes-extended/ltp/ltp_20200930.bb
index 7acf15b360..9b17d68f67 100644
--- a/meta/recipes-extended/ltp/ltp_20200930.bb
+++ b/meta/recipes-extended/ltp/ltp_20200930.bb
@@ -14,7 +14,7 @@ LIC_FILES_CHKSUM = "\
file://utils/benchmark/kernbench-0.42/COPYING;md5=94d55d512a9ba36caa9b7df079bae19f
\
"
-DEPENDS = "attr libaio libcap acl openssl zip-native"
+DEPENDS = "attr libaio libcap acl openssl zip-native libtirpc"
DEPENDS_append_libc-musl = " fts "
EXTRA_OEMAKE_append_libc-musl = " LIBC=musl "
EXTRA_OECONF_append_libc-musl = " LIBS=-lfts "
@@ -44,12 +44,12 @@ TARGET_CC_ARCH += "${LDFLAGS}"
export prefix = "/opt/${PN}"
export exec_prefix = "/opt/${PN}"
+export libdir = "/usr/lib"
+export incdir = "/usr/include"
PACKAGECONFIG[numa] = "--with-numa, --without-numa, numactl,"
EXTRA_AUTORECONF += "-I ${S}/testcases/realtime/m4"
EXTRA_OECONF = " --with-realtime-testsuite --with-open-posix-testsuite "
-# ltp network/rpc test cases ftbfs when libtirpc is found
-EXTRA_OECONF += " --without-tirpc "
do_install(){
install -d ${D}${prefix}/
--
2.17.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145811):
https://lists.openembedded.org/g/openembedded-core/message/145811
Mute This Topic: https://lists.openembedded.org/mt/79033325/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] ltp: fix rpc build error
On 17.12.20 10:55, Kory Maincent wrote:
Signed-off-by: Kory Maincent
---
meta/recipes-extended/ltp/ltp_20200930.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-extended/ltp/ltp_20200930.bb
b/meta/recipes-extended/ltp/ltp_20200930.bb
index 7acf15b360..9b17d68f67 100644
--- a/meta/recipes-extended/ltp/ltp_20200930.bb
+++ b/meta/recipes-extended/ltp/ltp_20200930.bb
@@ -14,7 +14,7 @@ LIC_FILES_CHKSUM = "\
file://utils/benchmark/kernbench-0.42/COPYING;md5=94d55d512a9ba36caa9b7df079bae19f
\
"
-DEPENDS = "attr libaio libcap acl openssl zip-native"
+DEPENDS = "attr libaio libcap acl openssl zip-native libtirpc"
Can we have that as a PACKAGECONFIG entry, please? otherwise I think it
will change the behavior of the existing setups.
Also the dependency has a `PROVIDES = "virtual/librpc"`, so I think
depending on virtual/librpc should be the safest way
DEPENDS_append_libc-musl = " fts "
EXTRA_OEMAKE_append_libc-musl = " LIBC=musl "
EXTRA_OECONF_append_libc-musl = " LIBS=-lfts "
@@ -44,12 +44,12 @@ TARGET_CC_ARCH += "${LDFLAGS}"
export prefix = "/opt/${PN}"
export exec_prefix = "/opt/${PN}"
+export libdir = "/usr/lib"
+export incdir = "/usr/include"
Those should be ${libdir} and ${incdir} if I'm not mistaken
PACKAGECONFIG[numa] = "--with-numa, --without-numa, numactl,"
EXTRA_AUTORECONF += "-I ${S}/testcases/realtime/m4"
EXTRA_OECONF = " --with-realtime-testsuite --with-open-posix-testsuite "
-# ltp network/rpc test cases ftbfs when libtirpc is found
-EXTRA_OECONF += " --without-tirpc "
Should be part of a PACKAGECONFIG entry, with `--without-tirpc` being
the default
do_install(){
install -d ${D}${prefix}/
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145812):
https://lists.openembedded.org/g/openembedded-core/message/145812
Mute This Topic: https://lists.openembedded.org/mt/79033325/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] ltp: fix rpc build error
Hello Konrad,
On Thu, 17 Dec 2020 11:03:39 +0100
Konrad Weihmann wrote:
> On 17.12.20 10:55, Kory Maincent wrote:
> > Signed-off-by: Kory Maincent
> > ---
> > meta/recipes-extended/ltp/ltp_20200930.bb | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/meta/recipes-extended/ltp/ltp_20200930.bb
> > b/meta/recipes-extended/ltp/ltp_20200930.bb index 7acf15b360..9b17d68f67
> > 100644 --- a/meta/recipes-extended/ltp/ltp_20200930.bb
> > +++ b/meta/recipes-extended/ltp/ltp_20200930.bb
> > @@ -14,7 +14,7 @@ LIC_FILES_CHKSUM = "\
> >
> > file://utils/benchmark/kernbench-0.42/COPYING;md5=94d55d512a9ba36caa9b7df079bae19f
> > \ "
> >
> > -DEPENDS = "attr libaio libcap acl openssl zip-native"
> > +DEPENDS = "attr libaio libcap acl openssl zip-native libtirpc"
>
> Can we have that as a PACKAGECONFIG entry, please? otherwise I think it
> will change the behavior of the existing setups.
I assumed the tirpc was disable because they did not manage to build with
it "# ltp network/rpc test cases ftbfs when libtirpc is found".
But if you prefer I can add it to PACKAGECONFIG.
> Also the dependency has a `PROVIDES = "virtual/librpc"`, so I think
> depending on virtual/librpc should be the safest way
Ok
>
> > DEPENDS_append_libc-musl = " fts "
> > EXTRA_OEMAKE_append_libc-musl = " LIBC=musl "
> > EXTRA_OECONF_append_libc-musl = " LIBS=-lfts "
> > @@ -44,12 +44,12 @@ TARGET_CC_ARCH += "${LDFLAGS}"
> >
> > export prefix = "/opt/${PN}"
> > export exec_prefix = "/opt/${PN}"
> > +export libdir = "/usr/lib"
> > +export incdir = "/usr/include"
>
> Those should be ${libdir} and ${incdir} if I'm not mistaken
The exporting of ${prefix} change the value of ${libdir} and ${incdir}
therefore I set them up again. If I don't set it, pkgconfig can not find
libtirpc.
>
> >
> > PACKAGECONFIG[numa] = "--with-numa, --without-numa, numactl,"
>
>
> > EXTRA_AUTORECONF += "-I ${S}/testcases/realtime/m4"
> > EXTRA_OECONF = " --with-realtime-testsuite --with-open-posix-testsuite "
> > -# ltp network/rpc test cases ftbfs when libtirpc is found
> > -EXTRA_OECONF += " --without-tirpc "
>
> Should be part of a PACKAGECONFIG entry, with `--without-tirpc` being
> the default
Ok,
Köry
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145813):
https://lists.openembedded.org/g/openembedded-core/message/145813
Mute This Topic: https://lists.openembedded.org/mt/79033325/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] ltp: fix rpc build error
On 17.12.20 11:25, Köry Maincent wrote:
Hello Konrad,
On Thu, 17 Dec 2020 11:03:39 +0100
Konrad Weihmann wrote:
On 17.12.20 10:55, Kory Maincent wrote:
Signed-off-by: Kory Maincent
---
meta/recipes-extended/ltp/ltp_20200930.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-extended/ltp/ltp_20200930.bb
b/meta/recipes-extended/ltp/ltp_20200930.bb index 7acf15b360..9b17d68f67
100644 --- a/meta/recipes-extended/ltp/ltp_20200930.bb
+++ b/meta/recipes-extended/ltp/ltp_20200930.bb
@@ -14,7 +14,7 @@ LIC_FILES_CHKSUM = "\
file://utils/benchmark/kernbench-0.42/COPYING;md5=94d55d512a9ba36caa9b7df079bae19f
\ "
-DEPENDS = "attr libaio libcap acl openssl zip-native"
+DEPENDS = "attr libaio libcap acl openssl zip-native libtirpc"
Can we have that as a PACKAGECONFIG entry, please? otherwise I think it
will change the behavior of the existing setups.
I assumed the tirpc was disable because they did not manage to build with
it "# ltp network/rpc test cases ftbfs when libtirpc is found".
But if you prefer I can add it to PACKAGECONFIG.
Also the dependency has a `PROVIDES = "virtual/librpc"`, so I think
depending on virtual/librpc should be the safest way
Ok
DEPENDS_append_libc-musl = " fts "
EXTRA_OEMAKE_append_libc-musl = " LIBC=musl "
EXTRA_OECONF_append_libc-musl = " LIBS=-lfts "
@@ -44,12 +44,12 @@ TARGET_CC_ARCH += "${LDFLAGS}"
export prefix = "/opt/${PN}"
export exec_prefix = "/opt/${PN}"
+export libdir = "/usr/lib"
+export incdir = "/usr/include"
Those should be ${libdir} and ${incdir} if I'm not mistaken
The exporting of ${prefix} change the value of ${libdir} and ${incdir}
therefore I set them up again. If I don't set it, pkgconfig can not find
libtirpc.
I see, then please just export ${libdir} and ${incdir} before the prefix
export and it should be fine
PACKAGECONFIG[numa] = "--with-numa, --without-numa, numactl,"
EXTRA_AUTORECONF += "-I ${S}/testcases/realtime/m4"
EXTRA_OECONF = " --with-realtime-testsuite --with-open-posix-testsuite "
-# ltp network/rpc test cases ftbfs when libtirpc is found
-EXTRA_OECONF += " --without-tirpc "
Should be part of a PACKAGECONFIG entry, with `--without-tirpc` being
the default
Ok,
Köry
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145814):
https://lists.openembedded.org/g/openembedded-core/message/145814
Mute This Topic: https://lists.openembedded.org/mt/79033325/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] ltp: fix rpc build error
On 17.12.20 11:47, Konrad Weihmann wrote:
On 17.12.20 11:25, Köry Maincent wrote:
Hello Konrad,
On Thu, 17 Dec 2020 11:03:39 +0100
Konrad Weihmann wrote:
On 17.12.20 10:55, Kory Maincent wrote:
Signed-off-by: Kory Maincent
---
meta/recipes-extended/ltp/ltp_20200930.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-extended/ltp/ltp_20200930.bb
b/meta/recipes-extended/ltp/ltp_20200930.bb index
7acf15b360..9b17d68f67
100644 --- a/meta/recipes-extended/ltp/ltp_20200930.bb
+++ b/meta/recipes-extended/ltp/ltp_20200930.bb
@@ -14,7 +14,7 @@ LIC_FILES_CHKSUM = "\
file://utils/benchmark/kernbench-0.42/COPYING;md5=94d55d512a9ba36caa9b7df079bae19f
\ "
-DEPENDS = "attr libaio libcap acl openssl zip-native"
+DEPENDS = "attr libaio libcap acl openssl zip-native libtirpc"
Can we have that as a PACKAGECONFIG entry, please? otherwise I think it
will change the behavior of the existing setups.
I assumed the tirpc was disable because they did not manage to build with
it "# ltp network/rpc test cases ftbfs when libtirpc is found".
But if you prefer I can add it to PACKAGECONFIG.
Also the dependency has a `PROVIDES = "virtual/librpc"`, so I think
depending on virtual/librpc should be the safest way
Ok
DEPENDS_append_libc-musl = " fts "
EXTRA_OEMAKE_append_libc-musl = " LIBC=musl "
EXTRA_OECONF_append_libc-musl = " LIBS=-lfts "
@@ -44,12 +44,12 @@ TARGET_CC_ARCH += "${LDFLAGS}"
export prefix = "/opt/${PN}"
export exec_prefix = "/opt/${PN}"
+export libdir = "/usr/lib"
+export incdir = "/usr/include"
Those should be ${libdir} and ${incdir} if I'm not mistaken
The exporting of ${prefix} change the value of ${libdir} and ${incdir}
therefore I set them up again. If I don't set it, pkgconfig can not find
libtirpc.
I see, then please just export ${libdir} and ${incdir} before the prefix
export and it should be fine
for instance that here
export libdir := "${exec_prefix}/${baselib}"
export incdir := "${exec_prefix}/include"
export prefix = "/opt/${PN}"
worked for me
PACKAGECONFIG[numa] = "--with-numa, --without-numa, numactl,"
EXTRA_AUTORECONF += "-I ${S}/testcases/realtime/m4"
EXTRA_OECONF = " --with-realtime-testsuite
--with-open-posix-testsuite "
-# ltp network/rpc test cases ftbfs when libtirpc is found
-EXTRA_OECONF += " --without-tirpc "
Should be part of a PACKAGECONFIG entry, with `--without-tirpc` being
the default
Ok,
Köry
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145815):
https://lists.openembedded.org/g/openembedded-core/message/145815
Mute This Topic: https://lists.openembedded.org/mt/79033325/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] u-boot: set SYSROOT_DIRS to /boot and ${sysconfdir}
Since do_install is shipping files to /boot and ${sysconfdir}, we can set
SYSROOT_DIRS to these dirs and allow recipes which depend on u-boot to use its
installed artifacts from RECIPE_SYSROOT instead of DEPLOY_DIR_IMAGE.
Signed-off-by: Diego Sueiro
---
meta/recipes-bsp/u-boot/u-boot.inc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/meta/recipes-bsp/u-boot/u-boot.inc
b/meta/recipes-bsp/u-boot/u-boot.inc
index 251178db33..6af0766571 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -266,6 +266,8 @@ FILES_${PN}-env = " \
"
FILES_${PN} = "/boot ${datadir}"
+SYSROOT_DIRS = "/boot ${sysconfdir}"
+
RDEPENDS_${PN} += "${PN}-env"
do_deploy () {
--
2.17.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145816):
https://lists.openembedded.org/g/openembedded-core/message/145816
Mute This Topic: https://lists.openembedded.org/mt/79034125/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] ltp: fix rpc build error
Add support to tirpc tests build.
Export incdir and libdir to have the right path.
Signed-off-by: Kory Maincent
---
Change since v1:
- Use PACKAGECONFIG to enable rpc tests
- Improve ${libdir} and ${incdir} export
meta/recipes-extended/ltp/ltp_20200930.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-extended/ltp/ltp_20200930.bb
b/meta/recipes-extended/ltp/ltp_20200930.bb
index 7acf15b360..9da5d90353 100644
--- a/meta/recipes-extended/ltp/ltp_20200930.bb
+++ b/meta/recipes-extended/ltp/ltp_20200930.bb
@@ -42,14 +42,14 @@ inherit autotools-brokensep pkgconfig
TARGET_CC_ARCH += "${LDFLAGS}"
+export libdir := "${exec_prefix}/${baselib}"
+export incdir := "${exec_prefix}/include"
export prefix = "/opt/${PN}"
-export exec_prefix = "/opt/${PN}"
PACKAGECONFIG[numa] = "--with-numa, --without-numa, numactl,"
+PACKAGECONFIG[rpc] = ", --without-tirpc, virtual/librpc,"
EXTRA_AUTORECONF += "-I ${S}/testcases/realtime/m4"
EXTRA_OECONF = " --with-realtime-testsuite --with-open-posix-testsuite "
-# ltp network/rpc test cases ftbfs when libtirpc is found
-EXTRA_OECONF += " --without-tirpc "
do_install(){
install -d ${D}${prefix}/
--
2.17.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145817):
https://lists.openembedded.org/g/openembedded-core/message/145817
Mute This Topic: https://lists.openembedded.org/mt/79036634/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] ltp: fix rpc build error
> -Original Message-
> From: openembedded-core@lists.openembedded.org c...@lists.openembedded.org> On Behalf Of Kory Maincent
> Sent: den 17 december 2020 15:08
> To: openembedded-core@lists.openembedded.org
> Cc: thomas.petazz...@bootlin.com
> Subject: [OE-core] [PATCH v2] ltp: fix rpc build error
>
> Add support to tirpc tests build.
> Export incdir and libdir to have the right path.
>
> Signed-off-by: Kory Maincent
> ---
>
> Change since v1:
> - Use PACKAGECONFIG to enable rpc tests
> - Improve ${libdir} and ${incdir} export
>
>
> meta/recipes-extended/ltp/ltp_20200930.bb | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/meta/recipes-extended/ltp/ltp_20200930.bb b/meta/recipes-
> extended/ltp/ltp_20200930.bb
> index 7acf15b360..9da5d90353 100644
> --- a/meta/recipes-extended/ltp/ltp_20200930.bb
> +++ b/meta/recipes-extended/ltp/ltp_20200930.bb
> @@ -42,14 +42,14 @@ inherit autotools-brokensep pkgconfig
>
> TARGET_CC_ARCH += "${LDFLAGS}"
>
> +export libdir := "${exec_prefix}/${baselib}"
> +export incdir := "${exec_prefix}/include"
This should be a little bit better:
export libdir := "${libdir}"
export incdir := "${incdir}"
> export prefix = "/opt/${PN}"
> -export exec_prefix = "/opt/${PN}"
>
> PACKAGECONFIG[numa] = "--with-numa, --without-numa, numactl,"
> +PACKAGECONFIG[rpc] = ", --without-tirpc, virtual/librpc,"
I assume there is a --with-tirpc as well, so this should be:
PACKAGECONFIG[rpc] = "--with-tirpc, --without-tirpc, virtual/librpc,"
> EXTRA_AUTORECONF += "-I ${S}/testcases/realtime/m4"
> EXTRA_OECONF = " --with-realtime-testsuite --with-open-posix-testsuite "
> -# ltp network/rpc test cases ftbfs when libtirpc is found
> -EXTRA_OECONF += " --without-tirpc "
>
> do_install(){
> install -d ${D}${prefix}/
> --
> 2.17.1
//Peter
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145818):
https://lists.openembedded.org/g/openembedded-core/message/145818
Mute This Topic: https://lists.openembedded.org/mt/79036634/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH v2 1/5] systemd: update 246 -> 247
On Wed, 2020-12-16 at 18:07 +, Richard Purdie wrote:
> On Mon, 2020-12-14 at 12:34 +, Luca Bocassi wrote:
> > From: Luca Boccassi
> >
> > Update systemd to v247.1.
> > Add rule for new oomd dbus conf and for new pam.d
> > conf directory in /usr/lib.
> > Drop selinux-hook-handling-to-enumerate-nexthop.patch,
> > merged upstream.
> > Backport 0001-meson-set-cxx-variable-before-using-it.patch
> > from v247-stable to fix builds without C++.
> > Refresh musl patches.
> >
> > Signed-off-by: Luca Boccassi
> > ---
> > NOTE: the musl build has only been build-tested, and only on x86_64.
> > The glibc build has been built and runtime-tested on x86_64 and
> > arm64.
>
> [sorry, hit send early when writing]
>
> Unfortunately there is still something not quite right in this patchset
> as it failed again during testing. There are a few different problems:
>
> meta-intel in systemd-boot:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/100/builds/1016
Looking into this, thanks
> a reproducibility issue in oe-selftest:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/79/builds/1651
> (which did happen last time as well on centos but I don't think was
> reported on the list, sorry)
>
> These urls may help understand why its not reproducible:
>
> https://autobuilder.yocto.io/pub/repro-fail/oe-reproducible-20201215-mhq8wl33/packages/diff-html/
> https://autobuilder.yocto.io/pub/repro-fail/oe-reproducible-20201215-mhq8wl33/
>
> (looks like the uid of messagebus is changing at a really quick glance)
Given your follow-up message about host contamination, is there
anything I can do here to fix it/mitigate it, or is it a CI issue?
> and a build issue in world-lsb:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/108/builds/1189
>
> At the top of the logs you can see the configuration that is being used
> for a given build, so for example in the last one, the key difference
> to some of the other builds might be any one of:
>
> DISTRO = "poky-altcfg"
> DISTRO_FEATURES_append = ' api-documentation'
> require conf/multilib.conf
> MULTILIBS = 'multilib:lib32'
> DEFAULTTUNE_virtclass-multilib-lib32 = 'x86'
> QEMU_USE_KVM = 'True'
> INHERIT += 'testimage'
> SDK_EXT_TYPE = 'minimal'
> SDK_INCLUDE_TOOLCHAIN = '1'
>
> but given its a pam error, I'd strongly suspect the DISTRO = "poky-
> altcfg" would be the likely trigger.
ERROR: systemd-1_247.1-r0 do_package: QA Issue: systemd: Files/directories were
installed but not shipped in any packa
ge:
/usr/lib/pam.d
/usr/lib/pam.d/systemd-user
But the recipe has:
FILES_${PN} = " ${base_bindir}/* \
<...>
${libdir}/pam.d/ \
Shouldn't that match? Did I use the wrong variable?
Thanks!
--
Kind regards,
Luca Boccassi
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145819):
https://lists.openembedded.org/g/openembedded-core/message/145819
Mute This Topic: https://lists.openembedded.org/mt/78949130/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH v3] ltp: fix rpc build error
Add support to tirpc tests build.
Export incdir and libdir to have the right path.
Signed-off-by: Kory Maincent
---
Change since v1:
- Use PACKAGECONFIG to enable rpc tests
- Improve ${libdir} and ${incdir} export
Change since v2:
- Improve ${libdir} and ${incdir} export
meta/recipes-extended/ltp/ltp_20200930.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-extended/ltp/ltp_20200930.bb
b/meta/recipes-extended/ltp/ltp_20200930.bb
index 7acf15b360..711fce01f1 100644
--- a/meta/recipes-extended/ltp/ltp_20200930.bb
+++ b/meta/recipes-extended/ltp/ltp_20200930.bb
@@ -42,14 +42,14 @@ inherit autotools-brokensep pkgconfig
TARGET_CC_ARCH += "${LDFLAGS}"
+export libdir := "${libdir}"
+export incdir := "${incdir}"
export prefix = "/opt/${PN}"
-export exec_prefix = "/opt/${PN}"
PACKAGECONFIG[numa] = "--with-numa, --without-numa, numactl,"
+PACKAGECONFIG[rpc] = "--with-tirpc, --without-tirpc, virtual/librpc,"
EXTRA_AUTORECONF += "-I ${S}/testcases/realtime/m4"
EXTRA_OECONF = " --with-realtime-testsuite --with-open-posix-testsuite "
-# ltp network/rpc test cases ftbfs when libtirpc is found
-EXTRA_OECONF += " --without-tirpc "
do_install(){
install -d ${D}${prefix}/
--
2.17.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145820):
https://lists.openembedded.org/g/openembedded-core/message/145820
Mute This Topic: https://lists.openembedded.org/mt/79042035/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH v2 1/5] systemd: update 246 -> 247
On Thu, 2020-12-17 at 17:25 +, Luca Boccassi wrote:
> On Wed, 2020-12-16 at 18:07 +, Richard Purdie wrote:
> > On Mon, 2020-12-14 at 12:34 +, Luca Bocassi wrote:
> > > From: Luca Boccassi
> > >
> > > Update systemd to v247.1.
> > > Add rule for new oomd dbus conf and for new pam.d
> > > conf directory in /usr/lib.
> > > Drop selinux-hook-handling-to-enumerate-nexthop.patch,
> > > merged upstream.
> > > Backport 0001-meson-set-cxx-variable-before-using-it.patch
> > > from v247-stable to fix builds without C++.
> > > Refresh musl patches.
> > >
> > > Signed-off-by: Luca Boccassi
> > > ---
> > > NOTE: the musl build has only been build-tested, and only on x86_64.
> > > The glibc build has been built and runtime-tested on x86_64 and
> > > arm64.
> >
> > [sorry, hit send early when writing]
> >
> > Unfortunately there is still something not quite right in this patchset
> > as it failed again during testing. There are a few different problems:
> >
> > meta-intel in systemd-boot:
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/100/builds/1016
>
> Looking into this, thanks
>
> > a reproducibility issue in oe-selftest:
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/79/builds/1651
> > (which did happen last time as well on centos but I don't think was
> > reported on the list, sorry)
> >
> > These urls may help understand why its not reproducible:
> >
> > https://autobuilder.yocto.io/pub/repro-fail/oe-reproducible-20201215-mhq8wl33/packages/diff-html/
> > https://autobuilder.yocto.io/pub/repro-fail/oe-reproducible-20201215-mhq8wl33/
> >
> > (looks like the uid of messagebus is changing at a really quick glance)
>
> Given your follow-up message about host contamination, is there
> anything I can do here to fix it/mitigate it, or is it a CI issue?
We'll have to look into why building it on a centos system seems to
result in a different binary from building it on other distros. Its not
a CI problem, our multi distro setup is just exposing the problem (as
its intended to).
>
> > and a build issue in world-lsb:
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/108/builds/1189
> >
> > At the top of the logs you can see the configuration that is being used
> > for a given build, so for example in the last one, the key difference
> > to some of the other builds might be any one of:
> >
> > DISTRO = "poky-altcfg"
> > DISTRO_FEATURES_append = ' api-documentation'
> > require conf/multilib.conf
> > MULTILIBS = 'multilib:lib32'
> > DEFAULTTUNE_virtclass-multilib-lib32 = 'x86'
> > QEMU_USE_KVM = 'True'
> > INHERIT += 'testimage'
> > SDK_EXT_TYPE = 'minimal'
> > SDK_INCLUDE_TOOLCHAIN = '1'
> >
> > but given its a pam error, I'd strongly suspect the DISTRO = "poky-
> > altcfg" would be the likely trigger.
>
> ERROR: systemd-1_247.1-r0 do_package: QA Issue: systemd: Files/directories
> were installed but not shipped in any packa
> ge:
> /usr/lib/pam.d
> /usr/lib/pam.d/systemd-user
>
> But the recipe has:
>
> FILES_${PN} = " ${base_bindir}/* \
><...>
>${libdir}/pam.d/ \
>
> Shouldn't that match? Did I use the wrong variable?
I think the trailing slash may make it match the directory only? I'm
going from memory though...
Cheers,
Richard
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145821):
https://lists.openembedded.org/g/openembedded-core/message/145821
Mute This Topic: https://lists.openembedded.org/mt/78949130/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [meta-oe][PATCH v5] iproute2: split ip to individual package
Move the ip tool into its own package. Useful for size constrained
systems that only want the ip tool.
Signed-off-by: Sinan Kaya
---
meta/recipes-connectivity/iproute2/iproute2.inc | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc
b/meta/recipes-connectivity/iproute2/iproute2.inc
index 403d264308c..dd42700092a 100644
--- a/meta/recipes-connectivity/iproute2/iproute2.inc
+++ b/meta/recipes-connectivity/iproute2/iproute2.inc
@@ -46,10 +46,11 @@ do_install () {
# The .so files in iproute2-tc are modules, not traditional libraries
INSANE_SKIP_${PN}-tc = "dev-so"
-PACKAGES =+ "\
+IPROUTE2_PACKAGES =+ "\
${PN}-devlink \
${PN}-genl \
${PN}-ifstat \
+${PN}-ip \
${PN}-lnstat \
${PN}-nstat \
${PN}-rtacct \
@@ -58,12 +59,16 @@ PACKAGES =+ "\
${PN}-tipc \
"
+PACKAGE_BEFORE_PN = "${IPROUTE2_PACKAGES}"
+RDEPENDS_${PN} += "${PN}-ip"
+
FILES_${PN}-tc = "${base_sbindir}/tc* \
${libdir}/tc/*.so"
FILES_${PN}-lnstat = "${base_sbindir}/lnstat \
${base_sbindir}/ctstat \
${base_sbindir}/rtstat"
FILES_${PN}-ifstat = "${base_sbindir}/ifstat"
+FILES_${PN}-ip = "${base_sbindir}/ip.${PN} ${sysconfdir}/iproute2"
FILES_${PN}-genl = "${base_sbindir}/genl"
FILES_${PN}-rtacct = "${base_sbindir}/rtacct"
FILES_${PN}-nstat = "${base_sbindir}/nstat"
@@ -71,7 +76,7 @@ FILES_${PN}-ss = "${base_sbindir}/ss"
FILES_${PN}-tipc = "${base_sbindir}/tipc"
FILES_${PN}-devlink = "${base_sbindir}/devlink"
-ALTERNATIVE_${PN} = "ip"
+ALTERNATIVE_${PN}-ip = "ip"
ALTERNATIVE_TARGET[ip] = "${base_sbindir}/ip.${BPN}"
ALTERNATIVE_LINK_NAME[ip] = "${base_sbindir}/ip"
ALTERNATIVE_PRIORITY = "100"
--
2.17.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145822):
https://lists.openembedded.org/g/openembedded-core/message/145822
Mute This Topic: https://lists.openembedded.org/mt/79044672/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 1/4] pulseaudio: Remove OE_LT_RPATH_ALLOW
I don't know what these variables were supposed to do, but there doesn't
seem to be anything that would use these variables, so removing them
should be safe. The PulseAudio recipe is the only place where these
appear in the repository.
These lines were included already in the original PulseAudio recipe that
was added in commit d9a4c588ef24e7e4019fbe5a2314addbcd3a6bb8 in 2007,
and already at that time there didn't seem to be anything using these
variables.
Signed-off-by: Tanu Kaskinen
---
meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 5 -
1 file changed, 5 deletions(-)
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
index 3a26cb59dd..4aea8bc6aa 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
@@ -129,11 +129,6 @@ EXTRA_OECONF_append_armeb =
"${@bb.utils.contains("TUNE_FEATURES", "neon", "", "
export TARGET_PFPU = "${TARGET_FPU}"
-# TODO: Use more fine granular version
-#OE_LT_RPATH_ALLOW=":${libdir}/pulse-0.9:"
-OE_LT_RPATH_ALLOW = "any"
-OE_LT_RPATH_ALLOW[export]="1"
-
set_cfg_value () {
sed -i -e "s/\(; *\)\?$2 =.*/$2 = $3/" "$1"
if ! grep -q "^$2 = $3\$" "$1"; then
--
2.20.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145824):
https://lists.openembedded.org/g/openembedded-core/message/145824
Mute This Topic: https://lists.openembedded.org/mt/79044756/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 2/4] pulseaudio: disable EsounD support
This disables PulseAudio's EsounD emulation and the EsounD sink module. EsounD has been obsolete for a long time, and doesn't seem to be packaged for OpenEmbedded, so probably there are no applications around that would need PulseAudio's EsounD support. Signed-off-by: Tanu Kaskinen --- meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc index 4aea8bc6aa..ad059404f8 100644 --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc @@ -79,6 +79,7 @@ EXTRA_OECONF = "\ ac_cv_header_valgrind_memcheck_h=no \ --disable-tests \ --disable-running-from-build-tree \ + --disable-esound \ " # soxr (the SoX Resampler library) doesn't seem to be currently packaged in -- 2.20.1 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145825): https://lists.openembedded.org/g/openembedded-core/message/145825 Mute This Topic: https://lists.openembedded.org/mt/79044757/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 0/4] pulseaudio: Autotools to Meson
The PulseAudio project plans to drop Autotools support in 15.0, so let's switch to Meson. The first patch removes OE_LT_RPATH_ALLOW, which is some old cruft that sounds like it has something to do Libtool, so it's perhaps relevant for the goal of dropping Autotools. The next two patches disable features that aren't available with Meson. The last patch does the build system switch. Tanu Kaskinen (4): pulseaudio: Remove OE_LT_RPATH_ALLOW pulseaudio: disable EsounD support pulseaudio: disable GConf support pulseaudio: switch build system from Autotools to Meson .../pulseaudio/pulseaudio.inc | 69 --- ...-option-for-enabling-disabling-Valgr.patch | 67 ++ .../pulseaudio/pulseaudio_14.0.bb | 1 + 3 files changed, 97 insertions(+), 40 deletions(-) create mode 100644 meta/recipes-multimedia/pulseaudio/pulseaudio/0001-build-sys-Add-an-option-for-enabling-disabling-Valgr.patch -- 2.20.1 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145823): https://lists.openembedded.org/g/openembedded-core/message/145823 Mute This Topic: https://lists.openembedded.org/mt/79044755/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 4/4] pulseaudio: switch build system from Autotools to Meson
Upstream is moving from Autotools to Meson, Autotools support will be dropped in 15.0. I dropped some configure options: * --enable-tcpwrap=no doesn't (currently) have a counterpart in Meson, TCP Wrappers support is always disabled. * --disable-esound doesn't have a counterpart in Meson, EsounD support is always disabled. * --disable-gconf doesn't have a counterpart in Meson, GConf support is always disabled. I backported a patch that implements support for the -Dvalgrind=disabled option. I checked with buildhistory what the differences are before and after this patch (with qemuarm with neon removed from TUNE_FEATURES, all PACKAGECONFIGs enabled): * Obvious differences in DEPENDS: Autotools stuff removed and Meson stuff added. There wasn't anything strange here. * Packages have superfluous RDEPENDS removed from them. With Autotools something caused for example X11 libraries to be added to RDEPENDS of packages that don't have anything to do with X11. * The pulseaudio-src package had MMX and SSE related files removed and Neon related files added. I don't know why the ARM build previously had MMX and SSE files included, the addition of Neon files is explained by the fact that with Meson the Neon optimizations can't be disabled if the compiler supports Neon (see below). * libfoo.so symlinks changed to point to libfoo.so.X rather than directly to libfoo.so.X.Y.Z. To my understading that's fine, since libfoo.so.X is a symlink that points to libfoo.so.X.Y.Z. * There were various file size changes, which I didn't investigate. Previously the recipe disabled ARM Neon optimizations when "neon" was not in TUNE_FEATURES. That was originally added in commit 4e7b91b5a2613b957b08aefbee1aac28fdd19598 at a time when PulseAudio's build system didn't check the availability of the arm_neon.h header, causing compilation errors when the header wasn't available. That issue was fixed a long time ago, so there was little need for the TUNE_FEATURES check, although it was still possible to make the build fail if non-neon -mfpu was passed in CFLAGS, so the TUNE_FEATURES check still had some theoretical benefit (theoretical, because generally OE passes -mfpu in CC instead of CFLAGS, so OE's -mfpu option appears very early in the compiler command line, which doesn't trip up PulseAudio's current Autotools build system). With Meson there's no option for disabling Neon optimizations if the compiler supports Neon. If -mfpu is set in CC or CFLAGS, it's ignored, because the build system adds -mfpu=neon at the end of the compiler command line, overriding any earlier -mfpu options. This shouldn't be a problem, because PulseAudio detects at runtime whether the CPU supports Neon instructions. Signed-off-by: Tanu Kaskinen --- .../pulseaudio/pulseaudio.inc | 64 -- ...-option-for-enabling-disabling-Valgr.patch | 67 +++ .../pulseaudio/pulseaudio_14.0.bb | 1 + 3 files changed, 97 insertions(+), 35 deletions(-) create mode 100644 meta/recipes-multimedia/pulseaudio/pulseaudio/0001-build-sys-Add-an-option-for-enabling-disabling-Valgr.patch diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc index 48759bca62..cf4be7ed40 100644 --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc @@ -56,49 +56,47 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=0e5cd938de1a7a53ea5adac38cc10c39 \ file://src/pulsecore/filter/biquad.h;beginline=1;endline=4;md5=6d46d1365206528a20137355924233c1 \ " +# libtool is needed for libltdl, used in module loading. DEPENDS = "libatomic-ops libsndfile1 libtool" # optional DEPENDS += "udev alsa-lib glib-2.0" DEPENDS += "speexdsp libxml-parser-perl-native libcap" -inherit autotools bash-completion pkgconfig useradd gettext perlnative systemd manpages gsettings +inherit bash-completion meson pkgconfig useradd gettext perlnative systemd manpages gsettings # *.desktop rules wont be generated during configure and build will fail # if using --disable-nls USE_NLS = "yes" -EXTRA_OECONF = "\ - --disable-hal-compat \ - --disable-orc \ - --enable-tcpwrap=no \ - --with-access-group=audio \ - --disable-openssl \ - --with-database=simple \ - --without-zsh-completion-dir \ - --with-udev-rules-dir=`pkg-config --variable=udevdir udev`/rules.d \ - ac_cv_header_valgrind_memcheck_h=no \ - --disable-tests \ - --disable-running-from-build-tree \ - --disable-esound \ - --disable-gconf \ +EXTRA_OEMESON = "\ + -Dhal-compat=false \ + -Dorc=disabled \ + -Daccess_group=audio \ + -Dopenssl=disabled \ + -Ddatabase=simple \ + -Dzshcompl
[OE-core] [PATCH 3/4] pulseaudio: disable GConf support
The GConf module is only needed for older versions of paprefs, other
applications shouldn't be interacting with PulseAudio's GConf settings.
Paprefs isn't packaged in OpenEmbedded, so there probably aren't any
users of the GConf functionality.
My immediate motivation for doing this is that I'm converting the
pulseaudio recipe to use Meson, and the GConf module isn't supported by
the Meson build system. Adding support for it would be possible, but
pretty pointless.
The GConf module will be removed in PulseAudio 15.0.
Signed-off-by: Tanu Kaskinen
---
meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
index ad059404f8..48759bca62 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
@@ -80,6 +80,7 @@ EXTRA_OECONF = "\
--disable-tests \
--disable-running-from-build-tree \
--disable-esound \
+ --disable-gconf \
"
# soxr (the SoX Resampler library) doesn't seem to be currently packaged in
@@ -108,7 +109,6 @@ PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES',
'bluetooth', 'bluez5'
PACKAGECONFIG[dbus] = "--enable-dbus,--disable-dbus,dbus"
PACKAGECONFIG[bluez5] = "--enable-bluez5,--disable-bluez5,bluez5 sbc"
-PACKAGECONFIG[gconf] = "--enable-gconf,--disable-gconf,gconf"
PACKAGECONFIG[gsettings] =
"--enable-gsettings,--disable-gsettings,glib-2.0-native glib-2.0"
PACKAGECONFIG[ofono] =
"--enable-bluez5-ofono-headset,--disable-bluez5-ofono-headset,ofono"
PACKAGECONFIG[gtk] = "--enable-gtk3,--disable-gtk3,gtk+3"
@@ -265,7 +265,6 @@ RDEPENDS_pulseaudio-module-console-kit =+
"${@bb.utils.contains('DISTRO_FEATURES
RDEPENDS_pulseaudio-misc += "pulseaudio-module-cli-protocol-unix"
FILES_${PN}-module-alsa-card += "${datadir}/pulseaudio/alsa-mixer"
-FILES_${PN}-module-gconf += "${libexecdir}/pulse/gconf-helper"
GSETTINGS_PACKAGE = "${@bb.utils.contains('PACKAGECONFIG', 'gsettings',
'${PN}-module-gsettings', '', d)}"
FILES_${PN}-module-gsettings += "${libexecdir}/pulse/gsettings-helper
${datadir}/GConf/gsettings ${datadir}/glib-2.0/schemas"
--
2.20.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145826):
https://lists.openembedded.org/g/openembedded-core/message/145826
Mute This Topic: https://lists.openembedded.org/mt/79044759/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 1/4] initscripts: use quotes for shell variable comparision
Helps to execute it with busybox shell Signed-off-by: Khem Raj --- meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh index 02f0351fcb..a63e71b780 100755 --- a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh +++ b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh @@ -74,7 +74,7 @@ test "$VERBOSE" != no && echo "Activating swap" # # Check the root filesystem. # -if test -f /fastboot || test $rootcheck = no +if test -f /fastboot || test "$rootcheck" = "no" then test $rootcheck = yes && echo "Fast boot, no filesystem check" else -- 2.29.2 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145828): https://lists.openembedded.org/g/openembedded-core/message/145828 Mute This Topic: https://lists.openembedded.org/mt/79049546/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 4/4] initscripts: Use initctl on sysvinit only
Check if init system is sysvinit to recreate initctl, this ensures that it can be used with busybox init system as well Signed-off-by: Khem Raj --- .../initscripts/initscripts-1.0/mountall.sh | 22 --- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/meta/recipes-core/initscripts/initscripts-1.0/mountall.sh b/meta/recipes-core/initscripts/initscripts-1.0/mountall.sh index c719be5d9a..ead4dba46d 100755 --- a/meta/recipes-core/initscripts/initscripts-1.0/mountall.sh +++ b/meta/recipes-core/initscripts/initscripts-1.0/mountall.sh @@ -19,15 +19,21 @@ test "$VERBOSE" != no && echo "Mounting local filesystems..." mount -at nonfs,nosmbfs,noncpfs 2>/dev/null -# -# We might have mounted something over /dev, see if /dev/initctl is there. -# -if test ! -p /dev/initctl -then - rm -f /dev/initctl - mknod -m 600 /dev/initctl p + +# We might have mounted something over /run; see if +# /dev/initctl is present. Look for +# /sbin/init.sysvinit to verify that sysvinit (and +# not busybox or systemd) is installed as default init). +INITCTL="/dev/initctl" +if [ ! -p "$INITCTL" ] && [ "$(readlink -- "/sbin/init")" = "/sbin/init.sysvinit" ]; then +# Create new control channel + rm -f "$INITCTL" + mknod -m 600 "$INITCTL" p + + # Reopen control channel. + PID="$(pidof -s /sbin/init || echo 1)" + [ -n "$PID" ] && kill -s USR1 "$PID" fi -kill -USR1 1 # # Execute swapon command again, in case we want to swap to -- 2.29.2 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145831): https://lists.openembedded.org/g/openembedded-core/message/145831 Mute This Topic: https://lists.openembedded.org/mt/79049550/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 2/4] busybox: Install /etc/default/rcS when used as init system
This helps in using sysvinit scripts with busybox init system as well
Signed-off-by: Khem Raj
---
meta/recipes-core/busybox/busybox.inc | 1 +
meta/recipes-core/busybox/busybox_1.32.0.bb | 1 +
meta/recipes-core/busybox/files/rcS.default | 1 +
3 files changed, 3 insertions(+)
create mode 100644 meta/recipes-core/busybox/files/rcS.default
diff --git a/meta/recipes-core/busybox/busybox.inc
b/meta/recipes-core/busybox/busybox.inc
index e0522be729..c3a97261d5 100644
--- a/meta/recipes-core/busybox/busybox.inc
+++ b/meta/recipes-core/busybox/busybox.inc
@@ -312,6 +312,7 @@ do_install () {
if grep -q "CONFIG_INIT=y" ${B}/.config; then
install -D -m 0755 ${WORKDIR}/rcS ${D}${sysconfdir}/init.d/rcS
install -D -m 0755 ${WORKDIR}/rcK ${D}${sysconfdir}/init.d/rcK
+ install -D -m 0755 ${WORKDIR}/rcS.default
${D}${sysconfdir}/default/rcS
fi
if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)};
then
diff --git a/meta/recipes-core/busybox/busybox_1.32.0.bb
b/meta/recipes-core/busybox/busybox_1.32.0.bb
index 8e23b0d4a2..3a669444dd 100644
--- a/meta/recipes-core/busybox/busybox_1.32.0.bb
+++ b/meta/recipes-core/busybox/busybox_1.32.0.bb
@@ -33,6 +33,7 @@ SRC_URI =
"https://busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
file://getopts.cfg \
file://resize.cfg \
${@["",
"file://init.cfg"][(d.getVar('VIRTUAL-RUNTIME_init_manager') == 'busybox')]} \
+ ${@["",
"file://rcS.default"][(d.getVar('VIRTUAL-RUNTIME_init_manager') == 'busybox')]}
\
${@["", "file://mdev.cfg"][(d.getVar('VIRTUAL-RUNTIME_dev_manager')
== 'busybox-mdev')]} \
file://syslog.cfg \
file://unicode.cfg \
diff --git a/meta/recipes-core/busybox/files/rcS.default
b/meta/recipes-core/busybox/files/rcS.default
new file mode 100644
index 00..262cad7d2a
--- /dev/null
+++ b/meta/recipes-core/busybox/files/rcS.default
@@ -0,0 +1 @@
+# SULOGIN = "yes"
--
2.29.2
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145829):
https://lists.openembedded.org/g/openembedded-core/message/145829
Mute This Topic: https://lists.openembedded.org/mt/79049548/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 3/4] busybox: Run mdev as daemon
When busybox is used for device management, kernel needs to support
older/obsolete mechanism via CONFIG_UEVENT_HELPER and
CONFIG_UEVENT_HELPER_PATH to enable /proc/sys/kernel/hotplug but this
would require kernel defconfig change and will always be needed when
mdev is used, intead run it in daemon mode
Update mdev init script to run mdev in daemon mode
Signed-off-by: Khem Raj
---
meta/recipes-core/busybox/busybox/mdev.cfg | 2 +
meta/recipes-core/busybox/files/mdev | 56 +++---
2 files changed, 41 insertions(+), 17 deletions(-)
diff --git a/meta/recipes-core/busybox/busybox/mdev.cfg
b/meta/recipes-core/busybox/busybox/mdev.cfg
index 6aefe90e43..143e6097cb 100644
--- a/meta/recipes-core/busybox/busybox/mdev.cfg
+++ b/meta/recipes-core/busybox/busybox/mdev.cfg
@@ -9,3 +9,5 @@ CONFIG_SETSID=y
CONFIG_CTTYHACK=y
CONFIG_FEATURE_SHADOWPASSWDS=y
+CONFIG_FEATURE_XARGS_SUPPORT_ZERO_TERM=y
+CONFIG_FEATURE_MDEV_DAEMON=y
diff --git a/meta/recipes-core/busybox/files/mdev
b/meta/recipes-core/busybox/files/mdev
index 8c9c06e96c..2fbdfb073e 100755
--- a/meta/recipes-core/busybox/files/mdev
+++ b/meta/recipes-core/busybox/files/mdev
@@ -1,21 +1,43 @@
#!/bin/sh
-mount -t proc proc /proc
-mount -t sysfs sysfs /sys
-mount -t tmpfs tmpfs /dev -o size=64k,mode=0755
-mkdir /dev/pts /dev/shm
-chmod 777 /dev/shm
-mount -t devpts devpts /dev/pts
-touch /dev/mdev.seq
-#sysctl -w kernel.hotplug=/sbin/mdev
-echo "/sbin/mdev" > /proc/sys/kernel/hotplug
-mdev -s
-
#
-# We might have mounted something over /dev, see if /dev/initctl is there.
+# Run the mdev daemon
#
-if test ! -p /dev/initctl
-then
-rm -f /dev/initctl
-mknod -m 600 /dev/initctl p
-fi
+
+DAEMON="mdev"
+PIDFILE="/var/run/$DAEMON.pid"
+
+
+start() {
+ echo -n "Starting $DAEMON... "
+ start-stop-daemon -S -b -m -p $PIDFILE -x /sbin/mdev -- -df
+ [ $? -eq 0 ] && echo "OK" || echo "ERROR"
+
+ # coldplug modules
+ find /sys/ -name modalias -print0 | \
+xargs -0 sort -u | \
+tr '\n' '\0' | \
+xargs -0 modprobe -abq
+}
+
+stop() {
+ echo -n "Stopping $DAEMON... "
+ start-stop-daemon -K -p $PIDFILE
+ [ $? -eq 0 ] && echo "OK" || echo "ERROR"
+}
+
+restart() {
+ stop
+ start
+}
+
+case "$1" in
+ start|stop|restart)
+ "$1"
+ ;;
+ *)
+ echo "Usage: $0 {start|stop|restart}"
+ exit 1
+esac
+
+exit $?
--
2.29.2
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145830):
https://lists.openembedded.org/g/openembedded-core/message/145830
Mute This Topic: https://lists.openembedded.org/mt/79049549/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/4] initscripts: use quotes for shell variable comparision
On Thu, Dec 17, 2020 at 2:54 PM Khem Raj wrote: > > Helps to execute it with busybox shell How does it help? Adding quotes would help if $rootcheck is an empty string, but that's not specific to busybox. > Signed-off-by: Khem Raj > --- > meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > index 02f0351fcb..a63e71b780 100755 > --- a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > +++ b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > @@ -74,7 +74,7 @@ test "$VERBOSE" != no && echo "Activating swap" > # > # Check the root filesystem. > # > -if test -f /fastboot || test $rootcheck = no > +if test -f /fastboot || test "$rootcheck" = "no" > then >test $rootcheck = yes && echo "Fast boot, no filesystem check" Doesn't this instance need to be quoted too? > else > -- > 2.29.2 > > > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145832): https://lists.openembedded.org/g/openembedded-core/message/145832 Mute This Topic: https://lists.openembedded.org/mt/79049546/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/4] initscripts: use quotes for shell variable comparision
On Thu, Dec 17, 2020 at 3:51 PM Andre McCurdy wrote: > > On Thu, Dec 17, 2020 at 2:54 PM Khem Raj wrote: > > > > Helps to execute it with busybox shell > > How does it help? Adding quotes would help if $rootcheck is an empty > string, but that's not specific to busybox. > > > Signed-off-by: Khem Raj > > --- > > meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > index 02f0351fcb..a63e71b780 100755 > > --- a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > +++ b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > @@ -74,7 +74,7 @@ test "$VERBOSE" != no && echo "Activating swap" > > # > > # Check the root filesystem. > > # > > -if test -f /fastboot || test $rootcheck = no > > +if test -f /fastboot || test "$rootcheck" = "no" > > then > >test $rootcheck = yes && echo "Fast boot, no filesystem check" > > Doesn't this instance need to be quoted too? yes infact there are more cleanup to follow, this series just fixed default boot errors on reference images. > > > else > > -- > > 2.29.2 > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145833): https://lists.openembedded.org/g/openembedded-core/message/145833 Mute This Topic: https://lists.openembedded.org/mt/79049546/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 3/4] busybox: Run mdev as daemon
On Thu, Dec 17, 2020 at 2:54 PM Khem Raj wrote:
>
> When busybox is used for device management, kernel needs to support
> older/obsolete mechanism via CONFIG_UEVENT_HELPER and
> CONFIG_UEVENT_HELPER_PATH to enable /proc/sys/kernel/hotplug but this
> would require kernel defconfig change and will always be needed when
> mdev is used, intead run it in daemon mode
>
> Update mdev init script to run mdev in daemon mode
>
> Signed-off-by: Khem Raj
> ---
> meta/recipes-core/busybox/busybox/mdev.cfg | 2 +
> meta/recipes-core/busybox/files/mdev | 56 +++---
> 2 files changed, 41 insertions(+), 17 deletions(-)
>
> diff --git a/meta/recipes-core/busybox/busybox/mdev.cfg
> b/meta/recipes-core/busybox/busybox/mdev.cfg
> index 6aefe90e43..143e6097cb 100644
> --- a/meta/recipes-core/busybox/busybox/mdev.cfg
> +++ b/meta/recipes-core/busybox/busybox/mdev.cfg
> @@ -9,3 +9,5 @@ CONFIG_SETSID=y
> CONFIG_CTTYHACK=y
>
> CONFIG_FEATURE_SHADOWPASSWDS=y
> +CONFIG_FEATURE_XARGS_SUPPORT_ZERO_TERM=y
> +CONFIG_FEATURE_MDEV_DAEMON=y
> diff --git a/meta/recipes-core/busybox/files/mdev
> b/meta/recipes-core/busybox/files/mdev
> index 8c9c06e96c..2fbdfb073e 100755
> --- a/meta/recipes-core/busybox/files/mdev
> +++ b/meta/recipes-core/busybox/files/mdev
> @@ -1,21 +1,43 @@
> #!/bin/sh
> -mount -t proc proc /proc
> -mount -t sysfs sysfs /sys
> -mount -t tmpfs tmpfs /dev -o size=64k,mode=0755
> -mkdir /dev/pts /dev/shm
> -chmod 777 /dev/shm
> -mount -t devpts devpts /dev/pts
> -touch /dev/mdev.seq
> -#sysctl -w kernel.hotplug=/sbin/mdev
> -echo "/sbin/mdev" > /proc/sys/kernel/hotplug
> -mdev -s
> -
> #
> -# We might have mounted something over /dev, see if /dev/initctl is there.
> +# Run the mdev daemon
> #
> -if test ! -p /dev/initctl
> -then
> -rm -f /dev/initctl
> -mknod -m 600 /dev/initctl p
> -fi
> +
> +DAEMON="mdev"
> +PIDFILE="/var/run/$DAEMON.pid"
> +
> +
> +start() {
> + echo -n "Starting $DAEMON... "
> + start-stop-daemon -S -b -m -p $PIDFILE -x /sbin/mdev -- -df
Where do these start-stop-daemon options come from? Using -b for an
application which is designed to run as a daemon (and deliberately
telling that app to run in the foreground) looks odd, etc.
Are there bugs or limitations in mdev which you are trying to workaround?
> + [ $? -eq 0 ] && echo "OK" || echo "ERROR"
> +
> + # coldplug modules
> + find /sys/ -name modalias -print0 | \
> +xargs -0 sort -u | \
> +tr '\n' '\0' | \
> +xargs -0 modprobe -abq
> +}
> +
> +stop() {
> + echo -n "Stopping $DAEMON... "
> + start-stop-daemon -K -p $PIDFILE
> + [ $? -eq 0 ] && echo "OK" || echo "ERROR"
> +}
> +
> +restart() {
> + stop
> + start
> +}
> +
> +case "$1" in
> + start|stop|restart)
> + "$1"
> + ;;
> + *)
> + echo "Usage: $0 {start|stop|restart}"
> + exit 1
> +esac
> +
> +exit $?
>
> --
> 2.29.2
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145834):
https://lists.openembedded.org/g/openembedded-core/message/145834
Mute This Topic: https://lists.openembedded.org/mt/79049549/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/4] initscripts: use quotes for shell variable comparision
On Thu, Dec 17, 2020 at 3:54 PM Khem Raj wrote: > > On Thu, Dec 17, 2020 at 3:51 PM Andre McCurdy wrote: > > > > On Thu, Dec 17, 2020 at 2:54 PM Khem Raj wrote: > > > > > > Helps to execute it with busybox shell > > > > How does it help? Adding quotes would help if $rootcheck is an empty > > string, but that's not specific to busybox. > > > > > Signed-off-by: Khem Raj > > > --- > > > meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > > b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > > index 02f0351fcb..a63e71b780 100755 > > > --- a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > > +++ b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > > @@ -74,7 +74,7 @@ test "$VERBOSE" != no && echo "Activating swap" > > > # > > > # Check the root filesystem. > > > # > > > -if test -f /fastboot || test $rootcheck = no > > > +if test -f /fastboot || test "$rootcheck" = "no" > > > then > > >test $rootcheck = yes && echo "Fast boot, no filesystem check" > > > > Doesn't this instance need to be quoted too? > > yes infact there are more cleanup to follow, this series just fixed > default boot errors on reference images. Adding quotes around a variable should probably be done consistently, so maybe it makes sense to hold off on this one until you've finished the cleanup? -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145835): https://lists.openembedded.org/g/openembedded-core/message/145835 Mute This Topic: https://lists.openembedded.org/mt/79049546/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH v2 1/5] systemd: update 246 -> 247
On 12/18/2020 01:25 AM, Luca Bocassi wrote:
On Wed, 2020-12-16 at 18:07 +, Richard Purdie wrote:
On Mon, 2020-12-14 at 12:34 +, Luca Bocassi wrote:
From: Luca Boccassi
Update systemd to v247.1.
Add rule for new oomd dbus conf and for new pam.d
conf directory in /usr/lib.
Drop selinux-hook-handling-to-enumerate-nexthop.patch,
merged upstream.
Backport 0001-meson-set-cxx-variable-before-using-it.patch
from v247-stable to fix builds without C++.
Refresh musl patches.
Signed-off-by: Luca Boccassi
---
NOTE: the musl build has only been build-tested, and only on x86_64.
The glibc build has been built and runtime-tested on x86_64 and
arm64.
[sorry, hit send early when writing]
Unfortunately there is still something not quite right in this patchset
as it failed again during testing. There are a few different problems:
meta-intel in systemd-boot:
https://autobuilder.yoctoproject.org/typhoon/#/builders/100/builds/1016
Looking into this, thanks
a reproducibility issue in oe-selftest:
https://autobuilder.yoctoproject.org/typhoon/#/builders/79/builds/1651
(which did happen last time as well on centos but I don't think was
reported on the list, sorry)
These urls may help understand why its not reproducible:
https://autobuilder.yocto.io/pub/repro-fail/oe-reproducible-20201215-mhq8wl33/packages/diff-html/
https://autobuilder.yocto.io/pub/repro-fail/oe-reproducible-20201215-mhq8wl33/
(looks like the uid of messagebus is changing at a really quick glance)
Given your follow-up message about host contamination, is there
anything I can do here to fix it/mitigate it, or is it a CI issue?
and a build issue in world-lsb:
https://autobuilder.yoctoproject.org/typhoon/#/builders/108/builds/1189
At the top of the logs you can see the configuration that is being used
for a given build, so for example in the last one, the key difference
to some of the other builds might be any one of:
DISTRO = "poky-altcfg"
DISTRO_FEATURES_append = ' api-documentation'
require conf/multilib.conf
MULTILIBS = 'multilib:lib32'
DEFAULTTUNE_virtclass-multilib-lib32 = 'x86'
QEMU_USE_KVM = 'True'
INHERIT += 'testimage'
SDK_EXT_TYPE = 'minimal'
SDK_INCLUDE_TOOLCHAIN = '1'
but given its a pam error, I'd strongly suspect the DISTRO = "poky-
altcfg" would be the likely trigger.
ERROR: systemd-1_247.1-r0 do_package: QA Issue: systemd: Files/directories were
installed but not shipped in any packa
ge:
/usr/lib/pam.d
/usr/lib/pam.d/systemd-user
But the recipe has:
FILES_${PN} = " ${base_bindir}/* \
<...>
${libdir}/pam.d/ \
Shouldn't that match? Did I use the wrong variable?
Thanks!
I think it should be ${nonarch_libdir}, as ${libdir} will be expanded to
'/usr/lib64' in the above multilib config.
Regards,
Chen Qi
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145836):
https://lists.openembedded.org/g/openembedded-core/message/145836
Mute This Topic: https://lists.openembedded.org/mt/78949130/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH v2 1/5] systemd: update 246 -> 247
On Wed, 2020-12-16 at 18:07 +, Richard Purdie wrote: > > Unfortunately there is still something not quite right in this > patchset > as it failed again during testing. There are a few different > problems: > > meta-intel in systemd-boot: > > https://autobuilder.yoctoproject.org/typhoon/#/builders/100/builds/1016 It looks like this happened because of systemd-boot change from Ross. I have removed the bbappend from meta-intel so this shouldn't fail now. Thanks, Anuj -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145837): https://lists.openembedded.org/g/openembedded-core/message/145837 Mute This Topic: https://lists.openembedded.org/mt/78949130/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 3/4] busybox: Run mdev as daemon
Afaik, running a daemon in foreground mode and instructing start-stop-daemon to
background it is common practice if the daemon doesn’t support writing a
pidfile itself, so start-stop-daemon can handle it itself. Without using
foregrounded process + -b, it wouldn’t know the resulting process id. If
busybox mdev supports a pidfile, this wouldn’t be necessary.
--
Christopher “kergoth” Larson
chris_lar...@mentor.com, chris.lar...@siemens.com, kerg...@gmail.com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics, a Siemens Business
On Dec 17, 2020, 4:59 PM -0700, Andre McCurdy , wrote:
> On Thu, Dec 17, 2020 at 2:54 PM Khem Raj wrote:
> >
> > When busybox is used for device management, kernel needs to support
> > older/obsolete mechanism via CONFIG_UEVENT_HELPER and
> > CONFIG_UEVENT_HELPER_PATH to enable /proc/sys/kernel/hotplug but this
> > would require kernel defconfig change and will always be needed when
> > mdev is used, intead run it in daemon mode
> >
> > Update mdev init script to run mdev in daemon mode
> >
> > Signed-off-by: Khem Raj
> > ---
> > meta/recipes-core/busybox/busybox/mdev.cfg | 2 +
> > meta/recipes-core/busybox/files/mdev | 56 +++---
> > 2 files changed, 41 insertions(+), 17 deletions(-)
> >
> > diff --git a/meta/recipes-core/busybox/busybox/mdev.cfg
> > b/meta/recipes-core/busybox/busybox/mdev.cfg
> > index 6aefe90e43..143e6097cb 100644
> > --- a/meta/recipes-core/busybox/busybox/mdev.cfg
> > +++ b/meta/recipes-core/busybox/busybox/mdev.cfg
> > @@ -9,3 +9,5 @@ CONFIG_SETSID=y
> > CONFIG_CTTYHACK=y
> >
> > CONFIG_FEATURE_SHADOWPASSWDS=y
> > +CONFIG_FEATURE_XARGS_SUPPORT_ZERO_TERM=y
> > +CONFIG_FEATURE_MDEV_DAEMON=y
> > diff --git a/meta/recipes-core/busybox/files/mdev
> > b/meta/recipes-core/busybox/files/mdev
> > index 8c9c06e96c..2fbdfb073e 100755
> > --- a/meta/recipes-core/busybox/files/mdev
> > +++ b/meta/recipes-core/busybox/files/mdev
> > @@ -1,21 +1,43 @@
> > #!/bin/sh
> > -mount -t proc proc /proc
> > -mount -t sysfs sysfs /sys
> > -mount -t tmpfs tmpfs /dev -o size=64k,mode=0755
> > -mkdir /dev/pts /dev/shm
> > -chmod 777 /dev/shm
> > -mount -t devpts devpts /dev/pts
> > -touch /dev/mdev.seq
> > -#sysctl -w kernel.hotplug=/sbin/mdev
> > -echo "/sbin/mdev" > /proc/sys/kernel/hotplug
> > -mdev -s
> > -
> > #
> > -# We might have mounted something over /dev, see if /dev/initctl is there.
> > +# Run the mdev daemon
> > #
> > -if test ! -p /dev/initctl
> > -then
> > - rm -f /dev/initctl
> > - mknod -m 600 /dev/initctl p
> > -fi
> > +
> > +DAEMON="mdev"
> > +PIDFILE="/var/run/$DAEMON.pid"
> > +
> > +
> > +start() {
> > + echo -n "Starting $DAEMON... "
> > + start-stop-daemon -S -b -m -p $PIDFILE -x /sbin/mdev -- -df
>
> Where do these start-stop-daemon options come from? Using -b for an
> application which is designed to run as a daemon (and deliberately
> telling that app to run in the foreground) looks odd, etc.
>
> Are there bugs or limitations in mdev which you are trying to workaround?
>
> > + [ $? -eq 0 ] && echo "OK" || echo "ERROR"
> > +
> > + # coldplug modules
> > + find /sys/ -name modalias -print0 | \
> > + xargs -0 sort -u | \
> > + tr '\n' '\0' | \
> > + xargs -0 modprobe -abq
> > +}
> > +
> > +stop() {
> > + echo -n "Stopping $DAEMON... "
> > + start-stop-daemon -K -p $PIDFILE
> > + [ $? -eq 0 ] && echo "OK" || echo "ERROR"
> > +}
> > +
> > +restart() {
> > + stop
> > + start
> > +}
> > +
> > +case "$1" in
> > + start|stop|restart)
> > + "$1"
> > + ;;
> > + *)
> > + echo "Usage: $0 {start|stop|restart}"
> > + exit 1
> > +esac
> > +
> > +exit $?
> >
> > --
> > 2.29.2
> >
> >
> >
> >
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145838):
https://lists.openembedded.org/g/openembedded-core/message/145838
Mute This Topic: https://lists.openembedded.org/mt/79049549/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 3/4] busybox: Run mdev as daemon
On Thu, Dec 17, 2020 at 3:59 PM Andre McCurdy wrote:
>
> On Thu, Dec 17, 2020 at 2:54 PM Khem Raj wrote:
> >
> > When busybox is used for device management, kernel needs to support
> > older/obsolete mechanism via CONFIG_UEVENT_HELPER and
> > CONFIG_UEVENT_HELPER_PATH to enable /proc/sys/kernel/hotplug but this
> > would require kernel defconfig change and will always be needed when
> > mdev is used, intead run it in daemon mode
> >
> > Update mdev init script to run mdev in daemon mode
> >
> > Signed-off-by: Khem Raj
> > ---
> > meta/recipes-core/busybox/busybox/mdev.cfg | 2 +
> > meta/recipes-core/busybox/files/mdev | 56 +++---
> > 2 files changed, 41 insertions(+), 17 deletions(-)
> >
> > diff --git a/meta/recipes-core/busybox/busybox/mdev.cfg
> > b/meta/recipes-core/busybox/busybox/mdev.cfg
> > index 6aefe90e43..143e6097cb 100644
> > --- a/meta/recipes-core/busybox/busybox/mdev.cfg
> > +++ b/meta/recipes-core/busybox/busybox/mdev.cfg
> > @@ -9,3 +9,5 @@ CONFIG_SETSID=y
> > CONFIG_CTTYHACK=y
> >
> > CONFIG_FEATURE_SHADOWPASSWDS=y
> > +CONFIG_FEATURE_XARGS_SUPPORT_ZERO_TERM=y
> > +CONFIG_FEATURE_MDEV_DAEMON=y
> > diff --git a/meta/recipes-core/busybox/files/mdev
> > b/meta/recipes-core/busybox/files/mdev
> > index 8c9c06e96c..2fbdfb073e 100755
> > --- a/meta/recipes-core/busybox/files/mdev
> > +++ b/meta/recipes-core/busybox/files/mdev
> > @@ -1,21 +1,43 @@
> > #!/bin/sh
> > -mount -t proc proc /proc
> > -mount -t sysfs sysfs /sys
> > -mount -t tmpfs tmpfs /dev -o size=64k,mode=0755
> > -mkdir /dev/pts /dev/shm
> > -chmod 777 /dev/shm
> > -mount -t devpts devpts /dev/pts
> > -touch /dev/mdev.seq
> > -#sysctl -w kernel.hotplug=/sbin/mdev
> > -echo "/sbin/mdev" > /proc/sys/kernel/hotplug
> > -mdev -s
> > -
> > #
> > -# We might have mounted something over /dev, see if /dev/initctl is there.
> > +# Run the mdev daemon
> > #
> > -if test ! -p /dev/initctl
> > -then
> > -rm -f /dev/initctl
> > -mknod -m 600 /dev/initctl p
> > -fi
> > +
> > +DAEMON="mdev"
> > +PIDFILE="/var/run/$DAEMON.pid"
> > +
> > +
> > +start() {
> > + echo -n "Starting $DAEMON... "
> > + start-stop-daemon -S -b -m -p $PIDFILE -x /sbin/mdev -- -df
>
> Where do these start-stop-daemon options come from? Using -b for an
> application which is designed to run as a daemon (and deliberately
> telling that app to run in the foreground) looks odd, etc.
>
> Are there bugs or limitations in mdev which you are trying to workaround?
>
its using start-stop-daemon applet from busybox which does have these options
implemented, its so that start-stop-daemon takes the control of the daemon
> > + [ $? -eq 0 ] && echo "OK" || echo "ERROR"
> > +
> > + # coldplug modules
> > + find /sys/ -name modalias -print0 | \
> > +xargs -0 sort -u | \
> > +tr '\n' '\0' | \
> > +xargs -0 modprobe -abq
> > +}
> > +
> > +stop() {
> > + echo -n "Stopping $DAEMON... "
> > + start-stop-daemon -K -p $PIDFILE
> > + [ $? -eq 0 ] && echo "OK" || echo "ERROR"
> > +}
> > +
> > +restart() {
> > + stop
> > + start
> > +}
> > +
> > +case "$1" in
> > + start|stop|restart)
> > + "$1"
> > + ;;
> > + *)
> > + echo "Usage: $0 {start|stop|restart}"
> > + exit 1
> > +esac
> > +
> > +exit $?
> >
> > --
> > 2.29.2
> >
> >
> >
> >
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145839):
https://lists.openembedded.org/g/openembedded-core/message/145839
Mute This Topic: https://lists.openembedded.org/mt/79049549/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/4] initscripts: use quotes for shell variable comparision
On Thu, Dec 17, 2020 at 4:02 PM Andre McCurdy wrote: > > On Thu, Dec 17, 2020 at 3:54 PM Khem Raj wrote: > > > > On Thu, Dec 17, 2020 at 3:51 PM Andre McCurdy wrote: > > > > > > On Thu, Dec 17, 2020 at 2:54 PM Khem Raj wrote: > > > > > > > > Helps to execute it with busybox shell > > > > > > How does it help? Adding quotes would help if $rootcheck is an empty > > > string, but that's not specific to busybox. > > > > > > > Signed-off-by: Khem Raj > > > > --- > > > > meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > > > b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > > > index 02f0351fcb..a63e71b780 100755 > > > > --- a/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > > > +++ b/meta/recipes-core/initscripts/initscripts-1.0/checkroot.sh > > > > @@ -74,7 +74,7 @@ test "$VERBOSE" != no && echo "Activating swap" > > > > # > > > > # Check the root filesystem. > > > > # > > > > -if test -f /fastboot || test $rootcheck = no > > > > +if test -f /fastboot || test "$rootcheck" = "no" > > > > then > > > >test $rootcheck = yes && echo "Fast boot, no filesystem check" > > > > > > Doesn't this instance need to be quoted too? > > > > yes infact there are more cleanup to follow, this series just fixed > > default boot errors on reference images. > > Adding quotes around a variable should probably be done consistently, > so maybe it makes sense to hold off on this one until you've finished > the cleanup? it doesn't matter, I would be creating per package patches. -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145840): https://lists.openembedded.org/g/openembedded-core/message/145840 Mute This Topic: https://lists.openembedded.org/mt/79049546/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [dunfell][PATCH] systemd: Fix reallocarray check
From: Khem Raj Musl brought this latent issue to fore Signed-off-by: Khem Raj Signed-off-by: Richard Purdie --- .../0001-meson-Fix-reallocarray-check.patch | 30 +++ meta/recipes-core/systemd/systemd_246.6.bb| 1 + 2 files changed, 31 insertions(+) create mode 100644 meta/recipes-core/systemd/systemd/0001-meson-Fix-reallocarray-check.patch diff --git a/meta/recipes-core/systemd/systemd/0001-meson-Fix-reallocarray-check.patch b/meta/recipes-core/systemd/systemd/0001-meson-Fix-reallocarray-check.patch new file mode 100644 index ..1229dd6e175a --- /dev/null +++ b/meta/recipes-core/systemd/systemd/0001-meson-Fix-reallocarray-check.patch @@ -0,0 +1,30 @@ +From 1ebf1a1df17afd8b89f84b1928a89069035bf20b Mon Sep 17 00:00:00 2001 +From: Khem Raj +Date: Sat, 12 Dec 2020 16:15:57 -0800 +Subject: [PATCH] meson: Fix reallocarray check + +reallocarray() is defined in stdlib.h, so that would be right header to +check for its presense. + +Upstream-Status: Submitted [https://github.com/systemd/systemd/pull/17951] +Signed-off-by: Khem Raj +--- + meson.build | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/meson.build b/meson.build +index d205f846c7..e0a3c53529 100644 +--- a/meson.build b/meson.build +@@ -517,7 +517,7 @@ foreach ident : [ + #include + #include '''], + ['explicit_bzero' , '''#include '''], +-['reallocarray', '''#include '''], ++['reallocarray', '''#include '''], + ['set_mempolicy', '''#include + #include '''], + ['get_mempolicy', '''#include +-- +2.29.2 + diff --git a/meta/recipes-core/systemd/systemd_246.6.bb b/meta/recipes-core/systemd/systemd_246.6.bb index d9e7b1a00c3a..b89eff6be2e4 100644 --- a/meta/recipes-core/systemd/systemd_246.6.bb +++ b/meta/recipes-core/systemd/systemd_246.6.bb @@ -24,6 +24,7 @@ SRC_URI += "file://touchscreen.rules \ file://0001-systemd.pc.in-use-ROOTPREFIX-without-suffixed-slash.patch \ file://selinux-hook-handling-to-enumerate-nexthop.patch \ file://0001-logind-Restore-chvt-as-non-root-user-without-polkit.patch \ + file://0001-meson-Fix-reallocarray-check.patch \ " # patches needed by musl -- 2.29.2 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145841): https://lists.openembedded.org/g/openembedded-core/message/145841 Mute This Topic: https://lists.openembedded.org/mt/79054746/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [dunfell][PATCH] perl: fix installation failure because of shell issue
From: Dmitry Baryshkov
On one of my buildservers I noticed perl do_install failing with the
following message:
| rm: cannot remove
'/tmp-rpb-glibc/work/armv8-2a-linaro-linux/perl/5.32.0-r0/image//usr/lib/perl5/5.32.0/*/CORE/libperl.so':
No such file or directory
I tracked this down to shell being dash rather than bash not being able
to expand this glob in the middle of the filename. So replace the glob
expansion with the simpler one which works in all cases.
Signed-off-by: Dmitry Baryshkov
---
meta/recipes-devtools/perl/perl_5.30.1.bb | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-devtools/perl/perl_5.30.1.bb
b/meta/recipes-devtools/perl/perl_5.30.1.bb
index b53aff12160e..ee6eb6ef0f01 100644
--- a/meta/recipes-devtools/perl/perl_5.30.1.bb
+++ b/meta/recipes-devtools/perl/perl_5.30.1.bb
@@ -146,8 +146,9 @@ do_install() {
install lib/ExtUtils/typemap ${D}${libdir}/perl5/${PV}/ExtUtils/
# Fix up shared library
-rm ${D}/${libdir}/perl5/${PV}/*/CORE/libperl.so
-ln -sf ../../../../libperl.so.${PERL_LIB_VER} $(echo
${D}/${libdir}/perl5/${PV}/*/CORE)/libperl.so
+dir=$(echo ${D}/${libdir}/perl5/${PV}/*/CORE)
+rm $dir/libperl.so
+ln -sf ../../../../libperl.so.${PERL_LIB_VER} $dir/libperl.so
}
do_install_append_class-target() {
--
2.29.2
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145842):
https://lists.openembedded.org/g/openembedded-core/message/145842
Mute This Topic: https://lists.openembedded.org/mt/79054754/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] curl: Update 7.73.0 -> 7.74.0
From: Khairul Rohaizzat Jamaluddin
update to version 7.74.0
curl 7.74.0 hsts: add experimental support for Strict-Transport-Security with
various bug fixes
Reference:
https://curl.se/changes.html#7_74_0
update includes fix for CVE:
CVE-2020-8284
CVE-2020-8285
CVE-2020-8286
Signed-off-by: Khairul Rohaizzat Jamaluddin
---
meta/recipes-support/curl/{curl_7.73.0.bb => curl_7.74.0.bb} | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
rename meta/recipes-support/curl/{curl_7.73.0.bb => curl_7.74.0.bb} (97%)
diff --git a/meta/recipes-support/curl/curl_7.73.0.bb
b/meta/recipes-support/curl/curl_7.74.0.bb
similarity index 97%
rename from meta/recipes-support/curl/curl_7.73.0.bb
rename to meta/recipes-support/curl/curl_7.74.0.bb
index 0f26b0f..873bbe8 100644
--- a/meta/recipes-support/curl/curl_7.73.0.bb
+++ b/meta/recipes-support/curl/curl_7.74.0.bb
@@ -9,7 +9,7 @@ SRC_URI = "https://curl.haxx.se/download/curl-${PV}.tar.bz2 \
file://0001-replace-krb5-config-with-pkg-config.patch \
"
-SRC_URI[sha256sum] =
"cf34fe0b07b800f1c01a499a6e8b2af548f6d0e044dca4a29d88a4bee146d131"
+SRC_URI[sha256sum] =
"0f4d63e6681636539dc88fa8e929f934cd3a840c46e0bf28c73be11e521b77a5"
# Curl has used many names over the years...
CVE_PRODUCT = "haxx:curl haxx:libcurl curl:curl curl:libcurl libcurl:libcurl
daniel_stenberg:curl"
--
2.7.4
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145843):
https://lists.openembedded.org/g/openembedded-core/message/145843
Mute This Topic: https://lists.openembedded.org/mt/79054773/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] [dunfell][PATCH] systemd: Fix reallocarray check
Hello, пт, 18 дек. 2020 г. в 06:57, Dmitry Baryshkov : > > From: Khem Raj > > Musl brought this latent issue to fore > > Signed-off-by: Khem Raj > Signed-off-by: Richard Purdie Please ignore this email, it was sent by mistake. -- With best wishes Dmitry -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145844): https://lists.openembedded.org/g/openembedded-core/message/145844 Mute This Topic: https://lists.openembedded.org/mt/79054746/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] mesa: update 20.2.4 → 20.3.1
Two of the patches have been applied upstream:
0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch
0005-vc4-use-intmax_t-for-formatted-output-of-timespec-me.patch
The 'surfaceless' platform is now enabled by default, therefore it is no
longer a valid choice. The meson build system is intelligent enough now to
decide if 'drm' needs to be enabled based on whether or not gbm is available.
Therefore 'drm' also is no longer a valid platform choice:
ERROR: Options "drm, surfaceless" are not in allowed choices: "auto,
x11, wayland, haiku, android, windows"
Signed-off-by: Trevor Woerner
---
...xes-to-loader-when-using-x11-and-dri.patch | 36 -
...-for-formatted-output-of-timespec-me.patch | 51 ---
meta/recipes-graphics/mesa/mesa.inc | 8 +--
.../mesa/{mesa_20.2.4.bb => mesa_20.3.1.bb} | 0
4 files changed, 2 insertions(+), 93 deletions(-)
delete mode 100644
meta/recipes-graphics/mesa/files/0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch
delete mode 100644
meta/recipes-graphics/mesa/files/0005-vc4-use-intmax_t-for-formatted-output-of-timespec-me.patch
rename meta/recipes-graphics/mesa/{mesa_20.2.4.bb => mesa_20.3.1.bb} (100%)
diff --git
a/meta/recipes-graphics/mesa/files/0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch
b/meta/recipes-graphics/mesa/files/0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch
deleted file mode 100644
index 9ee72880a2..00
---
a/meta/recipes-graphics/mesa/files/0001-meson-Add-xcb-fixes-to-loader-when-using-x11-and-dri.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-From cf17d6251653f4a98e7c4f904ea2f0bc0ecedd5c Mon Sep 17 00:00:00 2001
-From: Duncan Hopkins
-Date: Thu, 15 Oct 2020 12:14:57 +0100
-Subject: [PATCH] meson: Add xcb-fixes to loader when using x11 and dri3. Fixes
- undefined symbol for xcb_xfixes_create_region in loader_dri3_helper.c
-
-loader_dr3_helper.c uses xcb_xfixes_create_region() that requires
dep_xcb_xfixes to link. This is dependent on with_platform_x11 and with_dri3.
-But the source meson file does not set this up dependent on with_dri3.
-The build was initialsed using platforms=x11 and gallium-drivers=zink,swrast.
-
-Reviewed-by: Eric Anholt
-Part-of:
-
-Upstream-Status: Backport [cf17d6251653f4a98e7c4f904ea2f0bc0ecedd5c]
-
- meson.build | 3 ++-
- 1 file changed, 2 insertions(+), 1 deletion(-)
-
-diff --git a/meson.build b/meson.build
-index cfe02fa6373..3cb3c904927 100644
a/meson.build
-+++ b/meson.build
-@@ -1782,7 +1782,8 @@ if with_platform_x11
- dep_xxf86vm = dependency('xxf86vm')
- endif
- endif
-- if (with_egl or (
-+ if (with_egl or
-+ with_dri3 or (
- with_gallium_vdpau or with_gallium_xvmc or with_gallium_xa or
- with_gallium_omx != 'disabled'))
- dep_xcb_xfixes = dependency('xcb-xfixes')
---
-2.17.1
-
diff --git
a/meta/recipes-graphics/mesa/files/0005-vc4-use-intmax_t-for-formatted-output-of-timespec-me.patch
b/meta/recipes-graphics/mesa/files/0005-vc4-use-intmax_t-for-formatted-output-of-timespec-me.patch
deleted file mode 100644
index dacb1ea1c8..00
---
a/meta/recipes-graphics/mesa/files/0005-vc4-use-intmax_t-for-formatted-output-of-timespec-me.patch
+++ /dev/null
@@ -1,51 +0,0 @@
-From 281a636353666bfdd373c62591e744087e750e89 Mon Sep 17 00:00:00 2001
-From: Khem Raj
-Date: Wed, 4 Dec 2019 14:15:28 -0800
-Subject: [PATCH] vc4: use intmax_t for formatted output of timespec members
-
-32bit architectures which have 64bit time_t does not fit the assumption
-of time_t being same as system long int
-
-Fixes
-error: format specifies type 'long' but the argument has type 'time_t' (aka
'long long') [-Werror,-Wformat]
-time.tv_sec);
-^~~
-
-Upstream-Status: Submitted
[https://gitlab.freedesktop.org/mesa/mesa/merge_requests/2966]
-Signed-off-by: Khem Raj
-
- src/gallium/drivers/v3d/v3d_bufmgr.c | 4 ++--
- src/gallium/drivers/vc4/vc4_bufmgr.c | 4 ++--
- 2 files changed, 4 insertions(+), 4 deletions(-)
-
-diff --git a/src/gallium/drivers/v3d/v3d_bufmgr.c
b/src/gallium/drivers/v3d/v3d_bufmgr.c
-index 31a0803..cc2e2af 100644
a/src/gallium/drivers/v3d/v3d_bufmgr.c
-+++ b/src/gallium/drivers/v3d/v3d_bufmgr.c
-@@ -80,8 +80,8 @@ v3d_bo_dump_stats(struct v3d_screen *screen)
-
- struct timespec time;
- clock_gettime(CLOCK_MONOTONIC, &time);
--fprintf(stderr, " now: %ld\n",
--(long)time.tv_sec);
-+fprintf(stderr, " now: %jd\n",
-+(intmax_t)time.tv_sec);
- }
- }
-
-diff --git a/src/gallium/drivers/vc4/vc4_bufmgr.c
b/src/gallium/drivers/vc4/vc4_bufmgr.c
-index a786e8e..975d49e 100644
a/src/gallium/drivers/vc4/vc4_bufmgr.c
-+++ b/src/gallium/drivers/vc4/vc4_bufmgr.c
-@@ -99,8 +99,8 @@ vc4_bo_dum
[OE-core] [PATCH] packagegroup-core-base-utils.bb: check the gplv3 license
From: Mingli Yu
packagegroup-core-base-utils repends on some packages which are
gplv3 compatilbe.
Signed-off-by: Mingli Yu
---
.../packagegroups/packagegroup-core-base-utils.bb | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git
a/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
index 1e63da7f16..0a3cb63bc2 100644
--- a/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
+++ b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
@@ -33,18 +33,18 @@ RDEPENDS_${PN} = "\
grep \
gzip \
${@bb.utils.contains("DISTRO_FEATURES", "systemd", "", "ifupdown", d)} \
-inetutils \
-inetutils-ping \
-inetutils-telnet \
-inetutils-tftp \
-inetutils-traceroute \
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 'inetutils', d)}
\
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
'inetutils-ping', d)} \
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
'inetutils-telnet', d)} \
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
'inetutils-tftp', d)} \
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
'inetutils-traceroute', d)} \
iproute2 \
${@bb.utils.contains("MACHINE_FEATURES", "keyboard", "kbd", "", d)} \
kmod \
less \
ncurses-tools \
net-tools \
-parted \
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 'parted', d)} \
patch \
procps \
psmisc \
@@ -55,7 +55,7 @@ RDEPENDS_${PN} = "\
unzip \
util-linux \
${VIRTUAL-RUNTIME_vim} \
-wget \
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 'wget', d)} \
which \
xz \
"
--
2.26.2
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145846):
https://lists.openembedded.org/g/openembedded-core/message/145846
Mute This Topic: https://lists.openembedded.org/mt/79055799/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 3/4] busybox: Run mdev as daemon
On Thu, Dec 17, 2020 at 7:52 PM Khem Raj wrote:
>
> On Thu, Dec 17, 2020 at 3:59 PM Andre McCurdy wrote:
> >
> > On Thu, Dec 17, 2020 at 2:54 PM Khem Raj wrote:
> > >
> > > When busybox is used for device management, kernel needs to support
> > > older/obsolete mechanism via CONFIG_UEVENT_HELPER and
> > > CONFIG_UEVENT_HELPER_PATH to enable /proc/sys/kernel/hotplug but this
> > > would require kernel defconfig change and will always be needed when
> > > mdev is used, intead run it in daemon mode
> > >
> > > Update mdev init script to run mdev in daemon mode
> > >
> > > Signed-off-by: Khem Raj
> > > ---
> > > meta/recipes-core/busybox/busybox/mdev.cfg | 2 +
> > > meta/recipes-core/busybox/files/mdev | 56 +++---
> > > 2 files changed, 41 insertions(+), 17 deletions(-)
> > >
> > > diff --git a/meta/recipes-core/busybox/busybox/mdev.cfg
> > > b/meta/recipes-core/busybox/busybox/mdev.cfg
> > > index 6aefe90e43..143e6097cb 100644
> > > --- a/meta/recipes-core/busybox/busybox/mdev.cfg
> > > +++ b/meta/recipes-core/busybox/busybox/mdev.cfg
> > > @@ -9,3 +9,5 @@ CONFIG_SETSID=y
> > > CONFIG_CTTYHACK=y
> > >
> > > CONFIG_FEATURE_SHADOWPASSWDS=y
> > > +CONFIG_FEATURE_XARGS_SUPPORT_ZERO_TERM=y
> > > +CONFIG_FEATURE_MDEV_DAEMON=y
> > > diff --git a/meta/recipes-core/busybox/files/mdev
> > > b/meta/recipes-core/busybox/files/mdev
> > > index 8c9c06e96c..2fbdfb073e 100755
> > > --- a/meta/recipes-core/busybox/files/mdev
> > > +++ b/meta/recipes-core/busybox/files/mdev
> > > @@ -1,21 +1,43 @@
> > > #!/bin/sh
> > > -mount -t proc proc /proc
> > > -mount -t sysfs sysfs /sys
> > > -mount -t tmpfs tmpfs /dev -o size=64k,mode=0755
> > > -mkdir /dev/pts /dev/shm
> > > -chmod 777 /dev/shm
> > > -mount -t devpts devpts /dev/pts
> > > -touch /dev/mdev.seq
> > > -#sysctl -w kernel.hotplug=/sbin/mdev
> > > -echo "/sbin/mdev" > /proc/sys/kernel/hotplug
> > > -mdev -s
> > > -
> > > #
> > > -# We might have mounted something over /dev, see if /dev/initctl is
> > > there.
> > > +# Run the mdev daemon
> > > #
> > > -if test ! -p /dev/initctl
> > > -then
> > > -rm -f /dev/initctl
> > > -mknod -m 600 /dev/initctl p
> > > -fi
> > > +
> > > +DAEMON="mdev"
> > > +PIDFILE="/var/run/$DAEMON.pid"
> > > +
> > > +
> > > +start() {
> > > + echo -n "Starting $DAEMON... "
> > > + start-stop-daemon -S -b -m -p $PIDFILE -x /sbin/mdev -- -df
> >
> > Where do these start-stop-daemon options come from? Using -b for an
> > application which is designed to run as a daemon (and deliberately
> > telling that app to run in the foreground) looks odd, etc.
> >
> > Are there bugs or limitations in mdev which you are trying to workaround?
>
> its using start-stop-daemon applet from busybox which does have these options
> implemented, its so that start-stop-daemon takes the control of the daemon
The question is not whether start-stop-daemon supports using PID files
but whether the busybox mdev application supports creating them.
It looks like although most of the busybox daemons do, mdev does not:
.../busybox-1.32.0 $ grep -r write_pidfile *
debianutils/start_stop_daemon.c:write_pidfile(pidfile);
include/libbb.h:void write_pidfile(const char *path) FAST_FUNC;
include/libbb.h:void write_pidfile_std_path_and_ext(const char *path) FAST_FUNC;
include/libbb.h:#define write_pidfile_std_path_and_ext(path) ((void)0)
include/libbb.h:#define write_pidfile(path) ((void)0)
libbb/pidfile.c:void FAST_FUNC write_pidfile(const char *path)
libbb/pidfile.c:void FAST_FUNC write_pidfile_std_path_and_ext(const char *name)
libbb/pidfile.c:write_pidfile(buf);
miscutils/watchdog.c:write_pidfile_std_path_and_ext("watchdog");
miscutils/crond.c:write_pidfile_std_path_and_ext("crond");
networking/ifplugd.c:write_pidfile(pidfile_name);
networking/ntpd.c:write_pidfile_std_path_and_ext("ntpd");
networking/inetd.c:write_pidfile_std_path_and_ext("inetd");
networking/udhcp/dhcpc.c:write_pidfile(client_data.pidfile);
networking/udhcp/dhcpc.c:write_pidfile(client_data.pidfile);
networking/udhcp/dhcpd.c:write_pidfile(server_data.pidfile);
networking/udhcp/d6_dhcpc.c:write_pidfile(client_data.pidfile);
networking/udhcp/d6_dhcpc.c:write_pidfile(client_data.pidfile);
sysklogd/syslogd.c:write_pidfile_std_path_and_ext("syslogd");
sysklogd/klogd.c:write_pidfile_std_path_and_ext("klogd");
util-linux/acpid.c:write_pidfile(opt_pidfile);
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145847):
https://lists.openembedded.org/g/openembedded-core/message/145847
Mute This Topic: https://lists.openembedded.org/mt/79049549/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] packagegroup-core-base-utils.bb: check the gplv3 license
I’m not sure I like this approach. It doesn’t look like the right place to
decide whether to include or not some packages based on specific licensing
requirements, licenses for these and other items change over time, and
there may be more undesirable licenses than just this one.
Why not simply create a new package group?
Alex
On Fri 18. Dec 2020 at 6.37, Yu, Mingli wrote:
> From: Mingli Yu
>
> packagegroup-core-base-utils repends on some packages which are
> gplv3 compatilbe.
>
> Signed-off-by: Mingli Yu
> ---
> .../packagegroups/packagegroup-core-base-utils.bb | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/meta/recipes-extended/packagegroups/
> packagegroup-core-base-utils.bb b/meta/recipes-extended/packagegroups/
> packagegroup-core-base-utils.bb
> index 1e63da7f16..0a3cb63bc2 100644
> --- a/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
> +++ b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
> @@ -33,18 +33,18 @@ RDEPENDS_${PN} = "\
> grep \
> gzip \
> ${@bb.utils.contains("DISTRO_FEATURES", "systemd", "", "ifupdown",
> d)} \
> -inetutils \
> -inetutils-ping \
> -inetutils-telnet \
> -inetutils-tftp \
> -inetutils-traceroute \
> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
> 'inetutils', d)} \
> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
> 'inetutils-ping', d)} \
> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
> 'inetutils-telnet', d)} \
> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
> 'inetutils-tftp', d)} \
> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
> 'inetutils-traceroute', d)} \
> iproute2 \
> ${@bb.utils.contains("MACHINE_FEATURES", "keyboard", "kbd", "", d)} \
> kmod \
> less \
> ncurses-tools \
> net-tools \
> -parted \
> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 'parted',
> d)} \
> patch \
> procps \
> psmisc \
> @@ -55,7 +55,7 @@ RDEPENDS_${PN} = "\
> unzip \
> util-linux \
> ${VIRTUAL-RUNTIME_vim} \
> -wget \
> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 'wget', d)}
> \
> which \
> xz \
> "
> --
> 2.26.2
>
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145848):
https://lists.openembedded.org/g/openembedded-core/message/145848
Mute This Topic: https://lists.openembedded.org/mt/79055799/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
