[OE-core] [PATCH 1/1] gstreamer1.0-plugins-good: enable PACKAGECONFIG gtk conditionally

2018-05-16 Thread kai.kang
From: Kai Kang 

PACKAGECONFIG 'gtk' is enabled by default for gstreamer1.0-plugins-good
and it makes gstreamer1.0-plugins-good depends on package gtk+3 which
requires any distro feature in ${GTK3DISTROFEATURES}. When none distro
feature is enabled from ${GTK3DISTROFEATURES}, it causes error:

| ERROR: Nothing PROVIDES 'gtk+3' (but 
.../meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.14.0.bb
| DEPENDS on or otherwise requires it)
| gtk+3 was skipped: one of '['x11', 'wayland']' needs to be in DISTRO_FEATURES

So enable PACKAGECONFIG gtk conditionally for gstreamer1.0-plugins-good.

Signed-off-by: Kai Kang 
---
 meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.14.0.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.14.0.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.14.0.bb
index a420039fa85..b269fce1735 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.14.0.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.14.0.bb
@@ -27,7 +27,8 @@ inherit gettext
 PACKAGECONFIG ??= " \
 ${GSTREAMER_ORC} \
 ${@bb.utils.filter('DISTRO_FEATURES', 'pulseaudio x11', d)} \
-cairo flac gdk-pixbuf gudev jpeg libpng soup speex taglib v4l2 bz2 zlib 
gtk mpg123 lame \
+${@bb.utils.contains_any('DISTRO_FEATURES', 
d.getVar('GTK3DISTROFEATURES'), 'gtk', '', d)} \
+cairo flac gdk-pixbuf gudev jpeg libpng soup speex taglib v4l2 bz2 zlib 
mpg123 lame \
 "
 
 X11DEPENDS = "virtual/libx11 libsm libxrender libxfixes libxdamage"
-- 
2.14.1

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


[OE-core] [PATCH v2 3/3] rootfs.py: for dpkg/opkg, don't install postinsts if package management is present

2018-05-16 Thread Stefan Agner
From: Stefan Agner 

If package management is present opkg/dpkg will bring the original
copy of the postinsts scripts with the metadata and will be able to
handle postinsts just fine. In fact, it is preferred to let package
management handle the postinsts scripts in this case since it will
keep the package managers database up-to-date too. The run-postinsts
scripts will make sure the package manager gets invoked instead of
the scripts directly.

