Re: [OE-core] [RESEND PATCH v2] make: 4.2.1 -> 4.3

[Khem Raj]

On 2/13/20 5:36 AM, Jens Rehsack wrote:

Announcement: https://lists.gnu.org/archive/html/bug-make/2020-01/msg00057.html

1) Remove upstream provided patches 
0001-glob-Do-not-assume-glibc-glob-internals.patch
and 0002-glob-Do-not-assume-glibc-glob-internals.patch.

2) License has been changed to GPLv3 only

3) Important bug-fix is
* https://lists.gnu.org/archive/html/bug-make/2018-09/msg6.html

4) Backward-incompatibilities:
* Number signs (#) appearing inside a macro reference or function invocation
  no longer introduce comments and should not be escaped with backslashes
* Previously appending using '+=' to an empty variable would result in a 
value
  starting with a space.  Now the initial space is only added if the 
variable
  already contains some value.  Similarly, appending an empty string does 
not
  add a trailing space.



Just assessing forward port work this will ensue, did you try some world 
builds ?




Signed-off-by: Jens Rehsack 
---
  meta/recipes-devtools/make/make.inc   |  4 +-
  ...b-Do-not-assume-glibc-glob-internals.patch | 70 ---
  ...b-Do-not-assume-glibc-glob-internals.patch | 38 --
  meta/recipes-devtools/make/make_4.2.1.bb  | 12 
  meta/recipes-devtools/make/make_4.3.bb| 10 +++
  5 files changed, 11 insertions(+), 123 deletions(-)
  delete mode 100644 
meta/recipes-devtools/make/make/0001-glob-Do-not-assume-glibc-glob-internals.patch
  delete mode 100644 
meta/recipes-devtools/make/make/0002-glob-Do-not-assume-glibc-glob-internals.patch
  delete mode 100644 meta/recipes-devtools/make/make_4.2.1.bb
  create mode 100644 meta/recipes-devtools/make/make_4.3.bb

diff --git a/meta/recipes-devtools/make/make.inc 
b/meta/recipes-devtools/make/make.inc
index b8905bc6d3..4142cf23d3 100644
--- a/meta/recipes-devtools/make/make.inc
+++ b/meta/recipes-devtools/make/make.inc
@@ -5,9 +5,7 @@ called the makefile, which lists each of the non-source files 
and how to compute
  HOMEPAGE = "http://www.gnu.org/software/make/;
  SECTION = "devel"
  
-SRC_URI = "${GNU_MIRROR}/make/make-${PV}.tar.bz2 \

-   file://0001-glob-Do-not-assume-glibc-glob-internals.patch \
-   file://0002-glob-Do-not-assume-glibc-glob-internals.patch \
+SRC_URI = "${GNU_MIRROR}/make/make-${PV}.tar.lz \
 "
  
  inherit autotools gettext pkgconfig texinfo

diff --git 
a/meta/recipes-devtools/make/make/0001-glob-Do-not-assume-glibc-glob-internals.patch
 
b/meta/recipes-devtools/make/make/0001-glob-Do-not-assume-glibc-glob-internals.patch
deleted file mode 100644
index 2b6e4d40c3..00
--- 
a/meta/recipes-devtools/make/make/0001-glob-Do-not-assume-glibc-glob-internals.patch
+++ /dev/null
@@ -1,70 +0,0 @@
-From c90a7dda6c572f79b8e78da44b6ebf8704edef65 Mon Sep 17 00:00:00 2001
-From: Paul Eggert 
-Date: Sun, 24 Sep 2017 09:12:58 -0400
-Subject: [PATCH 1/2] glob: Do not assume glibc glob internals.
-
-It has been proposed that glibc glob start using gl_lstat,
-which the API allows it to do.  GNU 'make' should not get in
-the way of this.  See:
-https://sourceware.org/ml/libc-alpha/2017-09/msg00409.html
-
-* dir.c (local_lstat): New function, like local_stat.
-(dir_setup_glob): Use it to initialize gl_lstat too, as the API
-requires.

-Upstream-Status: Backport
-Signed-off-by: Khem Raj 
-
- dir.c | 29 +++--
- 1 file changed, 27 insertions(+), 2 deletions(-)
-
-diff --git a/dir.c b/dir.c
-index f34bbf5..12eef30 100644
 a/dir.c
-+++ b/dir.c
-@@ -1299,15 +1299,40 @@ local_stat (const char *path, struct stat *buf)
- }
- #endif
-
-+/* Similarly for lstat.  */
-+#if !defined(lstat) && !defined(WINDOWS32) || defined(VMS)
-+# ifndef VMS
-+#  ifndef HAVE_SYS_STAT_H
-+int lstat (const char *path, struct stat *sbuf);
-+#  endif
-+# else
-+/* We are done with the fake lstat.  Go back to the real lstat */
-+#   ifdef lstat
-+# undef lstat
-+#   endif
-+# endif
-+# define local_lstat lstat
-+#elif defined(WINDOWS32)
-+/* Windows doesn't support lstat().  */
-+# define local_lstat local_stat
-+#else
-+static int
-+local_lstat (const char *path, struct stat *buf)
-+{
-+  int e;
-+  EINTRLOOP (e, lstat (path, buf));
-+  return e;
-+}
-+#endif
-+
- void
- dir_setup_glob (glob_t *gl)
- {
-   gl->gl_opendir = open_dirstream;
-   gl->gl_readdir = read_dirstream;
-   gl->gl_closedir = free;
-+  gl->gl_lstat = local_lstat;
-   gl->gl_stat = local_stat;
--  /* We don't bother setting gl_lstat, since glob never calls it.
-- The slot is only there for compatibility with 4.4 BSD.  */
- }
-
- void
---
-2.16.1
-
diff --git 
a/meta/recipes-devtools/make/make/0002-glob-Do-not-assume-glibc-glob-internals.patch
 
b/meta/recipes-devtools/make/make/0002-glob-Do-not-assume-glibc-glob-internals.patch
deleted file mode 100644
index d49acd9f6e..00
--- 
a/meta/recipes-devtools/make/make/0002-glob-Do-not-assume-glibc-glob-internals.patch
+++ /dev/null
@@ -1,38 +0,0 @@
-From 

Re: [OE-core] [OE-Core][PATCH v2] systemd: upgrade v244.1 -> v244.3

[Alex Kiernan]

On Fri, Feb 14, 2020 at 7:17 AM Khem Raj  wrote:
>
> On Thu, Feb 13, 2020 at 3:40 PM Richard Purdie
>  wrote:
> >
> > On Thu, 2020-02-13 at 06:16 +, Alex Kiernan wrote:
> > > Commits from v244-stable:
> >
> > I think this causes:
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/48/builds/1557
> >
> > (systemd-boot patch failures)
>
> right
> http://errors.yoctoproject.org/Errors/Details/390912/
>

Yeah... just trying to work out if there's a better way to handle the
efi-cc stuff w/o having an even larger set of reverts against
meson.build :|

-- 
Alex Kiernan
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [OE-Core][PATCH v2] systemd: upgrade v244.1 -> v244.3

[Khem Raj]

On Thu, Feb 13, 2020 at 3:40 PM Richard Purdie
 wrote:
>
> On Thu, 2020-02-13 at 06:16 +, Alex Kiernan wrote:
> > Commits from v244-stable:
>
> I think this causes:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/48/builds/1557
>
> (systemd-boot patch failures)

right
http://errors.yoctoproject.org/Errors/Details/390912/

>
> Cheers,
>
> Richard
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Khem Raj]

On Thu, Feb 13, 2020 at 11:05 PM Alexander Kanavin
 wrote:
>
> On Fri, 14 Feb 2020 at 02:40, Khem Raj  wrote:
>>
>>
>> autobuilder runs poky, to reproduce this you will need to use
>> oe-core+bitbake checkout
>>
>> and then for qemux86-64
>>
>> bitbake -ctestimage core-image-sato-sdk-ptest
>
>
> If I set up a nodistro build, ptest packages are no longer available and 
> build doesn't start:
>
> akanavin@ubuntu1804-ty:~/poky/build-nodistro$ bitbake 
> core-image-sato-sdk-ptest
> ...
> ERROR: Nothing RPROVIDES 'acl-ptest' (but 
> /home/akanavin/poky/meta/recipes-sato/images/core-image-sato-sdk-ptest.bb 
> RDEPENDS on or otherwise requires it)
>
local.conf I have

DISTRO_FEATURES_append = " largefile opengl ptest multiarch wayland pam polkit "
IMAGE_CLASSES += "testimage testsdk"

> Can you share the config (local.conf) I need to use? Or maybe you can show 
> the failure you're getting. The tests should be skipped if /var/lib/rpm is 
> absent on the target image, so I would like to find out why it is there 
> (presumably).
>
>>
>> it might also work in poky setup if you remove
>> PACKAGE_CLASSES ?= "package_rpm"
>> from local.conf
>
>
> That's what the autobuilder test does, and then it passes, so I don't think 
> this will help.
>
> Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Alexander Kanavin]

On Fri, 14 Feb 2020 at 02:40, Khem Raj  wrote:

>
> autobuilder runs poky, to reproduce this you will need to use
> oe-core+bitbake checkout
>
> and then for qemux86-64
>
> bitbake -ctestimage core-image-sato-sdk-ptest
>

If I set up a nodistro build, ptest packages are no longer available and
build doesn't start:

akanavin@ubuntu1804-ty:~/poky/build-nodistro$ bitbake
core-image-sato-sdk-ptest
...
ERROR: Nothing RPROVIDES 'acl-ptest' (but
/home/akanavin/poky/meta/recipes-sato/images/core-image-sato-sdk-ptest.bb
RDEPENDS on or otherwise requires it)

Can you share the config (local.conf) I need to use? Or maybe you can show
the failure you're getting. The tests should be skipped if /var/lib/rpm is
absent on the target image, so I would like to find out why it is there
(presumably).


> it might also work in poky setup if you remove
> PACKAGE_CLASSES ?= "package_rpm"
> from local.conf
>

That's what the autobuilder test does, and then it passes, so I don't think
this will help.

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] swuashfs-tools: Enable on musl

[Khem Raj]

Upstream has a patch [1] to fix musl builds, Prior to this it would not
build on musl due to missing GNU extentions for fnmatch

[1] 
https://github.com/plougher/squashfs-tools/commit/4280e74de14070869787a9166242f9ce2dafd82e

Signed-off-by: Khem Raj 
---
 meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb 
b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
index 4fd33f8048..9bcf026ce2 100644
--- a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
+++ b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
@@ -12,9 +12,6 @@ SRC_URI = 
"git://github.com/plougher/squashfs-tools.git;protocol=https \
 
 S = "${WORKDIR}/git/squashfs-tools"
 
-# needs FNM_EXTMATCH
-COMPATIBLE_HOST_libc-musl = 'null'
-
 EXTRA_OEMAKE = "${PACKAGECONFIG_CONFARGS}"
 
 PACKAGECONFIG ??= "gzip xz lzo lz4 lzma xattr reproducible"
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 3/4] recipes-support: Add diffoscope recipe

[Khem Raj]

Hi Joshua

I think it needs to be ignored on musl

15:37:23 ERROR: Nothing RPROVIDES 'squashfs-tools' (but
/mnt/jenkins/workspace/Yocto-world-musl/sources/openembedded-core/meta/recipes-support/diffoscope/diffoscope_136.bb
RDEPENDS on or otherwise requires it)
15:37:23 squashfs-tools was skipped: incompatible with host
aarch64-yoe-linux-musl (not in COMPATIBLE_HOST)

On Tue, Feb 11, 2020 at 7:15 PM Joshua Watt  wrote:
>
> Diffoscope is the universal diff tool, capable of comparing many
> different formats.
>
> Signed-off-by: Joshua Watt 
> ---
>  meta/conf/distro/include/maintainers.inc  |  1 +
>  meta/recipes-support/diffoscope/diffoscope_136.bb | 15 +++
>  2 files changed, 16 insertions(+)
>  create mode 100644 meta/recipes-support/diffoscope/diffoscope_136.bb
>
> diff --git a/meta/conf/distro/include/maintainers.inc 
> b/meta/conf/distro/include/maintainers.inc
> index 171a43615e3..4a267faabce 100644
> --- a/meta/conf/distro/include/maintainers.inc
> +++ b/meta/conf/distro/include/maintainers.inc
> @@ -146,6 +146,7 @@ RECIPE_MAINTAINER_pn-dejagnu = "Nathan Rossi 
> "
>  RECIPE_MAINTAINER_pn-depmodwrapper-cross = "Mark Hatle 
> "
>  RECIPE_MAINTAINER_pn-desktop-file-utils = "Alexander Kanavin 
> "
>  RECIPE_MAINTAINER_pn-dhcp = "Hongxu Jia "
> +RECIPE_MAINTAINER_pn-diffoscope = "Joshua Watt "
>  RECIPE_MAINTAINER_pn-diffstat = "Chen Qi "
>  RECIPE_MAINTAINER_pn-diffutils = "Chen Qi "
>  RECIPE_MAINTAINER_pn-distcc = "Hongxu Jia "
> diff --git a/meta/recipes-support/diffoscope/diffoscope_136.bb 
> b/meta/recipes-support/diffoscope/diffoscope_136.bb
> new file mode 100644
> index 000..a00f281deb0
> --- /dev/null
> +++ b/meta/recipes-support/diffoscope/diffoscope_136.bb
> @@ -0,0 +1,15 @@
> +SUMMARY = "in-depth comparison of files, archives, and directories"
> +HOMEPAGE = "https://diffoscope.org/;
> +LICENSE = "GPL-3.0+"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
> +
> +PYPI_PACKAGE = "diffoscope"
> +
> +inherit pypi setuptools3
> +
> +SRC_URI[md5sum] = "c84d8d308a40176ba2f5dc4abdbf6f73"
> +SRC_URI[sha256sum] = 
> "0d6486d6eb6e0445ba21fee2e8bdd3a366ce786bfac98e00e5a95038b7815f15"
> +
> +RDEPENDS_${PN} += "binutils vim squashfs-tools python3-libarchive-c 
> python3-magic"
> +
> +BBCLASSEXTEND = "native"
> --
> 2.23.0
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 2/2] glibc-testsuite: Exclude this recipe from world builds

[Nathan Rossi]

This recipe is intended to be run manually or via oeqa.

[YOCTO #13737]

Signed-off-by: Nathan Rossi 
---
 meta/recipes-core/glibc/glibc-testsuite_2.31.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-core/glibc/glibc-testsuite_2.31.bb 
b/meta/recipes-core/glibc/glibc-testsuite_2.31.bb
index c44f30bf1c..d887aeff79 100644
--- a/meta/recipes-core/glibc/glibc-testsuite_2.31.bb
+++ b/meta/recipes-core/glibc/glibc-testsuite_2.31.bb
@@ -1,5 +1,7 @@
 require glibc_${PV}.bb
 
+EXCLUDE_FROM_WORLD = "1"
+
 # handle PN differences
 FILESEXTRAPATHS_prepend := "${THISDIR}/glibc:"
 
---
2.24.1
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 1/2] glibc-testsuite: Remove the do_install task

[Nathan Rossi]

This task is not intended to be run as this recipe does not produce any
packages or install any sysroot content. Additionally running this task
behaves differently from a normal glibc build, and can result in issues.

[YOCTO #13737]

Signed-off-by: Nathan Rossi 
---
 meta/recipes-core/glibc/glibc-testsuite_2.31.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/glibc/glibc-testsuite_2.31.bb 
b/meta/recipes-core/glibc/glibc-testsuite_2.31.bb
index 657fd4dbc1..c44f30bf1c 100644
--- a/meta/recipes-core/glibc/glibc-testsuite_2.31.bb
+++ b/meta/recipes-core/glibc/glibc-testsuite_2.31.bb
@@ -58,3 +58,4 @@ addtask do_check after do_compile
 
 inherit nopackages
 deltask do_stash_locale
+deltask do_install
---
2.24.1
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] bitbake.conf: do not let DISTROOVERRIDES fall back to empty string

[Masahiro Yamada]

Hi.

