Re: [OE-core] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

[Quentin Schulz]

Hi Alex,

On Fri, Apr 23, 2021 at 07:40:24PM +0200, Alexander Kanavin wrote:
> Particularly,
> - correctly describe the use of DEBUGINFOD_URLS; drop it from bitbake 
> variables
> - all necessary component tweaks are enabled by default via DISTRO_FEATURES
> - provide on-target examples of what to look for when things work properly
> 
> Signed-off-by: Alexander Kanavin 
> ---
>  documentation/dev-manual/common-tasks.rst | 40 ++-
>  documentation/ref-manual/variables.rst|  6 
>  2 files changed, 24 insertions(+), 22 deletions(-)
> 
> diff --git a/documentation/dev-manual/common-tasks.rst 
> b/documentation/dev-manual/common-tasks.rst
> index dcd3a11c3e..e3b9dcef18 100644
> --- a/documentation/dev-manual/common-tasks.rst
> +++ b/documentation/dev-manual/common-tasks.rst
> @@ -9705,39 +9705,47 @@ methods you can use: running a debuginfod server and 
> using gdbserver.
>  Using the debuginfod server method
>  
>  
> -"debuginfod" from "elfutils" is a way to distribute "debuginfo" files.
> -Running a "debuginfod" server makes debug symbols readily available,
> +``debuginfod`` from ``elfutils`` is a way to distribute ``debuginfo`` files.
> +Running a ``debuginfod`` server makes debug symbols readily available,
>  which means you don't need to download debugging information
>  and the binaries of the process being debugged. You can just fetch
>  debug symbols from the server.
>  
> -To run a debuginfod server, you need to do the following:
> +To run a ``debuginfod`` server, you need to do the following:
>  
> --  Ensure that this variable is set in your ``local.conf`` file::
> +-  Ensure that ``debuginfod`` is present in :term:`DISTRO_FEATURES`
> +   (it already is in ``OpenEmbedded-core`` defaults and ``poky`` reference 
> distribution).
> +   If not, set::

I would add where this is supposed to be set (in a distro conf file or
in local.conf if I'm not mistaken?), since it was an information that
was there initially.

For the rest:
Reviewed-by: Quentin Schulz 

Thanks,
Quentin

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

[OE-core] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

[Alexander Kanavin]

Particularly,
- correctly describe the use of DEBUGINFOD_URLS; drop it from bitbake variables
- all necessary component tweaks are enabled by default via DISTRO_FEATURES
- provide on-target examples of what to look for when things work properly

Signed-off-by: Alexander Kanavin 
---
 documentation/dev-manual/common-tasks.rst | 40 ++-
 documentation/ref-manual/variables.rst|  6 
 2 files changed, 24 insertions(+), 22 deletions(-)

diff --git a/documentation/dev-manual/common-tasks.rst 
b/documentation/dev-manual/common-tasks.rst
index dcd3a11c3e..37c7a19bfa 100644
--- a/documentation/dev-manual/common-tasks.rst
+++ b/documentation/dev-manual/common-tasks.rst
@@ -9705,39 +9705,47 @@ methods you can use: running a debuginfod server and 
using gdbserver.
 Using the debuginfod server method
 
 
-"debuginfod" from "elfutils" is a way to distribute "debuginfo" files.
-Running a "debuginfod" server makes debug symbols readily available,
+``debuginfod`` from ``elfutils`` is a way to distribute ``debuginfo`` files.
+Running a ``debuginfod`` server makes debug symbols readily available,
 which means you don't need to download debugging information
 and the binaries of the process being debugged. You can just fetch
 debug symbols from the server.
 
-To run a debuginfod server, you need to do the following:
+To run a ``debuginfod`` server, you need to do the following:
 
--  Ensure that this variable is set in your ``local.conf`` file::
+-  Ensure that ``debuginfod`` is present in :term:`DISTRO_FEATURES`
+   (it already is in ``OpenEmbedded-core`` defaults and ``poky`` reference 
distribution).
+   If not, set in your distro config file or in ``local.conf``::
 
-  PACKAGECONFIG_pn-elfutils-native = "debuginfod libdebuginfod"
+  DISTRO_FEATURES_append = " debuginfod"
 
-   This :term:`PACKAGECONFIG` option enables debuginfod and libdebuginfod for
-   "elfutils-native".
+   This distro feature enables the server and client library in ``elfutils``,
+   and enables ``debuginfod`` support in clients (at the moment, ``gdb`` and 
``binutils``).
 
--  Run the following commands to set up the "debuginfod" server::
+-  Run the following commands to launch the ``debuginfod`` server on the host::
 
   $ oe-debuginfod
 
+-  To use ``debuginfod`` on the target, you need to know the ip:port where
+   ``debuginfod`` is listening on the host (port defaults to 8002), and export
+   that into the shell environment, for example in ``qemu``::
 
-To use debuginfod on the target, you need the following:
+  root@qemux86-64:~# export DEBUGINFOD_URLS="http://192.168.7.1:8002/";
 
--  Ensure that this variable is set in your ``local.conf`` file::
+-  Then debug info fetching should simply work when running the target ``gdb``,
+   ``readelf`` or ``objdump``, for example::
 
-  DEBUGINFOD_URLS = "http://localhost:8002/";
+  root@qemux86-64:~# gdb /bin/cat
+  ...
+  Reading symbols from /bin/cat...
+  Downloading separate debug info for /bin/cat...
+  Reading symbols from 
/home/root/.cache/debuginfod_client/923dc4780cfbc545850c616bffa884b6b5eaf322/debuginfo...
 
-   This :term:`DEBUGINFOD_URLS` option does the client configuration.
+-  It's also possible to use ``debuginfod-find`` to just query the server::
 
-   ::
-
-PACKAGECONFIG_pn-gdb = "debuginfod"
+  root@qemux86-64:~# debuginfod-find debuginfo /bin/ls
+  
/home/root/.cache/debuginfod_client/356edc585f7f82d46f94fcb87a86a3fe2d2e60bd/debuginfo
 
-   This :term:`PACKAGECONFIG` option enables "debuginfod" for "gdb".
 
 Using the gdbserver method
 ~
diff --git a/documentation/ref-manual/variables.rst 
b/documentation/ref-manual/variables.rst
index 3297a69edf..c339d45e18 100644
--- a/documentation/ref-manual/variables.rst
+++ b/documentation/ref-manual/variables.rst
@@ -1523,12 +1523,6 @@ system and gives an overview of their function and 
contents.
 
  DEBIANNAME_${PN} = "dbus-1"
 
-   :term:`DEBUGINFOD_URLS`
-  Points to the URL of the "debuginfod" server. Such that for every
-  debugging information lookup, the debuginfod client will query the
-  server and return the requested information. You set this variable
-  in your ``local.conf`` file.
-
:term:`DEBUG_BUILD`
   Specifies to build packages with debugging information. This
   influences the value of the ``SELECTED_OPTIMIZATION`` variable.
-- 
2.30.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151003): 
https://lists.openembedded.org/g/openembedded-core/message/151003
Mute This Topic: https://lists.openembedded.org/mt/82374688/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] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

[Alexander Kanavin]

On Mon, 26 Apr 2021 at 10:39, Quentin Schulz <
quentin.sch...@streamunlimited.com> wrote:

> > -To run a debuginfod server, you need to do the following:
> > +To run a ``debuginfod`` server, you need to do the following:
> >
> > --  Ensure that this variable is set in your ``local.conf`` file::
> > +-  Ensure that ``debuginfod`` is present in :term:`DISTRO_FEATURES`
> > +   (it already is in ``OpenEmbedded-core`` defaults and ``poky``
> reference distribution).
> > +   If not, set::
>
> I would add where this is supposed to be set (in a distro conf file or
> in local.conf if I'm not mistaken?), since it was an information that
> was there initially.
>
> For the rest:
> Reviewed-by: Quentin Schulz 
>

Thanks, corrected and resent.

Alex

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151004): 
https://lists.openembedded.org/g/openembedded-core/message/151004
Mute This Topic: https://lists.openembedded.org/mt/82373405/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] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

[Quentin Schulz]

Hi Alex,

Could you please add the v2/v3/vwhatever in the mail subject next time?
You can do so by passing `-v 2` to `git format-patch`. This helps us
keep track of which version of a patch should be merged (well, it helps
Michael :p) and on which patch series to answer/rewview.

No need to do it this time :)

Thanks for the patch, LGTM!
Quentin

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151005): 
https://lists.openembedded.org/g/openembedded-core/message/151005
Mute This Topic: https://lists.openembedded.org/mt/82374738/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] gdk-pixbuf: upgrade from 2.40.0 -> 2.42.6

[Alexander Kanavin]

Thanks Changqing, I have an alternative update patch that also simplifies
the custom patch structure. If my approach doesn't work out, I'll take
yours, until then I'd like it to be on hold.

Alex

On Mon, 26 Apr 2021 at 02:55, Changqing Li 
wrote:

> From: Changqing Li 
>
> Signed-off-by: Changqing Li 
> ---
>  ...-around-thumbnailer-cross-compile-fa.patch | 61 ++-
>  .../0004-Do-not-run-tests-when-building.patch | 21 +++
>  ...ailer-and-tests-also-in-cross-builds.patch |  4 +-
>  .../gdk-pixbuf/CVE-2020-29385.patch   | 55 -
>  .../gdk-pixbuf/missing-test-data.patch| 30 -
>  ...-pixbuf_2.40.0.bb => gdk-pixbuf_2.42.6.bb} | 14 ++---
>  6 files changed, 49 insertions(+), 136 deletions(-)
>  delete mode 100644
> meta/recipes-gnome/gdk-pixbuf/gdk-pixbuf/CVE-2020-29385.patch
>  delete mode 100644
> meta/recipes-gnome/gdk-pixbuf/gdk-pixbuf/missing-test-data.patch
>  rename meta/recipes-gnome/gdk-pixbuf/{gdk-pixbuf_2.40.0.bb =>
> gdk-pixbuf_2.42.6.bb} (91%)
>
> diff --git
> a/meta/recipes-gnome/gdk-pixbuf/gdk-pixbuf/0003-target-only-Work-around-thumbnailer-cross-compile-fa.patch
> b/meta/recipes-gnome/gdk-pixbuf/gdk-pixbuf/0003-target-only-Work-around-thumbnailer-cross-compile-fa.patch
> index a9c7600eb3..342c47cca3 100644
> ---
> a/meta/recipes-gnome/gdk-pixbuf/gdk-pixbuf/0003-target-only-Work-around-thumbnailer-cross-compile-fa.patch
> +++
> b/meta/recipes-gnome/gdk-pixbuf/gdk-pixbuf/0003-target-only-Work-around-thumbnailer-cross-compile-fa.patch
> @@ -1,4 +1,4 @@
> -From 2d1b65bd1272ad63b7fbd4babd9a8e8c296d15b5 Mon Sep 17 00:00:00 2001
> +From 483e6818e9aebc5637f41856e327995ef967c451 Mon Sep 17 00:00:00 2001
>  From: Alexander Kanavin 
>  Date: Fri, 22 Feb 2019 13:22:06 +0100
>  Subject: [PATCH] (target only) Work-around thumbnailer and pixdata
> @@ -15,7 +15,6 @@ The upstream issue is
> https://bugzilla.gnome.org/show_bug.cgi?id=779057
>
>  Upstream-Status: Inappropriate [workaround]
>  Signed-off-by: Alexander Kanavin 
> -
>  ---
>   build-aux/gen-thumbnailer.py |  2 --
>   tests/meson.build| 11 +--
> @@ -36,13 +35,13 @@ index 05ac821..c5b99ab 100644
>   if os.name == 'nt':
>   gdk_pixbuf_dll_buildpath = os.path.dirname(args.pixdata)
>  diff --git a/tests/meson.build b/tests/meson.build
> -index 8ed7cc1..e011b77 100644
> +index 3f3ee54..4100520 100644
>  --- a/tests/meson.build
>  +++ b/tests/meson.build
> -@@ -1,12 +1,19 @@
> - # Resources; we cannot use gnome.compile_resources() here, because we
> need to
> - # override the environment in order to use the utilities we just built
> instead
> - # of the system ones
> +@@ -8,13 +8,20 @@ if enabled_loaders.contains('png') and host_system !=
> 'windows'
> +   # Resources; we cannot use gnome.compile_resources() here, because we
> need to
> +   # override the environment in order to use the utilities we just built
> instead
> +   # of the system ones
>  +
>  +if not meson.is_cross_build()
>  +pixdata_binary = gdk_pixbuf_pixdata.full_path()
> @@ -50,30 +49,31 @@ index 8ed7cc1..e011b77 100644
>  +pixdata_binary = 'gdk-pixbuf-pixdata'
>  +endif
>  +
> - resources_c = custom_target('resources.c',
> -   input: 'resources.gresource.xml',
> -   output: 'resources.c',
> -   command: [
> - gen_resources,
> --'--pixdata=@0@'.format(gdk_pixbuf_pixdata.full_path()),
> -+'--pixdata=@0@'.format(pixdata_binary),
> - '--loaders=@0@'.format(loaders_cache.full_path()),
> - '--sourcedir=@0@'.format(meson.current_source_dir()),
> - '--source',
> -@@ -24,7 +31,7 @@ resources_h = custom_target('resources.h',
> -   output: 'resources.h',
> -   command: [
> - gen_resources,
> --'--pixdata=@0@'.format(gdk_pixbuf_pixdata.full_path()),
> -+'--pixdata=@0@'.format(pixdata_binary),
> - '--loaders=@0@'.format(loaders_cache.full_path()),
> - '--sourcedir=@0@'.format(meson.current_source_dir()),
> - '--header',
> +   resources_c = custom_target('resources.c',
> + input: 'resources.gresource.xml',
> + output: 'resources.c',
> + command: [
> +   gen_resources,
> +   '--glib-compile-resources=@0@
> '.format(glib_compile_resources.full_path()),
> +-  '--pixdata=@0@'.format(gdk_pixbuf_pixdata.full_path()),
> ++  '--pixdata=@0@'.format(pixdata_binary),
> +   '--loaders=@0@'.format(loaders_cache.full_path()),
> +   '--sourcedir=@0@'.format(meson.current_source_dir()),
> +   '--source',
> +@@ -33,7 +40,7 @@ if enabled_loaders.contains('png') and host_system !=
> 'windows'
> + command: [
> +   gen_resources,
> +   '--glib-compile-resources=@0@
> '.format(glib_compile_resources.full_path()),
> +-  '--pixdata=@0@'.format(gdk_pixbuf_pixdata.full_path()),
> ++  '--pixdata=@0@'.format(pixdata_binary),
> +   '--loaders=@0@'.format(loaders_cache.full_path()),
> +   '--sourcedir=@0@'.format(meson.current_source_dir()),
> +   '--header',
>  diff --git a/thumbnailer/meson.build b/thumbnailer/meson.build
> -index a

Re: [OE-core] [PATCH] ca-certificates: Fix openssl runtime cert dependencies

[Richard Purdie]

On Fri, 2021-04-23 at 20:56 +0100, Andrei Gherzan wrote:
> On Sun, 18 Apr 2021, at 23:53, Khem Raj wrote:
> > With commit dc778c70449ee5401b5a24ad18b22b88338c47c5, dependency was
> > moved to openssl-bin which in itself was a fine change, but dropping
> > dependency on openssl too should have been kept along, dropping this
> > meant that openssl binary wont be able to validate secure connections as
> > the CApath files wont be installed, which infact are required for
> > openssl bins to work, following call e.g. fails
> > 
> > $ openssl s_client -connect google.com:443
> > 
> > 
> > New, TLSv1.3, Cipher is TLS_AES_256_GCM_SHA384
> > Server public key is 256 bit
> > Secure Renegotiation IS NOT supported
> > Compression: NONE
> > Expansion: NONE
> > No ALPN negotiated
> > Early data was not sent
> > Verify return code: 20 (unable to get local issuer certificate)
> > 
> > 
> > The local issuer certs are not found in default location
> > /usr/lib/ssh-1.1/certs, this dir and its content is installed by openssl 
> > package
> > therefore re-add the dependency on openssl
> 
> Good idea for a ptest. Change looks good to me.

Might be worth a bug opening so we don't forget?

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151007): 
https://lists.openembedded.org/g/openembedded-core/message/151007
Mute This Topic: https://lists.openembedded.org/mt/82197118/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/6] libseccomp: move recipe from meta-security to core

[Armin Kuster]

On 4/25/21 5:10 PM, Khem Raj wrote:
> On Sun, Apr 25, 2021 at 11:26 AM akuster808  wrote:
>>
>>
>> On 4/24/21 3:16 PM, Khem Raj wrote:
>>> riscv32 is not happy
>> that is not supported by libseccomp per their supported arch list. I
>> came across that yesterday.
> I think the problem is when we enable it by default in DISTRO_FEATURES
> perhaps we should add an explicit
>
> DISTRO_FEATURES_remove_riscv32 = "seccomp"
>
> in default-distrovars.inc
sounds good. libseccomp hit master this morning.

-armin
>
>>> ERROR: Nothing PROVIDES 'libseccomp' (but
>>> /home/jenkins/oe/world/yoe/sources/openembedded-core/meta/recipes-core/systemd/systemd_247.6.bb,
>>> /home/jenkins/oe/world/yoe/sources/openembedded-core/meta/recipes-support/gnutls/gnutls_3.7.1.bb
>>> DEPENDS on or otherwise requires it)
>>> libseccomp was skipped: incompatible with host riscv32-yoe-linux (not
>>> in COMPATIBLE_HOST)
>> I suspect we need to exclude the arch for now. I didn't notice any patch
>> to add that yet.
>>
>> -armin
>>> see
>>> http://jenkins.nas-admin.org/view/OE/job/oe_world_qemuriscv32/1123/consoleFull
>>>
>>> On Sat, Apr 24, 2021 at 8:56 AM Armin Kuster  wrote:
 ptest results:
 Regression Test Summary
  tests run: 1404
  tests skipped: 369
  tests passed: 1402
  tests failed: 2
  tests errored: 154

 Add feature_check so that the other recipes who can take
 advantage of this funtionality can enable it.

 Signed-off-by: Armin Kuster 
 ---
  .../libseccomp/files/run-ptest|  4 ++
  .../libseccomp/libseccomp_2.5.1.bb| 49 +++
  2 files changed, 53 insertions(+)
  create mode 100644 meta/recipes-support/libseccomp/files/run-ptest
  create mode 100644 meta/recipes-support/libseccomp/libseccomp_2.5.1.bb

 diff --git a/meta/recipes-support/libseccomp/files/run-ptest 
 b/meta/recipes-support/libseccomp/files/run-ptest
 new file mode 100644
 index 000..54b4a63cd2c
 --- /dev/null
 +++ b/meta/recipes-support/libseccomp/files/run-ptest
 @@ -0,0 +1,4 @@
 +#!/bin/sh
 +
 +cd tests
 +./regression -a
 diff --git a/meta/recipes-support/libseccomp/libseccomp_2.5.1.bb 
 b/meta/recipes-support/libseccomp/libseccomp_2.5.1.bb
 new file mode 100644
 index 000..667d5da8242
 --- /dev/null
 +++ b/meta/recipes-support/libseccomp/libseccomp_2.5.1.bb
 @@ -0,0 +1,49 @@
 +SUMMARY = "interface to seccomp filtering mechanism"
 +DESCRIPTION = "The libseccomp library provides and easy to use, platform 
 independent,interface to the Linux Kernel's syscall filtering mechanism: 
 seccomp."
 +SECTION = "security"
 +LICENSE = "LGPL-2.1"
 +LIC_FILES_CHKSUM = 
 "file://LICENSE;beginline=0;endline=1;md5=8eac08d22113880357ceb8e7c37f989f"
 +
 +DEPENDS += "gperf-native"
 +
 +SRCREV = "4bf70431a339a2886ab8c82e9a45378f30c6e6c7"
 +
 +SRC_URI = "git://github.com/seccomp/libseccomp.git;branch=release-2.5 \
 +   file://run-ptest \
 +   "
 +
 +COMPATIBLE_HOST_riscv32 = "null"
 +
 +S = "${WORKDIR}/git"
 +
 +inherit autotools-brokensep pkgconfig ptest features_check
 +
 +REQUIRED_DISTRO_FEATURES = "seccomp"
 +
 +PACKAGECONFIG ??= ""
 +PACKAGECONFIG[python] = "--enable-python, --disable-python, python3"
 +
 +DISABLE_STATIC = ""
 +
 +do_compile_ptest() {
 +oe_runmake -C tests check-build
 +}
 +
 +do_install_ptest() {
 +install -d ${D}${PTEST_PATH}/tests
 +install -d ${D}${PTEST_PATH}/tools
 +for file in $(find tests/* -executable -type f); do
 +install -m 744 ${S}/${file} ${D}/${PTEST_PATH}/tests
 +done
 +for file in $(find tests/*.tests -type f); do
 +install -m 744 ${S}/${file} ${D}/${PTEST_PATH}/tests
 +done
 +for file in $(find tools/* -executable -type f); do
 +install -m 744 ${S}/${file} ${D}/${PTEST_PATH}/tools
 +done
 +}
 +
 +FILES_${PN} = "${bindir} ${libdir}/${BPN}.so*"
 +FILES_${PN}-dbg += "${libdir}/${PN}/tests/.debug/* 
 ${libdir}/${PN}/tools/.debug"
 +
 +RDEPENDS_${PN}-ptest = "coreutils bash"
 --
 2.25.1


 

>>


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151008): 
https://lists.openembedded.org/g/openembedded-core/message/151008
Mute This Topic: https://lists.openembedded.org/mt/82336292/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] Binutils: Fix CVE-2021-20197

[Steve Sakoman]

On Sun, Apr 25, 2021 at 2:44 AM Vinay Kumar  wrote:
>
> Source: git://sourceware.org/git/binutils-gdb.git
> Tracking -- https://sourceware.org/bugzilla/show_bug.cgi?id=26945
>
> Backported upstream commit d3edaa91d4cf7202ec14342410194841e2f67f12 to
> binutils-2.34 source, along with commit id dependencies
> (8e03235147a9e774d3ba084e93c2daaa94d1cec and 
> 365f5fb6d0f0da83817431a275e99e6f6babbe04).
>
> Upstream-Status: Backport 
> [https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=d3edaa91d4cf7202ec14342410194841e2f67f12]
>
> Signed-off-by: Vinay Kumar 
> ---
> This patch is verified using poky image build and toolchain build for
> ARM and X86_64 targets. Also, verified for binutils regression testing with 
> ARM target.

Unfortunately I am getting autobuilder compilation errors on the
meta-mingw build:

https://errors.yoctoproject.org/Errors/Details/577768/
https://errors.yoctoproject.org/Errors/Details/577769/

Steve

>
>  .../binutils/binutils-2.34.inc|   1 +
>  .../binutils/binutils/CVE-2021-20197.patch| 479 ++
>  2 files changed, 480 insertions(+)
>  create mode 100644 
> meta/recipes-devtools/binutils/binutils/CVE-2021-20197.patch
>
> diff --git a/meta/recipes-devtools/binutils/binutils-2.34.inc 
> b/meta/recipes-devtools/binutils/binutils-2.34.inc
> index f557fe970c..a586faf5ab 100644
> --- a/meta/recipes-devtools/binutils/binutils-2.34.inc
> +++ b/meta/recipes-devtools/binutils/binutils-2.34.inc
> @@ -46,5 +46,6 @@ SRC_URI = "\
>   file://0001-gas-improve-reproducibility-for-stabs-debugging-data.patch \
>   file://CVE-2020-16592.patch \
>   file://CVE-2020-16598.patch \
> + file://CVE-2021-20197.patch \
>  "
>  S  = "${WORKDIR}/git"
> diff --git a/meta/recipes-devtools/binutils/binutils/CVE-2021-20197.patch 
> b/meta/recipes-devtools/binutils/binutils/CVE-2021-20197.patch
> new file mode 100644
> index 00..c207afec4d
> --- /dev/null
> +++ b/meta/recipes-devtools/binutils/binutils/CVE-2021-20197.patch
> @@ -0,0 +1,479 @@
> +From d3edaa91d4cf7202ec14342410194841e2f67f12 Mon Sep 17 00:00:00 2001
> +From: Alan Modra 
> +Date: Fri, 26 Feb 2021 11:30:32 +1030
> +Subject: [PATCH] Reinstate various pieces backed out from smart_rename 
> changes
> +
> +In the interests of a stable release various last minute smart_rename
> +patches were backed out of the 2.36 branch.  The main reason to
> +reinstate some of those backed out changes here is to make necessary
> +followup fixes to commit 8e03235147a9 simple cherry-picks from
> +mainline.  A secondary reason is that ar -M support isn't fixed for
> +pr26945 without this patch.
> +
> +PR 26945
> +* ar.c: Don't include libbfd.h.
> +(write_archive): Replace xmalloc+strcpy with xstrdup.
> +* arsup.c (temp_name, real_ofd): New static variables.
> +(ar_open): Use make_tempname and bfd_fdopenw.
> +(ar_save): Adjust to suit ar_open changes.
> +* objcopy.c: Don't include libbfd.h.
> +* rename.c: Rename and reorder variables.
> +
> +(cherry picked from commit 95b91a043aeaeb546d2fea556d84a2de1e917770)
> +
> +Upstream-Status: Backport 
> [https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=d3edaa91d4cf7202ec14342410194841e2f67f12]
> +CVE: CVE-2021-20197
> +Signed-off-by: Vinay Kumar 
> +
> +---
> + bfd/bfd-in2.h  |  2 ++
> + bfd/opncls.c   | 33 +
> + binutils/ar.c  | 15 +++-
> + binutils/arsup.c   | 37 +++-
> + binutils/bucomm.c  |  4 +--
> + binutils/bucomm.h  |  5 ++--
> + binutils/objcopy.c | 37 +---
> + binutils/rename.c  | 61 ++
> + 8 files changed, 110 insertions(+), 84 deletions(-)
> +
> +diff --git a/bfd/bfd-in2.h b/bfd/bfd-in2.h
> +index 2e453c50c18..e53f54a8ab7 100644
> +--- a/bfd/bfd-in2.h
>  b/bfd/bfd-in2.h
> +@@ -588,6 +588,8 @@ bfd *bfd_openr (const char *filename, const char 
> *target);
> +
> + bfd *bfd_fdopenr (const char *filename, const char *target, int fd);
> +
> ++bfd *bfd_fdopenw (const char *filename, const char *target, int fd);
> ++
> + bfd *bfd_openstreamr (const char * filename, const char * target,
> + void * stream);
> +
> +diff --git a/bfd/opncls.c b/bfd/opncls.c
> +index a03ad51c8fa..f9da97ed710 100644
> +--- a/bfd/opncls.c
>  b/bfd/opncls.c
> +@@ -370,6 +370,39 @@ bfd_fdopenr (const char *filename, const char *target, 
> int fd)
> +   return bfd_fopen (filename, target, mode, fd);
> + }
> +
> ++/*
> ++FUNCTION
> ++  bfd_fdopenw
> ++
> ++SYNOPSIS
> ++  bfd *bfd_fdopenw (const char *filename, const char *target, int fd);
> ++
> ++DESCRIPTION
> ++  <> is exactly like <> with the exception 
> that
> ++  the resulting BFD is suitable for output.
> ++*/
> ++
> ++bfd *
> ++bfd_fdopenw (const char *filename, const char *target, int fd)
> ++{
> ++  bfd *out = bfd_fdopenr (filename, target, fd);
> ++
> ++  if (out != NULL)
> ++{
> ++  if (!bfd_wri

[OE-core] [PATCH 2/2] qemurunner: Add support for qmp commands

[Saul Wold]

This adds support for the Qemu Machine Protocol [0] extending
the current dump process for Host and Target. The commands are
added in the testimage.bbclass.

Currently, we setup qemu to stall until qmp gets connected and
sends the initialization and continue commands, this works
correctly. If the UNIX Socket does not exist, we wait an timeout
to ensure to socket file is created.

With this version, the monitor_dumper is created in OEQemuTarget
but then set in OESSHTarget as that's where we get the SSH failure
happens. Python's @property is used to create a setter/getter type
of setup in OESSHTarget to get overridden by OEQemuTarget.

By default the data is currently dumped to files for each command in
TMPDIR/log/runtime-hostdump/_qmp/unknown__qemu_monitor as
this is the naming convenstion in the dump.py code.

We use the qmp.py from qemu, which needs to get installed in the
recipe-sysroot-native of the target image.

[0] https://github.com/qemu/qemu/blob/master/docs/interop/qmp-spec.txt

Signed-off-by: Saul Wold 
---
 meta/classes/testimage.bbclass  |  6 +++
 meta/lib/oeqa/core/target/qemu.py   |  6 +++
 meta/lib/oeqa/core/target/ssh.py| 17 ++-
 meta/lib/oeqa/targetcontrol.py  |  3 ++
 meta/lib/oeqa/utils/dump.py | 32 +++--
 meta/lib/oeqa/utils/qemurunner.py   | 70 -
 meta/recipes-devtools/qemu/qemu.inc |  2 +-
 7 files changed, 129 insertions(+), 7 deletions(-)

diff --git a/meta/classes/testimage.bbclass b/meta/classes/testimage.bbclass
index e6137595031..43de9d4d767 100644
--- a/meta/classes/testimage.bbclass
+++ b/meta/classes/testimage.bbclass
@@ -127,6 +127,11 @@ testimage_dump_host () {
 netstat -an
 }
 
+testimage_dump_monitor () {
+query-status
+query-block
+}
+
 python do_testimage() {
 testimage_main(d)
 }
@@ -320,6 +325,7 @@ def testimage_main(d):
 target_kwargs['powercontrol_extra_args'] = 
d.getVar("TEST_POWERCONTROL_EXTRA_ARGS") or ""
 target_kwargs['serialcontrol_cmd'] = d.getVar("TEST_SERIALCONTROL_CMD") or 
None
 target_kwargs['serialcontrol_extra_args'] = 
d.getVar("TEST_SERIALCONTROL_EXTRA_ARGS") or ""
+target_kwargs['testimage_dump_monitor'] = 
d.getVar("testimage_dump_monitor") or ""
 target_kwargs['testimage_dump_target'] = d.getVar("testimage_dump_target") 
or ""
 
 def export_ssh_agent(d):
diff --git a/meta/lib/oeqa/core/target/qemu.py 
b/meta/lib/oeqa/core/target/qemu.py
index 792efca1f84..4a5df4a9a81 100644
--- a/meta/lib/oeqa/core/target/qemu.py
+++ b/meta/lib/oeqa/core/target/qemu.py
@@ -12,6 +12,7 @@ from collections import defaultdict
 
 from .ssh import OESSHTarget
 from oeqa.utils.qemurunner import QemuRunner
+from oeqa.utils.dump import MonitorDumper
 from oeqa.utils.dump import TargetDumper
 
 supported_fstypes = ['ext3', 'ext4', 'cpio.gz', 'wic']
@@ -43,6 +44,11 @@ class OEQemuTarget(OESSHTarget):
  dump_host_cmds=dump_host_cmds, logger=logger,
  serial_ports=serial_ports, boot_patterns = 
boot_patterns, 
  use_ovmf=ovmf, tmpfsdir=tmpfsdir)
+dump_monitor_cmds = kwargs.get("testimage_dump_monitor")
+self.monitor_dumper = MonitorDumper(dump_monitor_cmds, dump_dir, 
self.runner)
+if self.monitor_dumper:
+self.monitor_dumper.create_dir("qmp")
+
 dump_target_cmds = kwargs.get("testimage_dump_target")
 self.target_dumper = TargetDumper(dump_target_cmds, dump_dir, 
self.runner)
 self.target_dumper.create_dir("qemu")
diff --git a/meta/lib/oeqa/core/target/ssh.py b/meta/lib/oeqa/core/target/ssh.py
index 461448dbc56..923a223b25b 100644
--- a/meta/lib/oeqa/core/target/ssh.py
+++ b/meta/lib/oeqa/core/target/ssh.py
@@ -43,6 +43,7 @@ class OESSHTarget(OETarget):
 if port:
 self.ssh = self.ssh + [ '-p', port ]
 self.scp = self.scp + [ '-P', port ]
+self._monitor_dumper = None
 
 def start(self, **kwargs):
 pass
@@ -50,6 +51,15 @@ class OESSHTarget(OETarget):
 def stop(self, **kwargs):
 pass
 
+@property
+def monitor_dumper(self):
+return self._monitor_dumper
+
+@monitor_dumper.setter
+def monitor_dumper(self, dumper):
+self._monitor_dumper = dumper
+self.monitor_dumper.dump_monitor()
+
 def _run(self, command, timeout=None, ignore_status=True):
 """
 Runs command in target using SSHProcess.
@@ -87,9 +97,14 @@ class OESSHTarget(OETarget):
 processTimeout = self.timeout
 
 status, output = self._run(sshCmd, processTimeout, True)
-self.logger.debug('Command: %s\nOutput:  %s\n' % (command, output))
+self.logger.debug('Command: %s\nStatus: %d Output:  %s\n' % (command, 
status, output))
 if (status == 255) and (('No route to host') in output):
+if self.monitor_dumper:
+self.monitor_dumper.dump_monitor()
+if status == 255:
 

[OE-core] [PATCH 0/2] Enable QMP Dumping for testimage

[Saul Wold]

This is the forth (and maybe final) pass at enabling getting debug
information from QEMU via the Qemu Machine Protocol interface. The
Qemu source provides a qmp.py module which is installed in the 
recipe-sysroot-native site-packages.

The initial commands that are issued to qmp when a failure is detected
are: query-status and query-block. The output goes into formated json
files as follows:

tmp/log/runtime-hostdump/202101061054_qmp
├── qmp_00_query-block
├── qmp_00_query-status
├── qmp_01_query-block
└── qmp_01_query-status

This has been tested by calling the monitor_dump code directly in the
target/ssh.py code instead of waiting for a given failure.

This version adds a time-out loop to check for the Unix Socket file
that was being problematic on the CentOS-8 builders.

Sau!

Saul Wold (2):
  qemu-system-native: install qmp python module
  qemurunner: Add support for qmp commands

 meta/classes/testimage.bbclass|  6 
 meta/lib/oeqa/core/target/qemu.py |  6 
 meta/lib/oeqa/core/target/ssh.py  | 17 +-
 meta/lib/oeqa/targetcontrol.py|  3 ++
 meta/lib/oeqa/utils/dump.py   | 31 +--
 meta/lib/oeqa/utils/qemurunner.py | 29 -
 .../qemu/qemu-system-native_5.1.0.bb  |  4 +++
 7 files changed, 91 insertions(+), 5 deletions(-)

-- 
2.25.1


Saul Wold (2):
  qemu-system-native: install qmp python module
  qemurunner: Add support for qmp commands

 meta/classes/testimage.bbclass|  6 ++
 meta/lib/oeqa/core/target/qemu.py |  6 ++
 meta/lib/oeqa/core/target/ssh.py  | 17 +-
 meta/lib/oeqa/targetcontrol.py|  3 +
 meta/lib/oeqa/utils/dump.py   | 31 +-
 meta/lib/oeqa/utils/qemurunner.py | 56 ++-
 .../qemu/qemu-system-native_5.2.0.bb  |  5 ++
 7 files changed, 118 insertions(+), 6 deletions(-)

-- 
2.25.1

*** BLURB HERE ***

Saul Wold (2):
  qemu-system-native: install qmp python module
  qemurunner: Add support for qmp commands

 meta/classes/testimage.bbclass|  6 ++
 meta/lib/oeqa/core/target/qemu.py |  6 ++
 meta/lib/oeqa/core/target/ssh.py  | 17 -
 meta/lib/oeqa/targetcontrol.py|  3 +
 meta/lib/oeqa/utils/dump.py   | 32 -
 meta/lib/oeqa/utils/qemurunner.py | 70 ++-
 .../qemu/qemu-system-native_5.2.0.bb  |  5 ++
 meta/recipes-devtools/qemu/qemu.inc   |  2 +-
 8 files changed, 134 insertions(+), 7 deletions(-)

-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151012): 
https://lists.openembedded.org/g/openembedded-core/message/151012
Mute This Topic: https://lists.openembedded.org/mt/82379591/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] qemu-system-native: install qmp python module

[Saul Wold]

The qmp python module supports the Qemu Machine Protocol [0].
This module needs to be installed in a known location so the
qemurunner python script can find the qmp module.

This change causes it to be installed in the recipe-sysroot-native
of the target image and that directory can be added to the python
sys.path that needs to use the qmp.py module.

[0] https://github.com/qemu/qemu/blob/master/docs/interop/qmp-spec.txt

Signed-off-by: Saul Wold 
---
 meta/recipes-devtools/qemu/qemu-system-native_5.2.0.bb | 5 +
 1 file changed, 5 insertions(+)

diff --git a/meta/recipes-devtools/qemu/qemu-system-native_5.2.0.bb 
b/meta/recipes-devtools/qemu/qemu-system-native_5.2.0.bb
index 222b55cbc6e..390dadea48a 100644
--- a/meta/recipes-devtools/qemu/qemu-system-native_5.2.0.bb
+++ b/meta/recipes-devtools/qemu/qemu-system-native_5.2.0.bb
@@ -1,5 +1,7 @@
 BPN = "qemu"
 
+inherit python3-dir
+
 require qemu-native.inc
 
 # As some of the files installed by qemu-native and qemu-system-native
@@ -23,4 +25,7 @@ do_install_append() {
 rm -f ${D}${datadir}/qemu/trace-events-all
 rm -rf ${D}${datadir}/qemu/keymaps
 rm -rf ${D}${datadir}/icons/
+
+# Install qmp.py to be used with testimage
+install -D ${S}/python/qemu/qmp.py ${D}${PYTHON_SITEPACKAGES_DIR}/qmp.py
 }
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151010): 
https://lists.openembedded.org/g/openembedded-core/message/151010
Mute This Topic: https://lists.openembedded.org/mt/82379589/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] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

[Alexander Kanavin]

Thanks, I'll remember to send versioned patches!

Alex

On Mon, 26 Apr 2021 at 18:11, Michael Opdenacker <
michael.opdenac...@bootlin.com> wrote:

> Hi Alex,
>
> On 4/26/21 12:46 PM, Quentin Schulz wrote:
> > Hi Alex,
> >
> > Could you please add the v2/v3/vwhatever in the mail subject next time?
> > You can do so by passing `-v 2` to `git format-patch`. This helps us
> > keep track of which version of a patch should be merged (well, it helps
> > Michael :p) and on which patch series to answer/rewview.
> >
> > No need to do it this time :)
>
>
> I agree :)
>
> Thanks for the patch.
>
> Reviewed-by: Michael Opdenacker 
>
> Merged into the "master-next" branch of the yocto-docs repository
> (http://git.yoctoproject.org/cgit/cgit.cgi/yocto-docs/log/?h=master-next).
> I should send a new pull-request for inclusion in "master" by the end of
> the week.
>
> Thanks again,
>
> Michael.
>
> --
> Michael Opdenacker, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151013): 
https://lists.openembedded.org/g/openembedded-core/message/151013
Mute This Topic: https://lists.openembedded.org/mt/82374738/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] gnutls: Point to staging area for finding seccomp libs and includes

[Khem Raj]

This ensures that if libseccomp is installed on build host then it does
not resort to use it.

Fixes
checking for libseccomp... (cached) yes
checking how to link with libseccomp... /usr/lib/libseccomp.so

Signed-off-by: Khem Raj 
Cc: Armin Kuster 
---
 meta/recipes-support/gnutls/gnutls_3.7.1.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-support/gnutls/gnutls_3.7.1.bb 
b/meta/recipes-support/gnutls/gnutls_3.7.1.bb
index 3e1958c969..350d0a018b 100644
--- a/meta/recipes-support/gnutls/gnutls_3.7.1.bb
+++ b/meta/recipes-support/gnutls/gnutls_3.7.1.bb
@@ -31,7 +31,7 @@ PACKAGECONFIG ??= "libidn  
${@bb.utils.filter('DISTRO_FEATURES', 'seccomp', d)}"
 
 # You must also have CONFIG_SECCOMP enabled in the kernel for
 # seccomp to work.
-PACKAGECONFIG[seccomp] = "ac_cv_libseccomp=yes,ac_cv_libseccomp=no,libseccomp"
+PACKAGECONFIG[seccomp] = 
"--with-libseccomp-prefix=${STAGING_EXECPREFIXDIR},ac_cv_libseccomp=no,libseccomp"
 PACKAGECONFIG[libidn] = "--with-idn,--without-idn,libidn2"
 PACKAGECONFIG[libtasn1] = 
"--with-included-libtasn1=no,--with-included-libtasn1,libtasn1"
 PACKAGECONFIG[p11-kit] = "--with-p11-kit,--without-p11-kit,p11-kit"
-- 
2.31.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151014): 
https://lists.openembedded.org/g/openembedded-core/message/151014
Mute This Topic: https://lists.openembedded.org/mt/82384782/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] gnutls: Point to staging area for finding seccomp libs and includes

[Armin Kuster]

On 4/26/21 11:06 AM, Khem Raj wrote:
> This ensures that if libseccomp is installed on build host then it does
> not resort to use it.
>
> Fixes
> checking for libseccomp... (cached) yes
> checking how to link with libseccomp... /usr/lib/libseccomp.so

Thanks
-armin
>
> Signed-off-by: Khem Raj 
> Cc: Armin Kuster 
> ---
>  meta/recipes-support/gnutls/gnutls_3.7.1.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-support/gnutls/gnutls_3.7.1.bb 
> b/meta/recipes-support/gnutls/gnutls_3.7.1.bb
> index 3e1958c969..350d0a018b 100644
> --- a/meta/recipes-support/gnutls/gnutls_3.7.1.bb
> +++ b/meta/recipes-support/gnutls/gnutls_3.7.1.bb
> @@ -31,7 +31,7 @@ PACKAGECONFIG ??= "libidn  
> ${@bb.utils.filter('DISTRO_FEATURES', 'seccomp', d)}"
>  
>  # You must also have CONFIG_SECCOMP enabled in the kernel for
>  # seccomp to work.
> -PACKAGECONFIG[seccomp] = 
> "ac_cv_libseccomp=yes,ac_cv_libseccomp=no,libseccomp"
> +PACKAGECONFIG[seccomp] = 
> "--with-libseccomp-prefix=${STAGING_EXECPREFIXDIR},ac_cv_libseccomp=no,libseccomp"
>  PACKAGECONFIG[libidn] = "--with-idn,--without-idn,libidn2"
>  PACKAGECONFIG[libtasn1] = 
> "--with-included-libtasn1=no,--with-included-libtasn1,libtasn1"
>  PACKAGECONFIG[p11-kit] = "--with-p11-kit,--without-p11-kit,p11-kit"


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

[OE-core] LTP drop MUSL specific patch

[Petr Vorel]

Hi Khem, Richard,

I'd like to replace MUSL specific patch
meta/recipes-extended/ltp/ltp/0001-build-Add-option-to-select-libc-implementation.patch
with just removing broken files. We use this in upstream CI for Alpine, I ported
it to Buildroot [2], where works well. It's better because it does allow to
handle MUSL without rebasing this patch.

I was looking into the docs, it looks to me that do_configure [3] might be good 
hook
to add it into. But it looks like by default it contains oe_runmake clean [4].
Or should I use different hook? And how to find the default content of chosen
hook? (if I need to extend it, I'll have to provide the original as well)

Thanks for info.

Kind regards,
Petr

[1] https://github.com/linux-test-project/ltp/blob/master/travis/alpine.sh#L35
[2] 
https://git.busybox.net/buildroot/tree/package/ltp-testsuite/ltp-testsuite.mk#n72
[3] 
https://www.yoctoproject.org/docs/2.5/dev-manual/dev-manual.html#new-dependencies
[4] 
https://www.yoctoproject.org/docs/2.5/ref-manual/ref-manual.html#ref-tasks-configure


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151016): 
https://lists.openembedded.org/g/openembedded-core/message/151016
Mute This Topic: https://lists.openembedded.org/mt/82386051/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] LTP drop MUSL specific patch

[Andre McCurdy]

On Mon, Apr 26, 2021 at 11:53 AM Petr Vorel  wrote:
>
> Hi Khem, Richard,
>
> I'd like to replace MUSL specific patch
> meta/recipes-extended/ltp/ltp/0001-build-Add-option-to-select-libc-implementation.patch
> with just removing broken files. We use this in upstream CI for Alpine, I 
> ported
> it to Buildroot [2], where works well. It's better because it does allow to
> handle MUSL without rebasing this patch.
>
> I was looking into the docs, it looks to me that do_configure [3] might be 
> good hook
> to add it into. But it looks like by default it contains oe_runmake clean [4].
> Or should I use different hook? And how to find the default content of chosen
> hook? (if I need to extend it, I'll have to provide the original as well)

Changes to source files, including removing them, should be done as
part of the do_patch task. Since do_patch is implemented in python you
can't simply _append shell script commands to it, but you can add a
shell function via do_patch[postfuncs]. If you grep for
"do_patch[postfuncs]" in oe-core you should find a few examples.

> Thanks for info.
>
> Kind regards,
> Petr
>
> [1] https://github.com/linux-test-project/ltp/blob/master/travis/alpine.sh#L35
> [2] 
> https://git.busybox.net/buildroot/tree/package/ltp-testsuite/ltp-testsuite.mk#n72
> [3] 
> https://www.yoctoproject.org/docs/2.5/dev-manual/dev-manual.html#new-dependencies
> [4] 
> https://www.yoctoproject.org/docs/2.5/ref-manual/ref-manual.html#ref-tasks-configure
>
>
> 
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151017): 
https://lists.openembedded.org/g/openembedded-core/message/151017
Mute This Topic: https://lists.openembedded.org/mt/82386051/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] oe-time-dd-test.sh: collect cooker log when timeout is exceeded

[Sakib Sajal]

Collect the last 30 lines from the cooker.log
whenever the timeout is exceeded.

Signed-off-by: Sakib Sajal 
---
 scripts/oe-time-dd-test.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scripts/oe-time-dd-test.sh b/scripts/oe-time-dd-test.sh
index 459071e732..46b218b76a 100755
--- a/scripts/oe-time-dd-test.sh
+++ b/scripts/oe-time-dd-test.sh
@@ -25,4 +25,5 @@ timeout ${TIMEOUT} dd if=/dev/zero of=oe-time-dd-test.dat 
bs=1024 count=$1 conv=
 if [ $? -ne 0 ]; then
echo "Timeout used: ${TIMEOUT}"
top -c -b -n1 -w 512
+   tail -30 tmp*/log/cooker/*/console-latest.log
 fi
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151018): 
https://lists.openembedded.org/g/openembedded-core/message/151018
Mute This Topic: https://lists.openembedded.org/mt/82387582/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] license_image.bbclass: Fix symlink to generic license files

[Reto Schneider]

From: Reto Schneider 

Link to the canonical filename of a license as only this one exists.

Fixes commit 70fe71dd18ea675f35581db4a61fda137f8bf
[license_image.bbclass: use canonical name for license files].

Signed-off-by: Reto Schneider 
---
 meta/classes/license_image.bbclass | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/classes/license_image.bbclass 
b/meta/classes/license_image.bbclass
index b9a0f2359b..73cebb4d55 100644
--- a/meta/classes/license_image.bbclass
+++ b/meta/classes/license_image.bbclass
@@ -144,12 +144,13 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
 continue
 
 # Make sure we use only canonical name for the license 
file
-rootfs_license = os.path.join(rootfs_license_dir, 
"generic_%s" % generic_lic)
+generic_lic_file = "generic_%s" % generic_lic
+rootfs_license = os.path.join(rootfs_license_dir, 
generic_lic_file)
 if not os.path.exists(rootfs_license):
 oe.path.copyhardlink(pkg_license, rootfs_license)
 
 if not os.path.exists(pkg_rootfs_license):
-os.symlink(os.path.join('..', lic), 
pkg_rootfs_license)
+os.symlink(os.path.join('..', generic_lic_file), 
pkg_rootfs_license)
 else:
 if (oe.license.license_ok(canonical_license(d,
 lic), bad_licenses) == False or
-- 
2.29.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151020): 
https://lists.openembedded.org/g/openembedded-core/message/151020
Mute This Topic: https://lists.openembedded.org/mt/8237/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] license_image.bbclass: Detect broken symlinks

[Reto Schneider]

From: Reto Schneider 

Find and report symlinks which point to a non-existing file.

Signed-off-by: Reto Schneider 
---
 meta/classes/license_image.bbclass | 15 +--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/meta/classes/license_image.bbclass 
b/meta/classes/license_image.bbclass
index c96b032ebd..b9a0f2359b 100644
--- a/meta/classes/license_image.bbclass
+++ b/meta/classes/license_image.bbclass
@@ -1,3 +1,5 @@
+ROOTFS_LICENSE_DIR = "${IMAGE_ROOTFS}/usr/share/common-licenses"
+
 python write_package_manifest() {
 # Get list of installed packages
 license_image_dir = d.expand('${LICENSE_DIRECTORY}/${IMAGE_NAME}')
@@ -104,8 +106,7 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
 copy_lic_manifest = d.getVar('COPY_LIC_MANIFEST')
 copy_lic_dirs = d.getVar('COPY_LIC_DIRS')
 if rootfs and copy_lic_manifest == "1":
-rootfs_license_dir = os.path.join(d.getVar('IMAGE_ROOTFS'), 
-'usr', 'share', 'common-licenses')
+rootfs_license_dir = d.getVar('ROOTFS_LICENSE_DIR')
 bb.utils.mkdirhier(rootfs_license_dir)
 rootfs_license_manifest = os.path.join(rootfs_license_dir,
 os.path.split(license_manifest)[1])
@@ -267,3 +268,13 @@ python do_populate_lic_deploy() {
 addtask populate_lic_deploy before do_build after do_image_complete
 do_populate_lic_deploy[recrdeptask] += "do_populate_lic do_deploy"
 
+python license_qa_dead_symlink() {
+import os
+
+for root, dirs, files in os.walk(d.getVar('ROOTFS_LICENSE_DIR')):
+for file in files:
+full_path = root + "/" + file
+if os.path.islink(full_path) and not os.path.exists(full_path):
+bb.error("broken symlink: " + full_path)
+}
+IMAGE_QA_COMMANDS += "license_qa_dead_symlink"
-- 
2.29.2


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

[OE-core] [PATCH v2 2/2] license_image.bbclass: Fix symlink to generic license files

[Reto Schneider]

From: Reto Schneider 

Link to the canonical filename of a license as only this one exists.

Fixes commit 670fe71dd18ea675f35581db4a61fda137f8bf00
[license_image.bbclass: use canonical name for license files].

Signed-off-by: Reto Schneider 
---
Version 2: Fix link to commit

 meta/classes/license_image.bbclass | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/classes/license_image.bbclass 
b/meta/classes/license_image.bbclass
index b9a0f2359b..73cebb4d55 100644
--- a/meta/classes/license_image.bbclass
+++ b/meta/classes/license_image.bbclass
@@ -144,12 +144,13 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
 continue
 
 # Make sure we use only canonical name for the license 
file
-rootfs_license = os.path.join(rootfs_license_dir, 
"generic_%s" % generic_lic)
+generic_lic_file = "generic_%s" % generic_lic
+rootfs_license = os.path.join(rootfs_license_dir, 
generic_lic_file)
 if not os.path.exists(rootfs_license):
 oe.path.copyhardlink(pkg_license, rootfs_license)
 
 if not os.path.exists(pkg_rootfs_license):
-os.symlink(os.path.join('..', lic), 
pkg_rootfs_license)
+os.symlink(os.path.join('..', generic_lic_file), 
pkg_rootfs_license)
 else:
 if (oe.license.license_ok(canonical_license(d,
 lic), bad_licenses) == False or
-- 
2.29.2


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

[OE-core] [PATCH v2 1/2] license_image.bbclass: Detect broken symlinks

[Reto Schneider]

From: Reto Schneider 

Find and report symlinks which point to a non-existing file.

Signed-off-by: Reto Schneider 
---
Version 2: No changes

 meta/classes/license_image.bbclass | 15 +--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/meta/classes/license_image.bbclass 
b/meta/classes/license_image.bbclass
index c96b032ebd..b9a0f2359b 100644
--- a/meta/classes/license_image.bbclass
+++ b/meta/classes/license_image.bbclass
@@ -1,3 +1,5 @@
+ROOTFS_LICENSE_DIR = "${IMAGE_ROOTFS}/usr/share/common-licenses"
+
 python write_package_manifest() {
 # Get list of installed packages
 license_image_dir = d.expand('${LICENSE_DIRECTORY}/${IMAGE_NAME}')
@@ -104,8 +106,7 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
 copy_lic_manifest = d.getVar('COPY_LIC_MANIFEST')
 copy_lic_dirs = d.getVar('COPY_LIC_DIRS')
 if rootfs and copy_lic_manifest == "1":
-rootfs_license_dir = os.path.join(d.getVar('IMAGE_ROOTFS'), 
-'usr', 'share', 'common-licenses')
+rootfs_license_dir = d.getVar('ROOTFS_LICENSE_DIR')
 bb.utils.mkdirhier(rootfs_license_dir)
 rootfs_license_manifest = os.path.join(rootfs_license_dir,
 os.path.split(license_manifest)[1])
@@ -267,3 +268,13 @@ python do_populate_lic_deploy() {
 addtask populate_lic_deploy before do_build after do_image_complete
 do_populate_lic_deploy[recrdeptask] += "do_populate_lic do_deploy"
 
+python license_qa_dead_symlink() {
+import os
+
+for root, dirs, files in os.walk(d.getVar('ROOTFS_LICENSE_DIR')):
+for file in files:
+full_path = root + "/" + file
+if os.path.islink(full_path) and not os.path.exists(full_path):
+bb.error("broken symlink: " + full_path)
+}
+IMAGE_QA_COMMANDS += "license_qa_dead_symlink"
-- 
2.29.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151022): 
https://lists.openembedded.org/g/openembedded-core/message/151022
Mute This Topic: https://lists.openembedded.org/mt/82389805/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] LTP drop MUSL specific patch

[Petr Vorel]

Hi Andre,

> On Mon, Apr 26, 2021 at 11:53 AM Petr Vorel  wrote:

> > Hi Khem, Richard,

> > I'd like to replace MUSL specific patch
> > meta/recipes-extended/ltp/ltp/0001-build-Add-option-to-select-libc-implementation.patch
> > with just removing broken files. We use this in upstream CI for Alpine, I 
> > ported
> > it to Buildroot [2], where works well. It's better because it does allow to
> > handle MUSL without rebasing this patch.

> > I was looking into the docs, it looks to me that do_configure [3] might be 
> > good hook
> > to add it into. But it looks like by default it contains oe_runmake clean 
> > [4].
> > Or should I use different hook? And how to find the default content of 
> > chosen
> > hook? (if I need to extend it, I'll have to provide the original as well)

> Changes to source files, including removing them, should be done as
> part of the do_patch task. Since do_patch is implemented in python you
> can't simply _append shell script commands to it, but you can add a
> shell function via do_patch[postfuncs]. If you grep for
> "do_patch[postfuncs]" in oe-core you should find a few examples.

Great, thanks a lot!

Kind regards,
Petr

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

[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 356
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.2", "3.3, "3.99" and "Future", the more pressing/urgent issues
being in "3.2" and then "3.3".

 

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 (#151024): 
https://lists.openembedded.org/g/openembedded-core/message/151024
Mute This Topic: https://lists.openembedded.org/mt/82392803/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] oe-pkgdata-util: ignore SIGPIPE

[Chen Qi]

ping

On 04/20/2021 04:37 PM, Chen Qi wrote:

oe-pkgdata-util sometimes outputs a large amount of data. When used
with pipe, it's likely to get the following error.

   BrokenPipeError: [Errno 32] Broken pipe

The problem could be reproduced by running `oe-pkgdata-util list-pkg | less'.
Type 'q' after running the above command, and we get the error.

Signed-off-by: Chen Qi 
---
  scripts/oe-pkgdata-util | 5 +
  1 file changed, 5 insertions(+)

diff --git a/scripts/oe-pkgdata-util b/scripts/oe-pkgdata-util
index 75dd23efa3..4aeb28879d 100755
--- a/scripts/oe-pkgdata-util
+++ b/scripts/oe-pkgdata-util
@@ -17,6 +17,7 @@ import re
  import argparse
  import logging
  from collections import defaultdict, OrderedDict
+from signal import signal, SIGPIPE, SIG_DFL
  
  scripts_path = os.path.dirname(os.path.realpath(__file__))

  lib_path = scripts_path + '/lib'
@@ -615,6 +616,10 @@ def main():
  logger.error('Unable to find pkgdata directory %s' % args.pkgdata_dir)
  sys.exit(1)
  
+# It's possible that this program will output large contents, and when used with a pipe in command line,

+# we will get a 'BrokenPipeError: [Errno 32] Broken pipe'. Ignore the 
SIGPIPE to avoid such error.
+signal(SIGPIPE, SIG_DFL)
+
  ret = args.func(args)
  
  return ret








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