[OE-core] [PATCH 2/2] package.bbclass: Emit HOMEPAGE pkgdata.

[Rob Walton]

Emit the contents of HOMEPAGE in the extra pkgdata. This is useful
information to have when creating license manifests.

Signed-off-by: Rob Walton 
---
 meta/classes/package.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index 4c0a859..a08aefe 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -1458,6 +1458,7 @@ fi
 write_if_exists(sf, pkg, 'PKGR')
 write_if_exists(sf, pkg, 'LICENSE')
 write_if_exists(sf, pkg, 'DESCRIPTION')
+write_if_exists(sf, pkg, 'HOMEPAGE')
 write_if_exists(sf, pkg, 'SUMMARY')
 write_if_exists(sf, pkg, 'RDEPENDS')
 rprov = write_if_exists(sf, pkg, 'RPROVIDES')
--
2.7.4

IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 1/2] image_license.bbclass: Write JSON files containing license data.

[Rob Walton]

For easier machine readability of license manifests, write a
manifest.json containing license data and package info.

Signed-off-by: Rob Walton 
---
 meta/classes/license_image.bbclass | 23 ++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/meta/classes/license_image.bbclass 
b/meta/classes/license_image.bbclass
index 6750038..6952417 100644
--- a/meta/classes/license_image.bbclass
+++ b/meta/classes/license_image.bbclass
@@ -36,6 +36,7 @@ python license_create_manifest() {
 }

 def write_license_files(d, license_manifest, pkg_dic, rootfs=True):
+import json
 import re
 import stat

@@ -43,6 +44,7 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
 bad_licenses = map(lambda l: canonical_license(d, l), bad_licenses)
 bad_licenses = expand_wildcard_licenses(d, bad_licenses)

+json_data = dict()
 with open(license_manifest, "w") as license_file:
 for pkg in sorted(pkg_dic):
 if bad_licenses:
@@ -63,7 +65,14 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
 license_file.write("PACKAGE VERSION: %s\n" % 
pkg_dic[pkg]["PV"])
 license_file.write("RECIPE NAME: %s\n" % pkg_dic[pkg]["PN"])
 license_file.write("LICENSE: %s\n\n" % pkg_dic[pkg]["LICENSE"])
-
+json_data[pkg] = {
+"RECIPE NAME": pkg_dic[pkg]["PN"],
+"PACKAGE VERSION": pkg_dic[pkg]["PV"],
+"PACKAGE NAME": pkg,
+"LICENSE": pkg_dic[pkg]["LICENSE"],
+"HOMEPAGE": pkg_dic[pkg].get("HOMEPAGE", ""),
+"SUMMARY": pkg_dic[pkg].get("SUMMARY", ""),
+}
 # If the package doesn't contain any file, that is, its size 
is 0, the license
 # isn't relevant as far as the final image is concerned. So 
doing license check
 # doesn't make much sense, skip it.
@@ -75,6 +84,14 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
 license_file.write("VERSION: %s\n" % pkg_dic[pkg]["PV"])
 license_file.write("LICENSE: %s\n" % pkg_dic[pkg]["LICENSE"])
 license_file.write("FILES: %s\n\n" % pkg_dic[pkg]["FILES"])
+json_data[pkg] = {
+"RECIPE NAME": pkg_dic[pkg]["PN"],
+"VERSION": pkg_dic[pkg]["PV"],
+"FILES": pkg_dic[pkg]["FILES"],
+"LICENSE": pkg_dic[pkg]["LICENSE"],
+"HOMEPAGE": pkg_dic[pkg].get("HOMEPAGE", ""),
+"SUMMARY": pkg_dic[pkg].get("SUMMARY", ""),
+}

 for lic in pkg_dic[pkg]["LICENSES"]:
 lic_file = os.path.join(d.getVar('LICENSE_DIRECTORY'),
@@ -89,6 +106,10 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
 "licenses collected for recipe %s"
 % (lic, pkg_dic[pkg]["PN"]))

+# Write a json file containing the license data.
+with open("{}.json".format(license_manifest), "w") as license_file_json:
+license_file_json.write(json.dumps(json_data))
+
 # Two options here:
 # - Just copy the manifest
 # - Copy the manifest and the license directories
--
2.7.4

IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] squashfs-tools: Mark as incompatible with musl

[Andre McCurdy]

On Wed, May 22, 2019 at 6:35 PM Khem Raj  wrote:
> On Wed, May 22, 2019 at 1:30 AM Adrian Bunk  wrote:
> >
> > Silently ignoring FNM_EXTMATCH makes fnmatch()
> > behave different from what callers expect.
> >
> > Mark as incompatible with musl instead of risking
> > broken runtime behaviour.
> >
> > The practical consequences should be small since
> > what is used in OE is mainly squashfs-tools-native.

The OE patch seems to take the same approach as the (3 year old)
upstream fix for the same issue, so if there are concerns maybe good
to raise them with upstream...

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

It does look dubious to me though... and since the patterns passed to
fnmatch() look like they ultimately come from the command line,
problems might only show up with more advanced usage of the squashfs
tools.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH v2 1/1] systemd-conf: configure wired network with dhcp

[Kang Kai]

On 2019/5/21 下午8:09, Richard Purdie wrote:

On Tue, 2019-05-21 at 05:22 -0400, kai.k...@windriver.com wrote:

From: Kai Kang 

Add a configure file for systemd.networkd to configure wired network
interfaces with dhcp. It works with common network interfaces such
eth0
and eno1.

Refer to
https://github.com/YoeDistro/meta-yoe/tree/master/recipes-core/systemd