On Mon, Jan 13, 2020 at 9:37 PM Masahiro Yamada  wrote:
>
> Commit abacd7243a2e ("bitbake.conf: fix distro-less failure expanding
> FILESPATH") mended the littered DISROOVERRIDES when DISTRO is unset.
>
> Some time later, commit b7279f996397 ("bitbake.conf: Default DISTRO to
> nodistro") added a weak default to 'DISTRO'.
>
> So, this line can go back to the original simple definition.
>
> Signed-off-by: Masahiro Yamada 


Please let me ping this in case this was missed.
This looks like a good clean-up to me.



> ---
>
>  meta/conf/bitbake.conf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
> index 62b5466b71..d2b6ca840f 100644
> --- a/meta/conf/bitbake.conf
> +++ b/meta/conf/bitbake.conf
> @@ -736,7 +736,7 @@ DISTRO_NAME ??= "OpenEmbedded"
>  OVERRIDES = 
> "${TARGET_OS}:${TRANSLATED_TARGET_ARCH}:pn-${PN}:${MACHINEOVERRIDES}:${DISTROOVERRIDES}:${CLASSOVERRIDE}${LIBCOVERRIDE}:forcevariable"
>  LIBCOVERRIDE ?= ""
>  CLASSOVERRIDE ?= "class-target"
> -DISTROOVERRIDES ?= "${@d.getVar('DISTRO') or ''}"
> +DISTROOVERRIDES ?= "${DISTRO}"
>  MACHINEOVERRIDES ?= "${MACHINE}"
>
>  FILESOVERRIDES = 
> "${TRANSLATED_TARGET_ARCH}:${MACHINEOVERRIDES}:${DISTROOVERRIDES}"
> --
> 2.17.1
>


-- 
Best Regards
Masahiro Yamada
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH v3] perf: Correct the substitution of python shebangs

[Chen, Qi]

TA


From: openembedded-core-boun...@lists.openembedded.org 
[openembedded-core-boun...@lists.openembedded.org] on behalf of 
zhe...@windriver.com [zhe...@windriver.com]
Sent: Friday, February 14, 2020 11:57 AM
To: openembedded-core@lists.openembedded.org
Subject: [OE-core] [PATCH v3] perf: Correct the substitution of python  shebangs

From: He Zhe 

To make the native python3 is always used,

- Use sed one-liner instead
- Add substitution for ${S}/scripts/bpf_helpers_doc.py to fix the
  following warning.

File "/usr/lib/python3.6/sysconfig.py", line 421, in _init_posix
_temp = __import__(name, globals(), locals(), ['build_time_vars'], 0)
ModuleNotFoundError: No module named '_sysconfigdata'

This issue is first reported by Joel Stanley 
The sed one-liner is credited to Anuj Mittal 

Signed-off-by: He Zhe 
---
 meta/recipes-kernel/perf/perf.bb | 7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-kernel/perf/perf.bb b/meta/recipes-kernel/perf/perf.bb
index 6d1b06693d..a6fb51d3db 100644
--- a/meta/recipes-kernel/perf/perf.bb
+++ b/meta/recipes-kernel/perf/perf.bb
@@ -239,11 +239,8 @@ do_configure_prepend () {
 fi

 # use /usr/bin/env instead of version specific python
-for s in `find ${S}/tools/perf/ -name '*.py'`; do
-sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
+for s in `find ${S}/tools/perf/ -name '*.py'` 
${S}/scripts/bpf_helpers_doc.py; do
+sed -i -e "s,#!.*python.*,#!${USRBINPATH}/env python3," ${s}
 done

 # unistd.h can be out of sync between libc-headers and the captured 
version in the perf source
--
2.24.1

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH v3] perf: Correct the substitution of python shebangs

[zhe.he]

From: He Zhe 

To make the native python3 is always used,

- Use sed one-liner instead
- Add substitution for ${S}/scripts/bpf_helpers_doc.py to fix the
  following warning.

File "/usr/lib/python3.6/sysconfig.py", line 421, in _init_posix
_temp = __import__(name, globals(), locals(), ['build_time_vars'], 0)
ModuleNotFoundError: No module named '_sysconfigdata'

This issue is first reported by Joel Stanley 
The sed one-liner is credited to Anuj Mittal 

Signed-off-by: He Zhe 
---
 meta/recipes-kernel/perf/perf.bb | 7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-kernel/perf/perf.bb b/meta/recipes-kernel/perf/perf.bb
index 6d1b06693d..a6fb51d3db 100644
--- a/meta/recipes-kernel/perf/perf.bb
+++ b/meta/recipes-kernel/perf/perf.bb
@@ -239,11 +239,8 @@ do_configure_prepend () {
 fi
 
 # use /usr/bin/env instead of version specific python
-for s in `find ${S}/tools/perf/ -name '*.py'`; do
-sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
+for s in `find ${S}/tools/perf/ -name '*.py'` 
${S}/scripts/bpf_helpers_doc.py; do
+sed -i -e "s,#!.*python.*,#!${USRBINPATH}/env python3," ${s}
 done
 
 # unistd.h can be out of sync between libc-headers and the captured 
version in the perf source
-- 
2.24.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH v2] perf: Correct the substitution of python shebangs

[He Zhe]

commit log broken. Please ignore. I'll send v3.

Zhe

On 2/14/20 11:44 AM, zhe...@windriver.com wrote:
> From: He Zhe 
>
> To make the native python3 is always used,
>
> - Move the substitution of /usr/bin/python3 to first, otherwise the
>   possible original /usr/bin/python3 would be changed to
>   /usr/bin/env python33.
> - Add substitution for ${S}/scripts/bpf_helpers_doc.py to fix the
>   following warning.
>
> File "/usr/lib/python3.6/sysconfig.py", line 421, in _init_posix
> _temp = __import__(name, globals(), locals(), ['build_time_vars'], 0)
> ModuleNotFoundError: No module named '_sysconfigdata'
>
> This issue is first reported by Joel Stanley 
> The sed one-liner is credited to Anuj Mittal 
>
> Signed-off-by: He Zhe 
> ---
>  meta/recipes-kernel/perf/perf.bb | 7 ++-
>  1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/meta/recipes-kernel/perf/perf.bb 
> b/meta/recipes-kernel/perf/perf.bb
> index 6d1b06693d..a6fb51d3db 100644
> --- a/meta/recipes-kernel/perf/perf.bb
> +++ b/meta/recipes-kernel/perf/perf.bb
> @@ -239,11 +239,8 @@ do_configure_prepend () {
>  fi
>  
>  # use /usr/bin/env instead of version specific python
> -for s in `find ${S}/tools/perf/ -name '*.py'`; do
> -sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
> -sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
> -sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
> -sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
> +for s in `find ${S}/tools/perf/ -name '*.py'` 
> ${S}/scripts/bpf_helpers_doc.py; do
> +sed -i -e "s,#!.*python.*,#!${USRBINPATH}/env python3," ${s}
>  done
>  
>  # unistd.h can be out of sync between libc-headers and the captured 
> version in the perf source

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH v2] perf: Correct the substitution of python shebangs

[zhe.he]

From: He Zhe 

To make the native python3 is always used,

- Move the substitution of /usr/bin/python3 to first, otherwise the
  possible original /usr/bin/python3 would be changed to
  /usr/bin/env python33.
- Add substitution for ${S}/scripts/bpf_helpers_doc.py to fix the
  following warning.

File "/usr/lib/python3.6/sysconfig.py", line 421, in _init_posix
_temp = __import__(name, globals(), locals(), ['build_time_vars'], 0)
ModuleNotFoundError: No module named '_sysconfigdata'

This issue is first reported by Joel Stanley 
The sed one-liner is credited to Anuj Mittal 

Signed-off-by: He Zhe 
---
 meta/recipes-kernel/perf/perf.bb | 7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-kernel/perf/perf.bb b/meta/recipes-kernel/perf/perf.bb
index 6d1b06693d..a6fb51d3db 100644
--- a/meta/recipes-kernel/perf/perf.bb
+++ b/meta/recipes-kernel/perf/perf.bb
@@ -239,11 +239,8 @@ do_configure_prepend () {
 fi
 
 # use /usr/bin/env instead of version specific python
-for s in `find ${S}/tools/perf/ -name '*.py'`; do
-sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
-sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
+for s in `find ${S}/tools/perf/ -name '*.py'` 
${S}/scripts/bpf_helpers_doc.py; do
+sed -i -e "s,#!.*python.*,#!${USRBINPATH}/env python3," ${s}
 done
 
 # unistd.h can be out of sync between libc-headers and the captured 
version in the perf source
-- 
2.24.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] perf: Correct the substitution of python shebangs

[He Zhe]

Thanks for your input.

Since the old one doesn't make it to next branch and the thread hasn't had 
following replies, I'll send v2 as a summary.

Zhe

On 2/14/20 4:45 AM, Bruce Ashfield wrote:
> On Thu, Feb 13, 2020 at 1:39 PM  wrote:
>> From: He Zhe 
>>
>> To make the native python3 are always used,
>>
>> - Move the substitution of /usr/bin/python3 to first, otherwise the
>>   possible original /usr/bin/python3 would be changed to
>>   /usr/bin/env python33.
>> - Add substitution for ${S}/scripts/
>>
>> Signed-off-by: He Zhe 
>> ---
>>  meta/recipes-kernel/perf/perf.bb | 7 +++
>>  1 file changed, 7 insertions(+)
>>
>> diff --git a/meta/recipes-kernel/perf/perf.bb 
>> b/meta/recipes-kernel/perf/perf.bb
>> index 6d1b066..14a9090 100644
>> --- a/meta/recipes-kernel/perf/perf.bb
>> +++ b/meta/recipes-kernel/perf/perf.bb
>> @@ -240,10 +240,17 @@ do_configure_prepend () {
>>
>>  # use /usr/bin/env instead of version specific python
>>  for s in `find ${S}/tools/perf/ -name '*.py'`; do
>> +sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
> Rather than messing around with the order of these, we should just
> combine them into a single sed invocation. When I started those
> substitutions, there weren't so many.
>
> As was mentioned in the other perf patch  and review:
>
> sed -i -e "1s,#!.*python.*,#!${USRBINPATH}/env python3," ${s}
>
> or some variant of the regex should fix them all.
>
>>  sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
>>  sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
>>  sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
>> +done
>> +
>> +for s in `find ${S}/scripts/ -name '*.py'`; do
> It would be better to not have two loops doing the same thing, why not
> combine the two ? In the other review, the single script that was
> causing the problem was specifically pointed out .. and while I also
> agree that we can just substitute them all, technically, if we haven't
> run them, we don't know they are py3 safe.
>
> But minimally, getting them into a single for loop is something we
> should do (also part of that other patch/review).
>
> Bruce
>
>>  sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
>> +sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
>> +sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
>> +sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
>>  done
>>
>>  # unistd.h can be out of sync between libc-headers and the captured 
>> version in the perf source
>> --
>> 2.7.4
>>
>> --
>> ___
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
> --
> - Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end
> - "Use the force Harry" - Gandalf, Star Trek II

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] Unknown Postinstall Scriptlet Failing to Defer Boot

[Jeremy A. Puhlman]

In these kind of cases, by providing the actual error message, people 
can provide more help.


The recipe inherits update-alternatives and sets a large number of 
alternatives. The alternatives are done in the automatically added post 
install scripts. In all likelihood one of the listed links is wrong, or 
already a symlink or something similar.

On 2/13/2020 4:49 PM, Andrew Boos wrote:

Hi,

I am working on a bitbake build and I am running into an issue where a 
postinst scriptlet is failing via "exit 1" to defer to first boot and 
I cannot figure out which recipe is causing the issue. Does anybody 
have any tips on identifying which recipe is actually exiting with a 
failure? The error log says that it is in imagemagick but the recipe 
does not have any postinst scripts defined. Also, there are no 
imagemagick .bbappend files in my layers. 
(https://github.com/openembedded/meta-openembedded/blob/sumo/meta-oe/recipes-support/imagemagick/imagemagick_7.0.7.bb)


Thanks,
Andrew

 
	Virus-free. www.avg.com 
 



<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>



--
Jeremy A. Puhlman
jpuhl...@mvista.com

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Khem Raj]

On Thu, Feb 13, 2020 at 2:05 PM Alexander Kanavin
 wrote:
>
> On Thu, 13 Feb 2020 at 22:37, Khem Raj  wrote:
>>>
>>>
>>> Merging these patches will exclude rpm from images, and so will mask the 
>>> issue, so it's not orthogonal.
>>
>>
>> are you actively working to fix the test case ?
>> In that case it would be justified to hold them
>
>
> The test case that is failing for you (with ipk/opkg and core-image-sato-sdk) 
> is passing just fine on the autobuilder [1], so I am missing information on 
> how to reproduce. I have requested this information from you here [2] and got 
> no answer so far. Or is it some other test that is failing? That isn't 
> entirely clear.
>

autobuilder runs poky, to reproduce this you will need to use
oe-core+bitbake checkout

and then for qemux86-64

bitbake -ctestimage core-image-sato-sdk-ptest

it might also work in poky setup if you remove
PACKAGE_CLASSES ?= "package_rpm"
from local.conf

> [1] 
> https://autobuilder.yoctoproject.org/typhoon/#/builders/76/builds/1554/steps/8/logs/step1c
>
> [2] 
> http://lists.openembedded.org/pipermail/openembedded-core/2020-February/292774.html
>
> Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 6/9] gdb: Upgrade to 9.1 release

[Khem Raj]

On Thu, Feb 13, 2020 at 3:38 PM Richard Purdie
 wrote:
>
> On Wed, 2020-02-12 at 22:38 -0800, Khem Raj wrote:
> > Drop security patches which are now in 9.1 already
> > Forward port rest of patches to 9.1
> > Detailed changes [1]
> >
> > [1] https://lists.gnu.org/archive/html/info-gnu/2020-02/msg8.html
> >
> > Signed-off-by: Khem Raj 
>
> Something odd is going on as another patch failure:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/60/builds/1551
>

perhaps pick from pull branch
https://git.yoctoproject.org/clean/cgit.cgi/poky-contrib/log/?h=kraj/pu

> Cheers,
>
> Richard
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] strace: Update to 5.5

[Alistair Francis]

On Thu, Feb 13, 2020 at 4:24 PM akuster808  wrote:
>
>
>
> On 2/13/20 3:25 PM, Alistair Francis wrote:
> > Update strace to 5.5. This includes a small change in the license file
> > to change the date to 2020.
>
> Have you run the ptests for this?

I haven't.

> Will this help help address this following open defect?
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=13785

I'm not sure.

Alistair