Note: Before commit 5aae19959a44 ("rootfs.py: Change logic to
unistall packages") rootfs.py did not install /etc/$pm-postinsts
too. It is not clear whether that change was intentionally or just
a bug. This commit fixes/reverts that aspect of the commit.

Signed-off-by: Stefan Agner 
---
 meta/lib/oe/rootfs.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/lib/oe/rootfs.py b/meta/lib/oe/rootfs.py
index f8f717c050..0a57d87f71 100644
--- a/meta/lib/oe/rootfs.py
+++ b/meta/lib/oe/rootfs.py
@@ -560,6 +560,9 @@ class DpkgOpkgRootfs(Rootfs):
 return pkg_list
 
 def _save_postinsts_common(self, dst_postinst_dir, src_postinst_dir):
+if bb.utils.contains("IMAGE_FEATURES", "package-management",
+ True, False, self.d):
+return
 num = 0
 for p in self._get_delayed_postinsts():
 bb.utils.mkdirhier(dst_postinst_dir)
-- 
2.13.6

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


[OE-core] [PATCH v2 2/3] run-postinsts: for dpkg/opkg, do not rely on /etc/*-postinsts

2018-05-16 Thread Stefan Agner
From: Stefan Agner 

Start opkg/dpkg as soon as the respective package managers status
file is present, no matter whether /etc/$pm-postinsts exists. This
decouples the implicit link between postinsts scripts in /etc and
the package manager: Currently the package manager is only started
if those scripts are present, although the package manager does not
use those scripts at all! Package managers install their own set of
postinst scripts.

The behavior when using rpm packages stays the same.

Note that using the package managers capability to execute postinst
scripts is preferred for good reasons: It makes sure that the
package managers database reflects that the packages have been
completely installed and configured.

This change allows to drop installation of the postinsts scripts
when package management is present. This will be done in a separate
change.

Note: Before commit 5aae19959a44 ("rootfs.py: Change logic to
unistall packages") rootfs.py did not install /etc/$pm-postinsts
when package management is installed! The change caused YOCTO #8235
which lead to the behavior change of run-postinsts in first place.

Signed-off-by: Stefan Agner 
---
 .../run-postinsts/run-postinsts/run-postinsts   | 21 -
 .../run-postinsts/run-postinsts.service |  1 -
 2 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts 
b/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts
index 307feb7187..95eff04e17 100755
--- a/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts
+++ b/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts
@@ -6,7 +6,8 @@
 #
 
 # The following script will run all the scriptlets found in 
#SYSCONFDIR#/deb-postinsts,
-# #SYSCONFDIR#/ipk-postinsts or #SYSCONFDIR#/rpm-postinsts.
+# #SYSCONFDIR#/ipk-postinsts or #SYSCONFDIR#/rpm-postinsts or the package 
manager in
+# case available.
 
 # the order of this list is important, do not change!
 backend_list="rpm deb ipk"
@@ -14,27 +15,29 @@ backend_list="rpm deb ipk"
 pm_installed=false
 
 for pm in $backend_list; do
-   pi_dir="#SYSCONFDIR#/$pm-postinsts"
-
-   if [ ! -d $pi_dir ]; then
-   continue
-   fi
-
# found the package manager, it has postinsts
case $pm in
"deb")
if [ -s "#LOCALSTATEDIR#/lib/dpkg/status" ]; then
pm_installed=true
+   break
fi
;;
 
"ipk")
if [ -s "#LOCALSTATEDIR#/lib/opkg/status" ]; then
pm_installed=true
+   break
fi
;;
esac
-   break
+
+   pi_dir="#SYSCONFDIR#/$pm-postinsts"
+
+   # found postinsts directory
+   if [ -d $pi_dir ]; then
+   break
+   fi
 done
 
 remove_rcsd_link () {
@@ -43,7 +46,7 @@ remove_rcsd_link () {
fi
 }
 
-if ! [ -d $pi_dir ]; then
+if ! [ -d $pi_dir ] && ! $pm_installed; then
remove_rcsd_link
exit 0
 fi
diff --git 
a/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts.service 
b/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts.service
index 1b71a1f8be..d42addf510 100644
--- a/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts.service
+++ b/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts.service
@@ -3,7 +3,6 @@ Description=Run pending postinsts
 DefaultDependencies=no
 After=systemd-remount-fs.service systemd-tmpfiles-setup.service tmp.mount
 Before=sysinit.target
-ConditionPathExistsGlob=#SYSCONFDIR#/*-postinsts
 
 [Service]
 Type=oneshot
-- 
2.13.6

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


[OE-core] [PATCH v2 1/3] opkg: avoid running postinst scripts twice when using systemd

2018-05-16 Thread Stefan Agner
From: Stefan Agner 

OpenEmbedded has a built-in mechanism to run postinst scripts offline
at build time or, if necessary, on first boot (delayed execution). If
the latter is the case and systemd is in use, two services end up
doing the same thing:
- opkg-configure.service starts "opkg configure" directly.
- run-postinsts.service starts "/usr/sbin/run-postinsts" which runs
  postinst scripts stored in /etc/ipk-postinsts/ or "opkg configure"
  if package management is installed.

Since the run-postinsts.service is also used in cases where no
package management is in use, it is the primary means of handling
postinsts.

Get rid of the opkg-configure.service to avoid duplicate opkg
configure execution.

Signed-off-by: Stefan Agner 
---
 meta/recipes-devtools/opkg/opkg/opkg-configure.service | 17 -
 meta/recipes-devtools/opkg/opkg_0.3.6.bb   | 14 --
 2 files changed, 31 deletions(-)
 delete mode 100644 meta/recipes-devtools/opkg/opkg/opkg-configure.service

diff --git a/meta/recipes-devtools/opkg/opkg/opkg-configure.service 
b/meta/recipes-devtools/opkg/opkg/opkg-configure.service
deleted file mode 100644
index 432c3ddc28..00
--- a/meta/recipes-devtools/opkg/opkg/opkg-configure.service
+++ /dev/null
@@ -1,17 +0,0 @@
-[Unit]
-Description=Opkg first boot configure
-DefaultDependencies=no
-After=systemd-remount-fs.service systemd-tmpfiles-setup.service tmp.mount
-Before=sysinit.target
-
-[Service]
-Type=oneshot
-EnvironmentFile=-@SYSCONFDIR@/default/postinst
-ExecStart=-@BASE_BINDIR@/sh -c " if [ $POSTINST_LOGGING = '1' ]; then 
@BINDIR@/opkg configure > $LOGFILE 2>&1; else @BINDIR@/opkg configure; fi"
-ExecStartPost=@BASE_BINDIR@/systemctl --no-reload disable 
opkg-configure.service
-StandardOutput=syslog
-RemainAfterExit=No
-
-[Install]
-WantedBy=basic.target
-WantedBy=sysinit.target
diff --git a/meta/recipes-devtools/opkg/opkg_0.3.6.bb 
b/meta/recipes-devtools/opkg/opkg_0.3.6.bb
index 70f20af739..579b51166c 100644
--- a/meta/recipes-devtools/opkg/opkg_0.3.6.bb
+++ b/meta/recipes-devtools/opkg/opkg_0.3.6.bb
@@ -12,7 +12,6 @@ DEPENDS = "libarchive"
 PE = "1"
 
 SRC_URI = 
"http://downloads.yoctoproject.org/releases/${BPN}/${BPN}-${PV}.tar.gz \
-   file://opkg-configure.service \
file://opkg.conf \

file://0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch \
 "
@@ -22,8 +21,6 @@ SRC_URI[sha256sum] = 
"f607f0e61be8cf8a3bbd0d2dccd9ec9e9b6c21dd4307b671c600d6eeaf
 
 inherit autotools pkgconfig systemd
 
-SYSTEMD_SERVICE_${PN} = "opkg-configure.service"
-
 target_localstatedir := "${localstatedir}"
 OPKGLIBDIR = "${target_localstatedir}/lib"
 
@@ -46,16 +43,6 @@ do_install_append () {
 
# We need to create the lock directory
install -d ${D}${OPKGLIBDIR}/opkg
-
-   if 
${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)};then
-   install -d ${D}${systemd_unitdir}/system
-   install -m 0644 ${WORKDIR}/opkg-configure.service 
${D}${systemd_unitdir}/system/
-   sed -i -e 's,@BASE_BINDIR@,${base_bindir},g' \
-   -e 's,@SYSCONFDIR@,${sysconfdir},g' \
-   -e 's,@BINDIR@,${bindir},g' \
-   -e 's,@SYSTEMD_UNITDIR@,${systemd_unitdir},g' \
-   ${D}${systemd_unitdir}/system/opkg-configure.service
-   fi
 }
 
 RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_update-alternatives} opkg-arch-config 
libarchive"
@@ -68,7 +55,6 @@ RPROVIDES_${PN} = "opkg-collateral"
 PACKAGES =+ "libopkg"
 
 FILES_libopkg = "${libdir}/*.so.* ${OPKGLIBDIR}/opkg/"
-FILES_${PN} += "${systemd_unitdir}/system/"
 
 BBCLASSEXTEND = "native nativesdk"
 
-- 
2.13.6

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


[OE-core] [PATCH 1/4] gdb: Upgrade to 8.1 release

2018-05-16 Thread Khem Raj
cache strerror so it does not try to use rpl_* versions

Signed-off-by: Khem Raj 
---
 meta/conf/distro/include/tcmode-default.inc   |   2 +-
 .../gdb/{gdb-8.0.1.inc => gdb-8.1.inc}|   5 +-
 ...ian_8.0.1.bb => gdb-cross-canadian_8.1.bb} |   0
 .../{gdb-cross_8.0.1.bb => gdb-cross_8.1.bb}  |   0
 .../0001-include-sys-types.h-for-mode_t.patch |  12 +-
 ...port-for-Renesas-SH-sh4-architecture.patch | 153 --
 ...0012-Unbreak-GDBserver-build-for-x32.patch | 101 
 .../gdb/{gdb_8.0.1.bb => gdb_8.1.bb}  |   2 +
 8 files changed, 82 insertions(+), 193 deletions(-)
 rename meta/recipes-devtools/gdb/{gdb-8.0.1.inc => gdb-8.1.inc} (85%)
 rename meta/recipes-devtools/gdb/{gdb-cross-canadian_8.0.1.bb => 
gdb-cross-canadian_8.1.bb} (100%)
 rename meta/recipes-devtools/gdb/{gdb-cross_8.0.1.bb => gdb-cross_8.1.bb} 
(100%)
 delete mode 100644 
meta/recipes-devtools/gdb/gdb/0012-Unbreak-GDBserver-build-for-x32.patch
 rename meta/recipes-devtools/gdb/{gdb_8.0.1.bb => gdb_8.1.bb} (86%)

diff --git a/meta/conf/distro/include/tcmode-default.inc 
b/meta/conf/distro/include/tcmode-default.inc
index 00399ae820..4b58df0459 100644
--- a/meta/conf/distro/include/tcmode-default.inc
+++ b/meta/conf/distro/include/tcmode-default.inc
@@ -25,7 +25,7 @@ PREFERRED_PROVIDER_virtual/gettext ??= "gettext"
 GCCVERSION ?= "7.%"
 SDKGCCVERSION ?= "${GCCVERSION}"
 BINUVERSION ?= "2.30%"
-GDBVERSION ?= "8.0%"
+GDBVERSION ?= "8.1%"
 GLIBCVERSION ?= "2.27%"
 LINUXLIBCVERSION ?= "4.15%"
 QEMUVERSION ?= "2.11%"
diff --git a/meta/recipes-devtools/gdb/gdb-8.0.1.inc 
b/meta/recipes-devtools/gdb/gdb-8.1.inc
similarity index 85%
rename from meta/recipes-devtools/gdb/gdb-8.0.1.inc
rename to meta/recipes-devtools/gdb/gdb-8.1.inc
index 4a5299b73f..4d9faa2d4b 100644
--- a/meta/recipes-devtools/gdb/gdb-8.0.1.inc
+++ b/meta/recipes-devtools/gdb/gdb-8.1.inc
@@ -16,9 +16,8 @@ SRC_URI = "http://ftp.gnu.org/gnu/gdb/gdb-${PV}.tar.xz \
file://0009-Change-order-of-CFLAGS.patch \
file://0010-resolve-restrict-keyword-conflict.patch \
file://package_devel_gdb_patches_120-sigprocmask-invalid-call.patch 
\
-   file://0012-Unbreak-GDBserver-build-for-x32.patch \
file://gdb-Fix-ia64-defining-TRAP_HWBKPT-before-including-g.patch \
 "
-SRC_URI[md5sum] = "48cac527e6f3018b865ece021e9723ac"
-SRC_URI[sha256sum] = 
"3dbd5f93e36ba2815ad0efab030dcd0c7b211d7b353a40a53f4c02d7d56295e3"
+SRC_URI[md5sum] = "f46487561f9a16916a8102316f7fd105"
+SRC_URI[sha256sum] = 
"af61a0263858e69c5dce51eab26662ff3d2ad9aa68da9583e8143b5426be4b34"
 
diff --git a/meta/recipes-devtools/gdb/gdb-cross-canadian_8.0.1.bb 
b/meta/recipes-devtools/gdb/gdb-cross-canadian_8.1.bb
similarity index 100%
rename from meta/recipes-devtools/gdb/gdb-cross-canadian_8.0.1.bb
rename to meta/recipes-devtools/gdb/gdb-cross-canadian_8.1.bb
diff --git a/meta/recipes-devtools/gdb/gdb-cross_8.0.1.bb 
b/meta/recipes-devtools/gdb/gdb-cross_8.1.bb
similarity index 100%
rename from meta/recipes-devtools/gdb/gdb-cross_8.0.1.bb
rename to meta/recipes-devtools/gdb/gdb-cross_8.1.bb
diff --git 
a/meta/recipes-devtools/gdb/gdb/0001-include-sys-types.h-for-mode_t.patch 
b/meta/recipes-devtools/gdb/gdb/0001-include-sys-types.h-for-mode_t.patch
index 4f06d46d70..5d19380e30 100644
--- a/meta/recipes-devtools/gdb/gdb/0001-include-sys-types.h-for-mode_t.patch
+++ b/meta/recipes-devtools/gdb/gdb/0001-include-sys-types.h-for-mode_t.patch
@@ -1,7 +1,7 @@
-From 91da0458b333249eb9c2f4c1f1e53fa4bc085cc9 Mon Sep 17 00:00:00 2001
+From ff37f9360ab6f841f28ced9f23f66542febcc0f6 Mon Sep 17 00:00:00 2001
 From: Khem Raj 
 Date: Tue, 19 Jan 2016 18:18:52 -0800
-Subject: [PATCH 01/10] include sys/types.h for mode_t
+Subject: [PATCH 01/11] include sys/types.h for mode_t
 
 mode_t is used in target.h, so we need to include sys/types.h to get the
 defintion
@@ -14,17 +14,17 @@ Upstream-Status: Pending
  1 file changed, 1 insertion(+)
 
 diff --git a/gdb/gdbserver/target.h b/gdb/gdbserver/target.h
-index 3cc2bc4bab..e6b19b06b9 100644
+index 25accd2207..7f6bca8aa4 100644
 --- a/gdb/gdbserver/target.h
 +++ b/gdb/gdbserver/target.h
-@@ -28,6 +28,7 @@
- #include "target/waitstatus.h"
+@@ -29,6 +29,7 @@
  #include "mem-break.h"
  #include "btrace-common.h"
+ #include 
 +#include 
  
  struct emit_ops;
  struct buffer;
 -- 
-2.13.1
+2.17.0
 
diff --git 
a/meta/recipes-devtools/gdb/gdb/0005-Add-support-for-Renesas-SH-sh4-architecture.patch
 
b/meta/recipes-devtools/gdb/gdb/0005-Add-support-for-Renesas-SH-sh4-architecture.patch
index cb1b7abd07..f678598284 100644
--- 
a/meta/recipes-devtools/gdb/gdb/0005-Add-support-for-Renesas-SH-sh4-architecture.patch
+++ 
b/meta/recipes-devtools/gdb/gdb/0005-Add-support-for-Renesas-SH-sh4-architecture.patch
@@ -1,7 +1,7 @@
-From 60ac68f601885ea6480229a5c8a89a0257da376c Mon Sep 17 00:00:00 2001
+From 8757b36be6109f6d7ea0bd8dafbaed647e0d2192 Mon Sep 17 00:00:00 2001
 From: Khem Raj 
 

[OE-core] [PATCH 2/4] python2: Fix build with gcc8

2018-05-16 Thread Khem Raj
From: Ross Burton 

Signed-off-by: Ross Burton 
Signed-off-by: Khem Raj 
---
 .../python/python-native_2.7.14.bb|  1 +
 .../python/python/fix-gc-alignment.patch  | 43 +++
 meta/recipes-devtools/python/python_2.7.14.bb |  1 +
 3 files changed, 45 insertions(+)
 create mode 100644 meta/recipes-devtools/python/python/fix-gc-alignment.patch

diff --git a/meta/recipes-devtools/python/python-native_2.7.14.bb 
b/meta/recipes-devtools/python/python-native_2.7.14.bb
index 7c491fa3e0..b822583c53 100644
--- a/meta/recipes-devtools/python/python-native_2.7.14.bb
+++ b/meta/recipes-devtools/python/python-native_2.7.14.bb
@@ -16,6 +16,7 @@ SRC_URI += "\
 file://builddir.patch \
 file://parallel-makeinst-create-bindir.patch \
 file://revert_use_of_sysconfigdata.patch \
+file://fix-gc-alignment.patch \
"
 
 S = "${WORKDIR}/Python-${PV}"
diff --git a/meta/recipes-devtools/python/python/fix-gc-alignment.patch 
b/meta/recipes-devtools/python/python/fix-gc-alignment.patch
new file mode 100644
index 00..b63cd08747
--- /dev/null
+++ b/meta/recipes-devtools/python/python/fix-gc-alignment.patch
@@ -0,0 +1,43 @@
+Upstream-Status: Submitted
+Signed-off-by: Ross Burton 
+
+Fix for over-aligned GC info
+Patch by Florian Weimer
+
+See: https://bugzilla.redhat.com/show_bug.cgi?id=1540316
+Upstream discussion: 
https://mail.python.org/pipermail/python-dev/2018-January/152000.html
+
+diff --git a/Include/objimpl.h b/Include/objimpl.h
+index 55e83eced6..aa906144dc 100644
+--- a/Include/objimpl.h
 b/Include/objimpl.h
+@@ -248,6 +248,18 @@ PyAPI_FUNC(PyVarObject *) _PyObject_GC_Resize(PyVarObject 
*, Py_ssize_t);
+ /* for source compatibility with 2.2 */
+ #define _PyObject_GC_Del PyObject_GC_Del
+ 
++/* Former over-aligned definition of PyGC_Head, used to compute the
++   size of the padding for the new version below. */
++union _gc_head;
++union _gc_head_old {
++struct {
++union _gc_head *gc_next;
++union _gc_head *gc_prev;
++Py_ssize_t gc_refs;
++} gc;
++long double dummy;
++};
++
+ /* GC information is stored BEFORE the object structure. */
+ typedef union _gc_head {
+ struct {
+@@ -255,7 +267,8 @@ typedef union _gc_head {
+ union _gc_head *gc_prev;
+ Py_ssize_t gc_refs;
+ } gc;
+-long double dummy;  /* force worst-case alignment */
++double dummy;  /* force worst-case alignment */
++char dummy_padding[sizeof(union _gc_head_old)];
+ } PyGC_Head;
+ 
+ extern PyGC_Head *_PyGC_generation0;
+ 
\ No newline at end of file
diff --git a/meta/recipes-devtools/python/python_2.7.14.bb 
b/meta/recipes-devtools/python/python_2.7.14.bb
index 0a09926dfb..41a8609b15 100644
--- a/meta/recipes-devtools/python/python_2.7.14.bb
+++ b/meta/recipes-devtools/python/python_2.7.14.bb
@@ -30,6 +30,7 @@ SRC_URI += "\
   file://add-CROSSPYTHONPATH-for-PYTHON_FOR_BUILD.patch \
   file://pass-missing-libraries-to-Extension-for-mul.patch \
   file://support_SOURCE_DATE_EPOCH_in_py_compile_2.7.patch \
+  file://fix-gc-alignment.patch \
 "
 
 S = "${WORKDIR}/Python-${PV}"
-- 
2.17.0

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


[OE-core] [PATCH 0/4] gdb upgrade and misc fixes to enable gcc8

2018-05-16 Thread Khem Raj
The following changes since commit 9611485bba03ef77ff31121e3b1da7cd57990c3e:

  patchreview: don't disable malformed SoB check (2018-05-15 10:00:33 +0100)

are available in the Git repository at:

  git://git.openembedded.org/openembedded-core-contrib kraj/pu
  http://cgit.openembedded.org/openembedded-core-contrib/log/?h=kraj/pu

Khem Raj (3):
  gdb: Upgrade to 8.1 release
  ovmf: Fix build with gcc8
  musl: Make dev package rdep on staticdev

Ross Burton (1):
  python2: Fix build with gcc8

 meta/conf/distro/include/tcmode-default.inc   |   2 +-
 meta/recipes-core/musl/musl_git.bb|   2 +-
 makefile-add-Wno-stringop-truncatio.patch |  73 +
 ...ols-header.makefile-add-Wno-restrict.patch | 104 
 makefile-revert-gcc-8-Wno-xxx-optio.patch |  55 +++
 ...-silence-false-stringop-overflow-war.patch |  65 
 meta/recipes-core/ovmf/ovmf_git.bb|   4 +
 .../gdb/{gdb-8.0.1.inc => gdb-8.1.inc}|   5 +-
 ...ian_8.0.1.bb => gdb-cross-canadian_8.1.bb} |   0
 .../{gdb-cross_8.0.1.bb => gdb-cross_8.1.bb}  |   0
 .../0001-include-sys-types.h-for-mode_t.patch |  12 +-
 ...port-for-Renesas-SH-sh4-architecture.patch | 153 --
 ...0012-Unbreak-GDBserver-build-for-x32.patch | 101 
 .../gdb/{gdb_8.0.1.bb => gdb_8.1.bb}  |   2 +
 .../python/python-native_2.7.14.bb|   1 +
 .../python/python/fix-gc-alignment.patch  |  43 +
 meta/recipes-devtools/python/python_2.7.14.bb |   1 +
 17 files changed, 429 insertions(+), 194 deletions(-)
 create mode 100644 
meta/recipes-core/ovmf/ovmf/0001-BaseTools-header.makefile-add-Wno-stringop-truncatio.patch
 create mode 100644 
meta/recipes-core/ovmf/ovmf/0002-BaseTools-header.makefile-add-Wno-restrict.patch
 create mode 100644 
meta/recipes-core/ovmf/ovmf/0003-BaseTools-header.makefile-revert-gcc-8-Wno-xxx-optio.patch
 create mode 100644 
meta/recipes-core/ovmf/ovmf/0004-BaseTools-GenVtf-silence-false-stringop-overflow-war.patch
 rename meta/recipes-devtools/gdb/{gdb-8.0.1.inc => gdb-8.1.inc} (85%)
 rename meta/recipes-devtools/gdb/{gdb-cross-canadian_8.0.1.bb => 
gdb-cross-canadian_8.1.bb} (100%)
 rename meta/recipes-devtools/gdb/{gdb-cross_8.0.1.bb => gdb-cross_8.1.bb} 
(100%)
 delete mode 100644 
meta/recipes-devtools/gdb/gdb/0012-Unbreak-GDBserver-build-for-x32.patch
 rename meta/recipes-devtools/gdb/{gdb_8.0.1.bb => gdb_8.1.bb} (86%)
 create mode 100644 meta/recipes-devtools/python/python/fix-gc-alignment.patch

-- 
2.17.0

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


Re: [OE-core] [PATCH v2] rootfs-postcommands: put image testdata under sstate control

2018-05-16 Thread André Draszik
Is there anything wrong with this?

On Thu, 2018-05-10 at 09:05 +0100, André Draszik wrote:
> ping
> 
> On Wed, 2018-03-21 at 16:31 +, André Draszik wrote:
> > From: André Draszik 
> > 
> > The testdata.json is being written to DEPLOY_DIR_IMAGE directly,
> > thus bypassing sstate, which results in an ever growing list
> > of files.
> > 
> > Write them to IMGDEPLOYDIR instead, so as to benefit from the
> > automatic management via sstate.
> > 
> > Signed-off-by: André Draszik 
> > 
> > ---
> > v2: fix typo in subject line
> > ---
> >  meta/classes/rootfs-postcommands.bbclass | 15 ---
> >  1 file changed, 8 insertions(+), 7 deletions(-)
> > 
> > diff --git a/meta/classes/rootfs-postcommands.bbclass
> > b/meta/classes/rootfs-postcommands.bbclass
> > index a4e627fef8..01eee350b8 100644
> > --- a/meta/classes/rootfs-postcommands.bbclass
> > +++ b/meta/classes/rootfs-postcommands.bbclass
> > @@ -303,17 +303,18 @@ rootfs_sysroot_relativelinks () {
> >  python write_image_test_data() {
> >  from oe.data import export2json
> >  
> > -testdata = "%s/%s.testdata.json" % (d.getVar('DEPLOY_DIR_IMAGE'),
> > d.getVar('IMAGE_NAME'))
> > -testdata_link = "%s/%s.testdata.json" %
> > (d.getVar('DEPLOY_DIR_IMAGE'), d.getVar('IMAGE_LINK_NAME'))
> > +deploy_dir = d.getVar('IMGDEPLOYDIR')
> > +link_name = d.getVar('IMAGE_LINK_NAME')
> > +testdata_name = os.path.join(deploy_dir, "%s.testdata.json" %
> > d.getVar('IMAGE_NAME'))
> >  
> > -bb.utils.mkdirhier(os.path.dirname(testdata))
> >  searchString = "%s/"%(d.getVar("TOPDIR")).replace("//","/")
> > -export2json(d, testdata,searchString=searchString,replaceString="")
> > +export2json(d, testdata_name, searchString=searchString,
> > replaceString="")
> >  
> > -if testdata_link != testdata:
> > +if os.path.exists(testdata_name):
> > +testdata_link = os.path.join(deploy_dir, "%s.testdata.json" %
> > link_name)
> >  if os.path.lexists(testdata_link):
> > -   os.remove(testdata_link)
> > -os.symlink(os.path.basename(testdata), testdata_link)
> > +os.remove(testdata_link)
> > +os.symlink(os.path.basename(testdata_name), testdata_link)
> >  }
> >  write_image_test_data[vardepsexclude] += "TOPDIR"
> >  
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 3/4] ovmf: Fix build with gcc8

2018-05-16 Thread Khem Raj
Signed-off-by: Khem Raj 
---
 makefile-add-Wno-stringop-truncatio.patch |  73 
 ...ols-header.makefile-add-Wno-restrict.patch | 104 ++
 makefile-revert-gcc-8-Wno-xxx-optio.patch |  55 +
 ...-silence-false-stringop-overflow-war.patch |  65 +++
 meta/recipes-core/ovmf/ovmf_git.bb|   4 +
 5 files changed, 301 insertions(+)
 create mode 100644 
meta/recipes-core/ovmf/ovmf/0001-BaseTools-header.makefile-add-Wno-stringop-truncatio.patch
 create mode 100644 
meta/recipes-core/ovmf/ovmf/0002-BaseTools-header.makefile-add-Wno-restrict.patch
 create mode 100644 
meta/recipes-core/ovmf/ovmf/0003-BaseTools-header.makefile-revert-gcc-8-Wno-xxx-optio.patch
 create mode 100644 
meta/recipes-core/ovmf/ovmf/0004-BaseTools-GenVtf-silence-false-stringop-overflow-war.patch

diff --git 
a/meta/recipes-core/ovmf/ovmf/0001-BaseTools-header.makefile-add-Wno-stringop-truncatio.patch
 
b/meta/recipes-core/ovmf/ovmf/0001-BaseTools-header.makefile-add-Wno-stringop-truncatio.patch
new file mode 100644
index 00..1d1679fe7e
--- /dev/null
+++ 
b/meta/recipes-core/ovmf/ovmf/0001-BaseTools-header.makefile-add-Wno-stringop-truncatio.patch
@@ -0,0 +1,73 @@
+From 9fce4bab014b9aa618060eba13d6dd04b0fa1b70 Mon Sep 17 00:00:00 2001
+From: Laszlo Ersek 
+Date: Fri, 2 Mar 2018 17:11:52 +0100
+Subject: [PATCH 1/4] BaseTools/header.makefile: add "-Wno-stringop-truncation"
+
+gcc-8 (which is part of Fedora 28) enables the new warning
+"-Wstringop-truncation" in "-Wall". This warning is documented in detail
+at ; the
+introduction says
+
+> Warn for calls to bounded string manipulation functions such as strncat,
+> strncpy, and stpncpy that may either truncate the copied string or leave
+> the destination unchanged.
+
+It breaks the BaseTools build with:
+
+> EfiUtilityMsgs.c: In function 'PrintMessage':
+> EfiUtilityMsgs.c:484:9: error: 'strncat' output may be truncated copying
+> between 0 and 511 bytes from a string of length 511
+> [-Werror=stringop-truncation]
+>  strncat (Line, Line2, MAX_LINE_LEN - strlen (Line) - 1);
+>  ^~~
+> EfiUtilityMsgs.c:469:9: error: 'strncat' output may be truncated copying
+> between 0 and 511 bytes from a string of length 511
+> [-Werror=stringop-truncation]
+>  strncat (Line, Line2, MAX_LINE_LEN - strlen (Line) - 1);
+>  ^~~
+> EfiUtilityMsgs.c:511:5: error: 'strncat' output may be truncated copying
+> between 0 and 511 bytes from a string of length 511
+> [-Werror=stringop-truncation]
+>  strncat (Line, Line2, MAX_LINE_LEN - strlen (Line) - 1);
+>  ^~~
+
+The right way to fix the warning would be to implement string concat with
+snprintf(). However, Microsoft does not appear to support snprintf()
+before VS2015
+,
+so we just have to shut up the warning. The strncat() calls flagged above
+are valid BTW.
+
+Cc: Ard Biesheuvel 
+Cc: Cole Robinson 
+Cc: Liming Gao 
+Cc: Paolo Bonzini 
+Cc: Yonghong Zhu 
+Contributed-under: TianoCore Contribution Agreement 1.1
+Signed-off-by: Laszlo Ersek 
+Reviewed-by: Liming Gao 
+---
+Upstream-Status: Backport
+
+ BaseTools/Source/C/Makefiles/header.makefile | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/BaseTools/Source/C/Makefiles/header.makefile 
b/BaseTools/Source/C/Makefiles/header.makefile
+index 063982b82f..6c3826aecb 100644
+--- a/BaseTools/Source/C/Makefiles/header.makefile
 b/BaseTools/Source/C/Makefiles/header.makefile
+@@ -47,9 +47,9 @@ INCLUDE = $(TOOL_INCLUDE) -I $(MAKEROOT) -I 
$(MAKEROOT)/Include/Common -I $(MAKE
+ BUILD_CPPFLAGS = $(INCLUDE) -O2
+ ifeq ($(DARWIN),Darwin)
+ # assume clang or clang compatible flags on OS X
+-BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror 
-Wno-deprecated-declarations -Wno-self-assign -Wno-unused-result -nostdlib -c -g
++BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror 
-Wno-deprecated-declarations -Wno-stringop-truncation -Wno-self-assign 
-Wno-unused-result -nostdlib -c -g
+ else
+-BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror 
-Wno-deprecated-declarations -Wno-unused-result -nostdlib -c -g
++BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror 
-Wno-deprecated-declarations -Wno-stringop-truncation -Wno-unused-result 
-nostdlib -c -g
+ endif
+ BUILD_LFLAGS =
+ BUILD_CXXFLAGS = -Wno-unused-result
+-- 
+2.17.0
+
diff --git 
a/meta/recipes-core/ovmf/ovmf/0002-BaseTools-header.makefile-add-Wno-restrict.patch
 

[OE-core] [PATCH 4/4] musl: Make dev package rdep on staticdev

2018-05-16 Thread Khem Raj
This is needed for enabling static linking with SDK

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

diff --git a/meta/recipes-core/musl/musl_git.bb 
b/meta/recipes-core/musl/musl_git.bb
index a0a098f5b4..d7d201df9d 100644
--- a/meta/recipes-core/musl/musl_git.bb
+++ b/meta/recipes-core/musl/musl_git.bb
@@ -66,7 +66,7 @@ do_install() {
done
 }
 
-RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
libssp-nonshared-staticdev"
+RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
libssp-nonshared-staticdev ${PN}-staticdev"
 RPROVIDES_${PN}-dev += "libc-dev virtual-libc-dev"
 RPROVIDES_${PN} += "ldd libsegfault rtld(GNU_HASH)"
 
-- 
2.17.0

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


[OE-core] [PATCH v2 0/3] postinst fixes for opgk/dpkg

2018-05-16 Thread Stefan Agner
From: Stefan Agner 

Hi,

This follows up on the discussion a while ago:
https://www.mail-archive.com/openembedded-core@lists.openembedded.org/msg104996.html

Patch 1 is rather simple and really fixes the main issue. The patch
by itself has been tested with the relevant self test and passes.

Patch 2/3 get rid of /etc/*-postinsts script in cases where the package
management is present. This avoids a bunch of unnecessary scripts to be
present on the rootfs. It also avoids the systemd service to be present
forever with in case no postinst scripts have been deployed:
Condition: start condition failed at Tue 2018-05-15 10:57:43 UTC; 42s ago
   └─ ConditionPathExistsGlob=/etc/*-postinsts was not met

Self test executed using:
$ oe-selftest --run-tests runtime_test.Postinst.test_postinst_rootfs_and_boot

Changes since v1:
- Note that patches are specific for opkg/dpkg

Stefan Agner (3):
  opkg: avoid running postinst scripts twice when using systemd
  run-postinsts: for dpkg/opkg, do not rely on /etc/*-postinsts
  rootfs.py: for dpkg/opkg, don't install postinsts if package
management is present

 meta/lib/oe/rootfs.py   |  3 +++
 .../opkg/opkg/opkg-configure.service| 17 -
 meta/recipes-devtools/opkg/opkg_0.3.6.bb| 14 --
 .../run-postinsts/run-postinsts/run-postinsts   | 21 -
 .../run-postinsts/run-postinsts.service |  1 -
 5 files changed, 15 insertions(+), 41 deletions(-)
 delete mode 100644 meta/recipes-devtools/opkg/opkg/opkg-configure.service

-- 
2.13.6

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


Re: [OE-core] [PATCH 4/4] musl: Make dev package rdep on staticdev

2018-05-16 Thread Burton, Ross
Surely in that usecase you can add musl-staticdev yourself?  Forcing
it into every SDK seems wasteful.

Ross

On 16 May 2018 at 14:32, Khem Raj  wrote:
> On Wed, May 16, 2018 at 4:09 AM, Burton, Ross  wrote:
>> Surely if you want to support static linking in the SDK, you just need
>> to add staticdev-pkgs to the SDKIMAGE_FEATURES?
>>
>
> In some usecase, compiler runtime and c library is only required everything
> else is external. Adding whole staticdev bundle would conflict with that
>
>> Ross
>>
>> On 16 May 2018 at 07:32, Khem Raj  wrote:
>>> This is needed for enabling static linking with SDK
>>>
>>> Signed-off-by: Khem Raj 
>>> ---
>>>  meta/recipes-core/musl/musl_git.bb | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/meta/recipes-core/musl/musl_git.bb 
>>> b/meta/recipes-core/musl/musl_git.bb
>>> index a0a098f5b4..d7d201df9d 100644
>>> --- a/meta/recipes-core/musl/musl_git.bb
>>> +++ b/meta/recipes-core/musl/musl_git.bb
>>> @@ -66,7 +66,7 @@ do_install() {
>>> done
>>>  }
>>>
>>> -RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
>>> libssp-nonshared-staticdev"
>>> +RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
>>> libssp-nonshared-staticdev ${PN}-staticdev"
>>>  RPROVIDES_${PN}-dev += "libc-dev virtual-libc-dev"
>>>  RPROVIDES_${PN} += "ldd libsegfault rtld(GNU_HASH)"
>>>
>>> --
>>> 2.17.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


Re: [OE-core] [PATCH 4/4] musl: Make dev package rdep on staticdev

2018-05-16 Thread Khem Raj
On Wed, May 16, 2018 at 7:38 AM, Burton, Ross  wrote:
> Surely in that usecase you can add musl-staticdev yourself?  Forcing
> it into every SDK seems wasteful.
>

I guess thats a fair argument I can accept it. Please ignore this patch then

> Ross
>
> On 16 May 2018 at 14:32, Khem Raj  wrote:
>> On Wed, May 16, 2018 at 4:09 AM, Burton, Ross  wrote:
>>> Surely if you want to support static linking in the SDK, you just need
>>> to add staticdev-pkgs to the SDKIMAGE_FEATURES?
>>>
>>
>> In some usecase, compiler runtime and c library is only required everything
>> else is external. Adding whole staticdev bundle would conflict with that
>>
>>> Ross
>>>
>>> On 16 May 2018 at 07:32, Khem Raj  wrote:
 This is needed for enabling static linking with SDK

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

 diff --git a/meta/recipes-core/musl/musl_git.bb 
 b/meta/recipes-core/musl/musl_git.bb
 index a0a098f5b4..d7d201df9d 100644
 --- a/meta/recipes-core/musl/musl_git.bb
 +++ b/meta/recipes-core/musl/musl_git.bb
 @@ -66,7 +66,7 @@ do_install() {
 done
  }

 -RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
 libssp-nonshared-staticdev"
 +RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
 libssp-nonshared-staticdev ${PN}-staticdev"
  RPROVIDES_${PN}-dev += "libc-dev virtual-libc-dev"
  RPROVIDES_${PN} += "ldd libsegfault rtld(GNU_HASH)"

 --
 2.17.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] libxslt: Fix handling of RVTs returned from nested EXSLT functions

2018-05-16 Thread Andrej Valek
Set the context variable to NULL when evaluating EXSLT functions.
Fixes potential use-after-free errors or memory leaks.

Fixes bug 792580

Signed-off-by: Andrej Valek 
---
 .../libxslt/libxslt/fix-rvts-handling.patch| 80 ++
 meta/recipes-support/libxslt/libxslt_1.1.32.bb |  5 +-
 2 files changed, 84 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-support/libxslt/libxslt/fix-rvts-handling.patch

diff --git a/meta/recipes-support/libxslt/libxslt/fix-rvts-handling.patch 
b/meta/recipes-support/libxslt/libxslt/fix-rvts-handling.patch
new file mode 100644
index 00..424c976d9b
--- /dev/null
+++ b/meta/recipes-support/libxslt/libxslt/fix-rvts-handling.patch
@@ -0,0 +1,80 @@
+libxslt-1.1.32: Fix handling of RVTs returned from nested EXSLT functions
+
+[No upstream tracking] -- https://bugzilla.gnome.org/show_bug.cgi?id=792580
+
+Set the context variable to NULL when evaluating EXSLT functions.
+Fixes potential use-after-free errors or memory leaks.
+
+Upstream-Status: Backport 
[https://git.gnome.org/browse/libxslt/commit/?id=8bd32f7753ac253a54279a0b6a88d15a57076bb0]
+bug: 792580
+Signed-off-by: Andrej Valek 
+
+diff --git a/libexslt/functions.c b/libexslt/functions.c
+index dc794e3..8511cb0 100644
+--- a/libexslt/functions.c
 b/libexslt/functions.c
+@@ -280,6 +280,7 @@ exsltFuncFunctionFunction (xmlXPathParserContextPtr ctxt, 
int nargs) {
+ exsltFuncFunctionData *func;
+ xmlNodePtr paramNode, oldInsert, fake;
+ int oldBase;
++void *oldCtxtVar;
+ xsltStackElemPtr params = NULL, param;
+ xsltTransformContextPtr tctxt = xsltXPathGetTransformContext(ctxt);
+ int i, notSet;
+@@ -418,11 +419,14 @@ exsltFuncFunctionFunction (xmlXPathParserContextPtr 
ctxt, int nargs) {
+ fake = xmlNewDocNode(tctxt->output, NULL,
+(const xmlChar *)"fake", NULL);
+ oldInsert = tctxt->insert;
++oldCtxtVar = tctxt->contextVariable;
+ tctxt->insert = fake;
++tctxt->contextVariable = NULL;
+ xsltApplyOneTemplate (tctxt, tctxt->node,
+ func->content, NULL, NULL);
+ xsltLocalVariablePop(tctxt, tctxt->varsBase, -2);
+ tctxt->insert = oldInsert;
++tctxt->contextVariable = oldCtxtVar;
+ tctxt->varsBase = oldBase;/* restore original scope */
+ if (params != NULL)
+   xsltFreeStackElemList(params);
+diff --git a/tests/docs/bug-209.xml b/tests/docs/bug-209.xml
+new file mode 100644
+index 000..69d62f2
+--- /dev/null
 b/tests/docs/bug-209.xml
+@@ -0,0 +1 @@
++
+diff --git a/tests/general/bug-209.out b/tests/general/bug-209.out
+new file mode 100644
+index 000..e829790
+--- /dev/null
 b/tests/general/bug-209.out
+@@ -0,0 +1,2 @@
++
++
+diff --git a/tests/general/bug-209.xsl b/tests/general/bug-209.xsl
+new file mode 100644
+index 000..fe69ac6
+--- /dev/null
 b/tests/general/bug-209.xsl
+@@ -0,0 +1,21 @@
++http://www.w3.org/1999/XSL/Transform;
++xmlns:func="http://exslt.org/functions;
++extension-element-prefixes="func">
++
++
++
++
++
++
++
++
++
++
++
++
++
++
++
++
diff --git a/meta/recipes-support/libxslt/libxslt_1.1.32.bb 
b/meta/recipes-support/libxslt/libxslt_1.1.32.bb
index 6a03f77699..f0fa5e723f 100644
--- a/meta/recipes-support/libxslt/libxslt_1.1.32.bb
+++ b/meta/recipes-support/libxslt/libxslt_1.1.32.bb
@@ -8,7 +8,10 @@ LIC_FILES_CHKSUM = 
"file://Copyright;md5=0cd9a07afbeb24026c9b03aecfeba458"
 SECTION = "libs"
 DEPENDS = "libxml2"
 
-SRC_URI = "http://xmlsoft.org/sources/libxslt-${PV}.tar.gz;
+SRC_URI = "http://xmlsoft.org/sources/libxslt-${PV}.tar.gz \
+   file://fix-rvts-handling.patch \
+   "
+
 SRC_URI[md5sum] = "1fc72f98e98bf4443f1651165f3aa146"
 SRC_URI[sha256sum] = 
"526ecd0abaf4a7789041622c3950c0e7f2c4c8835471515fd77eec684a355460"
 
-- 
2.11.0

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


Re: [OE-core] [PATCH 4/4] musl: Make dev package rdep on staticdev

2018-05-16 Thread Burton, Ross
Surely if you want to support static linking in the SDK, you just need
to add staticdev-pkgs to the SDKIMAGE_FEATURES?

Ross

On 16 May 2018 at 07:32, Khem Raj  wrote:
> This is needed for enabling static linking with SDK
>
> Signed-off-by: Khem Raj 
> ---
>  meta/recipes-core/musl/musl_git.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-core/musl/musl_git.bb 
> b/meta/recipes-core/musl/musl_git.bb
> index a0a098f5b4..d7d201df9d 100644
> --- a/meta/recipes-core/musl/musl_git.bb
> +++ b/meta/recipes-core/musl/musl_git.bb
> @@ -66,7 +66,7 @@ do_install() {
> done
>  }
>
> -RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
> libssp-nonshared-staticdev"
> +RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
> libssp-nonshared-staticdev ${PN}-staticdev"
>  RPROVIDES_${PN}-dev += "libc-dev virtual-libc-dev"
>  RPROVIDES_${PN} += "ldd libsegfault rtld(GNU_HASH)"
>
> --
> 2.17.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


Re: [OE-core] [meta-oe][PATCH] package.bbclass: provide debug sources in externalsrc -dbg packages

2018-05-16 Thread Alexander Kanavin

On 05/16/2018 03:11 PM, Vignesh Rajendran (RBEI/ECF3) wrote:
As I am new to the patch submission, anyone can explain how long this 
process will take for acceptance in master/next release?


After several days of waiting by watching OE-mailing thread for any 
reply for the patch, Now I had a feeling like the patch may have been 
missed.


Generally we don't positively acknowledge the patches, to reduce the 
amount of noise on the mailing list. If the patch doesn't show up in 
master after a week or two, do ping or resend.



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


Re: [OE-core] [PATCH 4/4] musl: Make dev package rdep on staticdev

2018-05-16 Thread Khem Raj
On Wed, May 16, 2018 at 4:09 AM, Burton, Ross  wrote:
> Surely if you want to support static linking in the SDK, you just need
> to add staticdev-pkgs to the SDKIMAGE_FEATURES?
>

In some usecase, compiler runtime and c library is only required everything
else is external. Adding whole staticdev bundle would conflict with that

> Ross
>
> On 16 May 2018 at 07:32, Khem Raj  wrote:
>> This is needed for enabling static linking with SDK
>>
>> Signed-off-by: Khem Raj 
>> ---
>>  meta/recipes-core/musl/musl_git.bb | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-core/musl/musl_git.bb 
>> b/meta/recipes-core/musl/musl_git.bb
>> index a0a098f5b4..d7d201df9d 100644
>> --- a/meta/recipes-core/musl/musl_git.bb
>> +++ b/meta/recipes-core/musl/musl_git.bb
>> @@ -66,7 +66,7 @@ do_install() {
>> done
>>  }
>>
>> -RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
>> libssp-nonshared-staticdev"
>> +RDEPENDS_${PN}-dev += "linux-libc-headers-dev bsd-headers-dev 
>> libssp-nonshared-staticdev ${PN}-staticdev"
>>  RPROVIDES_${PN}-dev += "libc-dev virtual-libc-dev"
>>  RPROVIDES_${PN} += "ldd libsegfault rtld(GNU_HASH)"
>>
>> --
>> 2.17.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


Re: [OE-core] [meta-oe][PATCH] package.bbclass: provide debug sources in externalsrc -dbg packages

2018-05-16 Thread Vignesh Rajendran (RBEI/ECF3)
Hello OE-Core Members,



As I am new to the patch submission, anyone can explain how long this process 
will take for acceptance in master/next release?

After several days of waiting by watching OE-mailing thread for any reply for 
the patch, Now I had a feeling like the patch may have been missed.



>From the below reference, I am just giving a ping back to the community.



https://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded

If your patch has not had any feedback after a few days it may have been missed 
or the appropriate reviewers may not currently be around; it is perfectly fine 
to reply to it yourself with a "ping" / reminder request for feedback. NOTE: 
patch review for feature / recipe upgrade patches will likely be delayed during 
a feature freeze because these types of patches aren't merged during this time 
- you may have to wait until after the freeze is lifted.



Any feedback/acceptance will be highly appreciated.



Mit freundlichen Grüßen / Best regards



Rajendran Vignesh

RBEI/ECF33



Tel. +91 422 67-65103







-Original Message-
From: Vignesh Rajendran (RBEI/ECF3)
Sent: Monday, May 07, 2018 11:21 AM
To: openembedded-core@lists.openembedded.org
Cc: Vignesh Rajendran (RBEI/ECF3) 
Subject: [meta-oe][PATCH] package.bbclass: provide debug sources in externalsrc 
-dbg packages



The recipes which are using externalsrc method doesn't contain the

debug sources in their -dbg packages.



Add a handling in copydebugsources function to copy the debugsources for

externalsrc -dbg packages.



Signed-off-by: Vignesh Rajendran 
>

---

meta/classes/externalsrc.bbclass |  3 +++

meta/classes/package.bbclass | 14 +++---

2 files changed, 14 insertions(+), 3 deletions(-)



diff --git a/meta/classes/externalsrc.bbclass b/meta/classes/externalsrc.bbclass

index c9f5cf7..135670d 100644

--- a/meta/classes/externalsrc.bbclass

+++ b/meta/classes/externalsrc.bbclass

@@ -129,6 +129,9 @@ python () {

 d.setVar('CONFIGURESTAMPFILE', configstamp)

 d.setVar('STAMP', 
'${STAMPS_DIR}/work-shared/${PN}/${EXTENDPE}${PV}-${PR}')

 d.setVar('STAMPCLEAN', '${STAMPS_DIR}/work-shared/${PN}/*-*')

+

+ # set DEBUG_PREFIX_MAP to external source path

+ d.appendVar("DEBUG_PREFIX_MAP", 
"-fdebug-prefix-map=${EXTERNALSRC}=/usr/src/debug/${PN}/${EXTENDPE}${PV}-${PR}")

}

 python externalsrc_configure_prefunc() {

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass

index edeffa9..2c4472d 100644

--- a/meta/classes/package.bbclass

+++ b/meta/classes/package.bbclass

@@ -420,6 +420,7 @@ def copydebugsources(debugsrcdir, d):

 workdir = d.getVar("WORKDIR")

 workparentdir = os.path.dirname(os.path.dirname(workdir))

 workbasedir = os.path.basename(os.path.dirname(workdir)) + "/" + 
os.path.basename(workdir)

+ externalsrc = d.getVar('EXTERNALSRC', True)

 # If build path exists in sourcefile, it means toolchain did not use

 # -fdebug-prefix-map to compile

@@ -444,10 +445,17 @@ def copydebugsources(debugsrcdir, d):

 processdebugsrc += "fgrep -zw '%s' | "

 # Remove prefix in the source paths

 processdebugsrc += "sed 's#%s##g' | "

-processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner '%s%s' 
2>/dev/null)"

+# handle external sources to provide sources in -dbg packages

+if externalsrc:

+localsrc_prefix = "/usr/src/debug/"

+processdebugsrc += "sed 's#%s/##g' | "

+processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner 
'%s%s/%s' 2>/dev/null)"

+cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workbasedir, externalsrc, dvar, debugsrcdir, workbasedir)

