Re: [OE-core] [PATCH] icewm:include imlib2-loaders package

2022-04-11 Thread Martin Jansa 
Or just add it in the line above and save bitbake some work with the append.

On Tue, Apr 12, 2022 at 8:44 AM Martin Jansa via lists.openembedded.org
 wrote:

> wrong ML and you probably wanted RDEPENDS:${PN}:append not
> RDEPENDS:append:${PN}
>
> On Tue, Apr 12, 2022 at 8:40 AM  wrote:
>
>> From: KARN JYE LAU 
>>
>> icewm 2.0.0 above have use imlib2 as default
>> rendering engine.Update icewm  recipe to include
>> the essential packages for icewm to work properly.
>>
>> Signed-off-by: KARN JYE LAU 
>> ---
>>  meta-oe/recipes-extended/icewm/icewm_2.9.0.bb | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
>> b/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
>> index dac025de7..f28ab2fa0 100644
>> --- a/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
>> +++ b/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
>> @@ -24,6 +24,7 @@ DEPENDS = "asciidoc-native fontconfig fribidi
>> gdk-pixbuf imlib2   libxft libxpm li
>>  libxfixes"
>>  DEPENDS:append = " qemu-native"
>>  RDEPENDS:${PN} = "perl fribidi"
>> +RDEPENDS:append:${PN} = " imlib2 imlib2-loaders"
>>
>>  do_compile:prepend:class-target() {
>>
>> --
>> 2.25.1
>>
>>
>>
>>
>>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164254): 
https://lists.openembedded.org/g/openembedded-core/message/164254
Mute This Topic: https://lists.openembedded.org/mt/90413099/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] icewm:include imlib2-loaders package

2022-04-11 Thread Martin Jansa 
wrong ML and you probably wanted RDEPENDS:${PN}:append not
RDEPENDS:append:${PN}

On Tue, Apr 12, 2022 at 8:40 AM  wrote:

> From: KARN JYE LAU 
>
> icewm 2.0.0 above have use imlib2 as default
> rendering engine.Update icewm  recipe to include
> the essential packages for icewm to work properly.
>
> Signed-off-by: KARN JYE LAU 
> ---
>  meta-oe/recipes-extended/icewm/icewm_2.9.0.bb | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
> b/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
> index dac025de7..f28ab2fa0 100644
> --- a/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
> +++ b/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
> @@ -24,6 +24,7 @@ DEPENDS = "asciidoc-native fontconfig fribidi gdk-pixbuf
> imlib2   libxft libxpm li
>  libxfixes"
>  DEPENDS:append = " qemu-native"
>  RDEPENDS:${PN} = "perl fribidi"
> +RDEPENDS:append:${PN} = " imlib2 imlib2-loaders"
>
>  do_compile:prepend:class-target() {
>
> --
> 2.25.1
>
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164253): 
https://lists.openembedded.org/g/openembedded-core/message/164253
Mute This Topic: https://lists.openembedded.org/mt/90413099/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] icewm:include imlib2-loaders package

2022-04-11 Thread karn . jye . lau 
From: KARN JYE LAU 

icewm 2.0.0 above have use imlib2 as default
rendering engine.Update icewm  recipe to include
the essential packages for icewm to work properly.

Signed-off-by: KARN JYE LAU 
---
 meta-oe/recipes-extended/icewm/icewm_2.9.0.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb 
b/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
index dac025de7..f28ab2fa0 100644
--- a/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
+++ b/meta-oe/recipes-extended/icewm/icewm_2.9.0.bb
@@ -24,6 +24,7 @@ DEPENDS = "asciidoc-native fontconfig fribidi gdk-pixbuf 
imlib2   libxft libxpm li
 libxfixes"
 DEPENDS:append = " qemu-native"
 RDEPENDS:${PN} = "perl fribidi"
+RDEPENDS:append:${PN} = " imlib2 imlib2-loaders"
 
 do_compile:prepend:class-target() {
 
-- 
2.25.1


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

[OE-core] [PATCH 2/2] ccache: Fix build with gcc12/mips

2022-04-11 Thread Khem Raj 
Avoid gcc12 failure to inline function on mips

Signed-off-by: Khem Raj 
---
 ...t-Do-not-use-always_inline-with-mips.patch | 33 +++
 meta/recipes-devtools/ccache/ccache_4.6.bb|  5 ++-
 2 files changed, 37 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-devtools/ccache/ccache/0001-fmt-Do-not-use-always_inline-with-mips.patch

diff --git 
a/meta/recipes-devtools/ccache/ccache/0001-fmt-Do-not-use-always_inline-with-mips.patch
 
b/meta/recipes-devtools/ccache/ccache/0001-fmt-Do-not-use-always_inline-with-mips.patch
new file mode 100644
index 000..90cf27b4187
--- /dev/null
+++ 
b/meta/recipes-devtools/ccache/ccache/0001-fmt-Do-not-use-always_inline-with-mips.patch
@@ -0,0 +1,33 @@
+From e57ce7af6f1ef0400aa847d79a7d3bc8803838bd Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Mon, 11 Apr 2022 20:09:18 -0700
+Subject: [PATCH] fmt: Do not use always_inline with mips
+
+mips/gcc-12 is not able to inline this function as reported here [1]
+For now workaround this for mips by not forcing always_inline attribute
+on functions
+
+[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105234
+
+Upstream-Status: Inappropriate [GCC MIPS Workaround]
+Signed-off-by: Khem Raj 
+---
+ src/third_party/fmt/core.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/third_party/fmt/core.h b/src/third_party/fmt/core.h
+index 92a7aa1..e343ae0 100644
+--- a/src/third_party/fmt/core.h
 b/src/third_party/fmt/core.h
+@@ -212,7 +212,7 @@
+ #endif
+ 
+ #ifndef FMT_INLINE
+-#  if FMT_GCC_VERSION || FMT_CLANG_VERSION
++#  if (FMT_GCC_VERSION || FMT_CLANG_VERSION) && !defined (__mips__)
+ #define FMT_INLINE inline __attribute__((always_inline))
+ #  else
+ #define FMT_INLINE inline
+-- 
+2.35.1
+
diff --git a/meta/recipes-devtools/ccache/ccache_4.6.bb 
b/meta/recipes-devtools/ccache/ccache_4.6.bb
index f019679cf18..983b62c1359 100644
--- a/meta/recipes-devtools/ccache/ccache_4.6.bb
+++ b/meta/recipes-devtools/ccache/ccache_4.6.bb
@@ -11,7 +11,9 @@ LIC_FILES_CHKSUM = 
"file://LICENSE.adoc;md5=ff5327dc93e2b286c931dda3d6079da9"
 
 DEPENDS = "zstd"
 
-SRC_URI = 
"https://github.com/ccache/ccache/releases/download/v${PV}/${BP}.tar.gz";
+SRC_URI = 
"https://github.com/ccache/ccache/releases/download/v${PV}/${BP}.tar.gz \
+   file://0001-fmt-Do-not-use-always_inline-with-mips.patch \
+  "
 SRC_URI[sha256sum] = 
"73a1767ac6b7c0404a1a55f761a746d338e702883c7137fbf587023062258625"
 
 UPSTREAM_CHECK_URI = "https://github.com/ccache/ccache/releases/";
@@ -24,3 +26,4 @@ BBCLASSEXTEND = "native nativesdk"
 
 PACKAGECONFIG[docs] = 
"-DENABLE_DOCUMENTATION=ON,-DENABLE_DOCUMENTATION=OFF,asciidoc"
 PACKAGECONFIG[redis] = 
"-DREDIS_STORAGE_BACKEND=ON,-DREDIS_STORAGE_BACKEND=OFF,hiredis"
+
-- 
2.35.1


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

[OE-core] [PATCH 1/2] gnupg: Disable FORTIFY_SOURCES on mips

2022-04-11 Thread Khem Raj 
This is a workaround for issue reported with gcc12 [1]

[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105234

Signed-off-by: Khem Raj 
---
 meta/recipes-support/gnupg/gnupg_2.3.4.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-support/gnupg/gnupg_2.3.4.bb 
b/meta/recipes-support/gnupg/gnupg_2.3.4.bb
index 81361d902ce..d27bddb8bd9 100644
--- a/meta/recipes-support/gnupg/gnupg_2.3.4.bb
+++ b/meta/recipes-support/gnupg/gnupg_2.3.4.bb
@@ -83,3 +83,6 @@ PACKAGECONFIG[gnutls] = "--enable-gnutls, --disable-gnutls, 
gnutls"
 PACKAGECONFIG[sqlite3] = "--enable-sqlite, --disable-sqlite, sqlite3"
 
 BBCLASSEXTEND = "native nativesdk"
+
+lcl_maybe_fortify:mipsarch = ""
+
-- 
2.35.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164250): 
https://lists.openembedded.org/g/openembedded-core/message/164250
Mute This Topic: https://lists.openembedded.org/mt/90411447/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][dunfell] zlib: backport the fix for CVE-2018-25032

2022-04-11 Thread Steve Sakoman 
On Mon, Apr 11, 2022, 3:33 PM Ralph Siemsen 
wrote:

> On Mon, Apr 11, 2022 at 6:43 PM Steve Sakoman  wrote:
> >
> > I was using qemux86-64 for my experiment, so I guess I need to redo it
> > now using the correct machine!
>
> I have rebuild with MACHINE=qemux86. Same result, all tests appear to pass:
>
> RESULTS - apt.AptRepoTest.test_apt_install_from_repo: PASSED (44.80s)
> ...
> SUMMARY:
> core-image-sato () - Ran 64 tests in 61.570s
> core-image-sato - OK - All required tests passed (successes=15,
> skipped=49, failures=0, errors=0)
>

Same here. But I just realized something that is different from the
autobuilder.

What distro is your build machine? On the autobuilder it is using a non
debian distro for the debian build.

My build machine is Ubuntu, so that is a major difference!

Steve

>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164249): 
https://lists.openembedded.org/g/openembedded-core/message/164249
Mute This Topic: https://lists.openembedded.org/mt/90107518/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][dunfell] zlib: backport the fix for CVE-2018-25032

2022-04-11 Thread Ralph Siemsen 
On Mon, Apr 11, 2022 at 6:43 PM Steve Sakoman  wrote:
>
> I was using qemux86-64 for my experiment, so I guess I need to redo it
> now using the correct machine!

I have rebuild with MACHINE=qemux86. Same result, all tests appear to pass:

RESULTS - apt.AptRepoTest.test_apt_install_from_repo: PASSED (44.80s)
...
SUMMARY:
core-image-sato () - Ran 64 tests in 61.570s
core-image-sato - OK - All required tests passed (successes=15,
skipped=49, failures=0, errors=0)

Ralph

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164248): 
https://lists.openembedded.org/g/openembedded-core/message/164248
Mute This Topic: https://lists.openembedded.org/mt/90107518/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/3] systemtap: Fix build with gcc-12