>
> - armin
>
> > Signed-off-by: Alistair Francis 
> > ---
> >  ...0001-caps-abbrev.awk-fix-gawk-s-path.patch |  2 +-
> >  ...yscall_info-to-__ptrace_syscall_info.patch | 52 ---
> >  .../strace/strace/Makefile-ptest.patch|  6 ++-
> >  .../strace/disable-git-version-gen.patch  |  4 +-
> >  .../strace/strace/mips-SIGEMT.patch   |  2 +-
> >  ...st-test-for-m32-mx32-compile-support.patch | 43 ---
> >  .../strace/strace/ptest-spacesave.patch   | 17 --
> >  .../strace/strace/sys_headers.patch   | 25 -
> >  .../strace/strace/uintptr_t.patch | 12 -
> >  .../strace/strace/update-gawk-paths.patch |  2 +-
> >  .../strace/{strace_5.4.bb => strace_5.5.bb}   |  9 ++--
> >  11 files changed, 36 insertions(+), 138 deletions(-)
> >  delete mode 100644 
> > meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
> >  delete mode 100644 
> > meta/recipes-devtools/strace/strace/more-robust-test-for-m32-mx32-compile-support.patch
> >  delete mode 100644 meta/recipes-devtools/strace/strace/sys_headers.patch
> >  rename meta/recipes-devtools/strace/{strace_5.4.bb => strace_5.5.bb} (80%)
> >
> > diff --git 
> > a/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
> >  
> > b/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
> > index 52d2cdcdc5..235e803641 100644
> > --- 
> > a/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
> > +++ 
> > b/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
> > @@ -1,4 +1,4 @@
> > -From d225aaa8841f47ba8aa7b353b0ac3028d5913efe Mon Sep 17 00:00:00 2001
> > +From 597cc206d982e7237eb93fdc33e8c4bb6bb2d796 Mon Sep 17 00:00:00 2001
> >  From: Robert Yang 
> >  Date: Thu, 9 Feb 2017 01:27:49 -0800
> >  Subject: [PATCH] caps-abbrev.awk: fix gawk's path
> > diff --git 
> > a/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
> >  
> > b/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
> > deleted file mode 100644
> > index dd0617a15f..00
> > --- 
> > a/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
> > +++ /dev/null
> > @@ -1,52 +0,0 @@
> > -From 86bbe1135d5d13db1ced64141acfb513c03d2f30 Mon Sep 17 00:00:00 2001
> > -From: Khem Raj 
> > -Date: Sat, 21 Dec 2019 10:18:38 -0800
> > -Subject: [PATCH] define ptrace_syscall_info to __ptrace_syscall_info
> > -
> > -glibc 2.31+ has defined __ptrace_syscall_info [1]
> > -
> > -[1] 
> > https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=d1e411e5c786ce3028d98b4e6fc02c2fcf66ae37;hp=2e4e75727eb05eef4c10470d865bd42962d5fae7
> > -
> > -Upstream-Status: Pending
> > -Signed-off-by: Khem Raj 
> > 
> > - get_personality.h | 2 ++
> > - ptrace.h  | 2 ++
> > - 2 files changed, 4 insertions(+)
> > -
> >  a/get_personality.h
> > -+++ b/get_personality.h
> > -@@ -10,7 +10,9 @@
> > -
> > - # include "ptrace.h"
> > -
> > -+# if HAVE_STRUCT___PTRACE_SYSCALL_INFO
> > - extern int
> > - get_personality_from_syscall_info(const struct ptrace_syscall_info *);
> > -+# endif
> > -
> > - #endif /* !STRACE_GET_PERSONALITY_H */
> >  a/ptrace.h
> > -+++ b/ptrace.h
> > -@@ -186,6 +186,9 @@ struct ptrace_syscall_info {
> > - };
> > - # endif
> > -
> > -+# if HAVE_STRUCT___PTRACE_SYSCALL_INFO
> > -+#  define ptrace_syscall_info __ptrace_syscall_info
> > -+# endif
> > - # if !HAVE_DECL_PTRACE_PEEKUSER
> > - #  define PTRACE_PEEKUSER PTRACE_PEEKUSR
> > - # endif
> >  a/configure.ac
> > -+++ b/configure.ac
> > -@@ -317,7 +317,8 @@ AC_CHECK_MEMBERS([struct stat.st_mtime_n
> > - AC_CHECK_TYPES(m4_normalize([
> > - struct pt_all_user_regs,
> > - struct ia64_fpreg,
> > --struct ptrace_peeksiginfo_args
> > -+struct ptrace_peeksiginfo_args,
> > -+struct __ptrace_syscall_info
> > - ]),,, [#include ])
> > -
> > - # For kernels that do not have v3.10-rc1~201^2~11
> > diff --git a/meta/recipes-devtools/strace/strace/Makefile-ptest.patch 
> > b/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
> > index 1f01ef79f2..42c1893f39 100644
> > --- a/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
> > +++ b/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
> > @@ -1,4 +1,4 @@
> > -From ef5040b4f15006a22ac63a3bacfceac36ffc2045 Mon Sep 17 00:00:00 2001
> > +From ee47e205255928679701a5b769bd7ae0f815119d Mon Sep 17 00:00:00 2001
> >  From: Gabriel Barbu 
> 

[OE-core] Unknown Postinstall Scriptlet Failing to Defer Boot

[Andrew Boos]

Hi,

I am working on a bitbake build and I am running into an issue where a
postinst scriptlet is failing via "exit 1" to defer to first boot and I
cannot figure out which recipe is causing the issue. Does anybody have any
tips on identifying which recipe is actually exiting with a failure? The
error log says that it is in imagemagick but the recipe does not have any
postinst scripts defined. Also, there are no imagemagick .bbappend files in
my layers. (
https://github.com/openembedded/meta-openembedded/blob/sumo/meta-oe/recipes-support/imagemagick/imagemagick_7.0.7.bb
)

Thanks,
Andrew
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] strace: Update to 5.5

[akuster808]

On 2/13/20 3:25 PM, Alistair Francis wrote:
> Update strace to 5.5. This includes a small change in the license file
> to change the date to 2020.

Have you run the ptests for this?
Will this help help address this following open defect?
https://bugzilla.yoctoproject.org/show_bug.cgi?id=13785

- armin

> Signed-off-by: Alistair Francis 
> ---
>  ...0001-caps-abbrev.awk-fix-gawk-s-path.patch |  2 +-
>  ...yscall_info-to-__ptrace_syscall_info.patch | 52 ---
>  .../strace/strace/Makefile-ptest.patch|  6 ++-
>  .../strace/disable-git-version-gen.patch  |  4 +-
>  .../strace/strace/mips-SIGEMT.patch   |  2 +-
>  ...st-test-for-m32-mx32-compile-support.patch | 43 ---
>  .../strace/strace/ptest-spacesave.patch   | 17 --
>  .../strace/strace/sys_headers.patch   | 25 -
>  .../strace/strace/uintptr_t.patch | 12 -
>  .../strace/strace/update-gawk-paths.patch |  2 +-
>  .../strace/{strace_5.4.bb => strace_5.5.bb}   |  9 ++--
>  11 files changed, 36 insertions(+), 138 deletions(-)
>  delete mode 100644 
> meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
>  delete mode 100644 
> meta/recipes-devtools/strace/strace/more-robust-test-for-m32-mx32-compile-support.patch
>  delete mode 100644 meta/recipes-devtools/strace/strace/sys_headers.patch
>  rename meta/recipes-devtools/strace/{strace_5.4.bb => strace_5.5.bb} (80%)
>
> diff --git 
> a/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
>  
> b/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
> index 52d2cdcdc5..235e803641 100644
> --- 
> a/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
> +++ 
> b/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
> @@ -1,4 +1,4 @@
> -From d225aaa8841f47ba8aa7b353b0ac3028d5913efe Mon Sep 17 00:00:00 2001
> +From 597cc206d982e7237eb93fdc33e8c4bb6bb2d796 Mon Sep 17 00:00:00 2001
>  From: Robert Yang 
>  Date: Thu, 9 Feb 2017 01:27:49 -0800
>  Subject: [PATCH] caps-abbrev.awk: fix gawk's path
> diff --git 
> a/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
>  
> b/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
> deleted file mode 100644
> index dd0617a15f..00
> --- 
> a/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
> +++ /dev/null
> @@ -1,52 +0,0 @@
> -From 86bbe1135d5d13db1ced64141acfb513c03d2f30 Mon Sep 17 00:00:00 2001
> -From: Khem Raj 
> -Date: Sat, 21 Dec 2019 10:18:38 -0800
> -Subject: [PATCH] define ptrace_syscall_info to __ptrace_syscall_info
> -
> -glibc 2.31+ has defined __ptrace_syscall_info [1]
> -
> -[1] 
> https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=d1e411e5c786ce3028d98b4e6fc02c2fcf66ae37;hp=2e4e75727eb05eef4c10470d865bd42962d5fae7
> -
> -Upstream-Status: Pending
> -Signed-off-by: Khem Raj 
> 
> - get_personality.h | 2 ++
> - ptrace.h  | 2 ++
> - 2 files changed, 4 insertions(+)
> -
>  a/get_personality.h
> -+++ b/get_personality.h
> -@@ -10,7 +10,9 @@
> - 
> - # include "ptrace.h"
> - 
> -+# if HAVE_STRUCT___PTRACE_SYSCALL_INFO
> - extern int
> - get_personality_from_syscall_info(const struct ptrace_syscall_info *);
> -+# endif
> - 
> - #endif /* !STRACE_GET_PERSONALITY_H */
>  a/ptrace.h
> -+++ b/ptrace.h
> -@@ -186,6 +186,9 @@ struct ptrace_syscall_info {
> - };
> - # endif
> - 
> -+# if HAVE_STRUCT___PTRACE_SYSCALL_INFO
> -+#  define ptrace_syscall_info __ptrace_syscall_info
> -+# endif
> - # if !HAVE_DECL_PTRACE_PEEKUSER
> - #  define PTRACE_PEEKUSER PTRACE_PEEKUSR
> - # endif
>  a/configure.ac
> -+++ b/configure.ac
> -@@ -317,7 +317,8 @@ AC_CHECK_MEMBERS([struct stat.st_mtime_n
> - AC_CHECK_TYPES(m4_normalize([
> - struct pt_all_user_regs,
> - struct ia64_fpreg,
> --struct ptrace_peeksiginfo_args
> -+struct ptrace_peeksiginfo_args,
> -+struct __ptrace_syscall_info
> - ]),,, [#include ])
> - 
> - # For kernels that do not have v3.10-rc1~201^2~11
> diff --git a/meta/recipes-devtools/strace/strace/Makefile-ptest.patch 
> b/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
> index 1f01ef79f2..42c1893f39 100644
> --- a/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
> +++ b/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
> @@ -1,4 +1,4 @@
> -From ef5040b4f15006a22ac63a3bacfceac36ffc2045 Mon Sep 17 00:00:00 2001
> +From ee47e205255928679701a5b769bd7ae0f815119d Mon Sep 17 00:00:00 2001
>  From: Gabriel Barbu 
>  Date: Thu, 25 Jul 2013 15:28:33 +0200
>  Subject: [PATCH] strace: Add ptest
> @@ -13,6 +13,8 @@ Signed-off-by: Anuj Mittal 
>   tests/Makefile.am | 19 +++
>   1 file changed, 19 insertions(+)
>  
> +diff --git a/tests/Makefile.am b/tests/Makefile.am
> +index 8cf4c36..66162eb 100644
>  --- 

Re: [OE-core] [OE-Core][PATCH v2] systemd: upgrade v244.1 -> v244.3

[Richard Purdie]

On Thu, 2020-02-13 at 06:16 +, Alex Kiernan wrote:
> Commits from v244-stable:

I think this causes:

https://autobuilder.yoctoproject.org/typhoon/#/builders/48/builds/1557

(systemd-boot patch failures)

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 6/9] gdb: Upgrade to 9.1 release

[Richard Purdie]

On Wed, 2020-02-12 at 22:38 -0800, Khem Raj wrote:
> Drop security patches which are now in 9.1 already
> Forward port rest of patches to 9.1
> Detailed changes [1]
> 
> [1] https://lists.gnu.org/archive/html/info-gnu/2020-02/msg8.html
> 
> Signed-off-by: Khem Raj 

Something odd is going on as another patch failure:

https://autobuilder.yoctoproject.org/typhoon/#/builders/60/builds/1551

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] strace: Update to 5.5

[Alistair Francis]

Update strace to 5.5. This includes a small change in the license file
to change the date to 2020.

Signed-off-by: Alistair Francis 
---
 ...0001-caps-abbrev.awk-fix-gawk-s-path.patch |  2 +-
 ...yscall_info-to-__ptrace_syscall_info.patch | 52 ---
 .../strace/strace/Makefile-ptest.patch|  6 ++-
 .../strace/disable-git-version-gen.patch  |  4 +-
 .../strace/strace/mips-SIGEMT.patch   |  2 +-
 ...st-test-for-m32-mx32-compile-support.patch | 43 ---
 .../strace/strace/ptest-spacesave.patch   | 17 --
 .../strace/strace/sys_headers.patch   | 25 -
 .../strace/strace/uintptr_t.patch | 12 -
 .../strace/strace/update-gawk-paths.patch |  2 +-
 .../strace/{strace_5.4.bb => strace_5.5.bb}   |  9 ++--
 11 files changed, 36 insertions(+), 138 deletions(-)
 delete mode 100644 
meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
 delete mode 100644 
meta/recipes-devtools/strace/strace/more-robust-test-for-m32-mx32-compile-support.patch
 delete mode 100644 meta/recipes-devtools/strace/strace/sys_headers.patch
 rename meta/recipes-devtools/strace/{strace_5.4.bb => strace_5.5.bb} (80%)

diff --git 
a/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
 
b/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
index 52d2cdcdc5..235e803641 100644
--- 
a/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
+++ 
b/meta/recipes-devtools/strace/strace/0001-caps-abbrev.awk-fix-gawk-s-path.patch
@@ -1,4 +1,4 @@
-From d225aaa8841f47ba8aa7b353b0ac3028d5913efe Mon Sep 17 00:00:00 2001
+From 597cc206d982e7237eb93fdc33e8c4bb6bb2d796 Mon Sep 17 00:00:00 2001
 From: Robert Yang 
 Date: Thu, 9 Feb 2017 01:27:49 -0800
 Subject: [PATCH] caps-abbrev.awk: fix gawk's path
diff --git 
a/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
 
b/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
deleted file mode 100644
index dd0617a15f..00
--- 
a/meta/recipes-devtools/strace/strace/0001-define-ptrace_syscall_info-to-__ptrace_syscall_info.patch
+++ /dev/null
@@ -1,52 +0,0 @@
-From 86bbe1135d5d13db1ced64141acfb513c03d2f30 Mon Sep 17 00:00:00 2001
-From: Khem Raj 
-Date: Sat, 21 Dec 2019 10:18:38 -0800
-Subject: [PATCH] define ptrace_syscall_info to __ptrace_syscall_info
-
-glibc 2.31+ has defined __ptrace_syscall_info [1]
-
-[1] 
https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=d1e411e5c786ce3028d98b4e6fc02c2fcf66ae37;hp=2e4e75727eb05eef4c10470d865bd42962d5fae7
-
-Upstream-Status: Pending
-Signed-off-by: Khem Raj 

- get_personality.h | 2 ++
- ptrace.h  | 2 ++
- 2 files changed, 4 insertions(+)
-
 a/get_personality.h
-+++ b/get_personality.h
-@@ -10,7 +10,9 @@
- 
- # include "ptrace.h"
- 
-+# if HAVE_STRUCT___PTRACE_SYSCALL_INFO
- extern int
- get_personality_from_syscall_info(const struct ptrace_syscall_info *);
-+# endif
- 
- #endif /* !STRACE_GET_PERSONALITY_H */
 a/ptrace.h
-+++ b/ptrace.h
-@@ -186,6 +186,9 @@ struct ptrace_syscall_info {
- };
- # endif
- 
-+# if HAVE_STRUCT___PTRACE_SYSCALL_INFO
-+#  define ptrace_syscall_info __ptrace_syscall_info
-+# endif
- # if !HAVE_DECL_PTRACE_PEEKUSER
- #  define PTRACE_PEEKUSER PTRACE_PEEKUSR
- # endif
 a/configure.ac
-+++ b/configure.ac
-@@ -317,7 +317,8 @@ AC_CHECK_MEMBERS([struct stat.st_mtime_n
- AC_CHECK_TYPES(m4_normalize([
-   struct pt_all_user_regs,
-   struct ia64_fpreg,
--  struct ptrace_peeksiginfo_args
-+  struct ptrace_peeksiginfo_args,
-+  struct __ptrace_syscall_info
- ]),,, [#include ])
- 
- # For kernels that do not have v3.10-rc1~201^2~11
diff --git a/meta/recipes-devtools/strace/strace/Makefile-ptest.patch 
b/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
index 1f01ef79f2..42c1893f39 100644
--- a/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
+++ b/meta/recipes-devtools/strace/strace/Makefile-ptest.patch
@@ -1,4 +1,4 @@
-From ef5040b4f15006a22ac63a3bacfceac36ffc2045 Mon Sep 17 00:00:00 2001
+From ee47e205255928679701a5b769bd7ae0f815119d Mon Sep 17 00:00:00 2001
 From: Gabriel Barbu 
 Date: Thu, 25 Jul 2013 15:28:33 +0200
 Subject: [PATCH] strace: Add ptest
@@ -13,6 +13,8 @@ Signed-off-by: Anuj Mittal 
  tests/Makefile.am | 19 +++
  1 file changed, 19 insertions(+)
 
+diff --git a/tests/Makefile.am b/tests/Makefile.am
+index 8cf4c36..66162eb 100644
 --- a/tests/Makefile.am
 +++ b/tests/Makefile.am
 @@ -15,6 +15,7 @@ SIZEOF_LONG = @SIZEOF_LONG@
@@ -23,7 +25,7 @@ Signed-off-by: Anuj Mittal 
  AM_CFLAGS = $(WARN_CFLAGS)
  AM_CPPFLAGS = $(ARCH_MFLAGS) \
  -I$(builddir) \
-@@ -531,3 +532,21 @@ BUILT_SOURCES = ksysent.h
+@@ -538,3 +539,21 @@ BUILT_SOURCES = ksysent.h
  CLEANFILES = ksysent.h
  
  include ../scno.am
diff --git 

Re: [OE-core] [PATCH] wpa-supplicant: add virtual supplicant

[Andreas Oberritter]

On Thu, 13 Feb 2020 11:13:49 -0500
Maxime Roussin-Bélanger  wrote:

> On Thu, Feb 13, 2020 at 11:00 AM Alexander Kanavin 
> wrote:
> >
> > On Thu, 13 Feb 2020 at 16:55, Maxime Roussin-Bélanger <
> maxime.roussinbelan...@gmail.com> wrote:
> >>
> >> PACKAGECONFIG[iwd] = "--enable-iwd, --disable-wifi, iwd, iwd"
> >> PACKAGECONFIG[wpa-supplicant] = "--enable-wifi, --disable-wifi,
> wpa-supplicant, wpa-supplicant"
> >
> >
> > The problem with this, is that enabling one but not the other will
> combine into '--enable-iwd --disable-wifi' which is not what is intended.
> 
> Actually, it is what is intended, the `--disable-wifi` will disable
> wpa-supplicant support, but keep the wifi support from other plugins (i.e.
> iwd).
> 
> I agree that maybe the `--enable-wifi` is probably not the best option
> name, but I am not sure I can something about it :)

It should be the user's responsibility (or the image recipe's) to choose the
wifi daemon, and it should be possible to uninstall one and install the other
from a package feed, or even to install both at the same time and use one for
wlan0 and the other for wlan1.

Debian enables support for both daemons in connman and has the following
package metadata:

Recommends: wpasupplicant | iwd

I don't know whether there's a way to generate that line in OE. If not, you
could try to fake it by adding RPROVIDES_${PN} = "iwd-or-wpasupplicant" or
something similar to both daemon's recipes and recommending that in recipes
supporting both daemons.

Best regards,
Andreas
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Alexander Kanavin]

On Thu, 13 Feb 2020 at 22:37, Khem Raj  wrote:

>
>> Merging these patches will exclude rpm from images, and so will mask the
>> issue, so it's not orthogonal.
>>
>
> are you actively working to fix the test case ?
> In that case it would be justified to hold them
>

The test case that is failing for you (with ipk/opkg and
core-image-sato-sdk) is passing just fine on the autobuilder [1], so I am
missing information on how to reproduce. I have requested this information
from you here [2] and got no answer so far. Or is it some other test that
is failing? That isn't entirely clear.

[1]
https://autobuilder.yoctoproject.org/typhoon/#/builders/76/builds/1554/steps/8/logs/step1c

[2]
http://lists.openembedded.org/pipermail/openembedded-core/2020-February/292774.html

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Khem Raj]

On Thu, Feb 13, 2020 at 8:52 AM Alexander Kanavin 
wrote:

> On Thu, 13 Feb 2020 at 17:44, Khem Raj  wrote:
>
>> I'd like to request that these two are held back until we figure out what
> is going wrong with the rpm test (see separate thread).
>

 We already know what’s going on what more
 Do you want to know ?

>>>
>>>  As the rpm maintainer, yes I do. The test is not being skipped when it
>>> should be, and I'd like to know why.
>>>
>>
>> I see that as separate issue orthogonal to these patches
>>
>
> Merging these patches will exclude rpm from images, and so will mask the
> issue, so it's not orthogonal.
>

are you actively working to fix the test case ?
In that case it would be justified to hold them


> Alex
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2] dnf, libdnf: Ignore if PACKAGE_CLASSES does not have rpm

[Khem Raj]

On Thu, Feb 13, 2020 at 8:57 AM Alexander Kanavin 
wrote:

> On Thu, 13 Feb 2020 at 17:42, Khem Raj  wrote:
>
>> This still sneaks in pieces of rpm into distros where rpm is not needed
>> so better would be to retool dnf to not demand rpm support in libsolv and
>> break the dependency chain. Opkg is usually sought for small systems and
>> last thing we want is unneeded dependencies
>>
>> I would rather use the OE configuration powers as such but I am not
>> opposed to such a solution
>>
>
> You can always set these things (rpm disabling in libsolv, blacklisting
> libdnf/dnf) in your distro configuration, but I would like less complexity
> and special-casing in poky, not more, for the pieces that I maintain at
> least.
>

Btw it’s being proposed for Oe-core not poky and in Oe core rpm is not
default backend but opkg is
Moreover it is being done in a way where poky default are being taken into
account

I really fail to understand your line of arguments
Here I am sorry, there seems to be no technical arguments for alternatives
but for convenience

I certainly can this goes unsaid and it just takes distro one step away
from Oe-core


> Alex
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] perf: Correct the substitution of python shebangs

[Bruce Ashfield]

On Thu, Feb 13, 2020 at 1:39 PM  wrote:
>
> From: He Zhe 
>
> To make the native python3 are always used,
>
> - Move the substitution of /usr/bin/python3 to first, otherwise the
>   possible original /usr/bin/python3 would be changed to
>   /usr/bin/env python33.
> - Add substitution for ${S}/scripts/
>
> Signed-off-by: He Zhe 
> ---
>  meta/recipes-kernel/perf/perf.bb | 7 +++
>  1 file changed, 7 insertions(+)
>
> diff --git a/meta/recipes-kernel/perf/perf.bb 
> b/meta/recipes-kernel/perf/perf.bb
> index 6d1b066..14a9090 100644
> --- a/meta/recipes-kernel/perf/perf.bb
> +++ b/meta/recipes-kernel/perf/perf.bb
> @@ -240,10 +240,17 @@ do_configure_prepend () {
>
>  # use /usr/bin/env instead of version specific python
>  for s in `find ${S}/tools/perf/ -name '*.py'`; do
> +sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"

Rather than messing around with the order of these, we should just
combine them into a single sed invocation. When I started those
substitutions, there weren't so many.

As was mentioned in the other perf patch  and review:

sed -i -e "1s,#!.*python.*,#!${USRBINPATH}/env python3," ${s}

or some variant of the regex should fix them all.

>  sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
>  sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
>  sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
> +done
> +
> +for s in `find ${S}/scripts/ -name '*.py'`; do

It would be better to not have two loops doing the same thing, why not
combine the two ? In the other review, the single script that was
causing the problem was specifically pointed out .. and while I also
agree that we can just substitute them all, technically, if we haven't
run them, we don't know they are py3 safe.

But minimally, getting them into a single for loop is something we
should do (also part of that other patch/review).

Bruce

>  sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
> +sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
> +sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
> +sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
>  done
>
>  # unistd.h can be out of sync between libc-headers and the captured 
> version in the perf source
> --
> 2.7.4
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
--
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] perf: Correct the substitution of python shebangs

[Scott Branden via Openembedded-core]

On 2020-02-13 11:45 a.m., Bruce Ashfield wrote:

On Thu, Feb 13, 2020 at 1:39 PM  wrote:

From: He Zhe 

To make the native python3 are always used,


FWIW, a very similar patch was sent 10 days ago. Did you confirm that it isn't
in master-next already ?

It's not in master-next as far as I can tell.


see: "[OE-core] [PATCH] perf: Additional fixes for 5.5+"

Bruce


- Move the substitution of /usr/bin/python3 to first, otherwise the
   possible original /usr/bin/python3 would be changed to
   /usr/bin/env python33.
- Add substitution for ${S}/scripts/

Signed-off-by: He Zhe 
---
  meta/recipes-kernel/perf/perf.bb | 7 +++
  1 file changed, 7 insertions(+)

diff --git a/meta/recipes-kernel/perf/perf.bb b/meta/recipes-kernel/perf/perf.bb
index 6d1b066..14a9090 100644
--- a/meta/recipes-kernel/perf/perf.bb
+++ b/meta/recipes-kernel/perf/perf.bb
@@ -240,10 +240,17 @@ do_configure_prepend () {

  # use /usr/bin/env instead of version specific python
  for s in `find ${S}/tools/perf/ -name '*.py'`; do
+sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
  sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
  sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
  sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
+done
+
+for s in `find ${S}/scripts/ -name '*.py'`; do
  sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
  done

  # unistd.h can be out of sync between libc-headers and the captured 
version in the perf source
--
2.7.4

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core





--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] perf: Correct the substitution of python shebangs

[Bruce Ashfield]

On Thu, Feb 13, 2020 at 1:39 PM  wrote:
>
> From: He Zhe 
>
> To make the native python3 are always used,
>

FWIW, a very similar patch was sent 10 days ago. Did you confirm that it isn't
in master-next already ?

see: "[OE-core] [PATCH] perf: Additional fixes for 5.5+"

Bruce

> - Move the substitution of /usr/bin/python3 to first, otherwise the
>   possible original /usr/bin/python3 would be changed to
>   /usr/bin/env python33.
> - Add substitution for ${S}/scripts/
>
> Signed-off-by: He Zhe 
> ---
>  meta/recipes-kernel/perf/perf.bb | 7 +++
>  1 file changed, 7 insertions(+)
>
> diff --git a/meta/recipes-kernel/perf/perf.bb 
> b/meta/recipes-kernel/perf/perf.bb
> index 6d1b066..14a9090 100644
> --- a/meta/recipes-kernel/perf/perf.bb
> +++ b/meta/recipes-kernel/perf/perf.bb
> @@ -240,10 +240,17 @@ do_configure_prepend () {
>
>  # use /usr/bin/env instead of version specific python
>  for s in `find ${S}/tools/perf/ -name '*.py'`; do
> +sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
>  sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
>  sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
>  sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
> +done
> +
> +for s in `find ${S}/scripts/ -name '*.py'`; do
>  sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
> +sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
> +sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
> +sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
>  done
>
>  # unistd.h can be out of sync between libc-headers and the captured 
> version in the perf source
> --
> 2.7.4
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [meta-multimedia][PATCH] x265: add x265 recipe

[Alexander Kanavin]

Wrong mailing list, should go to oe-devel.

Alex

On Thu, 13 Feb 2020 at 17:39, Scott Branden via Openembedded-core <
openembedded-core@lists.openembedded.org> wrote:

> Add x265 recipe for latest tag of Release_3.2 branch (3.2.1).
>
> Signed-off-by: Scott Branden 
> ---
>  .../recipes-multimedia/x265/x265_3.2.1.bb | 24 +++
>  1 file changed, 24 insertions(+)
>  create mode 100644
> meta-openembedded/meta-multimedia/recipes-multimedia/x265/x265_3.2.1.bb
>
> diff --git a/meta-openembedded/meta-multimedia/recipes-multimedia/x265/
> x265_3.2.1.bb b/meta-openembedded/meta-multimedia/recipes-multimedia/x265/
> x265_3.2.1.bb
> new file mode 100644
> index 00..68c1a55c8c
> --- /dev/null
> +++ b/meta-openembedded/meta-multimedia/recipes-multimedia/x265/
> x265_3.2.1.bb
> @@ -0,0 +1,24 @@
> +SUMMARY = "H.265/HEVC video encoder"
> +DESCRIPTION = "A free software library and application for encoding video
> streams into the H.265/HEVC format."
> +HOMEPAGE = "http://www.videolan.org/developers/x265.html;
> +
> +LICENSE = "GPLv2"
> +LICENSE_FLAGS = "commercial"
> +LIC_FILES_CHKSUM =
> "file://../COPYING;md5=c9e0427bc58f129f99728c62d4ad4091"
> +
> +DEPENDS = "nasm-native gnutls zlib libpcre"
> +
> +SRC_URI = "http://ftp.videolan.org/pub/videolan/x265/x265_${PV}.tar.gz;
> +
> +S = "${WORKDIR}/x265_${PV}/source"
> +
> +SRC_URI[md5sum] = "94808045a34d88a857e5eaf3f68f4bca"
> +SRC_URI[sha256sum] =
> "fb9badcf92364fd3567f8b5aa0e5e952aeea7a39a2b864387cec31e3b58cbbcc"
> +
> +inherit lib_package pkgconfig cmake
> +
> +AS[unexport] = "1"
> +
> +COMPATIBLE_MACHINE = "^$"
> +COMPATIBLE_MACHINE_x86 = ".*"
> +COMPATIBLE_MACHINE_x86-64 = ".*"
> --
> 2.17.1
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] perf: Correct the substitution of python shebangs

[Scott Branden via Openembedded-core]

Thank you - works for me.  I've been waiting for a fix!

On 2020-02-13 10:38 a.m., zhe...@windriver.com wrote:

From: He Zhe 

To make the native python3 are always used,

- Move the substitution of /usr/bin/python3 to first, otherwise the
   possible original /usr/bin/python3 would be changed to
   /usr/bin/env python33.
- Add substitution for ${S}/scripts/

Signed-off-by: He Zhe 

Tested-by: Scott Branden 

---
  meta/recipes-kernel/perf/perf.bb | 7 +++
  1 file changed, 7 insertions(+)

diff --git a/meta/recipes-kernel/perf/perf.bb b/meta/recipes-kernel/perf/perf.bb
index 6d1b066..14a9090 100644
--- a/meta/recipes-kernel/perf/perf.bb
+++ b/meta/recipes-kernel/perf/perf.bb
@@ -240,10 +240,17 @@ do_configure_prepend () {
  
  # use /usr/bin/env instead of version specific python

  for s in `find ${S}/tools/perf/ -name '*.py'`; do
+sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
  sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
  sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
  sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
+done
+
+for s in `find ${S}/scripts/ -name '*.py'`; do
  sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
  done
  
  # unistd.h can be out of sync between libc-headers and the captured version in the perf source


--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] perf: Correct the substitution of python shebangs

[zhe.he]

From: He Zhe 

To make the native python3 are always used,

- Move the substitution of /usr/bin/python3 to first, otherwise the
  possible original /usr/bin/python3 would be changed to
  /usr/bin/env python33.
- Add substitution for ${S}/scripts/

Signed-off-by: He Zhe 
---
 meta/recipes-kernel/perf/perf.bb | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/meta/recipes-kernel/perf/perf.bb b/meta/recipes-kernel/perf/perf.bb
index 6d1b066..14a9090 100644
--- a/meta/recipes-kernel/perf/perf.bb
+++ b/meta/recipes-kernel/perf/perf.bb
@@ -240,10 +240,17 @@ do_configure_prepend () {
 
 # use /usr/bin/env instead of version specific python
 for s in `find ${S}/tools/perf/ -name '*.py'`; do
+sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
 sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
 sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
 sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
+done
+
+for s in `find ${S}/scripts/ -name '*.py'`; do
 sed -i 's,/usr/bin/python3,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/python,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/python2,/usr/bin/env python3,' "${s}"
+sed -i 's,/usr/bin/env python2,/usr/bin/env python3,' "${s}"
 done
 
 # unistd.h can be out of sync between libc-headers and the captured 
version in the perf source
-- 
2.7.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2] dnf, libdnf: Ignore if PACKAGE_CLASSES does not have rpm

[Alexander Kanavin]

On Thu, 13 Feb 2020 at 17:42, Khem Raj  wrote:

> This still sneaks in pieces of rpm into distros where rpm is not needed so
> better would be to retool dnf to not demand rpm support in libsolv and
> break the dependency chain. Opkg is usually sought for small systems and
> last thing we want is unneeded dependencies
>
> I would rather use the OE configuration powers as such but I am not
> opposed to such a solution
>

You can always set these things (rpm disabling in libsolv, blacklisting
libdnf/dnf) in your distro configuration, but I would like less complexity
and special-casing in poky, not more, for the pieces that I maintain at
least.

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Alexander Kanavin]

On Thu, 13 Feb 2020 at 17:44, Khem Raj  wrote:

> I'd like to request that these two are held back until we figure out what
 is going wrong with the rpm test (see separate thread).

>>>
>>> We already know what’s going on what more
>>> Do you want to know ?
>>>
>>
>>  As the rpm maintainer, yes I do. The test is not being skipped when it
>> should be, and I'd like to know why.
>>
>
> I see that as separate issue orthogonal to these patches
>

Merging these patches will exclude rpm from images, and so will mask the
issue, so it's not orthogonal.

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Khem Raj]

On Thu, Feb 13, 2020 at 8:19 AM Alexander Kanavin 
wrote:

> On Thu, 13 Feb 2020 at 17:07, Khem Raj  wrote:
>
>>
>>> I'd like to request that these two are held back until we figure out
>>> what is going wrong with the rpm test (see separate thread).
>>>
>>
>> We already know what’s going on what more
>> Do you want to know ?
>>
>
>  As the rpm maintainer, yes I do. The test is not being skipped when it
> should be, and I'd like to know why.
>

I see that as separate issue orthogonal to these patches


> Alex
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2] dnf, libdnf: Ignore if PACKAGE_CLASSES does not have rpm