+else:

+processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner 
'%s%s' 2>/dev/null)"

+cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workparentdir, dvar, debugsrcdir)

-cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workparentdir, dvar, debugsrcdir)

-(retval, output) = oe.utils.getstatusoutput(cmd)

+ (retval, output) = oe.utils.getstatusoutput(cmd)

 # Can "fail" if internal headers/transient sources are attempted

 #if retval:

 #bb.fatal("debug source copy failed with exit code %s (cmd was 
%s)" % (retval, cmd))

--

2.7.4


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


[OE-core] [OE-Core][PATCH] libnss-mdns: fix postinst and postrm

2018-05-16 Thread Dan McGregor
From: Dan McGregor 

Don't blindly add mdns_minimal, instead check for existing
mdns_minimal instances. Fixes continuously appending mdns settings
to nssswitch.conf when upgrading.

Signed-off-by: Dan McGregor 
---
 .../libnss-mdns/libnss-mdns_0.10.bb   | 15 +--
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/meta/recipes-connectivity/libnss-mdns/libnss-mdns_0.10.bb 
b/meta/recipes-connectivity/libnss-mdns/libnss-mdns_0.10.bb
index 8d2feec7698..5be5e4fa323 100644
--- a/meta/recipes-connectivity/libnss-mdns/libnss-mdns_0.10.bb
+++ b/meta/recipes-connectivity/libnss-mdns/libnss-mdns_0.10.bb
@@ -28,13 +28,16 @@ DEBIANNAME_${PN} = "libnss-mdns"
 RDEPENDS_${PN} = "avahi-daemon"
 
 pkg_postinst_${PN} () {
-   sed -e '/^hosts:/s/\s*\//' \
-   -e 's/\(^hosts:.*\)\(\\)\(.*\)\(\\)\(.*\)/\1\2 
mdns4_minimal [NOTFOUND=return]\3\4 mdns\5/' \
-   -i $D${sysconfdir}/nsswitch.conf
+   sed '
+   /^hosts:/ !b
+   /\/ b
+   s/\([[:blank:]]\+\)dns\>/\1mdns4_minimal [NOTFOUND=return] dns/g
+   ' -i $D${sysconfdir}/nsswitch.conf
 }
 
 pkg_prerm_${PN} () {
-   sed -e '/^hosts:/s/\s*\//' \
-   -e '/^hosts:/s/\s*mdns4_minimal\s\+\[NOTFOUND=return\]//' \
-   -i $D${sysconfdir}/nsswitch.conf
+   sed '
+   /^hosts:/ !b
+   s/[[:blank:]]\+mdns\(4\|6\)\?\(_minimal\( 
\[NOTFOUND=return\]\)\?\)\?//g
+   ' -i $D${sysconfdir}/nsswitch.conf
 }