[YOCTO #13057]

Signed-off-by: Kai Kang 
---
  meta/recipes-core/systemd/systemd-conf/dhcp.network | 9 +
  meta/recipes-core/systemd/systemd-conf_242.bb   | 5 +
  2 files changed, 14 insertions(+)
  create mode 100644 meta/recipes-core/systemd/systemd-
conf/dhcp.network

This seems to result in the failures in:

https://autobuilder.yoctoproject.org/typhoon/#/builders/72/builds/617


Sorry, there was something wrong during my verify build and test. v3 
will be sent.


Regards,
Kai




Cheers,

Richard




--
Kai Kang

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

Re: [OE-core] [PATCH] squashfs-tools: Mark as incompatible with musl

[Khem Raj]

On Wed, May 22, 2019 at 1:30 AM Adrian Bunk  wrote:
>
> Silently ignoring FNM_EXTMATCH makes fnmatch()
> behave different from what callers expect.
>
> Mark as incompatible with musl instead of risking
> broken runtime behaviour.
>
> The practical consequences should be small since
> what is used in OE is mainly squashfs-tools-native.
>

This has been working fine on musl for a while ar runtime as well
do you see issues ?

> Signed-off-by: Adrian Bunk 
> ---
>  .../squashfs-tools/fix-compat.patch   | 46 ---
>  .../squashfs-tools/squashfs-tools_git.bb  |  3 ++
>  2 files changed, 3 insertions(+), 46 deletions(-)
>
> diff --git 
> a/meta/recipes-devtools/squashfs-tools/squashfs-tools/fix-compat.patch 
> b/meta/recipes-devtools/squashfs-tools/squashfs-tools/fix-compat.patch
> index 0ce7b4e8ce..87c1e8cac0 100644
> --- a/meta/recipes-devtools/squashfs-tools/squashfs-tools/fix-compat.patch
> +++ b/meta/recipes-devtools/squashfs-tools/squashfs-tools/fix-compat.patch
> @@ -1,39 +1,8 @@
> -Define FNM_EXTMATCH if not defined its glibc specific define
>  include missing sys/stat.h for stat* function declarations
>
>  Upstream-Status: Pending
>  Signed-off-by: Khem Raj 
>
> -Index: squashfs-tools/action.c
> -===
>  squashfs-tools.orig/action.c
> -+++ squashfs-tools/action.c
> -@@ -44,6 +44,10 @@
> - #include "action.h"
> - #include "error.h"
> -
> -+#if !defined(FNM_EXTMATCH)
> -+#define FNM_EXTMATCH 0
> -+#endif
> -+
> - /*
> -  * code to parse actions
> -  */
> -Index: squashfs-tools/mksquashfs.c
> -===
>  squashfs-tools.orig/mksquashfs.c
> -+++ squashfs-tools/mksquashfs.c
> -@@ -1261,6 +1261,10 @@ void write_dir(squashfs_inode *inode, st
> -   dir_size + 3, directory_block, directory_offset, NULL, NULL,
> -   dir, 0);
> -
> -+#if !defined(FNM_EXTMATCH)
> -+#define FNM_EXTMATCH 0
> -+#endif
> -+
> - #ifdef SQUASHFS_TRACE
> -   {
> -   unsigned char *dirp;
>  Index: squashfs-tools/pseudo.c
>  ===
>  --- squashfs-tools.orig/pseudo.c
> @@ -46,18 +15,3 @@ Index: squashfs-tools/pseudo.c
>   #include 
>
>   #include "pseudo.h"
> -Index: squashfs-tools/unsquashfs.c
> -===
>  squashfs-tools.orig/unsquashfs.c
> -+++ squashfs-tools/unsquashfs.c
> -@@ -42,6 +42,10 @@
> - #include 
> - #endif
> -
> -+#ifndef FNM_EXTMATCH
> -+#define FNM_EXTMATCH 0
> -+#endif
> -+
> - struct cache *fragment_cache, *data_cache;
> - struct queue *to_reader, *to_inflate, *to_writer, *from_writer;
> - pthread_t *thread, *inflator_thread;
> diff --git a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb 
> b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
> index 5a8b5f77bd..dc1568a77c 100644
> --- a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
> +++ b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
> @@ -19,6 +19,9 @@ SRC_URI[lzma.sha256sum] = 
> "c935fd04dd8e0e8c688a3078f3675d699679a90be81c12686837e
>
>  S = "${WORKDIR}/git/squashfs-tools"
>
> +# needs FNM_EXTMATCH
> +COMPATIBLE_HOST_libc-musl = 'null'
> +
>  EXTRA_OEMAKE = "${PACKAGECONFIG_CONFARGS}"
>
>  PACKAGECONFIG ??= "gzip xz lzo lz4 lzma xattr"
> --
> 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 1/2] piglit: upgrade to latest revision

[Khem Raj]

On Wed, May 22, 2019 at 4:02 AM Alexander Kanavin
 wrote:
>
> Signed-off-by: Alexander Kanavin 
> ---
>  ...UPWARD-only-if-its-defined-in-fenv.h.patch | 75 ---
>  .../piglit/piglit/format-fix.patch| 69 -

are these patches upstreamed ?

>  meta/recipes-graphics/piglit/piglit_git.bb|  5 +-
>  3 files changed, 1 insertion(+), 148 deletions(-)
>  delete mode 100644 
> meta/recipes-graphics/piglit/piglit/0001-tests-Use-FE_UPWARD-only-if-its-defined-in-fenv.h.patch
>  delete mode 100644 meta/recipes-graphics/piglit/piglit/format-fix.patch
>
> diff --git 
> a/meta/recipes-graphics/piglit/piglit/0001-tests-Use-FE_UPWARD-only-if-its-defined-in-fenv.h.patch
>  
> b/meta/recipes-graphics/piglit/piglit/0001-tests-Use-FE_UPWARD-only-if-its-defined-in-fenv.h.patch
> deleted file mode 100644
> index 57eda2e207b..000
> --- 
> a/meta/recipes-graphics/piglit/piglit/0001-tests-Use-FE_UPWARD-only-if-its-defined-in-fenv.h.patch
> +++ /dev/null
> @@ -1,75 +0,0 @@
> -Upstream-Status: Submitted [mailing list]
> -Signed-off-by: Ross Burton 
> -
> -From 0e0a2a69261031d55d52b6045990e8982ea12912 Mon Sep 17 00:00:00 2001
> -From: Khem Raj 
> -Date: Sat, 9 Jul 2016 07:52:19 +
> -Subject: [PATCH] tests: only run rounding tests if FE_UPWARD is present
> -
> -On ARM, musl does not define FE_* when the architecture does not have VFP 
> (which
> -is the right interpretation).
> -
> -As these tests depend on calling fesetround(), skip the test if FE_UPWARD 
> isn't
> -available.
> -
> -Signed-off-by: Ross Burton 
> 
> - tests/general/roundmode-getintegerv.c | 12 
> - tests/general/roundmode-pixelstore.c  | 12 
> - 2 files changed, 16 insertions(+), 8 deletions(-)
> -
> -diff --git a/tests/general/roundmode-getintegerv.c 
> b/tests/general/roundmode-getintegerv.c
> -index 28ecfaf55..aa99044a1 100644
>  a/tests/general/roundmode-getintegerv.c
> -+++ b/tests/general/roundmode-getintegerv.c
> -@@ -79,13 +79,17 @@ test(float val, int expect)
> - void
> - piglit_init(int argc, char **argv)
> - {
> --  int ret;
> -   bool pass = true;
> --  ret = fesetround(FE_UPWARD);
> --  if (ret != 0) {
> --  printf("Couldn't set rounding mode\n");
> -+
> -+#ifdef FE_UPWARD
> -+  if (fesetround(FE_UPWARD) != 0) {
> -+  printf("Setting rounding mode failed\n");
> -   piglit_report_result(PIGLIT_SKIP);
> -   }
> -+#else
> -+  printf("Cannot set rounding mode\n");
> -+  piglit_report_result(PIGLIT_SKIP);
> -+#endif
> -
> -   pass = test(2.2, 2) && pass;
> -   pass = test(2.8, 3) && pass;
> -diff --git a/tests/general/roundmode-pixelstore.c 
> b/tests/general/roundmode-pixelstore.c
> -index 8a029b257..57ec11c09 100644
>  a/tests/general/roundmode-pixelstore.c
> -+++ b/tests/general/roundmode-pixelstore.c
> -@@ -79,13 +79,17 @@ test(float val, int expect)
> - void
> - piglit_init(int argc, char **argv)
> - {
> --  int ret;
> -   bool pass = true;
> --  ret = fesetround(FE_UPWARD);
> --  if (ret != 0) {
> --  printf("Couldn't set rounding mode\n");
> -+
> -+#ifdef FE_UPWARD
> -+  if (fesetround(FE_UPWARD) != 0) {
> -+  printf("Setting rounding mode failed\n");
> -   piglit_report_result(PIGLIT_SKIP);
> -   }
> -+#else
> -+  printf("Cannot set rounding mode\n");
> -+  piglit_report_result(PIGLIT_SKIP);
> -+#endif
> -
> -   pass = test(2.2, 2) && pass;
> -   pass = test(2.8, 3) && pass;
> ---
> -2.11.0
> -
> diff --git a/meta/recipes-graphics/piglit/piglit/format-fix.patch 
> b/meta/recipes-graphics/piglit/piglit/format-fix.patch
> deleted file mode 100644
> index 73d539fef27..000
> --- a/meta/recipes-graphics/piglit/piglit/format-fix.patch
> +++ /dev/null
> @@ -1,69 +0,0 @@
> -Upstream-Status: Submitted [mailing list]
> -Signed-off-by: Ross Burton 
> -
> -From f0c6981322807e179e39ce67aeebd42cf7a54d36 Mon Sep 17 00:00:00 2001
> -From: Ross Burton 
> -Date: Wed, 21 Nov 2018 12:44:36 +
> -Subject: [PATCH] arb_texture_view: fix security format warnings
> -
> -If built with -Werror=format-security then Piglit fails to build:
> -
> -/tests/spec/arb_texture_view/rendering-layers-image.c:150:8:
> -error: format not a string literal and no format arguments 
> [-Werror=format-security]
> - (desc)); \
> - ^~
> -
> -In this case test->uniform_type is being turned into a string using 
> snprintf()
> -and then passed to piglit_report_subtest_result() which takes a format 
> string,
> -but GCC can't verify the format.
> -
> -As _subtest_report() takes a format string, we can just remove the snprintf()
> -and let it construct the label.
> -
> -Also as X is used once and doesn't make the code clearer, just inline it.
> -
> -Signed-off-by: Ross Burton 
> 
> - tests/spec/arb_texture_view/rendering-layers-image.c | 19 
> ++-
> - 1 file changed, 6 insertions(+), 13 deletions(-)
> -
> -diff 

[OE-core] [PATCH v2 4/4] maintainers: Account for musl-obstack and libssp-nonshared

[Khem Raj]

Signed-off-by: Khem Raj 
Signed-off-by: Richard Purdie 
---
v2: Rebased

 meta/conf/distro/include/maintainers.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/conf/distro/include/maintainers.inc 
b/meta/conf/distro/include/maintainers.inc
index 67a51cdecc..2c10b104de 100644
--- a/meta/conf/distro/include/maintainers.inc
+++ b/meta/conf/distro/include/maintainers.inc
@@ -382,6 +382,7 @@ RECIPE_MAINTAINER_pn-libsm = "Armin Kuster 
"
 RECIPE_MAINTAINER_pn-libsndfile1 = "Tanu Kaskinen "
 RECIPE_MAINTAINER_pn-libsolv = "Anuj Mittal "
 RECIPE_MAINTAINER_pn-libsoup-2.4 = "Anuj Mittal "
+RECIPE_MAINTAINER_pn-libssp-nonshared = "Khem Raj "
 RECIPE_MAINTAINER_pn-libtasn1 = "Anuj Mittal "
 RECIPE_MAINTAINER_pn-libtest-needs-perl = "Tim Orling 
"
 RECIPE_MAINTAINER_pn-libtheora = "Alexander Kanavin "
@@ -512,6 +513,7 @@ RECIPE_MAINTAINER_pn-mtd-utils = "Denys Dmytriyenko 
"
 RECIPE_MAINTAINER_pn-mtdev = "Anuj Mittal "
 RECIPE_MAINTAINER_pn-mtools = "Anuj Mittal "
 RECIPE_MAINTAINER_pn-musl = "Khem Raj "
+RECIPE_MAINTAINER_pn-musl-obstack = "Khem Raj "
 RECIPE_MAINTAINER_pn-musl-utils = "Khem Raj "
 RECIPE_MAINTAINER_pn-mx-1.0 = "Anuj Mittal "
 RECIPE_MAINTAINER_pn-nasm = "Richard Purdie 
"
-- 
2.21.0

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

[OE-core] [PATCH V2 1/4] pam_systemd: Include missing.h for secure_getenv

[Khem Raj]

'secure_getenv' api is not uniformly implemented across all C libraries
therefore its good to include missing.h so it can use the alternative
implementation where its not awvailable

Fixes
../git/src/login/pam_systemd.c:344:13: error: implicit declaration of function 
'secure_getenv' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
v = secure_getenv(key);

Signed-off-by: Khem Raj 
---
v2: Rebased

 ...missing.h-check-for-missing-strndupa.patch | 71 +++
 1 file changed, 10 insertions(+), 61 deletions(-)

diff --git 
a/meta/recipes-core/systemd/systemd/0005-src-basic-missing.h-check-for-missing-strndupa.patch
 
b/meta/recipes-core/systemd/systemd/0005-src-basic-missing.h-check-for-missing-strndupa.patch
index a2e25a97df..df1043b27d 100644
--- 
a/meta/recipes-core/systemd/systemd/0005-src-basic-missing.h-check-for-missing-strndupa.patch
+++ 
b/meta/recipes-core/systemd/systemd/0005-src-basic-missing.h-check-for-missing-strndupa.patch
@@ -43,8 +43,6 @@ Signed-off-by: Andrej Valek 
  src/udev/udev-rules.c  |  1 +
  29 files changed, 40 insertions(+)
 
-diff --git a/meson.build b/meson.build
-index 79195c9..80d9564 100644
 --- a/meson.build
 +++ b/meson.build
 @@ -572,6 +572,7 @@ foreach ident : [
@@ -55,8 +53,6 @@ index 79195c9..80d9564 100644
  ]
  
  have = cc.has_function(ident[0], prefix : ident[1], args : 
'-D_GNU_SOURCE')
-diff --git a/src/backlight/backlight.c b/src/backlight/backlight.c
-index dfd6805..c2b2ace 100644
 --- a/src/backlight/backlight.c
 +++ b/src/backlight/backlight.c
 @@ -17,6 +17,7 @@
@@ -67,8 +63,6 @@ index dfd6805..c2b2ace 100644
  
  static int find_pci_or_platform_parent(sd_device *device, sd_device **ret) {
  const char *subsystem, *sysname, *value;
-diff --git a/src/basic/env-util.c b/src/basic/env-util.c
-index fd449dc..e2b0722 100644
 --- a/src/basic/env-util.c
 +++ b/src/basic/env-util.c
 @@ -16,6 +16,7 @@
@@ -79,8 +73,6 @@ index fd449dc..e2b0722 100644
  
  #define VALID_CHARS_ENV_NAME\
  DIGITS LETTERS  \
-diff --git a/src/basic/missing_stdlib.h b/src/basic/missing_stdlib.h
-index 188a8d4..1e16ec2 100644
 --- a/src/basic/missing_stdlib.h
 +++ b/src/basic/missing_stdlib.h
 @@ -11,3 +11,15 @@
@@ -99,8 +91,6 @@ index 188a8d4..1e16ec2 100644
 +(char *)memcpy(__new, __old, __len); \
 +  })
 +#endif
-diff --git a/src/basic/mkdir.c b/src/basic/mkdir.c
-index 6b82eab..51c6b78 100644
 --- a/src/basic/mkdir.c
 +++ b/src/basic/mkdir.c
 @@ -14,6 +14,7 @@
@@ -111,8 +101,6 @@ index 6b82eab..51c6b78 100644
  
  int mkdir_safe_internal(const char *path, mode_t mode, uid_t uid, gid_t gid, 
MkdirFlags flags, mkdir_func_t _mkdir) {
  struct stat st;
-diff --git a/src/basic/parse-util.c b/src/basic/parse-util.c
-index 87724af..d9c53bc 100644
 --- a/src/basic/parse-util.c
 +++ b/src/basic/parse-util.c
 @@ -19,6 +19,7 @@
@@ -123,8 +111,6 @@ index 87724af..d9c53bc 100644
  
  int parse_boolean(const char *v) {
  if (!v)
-diff --git a/src/basic/proc-cmdline.c b/src/basic/proc-cmdline.c
-index 1670001..b51feaa 100644
 --- a/src/basic/proc-cmdline.c
 +++ b/src/basic/proc-cmdline.c
 @@ -15,6 +15,7 @@
@@ -135,8 +121,6 @@ index 1670001..b51feaa 100644
  
  int proc_cmdline(char **ret) {
  const char *e;
-diff --git a/src/basic/procfs-util.c b/src/basic/procfs-util.c
-index 7aaf95b..42ce53d 100644
 --- a/src/basic/procfs-util.c
 +++ b/src/basic/procfs-util.c
 @@ -11,6 +11,7 @@
@@ -147,8 +131,6 @@ index 7aaf95b..42ce53d 100644
  
  int procfs_tasks_get_limit(uint64_t *ret) {
  _cleanup_free_ char *value = NULL;
-diff --git a/src/basic/time-util.c b/src/basic/time-util.c
-index daf952b..374b97f 100644
 --- a/src/basic/time-util.c
 +++ b/src/basic/time-util.c
 @@ -28,6 +28,7 @@
@@ -159,8 +141,6 @@ index daf952b..374b97f 100644
  
  static clockid_t map_clock_id(clockid_t c) {
  
-diff --git a/src/core/dbus-cgroup.c b/src/core/dbus-cgroup.c
-index 4615aea..bc1364f 100644
 --- a/src/core/dbus-cgroup.c
 +++ b/src/core/dbus-cgroup.c
 @@ -15,6 +15,7 @@
@@ -171,8 +151,6 @@ index 4615aea..bc1364f 100644
  
  static BUS_DEFINE_PROPERTY_GET_ENUM(property_get_cgroup_device_policy, 
cgroup_device_policy, CGroupDevicePolicy);
  
-diff --git a/src/core/dbus-util.c b/src/core/dbus-util.c
-index f4fbb72..0a1e3b5 100644
 --- a/src/core/dbus-util.c
 +++ b/src/core/dbus-util.c
 @@ -7,6 +7,7 @@
@@ -183,8 +161,6 @@ index f4fbb72..0a1e3b5 100644
  
  int bus_property_get_triggered_unit(
  sd_bus *bus,
-diff --git a/src/core/kmod-setup.c b/src/core/kmod-setup.c
-index a91cfeb..a459610 100644
 --- a/src/core/kmod-setup.c
 +++ b/src/core/kmod-setup.c
 @@ -11,6 +11,7 @@
@@ -195,8 +171,6 @@ index a91cfeb..a459610 100644
  
  #if HAVE_KMOD
  #include 
-diff --git a/src/core/service.c b/src/core/service.c
-index 0289990..0e725b5 100644
 --- a/src/core/service.c
 +++ b/src/core/service.c
 @@ -42,6 +42,7 @@
@@ -207,8 +181,6 @@ index 0289990..0e725b5 100644
  
  

[OE-core] [PATCH v3 2/4] musl-obstack: Add recipe

[Khem Raj]

obstacks from GNUlib is used in some OE-Core packages e.g. elfutils
and other packages outside OE-Core, this recipe helps provide this
functionality standalone on musl systems, and helps in getting full
versions of dependent packages

Signed-off-by: Khem Raj 
---
v3: Rebased

 meta/recipes-core/musl/musl-obstack.bb | 22 ++
 1 file changed, 22 insertions(+)
 create mode 100644 meta/recipes-core/musl/musl-obstack.bb

diff --git a/meta/recipes-core/musl/musl-obstack.bb 
b/meta/recipes-core/musl/musl-obstack.bb
new file mode 100644
index 00..3003935fe5
--- /dev/null
+++ b/meta/recipes-core/musl/musl-obstack.bb
@@ -0,0 +1,22 @@
+# Copyright (C) 2019 Khem Raj 
+# Released under the MIT license (see COPYING.MIT for the terms)
+
+SUMMARY = "A standalone library to implement GNU libc's obstack"
+DESCRIPTION = "copy + paste of the obstack functions and macros found in GNU 
gcc libiberty library for use with musl libc"
+HOMEPAGE = "https://github.com/pullmoll/musl-obstack;
+LICENSE = "GPL-2.0+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=3d23e4eef8243efcaab6f0a438078932"
+SECTION = "libs"
+
+PV = "1.1"
+SRCREV = "d2ad66b0df44a4b784956f7f7f2717131ddc05f4"
+SRC_URI = "git://github.com/pullmoll/musl-obstack"
+
+UPSTREAM_CHECK_COMMITS = "1"
+
+inherit autotools pkgconfig
+
+S = "${WORKDIR}/git"
+
+COMPATIBLE_HOST = ".*-musl.*"
+
-- 
2.21.0

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

[OE-core] [PATCH v2 3/4] elfutils: Fix eu-* utils builds for musl

[Khem Raj]

Re-organize the musl patches in three different areas namely
libs, utils and tests, this will help maintain them in future
version bumps

Add obstack dependency on musl targets which is needed for eu-*

PN and PN-binutils is not empty anymore on musl

Signed-off-by: Khem Raj 
Signed-off-by: Richard Purdie 
---
v2: Make FNM_EXTMATCH 0

 .../elfutils/elfutils_0.176.bb|  15 +-
 ...-err-variable-and-function-conflicts.patch |  91 
 ...ternatives-for-glibc-assumptions-hel.patch | 196 --
 .../elfutils/files/musl-libs.patch| 111 ++
 .../elfutils/files/musl-obstack-fts.patch | 105 ++
 .../files/{musl.patch => musl-tests.patch}|   0
 .../elfutils/files/musl-utils.patch   | 136 
 7 files changed, 359 insertions(+), 295 deletions(-)
 delete mode 100644 
meta/recipes-devtools/elfutils/files/0001-fix-err-variable-and-function-conflicts.patch
 delete mode 100644 
meta/recipes-devtools/elfutils/files/0008-build-Provide-alternatives-for-glibc-assumptions-hel.patch
 create mode 100644 meta/recipes-devtools/elfutils/files/musl-libs.patch
 create mode 100644 meta/recipes-devtools/elfutils/files/musl-obstack-fts.patch
 rename meta/recipes-devtools/elfutils/files/{musl.patch => musl-tests.patch} 
(100%)
 create mode 100644 meta/recipes-devtools/elfutils/files/musl-utils.patch

diff --git a/meta/recipes-devtools/elfutils/elfutils_0.176.bb 
b/meta/recipes-devtools/elfutils/elfutils_0.176.bb
index 85ff5f38d2..397723166c 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.176.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.176.bb
@@ -4,7 +4,7 @@ SECTION = "base"
 LICENSE = "GPLv2 & LGPLv3+ & GPLv3+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
 DEPENDS = "libtool bzip2 zlib virtual/libintl"
-DEPENDS_append_libc-musl = " argp-standalone fts "
+DEPENDS_append_libc-musl = " argp-standalone fts musl-obstack "
 # The Debian patches below are from:
 # 
http://ftp.de.debian.org/debian/pool/main/e/elfutils/elfutils_0.175-1.debian.tar.xz
 SRC_URI = "https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \
@@ -31,12 +31,13 @@ SRC_URI = 
"https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \
file://0001-skip-the-test-when-gcc-not-deployed.patch \
file://run-ptest \
file://ptest.patch \
-   file://musl.patch \
"
-SRC_URI_append_libc-musl = " 
file://0008-build-Provide-alternatives-for-glibc-assumptions-hel.patch \
- 
file://0001-fix-err-variable-and-function-conflicts.patch \
-"
-
+SRC_URI_append_libc-musl = " \
+   file://musl-obstack-fts.patch \
+   file://musl-libs.patch \
+   file://musl-utils.patch \
+   file://musl-tests.patch \
+   "
 SRC_URI[md5sum] = "077e4f49320cad82bf17a997068b1db9"
 SRC_URI[sha256sum] = 
"eb5747c371b0af0f71e86215a5ebb88728533c3a104a43d4231963f308cd1023"
 
@@ -75,8 +76,6 @@ do_install_ptest() {
 EXTRA_OEMAKE_class-native = ""
 EXTRA_OEMAKE_class-nativesdk = ""
 
-ALLOW_EMPTY_${PN}_libc-musl = "1"
-
 BBCLASSEXTEND = "native nativesdk"
 
 # Package utilities separately
diff --git 
a/meta/recipes-devtools/elfutils/files/0001-fix-err-variable-and-function-conflicts.patch
 
b/meta/recipes-devtools/elfutils/files/0001-fix-err-variable-and-function-conflicts.patch
deleted file mode 100644
index 433db133b2..00
--- 
a/meta/recipes-devtools/elfutils/files/0001-fix-err-variable-and-function-conflicts.patch
+++ /dev/null
@@ -1,91 +0,0 @@
-From 2c50fe7068bd6911958c6d851aef88179e73bb21 Mon Sep 17 00:00:00 2001
-From: Mingli Yu 
-Date: Tue, 16 Apr 2019 15:30:38 +0800
-Subject: [PATCH] fix err variable and function conflicts
-
-There comes below build failure with musl when
-ptest enabled.
-| In file included from ../../elfutils-0.176/tests/dwfl-proc-attach.c:33:
-| ../../elfutils-0.176/lib/system.h:63:35: error: called object 'err' is not a 
function or function pointer
-|  #define error(status, errno, ...) err(status, __VA_ARGS__)
-|^~~
-| ../../elfutils-0.176/tests/dwfl-proc-attach.c:92:5: note: in expansion of 
macro 'error'
-|  error (-1, 0, "dwfl_linux_proc_attach pid %d: %s", pid,
-|  ^
-| ../../elfutils-0.176/tests/dwfl-proc-attach.c:79:7: note: declared here
-|int err;
-|^~~
-
-It is because there is no error.h in musl and
-the patch 0008-build-Provide-alternatives-for-glibc-assumptions-hel.patch
-has updated to use err.h to replace error.h
-and also added macro definiton as below when
-use musl.
- #define error(status, errno, ...) err(status, __VA_ARGS__)
-
-And in err.h, there is below logic:
-_Noreturn void err(int, const char *, ...);
-
-But when ptest enabled, there comes below error
-as there is both variable and function defined
-to be err in tests/dwfl-proc-attach.c.
-So change the err variable's name to workaround
-the build failure with musl.
-
-Upstream-Status: Inappropriate [workaround 

Re: [OE-core] [PATCH 3/3] elfutils: Fix eu-* utils builds for musl

[Khem Raj]

On Tue, May 21, 2019 at 11:49 PM Adrian Bunk  wrote:
>
> On Tue, May 21, 2019 at 06:32:28PM -0700, Khem Raj wrote:
> >...
> > PN and PN-binutils is not empty anymore on musl
> >...
> > ++#if !defined(FNM_EXTMATCH)
> > ++# define FNM_EXTMATCH (1 << 5) /* Use ksh-like extended matching. */
> > ++#endif
> >...
>
> This might fix the compilation, but it is horribly wrong.
>
> It passes a flag to a function that this function does not understand.
>
> In the best case the function returns with an error on unknown flags.
>
> In the worst case the function does silently something different from
> what the caller expects it to do.
>

I forgot that the API is provided by libc here so mimic'ing the define
infact could
cause pain. I think its better to nullify it in this case where it is
being used in these
files.


> cu
> Adrian
>
> --
>
>"Is there not promise of rain?" Ling Tan asked suddenly out
> of the darkness. There had been need of rain for many days.
>"Only a promise," Lao Er said.
>Pearl S. Buck - Dragon Seed
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] bc: use u-a for bc as well

[Martin Jansa]

* bc can be provided by busybox as well (e.g. if you have your own
  defconfig and forget to explicitly disable it:
  ...
  *
  * Miscellaneous Utilities
  *
  adjtimex (4.7 kb) (ADJTIMEX) [N/y/?] n
  bbconfig (9.7 kb) (BBCONFIG) [N/y/?] n
  bc (45 kb) (BC) [Y/n/?] (NEW) dc (36 kb) (DC) [Y/n/?] y
Use bc code base for dc (larger, more features) (FEATURE_DC_BIG) [Y] (NEW) y
  Interactive mode (+4kb) (FEATURE_BC_INTERACTIVE) [Y/n/?] (NEW) Enable 
bc/dc long options (FEATURE_BC_LONG_OPTIONS) [Y/n] (NEW) beep (2.4 kb) (BEEP) 
[N/y/?] n
  chat (6.3 kb) (CHAT) [N/y/?] n
  conspy (10 kb) (CONSPY) [N/y/?] n
  ...
  ), causing conflict in u-a:

  update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid 
since /usr/bin/bc exists and is not a link

  and then whole do_rootfs or do_populate_sdk to fail because busybox postinst 
is failing:

  do_populate_sdk: Postinstall scriptlets of ['busybox'] have failed. If the 
intention is to defer them to first boot,
  then please place them into pkg_postinst_ontarget_${PN} (). Deferring to 
first boot via 'exit 1' is no longer supported.

Signed-off-by: Martin Jansa 
---
 meta/recipes-extended/bc/bc_1.07.1.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-extended/bc/bc_1.07.1.bb 
b/meta/recipes-extended/bc/bc_1.07.1.bb
index e80857745e..809b864c1a 100644
--- a/meta/recipes-extended/bc/bc_1.07.1.bb
+++ b/meta/recipes-extended/bc/bc_1.07.1.bb
@@ -27,7 +27,7 @@ do_compile_prepend() {
 cp -f ${WORKDIR}/libmath.h ${B}/bc/libmath.h
 }
 
-ALTERNATIVE_${PN} = "dc"
+ALTERNATIVE_${PN} = "bc dc"
 ALTERNATIVE_PRIORITY = "100"
 
-BBCLASSEXTEND = "native"
\ No newline at end of file
+BBCLASSEXTEND = "native"
-- 
2.17.1

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

Re: [OE-core] ptest with BBCLASSEXTEND

[richard . purdie]

On Wed, 2019-05-22 at 18:28 -0400, Jonathan Rajotte-Julien wrote:
> Hi,
> 
> > I think Jonathan Rajotte might be able to help us with lttng-tools.
> 
> Finally had some time for this.
> 
> See [1][2].
> 
> [2] does bring the matter of forcing lttng-ust support of lttng-tools 
> when ptest
> is deployed. "lttng-ust" is already set in PACKAGECONFIG by default
> but is there any
> way to "enforce" the lttng-ust config & dependency when lttng-tools-
> ptest is used?
> 
> [1] 
> http://lists.openembedded.org/pipermail/openembedded-core/2019-May/282733.html
> [2] 
> http://lists.openembedded.org/pipermail/openembedded-core/2019-May/282734.html

Thanks for looking at this. It sounds like we need some runtime
dependencies as well as the PACKAGECONFIG option. I think the lttng-
tool ust option should be changed:

-PACKAGECONFIG[lttng-ust] = "--with-lttng-ust, --without-lttng-ust, lttng-ust"
+PACKAGECONFIG[lttng-ust] = "--with-lttng-ust, --without-lttng-ust, 
lttng-ust,lttng-ust"

i.e. force the dependency at runtime as well as buildtime.

Cheers,

Richard

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

Re: [OE-core] ptest with BBCLASSEXTEND

[Jonathan Rajotte-Julien]

Hi,

> I think Jonathan Rajotte might be able to help us with lttng-tools.

Finally had some time for this.

See [1][2].

[2] does bring the matter of forcing lttng-ust support of lttng-tools when ptest
is deployed. "lttng-ust" is already set in PACKAGECONFIG by default but is 
there any
way to "enforce" the lttng-ust config & dependency when lttng-tools-ptest is 
used?

[1] 
http://lists.openembedded.org/pipermail/openembedded-core/2019-May/282733.html
[2] 
http://lists.openembedded.org/pipermail/openembedded-core/2019-May/282734.html

Cheers
-- 
Jonathan Rajotte-Julien
EfficiOS
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH v2] insane: add sanity checks to SRC_URI

[Richard Purdie]

On Wed, 2019-05-22 at 17:48 +0200, Martin Jansa wrote:
> Can we add an option to skip this with INSANE_SKIP?
> 
> It looks like QARECIPETEST doesn't use INSANE_SKIP or I don't see
> how.
> 
> Removing src-uri-bad from ERROR_QA/WARN_QA for some recipes works as
> well, is it worth adding INSANE_SKIP for consistency with other
> checks or not?

Ultimately I'd say that all these checks should work with INSANE_SKIP.
I was just wondering about this issue myself for the chkconfig
autobuilder warnings...

Cheers,

Richard

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

[OE-core] [PATCH v2 2/2] lttng-tools: add lttng-modules to ptest dependencies

[Jonathan Rajotte]

The lttng-tools project is essentially a "tracer" controller, the tests
depends heavily on lttng-ust and lttng-modules presence.

Signed-off-by: Jonathan Rajotte 
---
 meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb 
b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
index f1bb7224f3..7e80bb45d1 100644
--- a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
+++ b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
@@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = 
"file://LICENSE;md5=01d7fc4496aacf37d90df90b90b0cac1 \
 
 DEPENDS = "liburcu popt libxml2 util-linux"
 RDEPENDS_${PN} = "libgcc"
-RDEPENDS_${PN}-ptest += "make perl bash gawk ${PN} babeltrace procps 
perl-module-overloading coreutils util-linux kmod"
+RDEPENDS_${PN}-ptest += "make perl bash gawk ${PN} babeltrace procps 
perl-module-overloading coreutils util-linux kmod lttng-modules"
 RDEPENDS_${PN}-ptest_append_libc-glibc = " glibc-utils"
 RDEPENDS_${PN}-ptest_append_libc-musl = " musl-utils"
 # babelstats.pl wants getopt-long
-- 
2.17.1

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

[OE-core] [PATCH v2 1/2] lttng-tools: prevent test timeout when lttng-modules is not present

[Jonathan Rajotte]

Signed-off-by: Jonathan Rajotte 
---
 ...ttng-modules-presence-before-testing.patch | 26 +++
 .../lttng/lttng-tools_2.10.6.bb   |  1 +
 2 files changed, 27 insertions(+)
 create mode 100644 
meta/recipes-kernel/lttng/lttng-tools/0008-Fix-check-for-lttng-modules-presence-before-testing.patch

diff --git 
a/meta/recipes-kernel/lttng/lttng-tools/0008-Fix-check-for-lttng-modules-presence-before-testing.patch
 
b/meta/recipes-kernel/lttng/lttng-tools/0008-Fix-check-for-lttng-modules-presence-before-testing.patch
new file mode 100644
index 00..04a773e8c5
--- /dev/null
+++ 
b/meta/recipes-kernel/lttng/lttng-tools/0008-Fix-check-for-lttng-modules-presence-before-testing.patch
@@ -0,0 +1,26 @@
+From 58e4dcce48b48b68b65bffc0cd51d9e26b44c75d Mon Sep 17 00:00:00 2001
+From: Jonathan Rajotte 
+Date: Wed, 22 May 2019 16:44:54 -0400
+Subject: [PATCH] Fix: check for lttng modules presence before testing
+
+Upstream-status: Submitted [1]
+
+[1] https://lists.lttng.org/pipermail/lttng-dev/2019-May/028987.html
+
+Signed-off-by: Jonathan Rajotte 
+---
+ tests/regression/tools/notification/test_notification_multi_app | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/tests/regression/tools/notification/test_notification_multi_app 
b/tests/regression/tools/notification/test_notification_multi_app
+index 29b0f62..a6baf65 100755
+--- a/tests/regression/tools/notification/test_notification_multi_app
 b/tests/regression/tools/notification/test_notification_multi_app
+@@ -484,6 +484,7 @@ else
+ fi
+ 
+ if [ "$(id -u)" == "0" ]; then
++  validate_lttng_modules_present
+   TESTS+=(
+   test_multi_app_kernel
+   test_on_register_evaluation_kernel
diff --git a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb 
b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
index a3fabb20ec..f1bb7224f3 100644
--- a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
+++ b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
@@ -39,6 +39,7 @@ SRC_URI = 
"https://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \

file://0005-Tests-use-modprobe-to-test-for-the-presence-of-lttng.patch \
file://0006-Tests-check-for-lttng-modules-presence.patch \
file://0007-Fix-getgrnam-is-not-MT-Safe-use-getgrnam_r.patch \
+   
file://0008-Fix-check-for-lttng-modules-presence-before-testing.patch \
"
 
 SRC_URI[md5sum] = "e88c521b5da6bb48a8187af66ecc"
-- 
2.17.1

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

Re: [OE-core] [PATCH 1/2] lttng-tools: prevent test timeout when lttng-modules is not present

[Jonathan Rajotte-Julien]

Clearly the actual patch is missing, sending v2 shortly.


On Wed, May 22, 2019 at 10:06:44PM +, Jonathan Rajotte wrote:
> Signed-off-by: Jonathan Rajotte 
> ---
>  meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb 
> b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
> index a3fabb20ec..f1bb7224f3 100644
> --- a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
> +++ b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
> @@ -39,6 +39,7 @@ SRC_URI = 
> "https://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
> 
> file://0005-Tests-use-modprobe-to-test-for-the-presence-of-lttng.patch \
> file://0006-Tests-check-for-lttng-modules-presence.patch \
> file://0007-Fix-getgrnam-is-not-MT-Safe-use-getgrnam_r.patch \
> +   
> file://0008-Fix-check-for-lttng-modules-presence-before-testing.patch \
> "
>  
>  SRC_URI[md5sum] = "e88c521b5da6bb48a8187af66ecc"
> -- 
> 2.17.1
> 

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

[OE-core] [PATCH 2/2] lttng-tools: add lttng-modules to ptest dependencies

[Jonathan Rajotte]

The lttng-tools project is essentially a "tracer" controller, the tests
depends heavily on lttng-ust and lttng-modules presence.

Signed-off-by: Jonathan Rajotte 
---
 meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb 
b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
index f1bb7224f3..7e80bb45d1 100644
--- a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
+++ b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
@@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = 
"file://LICENSE;md5=01d7fc4496aacf37d90df90b90b0cac1 \
 
 DEPENDS = "liburcu popt libxml2 util-linux"
 RDEPENDS_${PN} = "libgcc"
-RDEPENDS_${PN}-ptest += "make perl bash gawk ${PN} babeltrace procps 
perl-module-overloading coreutils util-linux kmod"
+RDEPENDS_${PN}-ptest += "make perl bash gawk ${PN} babeltrace procps 
perl-module-overloading coreutils util-linux kmod lttng-modules"
 RDEPENDS_${PN}-ptest_append_libc-glibc = " glibc-utils"
 RDEPENDS_${PN}-ptest_append_libc-musl = " musl-utils"
 # babelstats.pl wants getopt-long
-- 
2.17.1

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

[OE-core] [PATCH 1/2] lttng-tools: prevent test timeout when lttng-modules is not present

[Jonathan Rajotte]

Signed-off-by: Jonathan Rajotte 
---
 meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb 
b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
index a3fabb20ec..f1bb7224f3 100644
--- a/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
+++ b/meta/recipes-kernel/lttng/lttng-tools_2.10.6.bb
@@ -39,6 +39,7 @@ SRC_URI = 
"https://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \

file://0005-Tests-use-modprobe-to-test-for-the-presence-of-lttng.patch \
file://0006-Tests-check-for-lttng-modules-presence.patch \
file://0007-Fix-getgrnam-is-not-MT-Safe-use-getgrnam_r.patch \
+   
file://0008-Fix-check-for-lttng-modules-presence-before-testing.patch \
"
 
 SRC_URI[md5sum] = "e88c521b5da6bb48a8187af66ecc"
-- 
2.17.1

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

Re: [OE-core] [PATCH] bluez5: remove udev dependency

[Adrian Bunk]

On Wed, May 22, 2019 at 02:55:41PM -0700, David Frey wrote:
> udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow
> users to decide if they want udev support.
> 
> Signed-off-by: David Frey 
> ---
>  meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc 
> b/meta/recipes-connectivity/bluez5/bluez5.inc
> index aaf2af975d..568de86d6d 100644
> --- a/meta/recipes-connectivity/bluez5/bluez5.inc
> +++ b/meta/recipes-connectivity/bluez5/bluez5.inc
> @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
>  file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \
>  
> file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e"
> -DEPENDS = "udev dbus-glib glib-2.0"
> +DEPENDS = "dbus-glib glib-2.0"
>  PROVIDES += "bluez-hcidump"
>  RPROVIDES_${PN} += "bluez-hcidump"
>  
> @@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = 
> "--enable-threads,--disable-threads"
>  PACKAGECONFIG[deprecated] = "--enable-deprecated,--disable-deprecated"
>  PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell"
>  PACKAGECONFIG[btpclient] = "--enable-btpclient,--disable-btpclient, ell"
> +PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev"
>...

Unless there is a good reason to do otherwise, this should be added to 
the default PACKAGECONFIG so that the set of features enabled by default 
stays the same.

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

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

Re: [OE-core] [PATCH] bluez5: remove udev dependency

[Joshua Watt]

On 5/22/19 4:55 PM, David Frey wrote:

udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow
users to decide if they want udev support.

Signed-off-by: David Frey 
---
  meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc 
b/meta/recipes-connectivity/bluez5/bluez5.inc
index aaf2af975d..568de86d6d 100644
--- a/meta/recipes-connectivity/bluez5/bluez5.inc
+++ b/meta/recipes-connectivity/bluez5/bluez5.inc
@@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+"
  LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
  file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \
  
file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e"
-DEPENDS = "udev dbus-glib glib-2.0"
+DEPENDS = "dbus-glib glib-2.0"
  PROVIDES += "bluez-hcidump"
  RPROVIDES_${PN} += "bluez-hcidump"


Should it be enabled by default?

  
@@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = "--enable-threads,--disable-threads"

  PACKAGECONFIG[deprecated] = "--enable-deprecated,--disable-deprecated"
  PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell"
  PACKAGECONFIG[btpclient] = "--enable-btpclient,--disable-btpclient, ell"
+PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev"
  
  SRC_URI = "\

  ${KERNELORG_MIRROR}/linux/bluetooth/bluez-${PV}.tar.xz \

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

[OE-core] [PATCH] bluez5: remove udev dependency

[David Frey]

udev is an optional dependency of bluez5, so use PACKAGECONFIG to allow
users to decide if they want udev support.

Signed-off-by: David Frey 
---
 meta/recipes-connectivity/bluez5/bluez5.inc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc 
b/meta/recipes-connectivity/bluez5/bluez5.inc
index aaf2af975d..568de86d6d 100644
--- a/meta/recipes-connectivity/bluez5/bluez5.inc
+++ b/meta/recipes-connectivity/bluez5/bluez5.inc
@@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
 file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \
 
file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e"
-DEPENDS = "udev dbus-glib glib-2.0"
+DEPENDS = "dbus-glib glib-2.0"
 PROVIDES += "bluez-hcidump"
 RPROVIDES_${PN} += "bluez-hcidump"
 
@@ -43,6 +43,7 @@ PACKAGECONFIG[threads] = "--enable-threads,--disable-threads"
 PACKAGECONFIG[deprecated] = "--enable-deprecated,--disable-deprecated"
 PACKAGECONFIG[mesh] = "--enable-mesh,--disable-mesh, json-c ell"
 PACKAGECONFIG[btpclient] = "--enable-btpclient,--disable-btpclient, ell"
+PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev"
 
 SRC_URI = "\
 ${KERNELORG_MIRROR}/linux/bluetooth/bluez-${PV}.tar.xz \
-- 
2.21.0

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

Re: [OE-core] [PATCH 1/1] util-linux: Add missing ptest dependencies

[Martin Jansa]

On Wed, May 22, 2019 at 10:09:16PM +0200, Martin Jansa wrote:
> On Wed, May 22, 2019 at 05:18:25PM +0300, Adrian Bunk wrote:
> > On Wed, May 22, 2019 at 01:03:37PM +0200, Martin Jansa wrote:
> > > This pulls bc to many images with busybox, causing busybox postinst to 
> > > fail
> > > with:
> > > 
> > > update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid
> > > since /usr/bin/bc exists and is not a link
> > 
> > Are you enabling CONFIG_BC in your busybox config?
> > It is not enabled in meta/recipes-core/busybox/busybox/defconfig
> 
> It's not disabled in my defconfig like it is in
> meta/recipes-core/busybox/busybox/defconfig, maybe it got auto enabled.
> 
> In log.do_configure I see:
> ...
> *
> * Miscellaneous Utilities
> *
> adjtimex (4.7 kb) (ADJTIMEX) [N/y/?] n
> bbconfig (9.7 kb) (BBCONFIG) [N/y/?] n
> bc (45 kb) (BC) [Y/n/?] (NEW) dc (36 kb) (DC) [Y/n/?] y
>   Use bc code base for dc (larger, more features) (FEATURE_DC_BIG) [Y] (NEW) y
> Interactive mode (+4kb) (FEATURE_BC_INTERACTIVE) [Y/n/?] (NEW) Enable 
> bc/dc long options (FEATURE_BC_LONG_OPTIONS) [Y/n] (NEW) beep (2.4 kb) (BEEP) 
> [N/y/?] n
> chat (6.3 kb) (CHAT) [N/y/?] n
> conspy (10 kb) (CONSPY) [N/y/?] n
> ...
> 
> > > I'll check which busybox defconfig entry enables BC and send patch to use
> > > u-a for bc in ./meta/recipes-extended/bc/bc_1.07.1.bb
> > > 
> > > There already is:
> > > ALTERNATIVE_${PN} = "dc"
> > > ALTERNATIVE_PRIORITY = "100"
> > > but maybe dc is a typo.
> > 
> > bc and dc are two different binaries built from bc.
> > 
> > busybox used to only have dc, bc was added 6 months ago.
> 
> Thanks I've never noticed dc before.
> 
> I see both dc and bc in 1.06 version from meta-gplv2 which was released
> on 2000-11-15
> 
> bc/1.06-r3/image/usr/bin/bc

Ah you were talking about busybox having only dc until 6 months ago.

Makes sense now, thanks.

> As there is already u-a for dc, I will sent u-a for bc as well.
> -- 
> Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com



-- 
Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com


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

Re: [OE-core] [PATCH 2/2] python-six: put python2/3 variant together

[Alejandro Hernandez]

On 5/22/2019 7:01 AM, Tim Orling wrote:



On Wed, May 22, 2019 at 4:27 AM Alexander Kanavin 
mailto:alex.kana...@gmail.com>> wrote:


On Wed, 22 May 2019 at 13:05, mailto:richard.pur...@linuxfoundation.org>> wrote:

> I don't think we need to switch our default, just be able to
build say
> oe-core without python/python2 in HOSTTOOLS and without needing
python
> 2.7 in OE-Core either.
>
> > Its good that project is taking lead here but we might leave many
> > users behind.
>
> I suspect python2 will end up in a layer of its down which I'm fine
> with, dependencies can then be clearly seen.

For what it's worth, Fedora is getting serious about Python 2 removal,
and they have a plan for it:
https://fedoraproject.org/wiki/Changes/RetirePython2

"All packages in Fedora that need Python 2 to run will be removed from
Fedora 32 regardless of their dependencies. All packages in Fedora
that need Python 2 to build will be removed from Fedora 32 regardless
of their dependencies. "



We have discussed moving meta-python to its own repository (not under 
meta-openembedded). As part of that move, I am proposing the existing 
python2 recipes move to a separate (deprecated) layer, as RP mentioned 
earlier. Then meta-python will become python3 only. We will also move 
recipetool/devtool to provide python3 (only) support. People that need 
python2 will very soon have to live with older stable branches. This 
should not be a surprise.


We also need to do whatever we can with upstream projects that are not 
feeling an urgency to move to python3. They should likewise not be 
surprised at the need to switch.



I would agree with this as well.


Cheers,

Alejandro





Alex
-- 
___

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 1/1] util-linux: Add missing ptest dependencies

[Martin Jansa]

On Wed, May 22, 2019 at 05:18:25PM +0300, Adrian Bunk wrote:
> On Wed, May 22, 2019 at 01:03:37PM +0200, Martin Jansa wrote:
> > This pulls bc to many images with busybox, causing busybox postinst to fail
> > with:
> > 
> > update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid
> > since /usr/bin/bc exists and is not a link
> 
> Are you enabling CONFIG_BC in your busybox config?
> It is not enabled in meta/recipes-core/busybox/busybox/defconfig

It's not disabled in my defconfig like it is in
meta/recipes-core/busybox/busybox/defconfig, maybe it got auto enabled.

In log.do_configure I see:
...
*
* Miscellaneous Utilities
*
adjtimex (4.7 kb) (ADJTIMEX) [N/y/?] n
bbconfig (9.7 kb) (BBCONFIG) [N/y/?] n
bc (45 kb) (BC) [Y/n/?] (NEW) dc (36 kb) (DC) [Y/n/?] y
  Use bc code base for dc (larger, more features) (FEATURE_DC_BIG) [Y] (NEW) y
Interactive mode (+4kb) (FEATURE_BC_INTERACTIVE) [Y/n/?] (NEW) Enable 
bc/dc long options (FEATURE_BC_LONG_OPTIONS) [Y/n] (NEW) beep (2.4 kb) (BEEP) 
[N/y/?] n
chat (6.3 kb) (CHAT) [N/y/?] n
conspy (10 kb) (CONSPY) [N/y/?] n
...

> > I'll check which busybox defconfig entry enables BC and send patch to use
> > u-a for bc in ./meta/recipes-extended/bc/bc_1.07.1.bb
> > 
> > There already is:
> > ALTERNATIVE_${PN} = "dc"
> > ALTERNATIVE_PRIORITY = "100"
> > but maybe dc is a typo.
> 
> bc and dc are two different binaries built from bc.
> 
> busybox used to only have dc, bc was added 6 months ago.

Thanks I've never noticed dc before.

I see both dc and bc in 1.06 version from meta-gplv2 which was released
on 2000-11-15

bc/1.06-r3/image/usr/bin/bc

As there is already u-a for dc, I will sent u-a for bc as well.
-- 
Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com


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

[OE-core] [PATCH V2] mesa: Fix a case when gbm is enabled but DRIDRIVERS is not defined

[Khem Raj]

This is a typical scenario with arm platforms, with meson if DRIDRIVERS
is empty then dri is disabled automatically, but gbm needs dri2 enabled
for it to have proper backend, in autotools we could use --enable-gbm
--enable-dri and get correct behaviour but with meson enabling dri2 only
works if DRIDRIVERS is not empty

This fixes mesa-gl build for rpi machines when using userland graphics

Signed-off-by: Khem Raj 
Signed-off-by: Andrei Gherzan 
---
v2: Add explicit meson knob to enable dri

 ...Allow-enable-DRI-without-DRI-drivers.patch | 45 +++
 meta/recipes-graphics/mesa/mesa.inc   |  2 +-
 meta/recipes-graphics/mesa/mesa_19.0.3.bb |  1 +
 3 files changed, 47 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-graphics/mesa/files/0001-Allow-enable-DRI-without-DRI-drivers.patch

diff --git 
a/meta/recipes-graphics/mesa/files/0001-Allow-enable-DRI-without-DRI-drivers.patch
 
b/meta/recipes-graphics/mesa/files/0001-Allow-enable-DRI-without-DRI-drivers.patch
new file mode 100644
index 00..52d670a296
--- /dev/null
+++ 
b/meta/recipes-graphics/mesa/files/0001-Allow-enable-DRI-without-DRI-drivers.patch
@@ -0,0 +1,45 @@
+From 8e0a58c8eea7ed70071cac139655700fdfa16445 Mon Sep 17 00:00:00 2001
+From: Andrei Gherzan 
+Date: Wed, 22 May 2019 18:32:07 +0100
+Subject: [PATCH] Allow enable DRI without DRI drivers
+
+Upstream-status: Pending
+Signed-off-by: Andrei Gherzan 
+---
+ meson.build   | 2 +-
+ meson_options.txt | 6 ++
+ 2 files changed, 7 insertions(+), 1 deletion(-)
+
+diff --git a/meson.build b/meson.build
+index 53d02e3..9a8578f 100644
+--- a/meson.build
 b/meson.build
+@@ -119,7 +119,7 @@ with_dri_r200 = _drivers.contains('r200')
+ with_dri_nouveau = _drivers.contains('nouveau')
+ with_dri_swrast = _drivers.contains('swrast')
+ 
+-with_dri = _drivers.length() != 0 and _drivers != ['']
++with_dri = get_option('dri') or (_drivers.length() != 0 and _drivers != [''])
+ 
+ _drivers = get_option('gallium-drivers')
+ if _drivers.contains('auto')
+diff --git a/meson_options.txt b/meson_options.txt
+index ccf7065..5174320 100644
+--- a/meson_options.txt
 b/meson_options.txt
+@@ -34,6 +34,12 @@ option(
+   choices : ['auto', 'true', 'false'],
+   description : 'enable support for dri3'
+ )
++option(
++  'dri',
++  type : 'boolean',
++  value : false,
++  description : 'enable support for dri'
++)
+ option(
+   'dri-drivers',
+   type : 'array',
+-- 
+2.17.1
+
diff --git a/meta/recipes-graphics/mesa/mesa.inc 
b/meta/recipes-graphics/mesa/mesa.inc
index cd3ddfd822..3ecfb8506c 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -70,7 +70,7 @@ DRIDRIVERS_class-nativesdk = "swrast"
 DRIDRIVERS_append_x86_class-target = ",r100,r200,nouveau,i965,i915"
 DRIDRIVERS_append_x86-64_class-target = ",r100,r200,nouveau,i965,i915"
 # "dri" requires "opengl"
-PACKAGECONFIG[dri] = "-Ddri-drivers=${DRIDRIVERS}, -Ddri-drivers='', xorgproto 
libdrm"
+PACKAGECONFIG[dri] = "-Ddri=true -Ddri-drivers=${DRIDRIVERS}, -Ddri=false 
-Ddri-drivers='', xorgproto libdrm"
 PACKAGECONFIG[dri3] = "-Ddri3=true, -Ddri3=false, xorgproto libxshmfence"
 
 # Vulkan drivers need dri3 enabled
diff --git a/meta/recipes-graphics/mesa/mesa_19.0.3.bb 
b/meta/recipes-graphics/mesa/mesa_19.0.3.bb
index 36faa4a99d..8a72df1014 100644
--- a/meta/recipes-graphics/mesa/mesa_19.0.3.bb
+++ b/meta/recipes-graphics/mesa/mesa_19.0.3.bb
@@ -3,6 +3,7 @@ require ${BPN}.inc
 SRC_URI = "https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \

file://0001-meson.build-check-for-all-linux-host_os-combinations.patch \
file://0001-meson.build-make-TLS-GLX-optional-again.patch \
+   file://0001-Allow-enable-DRI-without-DRI-drivers.patch \
"
 
 SRC_URI[md5sum] = "d03bf14e42c0e54ebae5730712ccc408"
-- 
2.21.0

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

[OE-core] [PATCH] m4/tar: Remove remove-gets.patch

[Adrian Bunk]

When the original problem was fixed in gnulib the
patches were rebased on top of the upstream fix...

Signed-off-by: Adrian Bunk 
---
 meta/recipes-devtools/m4/m4-1.4.18.inc|  1 -
 meta/recipes-devtools/m4/m4/remove-gets.patch | 24 ---
 .../tar/tar/remove-gets.patch | 29 ---
 meta/recipes-extended/tar/tar_1.32.bb |  1 -
 4 files changed, 55 deletions(-)
 delete mode 100644 meta/recipes-devtools/m4/m4/remove-gets.patch
 delete mode 100644 meta/recipes-extended/tar/tar/remove-gets.patch

diff --git a/meta/recipes-devtools/m4/m4-1.4.18.inc 
b/meta/recipes-devtools/m4/m4-1.4.18.inc
index 81ed65edd9..a9b63c1bf6 100644
--- a/meta/recipes-devtools/m4/m4-1.4.18.inc
+++ b/meta/recipes-devtools/m4/m4-1.4.18.inc
@@ -8,7 +8,6 @@ inherit autotools texinfo ptest
 
 SRC_URI = "${GNU_MIRROR}/m4/m4-${PV}.tar.gz \
file://ac_config_links.patch \
-   file://remove-gets.patch \
file://m4-1.4.18-glibc-change-work-around.patch \
"
 SRC_URI_append_class-target = " 
file://0001-Unset-need_charset_alias-when-building-for-musl.patch \
diff --git a/meta/recipes-devtools/m4/m4/remove-gets.patch 
b/meta/recipes-devtools/m4/m4/remove-gets.patch
deleted file mode 100644
index abe82f3b2d..00
--- a/meta/recipes-devtools/m4/m4/remove-gets.patch
+++ /dev/null
@@ -1,24 +0,0 @@
-eglibc has remove gets starting 2.16
-therefore check for its being there before 
-undefining it.
-
-Upstream-Status: Pending
-Signed-off-by: Khem Raj 

-diff --git a/lib/stdio.in.h b/lib/stdio.in.h
-index 5727452..f04a691 100644
 a/lib/stdio.in.h
-+++ b/lib/stdio.in.h
-@@ -742,10 +742,12 @@ _GL_WARN_ON_USE (getline, "getline is unportable - "
- /* It is very rare that the developer ever has full control of stdin,
-so any use of gets warrants an unconditional warning; besides, C11
-removed it.  */
-+#if defined gets
- #undef gets
- #if HAVE_RAW_DECL_GETS && !defined __cplusplus
- _GL_WARN_ON_USE (gets, "gets is a security hole - use fgets instead");
- #endif
-+#endif
- 
- #if @GNULIB_OBSTACK_PRINTF@ || @GNULIB_OBSTACK_PRINTF_POSIX@
- struct obstack;
diff --git a/meta/recipes-extended/tar/tar/remove-gets.patch 
b/meta/recipes-extended/tar/tar/remove-gets.patch
deleted file mode 100644
index f24de926a3..00
--- a/meta/recipes-extended/tar/tar/remove-gets.patch
+++ /dev/null
@@ -1,29 +0,0 @@
-ISO C11 removes the specification of gets() from the C language, eglibc 2.16+ 
removed it
-
-Signed-off-by: Khem Raj 
-
-Upstream-Status: Pending

- gnu/stdio.in.h | 2 ++
- 1 file changed, 2 insertions(+)
-
-diff --git a/gnu/stdio.in.h b/gnu/stdio.in.h
-index ec43874..502e3ae 100644
 a/gnu/stdio.in.h
-+++ b/gnu/stdio.in.h
-@@ -722,10 +722,12 @@ _GL_WARN_ON_USE (getline, "getline is unportable - "
- /* It is very rare that the developer ever has full control of stdin,
-so any use of gets warrants an unconditional warning; besides, C11
-removed it.  */
-+#if defined gets
- #undef gets
- #if HAVE_RAW_DECL_GETS && !defined __cplusplus
- _GL_WARN_ON_USE (gets, "gets is a security hole - use fgets instead");
- #endif
-+#endif
- 
- #if @GNULIB_OBSTACK_PRINTF@ || @GNULIB_OBSTACK_PRINTF_POSIX@
- struct obstack;
--- 
-2.8.3
-
diff --git a/meta/recipes-extended/tar/tar_1.32.bb 
b/meta/recipes-extended/tar/tar_1.32.bb
index 7240fdb7e1..18f09b5711 100644
--- a/meta/recipes-extended/tar/tar_1.32.bb
+++ b/meta/recipes-extended/tar/tar_1.32.bb
@@ -7,7 +7,6 @@ LICENSE = "GPLv3"
 LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
 
 SRC_URI = "${GNU_MIRROR}/tar/tar-${PV}.tar.bz2 \
-   file://remove-gets.patch \
file://musl_dirent.patch \
 "
 
-- 
2.17.1

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

[OE-core] [PATCH] mesa: Fix a case when gbm is enabled but DRIDRIVERS is not defined

[Khem Raj]

This is a typical scenario with arm platforms, with meson if DRIDRIVERS
is empty then dri is disabled automatically, but gbm needs dri2 enabled
for it to have proper backend, in autotools we could use --enable-gbm
--enable-dri and get correct behaviour but with meson enabling dri2 only
works if DRIDRIVERS is not empty

This fixes mesa-gl build for rpi machines when using userland graphics

Signed-off-by: Khem Raj 
---
 .../mesa/files/dri-drivers-auto.patch | 31 +++
 meta/recipes-graphics/mesa/mesa_19.0.3.bb |  1 +
 2 files changed, 32 insertions(+)
 create mode 100644 meta/recipes-graphics/mesa/files/dri-drivers-auto.patch

diff --git a/meta/recipes-graphics/mesa/files/dri-drivers-auto.patch 
b/meta/recipes-graphics/mesa/files/dri-drivers-auto.patch
new file mode 100644
index 00..532b2370b3
--- /dev/null
+++ b/meta/recipes-graphics/mesa/files/dri-drivers-auto.patch
@@ -0,0 +1,31 @@
+Fix meson regression for --enable-dri --enable-gbm combo equivalent
+
+For arm if DRIDRIVERS is set to 'auto' then respect that
+to enable dri support, this is required for devices where
+we use mesa to provide gl/libgbm implementation and rest
+comes from some proprietary driver, typical case is userland
+graphics driver for raspberrypi devices.
+
+This would mean that when we set 
+
+DRIDRIVERS_rpi = ",auto"
+
+Then the meson system will build the dri with gbm backend
+as was the case with autotools where --enable-dri did the
+same thing
+
+Upstream-Status: Inappropriate [Might be specific to arm systems]
+
+Signed-off-by: Khem Raj 
+
+--- a/meson.build
 b/meson.build
+@@ -98,7 +98,7 @@ if _drivers.contains('auto')
+ if ['x86', 'x86_64'].contains(host_machine.cpu_family())
+   _drivers = ['i915', 'i965', 'r100', 'r200', 'nouveau']
+ elif ['arm', 'aarch64'].contains(host_machine.cpu_family())
+-  _drivers = []
++  _drivers = ['auto']
+ else
+   error('Unknown architecture @0@. Please pass -Ddri-drivers to set 
driver options. Patches gladly accepted to fix this.'.format(
+ host_machine.cpu_family()))
diff --git a/meta/recipes-graphics/mesa/mesa_19.0.3.bb 
b/meta/recipes-graphics/mesa/mesa_19.0.3.bb
index 36faa4a99d..c1703e04d2 100644
--- a/meta/recipes-graphics/mesa/mesa_19.0.3.bb
+++ b/meta/recipes-graphics/mesa/mesa_19.0.3.bb
@@ -3,6 +3,7 @@ require ${BPN}.inc
 SRC_URI = "https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \

file://0001-meson.build-check-for-all-linux-host_os-combinations.patch \
file://0001-meson.build-make-TLS-GLX-optional-again.patch \
+   file://dri-drivers-auto.patch \
"
 
 SRC_URI[md5sum] = "d03bf14e42c0e54ebae5730712ccc408"
-- 
2.21.0

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

Re: [OE-core] [PATCH v2] insane: add sanity checks to SRC_URI

[Martin Jansa]

Can we add an option to skip this with INSANE_SKIP?

It looks like QARECIPETEST doesn't use INSANE_SKIP or I don't see how.

Removing src-uri-bad from ERROR_QA/WARN_QA for some recipes works as well,
is it worth adding INSANE_SKIP for consistency with other checks or not?


On Sat, May 18, 2019 at 1:37 AM Ross Burton  wrote:

> The SRC_URI almost definitely shouldn't be using ${PN}, and GitHub
> */archive/*
> tarballs are dynamically generated so the checksums will change over time.
>
> Detect both of these, and emit a QA warning if found.
>
> Signed-off-by: Ross Burton 
> ---
>  meta/classes/insane.bbclass | 13 -
>  1 file changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
> index 9ca5aefe544..59bb8be5470 100644
> --- a/meta/classes/insane.bbclass
> +++ b/meta/classes/insane.bbclass
> @@ -25,7 +25,7 @@ QA_SANE = "True"
>  WARN_QA ?= "ldflags useless-rpaths rpaths staticdev libdir
> xorg-driver-abi \
>  textrel already-stripped incompatible-license files-invalid \
>  installed-vs-shipped compile-host-path install-host-path \
> -pn-overrides infodir build-deps \
> +pn-overrides infodir build-deps src-uri-bad \
>  unknown-configure-option symlink-to-sysroot multilib \
>  invalid-packageconfig host-user-contaminated uppercase-pn
> patch-fuzz \
>  "
> @@ -898,6 +898,17 @@ def package_qa_check_host_user(path, name, d, elf,
> messages):
>  return False
>  return True
>
> +QARECIPETEST[src-uri-bad] = "package_qa_check_src_uri"
> +def package_qa_check_src_uri(pn, d, messages):
> +import re
> +
> +if "${PN}" in d.getVar("SRC_URI", False):
> +package_qa_handle_error("src-uri-bad", "%s: SRC_URI uses PN not
> BPN" % pn, d)
> +
> +pn = d.getVar("SRC_URI")
> +if re.search(r"github\.com/.+/.+/archive/.+", pn):
> +package_qa_handle_error("src-uri-bad", "%s: SRC_URI uses unstable
> GitHub archives" % pn, d)
> +
>
>  # The PACKAGE FUNC to scan each package
>  python do_package_qa () {
> --
> 2.20.1 (Apple Git-117)
>
> --
> ___
> 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 1/3] musl-obstack: Add recipe

[Khem Raj]

On Wed, May 22, 2019 at 4:10 AM Richard Purdie
 wrote:
>
> On Tue, 2019-05-21 at 18:32 -0700, Khem Raj wrote:
> > obstacks from GNUlib is used in some OE-Core packages e.g. elfutils
> > and other packages outside OE-Core, this recipe helps provide this
> > functionality standalone on musl systems, and helps in getting full
> > versions of dependent packages
> >
> > Signed-off-by: Khem Raj 
> > ---
> >  meta/recipes-core/musl/musl-obstack.bb | 22 ++
> >  1 file changed, 22 insertions(+)
> >  create mode 100644 meta/recipes-core/musl/musl-obstack.bb
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/64/builds/623
>

this was due to missing pkgconfig inherit I believe.  I have sent a v2 now see

https://patchwork.openembedded.org/patch/161515/

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

[OE-core] [PATCH V2] musl-obstack: Add recipe

[Khem Raj]

obstacks from GNUlib is used in some OE-Core packages e.g. elfutils
and other packages outside OE-Core, this recipe helps provide this
functionality standalone on musl systems, and helps in getting full
versions of dependent packages

Signed-off-by: Khem Raj 
---
v2: Inherit pkgconfig

 meta/recipes-core/musl/musl-obstack.bb | 22 ++
 1 file changed, 22 insertions(+)
 create mode 100644 meta/recipes-core/musl/musl-obstack.bb

diff --git a/meta/recipes-core/musl/musl-obstack.bb 
b/meta/recipes-core/musl/musl-obstack.bb
new file mode 100644
index 00..3003935fe5
--- /dev/null
+++ b/meta/recipes-core/musl/musl-obstack.bb
@@ -0,0 +1,22 @@
+# Copyright (C) 2019 Khem Raj 
+# Released under the MIT license (see COPYING.MIT for the terms)
+
+SUMMARY = "A standalone library to implement GNU libc's obstack"
+DESCRIPTION = "copy + paste of the obstack functions and macros found in GNU 
gcc libiberty library for use with musl libc"
+HOMEPAGE = "https://github.com/pullmoll/musl-obstack;
+LICENSE = "GPL-2.0+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=3d23e4eef8243efcaab6f0a438078932"
+SECTION = "libs"
+
+PV = "1.1"
+SRCREV = "d2ad66b0df44a4b784956f7f7f2717131ddc05f4"
+SRC_URI = "git://github.com/pullmoll/musl-obstack"
+
+UPSTREAM_CHECK_COMMITS = "1"
+
+inherit autotools pkgconfig
+
+S = "${WORKDIR}/git"
+
+COMPATIBLE_HOST = ".*-musl.*"
+
-- 
2.21.0

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

Re: [OE-core] [PATCH 1/1] util-linux: Add missing ptest dependencies

[Adrian Bunk]

On Wed, May 22, 2019 at 01:03:37PM +0200, Martin Jansa wrote:
> This pulls bc to many images with busybox, causing busybox postinst to fail
> with:
> 
> update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid
> since /usr/bin/bc exists and is not a link

Are you enabling CONFIG_BC in your busybox config?
It is not enabled in meta/recipes-core/busybox/busybox/defconfig

> I'll check which busybox defconfig entry enables BC and send patch to use
> u-a for bc in ./meta/recipes-extended/bc/bc_1.07.1.bb
> 
> There already is:
> ALTERNATIVE_${PN} = "dc"
> ALTERNATIVE_PRIORITY = "100"
> but maybe dc is a typo.

bc and dc are two different binaries built from bc.

busybox used to only have dc, bc was added 6 months ago.

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

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

Re: [OE-core] [PATCH 2/2] python-six: put python2/3 variant together

[Tim Orling]

On Wed, May 22, 2019 at 4:27 AM Alexander Kanavin 
wrote:

> On Wed, 22 May 2019 at 13:05,  wrote:
>
> > I don't think we need to switch our default, just be able to build say
> > oe-core without python/python2 in HOSTTOOLS and without needing python
> > 2.7 in OE-Core either.
> >
> > > Its good that project is taking lead here but we might leave many
> > > users behind.
> >
> > I suspect python2 will end up in a layer of its down which I'm fine
> > with, dependencies can then be clearly seen.
>
> For what it's worth, Fedora is getting serious about Python 2 removal,
> and they have a plan for it:
> https://fedoraproject.org/wiki/Changes/RetirePython2
>
> "All packages in Fedora that need Python 2 to run will be removed from
> Fedora 32 regardless of their dependencies. All packages in Fedora
> that need Python 2 to build will be removed from Fedora 32 regardless
> of their dependencies. "
>


We have discussed moving meta-python to its own repository (not under
meta-openembedded). As part of that move, I am proposing the existing
python2 recipes move to a separate (deprecated) layer, as RP mentioned
earlier. Then meta-python will become python3 only. We will also move
recipetool/devtool to provide python3 (only) support. People that need
python2 will very soon have to live with older stable branches. This should
not be a surprise.

We also need to do whatever we can with upstream projects that are not
feeling an urgency to move to python3. They should likewise not be
surprised at the need to switch.


> Alex
> --
> ___
> 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] libmodulemd: fix erroneous linking against v2 library when v1 was requested

[Khem Raj]

On Wed, May 22, 2019 at 4:21 AM Alexander Kanavin 
wrote:

> I'm not sure that would work, as at do_rootfs time both createrepo_c
> (v2 user) and libdnf (v1 user) need to be present in the
> sysroot-native, and so v1 and v2 files would clash at that point
> (libmodulemd.so would be provided by both).
>
> I am kind of hoping that libdnf would migrate off v1 to v2, and we
> could then simply disable v1 altogether. So far it hasn't happened,
> and libdnf update is now also blocked by the c++ code that older
> compilers don't like, so I'd leave thing as they are for now.
>
>
conflicts will happen only when dev packages
are bundled into image.

we probably need something like libusb for this case and then remove the
compat package when its
no longer needed.



> Alex
>
> On Wed, 22 May 2019 at 13:08, Richard Purdie
>  wrote:
> >
> > On Wed, 2019-05-22 at 12:28 +0200, Alexander Kanavin wrote:
> > > On Wed, 22 May 2019 at 04:04, Khem Raj  wrote:
> > > > > +
> > > > > + pkg.generate(
> > > > > +-libraries : modulemd_v1_lib,
> > > > > ++libraries : '-l:libmodulemd.so.1',
> > > >
> > > > shouldn't it be generating a separate package for v1 ?
> > > > hardcoding linker paths seems a bit hackish here, is it going
> > > > to generate a .pc file with .so.1 version ?
> > >
> > > The problem is that both v2 and v1 are installed into the sysroot; we
> > > don't have a mechanism to choose one or the other via DEPENDS (unlike
> > > desktop distros where you can install -v1-dev or -v2-dev package for
> > > your build).
> > > Then both v2 and v1 .pc files (without this patch) refer to
> > > '-lmodulemd' which in our case always resolves to v2 library.
> > >
> > > Wth the patch the generated v1 .pc will look like this, while v2
> > > continues to use the original '-lmodlulemd' in Libs:
> > >
> > > prefix=${pcfiledir}/../../../usr
> > > libdir=${prefix}/lib
> > > includedir=${prefix}/include
> > >
> > > Name: modulemd
> > > Description: Module metadata manipulation library
> > > Version: 1.8.10
> > > Requires: glib-2.0, gobject-2.0
> > > Libs: -l:libmodulemd.so.1
> > > Cflags: -I${includedir}/modulemd
> >
> > FWIW I think if there were two different recipes we could select which
> > was in the sysroot through DEPENDS?
> >
> > Cheers,
> >
> > Richard
> >
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] apt: Remove workaround patches for no longer supported host distributions

[Adrian Bunk]

On Wed, May 22, 2019 at 01:04:59PM +0200, Alexander Kanavin wrote:
> Our apt version is badly out of date; would you be able to work on an update?
>...

Not the kind of change I am keen to do on a recipe I don't use in OE.

> Alex

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

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

Re: [OE-core] [PATCH 2/2] python-six: put python2/3 variant together

[Alexander Kanavin]

On Wed, 22 May 2019 at 13:05,  wrote:

> I don't think we need to switch our default, just be able to build say
> oe-core without python/python2 in HOSTTOOLS and without needing python
> 2.7 in OE-Core either.
>
> > Its good that project is taking lead here but we might leave many
> > users behind.
>
> I suspect python2 will end up in a layer of its down which I'm fine
> with, dependencies can then be clearly seen.

For what it's worth, Fedora is getting serious about Python 2 removal,
and they have a plan for it:
https://fedoraproject.org/wiki/Changes/RetirePython2

"All packages in Fedora that need Python 2 to run will be removed from
Fedora 32 regardless of their dependencies. All packages in Fedora
that need Python 2 to build will be removed from Fedora 32 regardless
of their dependencies. "

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

Re: [OE-core] [PATCH] libmodulemd: fix erroneous linking against v2 library when v1 was requested

[Alexander Kanavin]

I'm not sure that would work, as at do_rootfs time both createrepo_c
(v2 user) and libdnf (v1 user) need to be present in the
sysroot-native, and so v1 and v2 files would clash at that point
(libmodulemd.so would be provided by both).

I am kind of hoping that libdnf would migrate off v1 to v2, and we
could then simply disable v1 altogether. So far it hasn't happened,
and libdnf update is now also blocked by the c++ code that older
compilers don't like, so I'd leave thing as they are for now.

Alex

On Wed, 22 May 2019 at 13:08, Richard Purdie
 wrote:
>
> On Wed, 2019-05-22 at 12:28 +0200, Alexander Kanavin wrote:
> > On Wed, 22 May 2019 at 04:04, Khem Raj  wrote:
> > > > +
> > > > + pkg.generate(
> > > > +-libraries : modulemd_v1_lib,
> > > > ++libraries : '-l:libmodulemd.so.1',
> > >
> > > shouldn't it be generating a separate package for v1 ?
> > > hardcoding linker paths seems a bit hackish here, is it going
> > > to generate a .pc file with .so.1 version ?
> >
> > The problem is that both v2 and v1 are installed into the sysroot; we
> > don't have a mechanism to choose one or the other via DEPENDS (unlike
> > desktop distros where you can install -v1-dev or -v2-dev package for
> > your build).
> > Then both v2 and v1 .pc files (without this patch) refer to
> > '-lmodulemd' which in our case always resolves to v2 library.
> >
> > Wth the patch the generated v1 .pc will look like this, while v2
> > continues to use the original '-lmodlulemd' in Libs:
> >
> > prefix=${pcfiledir}/../../../usr
> > libdir=${prefix}/lib
> > includedir=${prefix}/include
> >
> > Name: modulemd
> > Description: Module metadata manipulation library
> > Version: 1.8.10
> > Requires: glib-2.0, gobject-2.0
> > Libs: -l:libmodulemd.so.1
> > Cflags: -I${includedir}/modulemd
>
> FWIW I think if there were two different recipes we could select which
> was in the sysroot through DEPENDS?
>
> Cheers,
>
> Richard
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/3] musl-obstack: Add recipe

[Richard Purdie]

On Tue, 2019-05-21 at 18:32 -0700, Khem Raj wrote:
> obstacks from GNUlib is used in some OE-Core packages e.g. elfutils
> and other packages outside OE-Core, this recipe helps provide this
> functionality standalone on musl systems, and helps in getting full
> versions of dependent packages
> 
> Signed-off-by: Khem Raj 
> ---
>  meta/recipes-core/musl/musl-obstack.bb | 22 ++
>  1 file changed, 22 insertions(+)
>  create mode 100644 meta/recipes-core/musl/musl-obstack.bb

https://autobuilder.yoctoproject.org/typhoon/#/builders/64/builds/623

Cheers,

Richard

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

Re: [OE-core] [PATCH] libmodulemd: fix erroneous linking against v2 library when v1 was requested

[Richard Purdie]

On Wed, 2019-05-22 at 12:28 +0200, Alexander Kanavin wrote:
> On Wed, 22 May 2019 at 04:04, Khem Raj  wrote:
> > > +
> > > + pkg.generate(
> > > +-libraries : modulemd_v1_lib,
> > > ++libraries : '-l:libmodulemd.so.1',
> > 
> > shouldn't it be generating a separate package for v1 ?
> > hardcoding linker paths seems a bit hackish here, is it going
> > to generate a .pc file with .so.1 version ?
> 
> The problem is that both v2 and v1 are installed into the sysroot; we
> don't have a mechanism to choose one or the other via DEPENDS (unlike
> desktop distros where you can install -v1-dev or -v2-dev package for
> your build).
> Then both v2 and v1 .pc files (without this patch) refer to
> '-lmodulemd' which in our case always resolves to v2 library.
> 
> Wth the patch the generated v1 .pc will look like this, while v2
> continues to use the original '-lmodlulemd' in Libs:
> 
> prefix=${pcfiledir}/../../../usr
> libdir=${prefix}/lib
> includedir=${prefix}/include
> 
> Name: modulemd
> Description: Module metadata manipulation library
> Version: 1.8.10
> Requires: glib-2.0, gobject-2.0
> Libs: -l:libmodulemd.so.1
> Cflags: -I${includedir}/modulemd

FWIW I think if there were two different recipes we could select which
was in the sysroot through DEPENDS?

Cheers,

Richard

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

Re: [OE-core] [PATCH 2/2] python-six: put python2/3 variant together

[richard . purdie]

On Tue, 2019-05-21 at 18:49 -0700, Khem Raj wrote:
> On Tue, May 21, 2019 at 2:44 PM 
> wrote:
> > On Tue, 2019-05-21 at 22:59 +0200, Andreas Müller wrote:
> > > On Tue, May 21, 2019 at 8:38 PM <
> > > richard.pur...@linuxfoundation.org>
> > > wrote:
> > > > > No sorry mozjs uses old crap in their configurations scripts
> > > > > nobody
> > > > > else does (python which) and ask for python2 explicitly.
> > > > 
> > > > I've been giving this a bit more thought and its our intent to
> > > > try
> > > > and
> > > > remove python2 from OE-Core entirely so adding py2 modules back
> > > > seems
> > > > like a backwards step.
> > > > 
> > > > I think this will therefore need to go to one of the other
> > > > layers,
> > > > painful as that might be from an include perspective.
> > > Yeah a new burden out of thin air is born. You know what I do:
> > > Create
> > > my own layers and am happy as I am with my other layers.
> > > 
> > > Bye
> > 
> > Its not "out of thin air". It was discussed over two years ago when
> > we
> > started a plan of trying to remove python2 dependencies:
> > 
> > http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=1fc2c9315faf65ea05fecf81450dd40b9a330435
> > 
> > This was in line with what upstream python recommended. Upstream
> > python
> > has made a clear statement on this too:
> > 
> > https://python3statement.org/
> > 
> > For a release that will ship 3 months before that deadline, it
> > seems
> > unreasonable to add this back to oe-core, no?
> > 
> 
> the EOL is expected to be Jan 2020, lets see what happens but I think
> it would be good for us
> to first switch default python to be python3 and then remove python2
> completely IMO, I do not
> see any major distro planning to drop py2 at the stroke of it going
> EOL as of now, situation might
> change in coming months but many of them haven't even switched
> defaults.

I don't think we need to switch our default, just be able to build say
oe-core without python/python2 in HOSTTOOLS and without needing python
2.7 in OE-Core either. 

> Its good that project is taking lead here but we might leave many
> users behind.

I suspect python2 will end up in a layer of its down which I'm fine
with, dependencies can then be clearly seen.

Cheers,

Richard

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

Re: [OE-core] [PATCH] apt: Remove workaround patches for no longer supported host distributions

[Alexander Kanavin]

Our apt version is badly out of date; would you be able to work on an update?

As a starting point, you can take this:
http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=akanavin/package-version-updates-later=e27b40c022654302e4846e47cc7da498a68d9666

Alex

On Wed, 22 May 2019 at 12:59, Adrian Bunk  wrote:
>
> All still supported host distributions have gcc >= 4.8.
>
> Signed-off-by: Adrian Bunk 
> ---
>  meta/recipes-devtools/apt/apt-native.inc  |  2 -
>  .../apt/0001-fix-the-gcc-version-check.patch  | 74 ---
>  ...emove-Wsuggest-attribute-from-CFLAGS.patch | 43 ---
>  3 files changed, 119 deletions(-)
>  delete mode 100644 
> meta/recipes-devtools/apt/apt/0001-fix-the-gcc-version-check.patch
>  delete mode 100644 
> meta/recipes-devtools/apt/apt/0001-remove-Wsuggest-attribute-from-CFLAGS.patch
>
> diff --git a/meta/recipes-devtools/apt/apt-native.inc 
> b/meta/recipes-devtools/apt/apt-native.inc
> index ef232c1eb6..d826786329 100644
> --- a/meta/recipes-devtools/apt/apt-native.inc
> +++ b/meta/recipes-devtools/apt/apt-native.inc
> @@ -7,8 +7,6 @@ USE_NLS = "yes"
>
>  SRC_URI += "file://db_linking_hack.patch \
>  
> file://0001-Revert-always-run-dpkg-configure-a-at-the-end-of-our.patch \
> -file://0001-remove-Wsuggest-attribute-from-CFLAGS.patch \
> -file://0001-fix-the-gcc-version-check.patch \
>  "
>
>  python do_install () {
> diff --git 
> a/meta/recipes-devtools/apt/apt/0001-fix-the-gcc-version-check.patch 
> b/meta/recipes-devtools/apt/apt/0001-fix-the-gcc-version-check.patch
> deleted file mode 100644
> index 3ac92462c5..00
> --- a/meta/recipes-devtools/apt/apt/0001-fix-the-gcc-version-check.patch
> +++ /dev/null
> @@ -1,74 +0,0 @@
> -From 53c5d0982f03fd0e24c4195d6e1e42b9ade9d500 Mon Sep 17 00:00:00 2001
> -From: Roy Li 
> -Date: Wed, 27 May 2015 14:30:28 +0800
> -Subject: [PATCH] fix the gcc version check
> -
> -Upstream-Status: pending
> -
> -"GCC diagnostic push" is gcc 4.6 feature, gcc 4.4.7 on centos did not know it
> -
> -Signed-off-by: Roy Li 
> 
> - apt-pkg/contrib/macros.h | 2 +-
> - apt-pkg/deb/debsrcrecords.cc | 4 ++--
> - apt-pkg/srcrecords.cc| 4 ++--
> - 3 files changed, 5 insertions(+), 5 deletions(-)
> -
> -diff --git a/apt-pkg/contrib/macros.h b/apt-pkg/contrib/macros.h
> -index 2727fd8..0ecae50 100644
>  a/apt-pkg/contrib/macros.h
> -+++ b/apt-pkg/contrib/macros.h
> -@@ -136,7 +136,7 @@
> - #endif
> - #endif
> -
> --#if __GNUC__ >= 4
> -+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
> -   #define APT_IGNORE_DEPRECATED_PUSH \
> -   _Pragma("GCC diagnostic push") \
> -   _Pragma("GCC diagnostic ignored 
> \"-Wdeprecated-declarations\"")
> -diff --git a/apt-pkg/deb/debsrcrecords.cc b/apt-pkg/deb/debsrcrecords.cc
> -index e8295de..40160b2 100644
>  a/apt-pkg/deb/debsrcrecords.cc
> -+++ b/apt-pkg/deb/debsrcrecords.cc
> -@@ -139,13 +139,13 @@ bool 
> debSrcRecordParser::Files(std::vector )
> -for (std::vector::const_iterator f2 = F2.begin(); 
> f2 != F2.end(); ++f2)
> -{
> -   pkgSrcRecords::File2 f;
> --#if __GNUC__ >= 4
> -+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
> -   #pragma GCC diagnostic push
> -   #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
> - #endif
> -   f.MD5Hash = f2->MD5Hash;
> -   f.Size = f2->Size;
> --#if __GNUC__ >= 4
> -+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
> -   #pragma GCC diagnostic pop
> - #endif
> -   f.Path = f2->Path;
> -diff --git a/apt-pkg/srcrecords.cc b/apt-pkg/srcrecords.cc
> -index 53d7e60..1484828 100644
>  a/apt-pkg/srcrecords.cc
> -+++ b/apt-pkg/srcrecords.cc
> -@@ -157,7 +157,7 @@ bool 
> pkgSrcRecords::Parser::Files2(std::vector )/*{{{*/
> -for (std::vector::const_iterator f = F.begin(); f 
> != F.end(); ++f)
> -{
> -   pkgSrcRecords::File2 f2;
> --#if __GNUC__ >= 4
> -+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
> -   #pragma GCC diagnostic push
> -   #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
> - #endif
> -@@ -165,7 +165,7 @@ bool 
> pkgSrcRecords::Parser::Files2(std::vector )/*{{{*/
> -   f2.Size = f->Size;
> -   f2.Hashes.push_back(HashString("MD5Sum", f->MD5Hash));
> -   f2.FileSize = f->Size;
> --#if __GNUC__ >= 4
> -+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
> -   #pragma GCC diagnostic pop
> - #endif
> -   f2.Path = f->Path;
> ---
> -2.1.4
> -
> diff --git 
> a/meta/recipes-devtools/apt/apt/0001-remove-Wsuggest-attribute-from-CFLAGS.patch
>  
> b/meta/recipes-devtools/apt/apt/0001-remove-Wsuggest-attribute-from-CFLAGS.patch
> deleted file mode 100644
> index 47870e3c8b..00
> --- 
> a/meta/recipes-devtools/apt/apt/0001-remove-Wsuggest-attribute-from-CFLAGS.patch
> +++ /dev/null
> @@ -1,43 +0,0 @@
> -From 9fdf50d63df08ee65e7d8e62c35f19ac4841bca9 Mon Sep 17 00:00:00 2001
> -From: Roy Li 
> -Date: Wed, 27 May 2015 09:48:45 +0800
> -Subject: [PATCH] remove Wsuggest-attribute from CXXFLAGS
> -
> 

Re: [OE-core] [PATCH 1/1] util-linux: Add missing ptest dependencies

[Martin Jansa]

This pulls bc to many images with busybox, causing busybox postinst to fail
with:

update-alternatives: Error: not linking /usr/bin/bc to /bin/busybox.nosuid
since /usr/bin/bc exists and is not a link

I'll check which busybox defconfig entry enables BC and send patch to use
u-a for bc in ./meta/recipes-extended/bc/bc_1.07.1.bb

There already is:
ALTERNATIVE_${PN} = "dc"
ALTERNATIVE_PRIORITY = "100"
but maybe dc is a typo.

On Sun, May 19, 2019 at 11:21 PM Mariano López <
just.another.mari...@gmail.com> wrote:

> There are some missing dependencies for the util-linux-ptest package
> that causes inconsistencies in the package tests run in different images.
>
> The kernel module in RRECOMMENDS is not build at this time, it needs
> more testing and check if the configuration change can be part of the
> yocto-kernel-cache repository.
>
> Signed-off-by: Mariano López 
> ---
>  meta/recipes-core/util-linux/util-linux.inc | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-core/util-linux/util-linux.inc
> b/meta/recipes-core/util-linux/util-linux.inc
> index 34255a2dec..6dfbe0b683 100644
> --- a/meta/recipes-core/util-linux/util-linux.inc
> +++ b/meta/recipes-core/util-linux/util-linux.inc
> @@ -142,7 +142,8 @@ RDEPENDS_${PN}_class-nativesdk = ""
>  RPROVIDES_${PN}-dev = "${PN}-libblkid-dev ${PN}-libmount-dev
> ${PN}-libuuid-dev"
>
>  RDEPENDS_${PN}-bash-completion += "${PN}-lsblk"
> -RDEPENDS_${PN}-ptest = "bash grep coreutils which btrfs-tools ${PN} sed"
> +RDEPENDS_${PN}-ptest = "bash bc btrfs-tools coreutils e2fsprogs grep
> iproute2 kmod mdadm ${PN} procps sed socat which xz"
> +RRECOMMENDS_${PN}-ptest = "kernel-module-scsi-debug"
>  RDEPENDS_${PN}-swaponoff = "${PN}-swapon ${PN}-swapoff"
>  ALLOW_EMPTY_${PN}-swaponoff = "1"
>
> --
> 2.21.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] iproute2: upgrade 5.0.0 -> 5.1.0

[Alexander Kanavin]

Signed-off-by: Alexander Kanavin 
---
 .../iproute2/{iproute2_5.0.0.bb => iproute2_5.1.0.bb} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-connectivity/iproute2/{iproute2_5.0.0.bb => 
iproute2_5.1.0.bb} (68%)

diff --git a/meta/recipes-connectivity/iproute2/iproute2_5.0.0.bb 
b/meta/recipes-connectivity/iproute2/iproute2_5.1.0.bb
similarity index 68%
rename from meta/recipes-connectivity/iproute2/iproute2_5.0.0.bb
rename to meta/recipes-connectivity/iproute2/iproute2_5.1.0.bb
index 8f33c7fe194..6854a130042 100644
--- a/meta/recipes-connectivity/iproute2/iproute2_5.0.0.bb
+++ b/meta/recipes-connectivity/iproute2/iproute2_5.1.0.bb
@@ -5,8 +5,8 @@ SRC_URI = 
"${KERNELORG_MIRROR}/linux/utils/net/${BPN}/${BP}.tar.xz \
file://0001-libc-compat.h-add-musl-workaround.patch \
   "
 
-SRC_URI[md5sum] = "d22107b4d7cfb999eeb8ad8a0aec1124"
-SRC_URI[sha256sum] = 
"df047302a39650ef832c07e8dab5df7a23218cd398bd310c8628e386161d20ba"
+SRC_URI[md5sum] = "a2b8349abf4ae00e92155fda22de4d5e"
+SRC_URI[sha256sum] = 
"dc5a980873eabf6b00c0be976b6e5562b1400d47d1d07d2ac35d5e5acbcf7bcf"
 
 # CFLAGS are computed in Makefile and reference CCOPTS
 #
-- 
2.17.1

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

[OE-core] [PATCH 1/2] piglit: upgrade to latest revision

[Alexander Kanavin]

Signed-off-by: Alexander Kanavin 
---
 ...UPWARD-only-if-its-defined-in-fenv.h.patch | 75 ---
 .../piglit/piglit/format-fix.patch| 69 -
 meta/recipes-graphics/piglit/piglit_git.bb|  5 +-
 3 files changed, 1 insertion(+), 148 deletions(-)
 delete mode 100644 
meta/recipes-graphics/piglit/piglit/0001-tests-Use-FE_UPWARD-only-if-its-defined-in-fenv.h.patch
 delete mode 100644 meta/recipes-graphics/piglit/piglit/format-fix.patch

diff --git 
a/meta/recipes-graphics/piglit/piglit/0001-tests-Use-FE_UPWARD-only-if-its-defined-in-fenv.h.patch
 
b/meta/recipes-graphics/piglit/piglit/0001-tests-Use-FE_UPWARD-only-if-its-defined-in-fenv.h.patch
deleted file mode 100644
index 57eda2e207b..000
--- 
a/meta/recipes-graphics/piglit/piglit/0001-tests-Use-FE_UPWARD-only-if-its-defined-in-fenv.h.patch
+++ /dev/null
@@ -1,75 +0,0 @@
-Upstream-Status: Submitted [mailing list]
-Signed-off-by: Ross Burton 
-
-From 0e0a2a69261031d55d52b6045990e8982ea12912 Mon Sep 17 00:00:00 2001
-From: Khem Raj 
-Date: Sat, 9 Jul 2016 07:52:19 +
-Subject: [PATCH] tests: only run rounding tests if FE_UPWARD is present
-
-On ARM, musl does not define FE_* when the architecture does not have VFP 
(which
-is the right interpretation).
-
-As these tests depend on calling fesetround(), skip the test if FE_UPWARD isn't
-available.
-
-Signed-off-by: Ross Burton 

- tests/general/roundmode-getintegerv.c | 12 
- tests/general/roundmode-pixelstore.c  | 12 
- 2 files changed, 16 insertions(+), 8 deletions(-)
-
-diff --git a/tests/general/roundmode-getintegerv.c 
b/tests/general/roundmode-getintegerv.c
-index 28ecfaf55..aa99044a1 100644
 a/tests/general/roundmode-getintegerv.c
-+++ b/tests/general/roundmode-getintegerv.c
-@@ -79,13 +79,17 @@ test(float val, int expect)
- void
- piglit_init(int argc, char **argv)
- {
--  int ret;
-   bool pass = true;
--  ret = fesetround(FE_UPWARD);
--  if (ret != 0) {
--  printf("Couldn't set rounding mode\n");
-+
-+#ifdef FE_UPWARD
-+  if (fesetround(FE_UPWARD) != 0) {
-+  printf("Setting rounding mode failed\n");
-   piglit_report_result(PIGLIT_SKIP);
-   }
-+#else
-+  printf("Cannot set rounding mode\n");
-+  piglit_report_result(PIGLIT_SKIP);
-+#endif
- 
-   pass = test(2.2, 2) && pass;
-   pass = test(2.8, 3) && pass;
-diff --git a/tests/general/roundmode-pixelstore.c 
b/tests/general/roundmode-pixelstore.c
-index 8a029b257..57ec11c09 100644
 a/tests/general/roundmode-pixelstore.c
-+++ b/tests/general/roundmode-pixelstore.c
-@@ -79,13 +79,17 @@ test(float val, int expect)
- void
- piglit_init(int argc, char **argv)
- {
--  int ret;
-   bool pass = true;
--  ret = fesetround(FE_UPWARD);
--  if (ret != 0) {
--  printf("Couldn't set rounding mode\n");
-+
-+#ifdef FE_UPWARD
-+  if (fesetround(FE_UPWARD) != 0) {
-+  printf("Setting rounding mode failed\n");
-   piglit_report_result(PIGLIT_SKIP);
-   }
-+#else
-+  printf("Cannot set rounding mode\n");
-+  piglit_report_result(PIGLIT_SKIP);
-+#endif
- 
-   pass = test(2.2, 2) && pass;
-   pass = test(2.8, 3) && pass;
--- 
-2.11.0
-
diff --git a/meta/recipes-graphics/piglit/piglit/format-fix.patch 
b/meta/recipes-graphics/piglit/piglit/format-fix.patch
deleted file mode 100644
index 73d539fef27..000
--- a/meta/recipes-graphics/piglit/piglit/format-fix.patch
+++ /dev/null
@@ -1,69 +0,0 @@
-Upstream-Status: Submitted [mailing list]
-Signed-off-by: Ross Burton 
-
-From f0c6981322807e179e39ce67aeebd42cf7a54d36 Mon Sep 17 00:00:00 2001
-From: Ross Burton 
-Date: Wed, 21 Nov 2018 12:44:36 +
-Subject: [PATCH] arb_texture_view: fix security format warnings
-
-If built with -Werror=format-security then Piglit fails to build:
-
-/tests/spec/arb_texture_view/rendering-layers-image.c:150:8:
-error: format not a string literal and no format arguments 
[-Werror=format-security]
- (desc)); \
- ^~
-
-In this case test->uniform_type is being turned into a string using snprintf()
-and then passed to piglit_report_subtest_result() which takes a format string,
-but GCC can't verify the format.
-
-As _subtest_report() takes a format string, we can just remove the snprintf()
-and let it construct the label.
-
-Also as X is used once and doesn't make the code clearer, just inline it.
-
-Signed-off-by: Ross Burton 

- tests/spec/arb_texture_view/rendering-layers-image.c | 19 ++-
- 1 file changed, 6 insertions(+), 13 deletions(-)
-
-diff --git a/tests/spec/arb_texture_view/rendering-layers-image.c 
b/tests/spec/arb_texture_view/rendering-layers-image.c
-index 415b01657..86148075b 100644
 a/tests/spec/arb_texture_view/rendering-layers-image.c
-+++ b/tests/spec/arb_texture_view/rendering-layers-image.c
-@@ -142,26 +142,19 @@ test_render_layers(const struct test_info *test)
-   return pass;
- }
- 

Re: [OE-core] [PATCH v4] python*-setuptools: add separate packages for pkg_resources module

[Luca Boccassi]

On Tue, 2019-05-21 at 19:06 -0700, Khem Raj wrote:
> On Tue, May 21, 2019 at 5:36 AM <
> luca.bocca...@gmail.com
> > wrote:
> > From: Luca Boccassi <
> > luca.bocca...@microsoft.com
> > >
> > 
> > The pkg_resources Python module is useful by itself, for example
> > for
> > automatic loading of resources shipped in a Python package.
> > Add separate packages for it, so that users can depend on them
> > individually and avoid pulling in the entire setuptools, which
> > include scripts to download other packages, which might not be
> > desired on minimal images.
> > 
> > Other distributions like Debian and Ubuntu already split setuptools
> > and pkg-resources in this way.
> > 
> > The setuptools packages now depend on the new pkg-resources
> > packages,
> > to avoid regressions for other packages that depend on them
> > already.
> > 
> > Signed-off-by: Luca Boccassi <
> > luca.bocca...@microsoft.com
> > >
> > ---
> > v2: restrict new RDEPENDS to class-target. As advised by Alexander,
> > bitbake
> > cannot resolve native rdeps that mention package names rather
> > than
> > recipe names.
> > v3: manually add RPROVIDES to the native class instead of
> > restricting the
> > RDEPENDS to the target class as a better workaround. Also
> > document why
> > the package is being split.
> > v4: re-send to the correct thread, no changes.
> > 
> >  meta/recipes-devtools/python/python-setuptools.inc | 11
> > +++
> >  1 file changed, 11 insertions(+)
> > 
> > diff --git a/meta/recipes-devtools/python/python-setuptools.inc
> > b/meta/recipes-devtools/python/python-setuptools.inc
> > index 357aa07086..f49e078697 100644
> > --- a/meta/recipes-devtools/python/python-setuptools.inc
> > +++ b/meta/recipes-devtools/python/python-setuptools.inc
> > @@ -37,3 +37,14 @@ do_install_prepend() {
> >  }
> > 
> >  BBCLASSEXTEND = "native nativesdk"
> > +
> > +# The pkg-resources module can be used by itself, without the
> > package downloader
> > +# and easy_install. Ship it in a separate package so that it can
> > be used by
> > +# minimal distributions.
> > +PACKAGES =+ "${PYTHON_PN}-pkg-resources "
> > +FILES_${PYTHON_PN}-pkg-resources =
> > "${PYTHON_SITEPACKAGES_DIR}/pkg_resources/*"
> > +# Due to the way OE-Core implemented native recipes, the native
> > class cannot
> > +# have a dependency on something that is not a recipe name. Work
> > around that by
> > +# manually setting RPROVIDES.
> > +RDEPENDS_${PN}_append = " ${PYTHON_PN}-pkg-resources"
> > +RPROVIDES_append_class-native = " ${PYTHON_PN}-pkg-resources-
> > native"
> 
> do we need to handle nativesdk case ?

Hi,

The parsing step of "bitbake core-image-minimal -c populate_sdk" works,
while without the append_class-native workaround it fails immediately.
Is this enough or is there something else I should run to check?

Thanks!

-- 
Kind regards,
Luca Boccassi

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

[OE-core] [PATCH] apt: Remove workaround patches for no longer supported host distributions

[Adrian Bunk]

All still supported host distributions have gcc >= 4.8.

Signed-off-by: Adrian Bunk 
---
 meta/recipes-devtools/apt/apt-native.inc  |  2 -
 .../apt/0001-fix-the-gcc-version-check.patch  | 74 ---
 ...emove-Wsuggest-attribute-from-CFLAGS.patch | 43 ---
 3 files changed, 119 deletions(-)
 delete mode 100644 
meta/recipes-devtools/apt/apt/0001-fix-the-gcc-version-check.patch
 delete mode 100644 
meta/recipes-devtools/apt/apt/0001-remove-Wsuggest-attribute-from-CFLAGS.patch

diff --git a/meta/recipes-devtools/apt/apt-native.inc 
b/meta/recipes-devtools/apt/apt-native.inc
index ef232c1eb6..d826786329 100644
--- a/meta/recipes-devtools/apt/apt-native.inc
+++ b/meta/recipes-devtools/apt/apt-native.inc
@@ -7,8 +7,6 @@ USE_NLS = "yes"
 
 SRC_URI += "file://db_linking_hack.patch \
 
file://0001-Revert-always-run-dpkg-configure-a-at-the-end-of-our.patch \
-file://0001-remove-Wsuggest-attribute-from-CFLAGS.patch \
-file://0001-fix-the-gcc-version-check.patch \
 "
 
 python do_install () {
diff --git a/meta/recipes-devtools/apt/apt/0001-fix-the-gcc-version-check.patch 
b/meta/recipes-devtools/apt/apt/0001-fix-the-gcc-version-check.patch
deleted file mode 100644
index 3ac92462c5..00
--- a/meta/recipes-devtools/apt/apt/0001-fix-the-gcc-version-check.patch
+++ /dev/null
@@ -1,74 +0,0 @@
-From 53c5d0982f03fd0e24c4195d6e1e42b9ade9d500 Mon Sep 17 00:00:00 2001
-From: Roy Li 
-Date: Wed, 27 May 2015 14:30:28 +0800
-Subject: [PATCH] fix the gcc version check
-
-Upstream-Status: pending
-
-"GCC diagnostic push" is gcc 4.6 feature, gcc 4.4.7 on centos did not know it
-
-Signed-off-by: Roy Li 

- apt-pkg/contrib/macros.h | 2 +-
- apt-pkg/deb/debsrcrecords.cc | 4 ++--
- apt-pkg/srcrecords.cc| 4 ++--
- 3 files changed, 5 insertions(+), 5 deletions(-)
-
-diff --git a/apt-pkg/contrib/macros.h b/apt-pkg/contrib/macros.h
-index 2727fd8..0ecae50 100644
 a/apt-pkg/contrib/macros.h
-+++ b/apt-pkg/contrib/macros.h
-@@ -136,7 +136,7 @@
- #endif
- #endif
- 
--#if __GNUC__ >= 4
-+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
-   #define APT_IGNORE_DEPRECATED_PUSH \
-   _Pragma("GCC diagnostic push") \
-   _Pragma("GCC diagnostic ignored \"-Wdeprecated-declarations\"")
-diff --git a/apt-pkg/deb/debsrcrecords.cc b/apt-pkg/deb/debsrcrecords.cc
-index e8295de..40160b2 100644
 a/apt-pkg/deb/debsrcrecords.cc
-+++ b/apt-pkg/deb/debsrcrecords.cc
-@@ -139,13 +139,13 @@ bool 
debSrcRecordParser::Files(std::vector )
-for (std::vector::const_iterator f2 = F2.begin(); f2 
!= F2.end(); ++f2)
-{
-   pkgSrcRecords::File2 f;
--#if __GNUC__ >= 4
-+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
-   #pragma GCC diagnostic push
-   #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
- #endif
-   f.MD5Hash = f2->MD5Hash;
-   f.Size = f2->Size;
--#if __GNUC__ >= 4
-+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
-   #pragma GCC diagnostic pop
- #endif
-   f.Path = f2->Path;
-diff --git a/apt-pkg/srcrecords.cc b/apt-pkg/srcrecords.cc
-index 53d7e60..1484828 100644
 a/apt-pkg/srcrecords.cc
-+++ b/apt-pkg/srcrecords.cc
-@@ -157,7 +157,7 @@ bool 
pkgSrcRecords::Parser::Files2(std::vector )/*{{{*/
-for (std::vector::const_iterator f = F.begin(); f != 
F.end(); ++f)
-{
-   pkgSrcRecords::File2 f2;
--#if __GNUC__ >= 4
-+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
-   #pragma GCC diagnostic push
-   #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
- #endif
-@@ -165,7 +165,7 @@ bool 
pkgSrcRecords::Parser::Files2(std::vector )/*{{{*/
-   f2.Size = f->Size;
-   f2.Hashes.push_back(HashString("MD5Sum", f->MD5Hash));
-   f2.FileSize = f->Size;
--#if __GNUC__ >= 4
-+#if __GNUC__ >= 4 + (6 >= __GNUC_MINOR__)
-   #pragma GCC diagnostic pop
- #endif
-   f2.Path = f->Path;
--- 
-2.1.4
-
diff --git 
a/meta/recipes-devtools/apt/apt/0001-remove-Wsuggest-attribute-from-CFLAGS.patch
 
b/meta/recipes-devtools/apt/apt/0001-remove-Wsuggest-attribute-from-CFLAGS.patch
deleted file mode 100644
index 47870e3c8b..00
--- 
a/meta/recipes-devtools/apt/apt/0001-remove-Wsuggest-attribute-from-CFLAGS.patch
+++ /dev/null
@@ -1,43 +0,0 @@
-From 9fdf50d63df08ee65e7d8e62c35f19ac4841bca9 Mon Sep 17 00:00:00 2001
-From: Roy Li 
-Date: Wed, 27 May 2015 09:48:45 +0800
-Subject: [PATCH] remove Wsuggest-attribute from CXXFLAGS
-
-Upstream-Status: Inappropriate
-
-Wsuggest-attribute is GCC 4.6 feature, centos6 is using GCC
-4.4.7 which unknown this flag, so remove it from CXXFLAGS
-
-Signed-off-by: Roy Li 

- buildlib/environment.mak.in | 1 -
- test/libapt/makefile| 1 -
- 2 files changed, 2 deletions(-)
-
-diff --git a/buildlib/environment.mak.in b/buildlib/environment.mak.in
-index 9620722..c344d01 100644
 a/buildlib/environment.mak.in
-+++ b/buildlib/environment.mak.in
-@@ -14,7 +14,6 @@ CXXFLAGS+= @CXXFLAGS@ -Wall -Wextra
- CXXFLAGS+= -Wcast-align -Wlogical-op -Wredundant-decls 

[OE-core] [PATCH] dpkg: Remove workaround patches for no longer supported host distributions

[Adrian Bunk]

Signed-off-by: Adrian Bunk 
---
 .../0005-dpkg-compiler.m4-remove-Wvla.patch   |  40 ---
 .../dpkg/dpkg/glibc2.5-sync_file_range.patch  | 100 --
 meta/recipes-devtools/dpkg/dpkg_1.19.4.bb |   3 +-
 3 files changed, 1 insertion(+), 142 deletions(-)
 delete mode 100644 
meta/recipes-devtools/dpkg/dpkg/0005-dpkg-compiler.m4-remove-Wvla.patch
 delete mode 100644 
meta/recipes-devtools/dpkg/dpkg/glibc2.5-sync_file_range.patch

diff --git 
a/meta/recipes-devtools/dpkg/dpkg/0005-dpkg-compiler.m4-remove-Wvla.patch 
b/meta/recipes-devtools/dpkg/dpkg/0005-dpkg-compiler.m4-remove-Wvla.patch
deleted file mode 100644
index b368508862..00
--- a/meta/recipes-devtools/dpkg/dpkg/0005-dpkg-compiler.m4-remove-Wvla.patch
+++ /dev/null
@@ -1,40 +0,0 @@
-From 0ad7bba80d5b9035089ff2b2f77a774b5b201915 Mon Sep 17 00:00:00 2001
-From: Alexander Kanavin 
-Date: Wed, 26 Aug 2015 16:28:59 +0300
-Subject: [PATCH 5/5] dpkg-compiler.m4: remove -Wvla
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-Remove the -Wvla flag from the set of compiler warning flags, since gcc
-on old host systems such as CentOS 5.8 doesn't support it, and it
-causes a build error for dpkg-native.
-
-Upstream-Status: Pending
-
-Signed-off-by: Donn Seeley 
-Signed-off-by: Robert Yang 
-Signed-off-by: Aníbal Limón 
-
-Update patch context for dpkg 1.19.4.
-
-Signed-off-by: Kai Kang 

- m4/dpkg-compiler.m4 | 1 -
- 1 file changed, 1 deletion(-)
-
-diff --git a/m4/dpkg-compiler.m4 b/m4/dpkg-compiler.m4
-index a6192f6..4c7bf2e 100644
 a/m4/dpkg-compiler.m4
-+++ b/m4/dpkg-compiler.m4
-@@ -67,7 +67,6 @@ AC_DEFUN([DPKG_CHECK_COMPILER_WARNINGS], [
-   DPKG_CHECK_COMPILER_FLAG([-Wshift-negative-value])
-   DPKG_CHECK_COMPILER_FLAG([-Wsizeof-array-argument])
-   DPKG_CHECK_COMPILER_FLAG([-Wswitch-bool])
--  DPKG_CHECK_COMPILER_FLAG([-Wvla])
-   DPKG_CHECK_COMPILER_FLAG([-Wwrite-strings])
- 
-   AC_LANG_CASE(
--- 
-2.1.4
-
diff --git a/meta/recipes-devtools/dpkg/dpkg/glibc2.5-sync_file_range.patch 
b/meta/recipes-devtools/dpkg/dpkg/glibc2.5-sync_file_range.patch
deleted file mode 100644
index 5debcc5209..00
--- a/meta/recipes-devtools/dpkg/dpkg/glibc2.5-sync_file_range.patch
+++ /dev/null
@@ -1,100 +0,0 @@
-From 9d260d408f9e17abd1d1dccd685bd7e80a3655a9 Mon Sep 17 00:00:00 2001
-From: Donn Seeley 
-Date: Tue, 25 Feb 2014 17:44:04 +0800
-Subject: [PATCH] dpkg: fix a link problem for dpkg-native on CentOS 5.8
-
-CentOS 5.8 kernels and headers support the sync_file_range() system call,
-but glibc 2.5 doesn't provide the syscall stub.  It appears that this
-problem is known but will never be fixed:
-
-  https://bugzilla.redhat.com/show_bug.cgi?id=518581
-
-  Bug 518581 - [RHEL5] glibc misses sync_file_range syscall interface
-
-  Status:   CLOSED CANTFIX
-  Last Closed:  2009-11-22 22:19:55
-
-  Kirby Zhou 2009-08-20 23:37:55 EDT
-
-  Description of problem:
-
-  glibc misses sync_file_range syscall interface.  The header file and
-  man page both say 'sync_file_range' should exist.  From man page,
-  sync_file_range should exist sinc kernel-2.6.17
-
-  Andreas Schwab 2009-08-21 03:24:24 EDT
-
-  It has only been added to glibc 2.6, and cannot be backported due to
-  ABI breakage.  You can always fall back to syscall(3).
-
-  Ulrich Drepper 2009-11-22 22:19:55 EST
-
-  As comment #1 says, no chance to backport this.
-
-  See the syscall man page for instructions.
-
-  Jon E 2010-03-19 10:32:37 EDT
-
-  then why document it if it's broken and you're not going to fix it?
-  .. might want to FTFM over at sync_file_range(2) - in the meantime -
-  borrowing from glibc 2.6 .. any thoughts on this implementation for a
-  hacky workaround for those still on your "ancient releases" .. (eg:
-  RHEL5.3)?:
-
-  #ifdef ULI_WONT_FIX_THIS_IN_GLIBC2.5
-  #define NR_sync_file_range 277
-  int sync_file_range (int fd, __off64_t from, __off64_t to, unsigned int 
flags)
-  {
-return syscall (NR_sync_file_range, fd,
-   __LONG_LONG_PAIR ((long) (from >> 32), (long) from),
-   __LONG_LONG_PAIR ((long) (to >> 32), (long) to),
-   flags);
-  }
-  #endif
-
-  assuming of course that you're on an x86_64 and include/asm-
-  x86_64/unistd.h has the correct entry
-
-  (fwiw - fio is starting to use this now)
-
-Rather than attempting to provide an implementation using syscall(),
-we take the more conservative route and ignore header support for
-sync_file_range() flags when the glibc version is <= 2.5.
-
-Upstream-Status: Inappropriate [everyone else builds on newer hosts :-)]
-
-Signed-off-by: Donn Seeley 
-Signed-off-by: Lei Liu 
-
-Update patch context for dpkg 1.19.4.
-
-Signed-off-by: Kai Kang 

- src/archives.c | 4 ++--
- 1 file changed, 2 insertions(+), 2 deletions(-)
-
-diff --git a/src/archives.c b/src/archives.c
-index 8476a5c..b05a63f 100644
 a/src/archives.c
-+++ b/src/archives.c
-@@ -68,7 +68,7 @@ 

[OE-core] [PATCH] librsvg: Replace workaround for old host systems with upstream fix

[Adrian Bunk]

Signed-off-by: Adrian Bunk 
---
 ...t-Bsymbolic-fixes-configure-on-macOS.patch | 35 +++
 meta/recipes-gnome/librsvg/librsvg_2.40.20.bb |  8 ++---
 2 files changed, 38 insertions(+), 5 deletions(-)
 create mode 100644 
meta/recipes-gnome/librsvg/librsvg/0001-Auto-detect-Bsymbolic-fixes-configure-on-macOS.patch

diff --git 
a/meta/recipes-gnome/librsvg/librsvg/0001-Auto-detect-Bsymbolic-fixes-configure-on-macOS.patch
 
b/meta/recipes-gnome/librsvg/librsvg/0001-Auto-detect-Bsymbolic-fixes-configure-on-macOS.patch
new file mode 100644
index 00..954bb60880
--- /dev/null
+++ 
b/meta/recipes-gnome/librsvg/librsvg/0001-Auto-detect-Bsymbolic-fixes-configure-on-macOS.patch
@@ -0,0 +1,35 @@
+From b99891e31eb6ce550e7e1cb2ca592095b3050a93 Mon Sep 17 00:00:00 2001
+From: Brion Vibber 
+Date: Sun, 25 Feb 2018 18:42:36 -0800
+Subject: Auto-detect -Bsymbolic, fixes configure on macOS
+
+The -Bsymbolic linker option is ELF-specific, and was breaking
+configure on macOS unless --disable-Bsymbolic was explicitly passed.
+
+Switching the behavior from requiring -Bsymbolic to be available
+by default to just warning and continuing on without.
+
+Fixes https://gitlab.gnome.org/GNOME/librsvg/issues/211
+
+Upstream-Status: Backport
+Signed-off-by: Adrian Bunk 
+---
+ configure.ac | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index 15b26b2d..9f8dce29 100644
+--- a/configure.ac
 b/configure.ac
+@@ -216,7 +216,7 @@ AM_CONDITIONAL([ENABLE_PIXBUF_LOADER],[test 
"$enable_pixbuf_loader" = "yes"])
+ AC_ARG_ENABLE([Bsymbolic],
+   [AS_HELP_STRING([--disable-Bsymbolic],
+   [disable linking with -Bsymbolic])],
+-  [],[enable_Bsymbolic=yes])
++  [enable_Bsymbolic=no],[enable_Bsymbolic=auto])
+ 
+ BSYMBOLIC_LDFLAG=
+ if test "$enable_Bsymbolic" != "no"; then
+-- 
+2.20.1
+
diff --git a/meta/recipes-gnome/librsvg/librsvg_2.40.20.bb 
b/meta/recipes-gnome/librsvg/librsvg_2.40.20.bb
index e376d52ef3..6a798e6a91 100644
--- a/meta/recipes-gnome/librsvg/librsvg_2.40.20.bb
+++ b/meta/recipes-gnome/librsvg/librsvg_2.40.20.bb
@@ -14,17 +14,15 @@ BBCLASSEXTEND = "native"
 
 inherit gnomebase gtk-doc pixbufcache upstream-version-is-even 
gobject-introspection
 
-SRC_URI += "file://gtk-option.patch"
+SRC_URI += "file://gtk-option.patch \
+file://0001-Auto-detect-Bsymbolic-fixes-configure-on-macOS.patch \
+"
 
 SRC_URI[archive.md5sum] = "4949d313b0c5d9161a5c259104af5568"
 SRC_URI[archive.sha256sum] = 
"cff4dd3c3b78bfe99d8fcfad3b8ba1eee3289a0823c0e118d78106be6b84c92b"
 
 CACHED_CONFIGUREVARS = 
"ac_cv_path_GDK_PIXBUF_QUERYLOADERS=${STAGING_LIBDIR_NATIVE}/gdk-pixbuf-2.0/gdk-pixbuf-query-loaders"
 
-# The older ld (2.22) on the host (Centos 6.5) doesn't have the
-# -Bsymbolic-functions option, we can disable it for native.
-EXTRA_OECONF_append_class-native = " --enable-Bsymbolic=auto"
-
 PACKAGECONFIG ??= "gdkpixbuf"
 # The gdk-pixbuf loader
 PACKAGECONFIG[gdkpixbuf] = 
"--enable-pixbuf-loader,--disable-pixbuf-loader,gdk-pixbuf-native"
-- 
2.17.1

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

Re: [OE-core] [PATCH] libmodulemd: fix erroneous linking against v2 library when v1 was requested

[Alexander Kanavin]

On Wed, 22 May 2019 at 04:04, Khem Raj  wrote:
> > +
> > + pkg.generate(
> > +-libraries : modulemd_v1_lib,
> > ++libraries : '-l:libmodulemd.so.1',
>
> shouldn't it be generating a separate package for v1 ?
> hardcoding linker paths seems a bit hackish here, is it going
> to generate a .pc file with .so.1 version ?

The problem is that both v2 and v1 are installed into the sysroot; we
don't have a mechanism to choose one or the other via DEPENDS (unlike
desktop distros where you can install -v1-dev or -v2-dev package for
your build).
Then both v2 and v1 .pc files (without this patch) refer to
'-lmodulemd' which in our case always resolves to v2 library.

Wth the patch the generated v1 .pc will look like this, while v2
continues to use the original '-lmodlulemd' in Libs:

prefix=${pcfiledir}/../../../usr
libdir=${prefix}/lib
includedir=${prefix}/include

Name: modulemd
Description: Module metadata manipulation library
Version: 1.8.10
Requires: glib-2.0, gobject-2.0
Libs: -l:libmodulemd.so.1
Cflags: -I${includedir}/modulemd


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

Re: [OE-core] [PATCH 13/42] systemtap: upgrade 4.0 -> 4.1

[Alexander Kanavin]

Didn't test it at all; the patch was produced by AUH, so I simply picked it up.

Do we have anything in the autobuilder for it?

Alex

On Tue, 21 May 2019 at 23:37, Victor Kamensky (kamensky)
 wrote:
>
> Hi Alexander,
>
> I am curios how much did you test it? I had the same patch for
> a while, waited to post it till proper regressions are done.
>
> Thanks,
> Victor
>
> 
> From: openembedded-core-boun...@lists.openembedded.org 
>  on behalf of Alexander 
> Kanavin 
> Sent: Monday, May 20, 2019 7:10 AM
> To: openembedded-core@lists.openembedded.org
> Subject: [OE-core] [PATCH 13/42] systemtap: upgrade 4.0 -> 4.1
>
> Signed-off-by: Alexander Kanavin 
> ---
>  meta/recipes-kernel/systemtap/systemtap_git.inc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-kernel/systemtap/systemtap_git.inc 
> b/meta/recipes-kernel/systemtap/systemtap_git.inc
> index 54de7226608..c5348b3204b 100644
> --- a/meta/recipes-kernel/systemtap/systemtap_git.inc
> +++ b/meta/recipes-kernel/systemtap/systemtap_git.inc
> @@ -1,7 +1,7 @@
>  LICENSE = "GPLv2"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> -SRCREV = "90ff34ac0506f0d5440393a4c78565f6aaf9b726"
> -PV = "4.0"
> +SRCREV = "984d6d1696ed06626b07cb65ab55d6ae0ece1131"
> +PV = "4.1"
>
>  SRC_URI = "git://sourceware.org/git/systemtap.git \
> file://configure-allow-to-disable-libvirt.patch \
> --
> 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

[OE-core] [PATCH] elfutils: fix ptest failures

[mingli.yu]

From: Mingli Yu 

* Add missing -ptest package dependencies (needs
  ${PN}-dbg)

* Add missing files which needed by ptest test
  to fix below failures:
  | ./run-ar.sh: line 23: cd: /usr/lib64/elfutils/ptest/tests/..//src: No such 
file or directory
  | FAIL: run-ar.sh

  | sh: ../src/elflint: No such file or directory
  | FAIL: asm-tst4

* Rework 0001-skip-the-test-when-gcc-not-deployed.patch
  to skip the tests which depend on gcc

Before:

Recipe   | Passed| Failed   | Skipped

elfutils | 176   | 23   | 4


After:

Recipe   | Passed| Failed   | Skipped

elfutils | 184   | 15   | 4


Signed-off-by: Mingli Yu 
---
 meta/recipes-devtools/elfutils/elfutils_0.176.bb   |  5 ++-
 .../0001-skip-the-test-when-gcc-not-deployed.patch | 38 +++---
 2 files changed, 37 insertions(+), 6 deletions(-)

diff --git a/meta/recipes-devtools/elfutils/elfutils_0.176.bb 
b/meta/recipes-devtools/elfutils/elfutils_0.176.bb
index 157551e..85ff5f3 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.176.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.176.bb
@@ -44,7 +44,7 @@ inherit autotools gettext ptest
 
 EXTRA_OECONF = "--program-prefix=eu- --without-lzma"
 EXTRA_OECONF_append_class-native = " --without-bzlib"
-RDEPENDS_${PN}-ptest = "libasm libelf bash make coreutils ${PN}-binutils ${PN}"
+RDEPENDS_${PN}-ptest = "libasm libelf bash make coreutils ${PN}-binutils ${PN} 
${PN}-dbg"
 
 EXTRA_OECONF_append_class-target += "--disable-tests-rpath"
 
@@ -61,6 +61,8 @@ do_compile_ptest() {
 
 do_install_ptest() {
if [ ${PTEST_ENABLED} = "1" ]; then
+   install -d -m 755   ${D}${PTEST_PATH}/src
+   cp -r ${B}/src/*${D}${PTEST_PATH}/src
cp -r ${S}/tests/   ${D}${PTEST_PATH}
cp -r ${B}/tests/*  ${D}${PTEST_PATH}/tests
cp -r ${B}/config.h ${D}${PTEST_PATH}
@@ -109,3 +111,4 @@ FILES_libdw  = "${libdir}/libdw-${PV}.so 
${libdir}/libdw.so.* ${libdir}/elfutils
 
 # The package contains symlinks that trip up insane
 INSANE_SKIP_${MLPREFIX}libdw = "dev-so"
+INSANE_SKIP_${PN}-ptest += "debug-deps"
diff --git 
a/meta/recipes-devtools/elfutils/files/0001-skip-the-test-when-gcc-not-deployed.patch
 
b/meta/recipes-devtools/elfutils/files/0001-skip-the-test-when-gcc-not-deployed.patch
index 67ca0e4..de8c05f 100644
--- 
a/meta/recipes-devtools/elfutils/files/0001-skip-the-test-when-gcc-not-deployed.patch
+++ 
b/meta/recipes-devtools/elfutils/files/0001-skip-the-test-when-gcc-not-deployed.patch
@@ -1,19 +1,47 @@
-From 61939d2e552e7645ecd671fa01cf1c7a72caa82a Mon Sep 17 00:00:00 2001
+From e82a055f85e398cb03a4eaf5faf351a3a1f19344 Mon Sep 17 00:00:00 2001
 From: Mingli Yu 
-Date: Fri, 12 Apr 2019 16:29:58 +0800
-Subject: [PATCH] skip the test when gcc not deployed
+Date: Tue, 21 May 2019 15:20:34 +0800
+Subject: [PATCH v2] skip the test when gcc not deployed
 
 Skip the tests which depend on gcc when
 gcc not deployed.
 
-Upstream-Status: Inappropriate [oe specific]
+Upstream-Status: 
Submitted[https://sourceware.org/ml/elfutils-devel/2019-q2/msg00091.html]
 
 Signed-off-by: Mingli Yu 
 ---
+ tests/run-disasm-x86-64.sh | 2 ++
+ tests/run-disasm-x86.sh| 2 ++
  tests/run-strip-g.sh   | 2 ++
  tests/run-strip-nothing.sh | 2 ++
- 2 files changed, 4 insertions(+)
+ 4 files changed, 8 insertions(+)
 
+diff --git a/tests/run-disasm-x86-64.sh b/tests/run-disasm-x86-64.sh
+index a6be62b..c3ef238 100755
+--- a/tests/run-disasm-x86-64.sh
 b/tests/run-disasm-x86-64.sh
+@@ -22,6 +22,8 @@ case "`uname -m`" in
+   x86_64)
+ tempfiles testfile45.o
+ testfiles testfile45.S testfile45.expect
++# skip the case if no gcc deployed
++which gcc || exit 77
+ gcc -m64 -c -o testfile45.o testfile45.S
+ testrun_compare ${abs_top_builddir}/src/objdump -d testfile45.o < 
testfile45.expect
+ ;;
+diff --git a/tests/run-disasm-x86.sh b/tests/run-disasm-x86.sh
+index 28a3df7..544fc28 100755
+--- a/tests/run-disasm-x86.sh
 b/tests/run-disasm-x86.sh
+@@ -22,6 +22,8 @@ case "`uname -m`" in
+   x86_64 | i?86 )
+ tempfiles testfile44.o
+ testfiles testfile44.S testfile44.expect
++# skip the case if no gcc deployed
++which gcc || exit 77
+ gcc -m32 -c -o testfile44.o testfile44.S
+ testrun_compare ${abs_top_builddir}/src/objdump -d testfile44.o < 
testfile44.expect
+ ;;
 diff --git a/tests/run-strip-g.sh b/tests/run-strip-g.sh
 index 1303819..a943dec 100755
 --- a/tests/run-strip-g.sh
-- 
2.7.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org

[OE-core] [PATCH 2/2] mdadm: define MAILADDR for mdmonitor service

[Liwei Song]

by default in mdmonitor.service it use mail address to report alerts
but this need to set MAILADDR in /etc/mdadm.conf, or the service
will be failed excuted.

This patch will set MAILADD to "root" in /etc/mdadm.conf
to aviod service run with error.

Signed-off-by: Liwei Song 
---
 meta/recipes-extended/mdadm/mdadm_4.1.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-extended/mdadm/mdadm_4.1.bb 
b/meta/recipes-extended/mdadm/mdadm_4.1.bb
index 8ca42a89ddc5..8ec1c22a0c62 100644
--- a/meta/recipes-extended/mdadm/mdadm_4.1.bb
+++ b/meta/recipes-extended/mdadm/mdadm_4.1.bb
@@ -57,6 +57,7 @@ do_install() {
 do_install_append() {
 install -d ${D}/${sysconfdir}/
 install -m 644 ${S}/mdadm.conf-example ${D}${sysconfdir}/mdadm.conf
+sed -i "s/#MAILADDR r...@mydomain.tld/MAILADDR root/" 
${D}${sysconfdir}/mdadm.conf
 install -d ${D}/${sysconfdir}/init.d
 install -m 755 ${WORKDIR}/mdadm.init ${D}${sysconfdir}/init.d/mdmonitor
 if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true', 'false', 
d)}; then
-- 
2.7.4

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

[OE-core] [PATCH 1/2] mdadm: guarded with systemd when install mdadm service

[Liwei Song]

install mdadm related service only when systemd was included
in DISTRO_FEATURES.

Signed-off-by: Liwei Song 
---
 meta/recipes-extended/mdadm/mdadm_4.1.bb | 10 --
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/meta/recipes-extended/mdadm/mdadm_4.1.bb 
b/meta/recipes-extended/mdadm/mdadm_4.1.bb
index 597faf787a67..8ca42a89ddc5 100644
--- a/meta/recipes-extended/mdadm/mdadm_4.1.bb
+++ b/meta/recipes-extended/mdadm/mdadm_4.1.bb
@@ -59,13 +59,11 @@ do_install_append() {
 install -m 644 ${S}/mdadm.conf-example ${D}${sysconfdir}/mdadm.conf
 install -d ${D}/${sysconfdir}/init.d
 install -m 755 ${WORKDIR}/mdadm.init ${D}${sysconfdir}/init.d/mdmonitor
+if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true', 'false', 
d)}; then
+   oe_runmake install-systemd DESTDIR=${D}
+fi
 }
 
-do_install_append() {
-oe_runmake install-systemd DESTDIR=${D}
-}
-
-
 do_compile_ptest() {
oe_runmake test
 }
@@ -93,4 +91,4 @@ RRECOMMENDS_${PN}-ptest += " \
 kernel-module-raid456 \
 "
 
-FILES_${PN} += "/lib/systemd/*"
+FILES_${PN} += 
"${@bb.utils.contains('DISTRO_FEATURES','systemd','/lib/systemd/*', '', d)}"
-- 
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] mdadm: install the systemd service through Makefile

[Liwei Song]

Please ignore this one.

Because the v1 patch has been applied, so this one is not applicable now,
will send it together with another patch to cover the systemd related change.

Thanks,
Liwei.



On 05/21/2019 04:54 PM, Liwei Song wrote:
> The original mdadm service may include variable like BINDIR,
> It should use the real value of it, since the install method
> "install-systemd" in Makefile can easily do such work, so don't
> install it in bb file manually, use "make install-systemd" to
> install all the service of mdadm.
> 
> Signed-off-by: Liwei Song 
> ---
>  meta/recipes-extended/mdadm/mdadm_4.1.bb | 8 +---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/meta/recipes-extended/mdadm/mdadm_4.1.bb 
> b/meta/recipes-extended/mdadm/mdadm_4.1.bb
> index ef5ddf55db39..8ca42a89ddc5 100644
> --- a/meta/recipes-extended/mdadm/mdadm_4.1.bb
> +++ b/meta/recipes-extended/mdadm/mdadm_4.1.bb
> @@ -57,11 +57,11 @@ do_install() {
>  do_install_append() {
>  install -d ${D}/${sysconfdir}/
>  install -m 644 ${S}/mdadm.conf-example ${D}${sysconfdir}/mdadm.conf
> -install -d ${D}/${systemd_unitdir}/system
> -install -m 644 ${WORKDIR}/mdmonitor.service 
> ${D}/${systemd_unitdir}/system
> -install -m 644 ${S}/systemd/mdmon@.service 
> ${D}/${systemd_unitdir}/system
>  install -d ${D}/${sysconfdir}/init.d
>  install -m 755 ${WORKDIR}/mdadm.init 
> ${D}${sysconfdir}/init.d/mdmonitor
> +if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true', 'false', 
> d)}; then
> +   oe_runmake install-systemd DESTDIR=${D}
> +fi
>  }
>  
>  do_compile_ptest() {
> @@ -90,3 +90,5 @@ RRECOMMENDS_${PN}-ptest += " \
>  kernel-module-raid10 \
>  kernel-module-raid456 \
>  "
> +
> +FILES_${PN} += 
> "${@bb.utils.contains('DISTRO_FEATURES','systemd','/lib/systemd/*', '', d)}"
> 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] squashfs-tools: Mark as incompatible with musl

[Adrian Bunk]

Silently ignoring FNM_EXTMATCH makes fnmatch()
behave different from what callers expect.

Mark as incompatible with musl instead of risking
broken runtime behaviour.

The practical consequences should be small since
what is used in OE is mainly squashfs-tools-native.

Signed-off-by: Adrian Bunk 
---
 .../squashfs-tools/fix-compat.patch   | 46 ---
 .../squashfs-tools/squashfs-tools_git.bb  |  3 ++
 2 files changed, 3 insertions(+), 46 deletions(-)

diff --git 
a/meta/recipes-devtools/squashfs-tools/squashfs-tools/fix-compat.patch 
b/meta/recipes-devtools/squashfs-tools/squashfs-tools/fix-compat.patch
index 0ce7b4e8ce..87c1e8cac0 100644
--- a/meta/recipes-devtools/squashfs-tools/squashfs-tools/fix-compat.patch
+++ b/meta/recipes-devtools/squashfs-tools/squashfs-tools/fix-compat.patch
@@ -1,39 +1,8 @@
-Define FNM_EXTMATCH if not defined its glibc specific define
 include missing sys/stat.h for stat* function declarations
 
 Upstream-Status: Pending
 Signed-off-by: Khem Raj 
 
-Index: squashfs-tools/action.c
-===
 squashfs-tools.orig/action.c
-+++ squashfs-tools/action.c
-@@ -44,6 +44,10 @@
- #include "action.h"
- #include "error.h"
- 
-+#if !defined(FNM_EXTMATCH)
-+#define FNM_EXTMATCH 0
-+#endif
-+
- /*
-  * code to parse actions
-  */
-Index: squashfs-tools/mksquashfs.c
-===
 squashfs-tools.orig/mksquashfs.c
-+++ squashfs-tools/mksquashfs.c
-@@ -1261,6 +1261,10 @@ void write_dir(squashfs_inode *inode, st
-   dir_size + 3, directory_block, directory_offset, NULL, NULL,
-   dir, 0);
- 
-+#if !defined(FNM_EXTMATCH)
-+#define FNM_EXTMATCH 0
-+#endif
-+
- #ifdef SQUASHFS_TRACE
-   {
-   unsigned char *dirp;
 Index: squashfs-tools/pseudo.c
 ===
 --- squashfs-tools.orig/pseudo.c
@@ -46,18 +15,3 @@ Index: squashfs-tools/pseudo.c
  #include 
  
  #include "pseudo.h"
-Index: squashfs-tools/unsquashfs.c
-===
 squashfs-tools.orig/unsquashfs.c
-+++ squashfs-tools/unsquashfs.c
-@@ -42,6 +42,10 @@
- #include 
- #endif
- 
-+#ifndef FNM_EXTMATCH
-+#define FNM_EXTMATCH 0
-+#endif
-+
- struct cache *fragment_cache, *data_cache;
- struct queue *to_reader, *to_inflate, *to_writer, *from_writer;
- pthread_t *thread, *inflator_thread;
diff --git a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb 
b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
index 5a8b5f77bd..dc1568a77c 100644
--- a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
+++ b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
@@ -19,6 +19,9 @@ SRC_URI[lzma.sha256sum] = 
"c935fd04dd8e0e8c688a3078f3675d699679a90be81c12686837e
 
 S = "${WORKDIR}/git/squashfs-tools"
 
+# needs FNM_EXTMATCH
+COMPATIBLE_HOST_libc-musl = 'null'
+
 EXTRA_OEMAKE = "${PACKAGECONFIG_CONFARGS}"
 
 PACKAGECONFIG ??= "gzip xz lzo lz4 lzma xattr"
-- 
2.17.1

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

[OE-core] [PATCH v2] meson.bbclass: Make meson support aarch64_be.

[Lei Maohui]

Added aarch64_be into cpu family.

Signed-off-by: Lei Maohui 
---
 meta/classes/meson.bbclass | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/classes/meson.bbclass b/meta/classes/meson.bbclass
index 115d1ae..0edbfc1 100644
--- a/meta/classes/meson.bbclass
+++ b/meta/classes/meson.bbclass
@@ -56,6 +56,8 @@ def meson_cpu_family(var, d):
 return 'ppc64'
 elif arch == 'armeb':
 return 'arm'
+elif arch == 'aarch64_be':
+return 'aarch64'
 elif arch == 'mipsel':
 return 'mips'
 elif arch == 'mips64el':
-- 
2.7.4



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

Re: [OE-core] [PATCH 3/3] elfutils: Fix eu-* utils builds for musl

[Adrian Bunk]

On Tue, May 21, 2019 at 06:32:28PM -0700, Khem Raj wrote:
>...
> PN and PN-binutils is not empty anymore on musl
>...
> ++#if !defined(FNM_EXTMATCH)
> ++# define FNM_EXTMATCH (1 << 5) /* Use ksh-like extended matching. */
> ++#endif
>...

This might fix the compilation, but it is horribly wrong.

It passes a flag to a function that this function does not understand.

In the best case the function returns with an error on unknown flags.

In the worst case the function does silently something different from
what the caller expects it to do.

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

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