[Khem Raj]

On Thu, Feb 13, 2020 at 8:24 AM Alexander Kanavin 
wrote:

> On Thu, 13 Feb 2020 at 17:06, Khem Raj  wrote:
>
>> No, problem is that if One wants to use opkg backend then rpm drags along
>> but apt or dpkg don’t
>>
>> Even if test is made to pass rpm will be dragged into
>> Images which is not desired
>>
>
> We can modify the rpm packaging so that only the library from rpm that
> libresolv links with, but not the executables or other libraries are in the
> image. Would you be ok with that?
>

This still sneaks in pieces of rpm into distros where rpm is not needed so
better would be to retool dnf to not demand rpm support in libsolv and
break the dependency chain. Opkg is usually sought for small systems and
last thing we want is unneeded dependencies

I would rather use the OE configuration powers as such but I am not opposed
to such a solution


> Alex
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [meta-multimedia][PATCH] x265: add x265 recipe

[Scott Branden via Openembedded-core]

Add x265 recipe for latest tag of Release_3.2 branch (3.2.1).

Signed-off-by: Scott Branden 
---
 .../recipes-multimedia/x265/x265_3.2.1.bb | 24 +++
 1 file changed, 24 insertions(+)
 create mode 100644 
meta-openembedded/meta-multimedia/recipes-multimedia/x265/x265_3.2.1.bb

diff --git 
a/meta-openembedded/meta-multimedia/recipes-multimedia/x265/x265_3.2.1.bb 
b/meta-openembedded/meta-multimedia/recipes-multimedia/x265/x265_3.2.1.bb
new file mode 100644
index 00..68c1a55c8c
--- /dev/null
+++ b/meta-openembedded/meta-multimedia/recipes-multimedia/x265/x265_3.2.1.bb
@@ -0,0 +1,24 @@
+SUMMARY = "H.265/HEVC video encoder"
+DESCRIPTION = "A free software library and application for encoding video 
streams into the H.265/HEVC format."
+HOMEPAGE = "http://www.videolan.org/developers/x265.html;
+
+LICENSE = "GPLv2"
+LICENSE_FLAGS = "commercial"
+LIC_FILES_CHKSUM = "file://../COPYING;md5=c9e0427bc58f129f99728c62d4ad4091"
+
+DEPENDS = "nasm-native gnutls zlib libpcre"
+
+SRC_URI = "http://ftp.videolan.org/pub/videolan/x265/x265_${PV}.tar.gz;
+
+S = "${WORKDIR}/x265_${PV}/source"
+
+SRC_URI[md5sum] = "94808045a34d88a857e5eaf3f68f4bca"
+SRC_URI[sha256sum] = 
"fb9badcf92364fd3567f8b5aa0e5e952aeea7a39a2b864387cec31e3b58cbbcc"
+
+inherit lib_package pkgconfig cmake
+
+AS[unexport] = "1"
+
+COMPATIBLE_MACHINE = "^$"
+COMPATIBLE_MACHINE_x86 = ".*"
+COMPATIBLE_MACHINE_x86-64 = ".*"
-- 
2.17.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] ffmpeg: add PACKAGECONFIG support for x265

[Scott Branden via Openembedded-core]

Add PACKAGECONFIG support for x265 in ffmpeg.

Signed-off-by: Scott Branden 
---
 meta/recipes-multimedia/ffmpeg/ffmpeg_4.2.2.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-multimedia/ffmpeg/ffmpeg_4.2.2.bb 
b/meta/recipes-multimedia/ffmpeg/ffmpeg_4.2.2.bb
index a3cf0c1a37..20a8249b02 100644
--- a/meta/recipes-multimedia/ffmpeg/ffmpeg_4.2.2.bb
+++ b/meta/recipes-multimedia/ffmpeg/ffmpeg_4.2.2.bb
@@ -74,6 +74,7 @@ PACKAGECONFIG[vaapi] = "--enable-vaapi,--disable-vaapi,libva"
 PACKAGECONFIG[vdpau] = "--enable-vdpau,--disable-vdpau,libvdpau"
 PACKAGECONFIG[vpx] = "--enable-libvpx,--disable-libvpx,libvpx"
 PACKAGECONFIG[x264] = "--enable-libx264,--disable-libx264,x264"
+PACKAGECONFIG[x265] = "--enable-libx265,--disable-libx265,x265"
 PACKAGECONFIG[xcb] = "--enable-libxcb,--disable-libxcb,libxcb"
 PACKAGECONFIG[xv] = "--enable-outdev=xv,--disable-outdev=xv,libxv"
 PACKAGECONFIG[zlib] = "--enable-zlib,--disable-zlib,zlib"
-- 
2.17.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2] dnf, libdnf: Ignore if PACKAGE_CLASSES does not have rpm

[Alexander Kanavin]

On Thu, 13 Feb 2020 at 17:06, Khem Raj  wrote:

> No, problem is that if One wants to use opkg backend then rpm drags along
> but apt or dpkg don’t
>
> Even if test is made to pass rpm will be dragged into
> Images which is not desired
>

We can modify the rpm packaging so that only the library from rpm that
libresolv links with, but not the executables or other libraries are in the
image. Would you be ok with that?

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Alexander Kanavin]

On Thu, 13 Feb 2020 at 17:07, Khem Raj  wrote:

>
>> I'd like to request that these two are held back until we figure out what
>> is going wrong with the rpm test (see separate thread).
>>
>
> We already know what’s going on what more
> Do you want to know ?
>

 As the rpm maintainer, yes I do. The test is not being skipped when it
should be, and I'd like to know why.

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] wpa-supplicant: add virtual supplicant

[Maxime Roussin-Bélanger]

On Thu, Feb 13, 2020 at 11:00 AM Alexander Kanavin 
wrote:
>
> On Thu, 13 Feb 2020 at 16:55, Maxime Roussin-Bélanger <
maxime.roussinbelan...@gmail.com> wrote:
>>
>> PACKAGECONFIG[iwd] = "--enable-iwd, --disable-wifi, iwd, iwd"
>> PACKAGECONFIG[wpa-supplicant] = "--enable-wifi, --disable-wifi,
wpa-supplicant, wpa-supplicant"
>
>
> The problem with this, is that enabling one but not the other will
combine into '--enable-iwd --disable-wifi' which is not what is intended.

Actually, it is what is intended, the `--disable-wifi` will disable
wpa-supplicant support, but keep the wifi support from other plugins (i.e.
iwd).

I agree that maybe the `--enable-wifi` is probably not the best option
name, but I am not sure I can something about it :)

>
>
> I am not sure why you need to specify them as both build and runtime
dependencies, can you clarify why?

iwd is probably not needed as a build dependency.. I would have to recheck
that. ConnMan
needs the runtime for the dbus interface that iwd exposes.

wpa-supplicant is probably the same thing. I'm not sure why originally
wpa-supplicant is specified as
a build dependency. It's been there for 5 years..

Thanks,
Max.

>
> Alex

>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 8/9] binutils: Bail out if gold is used on 32/64 bit RISC-V

[Khem Raj]

On Wed, Feb 12, 2020 at 11:31 PM Adrian Bunk  wrote:

> On Wed, Feb 12, 2020 at 10:38:22PM -0800, Khem Raj wrote:
> > gold is not supporting risc-v yet and, this will cause subltle errors
> > during build e.g. autoconf not finding certain features like visibility
> > and it could take a while to unwind the problem. Its better to error out
> > early
> >...
> > @@ -163,3 +165,8 @@ inherit update-alternatives
> >  ALTERNATIVE_PRIORITY = "100"
> >
> >  ALTERNATIVE_${PN}_class-target = "${USE_ALTERNATIVES_FOR}"
> > +
> > +python () {
> > +if bb.utils.contains('DISTRO_FEATURES', 'ld-is-gold', True, False,
> d):
> > +bb.fatal("Gold linker does not _yet_ support RISC-V
> architecture please remove ld-is-gold from DISTRO_FEATURES")
> > +}
>
> This lacks some check for RISC-V,
> currently it errors for all architectures.


Yes this in itself was not enough please try the v2 series


>
> cu
> Adrian
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Khem Raj]

On Thu, Feb 13, 2020 at 12:21 AM Alexander Kanavin 
wrote:

> On Thu, 13 Feb 2020 at 07:38, Khem Raj  wrote:
>
>>   libsolv: Enable rpm packageconfig by default only if rpm O_P_M is
>> enabled
>>   dnf,libdnf: Ignore if PACKAGE_CLASSES does not have rpm
>>
>
> I'd like to request that these two are held back until we figure out what
> is going wrong with the rpm test (see separate thread).
>

We already know what’s going on what more
Do you want to know ?


> Alex
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2] dnf, libdnf: Ignore if PACKAGE_CLASSES does not have rpm

[Khem Raj]

On Wed, Feb 12, 2020 at 11:53 PM Alexander Kanavin 
wrote:

> World builds also enable apt and dpkg, but there is no problem with that
> for you? There is no consistency here; if you want to start excluding
> things from world based on what packaging is enabled, you need to treat all
> options equally, including deb and ipk.
>

No, problem is that if One wants to use opkg backend then rpm drags along
but apt or dpkg don’t
World builds are also problem but secondary
Please try to understand the entirety of problem

I tried everything to keep it building but i could not and I have been
saying that all along

If you can come up with patches to dnf to build with libresolv that is not
build with rpm support we might get to built however I don’t think it will
of any use at runtime


>
> The original problem was that libsolv pulls in the full set of rpm
> executables and libraries, which erroneously triggers the rpm test on
> target. We can solve this by making libsolv pull in only the rpm libraries
> and not executables,
>

Libsolve already has option to disable rpmdb support and we use it

so that rpm test will be skipped. Or maybe even make the test itself check
> whether rpm is the packaging format and skip otherwise.
>

No. Skipping test is just symptom of the problem I knew we could paper over
it but that’s not the point

Even if test is made to pass rpm will be dragged into
Images which is not desired


> Alex
>
> On Thu 13. Feb 2020 at 7.01, Khem Raj  wrote:
>
>> On Wed, Feb 12, 2020 at 8:49 PM Alexander Kanavin
>>  wrote:
>> >
>> > But you do not need to fix or touch dnf at all. You only need to adjust
>> rpm packaging and split it in two parts instead of bundling everything into
>> a single package.
>> >
>>
>> when we do world builds without package_rpm, dnf and related packages
>> get compiled which is unnecessary and moreover wrong and when we mark
>> libsolv to enable rpmdb only for package_rpm, this becomes even more
>> evident.
>>
>> Perhaps, I am missing what you intend to convey, in that case maybe you
>> can send
>> code changes to support what you are intending here.
>>
>> > Alex
>> >
>> > On Wed 12. Feb 2020 at 23.16, Khem Raj  wrote:
>> >>
>> >> On Wed, Feb 12, 2020 at 2:00 PM Alexander Kanavin
>> >>  wrote:
>> >> >
>> >> > I would still prefer to just split rpm packaging into binaries and
>> libraries, and avoid the need to blacklist and tweak recipes based on what
>> PACKAGE_CLASSES is set to altogether. That setting should not leak into
>> recipes, and should only matter from do_package onwards. Can you look into
>> the rpm recipe please?
>> >> >
>> >>
>> >>  dnf and dnf related packages expecting rpmdb support in libresolv. I
>> >> do not know dnf well enough to fix it and moreover it has no direct
>> >> use for my usecase either. I also don't see a point of entertaining
>> >> dnf for non-rpm backends. Its not intended for that either even if we
>> >> were to make dnf not want rpmdb what good will it do to build for
>> >> distros who do not use rpm. These changes do not change poky defaults
>> >> which use rpm as default. OE-core does not and I dont know of any
>> >> opkg user who also has rpmdb needed.
>> >>
>> >> > Alex
>> >> >
>> >> > On Wed, 12 Feb 2020 at 20:19, Khem Raj  wrote:
>> >> >>
>> >> >> dnf does not work with opkg or dpkg/apt anyway
>> >> >>
>> >> >> Signed-off-by: Khem Raj 
>> >> >> ---
>> >> >> v2: Use PNBLACKLIST instead of anon python
>> >> >>
>> >> >>  meta/recipes-devtools/dnf/dnf_4.2.2.bb| 2 ++
>> >> >>  meta/recipes-devtools/libdnf/libdnf_0.28.1.bb | 1 +
>> >> >>  2 files changed, 3 insertions(+)
>> >> >>
>> >> >> diff --git a/meta/recipes-devtools/dnf/dnf_4.2.2.bb
>> b/meta/recipes-devtools/dnf/dnf_4.2.2.bb
>> >> >> index f38167f1ad..220f1aabbd 100644
>> >> >> --- a/meta/recipes-devtools/dnf/dnf_4.2.2.bb
>> >> >> +++ b/meta/recipes-devtools/dnf/dnf_4.2.2.bb
>> >> >> @@ -84,3 +84,5 @@ SYSTEMD_SERVICE_${PN} = "dnf-makecache.service
>> dnf-makecache.timer \
>> >> >>   dnf-automatic-notifyonly.service
>> dnf-automatic-notifyonly.timer \
>> >> >>  "
>> >> >>  SYSTEMD_AUTO_ENABLE ?= "disable"
>> >> >> +
>> >> >> +PNBLACKLIST[dnf] ?= "${@bb.utils.contains('PACKAGE_CLASSES',
>> 'package_rpm', '', 'does not build correctly without package_rpm in
>> PACKAGE_CLASSES', d)}"
>> >> >> diff --git a/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
>> b/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
>> >> >> index 882c435b32..49afa04812 100644
>> >> >> --- a/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
>> >> >> +++ b/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
>> >> >> @@ -26,4 +26,5 @@ EXTRA_OECMAKE_append_class-native = "
>> -DWITH_GIR=OFF"
>> >> >>  EXTRA_OECMAKE_append_class-nativesdk = " -DWITH_GIR=OFF"
>> >> >>
>> >> >>  BBCLASSEXTEND = "native nativesdk"
>> >> >> +PNBLACKLIST[libdnf] ?= "${@bb.utils.contains('PACKAGE_CLASSES',
>> 'package_rpm', '', 'does not build correctly without package_rpm in
>> PACKAGE_CLASSES', d)}"
>> >> >>
>> >> >> --