-- 
2.18.0-dev

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


Re: [OE-core] [PATCH] u-boot: Upgrade to 2018.05 release

2018-05-16 Thread Marek Vasut
On 05/17/2018 12:00 AM, Otavio Salvador wrote:
> On Wed, May 16, 2018 at 5:14 PM, Marek Vasut  wrote:
>> This upgrades the U-Boot from 2018.03 to 2018.05 release.
>>
>> Signed-off-by: Marek Vasut 
>> Cc: Otavio Salvador 
>> Cc: Richard Purdie 
>> Cc: Ross Burton 
> 
> Nice work :-)

Except this kind of patch could (read should) be generated automatically
by the OE recipe auto-upgrade drone.

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


[OE-core] [PATCH][rocko] libnl: protect against buffer overflow (CVE-2017-0553)

2018-05-16 Thread Ross Burton
Signed-off-by: Ross Burton 
---
 meta/recipes-support/libnl/libnl/overflow.patch | 39 +
 meta/recipes-support/libnl/libnl_3.2.29.bb  |  1 +
 2 files changed, 40 insertions(+)
 create mode 100644 meta/recipes-support/libnl/libnl/overflow.patch

diff --git a/meta/recipes-support/libnl/libnl/overflow.patch 
b/meta/recipes-support/libnl/libnl/overflow.patch
new file mode 100644
index 000..777fac3ea1a
--- /dev/null
+++ b/meta/recipes-support/libnl/libnl/overflow.patch
@@ -0,0 +1,39 @@
+CVE: CVE-2017-0553
+Upstream-Status: Backport
+Signed-off-by: Ross Burton 
+
+From 3e18948f17148e6a3c4255bdeaaf01ef6081ceeb Mon Sep 17 00:00:00 2001
+From: Thomas Haller 
+Date: Mon, 6 Feb 2017 22:23:52 +0100
+Subject: [PATCH] lib: check for integer-overflow in nlmsg_reserve()
+
+In general, libnl functions are not robust against calling with
+invalid arguments. Thus, never call libnl functions with invalid
+arguments. In case of nlmsg_reserve() this means never provide
+a @len argument that causes overflow.
+
+Still, add an additional safeguard to avoid exploiting such bugs.
+
+Assume that @pad is a trusted, small integer.
+Assume that n->nm_size is a valid number of allocated bytes (and thus
+much smaller then SIZE_T_MAX).
+Assume, that @len may be set to an untrusted value. Then the patch
+avoids an integer overflow resulting in reserving too few bytes.
+---
+ lib/msg.c | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/lib/msg.c b/lib/msg.c
+index 9af3f3a0..3e27d4e0 100644
+--- a/lib/msg.c
 b/lib/msg.c