2022-04-11 Thread Khem Raj 
Backport a patch to fix
| ../git/util.cxx:1766:56: error: 'std::pointer_to_unary_function<_Arg, 
_Result> std::ptr_fun(_Result (*)(_Arg)) [with _Arg = int; _Result = int]' is 
deprecated: use 'std::function' instea
d [-Werror=deprecated-declarations]
|  1766 |std::not1(std::ptr_fun(std::isspace;
|   |  ~~^~

Signed-off-by: Khem Raj 
---
 ...ility-re-tweak-for-rhel6-use-functio.patch | 49 +++
 .../recipes-kernel/systemtap/systemtap_git.bb |  3 +-
 2 files changed, 51 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-kernel/systemtap/systemtap/0001-gcc12-c-compatibility-re-tweak-for-rhel6-use-functio.patch

diff --git 
a/meta/recipes-kernel/systemtap/systemtap/0001-gcc12-c-compatibility-re-tweak-for-rhel6-use-functio.patch
 
b/meta/recipes-kernel/systemtap/systemtap/0001-gcc12-c-compatibility-re-tweak-for-rhel6-use-functio.patch
new file mode 100644
index 000..f885c444609
--- /dev/null
+++ 
b/meta/recipes-kernel/systemtap/systemtap/0001-gcc12-c-compatibility-re-tweak-for-rhel6-use-functio.patch
@@ -0,0 +1,49 @@
+From f199d1982ef8a6c6d5c06c082d057b8793bcc6aa Mon Sep 17 00:00:00 2001
+From: Serhei Makarov 
+Date: Fri, 21 Jan 2022 18:21:46 -0500
+Subject: [PATCH] gcc12 c++ compatibility re-tweak for rhel6: use function
+ pointer instead of lambdas instead of ptr_fun<>
+
+Saving 2 lines in ltrim/rtrim is probably not a good reason to drop
+compatibility with the RHEL6 system compiler.  Actually declaring a
+named function and passing the function pointer is compatible with
+everything.
+
+Upstream-Status: Backport 
[https://sourceware.org/git/?p=systemtap.git;a=commit;h=f199d1982ef8a6c6d5c06c082d057b8793bcc6aa]
+Signed-off-by: Khem Raj 
+---
+ util.cxx | 13 -
+ 1 file changed, 8 insertions(+), 5 deletions(-)
+
+--- a/util.cxx
 b/util.cxx
+@@ -1757,21 +1757,24 @@ flush_to_stream (const string &fname, os
+   return 1; // Failure
+ }
+ 
++int
++not_isspace(unsigned char c)
++{
++  return !std::isspace(c);
++}
++
+ // trim from start (in place)
+ void
+ ltrim(std::string &s)
+ {
+-  s.erase(s.begin(),
+-std::find_if(s.begin(), s.end(),
+- std::not1(std::ptr_fun(std::isspace;
++  s.erase(s.begin(), std::find_if(s.begin(), s.end(), not_isspace));
+ }
+ 
+ // trim from end (in place)
+ void
+ rtrim(std::string &s)
+ {
+-  s.erase(std::find_if(s.rbegin(), s.rend(),
+-std::not1(std::ptr_fun(std::isspace))).base(), s.end());
++  s.erase(std::find_if(s.rbegin(), s.rend(), not_isspace).base(), s.end());
+ }
+ 
+ // trim from both ends (in place)
diff --git a/meta/recipes-kernel/systemtap/systemtap_git.bb 
b/meta/recipes-kernel/systemtap/systemtap_git.bb
index 04f7e03ab50..ce86d5274dd 100644
--- a/meta/recipes-kernel/systemtap/systemtap_git.bb
+++ b/meta/recipes-kernel/systemtap/systemtap_git.bb
@@ -7,7 +7,8 @@ HOMEPAGE = "https://sourceware.org/systemtap/";
 require systemtap_git.inc
 
 SRC_URI += "file://0001-improve-reproducibility-for-c-compiling.patch \
-   file://0001-staprun-address-ncurses-6.3-failures.patch \
+file://0001-staprun-address-ncurses-6.3-failures.patch \
+
file://0001-gcc12-c-compatibility-re-tweak-for-rhel6-use-functio.patch \
"
 
 DEPENDS = "elfutils"
-- 
2.35.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164247): 
https://lists.openembedded.org/g/openembedded-core/message/164247
Mute This Topic: https://lists.openembedded.org/mt/90409336/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/3] xserver-xorg: Fix build with gcc12

2022-04-11 Thread Khem Raj 
backport patch to silence new array-bounds warnings

Signed-off-by: Khem Raj 
---
 .../0001-render-Fix-build-with-gcc-12.patch   | 90 +++
 .../xorg-xserver/xserver-xorg_21.1.3.bb   |  1 +
 2 files changed, 91 insertions(+)
 create mode 100644 
meta/recipes-graphics/xorg-xserver/xserver-xorg/0001-render-Fix-build-with-gcc-12.patch

diff --git 
a/meta/recipes-graphics/xorg-xserver/xserver-xorg/0001-render-Fix-build-with-gcc-12.patch
 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/0001-render-Fix-build-with-gcc-12.patch
new file mode 100644
index 000..df9332fae7e
--- /dev/null
+++ 
b/meta/recipes-graphics/xorg-xserver/xserver-xorg/0001-render-Fix-build-with-gcc-12.patch
@@ -0,0 +1,90 @@
+From 12041ad0610f1345d6b9994c32943fd4dd01f65d Mon Sep 17 00:00:00 2001
+From: Olivier Fourdan 
+Date: Thu, 20 Jan 2022 10:20:38 +0100
+Subject: [PATCH] render: Fix build with gcc 12
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+The xserver fails to compile with the latest gcc 12:
+
+ render/picture.c: In function ‘CreateSolidPicture’:
+ render/picture.c:874:26: error: array subscript ‘union _SourcePict[0]’ is 
partly outside array bounds of ‘unsigned char[16]’ [-Werror=array-bounds]
+  874 | pPicture->pSourcePict->type = SourcePictTypeSolidFill;
+  |  ^~
+ render/picture.c:868:45: note: object of size 16 allocated by ‘malloc’
+  868 | pPicture->pSourcePict = (SourcePictPtr) 
malloc(sizeof(PictSolidFill));
+  | 
^
+ render/picture.c: In function ‘CreateLinearGradientPicture’:
+ render/picture.c:906:26: error: array subscript ‘union _SourcePict[0]’ is 
partly outside array bounds of ‘unsigned char[32]’ [-Werror=array-bounds]
+  906 | pPicture->pSourcePict->linear.type = SourcePictTypeLinear;
+  |  ^~
+ render/picture.c:899:45: note: object of size 32 allocated by ‘malloc’
+  899 | pPicture->pSourcePict = (SourcePictPtr) 
malloc(sizeof(PictLinearGradient));
+  | 
^~
+ render/picture.c: In function ‘CreateConicalGradientPicture’:
+ render/picture.c:989:26: error: array subscript ‘union _SourcePict[0]’ is 
partly outside array bounds of ‘unsigned char[32]’ [-Werror=array-bounds]
+  989 | pPicture->pSourcePict->conical.type = SourcePictTypeConical;
+  |  ^~
+ render/picture.c:982:45: note: object of size 32 allocated by ‘malloc’
+  982 | pPicture->pSourcePict = (SourcePictPtr) 
malloc(sizeof(PictConicalGradient));
+  | 
^~~
+ cc1: some warnings being treated as errors
+ ninja: build stopped: subcommand failed.
+
+This is because gcc 12 has become stricter and raises a warning now.
+
+Fix the warning/error by allocating enough memory to store the union
+struct.
+
+Upstream-Status: Backport 
[https://gitlab.freedesktop.org/xorg/xserver/-/commit/c6b0dcb82d4db07a2f32c09a8c09c85a5f57248e]
+Signed-off-by: Olivier Fourdan 
+Acked-by: Michel Dänzer 
+Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1256
+---
+ render/picture.c | 8 
+ 1 file changed, 4 insertions(+), 4 deletions(-)
+
+diff --git a/render/picture.c b/render/picture.c
+index afa0d25..2be4b19 100644
+--- a/render/picture.c
 b/render/picture.c
+@@ -865,7 +865,7 @@ CreateSolidPicture(Picture pid, xRenderColor * color, int 
*error)
+ }
+ 
+ pPicture->id = pid;
+-pPicture->pSourcePict = (SourcePictPtr) malloc(sizeof(PictSolidFill));
++pPicture->pSourcePict = (SourcePictPtr) malloc(sizeof(SourcePict));
+ if (!pPicture->pSourcePict) {
+ *error = BadAlloc;
+ free(pPicture);
+@@ -896,7 +896,7 @@ CreateLinearGradientPicture(Picture pid, xPointFixed * p1, 
xPointFixed * p2,
+ }
+ 
+ pPicture->id = pid;
+-pPicture->pSourcePict = (SourcePictPtr) 
malloc(sizeof(PictLinearGradient));
++pPicture->pSourcePict = (SourcePictPtr) malloc(sizeof(SourcePict));
+ if (!pPicture->pSourcePict) {
+ *error = BadAlloc;
+ free(pPicture);
+@@ -936,7 +936,7 @@ CreateRadialGradientPicture(Picture pid, xPointFixed * 
inner,
+ }
+ 
+ pPicture->id = pid;
+-pPicture->pSourcePict = (SourcePictPtr) 
malloc(sizeof(PictRadialGradient));
++pPicture->pSourcePict = (SourcePictPtr) malloc(sizeof(SourcePict));
+ if (!pPicture->pSourcePict) {
+ *error = BadAlloc;
+ free(pPicture);
+@@ -979,7 +979,7 @@ CreateConicalGradientPicture(Picture pid, xPointFixed * 
center, xFixed angle,
+ }
+ 
+ pPicture->id = pid;
+-pPicture->pSourcePict = (SourcePictPtr) 
malloc(sizeof(PictConicalGradient));
++pPicture->pSourcePict = (SourcePictPtr) malloc(sizeof(SourcePict));
+ if (!pPicture->pSourcePict) {
+ *error = BadAlloc;
+ free(pPicture);
+-- 
+2.35.1
+
d

[OE-core] [PATCH 1/3] babeltrace: Disable warnings as errors

2022-04-11 Thread Khem Raj 
These settings are good for developers/maintainers but for distributions
generally disabling them turns out to be better especially when there is
a knob to do so. This fixes build with gcc-12 which find additional
warnings

inlined from 'bt_ctf_object_set_parent' at 
../../../git/src/ctf-writer/object.h:120:6,
inlined from 'bt_ctf_trace_common_add_stream_class' at 
../../../git/src/ctf-writer/trace.c:1243:3:
../../../git/src/ctf-writer/object.h:141:26: error: null pointer dereference 
[-Werror=null-dereference]
  141 | if (child->parent) {
  | ~^~~~
../../../git/src/ctf-writer/object.h:141:26: error: null pointer dereference 
[-Werror=null-dereference]
cc1: all warnings being treated as errors

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

diff --git a/meta/recipes-kernel/lttng/babeltrace2_2.0.4.bb 
b/meta/recipes-kernel/lttng/babeltrace2_2.0.4.bb
index 248b84e98d3..b48f07ea0df 100644
--- a/meta/recipes-kernel/lttng/babeltrace2_2.0.4.bb
+++ b/meta/recipes-kernel/lttng/babeltrace2_2.0.4.bb
@@ -19,7 +19,7 @@ S = "${WORKDIR}/git"
 
 inherit autotools pkgconfig ptest python3targetconfig
 
-EXTRA_OECONF = "--disable-debug-info"
+EXTRA_OECONF = "--disable-debug-info --disable-Werror"
 
 PACKAGECONFIG ??= "manpages"
 PACKAGECONFIG[manpages] = ", --disable-man-pages, asciidoc-native xmlto-native"
-- 
2.35.1


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

[OE-core] Yocto Project Newcomer & Unassigned Bugs - Help Needed

2022-04-11 Thread Stephen Jolley 
All,

 

The triage team is starting to try and collect up and classify bugs which a
newcomer to the project would be able to work on in a way which means people
can find them. They're being listed on the triage page under the appropriate
heading:

https://wiki.yoctoproject.org/wiki/Bug_Triage#Newcomer_Bugs  Also please
review:
https://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded and
how to create a bugzilla account at:

https://bugzilla.yoctoproject.org/createaccount.cgi

The idea is these bugs should be straight forward for a person to help work
on who doesn't have deep experience with the project.  If anyone can help,
please take ownership of the bug and send patches!  If anyone needs
help/advice there are people on irc who can likely do so, or some of the
more experienced contributors will likely be happy to help too.

 

Also, the triage team meets weekly and does its best to handle the bugs
reported into the Bugzilla. The number of people attending that meeting has
fallen, as have the number of people available to help fix bugs. One of the
things we hear users report is they don't know how to help. We (the triage
team) are therefore going to start reporting out the currently 410
unassigned or newcomer bugs.

 

We're hoping people may be able to spare some time now and again to help out
with these.  Bugs are split into two types, "true bugs" where things don't
work as they should and "enhancements" which are features we'd want to add
to the system.  There are also roughly four different "priority" classes
right now,  "3.5, "3.6", "3.99" and "Future", the more pressing/urgent
issues being in "3.5" and then "3.6".

 

Please review this link and if a bug is something you would be able to help
with either take ownership of the bug, or send me (sjolley.yp...@gmail.com
 ) an e-mail with the bug number you would
like and I will assign it to you (please make sure you have a Bugzilla
account).  The list is at:
https://wiki.yoctoproject.org/wiki/Bug_Triage_Archive#Unassigned_or_Newcomer
_Bugs

 

Thanks,

 

Stephen K. Jolley

Yocto Project Program Manager

*Cell:(208) 244-4460

* Email:  sjolley.yp...@gmail.com
 

 


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164244): 
https://lists.openembedded.org/g/openembedded-core/message/164244
Mute This Topic: https://lists.openembedded.org/mt/90408240/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][dunfell] zlib: backport the fix for CVE-2018-25032

2022-04-11 Thread Steve Sakoman 
On Mon, Apr 11, 2022 at 12:30 PM Ralph Siemsen  wrote:
>
> On Mon, Apr 11, 2022 at 2:58 PM Steve Sakoman  wrote:
>
> > Let me know if you make any progress!
>
> After a clean build, and messing about with VNC, I am now able to
> 'bitbake core-image-sato:do_testimage'.
>
> > Yes, either change will trigger the error.  Without the zlib or xz
> > patches all is fine.  Adding just one or the other will fail the
> > pkgman-deb-non-deb builds (and probably the pkgman-rpm-non-rpm builds
> > too, though I've been concentrating on the deb version to try to
> > isolate the offending patches)
>
> I have both gzip and xz patches applied:
> $ git log --oneline
> 52269ab7e7 (HEAD -> dunfell) gzip: fix CVE-2022-1271
> e7fa7a2a2c xz: fix CVE-2022-1271
> 38c55bd388 (origin/dunfell) tzdata: update to 2022a
> 86285152bd python3-jinja2: Correct HOMEPAGE
>
> With default PACKAGE_CLASSES = package_rpm
> RESULTS - dnf.DnfBasicTest.test_dnf_help: PASSED (8.77s)
> ...
> SUMMARY:
> core-image-sato () - Ran 64 tests in 242.039s
> core-image-sato - OK - All required tests passed (successes=35,
> skipped=29, failures=0, errors=0)
>
> With PACKAGE_CLASSES = package_deb
> RESULTS - apt.AptRepoTest.test_apt_install_from_repo: PASSED (43.98s)
> ...
> SUMMARY:
> core-image-sato () - Ran 64 tests in 57.334s
> core-image-sato - OK - All required tests passed (successes=15,
> skipped=49, failures=0, errors=0)
>
> So I do not seem to reproduce the error. Any ideas what else might
> differ in the autobuilder versus my local?

Sigh, I just ran the same experiment here with the same results - all is fine.

The setup json for the autobuilder run is:

"pkgman-deb-non-deb" : {
"MACHINE" : "qemux86",
"PACKAGE_CLASSES" : "package_deb",
"BBTARGETS" : "core-image-sato core-image-sato-sdk
core-image-minimal core-image-minimal-dev
core-image-sato:do_populate_sdk",
"SANITYTARGETS" : "core-image-minimal:do_testimage
core-image-sato:do_testimage core-image-sato-sdk:do_testimage
core-image-sato:do_testsdk"
},

I was using qemux86-64 for my experiment, so I guess I need to redo it
now using the correct machine!

Steve

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164243): 
https://lists.openembedded.org/g/openembedded-core/message/164243
Mute This Topic: https://lists.openembedded.org/mt/90107518/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][dunfell] zlib: backport the fix for CVE-2018-25032

2022-04-11 Thread Ralph Siemsen 
On Mon, Apr 11, 2022 at 2:58 PM Steve Sakoman  wrote:

> Let me know if you make any progress!

After a clean build, and messing about with VNC, I am now able to
'bitbake core-image-sato:do_testimage'.

> Yes, either change will trigger the error.  Without the zlib or xz
> patches all is fine.  Adding just one or the other will fail the
> pkgman-deb-non-deb builds (and probably the pkgman-rpm-non-rpm builds
> too, though I've been concentrating on the deb version to try to
> isolate the offending patches)

I have both gzip and xz patches applied:
$ git log --oneline
52269ab7e7 (HEAD -> dunfell) gzip: fix CVE-2022-1271
e7fa7a2a2c xz: fix CVE-2022-1271
38c55bd388 (origin/dunfell) tzdata: update to 2022a
86285152bd python3-jinja2: Correct HOMEPAGE

With default PACKAGE_CLASSES = package_rpm
RESULTS - dnf.DnfBasicTest.test_dnf_help: PASSED (8.77s)
...
SUMMARY:
core-image-sato () - Ran 64 tests in 242.039s
core-image-sato - OK - All required tests passed (successes=35,
skipped=29, failures=0, errors=0)

With PACKAGE_CLASSES = package_deb
RESULTS - apt.AptRepoTest.test_apt_install_from_repo: PASSED (43.98s)
...
SUMMARY:
core-image-sato () - Ran 64 tests in 57.334s
core-image-sato - OK - All required tests passed (successes=15,
skipped=49, failures=0, errors=0)

So I do not seem to reproduce the error. Any ideas what else might
differ in the autobuilder versus my local?

Ralph

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

[OE-core] package_manager: support for signed DEB package feeds

2022-04-11 Thread Ferry Toth 
[PATCH v3 1/1] apt: add apt selftest to test signed package feeds

Since Gatesgarth apt (1.8.2) has become more strict and doesn’t allow unsigned 
repositories by default.
Currently when building images this requirement is worked around by using 
[allow-insecure=yes] and
equivalently when performing selftest.

Patches "gpg-sign: Add parameters to gpg signature function" and 
"package_manager: sign deb package feeds"
(already in master) enable signed deb package feeds. When called from 
`oe-selftest -r runtime_test.TestImage.test_testimage_apt` this patch adds a 
runtime test for apt 
derived from the  test_testimage_dnf test. It creates a signed deb package 
feed, runs a qemu 
image to install the key and performs some package management. To be able to 
install the key
the gnupg package is added to the testimage.

Changes in V3:
 - When called from `bitbake core-image-sato -c testimage` package feed is 
unsigned. Auto-detect
   this case and behave as before (Richard Purdie)

Changes in V2:
 - Added runtime test for signed deb package feeds (Richard Purdie)


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164240): 
https://lists.openembedded.org/g/openembedded-core/message/164240
Mute This Topic: https://lists.openembedded.org/mt/90404956/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 1/1] apt: add apt selftest to test signed package feeds

2022-04-11 Thread Ferry Toth 
From: Ferry Toth 

Since Gatesgarth apt (1.8.2) has become more strict and doesn’t allow unsigned 
repositories by default.
Currently when building images this requirement is worked around by using 
[allow-insecure=yes] and
equivalently when performing selftest.

Patches "gpg-sign: Add parameters to gpg signature function" and 
"package_manager: sign DEB package feeds"
enable signed DEB package feeds. This patch adds a runtime test for apt derived 
from the test_testimage_dnf
test. It creates a signed deb package feed, runs a qemu image to install the 
key and performs some package
management. To be able to install the key the gnupg package is added to the 
testimage.

Signed-off-by: Ferry Toth 
---
 meta/lib/oeqa/runtime/cases/apt.py   | 38 
 meta/lib/oeqa/selftest/cases/runtime_test.py | 38 
 2 files changed, 69 insertions(+), 7 deletions(-)

diff --git a/meta/lib/oeqa/runtime/cases/apt.py 
b/meta/lib/oeqa/runtime/cases/apt.py
index 53745df93f..574a34f148 100644
--- a/meta/lib/oeqa/runtime/cases/apt.py
+++ b/meta/lib/oeqa/runtime/cases/apt.py
@@ -21,7 +21,7 @@ class AptRepoTest(AptTest):
 
 @classmethod
 def setUpClass(cls):
-service_repo = os.path.join(cls.tc.td['DEPLOY_DIR_DEB'], 'all')
+service_repo = os.path.join(cls.tc.td['DEPLOY_DIR_DEB'], '')
 cls.repo_server = HTTPService(service_repo,
   '0.0.0.0', 
port=cls.tc.target.server_port,
   logger=cls.tc.logger)
@@ -34,20 +34,44 @@ class AptRepoTest(AptTest):
 def setup_source_config_for_package_install(self):
 apt_get_source_server = 'http://%s:%s/' % (self.tc.target.server_ip, 
self.repo_server.port)
 apt_get_sourceslist_dir = '/etc/apt/'
-self.target.run('cd %s; echo deb [ allow-insecure=yes ] %s ./ > 
sources.list' % (apt_get_sourceslist_dir, apt_get_source_server))
+self.target.run('cd %s; echo deb [ allow-insecure=yes ] %s/all ./ > 
sources.list' % (apt_get_sourceslist_dir, apt_get_source_server))
+
+def setup_source_config_for_package_install_signed(self):
+apt_get_source_server = 'http:\/\/%s:%s' % (self.tc.target.server_ip, 
self.repo_server.port)
+apt_get_sourceslist_dir = '/etc/apt/'
+self.target.run("cd %s; cp sources.list sources.list.bak; sed -i 
's/\[trusted=yes\] http:\/\/bogus_ip:bogus_port/%s/g' sources.list" % 
(apt_get_sourceslist_dir, apt_get_source_server))
 
 def cleanup_source_config_for_package_install(self):
 apt_get_sourceslist_dir = '/etc/apt/'
 self.target.run('cd %s; rm sources.list' % (apt_get_sourceslist_dir))
 
+def cleanup_source_config_for_package_install_signed(self):
+apt_get_sourceslist_dir = '/etc/apt/'
+self.target.run('cd %s; mv sources.list.bak sources.list' % 
(apt_get_sourceslist_dir))
+
+def setup_key(self):
+# the key is found on the target /etc/pki/packagefeed-gpg/
+# named PACKAGEFEED-GPG-KEY-poky-branch
+self.target.run('cd %s; apt-key add P*' % ('/etc/pki/packagefeed-gpg'))
+
 @skipIfNotFeature('package-management',
   'Test requires package-management to be in 
IMAGE_FEATURES')
 @skipIfNotDataVar('IMAGE_PKGTYPE', 'deb',
   'DEB is not the primary package manager')
 @OEHasPackage(['apt'])
 def test_apt_install_from_repo(self):
-self.setup_source_config_for_package_install()
-self.pkg('update')
-self.pkg('remove --yes run-postinsts-dev')
-self.pkg('install --yes --allow-unauthenticated run-postinsts-dev')
-self.cleanup_source_config_for_package_install()
+if not self.tc.td.get('PACKAGE_FEED_GPG_NAME'):
+self.setup_source_config_for_package_install()
+self.pkg('update')
+self.pkg('remove --yes run-postinsts-dev')
+self.pkg('install --yes --allow-unauthenticated run-postinsts-dev')
+self.cleanup_source_config_for_package_install()
+else:
+# when we are here a key has been set to sign the package feed and
+# public key and gnupg installed on the image by test_testimage_apt
+self.setup_source_config_for_package_install_signed()
+self.setup_key()
+self.pkg('update')
+self.pkg('install --yes run-postinsts-dev')
+self.pkg('remove --yes run-postinsts-dev')
+self.cleanup_source_config_for_package_install_signed()
diff --git a/meta/lib/oeqa/selftest/cases/runtime_test.py 
b/meta/lib/oeqa/selftest/cases/runtime_test.py
index 2ad89490fc..3ece617cb0 100644
--- a/meta/lib/oeqa/selftest/cases/runtime_test.py
+++ b/meta/lib/oeqa/selftest/cases/runtime_test.py
@@ -162,6 +162,44 @@ class TestImage(OESelftestTestCase):
 bitbake('core-image-full-cmdline socat')
 bitbake('-c testimage core-image-full-cmdline')
 
+def test_testimage_apt(self):
+"""
+

[OE-core] package_manager: support for signed DEB package feeds

2022-04-11 Thread Ferry Toth 
[PATCH v3 1/1] apt: add apt selftest to test signed package feeds

Since Gatesgarth apt (1.8.2) has become more strict and doesn’t allow unsigned 
repositories by default.
Currently when building images this requirement is worked around by using 
[allow-insecure=yes] and
equivalently when performing selftest.

Patches "gpg-sign: Add parameters to gpg signature function" and 
"package_manager: sign deb package feeds"
(in master) enabled signed deb package feeds. 

This patch adds a runtime test for apt derived from the test_testimage_dnf 
test. When called from 
`oe-selftest -r runtime_test.TestImage.test_testimage_apt` it creates a signed 
deb package feed, 
runs a qemu image to install the key and performs some package management. To 
be able to install 
the key the gnupg package is added to the testimage.

Changes in V3:
 - Changed to original behavior when called from `bitbake core-image-sato -c 
testimage` and no 
   signed feed was created (Richard Purdie)

Changes in V2:
 - Added runtime test for signed deb package feeds (Richard Purdie)


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164239): 
https://lists.openembedded.org/g/openembedded-core/message/164239
Mute This Topic: https://lists.openembedded.org/mt/90404956/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][dunfell] zlib: backport the fix for CVE-2018-25032

2022-04-11 Thread Steve Sakoman 
On Mon, Apr 11, 2022 at 8:17 AM Ralph Siemsen  wrote:
>
> On Mon, Apr 11, 2022 at 1:52 PM Steve Sakoman  wrote:
>
> > I see from irc that you've discovered INHERIT += "testimage".  I think
> > that might help with the above issue.
>
> Indeed, I added that, which solved the missing do_testimage problem.
> However even after rebuilding core-image sato, I do not have /etc/apt
> nor apt in the image.
>
> Looking at 
> https://git.yoctoproject.org/poky/tree/meta/lib/oeqa/runtime/cases/apt.py?h=dunfell#n43
> the apt test should be skipped. Seemingly relevant bits of "bitbake -e":
>
> IMAGE_FEATURES=" debug-tweaks"
> IMAGE_PKGTYPE="rpm"
> INHERIT=" cve-check testimage poky-sanity uninative reproducible_build
> package_rpm buildstats image-mklibs image-prelink debian devshell
> sstate license remove-l
> ibtool blacklist sanity"
> TESTIMAGEDEPENDS=" cpio-native:do_populate_sysroot
> dnf-native:do_populate_sysroot createrepo-c-native:do_populate_sysroot
>   qemu-native:do_populate_sysroot
> qemu-helper-native:do_populate_sysroot
> qemu-helper-native:do_addto_recipe_sysroot"

I've been busy setting up my systems to start doing the kirkstone LTS
maintenance too, so I haven't yet dug into this. But I will try to
reproduce this locally in the next day or so.  Let me know if you make
any progress!

> > FWIW, I'm seeing the same error with the xz CVE fix patch too.
>
> Interesting... and without those the tests are successful (with all
> other current changes)?

Yes, either change will trigger the error.  Without the zlib or xz
patches all is fine.  Adding just one or the other will fail the
pkgman-deb-non-deb builds (and probably the pkgman-rpm-non-rpm builds
too, though I've been concentrating on the deb version to try to
isolate the offending patches)

Steve

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164238): 
https://lists.openembedded.org/g/openembedded-core/message/164238
Mute This Topic: https://lists.openembedded.org/mt/90107518/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][dunfell] zlib: backport the fix for CVE-2018-25032

2022-04-11 Thread Ralph Siemsen 
On Mon, Apr 11, 2022 at 1:52 PM Steve Sakoman  wrote:

> I see from irc that you've discovered INHERIT += "testimage".  I think
> that might help with the above issue.

Indeed, I added that, which solved the missing do_testimage problem.
However even after rebuilding core-image sato, I do not have /etc/apt
nor apt in the image.

Looking at 
https://git.yoctoproject.org/poky/tree/meta/lib/oeqa/runtime/cases/apt.py?h=dunfell#n43
the apt test should be skipped. Seemingly relevant bits of "bitbake -e":

IMAGE_FEATURES=" debug-tweaks"
IMAGE_PKGTYPE="rpm"
INHERIT=" cve-check testimage poky-sanity uninative reproducible_build
package_rpm buildstats image-mklibs image-prelink debian devshell
sstate license remove-l
ibtool blacklist sanity"
TESTIMAGEDEPENDS=" cpio-native:do_populate_sysroot
dnf-native:do_populate_sysroot createrepo-c-native:do_populate_sysroot
  qemu-native:do_populate_sysroot
qemu-helper-native:do_populate_sysroot
qemu-helper-native:do_addto_recipe_sysroot"

> FWIW, I'm seeing the same error with the xz CVE fix patch too.

Interesting... and without those the tests are successful (with all
other current changes)?

Ralph

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164237): 
https://lists.openembedded.org/g/openembedded-core/message/164237
Mute This Topic: https://lists.openembedded.org/mt/90107518/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][dunfell] zlib: backport the fix for CVE-2018-25032

2022-04-11 Thread Steve Sakoman 
On Mon, Apr 11, 2022 at 7:31 AM Ralph Siemsen  wrote:
>
> On Mon, Apr 4, 2022 at 11:22 AM Steve Sakoman  wrote:
>
> > > Unfortunately this triggers other errors too :-(
> > >
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/50/builds/4987
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/76/builds/4949
>
> Both of these seem to fail in apt.AptRepoTest.test_apt_install_from_repo.
> This test tries to "cd /etc/apt", then append to sources.list, and
> then "apt-get update".
>
> However, in a locally built core-image-sato, I do not have "/etc/apt"
> directory. I also do not have "apt-get" or "apt" commands.
> I'm not sure if this is a recent change, or a mis-configuration on my part.

I see from irc that you've discovered INHERIT += "testimage".  I think
that might help with the above issue.

> But it seems the test failure is not due to the zlib CVE fix, as I am
> missing '/etc/apt' even without the zilb fix applied.

FWIW, I'm seeing the same error with the xz CVE fix patch too.

Steve

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164236): 
https://lists.openembedded.org/g/openembedded-core/message/164236
Mute This Topic: https://lists.openembedded.org/mt/90107518/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][dunfell] zlib: backport the fix for CVE-2018-25032

2022-04-11 Thread Ralph Siemsen 
On Mon, Apr 4, 2022 at 11:22 AM Steve Sakoman  wrote:

> > Unfortunately this triggers other errors too :-(
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/50/builds/4987
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/76/builds/4949

Both of these seem to fail in apt.AptRepoTest.test_apt_install_from_repo.
This test tries to "cd /etc/apt", then append to sources.list, and
then "apt-get update".

However, in a locally built core-image-sato, I do not have "/etc/apt"
directory. I also do not have "apt-get" or "apt" commands.
I'm not sure if this is a recent change, or a mis-configuration on my part.

But it seems the test failure is not due to the zlib CVE fix, as I am
missing '/etc/apt' even without the zilb fix applied.

Regards,
Ralph

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164235): 
https://lists.openembedded.org/g/openembedded-core/message/164235
Mute This Topic: https://lists.openembedded.org/mt/90107518/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] kernel-devsrc: Check for gen_vdso_offsets.sh before copying on riscv

2022-04-11 Thread Khem Raj 
This file is introduced in newer kernels >= 5.15 but some RISCV machines
still use 5.13 kernels, this ensures that kernel-devsrc can build on
older kernels for RISCV

Signed-off-by: Khem Raj 
---
 meta/recipes-kernel/linux/kernel-devsrc.bb | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-kernel/linux/kernel-devsrc.bb 
b/meta/recipes-kernel/linux/kernel-devsrc.bb
index 8d360ed3f38..9654488a540 100644
--- a/meta/recipes-kernel/linux/kernel-devsrc.bb
+++ b/meta/recipes-kernel/linux/kernel-devsrc.bb
@@ -112,7 +112,9 @@ do_install() {
fi
 
if [ "${ARCH}" = "arm64" -o "${ARCH}" = "riscv" ]; then
-   cp -a --parents arch/${ARCH}/kernel/vdso/vdso.lds $kerneldir/build/
+if [ -e arch/${ARCH}/kernel/vdso/vdso.lds ]; then
+   cp -a --parents arch/${ARCH}/kernel/vdso/vdso.lds 
$kerneldir/build/
+fi
fi
if [ "${ARCH}" = "powerpc" ]; then
cp -a --parents arch/powerpc/kernel/vdso32/vdso32.lds 
$kerneldir/build 2>/dev/null || :
@@ -190,7 +192,9 @@ do_install() {
if [ "${ARCH}" = "riscv" ]; then
 cp -a --parents arch/riscv/kernel/vdso/*gettimeofday.* 
$kerneldir/build/
 cp -a --parents arch/riscv/kernel/vdso/note.S $kerneldir/build/
-cp -a --parents arch/riscv/kernel/vdso/gen_vdso_offsets.sh 
$kerneldir/build/
+if [ -e arch/riscv/kernel/vdso/gen_vdso_offsets.sh ]; then
+cp -a --parents arch/riscv/kernel/vdso/gen_vdso_offsets.sh 
$kerneldir/build/
+fi
cp -a --parents arch/riscv/kernel/vdso/* $kerneldir/build/ 
2>/dev/null || :
fi
 
-- 
2.35.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164234): 
https://lists.openembedded.org/g/openembedded-core/message/164234
Mute This Topic: https://lists.openembedded.org/mt/90399709/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] kernel-devsrc: Check for gen_vdso_offsets.sh before copying on riscv

2022-04-11 Thread Khem Raj 
This file is introduced in newer kernels >= 5.15 but some RISCV machines
still use 5.13 kernels, this ensures that kernel-devsrc can build on
older kernels for RISCV

Signed-off-by: Khem Raj 
---
 meta/recipes-kernel/linux/kernel-devsrc.bb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/linux/kernel-devsrc.bb 
b/meta/recipes-kernel/linux/kernel-devsrc.bb
index 8d360ed3f38..508e13026f9 100644
--- a/meta/recipes-kernel/linux/kernel-devsrc.bb
+++ b/meta/recipes-kernel/linux/kernel-devsrc.bb
@@ -190,7 +190,9 @@ do_install() {
if [ "${ARCH}" = "riscv" ]; then
 cp -a --parents arch/riscv/kernel/vdso/*gettimeofday.* 
$kerneldir/build/
 cp -a --parents arch/riscv/kernel/vdso/note.S $kerneldir/build/
-cp -a --parents arch/riscv/kernel/vdso/gen_vdso_offsets.sh 
$kerneldir/build/
+if [ -e arch/riscv/kernel/vdso/gen_vdso_offsets.sh ]
+cp -a --parents arch/riscv/kernel/vdso/gen_vdso_offsets.sh 
$kerneldir/build/
+fi
cp -a --parents arch/riscv/kernel/vdso/* $kerneldir/build/ 
2>/dev/null || :
fi
 
-- 
2.35.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164233): 
https://lists.openembedded.org/g/openembedded-core/message/164233
Mute This Topic: https://lists.openembedded.org/mt/90399325/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] [master][PATCH v2] libarchive: backport patch to fix CVE-2022-26280

2022-04-11 Thread Khem Raj 
On Mon, Apr 11, 2022 at 6:01 AM Davide Gardenal
 wrote:
>
> Hi Quentin,
>
> I just missed the release so I'm going to send a v3 patch now, thanks!

This patch is perhaps still valid for kirkstone. For upgrading to
3.6.1 I already sent a patch yesterday [1]

[1] https://lists.openembedded.org/g/openembedded-core/message/164210

> 
>

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

2022-04-11 Thread Christopher Larson 
This function returns True if any of the specified packages are skipped due to
incompatible license.

License exceptions are obeyed. The user may specify the package's license for
cross-recipe checks.

This allows for additions to packagegroups only for non-incompatible builds. For
example:

RDEPENDS_${PN} += "${@'dbench' if not any_incompatible(d, 'dbench', 
'GPL-3.0-only') else ''}"

Signed-off-by: Christopher Larson 
---
 meta/classes/license.bbclass | 37 
 1 file changed, 37 insertions(+)

diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass
index 0c637e966e..db8c3f8584 100644
--- a/meta/classes/license.bbclass
+++ b/meta/classes/license.bbclass
@@ -320,6 +320,43 @@ def incompatible_license(d, dont_want_licenses, 
package=None):
 
 return incompatible_pkg_license(d, dont_want_licenses, license)
 
+def any_incompatible(d, packages, licensestring=None):
+"""Return True if any of the packages are skipped due to incompatible 
license.
+
+License exceptions are obeyed. The user may specify the package's license
+for cross-recipe checks.
+
+This allows for additions to packagegroups only for non-incompatible 
builds.
+For example:
+
+RDEPENDS_${PN} += "${@'dbench' if not any_incompatible(d, 'dbench', 
'GPL-3.0-only') else ''}"
+"""
+if not isinstance(packages, str):
+packages = packages.split()
+
+bad_licenses = (d.getVar("INCOMPATIBLE_LICENSE") or "").split()
+if not bad_licenses:
+return False
+bad_licenses = expand_wildcard_licenses(d, bad_licenses)
+
+if licensestring is None:
+licensestring = d.getVar("LICENSE:%s" % package) if package else None
+if not licensestring:
+licensestring = d.getVar("LICENSE")
+
+exceptions = (d.getVar("INCOMPATIBLE_LICENSE_EXCEPTIONS") or "").split()
+for pkg in packages:
+remaining_bad_licenses = license.apply_pkg_license_exception(
+pkg, bad_licenses, exceptions
+)
+
+incompatible_lic = incompatible_pkg_license(
+d, remaining_bad_licenses, licensestring
+)
+if incompatible_lic:
+return True
+return False
+
 def check_license_flags(d):
 """
 This function checks if a recipe has any LICENSE_FLAGS that
-- 
2.35.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164231): 
https://lists.openembedded.org/g/openembedded-core/message/164231
Mute This Topic: https://lists.openembedded.org/mt/90398685/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] go: Always pass interpreter to linker

2022-04-11 Thread Khem Raj 
On Mon, Apr 11, 2022 at 6:30 AM Jose Quaresma  wrote:
>
>
>
> J?rg Vehlow  escreveu no dia segunda, 11/04/2022 à(s) 13:35:
>>
>> From: Joerg Vehlow 
>>
>> When gos internal linker is used, it uses hardcoded paths to the
>> interpreter (dynamic linker). For x86_64 this hardcoded path is
>> /lib64/ld-linux-x86-64.so.2, but yocto's default dynamic linker path
>> is /lib64/ld-linux-x86-64.so.2.
>
>
> Is this correct? The first path is the seme one of the second.

its a typo, Sadly we do differ in baselib naming convention on x86_64
( without multilib )
compared to other distributions which are inherently multilib and use
/lib64 OE instead uses
/lib when multilib is not enabled but lib64/ when multilib is enabled.

>
> Jose
>
>>
>> Most of the time, the internal linker is not used and binutils linker
>> sets the correct path, but sometimes the internal linker is used and
>> the resulting binary will not work on x86_64.
>>
>> To ensure the path is always correct, pass  it to the linker.
>>
>> Signed-off-by: Joerg Vehlow 
>> ---
>>  meta/classes/go.bbclass | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/classes/go.bbclass b/meta/classes/go.bbclass
>> index 9c4c92bffd..df8d4db26d 100644
>> --- a/meta/classes/go.bbclass
>> +++ b/meta/classes/go.bbclass
>> @@ -1,4 +1,5 @@
>>  inherit goarch
>> +inherit linuxloader
>>
>>  GO_PARALLEL_BUILD ?= "${@oe.utils.parallel_make_argument(d, '-p %d')}"
>>
>> @@ -44,7 +45,7 @@ GO_LINKMODE ?= ""
>>  GO_LINKMODE:class-nativesdk = "--linkmode=external"
>>  GO_LINKMODE:class-native = "--linkmode=external"
>>  GO_EXTRA_LDFLAGS ?= ""
>> -GO_LDFLAGS ?= '-ldflags="${GO_RPATH} ${GO_LINKMODE} ${GO_EXTRA_LDFLAGS} 
>> -extldflags '${GO_EXTLDFLAGS}'"'
>> +GO_LDFLAGS ?= '-ldflags="${GO_RPATH} ${GO_LINKMODE} -I 
>> ${@get_linuxloader(d)} ${GO_EXTRA_LDFLAGS} -extldflags '${GO_EXTLDFLAGS}'"'
>>  export GOBUILDFLAGS ?= "-v ${GO_LDFLAGS} -trimpath"
>>  export GOPATH_OMIT_IN_ACTIONID ?= "1"
>>  export GOPTESTBUILDFLAGS ?= "${GOBUILDFLAGS} -c"
>> --
>> 2.25.1
>>
>>
>> 
>>
>
>
> --
> Best regards,
>
> José Quaresma

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

Re: [OE-core] [PATCH v2 0/6] poky-tiny clean-ups and qemuarm fixes

2022-04-11 Thread Jon Mason 
On Sat, Apr 9, 2022 at 5:17 PM Richard Purdie
 wrote:
>
> On Fri, 2022-04-08 at 15:25 -0400, Jon Mason wrote:
> > Changes in v2:
> > * Addressed Khem's comments
> > * Rebased to the latest version
> > * Added distro description for poky-tiny, as this makes it more obvious
> >   which distro is being booted when there are multiple
> >
> > The yocto-kernel-cache changes were sent, accepted, and should be
> > uploaded shortly.
>
> Still breaks on the autobuilder unfortunately:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/110/builds/3923
> https://autobuilder.yoctoproject.org/typhoon/#/builders/53/builds/5035
>
> :(

Weird, it passed in my CI.  Okay, I'll see if I can repo and fix.
Thanks for the testing.

Thanks,
Jon

>
> Cheers,
>
> Richard
>

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

[OE-core] bitbake.conf: reason for assignment of deprecated variable

2022-04-11 Thread Markus Niebel 
Hello,

with the deprecation of SERIAL_CONSOLE the following was introduced in 
bitbake.conf:

3f0d665384e6d1b7aa2854a9cc4f13e0961bacb7 (bitbake.conf: set SERIAL_CONSOLES from
SERIAL_CONSOLE by default)

Since this is done after inclusion of local.conf and .conf any weak 
default
done for a System-on-Module / CPU will be overwritten. Is this just by 
forgetting to
remove these lines after the deprecation or is there still a reason to keep 
this as
is?

Thank you

Markus Niebel



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164228): 
https://lists.openembedded.org/g/openembedded-core/message/164228
Mute This Topic: https://lists.openembedded.org/mt/90397045/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][master][PATCH v3] libarchive: upgrate from 3.6.0 to 3.6.1

2022-04-11 Thread Alexandre Belloni via lists.openembedded.org 
Hello,

I'm already carrying this upgrade from Khem, earlier today.

On 11/04/2022 15:21:45+0200, Davide Gardenal wrote:
> This release contains security fixes
> including CVE-2022-26280 fix.
> 
> Signed-off-by: Davide Gardenal 
> ---
> Update v3:
> Instead of backporting the patch upgrade to the latest release with
> the CVE fix. Thanks Quentin Schulz for the advise.  
> Update v2:
> Change title from [meta-oe] to [oe-core]
> ---
>  .../libarchive/{libarchive_3.6.0.bb => libarchive_3.6.1.bb} | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>  rename meta/recipes-extended/libarchive/{libarchive_3.6.0.bb => 
> libarchive_3.6.1.bb} (96%)
> 
> diff --git a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb 
> b/meta/recipes-extended/libarchive/libarchive_3.6.1.bb
> similarity index 96%
> rename from meta/recipes-extended/libarchive/libarchive_3.6.0.bb
> rename to meta/recipes-extended/libarchive/libarchive_3.6.1.bb
> index f078c8ad03..c795b41628 100644
> --- a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
> +++ b/meta/recipes-extended/libarchive/libarchive_3.6.1.bb
> @@ -35,7 +35,7 @@ EXTRA_OECONF += "--enable-largefile"
>  SRC_URI = "http://libarchive.org/downloads/libarchive-${PV}.tar.gz";
>  UPSTREAM_CHECK_URI = "http://libarchive.org/";
>  
> -SRC_URI[sha256sum] = 
> "a36613695ffa2905fdedc997b6df04a3006ccfd71d747a339b78aa8412c3d852"
> +SRC_URI[sha256sum] = 
> "c676146577d989189940f1959d9e3980d28513d74eedfbc6b7f15ea45fe54ee2"
>  
>  inherit autotools update-alternatives pkgconfig
>  
> -- 
> 2.32.0
> 

> 
> 
> 


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

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164227): 
https://lists.openembedded.org/g/openembedded-core/message/164227
Mute This Topic: https://lists.openembedded.org/mt/90394591/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] go: Always pass interpreter to linker

2022-04-11 Thread J?rg Vehlow 
Hi Jose,

Am 4/11/2022 um 3:30 PM schrieb Jose Quaresma:
> 
> 
> J?rg Vehlow mailto:l...@jv-coder.de>> escreveu no dia
> segunda, 11/04/2022 à(s) 13:35:
> 
> From: Joerg Vehlow mailto:joerg.veh...@aox.de>>
> 
> When gos internal linker is used, it uses hardcoded paths to the
> interpreter (dynamic linker). For x86_64 this hardcoded path is
> /lib64/ld-linux-x86-64.so.2, but yocto's default dynamic linker path
> is /lib64/ld-linux-x86-64.so.2.
> 
> 
> Is this correct? The first path is the seme one of the second.
Yes... of course this is wrong. /lib/ld-linux-x86-64.so.2 is the default
linker path for yocto on x86_64

Sorry about that

Joerg


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164226): 
https://lists.openembedded.org/g/openembedded-core/message/164226
Mute This Topic: https://lists.openembedded.org/mt/90393660/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] go: Always pass interpreter to linker

2022-04-11 Thread Jose Quaresma 
J?rg Vehlow  escreveu no dia segunda, 11/04/2022 à(s)
13:35:

> From: Joerg Vehlow 
>
> When gos internal linker is used, it uses hardcoded paths to the
> interpreter (dynamic linker). For x86_64 this hardcoded path is
> /lib64/ld-linux-x86-64.so.2, but yocto's default dynamic linker path
> is /lib64/ld-linux-x86-64.so.2.
>

Is this correct? The first path is the seme one of the second.

Jose


> Most of the time, the internal linker is not used and binutils linker
> sets the correct path, but sometimes the internal linker is used and
> the resulting binary will not work on x86_64.
>
> To ensure the path is always correct, pass  it to the linker.
>
> Signed-off-by: Joerg Vehlow 
> ---
>  meta/classes/go.bbclass | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/meta/classes/go.bbclass b/meta/classes/go.bbclass
> index 9c4c92bffd..df8d4db26d 100644
> --- a/meta/classes/go.bbclass
> +++ b/meta/classes/go.bbclass
> @@ -1,4 +1,5 @@
>  inherit goarch
> +inherit linuxloader
>
>  GO_PARALLEL_BUILD ?= "${@oe.utils.parallel_make_argument(d, '-p %d')}"
>
> @@ -44,7 +45,7 @@ GO_LINKMODE ?= ""
>  GO_LINKMODE:class-nativesdk = "--linkmode=external"
>  GO_LINKMODE:class-native = "--linkmode=external"
>  GO_EXTRA_LDFLAGS ?= ""
> -GO_LDFLAGS ?= '-ldflags="${GO_RPATH} ${GO_LINKMODE} ${GO_EXTRA_LDFLAGS}
> -extldflags '${GO_EXTLDFLAGS}'"'
> +GO_LDFLAGS ?= '-ldflags="${GO_RPATH} ${GO_LINKMODE} -I
> ${@get_linuxloader(d)} ${GO_EXTRA_LDFLAGS} -extldflags '${GO_EXTLDFLAGS}'"'
>  export GOBUILDFLAGS ?= "-v ${GO_LDFLAGS} -trimpath"
>  export GOPATH_OMIT_IN_ACTIONID ?= "1"
>  export GOPTESTBUILDFLAGS ?= "${GOBUILDFLAGS} -c"
> --
> 2.25.1
>
>
> 
>
>

-- 
Best regards,

José Quaresma

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

[oe-core][master][PATCH v3] libarchive: upgrate from 3.6.0 to 3.6.1

2022-04-11 Thread Davide Gardenal 
This release contains security fixes
including CVE-2022-26280 fix.

Signed-off-by: Davide Gardenal 
---
Update v3:
Instead of backporting the patch upgrade to the latest release with
the CVE fix. Thanks Quentin Schulz for the advise.  
Update v2:
Change title from [meta-oe] to [oe-core]
---
 .../libarchive/{libarchive_3.6.0.bb => libarchive_3.6.1.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-extended/libarchive/{libarchive_3.6.0.bb => 
libarchive_3.6.1.bb} (96%)

diff --git a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb 
b/meta/recipes-extended/libarchive/libarchive_3.6.1.bb
similarity index 96%
rename from meta/recipes-extended/libarchive/libarchive_3.6.0.bb
rename to meta/recipes-extended/libarchive/libarchive_3.6.1.bb
index f078c8ad03..c795b41628 100644
--- a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
+++ b/meta/recipes-extended/libarchive/libarchive_3.6.1.bb
@@ -35,7 +35,7 @@ EXTRA_OECONF += "--enable-largefile"
 SRC_URI = "http://libarchive.org/downloads/libarchive-${PV}.tar.gz";
 UPSTREAM_CHECK_URI = "http://libarchive.org/";
 
-SRC_URI[sha256sum] = 
"a36613695ffa2905fdedc997b6df04a3006ccfd71d747a339b78aa8412c3d852"
+SRC_URI[sha256sum] = 
"c676146577d989189940f1959d9e3980d28513d74eedfbc6b7f15ea45fe54ee2"
 
 inherit autotools update-alternatives pkgconfig
 
-- 
2.32.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164224): 
https://lists.openembedded.org/g/openembedded-core/message/164224
Mute This Topic: https://lists.openembedded.org/mt/90394591/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] [master][PATCH v2] libarchive: backport patch to fix CVE-2022-26280

2022-04-11 Thread Davide Gardenal 
Hi Quentin,

I just missed the release so I'm going to send a v3 patch now, thanks!

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164223): 
https://lists.openembedded.org/g/openembedded-core/message/164223
Mute This Topic: https://lists.openembedded.org/mt/90393740/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][master][PATCH v2] libarchive: backport patch to fix CVE-2022-26280

2022-04-11 Thread Quentin Schulz 

Hi Davide,

Any reason for not updating to 3.6.1 since it includes this patch (from 
the changelog info at least)?


Cheers,
Quentin

On 4/11/22 14:40, Davide Gardenal wrote:

Signed-off-by: Davide Gardenal 
---
Update v2:
 Change title from [meta-oe] to [oe-core]
---
  .../libarchive/CVE-2022-26280.patch   | 31 +++
  .../libarchive/libarchive_3.6.0.bb|  4 ++-
  2 files changed, 34 insertions(+), 1 deletion(-)
  create mode 100644 
meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch

diff --git a/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch 
b/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch
new file mode 100644
index 00..c322e12274
--- /dev/null
+++ b/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch
@@ -0,0 +1,31 @@
+From cfaa28168a07ea4a53276b63068f94fce37d6aff Mon Sep 17 00:00:00 2001
+From: Tim Kientzle 
+Date: Thu, 24 Mar 2022 10:35:00 +0100
+Subject: [PATCH] ZIP reader: fix possible out-of-bounds read in
+ zipx_lzma_alone_init()
+
+Fixes #1672
+
+CVE: CVE-2022-26280
+
+Upstream-Status: Backport
+https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_libarchive_libarchive_commit_cfaa28168a07ea4a53276b63068f94fce37d6aff&d=DwIDAg&c=_sEr5x9kUWhuk4_nFwjJtA&r=LYjLexDn7rXIzVmkNPvw5ymA1XTSqHGq8yBP6m6qZZ4njZguQhZhkI_-172IIy1t&m=gRPxVzB4EMk0PAljAspLUdHOoj0XBl1hDKvUE0nQ4-ZYFhgn62aUm-XFsQqztA9x&s=Fu-Kk2GhLiSc9o3rjHQaH-u-51djGq7bKAdxUIMTRuo&e=
+
+Signed-off-by: Davide Gardenal 
+---
+ libarchive/archive_read_support_format_zip.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/libarchive/archive_read_support_format_zip.c 
b/libarchive/archive_read_support_format_zip.c
+index 38ada70b5..9d6c900b2 100644
+--- a/libarchive/archive_read_support_format_zip.c
 b/libarchive/archive_read_support_format_zip.c
+@@ -1667,7 +1667,7 @@ zipx_lzma_alone_init(struct archive_read *a, struct zip 
*zip)
+*/
+
+   /* Read magic1,magic2,lzma_params from the ZIPX stream. */
+-  if((p = __archive_read_ahead(a, 9, NULL)) == NULL) {
++  if(zip->entry_bytes_remaining < 9 || (p = __archive_read_ahead(a, 9, 
NULL)) == NULL) {
+   archive_set_error(&a->archive, ARCHIVE_ERRNO_FILE_FORMAT,
+   "Truncated lzma data");
+   return (ARCHIVE_FATAL);
diff --git a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb 
b/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
index f078c8ad03..16d6e2af2d 100644
--- a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
+++ b/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
@@ -32,7 +32,9 @@ PACKAGECONFIG[zstd] = "--with-zstd,--without-zstd,zstd,"
  
  EXTRA_OECONF += "--enable-largefile"
  
-SRC_URI = "https://urldefense.proofpoint.com/v2/url?u=http-3A__libarchive.org_downloads_libarchive-2D-24-257BPV-257D.tar.gz&d=DwIDAg&c=_sEr5x9kUWhuk4_nFwjJtA&r=LYjLexDn7rXIzVmkNPvw5ymA1XTSqHGq8yBP6m6qZZ4njZguQhZhkI_-172IIy1t&m=gRPxVzB4EMk0PAljAspLUdHOoj0XBl1hDKvUE0nQ4-ZYFhgn62aUm-XFsQqztA9x&s=qjkmUgOa7iMTBMJje9R2Ea_YQJUs8P7nFC_iHFYKT4A&e= "

+SRC_URI = 
"https://urldefense.proofpoint.com/v2/url?u=http-3A__libarchive.org_downloads_libarchive-2D-24-257BPV-257D.tar.gz&d=DwIDAg&c=_sEr5x9kUWhuk4_nFwjJtA&r=LYjLexDn7rXIzVmkNPvw5ymA1XTSqHGq8yBP6m6qZZ4njZguQhZhkI_-172IIy1t&m=gRPxVzB4EMk0PAljAspLUdHOoj0XBl1hDKvUE0nQ4-ZYFhgn62aUm-XFsQqztA9x&s=qjkmUgOa7iMTBMJje9R2Ea_YQJUs8P7nFC_iHFYKT4A&e=
  \
+   file://CVE-2022-26280.patch"
+
  UPSTREAM_CHECK_URI = 
"https://urldefense.proofpoint.com/v2/url?u=http-3A__libarchive.org_&d=DwIDAg&c=_sEr5x9kUWhuk4_nFwjJtA&r=LYjLexDn7rXIzVmkNPvw5ymA1XTSqHGq8yBP6m6qZZ4njZguQhZhkI_-172IIy1t&m=gRPxVzB4EMk0PAljAspLUdHOoj0XBl1hDKvUE0nQ4-ZYFhgn62aUm-XFsQqztA9x&s=fBf6Ji0vNBD7Q-LH4pCiZVZki3Hx8oz4ZOsBhe3yQIw&e=
 "
  
  SRC_URI[sha256sum] = "a36613695ffa2905fdedc997b6df04a3006ccfd71d747a339b78aa8412c3d852"







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

[oe-core][master][PATCH v2] libarchive: backport patch to fix CVE-2022-26280

2022-04-11 Thread Davide Gardenal 
Signed-off-by: Davide Gardenal 
---
Update v2:
Change title from [meta-oe] to [oe-core]
---
 .../libarchive/CVE-2022-26280.patch   | 31 +++
 .../libarchive/libarchive_3.6.0.bb|  4 ++-
 2 files changed, 34 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch

diff --git a/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch 
b/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch
new file mode 100644
index 00..c322e12274
--- /dev/null
+++ b/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch
@@ -0,0 +1,31 @@
+From cfaa28168a07ea4a53276b63068f94fce37d6aff Mon Sep 17 00:00:00 2001
+From: Tim Kientzle 
+Date: Thu, 24 Mar 2022 10:35:00 +0100
+Subject: [PATCH] ZIP reader: fix possible out-of-bounds read in
+ zipx_lzma_alone_init()
+
+Fixes #1672
+
+CVE: CVE-2022-26280
+
+Upstream-Status: Backport
+https://github.com/libarchive/libarchive/commit/cfaa28168a07ea4a53276b63068f94fce37d6aff
+
+Signed-off-by: Davide Gardenal 
+---
+ libarchive/archive_read_support_format_zip.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/libarchive/archive_read_support_format_zip.c 
b/libarchive/archive_read_support_format_zip.c
+index 38ada70b5..9d6c900b2 100644
+--- a/libarchive/archive_read_support_format_zip.c
 b/libarchive/archive_read_support_format_zip.c
+@@ -1667,7 +1667,7 @@ zipx_lzma_alone_init(struct archive_read *a, struct zip 
*zip)
+*/
+ 
+   /* Read magic1,magic2,lzma_params from the ZIPX stream. */
+-  if((p = __archive_read_ahead(a, 9, NULL)) == NULL) {
++  if(zip->entry_bytes_remaining < 9 || (p = __archive_read_ahead(a, 9, 
NULL)) == NULL) {
+   archive_set_error(&a->archive, ARCHIVE_ERRNO_FILE_FORMAT,
+   "Truncated lzma data");
+   return (ARCHIVE_FATAL); 
diff --git a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb 
b/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
index f078c8ad03..16d6e2af2d 100644
--- a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
+++ b/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
@@ -32,7 +32,9 @@ PACKAGECONFIG[zstd] = "--with-zstd,--without-zstd,zstd,"
 
 EXTRA_OECONF += "--enable-largefile"
 
-SRC_URI = "http://libarchive.org/downloads/libarchive-${PV}.tar.gz";
+SRC_URI = "http://libarchive.org/downloads/libarchive-${PV}.tar.gz \
+   file://CVE-2022-26280.patch"
+
 UPSTREAM_CHECK_URI = "http://libarchive.org/";
 
 SRC_URI[sha256sum] = 
"a36613695ffa2905fdedc997b6df04a3006ccfd71d747a339b78aa8412c3d852"
-- 
2.32.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164221): 
https://lists.openembedded.org/g/openembedded-core/message/164221
Mute This Topic: https://lists.openembedded.org/mt/90393740/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][master][PATCH] libarchive: backport patch to fix CVE-2022-26280

2022-04-11 Thread Davide Gardenal 
Signed-off-by: Davide Gardenal 
---
 .../libarchive/CVE-2022-26280.patch   | 31 +++
 .../libarchive/libarchive_3.6.0.bb|  4 ++-
 2 files changed, 34 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch

diff --git a/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch 
b/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch
new file mode 100644
index 00..c322e12274
--- /dev/null
+++ b/meta/recipes-extended/libarchive/libarchive/CVE-2022-26280.patch
@@ -0,0 +1,31 @@
+From cfaa28168a07ea4a53276b63068f94fce37d6aff Mon Sep 17 00:00:00 2001
+From: Tim Kientzle 
+Date: Thu, 24 Mar 2022 10:35:00 +0100
+Subject: [PATCH] ZIP reader: fix possible out-of-bounds read in
+ zipx_lzma_alone_init()
+
+Fixes #1672
+
+CVE: CVE-2022-26280
+
+Upstream-Status: Backport
+https://github.com/libarchive/libarchive/commit/cfaa28168a07ea4a53276b63068f94fce37d6aff
+
+Signed-off-by: Davide Gardenal 
+---
+ libarchive/archive_read_support_format_zip.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/libarchive/archive_read_support_format_zip.c 
b/libarchive/archive_read_support_format_zip.c
+index 38ada70b5..9d6c900b2 100644
+--- a/libarchive/archive_read_support_format_zip.c
 b/libarchive/archive_read_support_format_zip.c
+@@ -1667,7 +1667,7 @@ zipx_lzma_alone_init(struct archive_read *a, struct zip 
*zip)
+*/
+ 
+   /* Read magic1,magic2,lzma_params from the ZIPX stream. */
+-  if((p = __archive_read_ahead(a, 9, NULL)) == NULL) {
++  if(zip->entry_bytes_remaining < 9 || (p = __archive_read_ahead(a, 9, 
NULL)) == NULL) {
+   archive_set_error(&a->archive, ARCHIVE_ERRNO_FILE_FORMAT,
+   "Truncated lzma data");
+   return (ARCHIVE_FATAL); 
diff --git a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb 
b/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
index f078c8ad03..16d6e2af2d 100644
--- a/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
+++ b/meta/recipes-extended/libarchive/libarchive_3.6.0.bb
@@ -32,7 +32,9 @@ PACKAGECONFIG[zstd] = "--with-zstd,--without-zstd,zstd,"
 
 EXTRA_OECONF += "--enable-largefile"
 
-SRC_URI = "http://libarchive.org/downloads/libarchive-${PV}.tar.gz";
+SRC_URI = "http://libarchive.org/downloads/libarchive-${PV}.tar.gz \
+   file://CVE-2022-26280.patch"
+
 UPSTREAM_CHECK_URI = "http://libarchive.org/";
 
 SRC_URI[sha256sum] = 
"a36613695ffa2905fdedc997b6df04a3006ccfd71d747a339b78aa8412c3d852"
-- 
2.32.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164220): 
https://lists.openembedded.org/g/openembedded-core/message/164220
Mute This Topic: https://lists.openembedded.org/mt/90393720/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] go: Always pass interpreter to linker

2022-04-11 Thread J?rg Vehlow 
From: Joerg Vehlow 

When gos internal linker is used, it uses hardcoded paths to the
interpreter (dynamic linker). For x86_64 this hardcoded path is
/lib64/ld-linux-x86-64.so.2, but yocto's default dynamic linker path
is /lib64/ld-linux-x86-64.so.2.
Most of the time, the internal linker is not used and binutils linker
sets the correct path, but sometimes the internal linker is used and
the resulting binary will not work on x86_64.

To ensure the path is always correct, pass  it to the linker.

Signed-off-by: Joerg Vehlow 
---
 meta/classes/go.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/go.bbclass b/meta/classes/go.bbclass
index 9c4c92bffd..df8d4db26d 100644
--- a/meta/classes/go.bbclass
+++ b/meta/classes/go.bbclass
@@ -1,4 +1,5 @@
 inherit goarch
+inherit linuxloader
 
 GO_PARALLEL_BUILD ?= "${@oe.utils.parallel_make_argument(d, '-p %d')}"
 
@@ -44,7 +45,7 @@ GO_LINKMODE ?= ""
 GO_LINKMODE:class-nativesdk = "--linkmode=external"
 GO_LINKMODE:class-native = "--linkmode=external"
 GO_EXTRA_LDFLAGS ?= ""
-GO_LDFLAGS ?= '-ldflags="${GO_RPATH} ${GO_LINKMODE} ${GO_EXTRA_LDFLAGS} 
-extldflags '${GO_EXTLDFLAGS}'"'
+GO_LDFLAGS ?= '-ldflags="${GO_RPATH} ${GO_LINKMODE} -I ${@get_linuxloader(d)} 
${GO_EXTRA_LDFLAGS} -extldflags '${GO_EXTLDFLAGS}'"'
 export GOBUILDFLAGS ?= "-v ${GO_LDFLAGS} -trimpath"
 export GOPATH_OMIT_IN_ACTIONID ?= "1"
 export GOPTESTBUILDFLAGS ?= "${GOBUILDFLAGS} -c"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164219): 
https://lists.openembedded.org/g/openembedded-core/message/164219
Mute This Topic: https://lists.openembedded.org/mt/90393660/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] itstool: correct upstream version check

2022-04-11 Thread Alexander Kanavin 
Signed-off-by: Alexander Kanavin 
---
 meta/recipes-support/itstool/itstool_2.0.7.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-support/itstool/itstool_2.0.7.bb 
b/meta/recipes-support/itstool/itstool_2.0.7.bb
index 5af16995d7..336fb636d5 100644
--- a/meta/recipes-support/itstool/itstool_2.0.7.bb
+++ b/meta/recipes-support/itstool/itstool_2.0.7.bb
@@ -13,6 +13,8 @@ DEPENDS = "libxml2-native"
 
 SRC_URI = "http://files.itstool.org/${BPN}/${BPN}-${PV}.tar.bz2 \
"
+UPSTREAM_CHECK_URI = "https://itstool.org/download.html";
+
 SRC_URI:append:class-native = " 
file://0001-Native-Don-t-use-build-time-hardcoded-python-binary-.patch"
 SRC_URI:append:class-nativesdk = " 
file://0001-Native-Don-t-use-build-time-hardcoded-python-binary-.patch"
 SRC_URI:append:class-target = " 
file://0002-Don-t-use-build-time-hardcoded-python-binary-path.patch"
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#164218): 
https://lists.openembedded.org/g/openembedded-core/message/164218
Mute This Topic: https://lists.openembedded.org/mt/90391661/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] bluez5: install /var/lib/bluetooth directory

2022-04-11 Thread Stefan Herbrechtsmeier 

Am 08.04.2022 um 22:03 schrieb Markus Volk via lists.openembedded.org:
also had this issue and found out, that the bluetooth service did come 
up, once bluetooth was enabled


My fix was to edit like this

ReadWritePaths=:/var/lib/bluetooth

The colon ensures that the service won't fail if the directory doesn't 
exist


Regarding the documentation the ReadOnlyPaths and ReadWritePaths makes 
not sense because ProtectSystem=full mounts /usr, boot loader and /etc 
directories read-only.


Have somebody report the problem to the bluez project?

Regards
  Stefan





Am 08.04.22 um 13:01 schrieb Yi Zhao:


On 4/8/22 18:51, Stefan Herbrechtsmeier wrote:

Am 08.04.2022 um 11:23 schrieb Yi Zhao:


On 4/8/22 16:34, Stefan Herbrechtsmeier wrote:

Am 08.04.2022 um 10:15 schrieb Yi Zhao via lists.openembedded.org:

There is a bluetooth service startup failure:
bluetooth.service: Failed at step NAMESPACE spawning 
/usr/libexec/bluetooth/bluetoothd: No such file or directory
bluetooth.service: Failed to set up mount namespacing: 
/run/systemd/unit-root/var/lib/bluetooth: No such file or directory


This is because the directory /var/lib/bluetooth has been listed in
ReadWritePaths= in the service unit file but does not exist. We 
need to

create it before service startup.

Signed-off-by: Yi Zhao 
---
  meta/recipes-connectivity/bluez5/bluez5.inc | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc 
b/meta/recipes-connectivity/bluez5/bluez5.inc

index 79d4645ca8..ee2cdbcc59 100644
--- a/meta/recipes-connectivity/bluez5/bluez5.inc
+++ b/meta/recipes-connectivity/bluez5/bluez5.inc
@@ -102,6 +102,8 @@ do_install:append() {
  # Patch python tools to use Python 3; they should be source 
compatible, but

  # still refer to Python 2 in the shebang
  sed -i -e '1s,#!.*python.*,#!${bindir}/python3,' 
${D}${libdir}/bluez/test/*

+
+    install -d ${D}${localstatedir}/lib/bluetooth


Please use systemd StateDirectory= [1]. Maybe it is reasonable to 
replace the ReadWritePaths= with StateDirectory=.


[1] 
https://www.freedesktop.org/software/systemd/man/systemd.exec.html#RuntimeDirectory= 




Thanks Stefank, but the ReadWritePaths= is set by bluez:

https://github.com/bluez/bluez/commit/442d211b5f30f00d5ddd69b43385a03c1428ac45 



And who creates the directory before the patch?


There is no need this directory before the patch. The service can 
startup without this directory.





The systemd solution to create a state directory is StateDirectory= 
(or tmpfiles.d for older versions of systemd). Systemd supports 
stateless system [1] and therefore every service must work with an 
empty /var/lib.



I'll try adding StateDirecotry to service unit file to  see if this 
can solve the problem.  Thanks.



//Yi



This patch hide a bug in bluez and the problem should be fixed in bluez.

[1] https://0pointer.net/blog/projects/stateless.html










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