Re: [OE-core] [PATCH] wpa-supplicant: add virtual supplicant

[Alexander Kanavin]

On Thu, 13 Feb 2020 at 16:55, Maxime Roussin-Bélanger <
maxime.roussinbelan...@gmail.com> wrote:

> PACKAGECONFIG[iwd] = "--enable-iwd, --disable-wifi, iwd, iwd"
> PACKAGECONFIG[wpa-supplicant] = "--enable-wifi, --disable-wifi,
> wpa-supplicant, wpa-supplicant"
>

The problem with this, is that enabling one but not the other will combine
into '--enable-iwd --disable-wifi' which is not what is intended.

I am not sure why you need to specify them as both build and runtime
dependencies, can you clarify why?

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] wpa-supplicant: add virtual supplicant

[Maxime Roussin-Bélanger]

On Thu, Feb 13, 2020 at 10:27 AM Richard Purdie <
richard.pur...@linuxfoundation.org> wrote:
>
> On Thu, 2020-02-13 at 10:22 -0500, Maxime Roussin-Bélanger wrote:
> > On Thu, Feb 13, 2020 at 9:24 AM Richard Purdie <
> > richard.pur...@linuxfoundation.org> wrote:
> > > I know its not easy to understand but we don't support virtual/*
> > > names in the runtime namespace, only the build time one
> > > (PROVIDES/DEPENDS).
> > > Grep for VIRTUAL-RUNTIME to see how we handle this (its ugly I
> > know).
> >
> > Hmm interesting!
> >
> > If I follow correctly that's mean I have to create:
> >
> > VIRTUAL-RUNTIME_wireless-daemon and then people can override it.
>
> Correct.
>
> > If the (R)DEPENDS on a recipe change depending on the wireless daemon
> > that
> > was chosen(i.e. connman), do you handle it with d.getVar and
> > d.setVar("DEPENDS... inside a
> > anonymous python function inside the recipe?
> >
> > The packageconfig would like this ?
> >
> > PACKAGECONFIG[wifi] = "--enable-wifi, --disable-wifi, ${VIRTUAL-
> > RUNTIME_wireless-daemon}, ${VIRTUAL-RUNTIME_wireless-daemon}"
> >
> > Example:
> >
> > if VIRTUAL-RUNTIME_wireless-daemon == "iwd":
> > DEPENDS += "iwd"
> > EXTRA_OECONF += "--enable-iwd"
> > elif VIRTUAL-RUNTIME_wireless-daemon == "wpa-supplicant":
> > DEPENDS += "wpa-supplicant"
>
> At this point it suggests they're not interchangeable and therefore not
> simple runtime switchable dependencies.
>
> The closest analogy is the "ld-is-gold" DISTRO_FEATURE when you have
> something that isn't compatible like this.

I realize that my example isn't the proper one.. sorry about that! We can
probably
simplify with PACKAGECONFIG.

Maybe the right solution would be to provide two packageconfig?

PACKAGECONFIG[iwd] = "--enable-iwd, --disable-wifi, iwd, iwd"
PACKAGECONFIG[wpa-supplicant] = "--enable-wifi, --disable-wifi,
wpa-supplicant, wpa-supplicant"

They are mutually exclusive, is this something that can be done through
PACKAGECONFIG?

Maybe we don't deal with the conflicts that can occur if a user activates
both? It would probably
fail at configure time? What is good practice here?

At the same time, it is VERY weird for a system to have iwd and
wpa_supplicant. If I recall correctly,
iwd has an error message with another wireless daemon run concurrently.

I would like to be able to force the user to choose only one... because I
think having both is an error.


Thanks,
Max.

>
> Cheers,
>
> Richard
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] wpa-supplicant: add virtual supplicant

[Richard Purdie]

On Thu, 2020-02-13 at 10:22 -0500, Maxime Roussin-Bélanger wrote:
> On Thu, Feb 13, 2020 at 9:24 AM Richard Purdie <
> richard.pur...@linuxfoundation.org> wrote:
> > I know its not easy to understand but we don't support virtual/* 
> > names in the runtime namespace, only the build time one
> > (PROVIDES/DEPENDS).
> > Grep for VIRTUAL-RUNTIME to see how we handle this (its ugly I
> know).
> 
> Hmm interesting! 
> 
> If I follow correctly that's mean I have to create:
> 
> VIRTUAL-RUNTIME_wireless-daemon and then people can override it.

Correct.

> If the (R)DEPENDS on a recipe change depending on the wireless daemon
> that
> was chosen(i.e. connman), do you handle it with d.getVar and
> d.setVar("DEPENDS... inside a 
> anonymous python function inside the recipe?
> 
> The packageconfig would like this ?
> 
> PACKAGECONFIG[wifi] = "--enable-wifi, --disable-wifi, ${VIRTUAL-
> RUNTIME_wireless-daemon}, ${VIRTUAL-RUNTIME_wireless-daemon}"
> 
> Example:
> 
> if VIRTUAL-RUNTIME_wireless-daemon == "iwd":
> DEPENDS += "iwd"
> EXTRA_OECONF += "--enable-iwd"
> elif VIRTUAL-RUNTIME_wireless-daemon == "wpa-supplicant":
> DEPENDS += "wpa-supplicant"

At this point it suggests they're not interchangeable and therefore not
simple runtime switchable dependencies.

The closest analogy is the "ld-is-gold" DISTRO_FEATURE when you have
something that isn't compatible like this.

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] wpa-supplicant: add virtual supplicant

[Maxime Roussin-Bélanger]

On Thu, Feb 13, 2020 at 9:24 AM Richard Purdie <
richard.pur...@linuxfoundation.org> wrote:
>
> On Mon, 2020-02-10 at 16:42 -0500, Maxime Roussin-Bélanger wrote:
> > There is a second supplicant raising, iwd. User should be
> > able to switch between wpa-supplicant and iwd. Let's provide
> > a virtual package.
> >
> > ConnMan, systemd-networkd and NetworkManager supports iwd.
> >
> > Signed-off-by: Maxime Roussin-Bélanger 
> > ---
> >  meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_2.9.bb | 3 +++
> >  meta/recipes-core/packagegroups/packagegroup-base.bb   | 2 +-
> >  2 files changed, 4 insertions(+), 1 deletion(-)
>
> I know its not easy to understand but we don't support virtual/* names
> in the runtime namespace, only the build time one (PROVIDES/DEPENDS).
>
> Grep for VIRTUAL-RUNTIME to see how we handle this (its ugly I know).

Hmm interesting!

If I follow correctly that's mean I have to create:

VIRTUAL-RUNTIME_wireless-daemon and then people can override it.

If the (R)DEPENDS on a recipe change depending on the wireless daemon that
was chosen(i.e. connman), do you handle it with d.getVar and
d.setVar("DEPENDS... inside a
anonymous python function inside the recipe?

The packageconfig would like this ?

PACKAGECONFIG[wifi] = "--enable-wifi, --disable-wifi,
${VIRTUAL-RUNTIME_wireless-daemon}, ${VIRTUAL-RUNTIME_wireless-daemon}"

Example:

if VIRTUAL-RUNTIME_wireless-daemon == "iwd":
DEPENDS += "iwd"
EXTRA_OECONF += "--enable-iwd"
elif VIRTUAL-RUNTIME_wireless-daemon == "wpa-supplicant":
DEPENDS += "wpa-supplicant"


Thanks,
Max.

>
> Cheers,
>
> Richard
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] oe-core: IMAGE_INSTALL_append = "bash"

[Robert P. J. Day]

  i have some scripts that wander the layers and look for possible
typos, and i ran across this:

meta/lib/oeqa/selftest/cases/incompatible_lic.py:IMAGE_INSTALL_append = "bash"

should that value have a leading space or not? i'll let someone else
deal with that if it needs fixing.

rday
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 07/10] busybox: Backport patches to support removal of __NR_clock_gettime

[Richard Purdie]

On Thu, 2020-02-13 at 00:21 -0800, Khem Raj wrote:
> This helps compiling with musl on 32bit arches now that musl has
> switched to 64bit time_t
> 
> Signed-off-by: Khem Raj 
> ---
>  ...rappers-around-clock_gettime-closes-.patch | 113 ++
>  ...-Use-64-prefix-syscall-if-we-have-to.patch |  53 
>  ...-Use-64-prefix-syscall-if-we-have-to.patch |  43 +++
>  meta/recipes-core/busybox/busybox_1.31.1.bb   |   3 +
>  4 files changed, 212 insertions(+)
>  create mode 100644 
> meta/recipes-core/busybox/busybox/0001-Remove-syscall-wrappers-around-clock_gettime-closes-.patch
>  create mode 100644 
> meta/recipes-core/busybox/busybox/0001-date-Use-64-prefix-syscall-if-we-have-to.patch
>  create mode 100644 
> meta/recipes-core/busybox/busybox/0001-time-Use-64-prefix-syscall-if-we-have-to.patch

Patches fail to apply:

https://autobuilder.yoctoproject.org/typhoon/#/builders/15/builds/1797

Will abort this test run, drop this and try again.

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] ✗ patchtest: failure for package_ipk: Minor import tweak + ensure packages regenerated

[Patchwork]

== Series Details ==

Series: package_ipk: Minor import tweak + ensure packages regenerated
Revision: 1
URL   : https://patchwork.openembedded.org/series/22606/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue Series does not apply on top of target branch 
[test_series_merge_on_head] 
  Suggested fixRebase your series on top of targeted branch
  Targeted branch  master (currently at ef24a545d1)



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] wpa-supplicant: add virtual supplicant

[Richard Purdie]

On Mon, 2020-02-10 at 16:42 -0500, Maxime Roussin-Bélanger wrote:
> There is a second supplicant raising, iwd. User should be
> able to switch between wpa-supplicant and iwd. Let's provide
> a virtual package.
> 
> ConnMan, systemd-networkd and NetworkManager supports iwd.
> 
> Signed-off-by: Maxime Roussin-Bélanger 
> ---
>  meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_2.9.bb | 3 +++
>  meta/recipes-core/packagegroups/packagegroup-base.bb   | 2 +-
>  2 files changed, 4 insertions(+), 1 deletion(-)

I know its not easy to understand but we don't support virtual/* names
in the runtime namespace, only the build time one (PROVIDES/DEPENDS).

Grep for VIRTUAL-RUNTIME to see how we handle this (its ugly I know).

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] package_ipk: Minor import tweak + ensure packages regenerated

[Richard Purdie]

The recent opkg-utils change didn't update all the packages since its
marked as ABISAFE (and has to be due to update-alternatives).

Fix a minor import issue to avoid multiple imports of glob which
also causes packages to re-generate.

Signed-off-by: Richard Purdie 
---
 meta/classes/package_ipk.bbclass | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/package_ipk.bbclass b/meta/classes/package_ipk.bbclass
index 4f23977032f..25d1ab725ad 100644
--- a/meta/classes/package_ipk.bbclass
+++ b/meta/classes/package_ipk.bbclass
@@ -45,6 +45,7 @@ def ipk_write_pkg(pkg, d):
 import subprocess
 import textwrap
 import collections
+import glob
 
 def cleanupcontrol(root):
 for p in ['CONTROL', 'DEBIAN']:
@@ -101,8 +102,7 @@ def ipk_write_pkg(pkg, d):
 bb.utils.mkdirhier(pkgoutdir)
 os.chdir(root)
 cleanupcontrol(root)
-from glob import glob
-g = glob('*')
+g = glob.glob('*')
 if not g and localdata.getVar('ALLOW_EMPTY', False) != "1":
 bb.note("Not creating empty archive for %s-%s-%s" % (pkg, 
localdata.getVar('PKGV'), localdata.getVar('PKGR')))
 return
-- 
2.20.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] icecc.bbclass: replace superfluous "+=" with "="

[Robert P. J. Day]

Signed-off-by: Robert P. J. Day 

---

  that's the only example of that i can see.

diff --git a/meta/classes/icecc.bbclass b/meta/classes/icecc.bbclass
index bc3d6f4cc8..d095305ed8 100644
--- a/meta/classes/icecc.bbclass
+++ b/meta/classes/icecc.bbclass
@@ -105,7 +105,7 @@ def icecc_dep_prepend(d):
 return "icecc-create-env-native"
 return ""

-DEPENDS_prepend += "${@icecc_dep_prepend(d)} "
+DEPENDS_prepend = "${@icecc_dep_prepend(d)} "

 get_cross_kernel_cc[vardepsexclude] += "KERNEL_CC"
 def get_cross_kernel_cc(bb,d):

-- 


Robert P. J. Day Ottawa, Ontario, CANADA
 http://crashcourse.ca

Twitter:   http://twitter.com/rpjday
LinkedIn:   http://ca.linkedin.com/in/rpjday

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [RESEND PATCH v2] make: 4.2.1 -> 4.3

[Jens Rehsack]

Announcement: https://lists.gnu.org/archive/html/bug-make/2020-01/msg00057.html

1) Remove upstream provided patches 
0001-glob-Do-not-assume-glibc-glob-internals.patch
   and 0002-glob-Do-not-assume-glibc-glob-internals.patch.

2) License has been changed to GPLv3 only

3) Important bug-fix is
   * https://lists.gnu.org/archive/html/bug-make/2018-09/msg6.html

4) Backward-incompatibilities:
   * Number signs (#) appearing inside a macro reference or function invocation
 no longer introduce comments and should not be escaped with backslashes
   * Previously appending using '+=' to an empty variable would result in a 
value
 starting with a space.  Now the initial space is only added if the variable
 already contains some value.  Similarly, appending an empty string does not
 add a trailing space.

Signed-off-by: Jens Rehsack 
---
 meta/recipes-devtools/make/make.inc   |  4 +-
 ...b-Do-not-assume-glibc-glob-internals.patch | 70 ---
 ...b-Do-not-assume-glibc-glob-internals.patch | 38 --
 meta/recipes-devtools/make/make_4.2.1.bb  | 12 
 meta/recipes-devtools/make/make_4.3.bb| 10 +++
 5 files changed, 11 insertions(+), 123 deletions(-)
 delete mode 100644 
meta/recipes-devtools/make/make/0001-glob-Do-not-assume-glibc-glob-internals.patch
 delete mode 100644 
meta/recipes-devtools/make/make/0002-glob-Do-not-assume-glibc-glob-internals.patch
 delete mode 100644 meta/recipes-devtools/make/make_4.2.1.bb
 create mode 100644 meta/recipes-devtools/make/make_4.3.bb

diff --git a/meta/recipes-devtools/make/make.inc 
b/meta/recipes-devtools/make/make.inc
index b8905bc6d3..4142cf23d3 100644
--- a/meta/recipes-devtools/make/make.inc
+++ b/meta/recipes-devtools/make/make.inc
@@ -5,9 +5,7 @@ called the makefile, which lists each of the non-source files 
and how to compute
 HOMEPAGE = "http://www.gnu.org/software/make/;
 SECTION = "devel"
 
-SRC_URI = "${GNU_MIRROR}/make/make-${PV}.tar.bz2 \
-   file://0001-glob-Do-not-assume-glibc-glob-internals.patch \
-   file://0002-glob-Do-not-assume-glibc-glob-internals.patch \
+SRC_URI = "${GNU_MIRROR}/make/make-${PV}.tar.lz \
"
 
 inherit autotools gettext pkgconfig texinfo
diff --git 
a/meta/recipes-devtools/make/make/0001-glob-Do-not-assume-glibc-glob-internals.patch
 
b/meta/recipes-devtools/make/make/0001-glob-Do-not-assume-glibc-glob-internals.patch
deleted file mode 100644
index 2b6e4d40c3..00
--- 
a/meta/recipes-devtools/make/make/0001-glob-Do-not-assume-glibc-glob-internals.patch
+++ /dev/null
@@ -1,70 +0,0 @@
-From c90a7dda6c572f79b8e78da44b6ebf8704edef65 Mon Sep 17 00:00:00 2001
-From: Paul Eggert 
-Date: Sun, 24 Sep 2017 09:12:58 -0400
-Subject: [PATCH 1/2] glob: Do not assume glibc glob internals.
-
-It has been proposed that glibc glob start using gl_lstat,
-which the API allows it to do.  GNU 'make' should not get in
-the way of this.  See:
-https://sourceware.org/ml/libc-alpha/2017-09/msg00409.html
-
-* dir.c (local_lstat): New function, like local_stat.
-(dir_setup_glob): Use it to initialize gl_lstat too, as the API
-requires.

-Upstream-Status: Backport
-Signed-off-by: Khem Raj 
-
- dir.c | 29 +++--
- 1 file changed, 27 insertions(+), 2 deletions(-)
-
-diff --git a/dir.c b/dir.c
-index f34bbf5..12eef30 100644
 a/dir.c
-+++ b/dir.c
-@@ -1299,15 +1299,40 @@ local_stat (const char *path, struct stat *buf)
- }
- #endif
- 
-+/* Similarly for lstat.  */
-+#if !defined(lstat) && !defined(WINDOWS32) || defined(VMS)
-+# ifndef VMS
-+#  ifndef HAVE_SYS_STAT_H
-+int lstat (const char *path, struct stat *sbuf);
-+#  endif
-+# else
-+/* We are done with the fake lstat.  Go back to the real lstat */
-+#   ifdef lstat
-+# undef lstat
-+#   endif
-+# endif
-+# define local_lstat lstat
-+#elif defined(WINDOWS32)
-+/* Windows doesn't support lstat().  */
-+# define local_lstat local_stat
-+#else
-+static int
-+local_lstat (const char *path, struct stat *buf)
-+{
-+  int e;
-+  EINTRLOOP (e, lstat (path, buf));
-+  return e;
-+}
-+#endif
-+
- void
- dir_setup_glob (glob_t *gl)
- {
-   gl->gl_opendir = open_dirstream;
-   gl->gl_readdir = read_dirstream;
-   gl->gl_closedir = free;
-+  gl->gl_lstat = local_lstat;
-   gl->gl_stat = local_stat;
--  /* We don't bother setting gl_lstat, since glob never calls it.
-- The slot is only there for compatibility with 4.4 BSD.  */
- }
- 
- void
--- 
-2.16.1
-
diff --git 
a/meta/recipes-devtools/make/make/0002-glob-Do-not-assume-glibc-glob-internals.patch
 
b/meta/recipes-devtools/make/make/0002-glob-Do-not-assume-glibc-glob-internals.patch
deleted file mode 100644
index d49acd9f6e..00
--- 
a/meta/recipes-devtools/make/make/0002-glob-Do-not-assume-glibc-glob-internals.patch
+++ /dev/null
@@ -1,38 +0,0 @@
-From 9858702dbd1e137262c06765919937660879f63c Mon Sep 17 00:00:00 2001
-From: Paul Eggert 
-Date: Sun, 24 Sep 2017 09:12:58 -0400
-Subject: [PATCH 2/2] glob: Do not assume glibc glob 

[OE-core] [OE-Core][PATCH v3] iputils: Fix systemd and ipv6 detection

[Alex Kiernan]

When systemd is enabled, ensure iputils detects it correctly. Split out
IPv6 only features based on ipv6 enabled in DISTRO_FEATURES.

Signed-off-by: Alex Kiernan 
---

Changes in v3:
- fix typo in SoB