+@@ -411,6 +411,9 @@ void *nlmsg_reserve(struct nl_msg *n, size_t len, int pad)
+   size_t nlmsg_len = n->nm_nlh->nlmsg_len;
+   size_t tlen;
+ 
++  if (len > n->nm_size)
++  return NULL;
++
+   tlen = pad ? ((len + (pad - 1)) & ~(pad - 1)) : len;
+ 
+   if ((tlen + nlmsg_len) > n->nm_size)
\ No newline at end of file
diff --git a/meta/recipes-support/libnl/libnl_3.2.29.bb 
b/meta/recipes-support/libnl/libnl_3.2.29.bb
index 7d4839ba506..746fd3d4153 100644
--- a/meta/recipes-support/libnl/libnl_3.2.29.bb
+++ b/meta/recipes-support/libnl/libnl_3.2.29.bb
@@ -12,6 +12,7 @@ DEPENDS = "flex-native bison-native"
 SRC_URI = 
"https://github.com/thom311/${BPN}/releases/download/${BPN}${@d.getVar('PV').replace('.','_')}/${BP}.tar.gz
 \
file://fix-pktloc_syntax_h-race.patch \
file://fix-pc-file.patch \
+   file://overflow.patch \
 "
 UPSTREAM_CHECK_URI = "https://github.com/thom311/${BPN}/releases;
 