Changes in v2:
- ninfod: fix systemd Documentation url error
- rarpd: rdisc: Drop PrivateUsers

 ...-fix-systemd-Documentation-url-error.patch | 28 +++
 .../0001-rarpd-rdisc-Drop-PrivateUsers.patch  | 46 +++
 .../iputils/iputils_s20190709.bb  | 23 +++---
 3 files changed, 91 insertions(+), 6 deletions(-)
 create mode 100644 
meta/recipes-extended/iputils/iputils/0001-ninfod-fix-systemd-Documentation-url-error.patch
 create mode 100644 
meta/recipes-extended/iputils/iputils/0001-rarpd-rdisc-Drop-PrivateUsers.patch

diff --git 
a/meta/recipes-extended/iputils/iputils/0001-ninfod-fix-systemd-Documentation-url-error.patch
 
b/meta/recipes-extended/iputils/iputils/0001-ninfod-fix-systemd-Documentation-url-error.patch
new file mode 100644
index ..03a3f5602dc6
--- /dev/null
+++ 
b/meta/recipes-extended/iputils/iputils/0001-ninfod-fix-systemd-Documentation-url-error.patch
@@ -0,0 +1,28 @@
+From c1f1527eb30d4a5feebf9a0757582bbf7fe3eae9 Mon Sep 17 00:00:00 2001
+From: Andrea Stevanato 
+Date: Tue, 5 Nov 2019 19:08:30 +
+Subject: [PATCH] ninfod: fix systemd Documentation url error
+
+systemd[1]: /usr/lib/systemd/system/ninfod.service:3: Invalid URL, ignoring: 
ninfod(8)
+
+Upstream-Status: Backport 
[https://github.com/iputils/iputils/commit/c1f1527eb30d4a5feebf9a0757582bbf7fe3eae9]
+Signed-off-by: Alex Kiernan 
+---
+ systemd/ninfod.service.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/systemd/ninfod.service.in b/systemd/ninfod.service.in
+index 5ab69ca00e96..8e79fcd9238e 100644
+--- a/systemd/ninfod.service.in
 b/systemd/ninfod.service.in
+@@ -1,6 +1,6 @@
+ [Unit]
+ Description=Respond to IPv6 Node Information Queries
+-Documentation=ninfod(8)
++Documentation=man:ninfod(8)
+ Requires=network.target
+ After=network.target
+ 
+-- 
+2.17.1
+
diff --git 
a/meta/recipes-extended/iputils/iputils/0001-rarpd-rdisc-Drop-PrivateUsers.patch
 
b/meta/recipes-extended/iputils/iputils/0001-rarpd-rdisc-Drop-PrivateUsers.patch
new file mode 100644
index ..d7367caf7877
--- /dev/null
+++ 
b/meta/recipes-extended/iputils/iputils/0001-rarpd-rdisc-Drop-PrivateUsers.patch
@@ -0,0 +1,46 @@
+From 6e51d529988cfc0bb357751fd767e9f1478e2b81 Mon Sep 17 00:00:00 2001
+From: Alex Kiernan 
+Date: Thu, 13 Feb 2020 06:08:45 +
+Subject: [PATCH] rarpd: rdisc: Drop PrivateUsers
+
+Neither rarpd nor rdisc can gain the necessary capabilities with
+PrivateUsers enabled.
+
+Upstream-Status: Pending
+Signed-off-by: Alex Kiernan 
+---
+ systemd/rarpd.service.in | 1 -
+ systemd/rdisc.service.in | 3 ++-
+ 2 files changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/systemd/rarpd.service.in b/systemd/rarpd.service.in
+index e600c10c93e6..f5d7621a7ce8 100644
+--- a/systemd/rarpd.service.in
 b/systemd/rarpd.service.in
+@@ -12,7 +12,6 @@ AmbientCapabilities=CAP_NET_RAW
+ DynamicUser=yes
+ PrivateTmp=yes
+ PrivateDevices=yes
+-PrivateUsers=yes
+ ProtectSystem=strict
+ ProtectHome=yes
+ ProtectControlGroups=yes
+diff --git a/systemd/rdisc.service.in b/systemd/rdisc.service.in
+index 4e2a1ec9d0e5..a71b87d36b37 100644
+--- a/systemd/rdisc.service.in
 b/systemd/rdisc.service.in
+@@ -8,9 +8,10 @@ After=network.target
+ EnvironmentFile=-/etc/sysconfig/rdisc
+ ExecStart=@sbindir@/rdisc -f -t $OPTIONS $SEND_ADDRESS $RECEIVE_ADDRESS
+ 
++CapabilityBoundingSet=CAP_NET_RAW
+ AmbientCapabilities=CAP_NET_RAW
+ PrivateTmp=yes
+-PrivateUsers=yes
++DynamicUser=yes
+ ProtectSystem=strict
+ ProtectHome=yes
+ ProtectControlGroups=yes
+-- 
+2.17.1
+
diff --git a/meta/recipes-extended/iputils/iputils_s20190709.bb 
b/meta/recipes-extended/iputils/iputils_s20190709.bb
index e0d2ae160ee0..2aed6b56ca99 100644
--- a/meta/recipes-extended/iputils/iputils_s20190709.bb
+++ b/meta/recipes-extended/iputils/iputils_s20190709.bb
@@ -12,6 +12,8 @@ DEPENDS = "gnutls"
 
 SRC_URI = "git://github.com/iputils/iputils \

file://0001-ninfod-change-variable-name-to-avoid-colliding-with-.patch \
+   file://0001-ninfod-fix-systemd-Documentation-url-error.patch \
+   file://0001-rarpd-rdisc-Drop-PrivateUsers.patch \
"
 SRCREV = "13e00847176aa23683d68fce1d17ffb523510946"
 
@@ -23,16 +25,20 @@ UPSTREAM_CHECK_GITTAGREGEX = "(?Ps\d+)"
 # breaks the version order.
 CVE_CHECK_WHITELIST += "CVE-2000-1213 CVE-2000-1214"
 
-PACKAGECONFIG ??= "libcap libgcrypt rarpd traceroute6"
+PACKAGECONFIG ??= "libcap libgcrypt rarpd \
+   ${@bb.utils.contains('DISTRO_FEATURES', 'ipv6', 'ninfod 
traceroute6', '', d)} \
+   ${@bb.utils.filter('DISTRO_FEATURES', 'systemd', d)}"
 PACKAGECONFIG[libcap] = "-DUSE_CAP=true, -DUSE_CAP=false, libcap"
 PACKAGECONFIG[libgcrypt] = "-DUSE_CRYPTO=gcrypt, -DUSE_CRYPTO=none, libgcrypt"
 PACKAGECONFIG[libidn] = 

[OE-core] [PATCH] ppp: Security fix CVE-2020-8597

[Yi Zhao]

CVE-2020-8597: eap.c in pppd in ppp 2.4.2 through 2.4.8 has an rhostname
buffer overflow in the eap_request and eap_response functions.

References:
https://nvd.nist.gov/vuln/detail/CVE-2020-8597

Patch from:
https://github.com/paulusmack/ppp/commit/8d7970b8f3db727fe798b65f3377fe6787575426

Signed-off-by: Yi Zhao 
---
 ...01-pppd-Fix-bounds-check-in-EAP-code.patch | 47 +++
 meta/recipes-connectivity/ppp/ppp_2.4.7.bb|  1 +
 2 files changed, 48 insertions(+)
 create mode 100644 
meta/recipes-connectivity/ppp/ppp/0001-pppd-Fix-bounds-check-in-EAP-code.patch

diff --git 
a/meta/recipes-connectivity/ppp/ppp/0001-pppd-Fix-bounds-check-in-EAP-code.patch
 
b/meta/recipes-connectivity/ppp/ppp/0001-pppd-Fix-bounds-check-in-EAP-code.patch
new file mode 100644
index 00..b7ba7ba643
--- /dev/null
+++ 
b/meta/recipes-connectivity/ppp/ppp/0001-pppd-Fix-bounds-check-in-EAP-code.patch
@@ -0,0 +1,47 @@
+From 8d7970b8f3db727fe798b65f3377fe6787575426 Mon Sep 17 00:00:00 2001
+From: Paul Mackerras 
+Date: Mon, 3 Feb 2020 15:53:28 +1100
+Subject: [PATCH] pppd: Fix bounds check in EAP code
+
+Given that we have just checked vallen < len, it can never be the case
+that vallen >= len + sizeof(rhostname).  This fixes the check so we
+actually avoid overflowing the rhostname array.
+
+Reported-by: Ilja Van Sprundel 
+Signed-off-by: Paul Mackerras 
+
+Upstream-Status: Backport
+[https://github.com/paulusmack/ppp/commit/8d7970b8f3db727fe798b65f3377fe6787575426]
+
+CVE: CVE-2020-8597
+
+Signed-off-by: Yi Zhao 
+---
+ pppd/eap.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/pppd/eap.c b/pppd/eap.c
+index 94407f5..1b93db0 100644
+--- a/pppd/eap.c
 b/pppd/eap.c
+@@ -1420,7 +1420,7 @@ int len;
+   }
+ 
+   /* Not so likely to happen. */
+-  if (vallen >= len + sizeof (rhostname)) {
++  if (len - vallen >= sizeof (rhostname)) {
+   dbglog("EAP: trimming really long peer name down");
+   BCOPY(inp + vallen, rhostname, sizeof (rhostname) - 1);
+   rhostname[sizeof (rhostname) - 1] = '\0';
+@@ -1846,7 +1846,7 @@ int len;
+   }
+ 
+   /* Not so likely to happen. */
+-  if (vallen >= len + sizeof (rhostname)) {
++  if (len - vallen >= sizeof (rhostname)) {
+   dbglog("EAP: trimming really long peer name down");
+   BCOPY(inp + vallen, rhostname, sizeof (rhostname) - 1);
+   rhostname[sizeof (rhostname) - 1] = '\0';
+-- 
+2.17.1
+
diff --git a/meta/recipes-connectivity/ppp/ppp_2.4.7.bb 
b/meta/recipes-connectivity/ppp/ppp_2.4.7.bb
index 644cde4562..60c56dd0bd 100644
--- a/meta/recipes-connectivity/ppp/ppp_2.4.7.bb
+++ b/meta/recipes-connectivity/ppp/ppp_2.4.7.bb
@@ -33,6 +33,7 @@ SRC_URI = "https://download.samba.org/pub/${BPN}/${BP}.tar.gz 
\
file://0001-pppoe-include-netinet-in.h-before-linux-in.h.patch \
file://0001-ppp-Remove-unneeded-include.patch \
file://ppp-2.4.7-DES-openssl.patch \
+   file://0001-pppd-Fix-bounds-check-in-EAP-code.patch \
 "
 
 SRC_URI_append_libc-musl = "\
-- 
2.17.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 10/10] binutils: Bail out if gold is used on 32/64 bit RISC-V

[Khem Raj]

gold is not supporting risc-v yet and, this will cause subltle errors
during build e.g. autoconf not finding certain features like visibility
and it could take a while to unwind the problem. Its better to error out
early

Remove dwp and ld.gold for rv32 as well

Signed-off-by: Khem Raj 
---
 meta/recipes-devtools/binutils/binutils.inc | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/meta/recipes-devtools/binutils/binutils.inc 
b/meta/recipes-devtools/binutils/binutils.inc
index 0fb6cae407..a4b9aa586d 100644
--- a/meta/recipes-devtools/binutils/binutils.inc
+++ b/meta/recipes-devtools/binutils/binutils.inc
@@ -40,6 +40,7 @@ FILES_${PN}-dev = " \
 
 LDGOLD_ALTS ?= "ld.gold dwp"
 LDGOLD_ALTS_riscv64 = ""
+LDGOLD_ALTS_riscv32 = ""
 
 USE_ALTERNATIVES_FOR = " \
addr2line \
@@ -83,6 +84,7 @@ LDGOLD_class-native = ""
 LDGOLD_class-crosssdk = ""
 LDGOLD ?= "${@bb.utils.contains('DISTRO_FEATURES', 'ld-is-gold', 
'--enable-gold=default --enable-threads', '--enable-gold --enable-ld=default 
--enable-threads', d)}"
 
+
 # This is necessary due to a bug in the binutils Makefiles
 # EXTRA_OEMAKE = "configure-build-libiberty all"
 
@@ -163,3 +165,8 @@ inherit update-alternatives
 ALTERNATIVE_PRIORITY = "100"
 
 ALTERNATIVE_${PN}_class-target = "${USE_ALTERNATIVES_FOR}"
+
+python () {
+if bb.utils.contains('DISTRO_FEATURES', 'ld-is-gold', True, False, d) and 
bb.utils.contains_any('TARGET_ARCH', 'riscv32 riscv64', True, False, d):
+bb.fatal("Gold linker does not _yet_ support RISC-V architecture 
please remove ld-is-gold from DISTRO_FEATURES")
+}
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 07/10] busybox: Backport patches to support removal of __NR_clock_gettime

[Khem Raj]

This helps compiling with musl on 32bit arches now that musl has
switched to 64bit time_t

Signed-off-by: Khem Raj 
---
 ...rappers-around-clock_gettime-closes-.patch | 113 ++
 ...-Use-64-prefix-syscall-if-we-have-to.patch |  53 
 ...-Use-64-prefix-syscall-if-we-have-to.patch |  43 +++
 meta/recipes-core/busybox/busybox_1.31.1.bb   |   3 +
 4 files changed, 212 insertions(+)
 create mode 100644 
meta/recipes-core/busybox/busybox/0001-Remove-syscall-wrappers-around-clock_gettime-closes-.patch
 create mode 100644 
meta/recipes-core/busybox/busybox/0001-date-Use-64-prefix-syscall-if-we-have-to.patch
 create mode 100644 
meta/recipes-core/busybox/busybox/0001-time-Use-64-prefix-syscall-if-we-have-to.patch

diff --git 
a/meta/recipes-core/busybox/busybox/0001-Remove-syscall-wrappers-around-clock_gettime-closes-.patch
 
b/meta/recipes-core/busybox/busybox/0001-Remove-syscall-wrappers-around-clock_gettime-closes-.patch
new file mode 100644
index 00..f6fbed338a
--- /dev/null
+++ 
b/meta/recipes-core/busybox/busybox/0001-Remove-syscall-wrappers-around-clock_gettime-closes-.patch
@@ -0,0 +1,113 @@
+From be5a505d771a77c640acc35ceaa470c80e62f954 Mon Sep 17 00:00:00 2001
+From: Denys Vlasenko 
+Date: Thu, 24 Oct 2019 16:26:55 +0200
+Subject: [PATCH] Remove syscall wrappers around clock_gettime, closes 12091
+
+12091 "Direct use of __NR_clock_gettime is not time64-safe".
+
+function old new   delta
+runsv_main  16981712 +14
+startservice 378 383  +5
+get_mono  31  25  -6
+date_main932 926  -6
+gettimeofday_ns   17   - -17
+--
+(add/remove: 0/1 grow/shrink: 2/2 up/down: 19/-29)Total: -10 bytes
+
+Signed-off-by: Denys Vlasenko 
+---
+Upstream-Status: Backport 
[https://git.busybox.net/busybox/commit/?id=be5a505d771a77c640acc35ceaa470c80e62f954]
+ Makefile.flags   |  6 --
+ coreutils/date.c | 16 +++-
+ libbb/time.c | 11 +--
+ runit/runsv.c| 11 +--
+ 4 files changed, 9 insertions(+), 35 deletions(-)
+
+--- a/Makefile.flags
 b/Makefile.flags
+@@ -129,10 +129,12 @@ endif
+ # fall back to using a temp file:
+ CRYPT_AVAILABLE := $(shell echo 'int main(void){return 0;}' >crypttest.c; 
$(CC) $(CFLAGS) -lcrypt -o /dev/null crypttest.c >/dev/null 2>&1 && echo "y"; 
rm crypttest.c)
+ ifeq ($(CRYPT_AVAILABLE),y)
+-LDLIBS += m crypt
++LDLIBS += m rt crypt
+ else
+-LDLIBS += m
++LDLIBS += m rt
+ endif
++# libm may be needed for dc, awk, ntpd
++# librt may be needed for clock_gettime()
+ 
+ # libpam may use libpthread, libdl and/or libaudit.
+ # On some platforms that requires an explicit -lpthread, -ldl, -laudit.
+--- a/coreutils/date.c
 b/coreutils/date.c
+@@ -33,10 +33,9 @@
+ //config: Enable option (-I) to output an ISO-8601 compliant
+ //config: date/time string.
+ //config:
+-//config:# defaults to "no": stat's nanosecond field is a bit non-portable
+ //config:config FEATURE_DATE_NANO
+ //config: bool "Support %[num]N nanosecond format specifier"
+-//config: default n  # syscall(__NR_clock_gettime) or 
syscall(__NR_clock_gettime64)
++//config: default n # stat's nanosecond field is a bit non-portable
+ //config: depends on DATE
+ //config: select PLATFORM_LINUX
+ //config: help
+@@ -271,17 +270,8 @@ int date_main(int argc UNUSED_PARAM, cha
+*/
+ #endif
+   } else {
+-#if ENABLE_FEATURE_DATE_NANO && defined(__NR_clock_gettime)
+-  /* libc has incredibly messy way of doing this,
+-   * typically requiring -lrt. We just skip all this mess */
+-  syscall(__NR_clock_gettime, CLOCK_REALTIME, );
+-#elif ENABLE_FEATURE_DATE_NANO && __TIMESIZE == 64
+-  /* Let's only suppor the 64 suffix syscalls for 64-bit time_t.
+-   * This simplifies the code for us as we don't need to convert
+-   * between 64-bit and 32-bit. We also don't have a way to
+-   * report overflow errors here.
+-   */
+-  syscall(__NR_clock_gettime64, CLOCK_REALTIME, );
++#if ENABLE_FEATURE_DATE_NANO
++  clock_gettime(CLOCK_REALTIME, );
+ #else
+   time(_sec);
+ #endif
+--- a/libbb/time.c
 b/libbb/time.c
+@@ -253,18 +253,9 @@ char* FAST_FUNC strftime_MMDDHHMMSS(
+ #define CLOCK_MONOTONIC 1
+ #endif
+ 
+-/* libc has incredibly messy way of doing this,
+- * typically requiring -lrt. We just skip all this mess */
+ static void get_mono(struct timespec *ts)
+ {
+-#if defined(__NR_clock_gettime)
+-  if (syscall(__NR_clock_gettime, CLOCK_MONOTONIC, ts))
+-#elif __TIMESIZE == 64
+-  if (syscall(__NR_clock_gettime64, 

[OE-core] [PATCH 08/10] ltp: Fix tescases with 64bit time_t using 32bit arches

[Khem Raj]

This helps it compile on musl

Fixes
| tst_clocks.c:31:17: error: 'SYS_clock_getres' undeclared (first use in this 
function); did you mean 'tst_clock_getres

Signed-off-by: Khem Raj 
---
 ...or-time64-unsafe-syscalls-before-usi.patch | 81 +++
 meta/recipes-extended/ltp/ltp_20190930.bb |  1 +
 2 files changed, 82 insertions(+)
 create mode 100644 
meta/recipes-extended/ltp/ltp/0001-syscalls-Check-for-time64-unsafe-syscalls-before-usi.patch

diff --git 
a/meta/recipes-extended/ltp/ltp/0001-syscalls-Check-for-time64-unsafe-syscalls-before-usi.patch
 
b/meta/recipes-extended/ltp/ltp/0001-syscalls-Check-for-time64-unsafe-syscalls-before-usi.patch
new file mode 100644
index 00..92e5fdfe6e
--- /dev/null
+++ 
b/meta/recipes-extended/ltp/ltp/0001-syscalls-Check-for-time64-unsafe-syscalls-before-usi.patch
@@ -0,0 +1,81 @@
+From 09e631419d9763a4ff08b32d9801c12b475d8ec5 Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Wed, 12 Feb 2020 22:22:17 -0800
+Subject: [PATCH] syscalls: Check for time64 unsafe syscalls before using them
+
+musl is using 64bit time_t now on 32bit architectures and these syscalls
+no longer exist, therefore its better to check for them being available
+before using them
+
+Upstream-Status: Submitted 
[http://lists.linux.it/pipermail/ltp/2020-February/015400.html]
+Signed-off-by: Khem Raj 
+---
+ lib/tst_clocks.c| 9 +
+ testcases/kernel/syscalls/gettimeofday/gettimeofday01.c | 4 
+ testcases/kernel/syscalls/gettimeofday/gettimeofday02.c | 4 
+ 3 files changed, 17 insertions(+)
+
+diff --git a/lib/tst_clocks.c b/lib/tst_clocks.c
+index 35798a4aaf..6a5b05c4ea 100644
+--- a/lib/tst_clocks.c
 b/lib/tst_clocks.c
+@@ -28,15 +28,24 @@
+ 
+ int tst_clock_getres(clockid_t clk_id, struct timespec *res)
+ {
++#if defined(__NR_clock_getres)
+   return syscall(SYS_clock_getres, clk_id, res);
++#endif
++  return -1;
+ }
+ 
+ int tst_clock_gettime(clockid_t clk_id, struct timespec *ts)
+ {
++#if defined(__NR_clock_gettime)
+   return syscall(SYS_clock_gettime, clk_id, ts);
++#endif
++  return -1;
+ }
+ 
+ int tst_clock_settime(clockid_t clk_id, struct timespec *ts)
+ {
++#if defined(__NR_clock_settime)
+   return syscall(SYS_clock_settime, clk_id, ts);
++#endif
++  return -1;
+ }
+diff --git a/testcases/kernel/syscalls/gettimeofday/gettimeofday01.c 
b/testcases/kernel/syscalls/gettimeofday/gettimeofday01.c
+index 583d8f7b9b..b498de5b68 100644
+--- a/testcases/kernel/syscalls/gettimeofday/gettimeofday01.c
 b/testcases/kernel/syscalls/gettimeofday/gettimeofday01.c
+@@ -41,7 +41,11 @@
+ #include 
+ #include 
+ 
++#ifdef __NR_gettimeofday
+ #define gettimeofday(a,b)  syscall(__NR_gettimeofday,a,b)
++#else
++#define gettimeofday(a,b) (-1)
++#endif
+ 
+ char *TCID = "gettimeofday01";
+ int TST_TOTAL = 1;
+diff --git a/testcases/kernel/syscalls/gettimeofday/gettimeofday02.c 
b/testcases/kernel/syscalls/gettimeofday/gettimeofday02.c
+index 1d60f448e8..218e017df8 100644
+--- a/testcases/kernel/syscalls/gettimeofday/gettimeofday02.c
 b/testcases/kernel/syscalls/gettimeofday/gettimeofday02.c
+@@ -23,7 +23,11 @@
+ 
+ #include "tst_test.h"
+ 
++#ifdef __NR_gettimeofday
+ #define gettimeofday(a,b)  syscall(__NR_gettimeofday,a,b)
++#else
++#define gettimeofday(a,b) (-1)
++#endif
+ 
+ static volatile sig_atomic_t done;
+ static char *str_rtime;
+-- 
+2.25.0
+
diff --git a/meta/recipes-extended/ltp/ltp_20190930.bb 
b/meta/recipes-extended/ltp/ltp_20190930.bb
index 2853b1c819..19bd7bc9c6 100644
--- a/meta/recipes-extended/ltp/ltp_20190930.bb
+++ b/meta/recipes-extended/ltp/ltp_20190930.bb
@@ -42,6 +42,7 @@ SRC_URI = "git://github.com/linux-test-project/ltp.git \
file://0001-testcases-fix-an-absent-format-string-issue.patch \
file://0001-Add-more-musl-exclusions.patch \

file://0001-nm01-Remove-prefix-zeros-of-the-addresses-output-by-.patch \
+   
file://0001-syscalls-Check-for-time64-unsafe-syscalls-before-usi.patch \
"
 
 S = "${WORKDIR}/git"
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 06/10] gdb: Upgrade to 9.1 release

[Khem Raj]

Drop security patches which are now in 9.1 already
Forward port rest of patches to 9.1
Detailed changes [1]

[1] https://lists.gnu.org/archive/html/info-gnu/2020-02/msg8.html

Signed-off-by: Khem Raj 
---
 meta/conf/distro/include/tcmode-default.inc   |   2 +-
 meta/recipes-devtools/gdb/gdb-8.3.1.inc   |  22 ---
 meta/recipes-devtools/gdb/gdb-9.1.inc |  21 +++
 ...ian_8.3.1.bb => gdb-cross-canadian_9.1.bb} |   0
 .../{gdb-cross_8.3.1.bb => gdb-cross_9.1.bb}  |   0
 ...ake-man-install-relative-to-DESTDIR.patch} |   9 +-
 ...x-nat-Define-_ABIO32-if-not-defined.patch} |   9 +-
 ...-pt_regs-uapi_pt_regs-on-GLIBC-syst.patch} |   7 +-
 ...ort-for-Renesas-SH-sh4-architecture.patch} |  41 +++---
 ...eadline.a-when-using-disable-static.patch} |  13 +-
 ...s.h.patch => 0006-use-asm-sgidefs.h.patch} |   9 +-
 ...Use-exorted-definitions-of-SIGRTMIN.patch} |  11 +-
 ...atch => 0008-Change-order-of-CFLAGS.patch} |  13 +-
 ...9-resolve-restrict-keyword-conflict.patch} |  17 +--
 ...> 0010-Fix-invalid-sigprocmask-call.patch} |  23 ++-
 ...h => 0011-gdbserver-ctrl-c-handling.patch} |  31 ++--
 .../gdb/gdb/CVE-2019-1010180.patch| 132 --
 .../gdb/{gdb_8.3.1.bb => gdb_9.1.bb}  |   0
 18 files changed, 107 insertions(+), 253 deletions(-)
 delete mode 100644 meta/recipes-devtools/gdb/gdb-8.3.1.inc
 create mode 100644 meta/recipes-devtools/gdb/gdb-9.1.inc
 rename meta/recipes-devtools/gdb/{gdb-cross-canadian_8.3.1.bb => 
gdb-cross-canadian_9.1.bb} (100%)
 rename meta/recipes-devtools/gdb/{gdb-cross_8.3.1.bb => gdb-cross_9.1.bb} 
(100%)
 rename 
meta/recipes-devtools/gdb/gdb/{0002-make-man-install-relative-to-DESTDIR.patch 
=> 0001-make-man-install-relative-to-DESTDIR.patch} (77%)
 rename 
meta/recipes-devtools/gdb/gdb/{0003-mips-linux-nat-Define-_ABIO32-if-not-defined.patch
 => 0002-mips-linux-nat-Define-_ABIO32-if-not-defined.patch} (82%)
 rename 
meta/recipes-devtools/gdb/gdb/{0004-ppc-ptrace-Define-pt_regs-uapi_pt_regs-on-GLIBC-syst.patch
 => 0003-ppc-ptrace-Define-pt_regs-uapi_pt_regs-on-GLIBC-syst.patch} (89%)
 rename 
meta/recipes-devtools/gdb/gdb/{0005-Add-support-for-Renesas-SH-sh4-architecture.patch
 => 0004-Add-support-for-Renesas-SH-sh4-architecture.patch} (96%)
 rename 
meta/recipes-devtools/gdb/gdb/{0006-Dont-disable-libreadline.a-when-using-disable-static.patch
 => 0005-Dont-disable-libreadline.a-when-using-disable-static.patch} (83%)
 rename meta/recipes-devtools/gdb/gdb/{0007-use-asm-sgidefs.h.patch => 
0006-use-asm-sgidefs.h.patch} (84%)
 rename 
meta/recipes-devtools/gdb/gdb/{0008-Use-exorted-definitions-of-SIGRTMIN.patch 
=> 0007-Use-exorted-definitions-of-SIGRTMIN.patch} (81%)
 rename meta/recipes-devtools/gdb/gdb/{0009-Change-order-of-CFLAGS.patch => 
0008-Change-order-of-CFLAGS.patch} (75%)
 rename 
meta/recipes-devtools/gdb/gdb/{0010-resolve-restrict-keyword-conflict.patch => 
0009-resolve-restrict-keyword-conflict.patch} (80%)
 rename meta/recipes-devtools/gdb/gdb/{0011-Fix-invalid-sigprocmask-call.patch 
=> 0010-Fix-invalid-sigprocmask-call.patch} (65%)
 rename meta/recipes-devtools/gdb/gdb/{0001-gdbserver-ctrl-c-handling.patch => 
0011-gdbserver-ctrl-c-handling.patch} (45%)
 delete mode 100644 meta/recipes-devtools/gdb/gdb/CVE-2019-1010180.patch
 rename meta/recipes-devtools/gdb/{gdb_8.3.1.bb => gdb_9.1.bb} (100%)

diff --git a/meta/conf/distro/include/tcmode-default.inc 
b/meta/conf/distro/include/tcmode-default.inc
index 936db5ae16..b451f0ba26 100644
--- a/meta/conf/distro/include/tcmode-default.inc
+++ b/meta/conf/distro/include/tcmode-default.inc
@@ -19,7 +19,7 @@ PREFERRED_PROVIDER_virtual/gettext ??= "gettext"
 GCCVERSION ?= "9.%"
 SDKGCCVERSION ?= "${GCCVERSION}"
 BINUVERSION ?= "2.33%"
-GDBVERSION ?= "8.3%"
+GDBVERSION ?= "9.%"
 GLIBCVERSION ?= "2.31"
 LINUXLIBCVERSION ?= "5.4%"
 QEMUVERSION ?= "4.1%"
diff --git a/meta/recipes-devtools/gdb/gdb-8.3.1.inc 
b/meta/recipes-devtools/gdb/gdb-8.3.1.inc
deleted file mode 100644
index aec913f3ce..00
--- a/meta/recipes-devtools/gdb/gdb-8.3.1.inc
+++ /dev/null
@@ -1,22 +0,0 @@
-LICENSE = "GPLv2 & GPLv3 & LGPLv2 & LGPLv3"
-LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552 \
-   file://COPYING3;md5=d32239bcb673463ab874e80d47fae504 \
-   file://COPYING3.LIB;md5=6a6a8e020838b23406c81b19c1d46df6 \
-   file://COPYING.LIB;md5=9f604d8a4f8e74f4f5140845a21b6674"
-
-SRC_URI = "${GNU_MIRROR}/gdb/gdb-${PV}.tar.xz \
-   file://0001-gdbserver-ctrl-c-handling.patch \
-   file://0002-make-man-install-relative-to-DESTDIR.patch \
-   file://0003-mips-linux-nat-Define-_ABIO32-if-not-defined.patch \
-   
file://0004-ppc-ptrace-Define-pt_regs-uapi_pt_regs-on-GLIBC-syst.patch \
-   file://0005-Add-support-for-Renesas-SH-sh4-architecture.patch \
-   
file://0006-Dont-disable-libreadline.a-when-using-disable-static.patch \
-   file://0007-use-asm-sgidefs.h.patch \
-   

[OE-core] [PATCH 09/10] crosssdk: Set natisdk specific distro features

[Khem Raj]

Currently, normal distro features e.g. ld-is-gold is impacting
crosssdk recipes, which actually should not be the case, since
that feature is essentially intended for target packages and not
nativesdk packages

Signed-off-by: Khem Raj 
---
 meta/classes/crosssdk.bbclass | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/meta/classes/crosssdk.bbclass b/meta/classes/crosssdk.bbclass
index c718a09025..04aecb694e 100644
--- a/meta/classes/crosssdk.bbclass
+++ b/meta/classes/crosssdk.bbclass
@@ -5,9 +5,15 @@ NATIVESDKLIBC ?= "libc-glibc"
 LIBCOVERRIDE = ":${NATIVESDKLIBC}"
 MACHINEOVERRIDES = ""
 PACKAGE_ARCH = "${SDK_ARCH}"
+
 python () {
 # set TUNE_PKGARCH to SDK_ARCH
 d.setVar('TUNE_PKGARCH', d.getVar('SDK_ARCH'))
+# Set features here to prevent appends and distro features backfill
+# from modifying nativesdk distro features
+features = set(d.getVar("DISTRO_FEATURES_NATIVESDK").split())
+filtered = set(bb.utils.filter("DISTRO_FEATURES", 
d.getVar("DISTRO_FEATURES_FILTER_NATIVESDK"), d).split())
+d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
 }
 
 STAGING_BINDIR_TOOLCHAIN = 
"${STAGING_DIR_NATIVE}${bindir_native}/${TARGET_ARCH}${TARGET_VENDOR}-${TARGET_OS}"
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 04/10] dnf, libdnf: Ignore if PACKAGE_CLASSES does not have rpm

[Khem Raj]

dnf does not work with opkg or dpkg/apt anyway

Signed-off-by: Khem Raj 
---
 meta/recipes-devtools/dnf/dnf_4.2.2.bb| 2 ++
 meta/recipes-devtools/libdnf/libdnf_0.28.1.bb | 1 +
 2 files changed, 3 insertions(+)

diff --git a/meta/recipes-devtools/dnf/dnf_4.2.2.bb 
b/meta/recipes-devtools/dnf/dnf_4.2.2.bb
index f38167f1ad..220f1aabbd 100644
--- a/meta/recipes-devtools/dnf/dnf_4.2.2.bb
+++ b/meta/recipes-devtools/dnf/dnf_4.2.2.bb
@@ -84,3 +84,5 @@ SYSTEMD_SERVICE_${PN} = "dnf-makecache.service 
dnf-makecache.timer \
  dnf-automatic-notifyonly.service 
dnf-automatic-notifyonly.timer \
 "
 SYSTEMD_AUTO_ENABLE ?= "disable"
+
+PNBLACKLIST[dnf] ?= "${@bb.utils.contains('PACKAGE_CLASSES', 'package_rpm', 
'', 'does not build correctly without package_rpm in PACKAGE_CLASSES', d)}"
diff --git a/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb 
b/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
index 882c435b32..49afa04812 100644
--- a/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
+++ b/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
@@ -26,4 +26,5 @@ EXTRA_OECMAKE_append_class-native = " -DWITH_GIR=OFF"
 EXTRA_OECMAKE_append_class-nativesdk = " -DWITH_GIR=OFF"
 
 BBCLASSEXTEND = "native nativesdk"
+PNBLACKLIST[libdnf] ?= "${@bb.utils.contains('PACKAGE_CLASSES', 'package_rpm', 
'', 'does not build correctly without package_rpm in PACKAGE_CLASSES', d)}"
 
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 05/10] musl: Update to latest tip

[Khem Raj]

Detailed changelog is here [1]

[1] 
https://git.musl-libc.org/cgit/musl/log/?qt=range=e6093b5a870a38ebfb3e54382acd48c698bde15d..a662220df547e5c2446518e74440a7d834f9ebe6

Signed-off-by: Khem Raj 
---
 meta/recipes-core/musl/musl_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/musl/musl_git.bb 
b/meta/recipes-core/musl/musl_git.bb
index f4853b4f2e..b6ec6fe9af 100644
--- a/meta/recipes-core/musl/musl_git.bb
+++ b/meta/recipes-core/musl/musl_git.bb
@@ -4,7 +4,7 @@
 require musl.inc
 inherit linuxloader
 
-SRCREV = "e6093b5a870a38ebfb3e54382acd48c698bde15d"
+SRCREV = "a662220df547e5c2446518e74440a7d834f9ebe6"
 
 BASEVER = "1.1.24"
 
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 03/10] libsolv: Enable rpm packageconfig by default only if rpm O_P_M is enabled

[Khem Raj]

opkg also depends on libsolv and can needlessly pull in rpm even if
the O_P_M does not desire to use rpm

Signed-off-by: Khem Raj 
---
 meta/recipes-extended/libsolv/libsolv_0.7.10.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/libsolv/libsolv_0.7.10.bb 
b/meta/recipes-extended/libsolv/libsolv_0.7.10.bb
index 502f4e0e85..265a27c00d 100644
--- a/meta/recipes-extended/libsolv/libsolv_0.7.10.bb
+++ b/meta/recipes-extended/libsolv/libsolv_0.7.10.bb
@@ -18,7 +18,7 @@ S = "${WORKDIR}/git"
 
 inherit cmake
 
-PACKAGECONFIG ??= "rpm"
+PACKAGECONFIG ??= 
"${@bb.utils.contains('PACKAGE_CLASSES','package_rpm','rpm','',d)}"
 PACKAGECONFIG[rpm] = "-DENABLE_RPMMD=ON -DENABLE_RPMDB=ON,,rpm"
 
 EXTRA_OECMAKE = "-DMULTI_SEMANTICS=ON -DENABLE_COMPLEX_DEPS=ON"
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH v2 00/10] recipe updates, testing fixes

[Khem Raj]

- Update gdb to 9.1 and musl to latest
- Fix packages busybox, ltp to build with musl exposed 64bit time_t
  problems
- rest of all fixes are found testing core-image-sato-sdk on
  qemuarm/qemux86_64
- Add a check for gold being used on rv32/rv64
- crosssdk is changed to not use target distro features

The following changes since commit 34535f3e0ca6f6e37e6457fc800dfbfff64d9298:

  bitbake: doc: minor tweaks to ch 1 of BB user manual (2020-02-08 17:41:17 
+)

are available in the Git repository at:

  git://git.yoctoproject.org/poky-contrib kraj/pu
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=kraj/pu

Khem Raj (10):
  qemuarm: Disable highmem when QB_MACHINE is virt
  oeqa: Use --disable-maintainer-mode configure option
  libsolv: Enable rpm packageconfig by default only if rpm O_P_M is
enabled
  dnf,libdnf: Ignore if PACKAGE_CLASSES does not have rpm
  musl: Update to latest tip
  gdb: Upgrade to 9.1 release
  busybox: Backport patches to support removal of __NR_clock_gettime
  ltp: Fix tescases with 64bit time_t using 32bit arches
  crosssdk: Set natisdk specific distro features
  binutils: Bail out if gold is used on 32/64 bit RISC-V

 meta/classes/crosssdk.bbclass |   6 +
 meta/conf/distro/include/tcmode-default.inc   |   2 +-
 meta/conf/machine/qemuarm.conf|   2 +-
 meta/lib/oeqa/runtime/cases/buildcpio.py  |   2 +-
 meta/lib/oeqa/sdk/cases/buildcpio.py  |   2 +-
 ...rappers-around-clock_gettime-closes-.patch | 113 +++
 ...-Use-64-prefix-syscall-if-we-have-to.patch |  53 +++
 ...-Use-64-prefix-syscall-if-we-have-to.patch |  43 ++
 meta/recipes-core/busybox/busybox_1.31.1.bb   |   3 +
 meta/recipes-core/musl/musl_git.bb|   2 +-
 meta/recipes-devtools/binutils/binutils.inc   |   7 +
 meta/recipes-devtools/dnf/dnf_4.2.2.bb|   2 +
 meta/recipes-devtools/gdb/gdb-8.3.1.inc   |  22 ---
 meta/recipes-devtools/gdb/gdb-9.1.inc |  21 +++
 ...ian_8.3.1.bb => gdb-cross-canadian_9.1.bb} |   0
 .../{gdb-cross_8.3.1.bb => gdb-cross_9.1.bb}  |   0
 ...ake-man-install-relative-to-DESTDIR.patch} |   9 +-
 ...x-nat-Define-_ABIO32-if-not-defined.patch} |   9 +-
 ...-pt_regs-uapi_pt_regs-on-GLIBC-syst.patch} |   7 +-
 ...ort-for-Renesas-SH-sh4-architecture.patch} |  41 +++---
 ...eadline.a-when-using-disable-static.patch} |  13 +-
 ...s.h.patch => 0006-use-asm-sgidefs.h.patch} |   9 +-
 ...Use-exorted-definitions-of-SIGRTMIN.patch} |  11 +-
 ...atch => 0008-Change-order-of-CFLAGS.patch} |  13 +-
 ...9-resolve-restrict-keyword-conflict.patch} |  17 +--
 ...> 0010-Fix-invalid-sigprocmask-call.patch} |  23 ++-
 ...h => 0011-gdbserver-ctrl-c-handling.patch} |  31 ++--
 .../gdb/gdb/CVE-2019-1010180.patch| 132 --
 .../gdb/{gdb_8.3.1.bb => gdb_9.1.bb}  |   0
 meta/recipes-devtools/libdnf/libdnf_0.28.1.bb |   1 +
 .../libsolv/libsolv_0.7.10.bb |   2 +-
 ...or-time64-unsafe-syscalls-before-usi.patch |  81 +++
 meta/recipes-extended/ltp/ltp_20190930.bb |   1 +
 33 files changed, 422 insertions(+), 258 deletions(-)
 create mode 100644 