-- 
2.11.0

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


Re: [OE-core] [PATCH] u-boot: Upgrade to 2018.05 release

2018-05-16 Thread Marek Vasut
On 05/17/2018 12:49 AM, Burton, Ross wrote:
> I was informed yesterday that AUH had just started, so you should be
> getting an email shortly...

Well, if it fails, feel free to pick this patch :)

> On 16 May 2018 at 23:37, Marek Vasut  wrote:
>> On 05/17/2018 12:00 AM, Otavio Salvador wrote:
>>> On Wed, May 16, 2018 at 5:14 PM, Marek Vasut  wrote:
 This upgrades the U-Boot from 2018.03 to 2018.05 release.

 Signed-off-by: Marek Vasut 
 Cc: Otavio Salvador 
 Cc: Richard Purdie 
 Cc: Ross Burton 
>>>
>>> Nice work :-)
>>
>> Except this kind of patch could (read should) be generated automatically
>> by the OE recipe auto-upgrade drone.
>>
>> --
>> Best regards,
>> Marek Vasut
>> --
>> ___
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core


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


Re: [OE-core] [PATCH] u-boot: Upgrade to 2018.05 release

2018-05-16 Thread Burton, Ross
I was informed yesterday that AUH had just started, so you should be
getting an email shortly...

On 16 May 2018 at 23:37, Marek Vasut  wrote:
> On 05/17/2018 12:00 AM, Otavio Salvador wrote:
>> On Wed, May 16, 2018 at 5:14 PM, Marek Vasut  wrote:
>>> This upgrades the U-Boot from 2018.03 to 2018.05 release.
>>>
>>> Signed-off-by: Marek Vasut 
>>> Cc: Otavio Salvador 
>>> Cc: Richard Purdie 
>>> Cc: Ross Burton 
>>
>> Nice work :-)
>
> Except this kind of patch could (read should) be generated automatically
> by the OE recipe auto-upgrade drone.
>
> --
> Best regards,
> Marek Vasut
> --
> ___
> 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 0/1] gstreamer1.0-plugins-good: enable PACKAGECONFIG gtk conditionally

2018-05-16 Thread kai.kang
From: Kai Kang 

The following changes since commit 13cc30cd7de4841990b600e83e1249c81a5171dd:

  local.conf.sample.extended: Drop obsolete rpc and libnsl (2018-05-15 11:07:50 
+0100)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib kangkai/gst-good
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=kangkai/gst-good

Kai Kang (1):
  gstreamer1.0-plugins-good: enable PACKAGECONFIG gtk conditionally

 meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.14.0.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

-- 
2.14.1

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


Re: [OE-core] [PATCH][rocko] libnl: protect against buffer overflow (CVE-2017-0553)

2018-05-16 Thread Andre McCurdy
On Wed, May 16, 2018 at 4:30 PM, Ross Burton  wrote:

Looks like a duplicate of:

  http://lists.openembedded.org/pipermail/openembedded-core/2018-May/150743.html

?

> Signed-off-by: Ross Burton 
> ---
>  meta/recipes-support/libnl/libnl/overflow.patch | 39 
> +
>  meta/recipes-support/libnl/libnl_3.2.29.bb  |  1 +
>  2 files changed, 40 insertions(+)
>  create mode 100644 meta/recipes-support/libnl/libnl/overflow.patch
>
> diff --git a/meta/recipes-support/libnl/libnl/overflow.patch 
> b/meta/recipes-support/libnl/libnl/overflow.patch
> new file mode 100644
> index 000..777fac3ea1a
> --- /dev/null
> +++ b/meta/recipes-support/libnl/libnl/overflow.patch
> @@ -0,0 +1,39 @@
> +CVE: CVE-2017-0553
> +Upstream-Status: Backport
> +Signed-off-by: Ross Burton 
> +
> +From 3e18948f17148e6a3c4255bdeaaf01ef6081ceeb Mon Sep 17 00:00:00 2001
> +From: Thomas Haller 
> +Date: Mon, 6 Feb 2017 22:23:52 +0100
> +Subject: [PATCH] lib: check for integer-overflow in nlmsg_reserve()
> +
> +In general, libnl functions are not robust against calling with
> +invalid arguments. Thus, never call libnl functions with invalid
> +arguments. In case of nlmsg_reserve() this means never provide
> +a @len argument that causes overflow.
> +
> +Still, add an additional safeguard to avoid exploiting such bugs.
> +
> +Assume that @pad is a trusted, small integer.
> +Assume that n->nm_size is a valid number of allocated bytes (and thus
> +much smaller then SIZE_T_MAX).
> +Assume, that @len may be set to an untrusted value. Then the patch
> +avoids an integer overflow resulting in reserving too few bytes.
> +---
> + lib/msg.c | 3 +++
> + 1 file changed, 3 insertions(+)
> +
> +diff --git a/lib/msg.c b/lib/msg.c
> +index 9af3f3a0..3e27d4e0 100644
> +--- a/lib/msg.c
>  b/lib/msg.c
> +@@ -411,6 +411,9 @@ void *nlmsg_reserve(struct nl_msg *n, size_t len, int 
> pad)
> +   size_t nlmsg_len = n->nm_nlh->nlmsg_len;
> +   size_t tlen;
> +
> ++  if (len > n->nm_size)
> ++  return NULL;
> ++
> +   tlen = pad ? ((len + (pad - 1)) & ~(pad - 1)) : len;
> +
> +   if ((tlen + nlmsg_len) > n->nm_size)
> \ No newline at end of file
> diff --git a/meta/recipes-support/libnl/libnl_3.2.29.bb 
> b/meta/recipes-support/libnl/libnl_3.2.29.bb
> index 7d4839ba506..746fd3d4153 100644
> --- a/meta/recipes-support/libnl/libnl_3.2.29.bb
> +++ b/meta/recipes-support/libnl/libnl_3.2.29.bb
> @@ -12,6 +12,7 @@ DEPENDS = "flex-native bison-native"
>  SRC_URI = 
> "https://github.com/thom311/${BPN}/releases/download/${BPN}${@d.getVar('PV').replace('.','_')}/${BP}.tar.gz
>  \
> file://fix-pktloc_syntax_h-race.patch \
> file://fix-pc-file.patch \
> +   file://overflow.patch \
>  "
>  UPSTREAM_CHECK_URI = "https://github.com/thom311/${BPN}/releases;
>
> --
> 2.11.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] [AUH] Upgrade status: 2018-05-16

2018-05-16 Thread auh
Recipe upgrade statistics:

* Succeeded: 48
python3-git, 2.1.9, Derek Straka 
gpgme, 1.11.1, Hongxu Jia 
gst-examples, 0.0.1-new-commits-available, Anuj Mittal 

mtd-utils, 2.0.2, Denys Dmytriyenko 
gnu-config, 20150728-new-commits-available, Robert Yang 

python3-pip, 10.0.1, Derek Straka 
ethtool, 4.16, Changhyeok Bae 
python3-dbus, 1.2.8, Derek Straka 
lttng-modules, 2.10.6, Richard Purdie 

python-setuptools, 39.1.0, Derek Straka 
iproute2, 4.16.0, Changhyeok Bae 
ca-certificates, 20180409, Alexander Kanavin 

puzzles, 0.0-new-commits-available, Maxin B. John 
gnupg, 2.2.7, Hongxu Jia 
linux-firmware, 0.0-new-commits-available, Otavio Salvador 

acpid, 2.0.29, Maxin B. John 
dbus-glib, 0.110, Chen Qi 
openssh, 7.7p1, Armin Kuster 
hwlatdetect, 1.3, Alexander Kanavin 
python3-setuptools, 39.1.0, Derek Straka 
xdg-utils, 1.1.3, Maxin B. John 
xserver-xorg, 1.20.0, Armin Kuster 
python-numpy, 1.14.3, Derek Straka 
man-pages, 4.16, Hongxu Jia 
man-db, 2.8.3, Hongxu Jia 
libdmx, 1.1.4, Armin Kuster 
cronie, 1.5.2, Maxin B. John 
sqlite3, 3.23.1, Maxin B. John 
libdrm, 2.4.92, Otavio Salvador 
lz4, 1.8.2, Denys Dmytriyenko 
hdparm, 9.56, Denys Dmytriyenko 
piglit, 1.0-new-commits-available, Maxin B. John 
python3-numpy, 1.14.3, Derek Straka 
libinput, 1.10.6, Maxin B. John 
alsa-plugins, 1.1.6, Tanu Kaskinen 
dbus, 1.12.8, Chen Qi 
dbus-test, 1.12.8, Chen Qi 
xf86-input-libinput, 0.27.1, Armin Kuster 
dtc, 1.4.6, Alexander Kanavin 
libepoxy, 1.5.1, Maxin B. John 
gtk+3, 3.22.30, Maxin B. John 
kexec-tools, 2.0.17, Armin Kuster 
wayland-protocols, 1.14, Denys Dmytriyenko 
python, 2.7.15, Derek Straka 
python3-pygobject, 3.28.2, Derek Straka 
vte, 0.52.1, Maxin B. John 
wget, 1.19.5, Yi Zhao 
libsoup-2.4, 2.62.2, Maxin B. John 
* Failed(do_compile): 17
systemd-boot, 238, Chen Qi 
subversion, 1.10.0, Richard Purdie 
git, 2.17.0, Robert Yang 
u-boot-mkimage, 2018.05, Marek Vasut 
libidn, 1.35, Alexander Kanavin 
llvm, 6.0-new-commits-available, Khem Raj 
webkitgtk, 2.20.2, Alexander Kanavin 
glib-networking, 2.56.0, Maxin B. John 
btrfs-tools, 4.16.1, Alexander Kanavin 
dpkg, 1.19.0.5, Aníbal Limón 
mesa, 18.0.3, Otavio Salvador 
rt-tests, 1.3, Alexander Kanavin 
python3-pycairo, 1.17.0, Derek Straka 
cups, 2.2.7, Chen Qi 
lttng-tools, 2.10.4, Richard Purdie 
freetype, 2.9.1, Maxin B. John 
harfbuzz, 1.7.6, Maxin B. John 
* Failed (devtool error): 48
gnu-efi, 3.0.8, Yi Zhao 
gdb, 8.1, Khem Raj 
kmscube, git-new-commits-available, Carlos Rafael Giani 

go, 1.10.2, Khem Raj 
busybox, 1.28.3, Armin Kuster 
bind, 9.12.1, Armin Kuster 
re2c, 1.0.1, Khem Raj 
gdk-pixbuf, 2.36.12, Maxin B. John 
lzop, 1.04, Denys Dmytriyenko 

Re: [OE-core] [rocko][PATCH] libnl: fix CVE-2017-0553

2018-05-16 Thread akuster808


On 05/15/2018 01:14 PM, Andre McCurdy wrote:
> On Fri, May 11, 2018 at 4:52 PM, Andre McCurdy  wrote:
>> An elevation of privilege vulnerability in libnl could enable a local
>> malicious application to execute arbitrary code within the context of
>> the Wi-Fi service. This issue is rated as Moderate because it first
>> requires compromising a privileged process and is mitigated by
>> current platform configurations. Product: Android. Versions: 5.0.2,
>> 5.1.1, 6.0, 6.0.1, 7.0, 7.1.1. Android ID: A-32342065. NOTE: this
>> issue also exists in the upstream libnl before 3.3.0 library.
>>
>>   https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-0553
>>
>> Backport fix from upstream libnl 3.3.0 release:
>>
>>   
>> https://github.com/thom311/libnl/commit/3e18948f17148e6a3c4255bdeaaf01ef6081ceeb
>>   http://lists.infradead.org/pipermail/libnl/2017-May/002313.html
> Armin, please let me know if this is OK and I'll send the same fix for
> pyro and morty.