meta/recipes-core/busybox/busybox/0001-Remove-syscall-wrappers-around-clock_gettime-closes-.patch
 create mode 100644 
meta/recipes-core/busybox/busybox/0001-date-Use-64-prefix-syscall-if-we-have-to.patch
 create mode 100644 
meta/recipes-core/busybox/busybox/0001-time-Use-64-prefix-syscall-if-we-have-to.patch
 delete mode 100644 meta/recipes-devtools/gdb/gdb-8.3.1.inc
 create mode 100644 meta/recipes-devtools/gdb/gdb-9.1.inc
 rename meta/recipes-devtools/gdb/{gdb-cross-canadian_8.3.1.bb => 
gdb-cross-canadian_9.1.bb} (100%)
 rename meta/recipes-devtools/gdb/{gdb-cross_8.3.1.bb => gdb-cross_9.1.bb} 
(100%)
 rename 
meta/recipes-devtools/gdb/gdb/{0002-make-man-install-relative-to-DESTDIR.patch 
=> 0001-make-man-install-relative-to-DESTDIR.patch} (77%)
 rename 
meta/recipes-devtools/gdb/gdb/{0003-mips-linux-nat-Define-_ABIO32-if-not-defined.patch
 => 0002-mips-linux-nat-Define-_ABIO32-if-not-defined.patch} (82%)
 rename 
meta/recipes-devtools/gdb/gdb/{0004-ppc-ptrace-Define-pt_regs-uapi_pt_regs-on-GLIBC-syst.patch
 => 0003-ppc-ptrace-Define-pt_regs-uapi_pt_regs-on-GLIBC-syst.patch} (89%)
 rename 
meta/recipes-devtools/gdb/gdb/{0005-Add-support-for-Renesas-SH-sh4-architecture.patch
 => 0004-Add-support-for-Renesas-SH-sh4-architecture.patch} (96%)
 rename 
meta/recipes-devtools/gdb/gdb/{0006-Dont-disable-libreadline.a-when-using-disable-static.patch
 => 0005-Dont-disable-libreadline.a-when-using-disable-static.patch} (83%)
 rename meta/recipes-devtools/gdb/gdb/{0007-use-asm-sgidefs.h.patch => 
0006-use-asm-sgidefs.h.patch} (84%)
 rename 
meta/recipes-devtools/gdb/gdb/{0008-Use-exorted-definitions-of-SIGRTMIN.patch 
=> 0007-Use-exorted-definitions-of-SIGRTMIN.patch} (81%)
 rename meta/recipes-devtools/gdb/gdb/{0009-Change-order-of-CFLAGS.patch => 
0008-Change-order-of-CFLAGS.patch} (75%)
 rename 

[OE-core] [PATCH 01/10] qemuarm: Disable highmem when QB_MACHINE is virt

[Khem Raj]

running ptests on qemuarm returns fails since it finds errors in kernel
logs like below

***
Central error: [4.338465] pci-host-generic 401000.pcie: ECAM ioremap 
failed
***

Since its a 32bit kernel 401000 address is truncated to 1000 and ends up
in conflicts with VIRT_PCIE_MMIO, which ranges from 0x1000 to 0x3efe

This is happening because the linux-yocto kernel is not compiled with
LPAE support, however, virt machine for qemuarm assumes that by default

Should LPAE be enabled by default in kernel config is a separate
question

Signed-off-by: Khem Raj 
---
 meta/conf/machine/qemuarm.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/machine/qemuarm.conf b/meta/conf/machine/qemuarm.conf
index 26f40b1419..367fcef019 100644
--- a/meta/conf/machine/qemuarm.conf
+++ b/meta/conf/machine/qemuarm.conf
@@ -11,7 +11,7 @@ SERIAL_CONSOLES ?= "115200;ttyAMA0 115200;hvc0"
 
 # For runqemu
 QB_SYSTEM_NAME = "qemu-system-arm"
-QB_MACHINE = "-machine virt"
+QB_MACHINE = "-machine virt,highmem=off"
 QB_CPU = "-cpu cortex-a15"
 # Standard Serial console
 QB_KERNEL_CMDLINE_APPEND = "console=ttyAMA0"
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/9] recipe updates, testing fixes

[Alexander Kanavin]

On Thu, 13 Feb 2020 at 07:38, Khem Raj  wrote:

>   libsolv: Enable rpm packageconfig by default only if rpm O_P_M is
> enabled
>   dnf,libdnf: Ignore if PACKAGE_CLASSES does not have rpm
>

I'd like to request that these two are held back until we figure out what
is going wrong with the rpm test (see separate thread).

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 02/10] oeqa: Use --disable-maintainer-mode configure option

[Khem Raj]

since the versions of autotools might differ on target and build host, plus 
difference in
timestamps for configure and system can result in reconfigure lets avoid
that by disabling maintainer mode

Avoids
error: newly created file is older than distributed files!

[YOCTO #13779]

Suggested-by: Andre McCurdy 
Signed-off-by: Khem Raj 
---
 meta/lib/oeqa/runtime/cases/buildcpio.py | 2 +-
 meta/lib/oeqa/sdk/cases/buildcpio.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oeqa/runtime/cases/buildcpio.py 
b/meta/lib/oeqa/runtime/cases/buildcpio.py
index 4bd72dd37e..d0f91668b2 100644
--- a/meta/lib/oeqa/runtime/cases/buildcpio.py
+++ b/meta/lib/oeqa/runtime/cases/buildcpio.py
@@ -27,6 +27,6 @@ class BuildCpioTest(OERuntimeTestCase):
 @OEHasPackage(['autoconf'])
 def test_cpio(self):
 self.project.download_archive()
-self.project.run_configure()
+self.project.run_configure('--disable-maintainer-mode','')
 self.project.run_make()
 self.project.run_install()
diff --git a/meta/lib/oeqa/sdk/cases/buildcpio.py 
b/meta/lib/oeqa/sdk/cases/buildcpio.py
index b0beafb38f..902e93f623 100644
--- a/meta/lib/oeqa/sdk/cases/buildcpio.py
+++ b/meta/lib/oeqa/sdk/cases/buildcpio.py
@@ -28,7 +28,7 @@ class BuildCpioTest(OESDKTestCase):
 self.assertTrue(os.path.isdir(dirs["source"]))
 os.makedirs(dirs["build"])
 
-self._run("cd {build} && {source}/configure 
$CONFIGURE_FLAGS".format(**dirs))
+self._run("cd {build} && {source}/configure 
--disable-maintainer-mode $CONFIGURE_FLAGS".format(**dirs))
 self._run("cd {build} && make -j".format(**dirs))
 self._run("cd {build} && make install 
DESTDIR={install}".format(**dirs))
 
-- 
2.25.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2] dnf, libdnf: Ignore if PACKAGE_CLASSES does not have rpm

[Alexander Kanavin]

I took a look at meta/lib/oeqa/runtime/cases/rpm.py.

All tests depend on test_rpm_query, which should be skipped if rpm is not
used for packaging:

status, output = self.target.run('ls /var/lib/rpm/')
if status != 0:
self.skipTest('No /var/lib/rpm on target')

Can you please check, how the tests fails for you exactly, and why
/var/lib/rpm exists (and what is the content of it)?

The test used to have the check I am talking about, but it was removed by

commit 633bd85270922d71ab4b1f94cd166da51fbe1458
Author: Richard Purdie 
Date:   Thu Nov 8 19:55:17 2018 +

oeqa/runtime/cases: Improve test dependency information
...
-@classmethod
-def setUpClass(cls):
-if cls.tc.td['PACKAGE_CLASSES'].split()[0] != 'package_rpm':
-cls.skipTest('Tests require image to be build from rpm')

Alex


On Thu, 13 Feb 2020 at 08:53, Alexander Kanavin 
wrote:

> World builds also enable apt and dpkg, but there is no problem with that
> for you? There is no consistency here; if you want to start excluding
> things from world based on what packaging is enabled, you need to treat all
> options equally, including deb and ipk.
>
> The original problem was that libsolv pulls in the full set of rpm
> executables and libraries, which erroneously triggers the rpm test on
> target. We can solve this by making libsolv pull in only the rpm libraries
> and not executables, so that rpm test will be skipped. Or maybe even make
> the test itself check whether rpm is the packaging format and skip
> otherwise.
>
> Alex
>
> On Thu 13. Feb 2020 at 7.01, Khem Raj  wrote:
>
>> On Wed, Feb 12, 2020 at 8:49 PM Alexander Kanavin
>>  wrote:
>> >
>> > But you do not need to fix or touch dnf at all. You only need to adjust
>> rpm packaging and split it in two parts instead of bundling everything into
>> a single package.
>> >
>>
>> when we do world builds without package_rpm, dnf and related packages
>> get compiled which is unnecessary and moreover wrong and when we mark
>> libsolv to enable rpmdb only for package_rpm, this becomes even more
>> evident.
>>
>> Perhaps, I am missing what you intend to convey, in that case maybe you
>> can send
>> code changes to support what you are intending here.
>>
>> > Alex
>> >
>> > On Wed 12. Feb 2020 at 23.16, Khem Raj  wrote:
>> >>
>> >> On Wed, Feb 12, 2020 at 2:00 PM Alexander Kanavin
>> >>  wrote:
>> >> >
>> >> > I would still prefer to just split rpm packaging into binaries and
>> libraries, and avoid the need to blacklist and tweak recipes based on what
>> PACKAGE_CLASSES is set to altogether. That setting should not leak into
>> recipes, and should only matter from do_package onwards. Can you look into
>> the rpm recipe please?
>> >> >
>> >>
>> >>  dnf and dnf related packages expecting rpmdb support in libresolv. I
>> >> do not know dnf well enough to fix it and moreover it has no direct
>> >> use for my usecase either. I also don't see a point of entertaining
>> >> dnf for non-rpm backends. Its not intended for that either even if we
>> >> were to make dnf not want rpmdb what good will it do to build for
>> >> distros who do not use rpm. These changes do not change poky defaults
>> >> which use rpm as default. OE-core does not and I dont know of any
>> >> opkg user who also has rpmdb needed.
>> >>
>> >> > Alex
>> >> >
>> >> > On Wed, 12 Feb 2020 at 20:19, Khem Raj  wrote:
>> >> >>
>> >> >> dnf does not work with opkg or dpkg/apt anyway
>> >> >>
>> >> >> Signed-off-by: Khem Raj 
>> >> >> ---
>> >> >> v2: Use PNBLACKLIST instead of anon python
>> >> >>
>> >> >>  meta/recipes-devtools/dnf/dnf_4.2.2.bb| 2 ++
>> >> >>  meta/recipes-devtools/libdnf/libdnf_0.28.1.bb | 1 +
>> >> >>  2 files changed, 3 insertions(+)
>> >> >>
>> >> >> diff --git a/meta/recipes-devtools/dnf/dnf_4.2.2.bb
>> b/meta/recipes-devtools/dnf/dnf_4.2.2.bb
>> >> >> index f38167f1ad..220f1aabbd 100644
>> >> >> --- a/meta/recipes-devtools/dnf/dnf_4.2.2.bb
>> >> >> +++ b/meta/recipes-devtools/dnf/dnf_4.2.2.bb
>> >> >> @@ -84,3 +84,5 @@ SYSTEMD_SERVICE_${PN} = "dnf-makecache.service
>> dnf-makecache.timer \
>> >> >>   dnf-automatic-notifyonly.service
>> dnf-automatic-notifyonly.timer \
>> >> >>  "
>> >> >>  SYSTEMD_AUTO_ENABLE ?= "disable"
>> >> >> +
>> >> >> +PNBLACKLIST[dnf] ?= "${@bb.utils.contains('PACKAGE_CLASSES',
>> 'package_rpm', '', 'does not build correctly without package_rpm in
>> PACKAGE_CLASSES', d)}"
>> >> >> diff --git a/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
>> b/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
>> >> >> index 882c435b32..49afa04812 100644
>> >> >> --- a/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
>> >> >> +++ b/meta/recipes-devtools/libdnf/libdnf_0.28.1.bb
>> >> >> @@ -26,4 +26,5 @@ EXTRA_OECMAKE_append_class-native = "
>> -DWITH_GIR=OFF"
>> >> >>  EXTRA_OECMAKE_append_class-nativesdk = " -DWITH_GIR=OFF"
>> >> >>
>> >> >>  BBCLASSEXTEND = "native nativesdk"
>> >> >> +PNBLACKLIST[libdnf] ?= 

Re: [OE-core] [PATCH] wayland: upgrade 1.17.0 -> 1.18.0

[Alexander Kanavin]

Can you prepare the meson conversion as a separate patch please?

Alex

On Thu 13. Feb 2020 at 7.29, Denys Dmytriyenko  wrote:

> From: Denys Dmytriyenko 
>
> This is the official release for Wayland 1.18. The main new features in
> this release are:
>
> - Add support for the Meson build system (autotools is still supported
>   but will be removed in a future release)
> - Add API to tag proxy objects to allow applications and toolkits to
>   share the same Wayland connection
> - Track wayland-server timers in user-space to prevent creating too
>   many FDs
> - Add wl_global_remove, a new function to mitigate race conditions with
>   globals
>
> https://lists.freedesktop.org/archives/wayland-devel/2020-February/041207.html
>
> 2 upstreamed patches are dropped.
>
> Signed-off-by: Denys Dmytriyenko 
> ---
>  ...1-scanner-Add-configure-check-for-strndup.patch |  50 ---
>  ...l_priv_signal-to-wayland-server-private.h.patch | 166
> -
>  .../{wayland_1.17.0.bb => wayland_1.18.0.bb}   |   8 +-
>  3 files changed, 3 insertions(+), 221 deletions(-)
>  delete mode 100644
> meta/recipes-graphics/wayland/wayland/0001-scanner-Add-configure-check-for-strndup.patch
>  delete mode 100644
> meta/recipes-graphics/wayland/wayland/0002-Move-wl_priv_signal-to-wayland-server-private.h.patch
>  rename meta/recipes-graphics/wayland/{wayland_1.17.0.bb =>
> wayland_1.18.0.bb} (84%)
>
> diff --git
> a/meta/recipes-graphics/wayland/wayland/0001-scanner-Add-configure-check-for-strndup.patch
> b/meta/recipes-graphics/wayland/wayland/0001-scanner-Add-configure-check-for-strndup.patch
> deleted file mode 100644
> index c443587..000
> ---
> a/meta/recipes-graphics/wayland/wayland/0001-scanner-Add-configure-check-for-strndup.patch
> +++ /dev/null
> @@ -1,50 +0,0 @@
> -From 3c8b4467a1ca229e72fb5223787ed400a19c65c0 Mon Sep 17 00:00:00 2001
> -From: Joshua Watt 
> -Date: Fri, 25 Oct 2019 21:03:23 -0500
> -Subject: [PATCH] scanner: Add configure check for strndup
> -
> -Some platforms may not have strndup() (e.g. MinGW), so provide a
> -equivalent implementation if it's not found.
> -
> -Upstream-Status: Accepted [4a1f348c20157db7bd7c759fdeb23fbe8729c571]
> -Signed-off-by: Joshua Watt 
> 
> - configure.ac  |  2 +-
> - src/scanner.c | 11 +++
> - 2 files changed, 12 insertions(+), 1 deletion(-)
> -
> -diff --git a/configure.ac b/configure.ac
> -index 8d56f2b..1c99e21 100644
>  a/configure.ac
> -+++ b/configure.ac
> -@@ -63,7 +63,7 @@ fi
> - AC_SUBST(GCC_CFLAGS)
> -
> - AC_CHECK_HEADERS([sys/prctl.h])
> --AC_CHECK_FUNCS([accept4 mkostemp posix_fallocate prctl])
> -+AC_CHECK_FUNCS([accept4 mkostemp posix_fallocate prctl strndup])
> -
> - AC_ARG_ENABLE([libraries],
> - [AC_HELP_STRING([--disable-libraries],
> -diff --git a/src/scanner.c b/src/scanner.c
> -index a94be5d..27004bc 100644
>  a/src/scanner.c
> -+++ b/src/scanner.c
> -@@ -916,6 +916,17 @@ verify_arguments(struct parse_context *ctx,
> -
> - }
> -
> -+#ifndef HAVE_STRNDUP
> -+char *
> -+strndup(const char *s, size_t size)
> -+{
> -+  char *r = malloc(size + 1);
> -+  strncpy(r, s, size);
> -+  r[size] = '\0';
> -+  return r;
> -+}
> -+#endif
> -+
> - static void
> - end_element(void *data, const XML_Char *name)
> - {
> diff --git
> a/meta/recipes-graphics/wayland/wayland/0002-Move-wl_priv_signal-to-wayland-server-private.h.patch
> b/meta/recipes-graphics/wayland/wayland/0002-Move-wl_priv_signal-to-wayland-server-private.h.patch
> deleted file mode 100644
> index b05b875..000
> ---
> a/meta/recipes-graphics/wayland/wayland/0002-Move-wl_priv_signal-to-wayland-server-private.h.patch
> +++ /dev/null
> @@ -1,166 +0,0 @@
> -From e746f3bf7c8bb2f5dcf9e93edffc23b56eff12d0 Mon Sep 17 00:00:00 2001
> -From: Joshua Watt 
> -Date: Tue, 5 Nov 2019 09:09:50 -0600
> -Subject: [PATCH] Move wl_priv_signal to wayland-server-private.h
> -
> -Including wayland-server-core.h in wayland-private.h is problematic
> -because wayland-private.h is included by wayland-scanner which should be
> -able to build against non-POSIX platforms (e.g. MinGW). The only reason
> -that wayland-server-core.h was included in wayland-private.h was for the
> -wl_private_signal definitions, so move those to a
> -wayland-server-private.h file that can be included by both
> -wayland-server.c and the tests.
> -
> -Upstream-Status: Accepted [e7d88f35eb89cf0cc77cbddd834cacc63683a9cc]
> -Signed-off-by: Joshua Watt 
> 
> - Makefile.am  |  3 +-
> - src/wayland-private.h| 22 +--
> - src/wayland-server-private.h | 53 
> - src/wayland-server.c |  1 +
> - tests/newsignal-test.c   |  2 +-
> - 5 files changed, 58 insertions(+), 23 deletions(-)
> - create mode 100644 src/wayland-server-private.h
> -
> -diff --git a/Makefile.am b/Makefile.am
> -index f47d055..026d981 100644
>  a/Makefile.am
> -+++ b/Makefile.am
> -@@ -56,7 +56,8 @@ libwayland_private_la_SOURCES =