this is fine. I have it in stable/rocko-mnut pending build rotation

thanks,
Armin
>
>> Signed-off-by: Andre McCurdy 
>> ---
>>  ...eck-for-integer-overflow-in-nlmsg_reserve.patch | 43 
>> ++
>>  meta/recipes-support/libnl/libnl_3.2.29.bb |  2 +
>>  2 files changed, 45 insertions(+)
>>  create mode 100644 
>> meta/recipes-support/libnl/libnl/lib-check-for-integer-overflow-in-nlmsg_reserve.patch
>>
>> diff --git 
>> a/meta/recipes-support/libnl/libnl/lib-check-for-integer-overflow-in-nlmsg_reserve.patch
>>  
>> b/meta/recipes-support/libnl/libnl/lib-check-for-integer-overflow-in-nlmsg_reserve.patch
>> new file mode 100644
>> index 000..594dd06
>> --- /dev/null
>> +++ 
>> b/meta/recipes-support/libnl/libnl/lib-check-for-integer-overflow-in-nlmsg_reserve.patch
>> @@ -0,0 +1,43 @@
>> +From 3e18948f17148e6a3c4255bdeaaf01ef6081ceeb Mon Sep 17 00:00:00 2001
>> +From: Thomas Haller 
>> +Date: Mon, 6 Feb 2017 22:23:52 +0100
>> +Subject: [PATCH] lib: check for integer-overflow in nlmsg_reserve()
>> +
>> +In general, libnl functions are not robust against calling with
>> +invalid arguments. Thus, never call libnl functions with invalid
>> +arguments. In case of nlmsg_reserve() this means never provide
>> +a @len argument that causes overflow.
>> +
>> +Still, add an additional safeguard to avoid exploiting such bugs.
>> +
>> +Assume that @pad is a trusted, small integer.
>> +Assume that n->nm_size is a valid number of allocated bytes (and thus
>> +much smaller then SIZE_T_MAX).
>> +Assume, that @len may be set to an untrusted value. Then the patch
>> +avoids an integer overflow resulting in reserving too few bytes.
>> +
>> +Upstream-Status: Backport 
>> [https://github.com/thom311/libnl/commit/3e18948f17148e6a3c4255bdeaaf01ef6081ceeb]
>> +CVE: CVE-2017-0553
>> +
>> +Signed-off-by: Andre McCurdy 
>> +---
>> + lib/msg.c | 3 +++
>> + 1 file changed, 3 insertions(+)
>> +
>> +diff --git a/lib/msg.c b/lib/msg.c
>> +index 9af3f3a..3e27d4e 100644
>> +--- a/lib/msg.c
>>  b/lib/msg.c
>> +@@ -411,6 +411,9 @@ void *nlmsg_reserve(struct nl_msg *n, size_t len, int 
>> pad)
>> +   size_t nlmsg_len = n->nm_nlh->nlmsg_len;
>> +   size_t tlen;
>> +
>> ++  if (len > n->nm_size)
>> ++  return NULL;
>> ++
>> +   tlen = pad ? ((len + (pad - 1)) & ~(pad - 1)) : len;
>> +
>> +   if ((tlen + nlmsg_len) > n->nm_size)
>> +--
>> +1.9.1
>> +
>> diff --git a/meta/recipes-support/libnl/libnl_3.2.29.bb 
>> b/meta/recipes-support/libnl/libnl_3.2.29.bb
>> index 7d4839b..4ce80e8 100644
>> --- a/meta/recipes-support/libnl/libnl_3.2.29.bb
>> +++ b/meta/recipes-support/libnl/libnl_3.2.29.bb
>> @@ -12,7 +12,9 @@ DEPENDS = "flex-native bison-native"
>>  SRC_URI = 
>> "https://github.com/thom311/${BPN}/releases/download/${BPN}${@d.getVar('PV').replace('.','_')}/${BP}.tar.gz
>>  \
>> file://fix-pktloc_syntax_h-race.patch \
>> file://fix-pc-file.patch \
>> +   file://lib-check-for-integer-overflow-in-nlmsg_reserve.patch \
>>  "
>> +
>>  UPSTREAM_CHECK_URI = "https://github.com/thom311/${BPN}/releases;
>>
>>  SRC_URI[md5sum] = "a8ba62a5c4f883f4e493a46d1f3733fe"
>> --
>> 1.9.1
>>

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


Re: [OE-core] read-only-rootfs and EFI configuration

2018-05-16 Thread Paul Eggleton
On Wednesday, 9 May 2018 8:19:19 AM NZST Cal Sullivan wrote:
> You're right. Since its adding to APPEND from rootfs-postcommands, which 
> is inherited by images, its in the local space instead of global and 
> won't be picked up by these recipes...
> 
> Right now it won't actually affect anything out of the box, as that 
> method of booting isn't default due to lack of random UUID support, but 
> its something that will need to be solved in the future.

I've filed the following bug to track this:

https://bugzilla.yoctoproject.org/show_bug.cgi?id=12755

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


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


[OE-core] [PATCH] strace: fix ARM Thumb build with frame pointers enabled

2018-05-16 Thread Andre McCurdy
Replace the previous (incomplete) workaround with better solution
backported from upstream.

Signed-off-by: Andre McCurdy 
---
 ...aw_syscall.h-avoid-r7-specified-register-.patch | 74 ++
 meta/recipes-devtools/strace/strace_4.22.bb| 16 ++---
 2 files changed, 81 insertions(+), 9 deletions(-)
 create mode 100644 
meta/recipes-devtools/strace/strace/0001-linux-arm-raw_syscall.h-avoid-r7-specified-register-.patch

diff --git 
a/meta/recipes-devtools/strace/strace/0001-linux-arm-raw_syscall.h-avoid-r7-specified-register-.patch
 
b/meta/recipes-devtools/strace/strace/0001-linux-arm-raw_syscall.h-avoid-r7-specified-register-.patch
new file mode 100644
index 000..8584291
--- /dev/null
+++ 
b/meta/recipes-devtools/strace/strace/0001-linux-arm-raw_syscall.h-avoid-r7-specified-register-.patch
@@ -0,0 +1,74 @@
+From 0c75ebaad09d6d3f2395dfe6160904af883dd0d9 Mon Sep 17 00:00:00 2001
+From: Andre McCurdy 
+Date: Tue, 15 May 2018 15:34:39 -0700
+Subject: [PATCH] linux/arm/raw_syscall.h: avoid r7 specified register
+ variables with Thumb
+
+If Thumb code is being generated and frame pointers are enabled, the
+Thumb frame pointer in r7 clashes with any local variable which may
+need to be assigned to r7 (e.g. the syscall NR when making a raw
+syscall).
+
+With gcc, the double use of r7 results in a build error, e.g.
+
+  strace-4.22/tests/inject-nf.c:86:1: error: r7 cannot be used in asm here
+
+With clang, the double use of r7 can result in the compiler silently
+generating broken code which crashes at run time due to frame pointer
+corruption:
+
+  https://bugs.llvm.org/show_bug.cgi?id=34165
+
+In most cases the problem isn't visible as frame pointers will be
+disabled automatically due to optimisation level. However to handle
+cases where frame pointers are enabled (e.g. when CFLAGS etc are set
+to support a debug build, etc) provide a version of raw_syscall_0
+which manually saves and restores the frame pointer value in r7
+to a temporary register before setting up the syscall NR in r7
+and invoking the syscall.
+
+* linux/arm/raw_syscall.h (raw_syscall_0) [__thumb__]: Provide
+an alternative version.
+
+Upstream-Status: Backport
+
+Signed-off-by: Andre McCurdy 
+---
+ linux/arm/raw_syscall.h | 15 +++
+ 1 file changed, 15 insertions(+)
+
+diff --git a/linux/arm/raw_syscall.h b/linux/arm/raw_syscall.h
+index 69c7e23..ec534ec 100644
+--- a/linux/arm/raw_syscall.h
 b/linux/arm/raw_syscall.h
+@@ -36,12 +36,27 @@ static inline kernel_ulong_t
+ raw_syscall_0(const kernel_ulong_t nr, kernel_ulong_t *err)
+ {
+   *err = 0;
++
++#ifdef __thumb__ /* && FRAME_POINTERS_ENABLED */
++
++  register kernel_ulong_t rt;
++  register kernel_ulong_t r0 __asm__("r0");
++  __asm__ __volatile__("mov %1,r7; mov r7,%2; swi 0x0; mov r7,%1"
++   : "=r"(r0), "="(rt)
++   : "r"(nr)
++   : "memory");
++
++#else
++
+   register kernel_ulong_t r7 __asm__("r7") = nr;
+   register kernel_ulong_t r0 __asm__("r0");
+   __asm__ __volatile__("swi 0x0"
+: "=r"(r0)
+: "r"(r7)
+: "memory");
++
++#endif
++
+   return r0;
+ }
+ # define raw_syscall_0 raw_syscall_0
+-- 
+1.9.1
+
diff --git a/meta/recipes-devtools/strace/strace_4.22.bb 
b/meta/recipes-devtools/strace/strace_4.22.bb
index 947b3f7..99691f8 100644
--- a/meta/recipes-devtools/strace/strace_4.22.bb
+++ b/meta/recipes-devtools/strace/strace_4.22.bb
@@ -14,21 +14,13 @@ SRC_URI = 
"https://strace.io/files/${PV}/strace-${PV}.tar.xz \
file://mips-SIGEMT.patch \
file://0001-caps-abbrev.awk-fix-gawk-s-path.patch \

file://0001-tests-sigaction-Check-for-mips-and-alpha-before-usin.patch \
+   
file://0001-linux-arm-raw_syscall.h-avoid-r7-specified-register-.patch \
"
 SRC_URI[md5sum] = "7a2a7d7715da6e6834bc65bd09bace1c"
 SRC_URI[sha256sum] = 
"068cd09264c95e4d591bbcd3ea08f99a693ed8663cd5169b0fdad72eb5bdb39d"
 
 inherit autotools ptest bluetooth
 
-EXTRA_OECONF += "--enable-mpers=no"
-
-CFLAGS_append_libc-musl = " -Dsigcontext_struct=sigcontext"
-# otherwise strace-4.22/tests/inject-nf.c fails to build as discussed here:
-# 
http://lists.openembedded.org/pipermail/openembedded-core/2018-May/150647.html
-DEBUG_OPTIMIZATION_remove = "${@bb.utils.contains('PTEST_ENABLED', '1', 
'-fno-omit-frame-pointer', '', d)}"
-
-RDEPENDS_${PN}-ptest += "make coreutils grep gawk sed"
-
 PACKAGECONFIG_class-target ??= "\
 ${@bb.utils.contains('DISTRO_FEATURES', 'bluetooth', 'bluez', '', d)} \
 "
@@ -36,6 +28,10 @@ PACKAGECONFIG_class-target ??= "\
 PACKAGECONFIG[bluez] = 
"ac_cv_header_bluetooth_bluetooth_h=yes,ac_cv_header_bluetooth_bluetooth_h=no,${BLUEZ}"
 PACKAGECONFIG[libunwind] = "--with-libunwind,--without-libunwind,libunwind"
 
+EXTRA_OECONF += "--enable-mpers=no"
+

Re: [OE-core] [PATCH] u-boot: Upgrade to 2018.05 release

2018-05-16 Thread Otavio Salvador
On Wed, May 16, 2018 at 5:14 PM, Marek Vasut  wrote:
> This upgrades the U-Boot from 2018.03 to 2018.05 release.
>
> Signed-off-by: Marek Vasut 
> Cc: Otavio Salvador 
> Cc: Richard Purdie 
> Cc: Ross Burton 

Nice work :-)

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] u-boot: Upgrade to 2018.05 release

2018-05-16 Thread Khem Raj
On Wed, May 16, 2018 at 3:00 PM, Otavio Salvador
 wrote:
> On Wed, May 16, 2018 at 5:14 PM, Marek Vasut  wrote:
>> This upgrades the U-Boot from 2018.03 to 2018.05 release.
>>
>> Signed-off-by: Marek Vasut 
>> Cc: Otavio Salvador 
>> Cc: Richard Purdie 
>> Cc: Ross Burton 
>
> Nice work :-)


is libfdt/dtc host/target mixup faux pas fixed in this version
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [morty][PATCH 1/2] glibc: add missing TRAP_BRANCH/TRAP_HWBKPT definitions

2018-05-16 Thread Daniel Díaz
On 6 April 2018 at 15:43, akuster808  wrote:
> On 04/06/2018 10:08 AM, Daniel Díaz Rodríguez wrote:
>> Ping on this series for Morty.
> This needs to be ported to Pyro first. I do it this weekend and morty too.

Ping on this series for Morty.

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


Re: [OE-core] [PATCH] u-boot: Upgrade to 2018.05 release

2018-05-16 Thread Marek Vasut
On 05/17/2018 12:05 AM, Khem Raj wrote:
> On Wed, May 16, 2018 at 3:00 PM, Otavio Salvador
>  wrote:
>> On Wed, May 16, 2018 at 5:14 PM, Marek Vasut  wrote:
>>> This upgrades the U-Boot from 2018.03 to 2018.05 release.
>>>
>>> Signed-off-by: Marek Vasut 
>>> Cc: Otavio Salvador 
>>> Cc: Richard Purdie 
>>> Cc: Ross Burton 
>>
>> Nice work :-)
> 
> 
> is libfdt/dtc host/target mixup faux pas fixed in this version

Do a build test please, I'm not triggering it here.

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


[OE-core] [PATCH] u-boot: Upgrade to 2018.05 release

2018-05-16 Thread Marek Vasut
This upgrades the U-Boot from 2018.03 to 2018.05 release.

Signed-off-by: Marek Vasut 
Cc: Otavio Salvador 
Cc: Richard Purdie 
Cc: Ross Burton 
---
 .../u-boot/{u-boot-common_2018.03.inc => u-boot-common_2018.05.inc} | 2 +-
 .../u-boot/{u-boot-fw-utils_2018.03.bb => u-boot-fw-utils_2018.05.bb}   | 0
 .../u-boot/{u-boot-mkimage_2018.03.bb => u-boot-mkimage_2018.05.bb} | 0
 meta/recipes-bsp/u-boot/{u-boot_2018.03.bb => u-boot_2018.05.bb}| 0
 4 files changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-bsp/u-boot/{u-boot-common_2018.03.inc => 
u-boot-common_2018.05.inc} (86%)
 rename meta/recipes-bsp/u-boot/{u-boot-fw-utils_2018.03.bb => 
u-boot-fw-utils_2018.05.bb} (100%)
 rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2018.03.bb => 
u-boot-mkimage_2018.05.bb} (100%)
 rename meta/recipes-bsp/u-boot/{u-boot_2018.03.bb => u-boot_2018.05.bb} (100%)

diff --git a/meta/recipes-bsp/u-boot/u-boot-common_2018.03.inc 
b/meta/recipes-bsp/u-boot/u-boot-common_2018.05.inc
similarity index 86%
rename from meta/recipes-bsp/u-boot/u-boot-common_2018.03.inc
rename to meta/recipes-bsp/u-boot/u-boot-common_2018.05.inc
index 8ffae365a9..61566f5856 100644
--- a/meta/recipes-bsp/u-boot/u-boot-common_2018.03.inc
+++ b/meta/recipes-bsp/u-boot/u-boot-common_2018.05.inc
@@ -7,7 +7,7 @@ PE = "1"
 
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "f95ab1fb6e37f0601f397091bb011edf7a98b890"
+SRCREV = "890e79f2b1c26c5ba1a86d179706348aec7feef7"
 
 SRC_URI = "git://git.denx.de/u-boot.git"
 
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2018.03.bb 
b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2018.05.bb
similarity index 100%
rename from meta/recipes-bsp/u-boot/u-boot-fw-utils_2018.03.bb
rename to meta/recipes-bsp/u-boot/u-boot-fw-utils_2018.05.bb
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2018.03.bb 
b/meta/recipes-bsp/u-boot/u-boot-mkimage_2018.05.bb
similarity index 100%
rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2018.03.bb
rename to meta/recipes-bsp/u-boot/u-boot-mkimage_2018.05.bb
diff --git a/meta/recipes-bsp/u-boot/u-boot_2018.03.bb 
b/meta/recipes-bsp/u-boot/u-boot_2018.05.bb
similarity index 100%
rename from meta/recipes-bsp/u-boot/u-boot_2018.03.bb
rename to meta/recipes-bsp/u-boot/u-boot_2018.05.bb
-- 
2.16.2

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


[OE-core] [meta-oe][PATCH] package.bbclass: provide debug sources in externalsrc -dbg packages

2018-05-16 Thread Vignesh Rajendran
The recipes which are using externalsrc method doesn't contain the
debug sources in their -dbg packages.

Add a handling in copydebugsources function to copy the debugsources for
externalsrc -dbg packages.

Signed-off-by: Vignesh Rajendran 
---
 meta/classes/externalsrc.bbclass |  3 +++
 meta/classes/package.bbclass | 12 ++--
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/meta/classes/externalsrc.bbclass b/meta/classes/externalsrc.bbclass
index c9f5cf7..135670d 100644
--- a/meta/classes/externalsrc.bbclass
+++ b/meta/classes/externalsrc.bbclass
@@ -129,6 +129,9 @@ python () {
 d.setVar('CONFIGURESTAMPFILE', configstamp)
 d.setVar('STAMP', 
'${STAMPS_DIR}/work-shared/${PN}/${EXTENDPE}${PV}-${PR}')
 d.setVar('STAMPCLEAN', '${STAMPS_DIR}/work-shared/${PN}/*-*')
+
+   # set DEBUG_PREFIX_MAP to external source path
+   d.appendVar("DEBUG_PREFIX_MAP", 
"-fdebug-prefix-map=${EXTERNALSRC}=/usr/src/debug/${PN}/${EXTENDPE}${PV}-${PR}")
 }
 
 python externalsrc_configure_prefunc() {
diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index edeffa9..f071a2e 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -420,6 +420,7 @@ def copydebugsources(debugsrcdir, d):
 workdir = d.getVar("WORKDIR")
 workparentdir = os.path.dirname(os.path.dirname(workdir))
 workbasedir = os.path.basename(os.path.dirname(workdir)) + "/" + 
os.path.basename(workdir)
+externalsrc = d.getVar('EXTERNALSRC', True)
 
 # If build path exists in sourcefile, it means toolchain did not use
 # -fdebug-prefix-map to compile
@@ -444,9 +445,16 @@ def copydebugsources(debugsrcdir, d):
 processdebugsrc += "fgrep -zw '%s' | "
 # Remove prefix in the source paths
 processdebugsrc += "sed 's#%s##g' | "
-processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner '%s%s' 
2>/dev/null)"
+# handle external sources to provide sources in -dbg packages
+if externalsrc:
+localsrc_prefix = "/usr/src/debug/"
+processdebugsrc += "sed 's#%s/##g' | "
+processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner 
'%s%s/%s' 2>/dev/null)"
+cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workbasedir, externalsrc, dvar, debugsrcdir, workbasedir)
+else:
+processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner 
'%s%s' 2>/dev/null)"
+cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workparentdir, dvar, debugsrcdir)
 
-cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workparentdir, dvar, debugsrcdir)
 (retval, output) = oe.utils.getstatusoutput(cmd)
 # Can "fail" if internal headers/transient sources are attempted
 #if retval:
-- 
2.7.4

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


Re: [OE-core] [meta-oe][PATCH] package.bbclass: provide debug sources in externalsrc -dbg packages

2018-05-16 Thread Vignesh Rajendran (RBEI/ECF3)
Hi OE-Core members,



As per the wiki notes,  This is a reminder and re-submission again the same 
patch posted some weeks before.



https://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded

If your patch has not had any feedback after a few days it may have been missed 
or the appropriate reviewers may not currently be around; it is perfectly fine 
to reply to it yourself with a "ping" / reminder request for feedback. NOTE: 
patch review for feature / recipe upgrade patches will likely be delayed during 
a feature freeze because these types of patches aren't merged during this time 
- you may have to wait until after the freeze is lifted.



I have noted that the patch not available in master and may have been missed.



Mit freundlichen Grüßen / Best regards



Rajendran Vignesh

RBEI/ECF33



Tel. +91 422 67-65103







-Original Message-
From: Vignesh Rajendran (RBEI/ECF3)
Sent: Thursday, May 17, 2018 11:20 AM
To: openembedded-core@lists.openembedded.org
Cc: Vignesh Rajendran (RBEI/ECF3) 
Subject: [meta-oe][PATCH] package.bbclass: provide debug sources in externalsrc 
-dbg packages



The recipes which are using externalsrc method doesn't contain the

debug sources in their -dbg packages.



Add a handling in copydebugsources function to copy the debugsources for

externalsrc -dbg packages.



Signed-off-by: Vignesh Rajendran 
>

---

meta/classes/externalsrc.bbclass |  3 +++

meta/classes/package.bbclass | 12 ++--

2 files changed, 13 insertions(+), 2 deletions(-)



diff --git a/meta/classes/externalsrc.bbclass b/meta/classes/externalsrc.bbclass

index c9f5cf7..135670d 100644

--- a/meta/classes/externalsrc.bbclass

+++ b/meta/classes/externalsrc.bbclass

@@ -129,6 +129,9 @@ python () {

 d.setVar('CONFIGURESTAMPFILE', configstamp)

 d.setVar('STAMP', 
'${STAMPS_DIR}/work-shared/${PN}/${EXTENDPE}${PV}-${PR}')

 d.setVar('STAMPCLEAN', '${STAMPS_DIR}/work-shared/${PN}/*-*')

+

+ # set DEBUG_PREFIX_MAP to external source path

+ d.appendVar("DEBUG_PREFIX_MAP", 
"-fdebug-prefix-map=${EXTERNALSRC}=/usr/src/debug/${PN}/${EXTENDPE}${PV}-${PR}")

}

 python externalsrc_configure_prefunc() {

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass

index edeffa9..f071a2e 100644

--- a/meta/classes/package.bbclass

+++ b/meta/classes/package.bbclass

@@ -420,6 +420,7 @@ def copydebugsources(debugsrcdir, d):

 workdir = d.getVar("WORKDIR")

 workparentdir = os.path.dirname(os.path.dirname(workdir))

 workbasedir = os.path.basename(os.path.dirname(workdir)) + "/" + 
os.path.basename(workdir)

+externalsrc = d.getVar('EXTERNALSRC', True)

 # If build path exists in sourcefile, it means toolchain did not use

 # -fdebug-prefix-map to compile

@@ -444,9 +445,16 @@ def copydebugsources(debugsrcdir, d):

 processdebugsrc += "fgrep -zw '%s' | "

 # Remove prefix in the source paths

 processdebugsrc += "sed 's#%s##g' | "

-processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner '%s%s' 
2>/dev/null)"

+# handle external sources to provide sources in -dbg packages

+if externalsrc:

+localsrc_prefix = "/usr/src/debug/"

+processdebugsrc += "sed 's#%s/##g' | "

+processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner 
'%s%s/%s' 2>/dev/null)"

+cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workbasedir, externalsrc, dvar, debugsrcdir, workbasedir)

+else:

+processdebugsrc += "(cd '%s' ; cpio -pd0mlL --no-preserve-owner 
'%s%s' 2>/dev/null)"

+cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workparentdir, dvar, debugsrcdir)

-cmd = processdebugsrc % (sourcefile, workbasedir, localsrc_prefix, 
workparentdir, dvar, debugsrcdir)

 (retval, output) = oe.utils.getstatusoutput(cmd)

 # Can "fail" if internal headers/transient sources are attempted

 #if retval:

--

2.7.4


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