[OE-core] [PATCH] recipes: change SRC_URI to use https

[Stefan Müller-Klieser]

Change all recipes to https where we get an http 301 permanent redirect.

Signed-off-by: Stefan Müller-Klieser 
---
 meta/recipes-core/busybox/busybox.inc   | 2 +-
 meta/recipes-core/busybox/busybox_1.31.0.bb | 2 +-
 meta/recipes-core/dbus/dbus-glib_0.110.bb   | 4 ++--
 meta/recipes-core/dbus/dbus_1.12.16.bb  | 4 ++--
 meta/recipes-core/udev/eudev_3.2.8.bb   | 2 +-
 meta/recipes-devtools/patchelf/patchelf_0.10.bb | 2 +-
 meta/recipes-extended/iptables/iptables_1.8.3.bb| 6 +++---
 meta/recipes-extended/libmnl/libmnl_1.0.4.bb| 4 ++--
 meta/recipes-extended/xdg-utils/xdg-utils_1.1.3.bb  | 2 +-
 meta/recipes-extended/xz/xz_5.2.4.bb| 4 ++--
 .../hicolor-icon-theme/hicolor-icon-theme_0.17.bb   | 2 +-
 meta/recipes-kernel/powertop/powertop_2.10.bb   | 6 +++---
 meta/recipes-sato/webkit/webkitgtk_2.26.2.bb| 6 +++---
 meta/recipes-support/db/db_5.3.28.bb| 4 ++--
 meta/recipes-support/libbsd/libbsd_0.10.0.bb| 4 ++--
 meta/recipes-support/libyaml/libyaml_0.2.2.bb   | 4 ++--
 meta/recipes-support/mpfr/mpfr_4.0.2.bb | 4 ++--
 17 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/meta/recipes-core/busybox/busybox.inc 
b/meta/recipes-core/busybox/busybox.inc
index d08fa8d3d96b..bf6ddae7d197 100644
--- a/meta/recipes-core/busybox/busybox.inc
+++ b/meta/recipes-core/busybox/busybox.inc
@@ -1,6 +1,6 @@
 SUMMARY = "Tiny versions of many common UNIX utilities in a single small 
executable"
 DESCRIPTION = "BusyBox combines tiny versions of many common UNIX utilities 
into a single small executable. It provides minimalist replacements for most of 
the utilities you usually find in GNU fileutils, shellutils, etc. The utilities 
in BusyBox generally have fewer options than their full-featured GNU cousins; 
however, the options that are included provide the expected functionality and 
behave very much like their GNU counterparts. BusyBox provides a fairly 
complete POSIX environment for any small or embedded system."
-HOMEPAGE = "http://www.busybox.net;
+HOMEPAGE = "https://www.busybox.net;
 BUGTRACKER = "https://bugs.busybox.net/;
 
 DEPENDS += "kern-tools-native virtual/crypt"
diff --git a/meta/recipes-core/busybox/busybox_1.31.0.bb 
b/meta/recipes-core/busybox/busybox_1.31.0.bb
index 34b1f2cc9166..5dd8d448daf6 100644
--- a/meta/recipes-core/busybox/busybox_1.31.0.bb
+++ b/meta/recipes-core/busybox/busybox_1.31.0.bb
@@ -1,6 +1,6 @@
 require busybox.inc
 
-SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball 
\
+SRC_URI = "https://busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
file://busybox-udhcpc-no_deconfig.patch \
file://find-touchscreen.sh \
file://busybox-cron \
diff --git a/meta/recipes-core/dbus/dbus-glib_0.110.bb 
b/meta/recipes-core/dbus/dbus-glib_0.110.bb
index 9afbc2b6a11a..0b45805cf469 100644
--- a/meta/recipes-core/dbus/dbus-glib_0.110.bb
+++ b/meta/recipes-core/dbus/dbus-glib_0.110.bb
@@ -1,7 +1,7 @@
 SUMMARY = "High level language (GLib) binding for D-Bus"
 DESCRIPTION = "GLib bindings for the D-Bus message bus that integrate \
 the D-Bus library with the GLib thread abstraction and main loop."
-HOMEPAGE = "http://www.freedesktop.org/Software/dbus;
+HOMEPAGE = "https://www.freedesktop.org/Software/dbus;
 LICENSE = "AFL-2.1 | GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=cf5b3a2f7083750d504333114e738656 \
 
file://dbus/dbus-glib.h;beginline=7;endline=21;md5=7755c9d7abccd5dbd25a6a974538bb3c"
@@ -10,7 +10,7 @@ SECTION = "base"
 DEPENDS = "expat glib-2.0 virtual/libintl dbus-glib-native dbus"
 DEPENDS_class-native = "glib-2.0-native dbus-native"
 
-SRC_URI = 
"http://dbus.freedesktop.org/releases/dbus-glib/dbus-glib-${PV}.tar.gz \
+SRC_URI = 
"https://dbus.freedesktop.org/releases/dbus-glib/dbus-glib-${PV}.tar.gz \
file://no-examples.patch \
file://test-install-makefile.patch \
 "
diff --git a/meta/recipes-core/dbus/dbus_1.12.16.bb 
b/meta/recipes-core/dbus/dbus_1.12.16.bb
index 96b503687067..82bb753731d8 100644
--- a/meta/recipes-core/dbus/dbus_1.12.16.bb
+++ b/meta/recipes-core/dbus/dbus_1.12.16.bb
@@ -1,6 +1,6 @@
 SUMMARY = "D-Bus message bus"
 DESCRIPTION = "D-Bus is a message bus system, a simple way for applications to 
talk to one another. In addition to interprocess communication, D-Bus helps 
coordinate process lifecycle; it makes it simple and reliable to code a 
\"single instance\" application or daemon, and to launch applications and 
daemons on demand when their services are needed."
-HOMEPAGE = "http://dbus.freedesktop.

[OE-core] [PATCH v2] cml1.bbclass: fix undefined behavior

[Stefan Müller-Klieser]

Whenever cml1 do_configure is used with a defconfig, oldconfig waits for
input. This silently fails on recent kconfig projects with:
"Error in reading or end of file."
We cannot use a more up to date kconfig target such as olddefconfig,
because busybox does not support it.

Signed-off-by: Stefan Müller-Klieser 
---
v2:
 - switch to good old yes '' | to support busybox
 - adapt commit message

 meta/classes/cml1.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/cml1.bbclass b/meta/classes/cml1.bbclass
index 98d24cec746b..c7f6723cb3de 100644
--- a/meta/classes/cml1.bbclass
+++ b/meta/classes/cml1.bbclass
@@ -1,7 +1,7 @@
 cml1_do_configure() {
set -e
unset CFLAGS CPPFLAGS CXXFLAGS LDFLAGS
-   oe_runmake oldconfig
+   yes '' | oe_runmake oldconfig
 }
 
 EXPORT_FUNCTIONS do_configure
-- 
2.20.1

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

[OE-core] [PATCH] cml1.bbclass: fix undefined behavior

[Stefan Müller-Klieser]

Whenever cml1 do_configure is used with a defconfig, oldconfig waits for
input. This silently fails with:
"Error in reading or end of file."
Replace the call with olddefconfig, which will have the same effect in
setting all defaults, but has a defined behavior.

Signed-off-by: Stefan Müller-Klieser 
---
 meta/classes/cml1.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/cml1.bbclass b/meta/classes/cml1.bbclass
index 98d24cec746b..f3b8a84073c8 100644
--- a/meta/classes/cml1.bbclass
+++ b/meta/classes/cml1.bbclass
@@ -1,7 +1,7 @@
 cml1_do_configure() {
set -e
unset CFLAGS CPPFLAGS CXXFLAGS LDFLAGS
-   oe_runmake oldconfig
+   oe_runmake olddefconfig
 }
 
 EXPORT_FUNCTIONS do_configure
-- 
2.20.1

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

Re: [OE-core] [sumo][PATCH] coreutils: 8.29: fix native build

[Stefan Müller-Klieser]

Hi Armin,

On 19.03.19 16:13, akuster808 wrote:

On 3/14/19 8:43 AM, Stefan Müller-Klieser wrote:

This fixes the native build on glibc 2.28 hosts, by backporting two
gnulib patches from master.

This will need to be done for Thud too.
- armin


coreutils 8.30 in thud already carries those patches.
Stefan



Signed-off-by: Stefan Müller-Klieser 
---
  ...adjust-to-glibc-2.28-libio.h-removal.patch | 193 ++
  ...e-more-paranoid-about-libio.h-change.patch | 187 +
  meta/recipes-core/coreutils/coreutils_8.29.bb |   2 +
  3 files changed, 382 insertions(+)
  create mode 100644 
meta/recipes-core/coreutils/coreutils/0001-fflush-adjust-to-glibc-2.28-libio.h-removal.patch
  create mode 100644 
meta/recipes-core/coreutils/coreutils/0002-fflush-be-more-paranoid-about-libio.h-change.patch

diff --git 
a/meta/recipes-core/coreutils/coreutils/0001-fflush-adjust-to-glibc-2.28-libio.h-removal.patch
 
b/meta/recipes-core/coreutils/coreutils/0001-fflush-adjust-to-glibc-2.28-libio.h-removal.patch
new file mode 100644
index ..110b34372db1
--- /dev/null
+++ 
b/meta/recipes-core/coreutils/coreutils/0001-fflush-adjust-to-glibc-2.28-libio.h-removal.patch
@@ -0,0 +1,193 @@
+From 269a8987224ec62b7a9aa6202772b07d5be424d2 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Stefan=20M=C3=BCller-Klieser?= 
+Date: Mon, 11 Mar 2019 16:00:33 +0100
+Subject: [PATCH 1/2] fflush: adjust to glibc 2.28 libio.h removal
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Backport of patch:
+4af4a4a71827c0bc5e0ec67af23edef4f15cee8e
+Paul Eggert 
+fflush: adjust to glibc 2.28 libio.h removal
+
+Upstream-Status: Backport [master]
+
+Signed-off-by: Stefan Müller-Klieser 
+---
+ lib/fflush.c | 6 +++---
+ lib/fpending.c   | 2 +-
+ lib/fpurge.c | 2 +-
+ lib/freadahead.c | 2 +-
+ lib/freading.c   | 2 +-
+ lib/freadptr.c   | 2 +-
+ lib/freadseek.c  | 2 +-
+ lib/fseeko.c | 4 ++--
+ lib/fseterr.c| 2 +-
+ lib/stdio-impl.h | 6 ++
+ 10 files changed, 18 insertions(+), 12 deletions(-)
+
+diff --git a/lib/fflush.c b/lib/fflush.c
+index 4e65692644b1..c16da5fdcf23 100644
+--- a/lib/fflush.c
 b/lib/fflush.c
+@@ -33,7 +33,7 @@
+ #undef fflush
+
+
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+
+ /* Clear the stream's ungetc buffer, preserving the value of ftello (fp).  */
+ static void
+@@ -72,7 +72,7 @@ clear_ungetc_buffer (FILE *fp)
+
+ #endif
+
+-#if ! (defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */)
++#if ! (defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */)
+
+ # if (defined __sferror || defined __DragonFly__ || defined __ANDROID__) && 
defined __SNPT
+ /* FreeBSD, NetBSD, OpenBSD, DragonFly, Mac OS X, Cygwin, Minix 3, Android */
+@@ -148,7 +148,7 @@ rpl_fflush (FILE *stream)
+   if (stream == NULL || ! freading (stream))
+ return fflush (stream);
+
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+
+   clear_ungetc_buffer_preserving_position (stream);
+
+diff --git a/lib/fpending.c b/lib/fpending.c
+index 5811a4a74750..9e21a165e220 100644
+--- a/lib/fpending.c
 b/lib/fpending.c
+@@ -32,7 +32,7 @@ __fpending (FILE *fp)
+   /* Most systems provide FILE as a struct and the necessary bitmask in
+  , because they need it for implementing getc() and putc() as
+  fast macros.  */
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+   return fp->_IO_write_ptr - fp->_IO_write_base;
+ #elif defined __sferror || defined __DragonFly__ || defined __ANDROID__
+   /* FreeBSD, NetBSD, OpenBSD, DragonFly, Mac OS X, Cygwin, Minix 3, Android 
*/
+diff --git a/lib/fpurge.c b/lib/fpurge.c
+index 408b8fcbb3b8..3a160004d49d 100644
+--- a/lib/fpurge.c
 b/lib/fpurge.c
+@@ -62,7 +62,7 @@ fpurge (FILE *fp)
+   /* Most systems provide FILE as a struct and the necessary bitmask in
+  , because they need it for implementing getc() and putc() as
+  fast macros.  */
+-# if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++# if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+   fp->_IO_read_end = fp->_IO_read_ptr;
+   fp->_IO_write_ptr = fp->_IO_write_base;
+   /* Avoid memory leak when there is an active ungetc buffer.  */
+diff --git a/lib/freadahead.c b/lib/freadahead.c
+index f335f041ee5e..e7cb77b6722b 100644
+--- a/lib/freadahead.c
 b/lib/freadahead.c
+@@ -30,7 +30,7 @@ extern size_t __sreadahead (FILE *);
+ size_t
+ freadahead (FILE *fp)
+ {
+-#i

[OE-core] [sumo][PATCH] coreutils: 8.29: fix native build

[Stefan Müller-Klieser]

This fixes the native build on glibc 2.28 hosts, by backporting two
gnulib patches from master.

Signed-off-by: Stefan Müller-Klieser 
---
 ...adjust-to-glibc-2.28-libio.h-removal.patch | 193 ++
 ...e-more-paranoid-about-libio.h-change.patch | 187 +
 meta/recipes-core/coreutils/coreutils_8.29.bb |   2 +
 3 files changed, 382 insertions(+)
 create mode 100644 
meta/recipes-core/coreutils/coreutils/0001-fflush-adjust-to-glibc-2.28-libio.h-removal.patch
 create mode 100644 
meta/recipes-core/coreutils/coreutils/0002-fflush-be-more-paranoid-about-libio.h-change.patch

diff --git 
a/meta/recipes-core/coreutils/coreutils/0001-fflush-adjust-to-glibc-2.28-libio.h-removal.patch
 
b/meta/recipes-core/coreutils/coreutils/0001-fflush-adjust-to-glibc-2.28-libio.h-removal.patch
new file mode 100644
index ..110b34372db1
--- /dev/null
+++ 
b/meta/recipes-core/coreutils/coreutils/0001-fflush-adjust-to-glibc-2.28-libio.h-removal.patch
@@ -0,0 +1,193 @@
+From 269a8987224ec62b7a9aa6202772b07d5be424d2 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Stefan=20M=C3=BCller-Klieser?= 
+Date: Mon, 11 Mar 2019 16:00:33 +0100
+Subject: [PATCH 1/2] fflush: adjust to glibc 2.28 libio.h removal
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Backport of patch:
+4af4a4a71827c0bc5e0ec67af23edef4f15cee8e
+Paul Eggert 
+fflush: adjust to glibc 2.28 libio.h removal
+
+Upstream-Status: Backport [master]
+
+Signed-off-by: Stefan Müller-Klieser 
+---
+ lib/fflush.c | 6 +++---
+ lib/fpending.c   | 2 +-
+ lib/fpurge.c | 2 +-
+ lib/freadahead.c | 2 +-
+ lib/freading.c   | 2 +-
+ lib/freadptr.c   | 2 +-
+ lib/freadseek.c  | 2 +-
+ lib/fseeko.c | 4 ++--
+ lib/fseterr.c| 2 +-
+ lib/stdio-impl.h | 6 ++
+ 10 files changed, 18 insertions(+), 12 deletions(-)
+
+diff --git a/lib/fflush.c b/lib/fflush.c
+index 4e65692644b1..c16da5fdcf23 100644
+--- a/lib/fflush.c
 b/lib/fflush.c
+@@ -33,7 +33,7 @@
+ #undef fflush
+ 
+ 
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+ 
+ /* Clear the stream's ungetc buffer, preserving the value of ftello (fp).  */
+ static void
+@@ -72,7 +72,7 @@ clear_ungetc_buffer (FILE *fp)
+ 
+ #endif
+ 
+-#if ! (defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */)
++#if ! (defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */)
+ 
+ # if (defined __sferror || defined __DragonFly__ || defined __ANDROID__) && 
defined __SNPT
+ /* FreeBSD, NetBSD, OpenBSD, DragonFly, Mac OS X, Cygwin, Minix 3, Android */
+@@ -148,7 +148,7 @@ rpl_fflush (FILE *stream)
+   if (stream == NULL || ! freading (stream))
+ return fflush (stream);
+ 
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+ 
+   clear_ungetc_buffer_preserving_position (stream);
+ 
+diff --git a/lib/fpending.c b/lib/fpending.c
+index 5811a4a74750..9e21a165e220 100644
+--- a/lib/fpending.c
 b/lib/fpending.c
+@@ -32,7 +32,7 @@ __fpending (FILE *fp)
+   /* Most systems provide FILE as a struct and the necessary bitmask in
+  , because they need it for implementing getc() and putc() as
+  fast macros.  */
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+   return fp->_IO_write_ptr - fp->_IO_write_base;
+ #elif defined __sferror || defined __DragonFly__ || defined __ANDROID__
+   /* FreeBSD, NetBSD, OpenBSD, DragonFly, Mac OS X, Cygwin, Minix 3, Android 
*/
+diff --git a/lib/fpurge.c b/lib/fpurge.c
+index 408b8fcbb3b8..3a160004d49d 100644
+--- a/lib/fpurge.c
 b/lib/fpurge.c
+@@ -62,7 +62,7 @@ fpurge (FILE *fp)
+   /* Most systems provide FILE as a struct and the necessary bitmask in
+  , because they need it for implementing getc() and putc() as
+  fast macros.  */
+-# if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++# if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+   fp->_IO_read_end = fp->_IO_read_ptr;
+   fp->_IO_write_ptr = fp->_IO_write_base;
+   /* Avoid memory leak when there is an active ungetc buffer.  */
+diff --git a/lib/freadahead.c b/lib/freadahead.c
+index f335f041ee5e..e7cb77b6722b 100644
+--- a/lib/freadahead.c
 b/lib/freadahead.c
+@@ -30,7 +30,7 @@ extern size_t __sreadahead (FILE *);
+ size_t
+ freadahead (FILE *fp)
+ {
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+   if (fp->_IO_w

[OE-core] [PATCH] libpcre2: add packages for 16-bit and 32-bit code unit support

[Stefan Müller-Klieser]

The packages loosely follow the debian package names. In that way more
projects, e.g. Qt5 for 16-bit, are able use system libraries. This does
not change the existing default package.

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-support/libpcre/libpcre2_10.30.bb | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-support/libpcre/libpcre2_10.30.bb 
b/meta/recipes-support/libpcre/libpcre2_10.30.bb
index a7df055..46c2d63 100644
--- a/meta/recipes-support/libpcre/libpcre2_10.30.bb
+++ b/meta/recipes-support/libpcre/libpcre2_10.30.bb
@@ -33,8 +33,9 @@ EXTRA_OECONF = "\
 --enable-rebuild-chartables \
 --with-link-size=2 \
 --with-match-limit=1000 \
+--enable-pcre2-16 \
+--enable-pcre2-32 \
 "
-
 # Set LINK_SIZE in BUILD_CFLAGS given that the autotools bbclass use it to
 # set CFLAGS_FOR_BUILD, required for the libpcre build.
 BUILD_CFLAGS =+ "-DLINK_SIZE=2 -I${B}/src"
@@ -43,13 +44,15 @@ CXXFLAGS_append_powerpc = " -lstdc++"
 
 export CCLD_FOR_BUILD ="${BUILD_CCLD}"
 
-PACKAGES =+ "pcre2grep pcre2grep-doc pcre2test pcre2test-doc"
+PACKAGES =+ "libpcre2-16 libpcre2-32 pcre2grep pcre2grep-doc pcre2test 
pcre2test-doc"
 
 SUMMARY_pcre2grep = "grep utility that uses perl 5 compatible regexes"
 SUMMARY_pcre2grep-doc = "grep utility that uses perl 5 compatible regexes - 
docs"
 SUMMARY_pcre2test = "program for testing Perl-comatible regular expressions"
 SUMMARY_pcre2test-doc = "program for testing Perl-comatible regular 
expressions - docs"
 
+FILES_libpcre2-16 = "${libdir}/libpcre2-16.so.*"
+FILES_libpcre2-32 = "${libdir}/libpcre2-32.so.*"
 FILES_pcre2grep = "${bindir}/pcre2grep"
 FILES_pcre2grep-doc = "${mandir}/man1/pcre2grep.1"
 FILES_pcre2test = "${bindir}/pcre2test"
-- 
2.7.4

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

[OE-core] [PATCH] gtk+3: remove superfluous vardepsexclude

[Stefan Müller-Klieser]

This seems to be a leftover from the recipe sysroot transition. Remove
it, as it might mask configuration errors.

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-gnome/gtk+/gtk+3.inc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta/recipes-gnome/gtk+/gtk+3.inc 
b/meta/recipes-gnome/gtk+/gtk+3.inc
index 34845ae04ecd..0a357db04913 100644
--- a/meta/recipes-gnome/gtk+/gtk+3.inc
+++ b/meta/recipes-gnome/gtk+/gtk+3.inc
@@ -38,7 +38,6 @@ EXTRA_OECONF += " \
  WAYLAND_PROTOCOLS_SYSROOT_DIR=${RECIPE_SYSROOT} \
  ${@bb.utils.contains("DISTRO_FEATURES", "x11", "", 
"--disable-gtk-doc", d)} \
  "
-EXTRA_OECONF[vardepsexclude] = "MACHINE"
 
 do_compile_prepend() {
 export GIR_EXTRA_LIBS_PATH="${B}/gdk/.libs"
-- 
1.9.1

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

[OE-core] [PATCH] systemd: fix lz4 build

[Stefan Müller-Klieser]

Updating lz4 in oe-core required back porting 3d4cf7de48a from master,
as versioning scheme changed.

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 ...heck-for-lz4-in-the-old-and-new-numbering.patch | 51 ++
 meta/recipes-core/systemd/systemd_232.bb   |  1 +
 2 files changed, 52 insertions(+)
 create mode 100644 
meta/recipes-core/systemd/systemd/0021-build-sys-check-for-lz4-in-the-old-and-new-numbering.patch

diff --git 
a/meta/recipes-core/systemd/systemd/0021-build-sys-check-for-lz4-in-the-old-and-new-numbering.patch
 
b/meta/recipes-core/systemd/systemd/0021-build-sys-check-for-lz4-in-the-old-and-new-numbering.patch
new file mode 100644
index 000..a7d7461
--- /dev/null
+++ 
b/meta/recipes-core/systemd/systemd/0021-build-sys-check-for-lz4-in-the-old-and-new-numbering.patch
@@ -0,0 +1,51 @@
+From 822233bd40acda4bbfb991a31d87f4224fea Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbys...@in.waw.pl>
+Date: Wed, 23 Nov 2016 10:18:30 -0500
+Subject: [PATCH 21/21] build-sys: check for lz4 in the old and new numbering
+ scheme (#4717)
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+lz4 upstream decided to switch to an incompatible numbering scheme
+(1.7.3 follows 131, to match the so version).
+PKG_CHECK_MODULES does not allow two version matches for the same package,
+so e.g. lz4 < 10 || lz4 >= 125 cannot be used. Check twice, once for
+"new" numbers (anything below 10 is assume to be new), once for the "old"
+numbers (anything above >= 125). This assumes that the "new" versioning
+will not get to 10 to quickly. I think that's a safe assumption, lz4 is a
+mature project.
+
+Fixed #4690.
+
+Upstream-Status: Backport
+
+Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
+---
+ configure.ac | 9 ++---
+ 1 file changed, 6 insertions(+), 3 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 279ce28..782b5d7 100644
+--- a/configure.ac
 b/configure.ac
+@@ -640,10 +640,13 @@ AM_CONDITIONAL(HAVE_BZIP2, [test "$have_bzip2" = "yes"])
+ have_lz4=no
+ AC_ARG_ENABLE(lz4, AS_HELP_STRING([--disable-lz4], [Disable optional LZ4 
support]))
+ AS_IF([test "x$enable_lz4" != "xno"], [
+-PKG_CHECK_MODULES(LZ4, [ liblz4 >= 125 ],
+-   [AC_DEFINE(HAVE_LZ4, 1, [Define in LZ4 is available])
++PKG_CHECK_MODULES(LZ4, [ liblz4 < 10 ],
++   [AC_DEFINE(HAVE_LZ4, 1, [Define if LZ4 is available])
+ have_lz4=yes],
+-have_lz4=no)
++[PKG_CHECK_MODULES(LZ4, [ liblz4 >= 125 ],
++  [AC_DEFINE(HAVE_LZ4, 1, [Define if LZ4 is available])
++  have_lz4=yes],
++  have_lz4=no)])
+ AS_IF([test "x$have_lz4" = xno -a "x$enable_lz4" = xyes],
+   [AC_MSG_ERROR([*** LZ4 support requested but libraries not 
found])])
+ ])
+-- 
+1.9.1
+
diff --git a/meta/recipes-core/systemd/systemd_232.bb 
b/meta/recipes-core/systemd/systemd_232.bb
index 398cb46..6513054 100644
--- a/meta/recipes-core/systemd/systemd_232.bb
+++ b/meta/recipes-core/systemd/systemd_232.bb
@@ -33,6 +33,7 @@ SRC_URI += " \
file://0018-check-for-uchar.h-in-configure.patch \

file://0019-socket-util-don-t-fail-if-libc-doesn-t-support-IDN.patch \

file://0020-back-port-233-don-t-use-the-unified-hierarchy-for-the-systemd.patch 
\
+   
file://0021-build-sys-check-for-lz4-in-the-old-and-new-numbering.patch \
 "
 SRC_URI_append_libc-uclibc = "\

file://0002-units-Prefer-getty-to-agetty-in-console-setup-system.patch \
-- 
1.9.1

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

[OE-core] [PATCH] perf: add PACKAGECONFIG for jvmti

[Stefan Müller-Klieser]

Recent kernel build jvmti, java profiling agent, if a host jdk is
detected. The library will be packaged in -dev and throws a qa error.
Disable the jvmti build per default.

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-kernel/perf/perf.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-kernel/perf/perf.bb b/meta/recipes-kernel/perf/perf.bb
index acbb1d9..ffaa5f6 100644
--- a/meta/recipes-kernel/perf/perf.bb
+++ b/meta/recipes-kernel/perf/perf.bb
@@ -21,6 +21,7 @@ PACKAGECONFIG[tui] = ",NO_NEWT=1,libnewt slang"
 PACKAGECONFIG[libunwind] = ",NO_LIBUNWIND=1 NO_LIBDW_DWARF_UNWIND=1,libunwind"
 PACKAGECONFIG[libnuma] = ",NO_LIBNUMA=1"
 PACKAGECONFIG[systemtap] = ",NO_SDT=1,systemtap"
+PACKAGECONFIG[jvmti] = ",NO_JVMTI=1"
 
 DEPENDS = " \
 virtual/${MLPREFIX}libc \
-- 
1.9.1

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

Re: [OE-core] [PATCH] u-boot: mkimage: fix build

[Stefan Müller-Klieser]

On 07.11.2016 16:00, Burton, Ross wrote:
> On 4 November 2016 at 11:06, Stefan Müller-Klieser <
> s.mueller-klie...@phytec.de> wrote:
> 
>> This fixes the mkimage build for situations where HOSTCC and friends
>> need to be overridden and the u-boot makefile defaults don't work.
>>
> 
> I applied this on top of the u-boot upgrades and it still breaks:
> 
> make -f ./scripts/Makefile.build obj=tools
>   x86_64-poky-linux-gcc  -m64 -march=corei7 -mtune=corei7 -mfpmath=sse
> -msse4.2 --sysroot=/data/poky-master/tmp-glibc/sysroots/intel-corei7-64
> -Wp,-MD,tools/.bin2header.d -O2 -pipe -g -feliminate-unused-debug-types
> -fdebug-prefix-map=/data/poky-master/tmp-glibc/work/corei7-64-poky-linux/u-boot-mkimage/2016.09.01-r0=/usr/src/debug/u-boot-mkimage/2016.09.01-r0
> -fdebug-prefix-map=/data/poky-master/tmp-glibc/sysroots/x86_64-linux=
> -fdebug-prefix-map=/data/poky-master/tmp-glibc/sysroots/intel-corei7-64=
>  -fstack-protector-strong -pie -fpie -D_FORTIFY_SOURCE=2 -Wformat
> -Wformat-security -Werror=format-security   -DCONFIG_FIT_SIGNATURE -include
> ./include/libfdt_env.h -idirafterinclude -idirafter./arch/sandbox/include
> -I./lib/libfdt -I./tools -DCONFIG_SYS_TEXT_BASE=0 -DUSE_HOSTCC
> -D__KERNEL_STRICT_NAMES -D_GNU_SOURCE  -o tools/bin2header
> tools/bin2header.c
> ...
> cat ./Licenses/gpl-2.0.txt | gzip -9 -c | \
> tools/bin2header license_gzip > ./include/license.h
> /bin/sh: 2: tools/bin2header: not found
> tools/Makefile:256: recipe for target 'include/license.h' failed
> 
> Looks like HOSTCC isn't being set correctly.

Ok, this bin2header issue seems to be unrelated, as Marek sent a v1 upstream, 
fixing
the execution path. I am not in a hurry with the oe-core, so please ignore the 
patch.
I will send it upstream first.

Stefan

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

[OE-core] [PATCH] u-boot: mkimage: fix build

[Stefan Müller-Klieser]

This fixes the mkimage build for situations where HOSTCC and friends
need to be overridden and the u-boot makefile defaults don't work.

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 ...file-improve-cross_tools-target-usability.patch | 35 ++
 ...ile-add-override-statements-for-cross_too.patch | 41 ++
 meta/recipes-bsp/u-boot/u-boot-mkimage_2016.03.bb  |  8 -
 3 files changed, 83 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-bsp/u-boot/u-boot-mkimage/0001-tools-Makefile-improve-cross_tools-target-usability.patch
 create mode 100644 
meta/recipes-bsp/u-boot/u-boot-mkimage/0002-tools-Makefile-add-override-statements-for-cross_too.patch

diff --git 
a/meta/recipes-bsp/u-boot/u-boot-mkimage/0001-tools-Makefile-improve-cross_tools-target-usability.patch
 
b/meta/recipes-bsp/u-boot/u-boot-mkimage/0001-tools-Makefile-improve-cross_tools-target-usability.patch
new file mode 100644
index 000..4b0d0e8
--- /dev/null
+++ 
b/meta/recipes-bsp/u-boot/u-boot-mkimage/0001-tools-Makefile-improve-cross_tools-target-usability.patch
@@ -0,0 +1,35 @@
+From ad129135402b38deeb37383c86781341cf239b6a Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Stefan=20M=C3=BCller-Klieser?= <s.mueller-klie...@phytec.de>
+Date: Mon, 31 Oct 2016 10:30:18 +0100
+Subject: [PATCH 1/2] tools: Makefile: improve cross_tools target usability
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+When building the cross_tools target, HOSTCFLAGS and HOSTLDFLAGS will
+propagate to the target build. This should not happen and is easy to
+prevent.
+
+Upstream-Status: Submitted
+
+Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
+---
+ tools/Makefile | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/tools/Makefile b/tools/Makefile
+index 400588c..305336c 100644
+--- a/tools/Makefile
 b/tools/Makefile
+@@ -263,6 +263,8 @@ subdir- += env
+ 
+ ifneq ($(CROSS_BUILD_TOOLS),)
+ HOSTCC = $(CC)
++HOSTCFLAGS = $(CFLAGS)
++HOSTLDFLAGS = $(LDFLAGS)
+ 
+ quiet_cmd_crosstools_strip = STRIP   $^
+   cmd_crosstools_strip = $(STRIP) $^; touch $@
+-- 
+1.9.1
+
diff --git 
a/meta/recipes-bsp/u-boot/u-boot-mkimage/0002-tools-Makefile-add-override-statements-for-cross_too.patch
 
b/meta/recipes-bsp/u-boot/u-boot-mkimage/0002-tools-Makefile-add-override-statements-for-cross_too.patch
new file mode 100644
index 000..4c7b6d0
--- /dev/null
+++ 
b/meta/recipes-bsp/u-boot/u-boot-mkimage/0002-tools-Makefile-add-override-statements-for-cross_too.patch
@@ -0,0 +1,41 @@
+From 2772e2ad0e097590e77fa338295d839d6ae6037c Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Stefan=20M=C3=BCller-Klieser?= <s.mueller-klie...@phytec.de>
+Date: Thu, 3 Nov 2016 08:48:02 +0100
+Subject: [PATCH 2/2] tools: Makefile: add override statements for cross_tools
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+This fixes the use case where one needs to override the host compiler
+flags for building the target tools. Usually the Makefile will build the
+basic tools first using the hostcc. After that cc will be assigned to
+hostcc to build the target tools. So in this scenario overriding HOSTCC
+fails to build. Using the override statement is one possible fix.
+
+Upstream-Status: Pending
+
+Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
+---
+ tools/Makefile | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/tools/Makefile b/tools/Makefile
+index 305336c..9cb1607 100644
+--- a/tools/Makefile
 b/tools/Makefile
+@@ -262,9 +262,9 @@ $(LICENSE_H): $(obj)/bin2header 
$(srctree)/Licenses/gpl-2.0.txt
+ subdir- += env
+ 
+ ifneq ($(CROSS_BUILD_TOOLS),)
+-HOSTCC = $(CC)
+-HOSTCFLAGS = $(CFLAGS)
+-HOSTLDFLAGS = $(LDFLAGS)
++override HOSTCC = $(CC)
++override HOSTCFLAGS = $(CFLAGS)
++override HOSTLDFLAGS = $(LDFLAGS)
+ 
+ quiet_cmd_crosstools_strip = STRIP   $^
+   cmd_crosstools_strip = $(STRIP) $^; touch $@
+-- 
+1.9.1
+
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.03.bb 
b/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.03.bb
index 5025961..0749408 100644
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.03.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.03.bb
@@ -3,7 +3,13 @@ require u-boot-common_${PV}.inc
 SUMMARY = "U-Boot bootloader image creation tool"
 DEPENDS = "openssl"
 
-EXTRA_OEMAKE = 'CROSS_COMPILE="${TARGET_PREFIX}" CC="${CC} ${CFLAGS} 
${LDFLAGS}" STRIP=true V=1'
+SRC_URI += " \
+file://0001-tools-Makefile-improve-cross_tools-target-usability.patch \
+file://0002-tools-Makefile-add-override-statements-for-cross_too.patch \
+"
+
+EXTRA_OEMAKE = 'CROSS_COMPILE="${TARGET_PREFIX}" CC="${CC}" CFLAGS="${CFLAGS}" 
LDFLAGS="${LDFLAGS}" STRIP=true V=1'
+EXTRA_OEMAKE += 'HOSTCC="${BUILD_CC}&quo

Re: [OE-core] [PATCH] u-boot: mkimage: Fix build of u-boot-mkimage

[Stefan Müller-Klieser]

On 03.11.2016 00:06, Burton, Ross wrote:
> On 2 November 2016 at 19:40, Marek Vasut  wrote:
> 
>>> But u-boot-common doesn't set EXTRA_OEMAKE...
>>
>> Should be u-boot.inc , sorry.
>>
> 
> Yes, but u-book-mkimage doesn't include that file. :)
> 
> Ross

I am seeing a different problem, but I guess my solution would fix this case
here, too. So I am sending it as a new thread. I hope it helps the case and
I get some discussion about the solution.

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

Re: [OE-core] [PATCH 1/4] u-boot: Update to 2016.09.01 release

[Stefan Müller-Klieser]

On 25.10.2016 21:24, Marek Vasut wrote:
> On 10/25/2016 08:32 PM, Denys Dmytriyenko wrote:
>> On Sat, Oct 22, 2016 at 10:32:12PM +0200, Marek Vasut wrote:
>>> On 10/21/2016 09:47 AM, Burton, Ross wrote:
>>>
>>> Hi!
>>>
 On 20 October 2016 at 14:35, Marek Vasut > wrote:

 Upgrade U-Boot to the latest version.


 As usual, u-boot-mkimage broke again:
>>>
>>> That's weird, I successfully built it for nios2 during my tests.
>>> Can you tell me how I can replicate the issue , so I can test for it to
>>> prevent regression and roll out a patch ?
>>
>> Marek, Ross,
>>
>> Any progress on this? Need any help testing?
>>
> Yeah, how do you replicate this issue ?
> 
Hi!
I am just looking at a similar problem and want to jump into the discussion.
As Ross said, the problem is to not respect host/target -- cc/cflags/ldflags.
So to replicate the issue, you can use a bare minimum build host with no
cross toolchain installed, and I guess all targets will fail to build.
As this has been broken so many times, I want to discuss some possible fixes:
In the top level Makefile we have:
HOSTCC = cc
HOSTCFLAGS = ...
The problem is, you cannot properly override those variables, as they get used
a lot to do different things, e.g. in tools/Makefile we have (for cross tools
target) HOSTCC = $(CC) and for HOSTCFLAGS we have appends for configuration. 
Thats why we have the current workaround with a squashed override. I see many
possible solutions and would like to hear your opinion:
1. Make top level Makefile HOST assignments conditional "?="
   - easy
   - will probably not be accepted upstream
2. add "override" to appends in sublevel Makefiles
   - adds complexity/one level of override hierarchy
3. Don't use appends for those variables (like in the kernel Makefile), 
overrides
   in the recipe
   - clean
   - quite some rework in uboot
4. Hack around in the recipe with class overrides and exports
   - quickfix, no patch required
   - fails easily in the future

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

[OE-core] [PATCH] kernel.bbclass: add user output to savedefconfig

[Stefan Müller-Klieser]

In a similar manner to diffconfig, tell the bitbake user where the
defconfig will be saved to.

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/classes/kernel.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index db42744..ed41125 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -434,6 +434,7 @@ kernel_do_configure() {
 }
 
 do_savedefconfig() {
+   bbplain "Saving defconfig to:\n${B}/defconfig"
oe_runmake -C ${B} savedefconfig
 }
 do_savedefconfig[nostamp] = "1"
-- 
1.9.1

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

[OE-core] [PATCH 7/9] linux-libc-headers: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-kernel/linux-libc-headers/linux-libc-headers.inc | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-headers.inc 
b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers.inc
index 92acf8a..c701b39 100644
--- a/meta/recipes-kernel/linux-libc-headers/linux-libc-headers.inc
+++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers.inc
@@ -50,9 +50,6 @@ SRC_URI = 
"${KERNELORG_MIRROR}/linux/kernel/v${HEADER_FETCH_VER}/linux-${PV}.tar
 
 S = "${WORKDIR}/linux-${PV}"
 
-# For the kernel, we don't want the '-e MAKEFLAGS=' in EXTRA_OEMAKE.
-EXTRA_OEMAKE = ""
-
 do_configure() {
oe_runmake allnoconfig
 }
-- 
1.9.1

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

[OE-core] [PATCH 5/9] musl: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-core/musl/musl_git.bb | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-core/musl/musl_git.bb 
b/meta/recipes-core/musl/musl_git.bb
index b2cea7f..c07101b 100644
--- a/meta/recipes-core/musl/musl_git.bb
+++ b/meta/recipes-core/musl/musl_git.bb
@@ -26,8 +26,6 @@ DEPENDS = "virtual/${TARGET_PREFIX}binutils \
 
 export CROSS_COMPILE="${TARGET_PREFIX}"
 
-EXTRA_OEMAKE = ""
-
 LDFLAGS += "-Wl,-soname,libc.so"
 
 CONFIGUREOPTS = " \
-- 
1.9.1

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

[OE-core] [PATCH 9/9] x264: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-multimedia/x264/x264_git.bb | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-multimedia/x264/x264_git.bb 
b/meta/recipes-multimedia/x264/x264_git.bb
index 0a69909..fa44f9c 100644
--- a/meta/recipes-multimedia/x264/x264_git.bb
+++ b/meta/recipes-multimedia/x264/x264_git.bb
@@ -42,8 +42,6 @@ do_configure() {
 ./configure ${EXTRA_OECONF}
 }
 
-# Get rid of -e
-EXTRA_OEMAKE = ""
 AS = "${TARGET_PREFIX}gcc"
 
 do_install() {
-- 
1.9.1

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

[OE-core] [PATCH 8/9] systemtap: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-kernel/systemtap/systemtap-uprobes_git.bb | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-kernel/systemtap/systemtap-uprobes_git.bb 
b/meta/recipes-kernel/systemtap/systemtap-uprobes_git.bb
index 3686c2f..46820ef 100644
--- a/meta/recipes-kernel/systemtap/systemtap-uprobes_git.bb
+++ b/meta/recipes-kernel/systemtap/systemtap-uprobes_git.bb
@@ -13,8 +13,6 @@ FILESEXTRAPATHS =. "${FILE_DIRNAME}/systemtap:"
 
 FILES_${PN} += "${datadir}/systemtap/runtime/uprobes"
 
-EXTRA_OEMAKE = ""
-
 # Compile and install the uprobes kernel module on machines with utrace
 # support.  Note that staprun expects it in the systemtap/runtime directory,
 # not in /lib/modules.
-- 
1.9.1

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

[OE-core] [PATCH 6/9] lsof: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-extended/lsof/lsof_4.89.bb | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-extended/lsof/lsof_4.89.bb 
b/meta/recipes-extended/lsof/lsof_4.89.bb
index c10337b..b732cf0 100644
--- a/meta/recipes-extended/lsof/lsof_4.89.bb
+++ b/meta/recipes-extended/lsof/lsof_4.89.bb
@@ -47,8 +47,6 @@ do_configure () {
 export I = "${STAGING_INCDIR}"
 export L = "${STAGING_INCDIR}"
 
-EXTRA_OEMAKE = ""
-
 do_compile () {
oe_runmake 'CC=${CC}' 'CFGL=${LDFLAGS} -L./lib -llsof' 'DEBUG=' 
'INCL=${CFLAGS}'
 }
-- 
1.9.1

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

[OE-core] [PATCH 0/9] remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

Hi,

since 2-2-2016 the EXTRA_OEMAKE default has been set empty. The
"-e MAKEFLAGS=" got removed. I guess it is time to clean up the recipes
which need to work around those defaults. I tried my best to walk
through the include hierarchy to find any double workarounds. But
everything seems fine and a handful of different machines and distros
build fine. I hope this is reasonable.

Regards, Stefan

Stefan Müller-Klieser (9):
  autotools.bbclass: remove EXTRA_OEMAKE workaround
  distutils-common-base.bbclass: remove EXTRA_OEMAKE workaround
  kernel.bbclass: remove EXTRA_OEMAKE workaround
  ifupdown: remove EXTRA_OEMAKE workaround
  musl: remove EXTRA_OEMAKE workaround
  lsof: remove EXTRA_OEMAKE workaround
  linux-libc-headers: remove EXTRA_OEMAKE workaround
  systemtap: remove EXTRA_OEMAKE workaround
  x264: remove EXTRA_OEMAKE workaround

 meta/classes/autotools.bbclass| 2 --
 meta/classes/distutils-common-base.bbclass| 2 --
 meta/classes/kernel.bbclass   | 4 
 meta/recipes-core/ifupdown/ifupdown_0.8.2.bb  | 2 --
 meta/recipes-core/musl/musl_git.bb| 2 --
 meta/recipes-extended/lsof/lsof_4.89.bb   | 2 --
 meta/recipes-kernel/linux-libc-headers/linux-libc-headers.inc | 3 ---
 meta/recipes-kernel/systemtap/systemtap-uprobes_git.bb| 2 --
 meta/recipes-multimedia/x264/x264_git.bb  | 2 --
 9 files changed, 21 deletions(-)

-- 
1.9.1

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

[OE-core] [PATCH 4/9] ifupdown: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-core/ifupdown/ifupdown_0.8.2.bb | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-core/ifupdown/ifupdown_0.8.2.bb 
b/meta/recipes-core/ifupdown/ifupdown_0.8.2.bb
index 732b295..9de01f9 100644
--- a/meta/recipes-core/ifupdown/ifupdown_0.8.2.bb
+++ b/meta/recipes-core/ifupdown/ifupdown_0.8.2.bb
@@ -13,8 +13,6 @@ SRC_URI = 
"git://anonscm.debian.org/git/collab-maint/ifupdown.git;protocol=https
  "
 SRCREV = "05ea2fd4f49bb1201aeef2a42efbcff8f336112f"
 
-EXTRA_OEMAKE = ""
-
 S = "${WORKDIR}/git"
 
 
-- 
1.9.1

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

[OE-core] [PATCH 2/9] distutils-common-base.bbclass: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/classes/distutils-common-base.bbclass | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/classes/distutils-common-base.bbclass 
b/meta/classes/distutils-common-base.bbclass
index dee06b4..824a1b6 100644
--- a/meta/classes/distutils-common-base.bbclass
+++ b/meta/classes/distutils-common-base.bbclass
@@ -1,5 +1,3 @@
-EXTRA_OEMAKE = ""
-
 export STAGING_INCDIR
 export STAGING_LIBDIR
 
-- 
1.9.1

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

[OE-core] [PATCH 1/9] autotools.bbclass: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/classes/autotools.bbclass | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/classes/autotools.bbclass b/meta/classes/autotools.bbclass
index 076899c..bd5452f 100644
--- a/meta/classes/autotools.bbclass
+++ b/meta/classes/autotools.bbclass
@@ -19,8 +19,6 @@ def autotools_dep_prepend(d):
 
 return deps + 'gnu-config-native '
 
-EXTRA_OEMAKE = ""
-
 DEPENDS_prepend = "${@autotools_dep_prepend(d)} "
 
 inherit siteinfo
-- 
1.9.1

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

[OE-core] [PATCH 3/9] kernel.bbclass: remove EXTRA_OEMAKE workaround

[Stefan Müller-Klieser]

The default of EXTRA_OEMAKE is already empty since commit:

OE-Core rev: aeb653861a0ec39ea7a014c0622980edcbf653fa
bitbake.conf: Remove unhelpful default value for EXTRA_OEMAKE

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/classes/kernel.bbclass | 4 
 1 file changed, 4 deletions(-)

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index db42744..7ee22e1 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -156,10 +156,6 @@ UBOOT_LOADADDRESS ?= "${UBOOT_ENTRYPOINT}"
 # Some Linux kernel configurations need additional parameters on the command 
line
 KERNEL_EXTRA_ARGS ?= ""
 
-# For the kernel, we don't want the '-e MAKEFLAGS=' in EXTRA_OEMAKE.
-# We don't want to override kernel Makefile variables from the environment
-EXTRA_OEMAKE = ""
-
 KERNEL_ALT_IMAGETYPE ??= ""
 
 copy_initramfs() {
-- 
1.9.1

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

[OE-core] [PATCH v2] alsa-utils: fix installed-vs-shipped for bat

[Stefan Müller-Klieser]

The bat PACKAGECONFIG does not install the test script correctly. Fix
this by following the packaging used for the other bash scripts. While
at it, fix some tabs.

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
v2:
 - fix mixed up install logic
 - retested all PACKAGECONFIG for utils and utils-scripts w/wo bat

 meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb | 4 
 meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb | 7 ---
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb 
b/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb
index a98f9b9..fef79d9 100644
--- a/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb
+++ b/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb
@@ -10,6 +10,7 @@ RDEPENDS_${PN} += "bash"
 
 FILES_${PN} = "${sbindir}/alsaconf \
${sbindir}/alsa-info.sh \
+   ${sbindir}/alsabat-test.sh \
   "
 
 S = "${WORKDIR}/alsa-utils-${PV}"
@@ -18,4 +19,7 @@ do_install() {
install -d ${D}${sbindir}
install -m 0755 ${B}/alsaconf/alsaconf ${D}${sbindir}/
install -m 0755 ${S}/alsa-info/alsa-info.sh ${D}${sbindir}/
+   if ${@bb.utils.contains('PACKAGECONFIG', 'bat', 'true', 'false', d)}; 
then
+   install -m 0755 ${S}/bat/alsabat-test.sh ${D}${sbindir}/
+   fi
 }
diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb 
b/meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb
index 08c4646..ddb1644 100644
--- a/meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb
+++ b/meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb
@@ -99,10 +99,11 @@ do_install() {
# See alsa-utils-scripts_${PV}.bb
rm ${D}${sbindir}/alsaconf
rm ${D}${sbindir}/alsa-info.sh
+   rm -f ${D}${sbindir}/alsabat-test.sh
 
if ${@bb.utils.contains('PACKAGECONFIG', 'udev', 'false', 'true', d)}; 
then
-  # This is where alsa-utils will install its rules if we don't tell 
it anything else.
-  rm -rf ${D}/lib/udev
-  rmdir --ignore-fail-on-non-empty ${D}/lib
+   # This is where alsa-utils will install its rules if we don't 
tell it anything else.
+   rm -rf ${D}/lib/udev
+   rmdir --ignore-fail-on-non-empty ${D}/lib
fi
 }
-- 
1.9.1

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

Re: [OE-core] [PATCH] alsa-utils: fix installed-vs-shipped for bat

[Stefan Müller-Klieser]

On 05.08.2016 15:19, Burton, Ross wrote:
> On 5 August 2016 at 14:15, Stefan Müller-Klieser <
> s.mueller-klie...@phytec.de> wrote:
> 
>> +   if ${@bb.utils.contains('PACKAGECONFIG', 'bat', 'false', 'true',
>> d)}; then
>>
> 
> This means the script is installed if bat isn't selected, which I don't
> think you meant.

It seems I tricked myself when testing. I will send a v2.
Stefan

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

[OE-core] [PATCH] alsa-utils: fix installed-vs-shipped for bat

[Stefan Müller-Klieser]

The bat PACKAGECONFIG does not install the test script correctly. Fix
this by following the packaging used for the other bash scripts. While
at it, fix some tabs.

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb | 4 
 meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb | 7 ---
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb 
b/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb
index a98f9b9..4e28fef 100644
--- a/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb
+++ b/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.1.bb
@@ -10,6 +10,7 @@ RDEPENDS_${PN} += "bash"
 
 FILES_${PN} = "${sbindir}/alsaconf \
${sbindir}/alsa-info.sh \
+   ${sbindir}/alsabat-test.sh \
   "
 
 S = "${WORKDIR}/alsa-utils-${PV}"
@@ -18,4 +19,7 @@ do_install() {
install -d ${D}${sbindir}
install -m 0755 ${B}/alsaconf/alsaconf ${D}${sbindir}/
install -m 0755 ${S}/alsa-info/alsa-info.sh ${D}${sbindir}/
+   if ${@bb.utils.contains('PACKAGECONFIG', 'bat', 'false', 'true', d)}; 
then
+   install -m 0755 ${S}/bat/alsabat-test.sh ${D}${sbindir}/
+   fi
 }
diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb 
b/meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb
index 08c4646..ab07fa8 100644
--- a/meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb
+++ b/meta/recipes-multimedia/alsa/alsa-utils_1.1.1.bb
@@ -99,10 +99,11 @@ do_install() {
# See alsa-utils-scripts_${PV}.bb
rm ${D}${sbindir}/alsaconf
rm ${D}${sbindir}/alsa-info.sh
+   rm ${D}${sbindir}/alsabat-test.sh
 
if ${@bb.utils.contains('PACKAGECONFIG', 'udev', 'false', 'true', d)}; 
then
-  # This is where alsa-utils will install its rules if we don't tell 
it anything else.
-  rm -rf ${D}/lib/udev
-  rmdir --ignore-fail-on-non-empty ${D}/lib
+   # This is where alsa-utils will install its rules if we don't 
tell it anything else.
+   rm -rf ${D}/lib/udev
+   rmdir --ignore-fail-on-non-empty ${D}/lib
fi
 }
-- 
1.9.1

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

Re: [OE-core] [PATCH 0/7] dangling patches

[Stefan Müller-Klieser]

On 04.08.2016 16:33, Burton, Ross wrote:
> On 4 August 2016 at 14:27, Stefan Müller-Klieser <
> s.mueller-klie...@phytec.de> wrote:
> 
>> There are some patches floating around with no reference to it. I guess
>> one just forgot to delete them during package upgrade. They should be
>> removed for consistency.
>>
> 
> I dont suppose you have tooling to find these, or was it manual review?

I found one and wrote a small bash script for the others, as it needs to
run outside of bitbake. Nothing special, though.
Stefan

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

[OE-core] [PATCH 6/7] matchbox-desktop: remove dangling patch

[Stefan Müller-Klieser]

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 ...thing-on-delete-event-when-not-STANDALONE.patch | 60 --
 1 file changed, 60 deletions(-)
 delete mode 100644 
meta/recipes-sato/matchbox-desktop/files/0001-Do-nothing-on-delete-event-when-not-STANDALONE.patch

diff --git 
a/meta/recipes-sato/matchbox-desktop/files/0001-Do-nothing-on-delete-event-when-not-STANDALONE.patch
 
b/meta/recipes-sato/matchbox-desktop/files/0001-Do-nothing-on-delete-event-when-not-STANDALONE.patch
deleted file mode 100644
index 46397dc..000
--- 
a/meta/recipes-sato/matchbox-desktop/files/0001-Do-nothing-on-delete-event-when-not-STANDALONE.patch
+++ /dev/null
@@ -1,60 +0,0 @@
-From 5fe37ef38500f858a22ecdf9bf69bf06e213bf68 Mon Sep 17 00:00:00 2001
-From: Jussi Kukkonen <jussi.kukko...@intel.com>
-Date: Tue, 22 Mar 2016 12:41:35 +0200
-Subject: [PATCH] Do nothing on delete-event when not STANDALONE
-
-This prevents an opportunistic alt-F4 from closing the desktop.
-
-Signed-off-by: Jussi Kukkonen <jussi.kukko...@intel.com>
-Signed-off-by: Ross Burton <ross.bur...@intel.com>
-
-
-Upstream-Status: Backport

- src/desktop.c | 12 +++-
- 1 file changed, 11 insertions(+), 1 deletion(-)
-
-diff --git a/src/desktop.c b/src/desktop.c
-index d4fc2fb..94b4ddf 100644
 a/src/desktop.c
-+++ b/src/desktop.c
-@@ -122,6 +122,15 @@ load_items (TakuMenu *menu)
- 
- }
- 
-+#ifndef STANDALONE
-+static gboolean
-+delete_event_cb (GtkWidget *widget, GdkEvent *event, gpointer user_data)
-+{
-+  /* prevent default handler from destroying the window */
-+  return TRUE;
-+}
-+#endif
-+
- static void
- workarea_changed (int x, int y, int w, int h)
- {
-@@ -144,11 +153,11 @@ create_desktop (void)
-   gtk_icon_size_register ("taku-icon", 64, 64);
- 
-   window = gtk_window_new (GTK_WINDOW_TOPLEVEL);
--  g_signal_connect (window, "delete-event", G_CALLBACK (gtk_main_quit), NULL);
-   gtk_widget_set_name (window, "TakuWindow");
-   gtk_window_set_title (GTK_WINDOW (window), _("Desktop"));
- 
- #ifndef STANDALONE
-+  g_signal_connect (window, "delete-event", G_CALLBACK (delete_event_cb), 
NULL);
-   gtk_window_set_type_hint (GTK_WINDOW (window), 
GDK_WINDOW_TYPE_HINT_DESKTOP);
-   gtk_window_set_skip_taskbar_hint (GTK_WINDOW (window), TRUE);
- 
-@@ -156,6 +165,7 @@ create_desktop (void)
-   screen_w = gdk_screen_get_width (screen);
-   screen_h = gdk_screen_get_height (screen);
- #else
-+  g_signal_connect (window, "delete-event", G_CALLBACK (gtk_main_quit), NULL);
-   screen_w = 640;
-   screen_h = 480;
- #endif
--- 
-2.7.0
-
-- 
1.9.1

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

[OE-core] [PATCH 1/7] gstreamer1.0-plugins-bad: remove two dangling patches

[Stefan Müller-Klieser]

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 .../0001-glimagesink-Downrank-to-marginal.patch| 32 
 ...window-fix-memory-leak-of-navigation-thre.patch | 35 --
 2 files changed, 67 deletions(-)
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-glimagesink-Downrank-to-marginal.patch
 delete mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-glplugin-glwindow-fix-memory-leak-of-navigation-thre.patch

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-glimagesink-Downrank-to-marginal.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-glimagesink-Downrank-to-marginal.patch
deleted file mode 100644
index 1085e95..000
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-glimagesink-Downrank-to-marginal.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-From c6b37a80806f9128de47f1ccc3f2354f8d436bb6 Mon Sep 17 00:00:00 2001
-From: Alexander Kanavin <alex.kana...@gmail.com>
-Date: Thu, 24 Sep 2015 19:47:32 +0300
-Subject: [PATCH] glimagesink: Downrank to marginal
-
-On desktop, where there is good OpenGL, xvimagesink will come up first,
-on other platforms, OpenGL can't be trusted because it's either software (like
-in a VM) or broken (like on embedded)., so let ximagesink come above.
-
-Upstream-Status: Submitted [https://bugzilla.gnome.org/show_bug.cgi?id=751684]
-
-Signed-off-by: Alexander Kanavin <alex.kana...@gmail.com>

- ext/gl/gstopengl.c | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/ext/gl/gstopengl.c b/ext/gl/gstopengl.c
-index a4b2540..0ccaacd 100644
 a/ext/gl/gstopengl.c
-+++ b/ext/gl/gstopengl.c
-@@ -101,7 +101,7 @@ plugin_init (GstPlugin * plugin)
- #endif
- 
-   if (!gst_element_register (plugin, "glimagesink",
--  GST_RANK_SECONDARY, gst_gl_image_sink_bin_get_type ())) {
-+  GST_RANK_MARGINAL, gst_gl_image_sink_bin_get_type ())) {
- return FALSE;
-   }
- 
--- 
-2.1.4
-
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-glplugin-glwindow-fix-memory-leak-of-navigation-thre.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-glplugin-glwindow-fix-memory-leak-of-navigation-thre.patch
deleted file mode 100755
index 3491a15..000
--- 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-glplugin-glwindow-fix-memory-leak-of-navigation-thre.patch
+++ /dev/null
@@ -1,35 +0,0 @@
-From 95cda7fbcf1a81289d9315c801c8e2b3d896f4cb Mon Sep 17 00:00:00 2001
-From: Haihua Hu <b55...@freescale.com>
-Date: Mon, 30 Nov 2015 09:36:09 +0800
-Subject: [PATCH 2/5] [glplugin] glwindow: fix memory leak of navigation
- thread
-
-When exit navigation thread, call g_thread_join() to release
-the resource hold by it.
-
-Upstream-Status: Backport [1.7.1]
-
-bugzilla URL: https://bugzilla.gnome.org/show_bug.cgi?id=758820
-
-Signed-off-by: Haihua Hu <b55...@freescale.com>

- gst-libs/gst/gl/gstglwindow.c |3 +++
- 1 file changed, 3 insertions(+)
-
-diff --git a/gst-libs/gst/gl/gstglwindow.c b/gst-libs/gst/gl/gstglwindow.c
-index 44b74ca..42ef296 100644
 a/gst-libs/gst/gl/gstglwindow.c
-+++ b/gst-libs/gst/gl/gstglwindow.c
-@@ -343,6 +343,9 @@ gst_gl_window_finalize (GObject * object)
- while (window->nav_alive) {
-   g_cond_wait (>nav_destroy_cond, >nav_lock);
- }
-+/* release resource hold by navigation thread */
-+g_thread_join(window->priv->navigation_thread);
-+window->priv->navigation_thread = NULL;
- g_mutex_unlock (>nav_lock);
-   }
- 
--- 
-1.7.9.5
-
-- 
1.9.1

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

[OE-core] [PATCH 0/7] dangling patches

[Stefan Müller-Klieser]

There are some patches floating around with no reference to it. I guess
one just forgot to delete them during package upgrade. They should be
removed for consistency.

Stefan Müller-Klieser (7):
  gstreamer1.0-plugins-bad: remove two dangling patches
  openssl: remove dangling patch
  guile: remove dangling patch
  python-numpy: remove dangling patch
  alsa-utils: remove dangling patch
  matchbox-desktop: remove dangling patch
  matchbox-panel-2: remove dangling patch

 .../openssl/crypto_use_bigint_in_x86-64_perl.patch |  35 -
 .../files/opensuse/guile-turn-off-gc-test.patch|  49 ---
 .../python-numpy/files/add-glibc-check.patch   |  36 -
 ...1-alsabat-rename-to-avoid-naming-conflict.patch | 153 -
 .../0001-glimagesink-Downrank-to-marginal.patch|  32 -
 ...window-fix-memory-leak-of-navigation-thre.patch |  35 -
 ...thing-on-delete-event-when-not-STANDALONE.patch |  60 
 ...top-Make-sure-active-state-is-initialized.patch |  50 ---
 8 files changed, 450 deletions(-)
 delete mode 100644 
meta/recipes-connectivity/openssl/openssl/crypto_use_bigint_in_x86-64_perl.patch
 delete mode 100644 
meta/recipes-devtools/guile/files/opensuse/guile-turn-off-gc-test.patch
 delete mode 100644 
meta/recipes-devtools/python-numpy/files/add-glibc-check.patch
 delete mode 100644 
meta/recipes-multimedia/alsa/alsa-utils/0001-alsabat-rename-to-avoid-naming-conflict.patch
 delete mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0001-glimagesink-Downrank-to-marginal.patch
 delete mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0002-glplugin-glwindow-fix-memory-leak-of-navigation-thre.patch
 delete mode 100644 
meta/recipes-sato/matchbox-desktop/files/0001-Do-nothing-on-delete-event-when-not-STANDALONE.patch
 delete mode 100644 
meta/recipes-sato/matchbox-panel-2/files/0001-showdesktop-Make-sure-active-state-is-initialized.patch

-- 
1.9.1

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

[OE-core] [PATCH 2/7] openssl: remove dangling patch

[Stefan Müller-Klieser]

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 .../openssl/crypto_use_bigint_in_x86-64_perl.patch | 35 --
 1 file changed, 35 deletions(-)
 delete mode 100644 
meta/recipes-connectivity/openssl/openssl/crypto_use_bigint_in_x86-64_perl.patch

diff --git 
a/meta/recipes-connectivity/openssl/openssl/crypto_use_bigint_in_x86-64_perl.patch
 
b/meta/recipes-connectivity/openssl/openssl/crypto_use_bigint_in_x86-64_perl.patch
deleted file mode 100644
index 4f81d85..000
--- 
a/meta/recipes-connectivity/openssl/openssl/crypto_use_bigint_in_x86-64_perl.patch
+++ /dev/null
@@ -1,35 +0,0 @@
-Upstream-Status: Backport
-
-When building on x32 systems where the default type is 32bit, make sure
-we can transparently represent 64bit integers.  Otherwise we end up with
-build errors like:
-/usr/bin/perl asm/ghash-x86_64.pl elf > ghash-x86_64.s
-Integer overflow in hexadecimal number at asm/../../perlasm/x86_64-xlate.pl 
line 201, <> line 890.
-...
-ghash-x86_64.s: Assembler messages:
-ghash-x86_64.s:890: Error: junk '.15473355479995e+19' after expression
-
-We don't enable this globally as there are some cases where we'd get
-32bit values interpreted as unsigned when we need them as signed.
-
-Reported-by: Bertrand Jacquin <bertr...@jacquin.bzh>
-URL: https://bugs.gentoo.org/542618
-
-Signed-off-by: Armin Kuster <akus...@mvista.com>
-
-Index: openssl-1.0.2a/crypto/perlasm/x86_64-xlate.pl
-===
 openssl-1.0.2a.orig/crypto/perlasm/x86_64-xlate.pl
-+++ openssl-1.0.2a/crypto/perlasm/x86_64-xlate.pl
-@@ -194,7 +194,10 @@ my %globals;
- }
- sub out {
-   my $self = shift;
--
-+  # When building on x32 ABIs, the expanded hex value might be too
-+  # big to fit into 32bits. Enable transparent 64bit support here
-+  # so we can safely print it out.
-+  use bigint;
-   if ($gas) {
-   # Solaris /usr/ccs/bin/as can't handle multiplications
-   # in $self->{value}
-- 
1.9.1

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

[OE-core] [PATCH 3/7] guile: remove dangling patch

[Stefan Müller-Klieser]

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 .../files/opensuse/guile-turn-off-gc-test.patch| 49 --
 1 file changed, 49 deletions(-)
 delete mode 100644 
meta/recipes-devtools/guile/files/opensuse/guile-turn-off-gc-test.patch

diff --git 
a/meta/recipes-devtools/guile/files/opensuse/guile-turn-off-gc-test.patch 
b/meta/recipes-devtools/guile/files/opensuse/guile-turn-off-gc-test.patch
deleted file mode 100644
index e2a353e..000
--- a/meta/recipes-devtools/guile/files/opensuse/guile-turn-off-gc-test.patch
+++ /dev/null
@@ -1,49 +0,0 @@
-Upstream-Status: Inappropriate [opensuse patch]
-
-Signed-off-by: Nitin A Kamble <nitin.a.kam...@intel.com>
-
-See http://debbugs.gnu.org/cgi/bugreport.cgi?bug=10096
-why this test is turned off.
-Index: guile-2.0.3/test-suite/tests/gc.test
-===
 guile-2.0.3.orig/test-suite/tests/gc.test  2011-10-15 09:34:40.0 
-0700
-+++ guile-2.0.3/test-suite/tests/gc.test   2012-01-13 21:52:10.282540355 
-0800
-@@ -65,23 +65,23 @@
-   foo)))
- 
- 
--(with-test-prefix "gc"
--  (pass-if "Unused modules are removed"
--(let* ((guard (make-guardian))
--   (total 1000))
--
--  (for-each (lambda (x) (guard (make-module))) (iota total))
-+;;(with-test-prefix "gc"
-+;;  (pass-if "Unused modules are removed"
-+;;(let* ((guard (make-guardian))
-+;;   (total 1000))
-+;;
-+;;  (for-each (lambda (x) (guard (make-module))) (iota total))
- 
-   ;; Avoid false references to the modules on the stack.
--  (stack-cleanup 20)
-+;;  (stack-cleanup 20)
- 
--  (gc)
--  (gc) ;; twice: have to kill the weak vectors.
--  (gc) ;; thrice: because the test doesn't succeed with only
-+;;  (gc)
-+;;  (gc) ;; twice: have to kill the weak vectors.
-+;;  (gc) ;; thrice: because the test doesn't succeed with only
-;; one gc round. not sure why.
- 
--  (= (let lp ((i 0))
--   (if (guard)
--   (lp (1+ i))
--   i))
-- total
-+;;  (= (let lp ((i 0))
-+;;   (if (guard)
-+;;   (lp (1+ i))
-+;;   i))
-+;; total
-- 
1.9.1

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

[OE-core] [PATCH 5/7] alsa-utils: remove dangling patch

[Stefan Müller-Klieser]

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 ...1-alsabat-rename-to-avoid-naming-conflict.patch | 153 -
 1 file changed, 153 deletions(-)
 delete mode 100644 
meta/recipes-multimedia/alsa/alsa-utils/0001-alsabat-rename-to-avoid-naming-conflict.patch

diff --git 
a/meta/recipes-multimedia/alsa/alsa-utils/0001-alsabat-rename-to-avoid-naming-conflict.patch
 
b/meta/recipes-multimedia/alsa/alsa-utils/0001-alsabat-rename-to-avoid-naming-conflict.patch
deleted file mode 100644
index b6d7d4f..000
--- 
a/meta/recipes-multimedia/alsa/alsa-utils/0001-alsabat-rename-to-avoid-naming-conflict.patch
+++ /dev/null
@@ -1,153 +0,0 @@
-From 28c29dece6c434e7d3d01e80ba7ddf6fba65 Mon Sep 17 00:00:00 2001
-From: "Lu, Han" <han...@intel.com>
-Date: Wed, 3 Feb 2016 14:20:33 +0800
-Subject: [PATCH] alsabat: rename to avoid naming conflict
-
-alsa-utils as well as bareos-bat (as well a some Bacula packages)
-all contain a program called /usr/bin/bat, which causes conflicts on
-various distributions ("basic audio tester" vs "bareos administration
-tool"("bacula administration tool")).
-Rename to avoid conflict.
-
-Signed-off-by: Lu, Han <han...@intel.com>
-Signed-off-by: Takashi Iwai <ti...@suse.de>
-
-Upstream-Status: Accepted [expected in version 1.1.1]
-
-Signed-off-by: Tanu Kaskinen <ta...@iki.fi>

- bat/Makefile.am  | 10 +-
- bat/{bat.1 => alsabat.1} | 43 ++-
- bat/bat.c|  2 +-
- 3 files changed, 28 insertions(+), 27 deletions(-)
- rename bat/{bat.1 => alsabat.1} (72%)
-
-diff --git a/bat/Makefile.am b/bat/Makefile.am
-index 842ae6b..985f00c 100644
 a/bat/Makefile.am
-+++ b/bat/Makefile.am
-@@ -1,9 +1,9 @@
--bin_PROGRAMS = bat
--man_MANS = bat.1
-+bin_PROGRAMS = alsabat
-+man_MANS = alsabat.1
- 
--EXTRA_DIST = bat.1
-+EXTRA_DIST = alsabat.1
- 
--bat_SOURCES = \
-+alsabat_SOURCES = \
-   bat.c \
-   common.c \
-   analyze.c \
-@@ -21,4 +21,4 @@ noinst_HEADERS = \
- AM_CPPFLAGS = \
- -Wall -I$(top_srcdir)/include
- 
--bat_LDADD = @FFTW_LIB@
-+alsabat_LDADD = @FFTW_LIB@
-diff --git a/bat/bat.1 b/bat/alsabat.1
-similarity index 72%
-rename from bat/bat.1
-rename to bat/alsabat.1
-index e00fc27..8d0b9c0 100644
 a/bat/bat.1
-+++ b/bat/alsabat.1
-@@ -1,23 +1,23 @@
--.TH BAT 1 "20th October 2015"
-+.TH ALSABAT 1 "20th October 2015"
- .SH NAME
--bat \- command\-line sound tester for ALSA sound card driver
-+alsabat \- command\-line sound tester for ALSA sound card driver
- 
- .SH SYNOPSIS
--\fBbat\fP [\fIflags\fP]
-+\fBalsabat\fP [\fIflags\fP]
- 
- .SH DESCRIPTION
--\fBBAT(Basic Audio Tester)\fP is a simple command\-line utility intended
--to help automate audio driver and sound server testing with little human
--interaction. BAT can be used to test audio quality, stress test features
--and test audio before and after PM state changes.
-+\fBALSABAT(ALSA Basic Audio Tester)\fP is a simple command\-line utility
-+intended to help automate audio driver and sound server testing with little
-+human interaction. ALSABAT can be used to test audio quality, stress test
-+features and test audio before and after PM state changes.
- 
--BAT's design is relatively simple. BAT plays an audio stream and captures
--the same stream in either a digital or analog loop back. It then compares
--the captured stream using a FFT to the original to determine if the test
--case passes or fails.
-+ALSABAT's design is relatively simple. ALSABAT plays an audio stream and
-+captures the same stream in either a digital or analog loop back. It then
-+compares the captured stream using a FFT to the original to determine if
-+the test case passes or fails.
- 
--BAT can either run wholly on the target machine being tested (standalone
--mode) or can run as a client/server mode where by bat client runs on the
-+ALSABAT can either run wholly on the target machine being tested (standalone
-+mode) or can run as a client/server mode where by alsabat client runs on the
- target and runs as a server on a separate tester machine. The client/server
- mode still requires some manual interaction for synchronization, but this
- is actively being developed for future releases.
-@@ -93,8 +93,8 @@ Frequencies with amplitude larger than threshold will be 
recognized as a
- peak, and the frequency with largest peak value will be recognized as a
- detected frequency.
- .br
--BAT then compares the detected frequency to target frequency, to decide
--if the detecting passes or fails.
-+ALSABAT then compares the detected frequency to target frequency, to
-+decide if the detecting passes or fails.
- .br
- The default value is 3.0.
- .TP
-@@ -117,18 +117,19 @@ Target WAV file to save capture test content.
- .TP
- \fI\-\-local\fP
- Internal loopback mode.
--Playback, capture and analysis internal to BAT only. This is intended for

[OE-core] [PATCH 7/7] matchbox-panel-2: remove dangling patch

[Stefan Müller-Klieser]

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 ...top-Make-sure-active-state-is-initialized.patch | 50 --
 1 file changed, 50 deletions(-)
 delete mode 100644 
meta/recipes-sato/matchbox-panel-2/files/0001-showdesktop-Make-sure-active-state-is-initialized.patch

diff --git 
a/meta/recipes-sato/matchbox-panel-2/files/0001-showdesktop-Make-sure-active-state-is-initialized.patch
 
b/meta/recipes-sato/matchbox-panel-2/files/0001-showdesktop-Make-sure-active-state-is-initialized.patch
deleted file mode 100644
index 3bab6d2..000
--- 
a/meta/recipes-sato/matchbox-panel-2/files/0001-showdesktop-Make-sure-active-state-is-initialized.patch
+++ /dev/null
@@ -1,50 +0,0 @@
-From a0b5fca04da208990f3c40e65dbb045da4cdb35e Mon Sep 17 00:00:00 2001
-From: Jussi Kukkonen <jussi.kukko...@intel.com>
-Date: Sun, 20 Mar 2016 13:24:37 +0200
-Subject: [PATCH] showdesktop: Make sure active state is initialized
-
-There are cases (in qemu at least) where set_active()
-is never called on startup. Make sure we initialize the active
-state so the icon gets loaded and the applet is not confused
-about the current state.
-
-Signed-off-by: Jussi Kukkonen <jussi.kukko...@intel.com>
-Signed-off-by: Ross Burton <ross.bur...@intel.com>
-
-
-Upstream-Status: Backport

- applets/showdesktop/showdesktop.c | 10 ++
- 1 file changed, 10 insertions(+)
-
-diff --git a/applets/showdesktop/showdesktop.c 
b/applets/showdesktop/showdesktop.c
-index 61dd5c2..33c13e1 100644
 a/applets/showdesktop/showdesktop.c
-+++ b/applets/showdesktop/showdesktop.c
-@@ -181,6 +181,12 @@ button_clicked_cb (GtkButton *button,
-   );
- }
- 
-+static void
-+realize_cb (GtkWidget *button, ShowDesktopApplet *applet)
-+{
-+sync_applet (applet);
-+}
-+
- G_MODULE_EXPORT GtkWidget *
- mb_panel_applet_create (const char*id,
- GtkOrientation orientation)
-@@ -216,6 +222,10 @@ mb_panel_applet_create (const char*id,
-   "clicked",
-   G_CALLBACK (button_clicked_cb),
-   applet);
-+g_signal_connect (button,
-+  "realize",
-+  G_CALLBACK (realize_cb),
-+  applet);
- 
- g_object_weak_ref (G_OBJECT (button),
-(GWeakNotify) show_desktop_applet_free,
--- 
-2.7.0
-
-- 
1.9.1

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

[OE-core] [PATCH 4/7] python-numpy: remove dangling patch

[Stefan Müller-Klieser]

Signed-off-by: Stefan Müller-Klieser <s.mueller-klie...@phytec.de>
---
 .../python-numpy/files/add-glibc-check.patch   | 36 --
 1 file changed, 36 deletions(-)
 delete mode 100644 
meta/recipes-devtools/python-numpy/files/add-glibc-check.patch

diff --git a/meta/recipes-devtools/python-numpy/files/add-glibc-check.patch 
b/meta/recipes-devtools/python-numpy/files/add-glibc-check.patch
deleted file mode 100644
index 29768f3..000
--- a/meta/recipes-devtools/python-numpy/files/add-glibc-check.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-Dont punish musl for glibc's trignometeric functions problems
-additionally ensure that glibc specific macros are not used
-unconditionally
-
-Signed-off-by: Khem Raj <raj.k...@gmail.com>
-Upstream-Status: Pending
-Index: numpy-1.10.4/numpy/core/src/private/npy_config.h
-===
 numpy-1.10.4.orig/numpy/core/src/private/npy_config.h
-+++ numpy-1.10.4/numpy/core/src/private/npy_config.h
-@@ -75,10 +75,12 @@
- 
- #if defined(HAVE_FEATURES_H)
- #include 
-+#ifdef __GLIBC__
- #define TRIG_OK __GLIBC_PREREQ(2, 16)
- #else
- #define TRIG_OK 0
- #endif
-+#endif
- 
- #if !TRIG_OK
- #undef HAVE_CASIN
-Index: numpy-1.10.4/numpy/core/src/npymath/ieee754.c.src
-===
 numpy-1.10.4.orig/numpy/core/src/npymath/ieee754.c.src
-+++ numpy-1.10.4/numpy/core/src/npymath/ieee754.c.src
-@@ -612,7 +612,7 @@ void npy_set_floatstatus_invalid(void)
- }
- 
- 
--#elif defined(__GLIBC__) || defined(__APPLE__) || \
-+#elif defined(__linux__) || defined(__APPLE__) || \
-   defined(__CYGWIN__) || defined(__MINGW32__) || \
-   (defined(__FreeBSD__) && (__FreeBSD_version >= 502114))
- #  include 
-- 
1.9.1

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

[OE-core] [PATCH] libvorbis: remove legacy options

[Stefan Müller-Klieser]

Those code fragments date back to ancient times. EXTRA_OECONF is not
required anymore and we should give gcc another try to do it right.
Testing on cortex-a8 with thumb and -O2 reveals a performance boost of
82 percent during encoding in comparison to -O0. -O4 gives another 7
percent.

Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
---
- this requires patch 0fe770ef9817319bb82 from master-next. It is
  already for version 1.3.5

 meta/recipes-multimedia/libvorbis/libvorbis_1.3.5.bb | 7 ---
 1 file changed, 7 deletions(-)

diff --git a/meta/recipes-multimedia/libvorbis/libvorbis_1.3.5.bb 
b/meta/recipes-multimedia/libvorbis/libvorbis_1.3.5.bb
index b7a0667..636e0f3 100644
--- a/meta/recipes-multimedia/libvorbis/libvorbis_1.3.5.bb
+++ b/meta/recipes-multimedia/libvorbis/libvorbis_1.3.5.bb
@@ -15,10 +15,3 @@ SRC_URI[md5sum] = 28cb28097c07a735d6af56e598e1c90f
 SRC_URI[sha256sum] = 
54f94a9527ff0a88477be0a71c0bab09a4c3febe0ed878b24824906cd4b0e1d1
 
 inherit autotools pkgconfig
-
-# vorbisfile.c reveals a problem in the gcc register spilling for the
-# thumb instruction set...
-FULL_OPTIMIZATION_thumb = -O0
-
-EXTRA_OECONF = --with-ogg-libraries=${STAGING_LIBDIR} \
---with-ogg-includes=${STAGING_INCDIR}
-- 
1.9.1

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

Re: [OE-core] [PATCH] kernel.bbclass: Fix do_shared_workdir task ordering

[Stefan Müller-Klieser]

On 11.08.2015 15:11, Bruce Ashfield wrote:

On Tue, Aug 11, 2015 at 9:06 AM, Bruce Ashfield
bruce.ashfi...@gmail.com wrote:

On Tue, Aug 11, 2015 at 8:47 AM, Otavio Salvador
otavio.salva...@ossystems.com.br wrote:

On Tue, Aug 11, 2015 at 12:23 AM, Bruce Ashfield
bruce.ashfi...@gmail.com wrote:

On Mon, Aug 10, 2015 at 11:21 AM, Stefan Müller-Klieser
s.mueller-klie...@phytec.de wrote:

commit 02d0a003d60326 [kernel.bbclass: Fix race condition] has surfaced
a bug in the generation of the shared_workdir. The task
do_compile_kernelmodules adds the exported symbols of the kernel modules
to the Module.symvers. By creating the shared_workdir before the modules
are compiled, the symbols of the modules are missing in the
shared_workdir. Subsequent external module builds will not include the
ABI CRC of functions exported in modules. Modprobe will fail to load the
external module if CONFIG_MODVERSIONS is enabled.\


Have you seen our bug: https://bugzilla.yoctoproject.org/show_bug.cgi?id=8127 ?

It's new .. so probably not.

I did not. Thanks.



The significant issue with this, is that we are now forcing anyone
that needs the
shared workdir artifacts to build kernel modules.

That's by design. The artifacts are modified by the module build.



That's performance issue for many workflows.

I had some changes where I was working to short cut parts of the process, but
they turned out to miss a few corner cases.

We need to do more thinking on this one, before we can bring in a change like
this .. since avoiding that overhead is something valuable.
So you are saying a fast build is more important than a correct build? 
That's quite a bold statement.




I agree that performance is important but correctness seems more
valuable for me. I think the optimization can come as a subsequent
patch ...


Let's disagree on this point.

There's time to get this right. We have a bug to track it, so we wont'
release with
the active bug, and this only hits a very tiny set of users.

So we are going to step back and try and fix this right.
Well, if you really want to do this then there should at least be a 
module-interdepend.bbclass not using the shared workdir and depending on 
the modules build. Fido and master are broken at the moment.


Regards,

Stefan



I hit send too soon. I have a suggestion in the bug already, so it isn't like
we are talking about letting this sit for weeks.

History shows that we are very unlikely to loop back and fix the performance
of perf or other builds once the change goes in. So in the absence of
other concrete suggestions, looking into some other small changes is a good
use of time.

Cheers,

Bruce




Bruce



--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750




--
Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end





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

[OE-core] [PATCH] kernel.bbclass: Fix do_shared_workdir task ordering

[Stefan Müller-Klieser]

commit 02d0a003d60326 [kernel.bbclass: Fix race condition] has surfaced
a bug in the generation of the shared_workdir. The task
do_compile_kernelmodules adds the exported symbols of the kernel modules
to the Module.symvers. By creating the shared_workdir before the modules
are compiled, the symbols of the modules are missing in the
shared_workdir. Subsequent external module builds will not include the
ABI CRC of functions exported in modules. Modprobe will fail to load the
external module if CONFIG_MODVERSIONS is enabled.

Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
---
 meta/classes/kernel.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index d06f6cf..473f1f8 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -249,7 +249,7 @@ kernel_do_install() {
 }
 do_install[prefuncs] += package_get_auto_pr
 
-addtask shared_workdir after do_compile before do_compile_kernelmodules
+addtask shared_workdir after do_compile_kernelmodules before do_install
 addtask shared_workdir_setscene
 
 do_shared_workdir_setscene () {
-- 
1.9.1

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

[OE-core] [PATCH] cml1.bbclass: make menuconfig task configurable

[Stefan Müller-Klieser]

Menuconfig is sometimes not the most convenient way to configure a
kernel, as the default color scheme can be hard to read. This variable
gives you access to the MENUCONFIG_COLOR variable or lets you select
nconfig, e.g. by setting it in your local.conf:

KCONFIG_CONFIG_COMMAND = MENUCONFIG_COLOR=mono menuconfig
or
KCONFIG_CONFIG_COMMAND = nconfig

Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
---
 meta/classes/cml1.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/cml1.bbclass b/meta/classes/cml1.bbclass
index 43acfd5..95cf584 100644
--- a/meta/classes/cml1.bbclass
+++ b/meta/classes/cml1.bbclass
@@ -16,6 +16,7 @@ CROSS_CURSES_LIB = -lncurses -ltinfo
 CROSS_CURSES_INC = '-DCURSES_LOC=curses.h'
 TERMINFO = ${STAGING_DATADIR_NATIVE}/terminfo
 
+KCONFIG_CONFIG_COMMAND ??= menuconfig
 python do_menuconfig() {
 import shutil
 
@@ -25,7 +26,7 @@ python do_menuconfig() {
 except OSError:
 mtime = 0
 
-oe_terminal(${SHELL} -c \make menuconfig; if [ \$? -ne 0 ]; then echo 
'Command failed.'; printf 'Press any key to continue... '; read r; fi\, 
'${PN} Configuration', d)
+oe_terminal(${SHELL} -c \make ${KCONFIG_CONFIG_COMMAND}; if [ \$? -ne 0 
]; then echo 'Command failed.'; printf 'Press any key to continue... '; read r; 
fi\, '${PN} Configuration', d)
 
 # FIXME this check can be removed when the minimum bitbake version has 
been bumped
 if hasattr(bb.build, 'write_taint'):
-- 
1.9.1

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

[OE-core] [PATCH v2] gcc-4.9: backport from gcc trunk r212178

[Stefan Müller-Klieser]

When compiling meta-toolchain-qt5 on cortexa8, the compiler throws an
internal compiler error:

...
qttools-opensource-src-5.3.2/src/linguist/shared/po.cpp:
In function 'bool loadPO(Translator, QIODevice, ConversionData)':
qttools-opensource-src-5.3.2/src/linguist/shared/po.cpp:717:1:
internal compiler error: in add_stores, at var-tracking.c:6000
...

Tracking this down led to https://bugs.linaro.org/show_bug.cgi?id=534
It seems the bug is well know and fixed upstream. So backporting from
trunk seems to be the right solution. This fixes the compiler problem
on cortexa8 and does not seem to be very invasive. The original commit
can be found at:

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@212178 
138bc75d-0d04-0410-961f-82ee72b054a4

Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
---
v2: bug is still present in master, so this patch applies on master.
It is relevant for more gcc versions. I tested master and dizzy.
---
 meta/recipes-devtools/gcc/gcc-4.9.inc  |  1 +
 ...racking.c-backport-from-gcc-trunk-r212178.patch | 39 ++
 2 files changed, 40 insertions(+)
 create mode 100644 
meta/recipes-devtools/gcc/gcc-4.9/0062-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch

diff --git a/meta/recipes-devtools/gcc/gcc-4.9.inc 
b/meta/recipes-devtools/gcc/gcc-4.9.inc
index 3af87d1..ee31f0b 100644
--- a/meta/recipes-devtools/gcc/gcc-4.9.inc
+++ b/meta/recipes-devtools/gcc/gcc-4.9.inc
@@ -77,6 +77,7 @@ SRC_URI = \
 file://0059-gcc-PR-rtl-optimization-63348.patch \
 file://0060-Only-allow-e500-double-in-SPE_SIMD_REGNO_P-registers.patch \
 file://0061-target-gcc-includedir.patch \
+file://0062-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch \
 
 SRC_URI[md5sum] = 4df8ee253b7f3863ad0b86359cd39c43
 SRC_URI[sha256sum] = 
2020c98295856aa13fda0f2f3a4794490757fc24bcca918d52cc8b4917b972dd
diff --git 
a/meta/recipes-devtools/gcc/gcc-4.9/0062-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch
 
b/meta/recipes-devtools/gcc/gcc-4.9/0062-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch
new file mode 100644
index 000..c0ea62f
--- /dev/null
+++ 
b/meta/recipes-devtools/gcc/gcc-4.9/0062-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch
@@ -0,0 +1,39 @@
+From b30ffb8097749fdb55704aa7d8307ca1a58255d6 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Stefan=20M=C3=BCller-Klieser?= s.mueller-klie...@phytec.de
+Date: Tue, 7 Apr 2015 16:15:11 +0200
+Subject: [PATCH] gcc/var-tracking.c: backport from gcc trunk r212178
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+resolves a bug seen on cortexa8 building qt5 libraries.
+
+2014-06-30  Joseph Myers  jos...@codesourcery.com
+
+* var-tracking.c (add_stores): Return instead of asserting if old
+and new values for conditional store are the same.
+
+git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@212178 
138bc75d-0d04-0410-961f-82ee72b054a4
+
+Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
+---
+ gcc/var-tracking.c | 3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+diff --git a/gcc/var-tracking.c b/gcc/var-tracking.c
+index 65d8285..7c38910 100644
+--- a/gcc/var-tracking.c
 b/gcc/var-tracking.c
+@@ -5997,7 +5997,8 @@ add_stores (rtx loc, const_rtx expr, void *cuip)
+ {
+   cselib_val *oval = cselib_lookup (oloc, GET_MODE (oloc), 0, VOIDmode);
+ 
+-  gcc_assert (oval != v);
++  if (oval == v)
++return;
+   gcc_assert (REG_P (oloc) || MEM_P (oloc));
+ 
+   if (oval  !cselib_preserved_value_p (oval))
+-- 
+1.9.1
+
-- 
1.9.1

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

Re: [OE-core] [meta-oe][dizzy][PATCH] gcc-4.9: backport from gcc trunk r212178

[Stefan Müller-Klieser]

Hi Richard,

Richard Purdie richard.pur...@linuxfoundation.org wrote on 04/20/2015 
04:34:20 PM:

 Von: Richard Purdie richard.pur...@linuxfoundation.org
 An: Stefan Müller-Klieser s.mueller-klie...@phytec.de
 Kopie: openembedded-core@lists.openembedded.org
 Datum: 04/20/2015 04:34 PM
 Betreff: Re: [OE-core] [meta-oe][dizzy][PATCH] gcc-4.9: backport 
 from gcc trunk r212178
 
 On Mon, 2015-04-20 at 12:18 +0200, Stefan Müller-Klieser wrote:
  When compiling meta-toolchain-qt5 on cortexa8, the compiler throws an
  internal compiler error:
  
  ...
  qttools-opensource-src-5.3.2/src/linguist/shared/po.cpp:
  In function 'bool loadPO(Translator, QIODevice, ConversionData)':
  qttools-opensource-src-5.3.2/src/linguist/shared/po.cpp:717:1:
  internal compiler error: in add_stores, at var-tracking.c:6000
  ...
  
  Tracking this down led to https://bugs.linaro.org/show_bug.cgi?id=534
  It seems the bug is well know and fixed upstream. So backporting from
  trunk seems to be the right solution. This fixes the compiler problem
  on cortexa8 and does not seem to be very invasive. The original commit
  can be found at:
  
  git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@212178 
 138bc75d-0d04-0410-961f-82ee72b054a4
  
  Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
  ---
   meta/recipes-devtools/gcc/gcc-4.9.inc  |  1 +
   ...racking.c-backport-from-gcc-trunk-r212178.patch | 39 +
 +
   2 files changed, 40 insertions(+)
   create mode 100644 meta/recipes-devtools/gcc/gcc-4.9/0060-gcc-
 var-tracking.c-backport-from-gcc-trunk-r212178.patch
 
 Does this need to be added to master before we can backport to dizzy?
I thought Yocto will resolve those dependencies automatically ;-) No for 
real, I don't use master at the moment. But I managed to quickly test the 
patch on master with qt5.3.2 on the beagleboneblack. The bug still exists 
and is fixed with the patch. I will send a new one...

Regards, Stefan

 
 Cheers,
 
 Richard
 

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

[OE-core] [meta-oe][dizzy][PATCH] gcc-4.9: backport from gcc trunk r212178

[Stefan Müller-Klieser]

When compiling meta-toolchain-qt5 on cortexa8, the compiler throws an
internal compiler error:

...
qttools-opensource-src-5.3.2/src/linguist/shared/po.cpp:
In function 'bool loadPO(Translator, QIODevice, ConversionData)':
qttools-opensource-src-5.3.2/src/linguist/shared/po.cpp:717:1:
internal compiler error: in add_stores, at var-tracking.c:6000
...

Tracking this down led to https://bugs.linaro.org/show_bug.cgi?id=534
It seems the bug is well know and fixed upstream. So backporting from
trunk seems to be the right solution. This fixes the compiler problem
on cortexa8 and does not seem to be very invasive. The original commit
can be found at:

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@212178 
138bc75d-0d04-0410-961f-82ee72b054a4

Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
---
 meta/recipes-devtools/gcc/gcc-4.9.inc  |  1 +
 ...racking.c-backport-from-gcc-trunk-r212178.patch | 39 ++
 2 files changed, 40 insertions(+)
 create mode 100644 
meta/recipes-devtools/gcc/gcc-4.9/0060-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch

diff --git a/meta/recipes-devtools/gcc/gcc-4.9.inc 
b/meta/recipes-devtools/gcc/gcc-4.9.inc
index c88a496..e9412e3 100644
--- a/meta/recipes-devtools/gcc/gcc-4.9.inc
+++ b/meta/recipes-devtools/gcc/gcc-4.9.inc
@@ -73,6 +73,7 @@ SRC_URI = \
 file://0056-top-level-reorder_gcc-bug-61144.patch \
 file://0058-gcc-r212171.patch \
 file://0059-gcc-PR-rtl-optimization-63348.patch \
+file://0060-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch \
 file://target-gcc-includedir.patch \
 
 SRC_URI[md5sum] = fddf71348546af523353bd43d34919c1
diff --git 
a/meta/recipes-devtools/gcc/gcc-4.9/0060-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch
 
b/meta/recipes-devtools/gcc/gcc-4.9/0060-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch
new file mode 100644
index 000..c0ea62f
--- /dev/null
+++ 
b/meta/recipes-devtools/gcc/gcc-4.9/0060-gcc-var-tracking.c-backport-from-gcc-trunk-r212178.patch
@@ -0,0 +1,39 @@
+From b30ffb8097749fdb55704aa7d8307ca1a58255d6 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Stefan=20M=C3=BCller-Klieser?= s.mueller-klie...@phytec.de
+Date: Tue, 7 Apr 2015 16:15:11 +0200
+Subject: [PATCH] gcc/var-tracking.c: backport from gcc trunk r212178
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+resolves a bug seen on cortexa8 building qt5 libraries.
+
+2014-06-30  Joseph Myers  jos...@codesourcery.com
+
+* var-tracking.c (add_stores): Return instead of asserting if old
+and new values for conditional store are the same.
+
+git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@212178 
138bc75d-0d04-0410-961f-82ee72b054a4
+
+Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
+---
+ gcc/var-tracking.c | 3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+diff --git a/gcc/var-tracking.c b/gcc/var-tracking.c
+index 65d8285..7c38910 100644
+--- a/gcc/var-tracking.c
 b/gcc/var-tracking.c
+@@ -5997,7 +5997,8 @@ add_stores (rtx loc, const_rtx expr, void *cuip)
+ {
+   cselib_val *oval = cselib_lookup (oloc, GET_MODE (oloc), 0, VOIDmode);
+ 
+-  gcc_assert (oval != v);
++  if (oval == v)
++return;
+   gcc_assert (REG_P (oloc) || MEM_P (oloc));
+ 
+   if (oval  !cselib_preserved_value_p (oval))
+-- 
+1.9.1
+
-- 
1.9.1

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

Re: [OE-core] [RFC 1/1] add support for the barebox bootloader

[Stefan Müller-Klieser]

On 30.07.2014 18:53, Saul Wold wrote:

On 07/30/2014 04:49 AM, Stefan Müller-Klieser wrote:

Barebox is a bootloader that inherits the best of U-Boot and the Linux
kernel. The barebox uses kconfig and kbuild mechanisms similar to the
Linux kernel. This is reflected in the recipe. As soon as this
functionality gets more abstraction in classes, the barebox recipe can
be simplified. The recipe supports: kconfig handling, kconfig fragments,
packages for dev tools and support for the initial program loader
creation.

Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
---
  meta/recipes-bsp/barebox/barebox-ipl.inc  |  37 +
  meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb |   4 +
  meta/recipes-bsp/barebox/barebox.inc  | 161
++
  meta/recipes-bsp/barebox/barebox_2014.07.0.bb |   4 +
  4 files changed, 206 insertions(+)
  create mode 100644 meta/recipes-bsp/barebox/barebox-ipl.inc
  create mode 100644 meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb
  create mode 100644 meta/recipes-bsp/barebox/barebox.inc
  create mode 100644 meta/recipes-bsp/barebox/barebox_2014.07.0.bb

diff --git a/meta/recipes-bsp/barebox/barebox-ipl.inc
b/meta/recipes-bsp/barebox/barebox-ipl.inc
new file mode 100644
index 000..f388522
--- /dev/null
+++ b/meta/recipes-bsp/barebox/barebox-ipl.inc
@@ -0,0 +1,37 @@
+# Copyright (C) 2014 Stefan Mueller-Klieser
s.mueller-klie...@phytec.de \
+# PHYTEC Messtechnik GmbH
+# Recipe released under the GPL-3.0 license
+

Before I review this much further, I have to ask why are you licensing
the recipe meta-data as GPL-3.0?  All the rest of the meta-data is MIT.
  I also note that barebox itself is GPL-2.0.

Thanks
 Sau!


Thats our default license. GPL3 is modern and better compatible with the 
huge number of other free licenses. I can easily remove the line for 
upstream work, so it will be MIT too.





+require barebox.inc
+
+DESCRIPTION = Barebox IPL Inintial Program Loader: some machines need \
+   a pre-bootloader. This one gets build by the barebox.
+PROVIDES = barebox-ipl
+S = ${WORKDIR}/barebox-${PV}
+
+# Pre Bootloader
+BAREBOX_IPL_BIN ?= barebox.bin
+BAREBOX_IPL_BIN_DEPLOY ?= IPL-${MACHINE}-${PV}-${PR}.bin
+BAREBOX_IPL_BIN_SYMLINK ?= IPL
+BAREBOX_IPL_DEFAULT_DEFCONFIG ?= am335x_mlo_defconfig
+
+do_default_defconfig () {
+bbnote generating .config for target
${BAREBOX_IPL_DEFAULT_DEFCONFIG}
+unset CFLAGS LDFLAGS
+oe_runmake ${BAREBOX_IPL_DEFAULT_DEFCONFIG}
+}
+
+do_install () {
+bbnote Installing barebox-ipl image on target rootfs
+install -d ${D}${base_bootdir}
+install ${S}/${BAREBOX_IPL_BIN}
${D}${base_bootdir}/${BAREBOX_IPL_BIN_DEPLOY}
+ln -sf ${BAREBOX_IPL_BIN_DEPLOY}
${D}${base_bootdir}/${BAREBOX_IPL_BIN_SYMLINK}
+}
+FILES_${PN} = 
+FILES_${PN} = ${base_bootdir}
+
+do_deploy () {
+bbnote Deploying barebox ipl
+install ${S}/${BAREBOX_IPL_BIN}
${DEPLOYDIR}/${BAREBOX_IPL_BIN_DEPLOY}
+ln -sf ${BAREBOX_IPL_BIN_DEPLOY}
${DEPLOYDIR}/${BAREBOX_IPL_BIN_SYMLINK}
+}
diff --git a/meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb
b/meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb
new file mode 100644
index 000..372828a
--- /dev/null
+++ b/meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb
@@ -0,0 +1,4 @@
+require barebox-ipl.inc
+
+SRC_URI[md5sum] = f3a93950777916d4b1c2cd366aec8d91
+SRC_URI[sha256sum] =
6dca5c78de8565a15aaa404edbf1bc4663415f0dc69e094f3b621636876285f2
diff --git a/meta/recipes-bsp/barebox/barebox.inc
b/meta/recipes-bsp/barebox/barebox.inc
new file mode 100644
index 000..b7885df
--- /dev/null
+++ b/meta/recipes-bsp/barebox/barebox.inc
@@ -0,0 +1,161 @@
+# Copyright (C) 2014 Stefan Mueller-Klieser
s.mueller-klie...@phytec.de
+# PHYTEC Messtechnik GmbH
+# Recipe released under the GPL-3.0 license
+#
+# TODO: see if there is some development towards moving all kconfig
+#   funcionality into the kconfig-frontends package and adjust
+#   the recipe accordingly.
+#
+
+SUMMARY = The Barebox Bootloader
+DESCRIPTION = Barebox (formerly known as u-boot-v2) is a bootloader
that \
+   inherits the best of U-Boot and the Linux kernel: The
size and \
+   look-and-feel of u-boot, with driver model and lots of
design \
+   concepts from the kernel. \
+   The bb recipe adds config fragment support and kconfig \
+   command handling using the yocto kern-tools-native.
+HOMEPAGE = http://www.barebox.org;
+SECTION = bootloaders
+PROVIDES = virtual/bootloader barebox
+LICENSE = GPL-2.0
+LIC_FILES_CHKSUM = file://COPYING;md5=057bf9e50e1ca857d0eb97bfe4ba8e5d
+SRC_URI = http://barebox.org/download/barebox-${PV}.tar.bz2;
+
+inherit kernel-arch cml1 deploy
+DEPENDS += kern-tools-native
+PACKAGE_ARCH = ${MACHINE_ARCH}
+# verbose build 0/1
+VERBOSE_BUILD ?= 0
+EXTRA_OEMAKE = 'V=${VERBOSE_BUILD} CROSS_COMPILE=${TARGET_PREFIX}
CC=${TARGET_PREFIX}gcc ${TOOLCHAIN_OPTIONS}'
+# target rootfs boot directory
+base_bootdir = /boot

Re: [OE-core] [PATCH] barebox: Add recipe

[Stefan Müller-Klieser]

Hello everybody,

it happened that I am also preparing our barebox recipe for OE-core. I 
started similar to David but it evolved already. I added support for 
packages, configuration fragments and an the ipl, needed for some SoCs. 
I will send it to the list for comments. Even if it will not be 
accepted, it might be useful for some people. I will provide an 
explanation for the maintainer.


Regards, Stefan

On 28.07.2014 12:23, Richard Purdie wrote:

On Mon, 2014-07-28 at 11:55 +0200, David Vincent wrote:

barebox (formerly known as u-boot-v2) is a bootloader that inherits the
best of U-Boot and the Linux kernel. This commit adds the possibility to
use it as a virtual/bootloader instead of U-Boot.

Signed-off-by: David Vincent freesili...@gmail.com
---
  meta/recipes-bsp/barebox/barebox.inc  |   73 +
  meta/recipes-bsp/barebox/barebox_2014.07.0.bb |   15 +
  2 files changed, 88 insertions(+)
  create mode 100644 meta/recipes-bsp/barebox/barebox.inc
  create mode 100644 meta/recipes-bsp/barebox/barebox_2014.07.0.bb


I have to admit I don't know a lot about this. To make it into OE-Core
we need some kind of story about why its necessary and commonly used.
Are there a number of BSPs using this? Can you give some background to
why this is needed?

Cheers,

Richard


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

[OE-core] [RFC 0/1] add support for the barebox bootloader

[Stefan Müller-Klieser]

I would be happy about comments on our barebox recipe and provide a 
story for it to be moved into OE-core:


The barebox is a bootloader created by and for kernel developers. Its 
system design and user interface is kept in close relation to the linux 
kernel. It has support for arm/blackfin/mips/ppc/x86... Most of the 
supported boards are current arm SoCs. As seen on

https://www.openhub.net/p/barebox
the community is very active with 1801 commits from 66 contributors in 
the last year (u-boot:3547/319). It gets maintained by pengutronix a big 
linux kernel contributor from Germany.
Currently, as far as I can see, only freescale supports barebox in the 
meta-fsl-arm bsp layer. As barebox is used by many arm SoC board 
vendors, e.g. we here at phytec.de use barebox, the missing support in 
OE-core could be one reason to see few public bsp layers to be pushed. 
Its the most crucial part of the bsp. We maintain our own version of the 
barebox recipe at the moment. It would be helpful to see it becoming 
mainline in OE-core as this could create some traction towards including 
barebox support in more bsp layers, as a lot of machines are supported 
by barebox.
Maybe there is a better place to put it in, or a better way of 
implementing it, as David already said.

Anyway, I hope this recipe will be helpful for someone.

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

[OE-core] [RFC 1/1] add support for the barebox bootloader

[Stefan Müller-Klieser]

Barebox is a bootloader that inherits the best of U-Boot and the Linux
kernel. The barebox uses kconfig and kbuild mechanisms similar to the
Linux kernel. This is reflected in the recipe. As soon as this
functionality gets more abstraction in classes, the barebox recipe can
be simplified. The recipe supports: kconfig handling, kconfig fragments,
packages for dev tools and support for the initial program loader
creation.

Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
---
 meta/recipes-bsp/barebox/barebox-ipl.inc  |  37 +
 meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb |   4 +
 meta/recipes-bsp/barebox/barebox.inc  | 161 ++
 meta/recipes-bsp/barebox/barebox_2014.07.0.bb |   4 +
 4 files changed, 206 insertions(+)
 create mode 100644 meta/recipes-bsp/barebox/barebox-ipl.inc
 create mode 100644 meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb
 create mode 100644 meta/recipes-bsp/barebox/barebox.inc
 create mode 100644 meta/recipes-bsp/barebox/barebox_2014.07.0.bb

diff --git a/meta/recipes-bsp/barebox/barebox-ipl.inc 
b/meta/recipes-bsp/barebox/barebox-ipl.inc
new file mode 100644
index 000..f388522
--- /dev/null
+++ b/meta/recipes-bsp/barebox/barebox-ipl.inc
@@ -0,0 +1,37 @@
+# Copyright (C) 2014 Stefan Mueller-Klieser s.mueller-klie...@phytec.de \
+# PHYTEC Messtechnik GmbH
+# Recipe released under the GPL-3.0 license
+
+require barebox.inc
+
+DESCRIPTION = Barebox IPL Inintial Program Loader: some machines need \
+   a pre-bootloader. This one gets build by the barebox.
+PROVIDES = barebox-ipl
+S = ${WORKDIR}/barebox-${PV}
+
+# Pre Bootloader 
+BAREBOX_IPL_BIN ?= barebox.bin
+BAREBOX_IPL_BIN_DEPLOY ?= IPL-${MACHINE}-${PV}-${PR}.bin
+BAREBOX_IPL_BIN_SYMLINK ?= IPL
+BAREBOX_IPL_DEFAULT_DEFCONFIG ?= am335x_mlo_defconfig
+
+do_default_defconfig () {
+   bbnote generating .config for target ${BAREBOX_IPL_DEFAULT_DEFCONFIG}
+   unset CFLAGS LDFLAGS
+   oe_runmake ${BAREBOX_IPL_DEFAULT_DEFCONFIG}
+}
+
+do_install () {
+   bbnote Installing barebox-ipl image on target rootfs
+   install -d ${D}${base_bootdir}
+   install ${S}/${BAREBOX_IPL_BIN} 
${D}${base_bootdir}/${BAREBOX_IPL_BIN_DEPLOY}
+   ln -sf ${BAREBOX_IPL_BIN_DEPLOY} 
${D}${base_bootdir}/${BAREBOX_IPL_BIN_SYMLINK}
+}
+FILES_${PN} = 
+FILES_${PN} = ${base_bootdir}
+
+do_deploy () {
+   bbnote Deploying barebox ipl
+   install ${S}/${BAREBOX_IPL_BIN} ${DEPLOYDIR}/${BAREBOX_IPL_BIN_DEPLOY}
+   ln -sf ${BAREBOX_IPL_BIN_DEPLOY} ${DEPLOYDIR}/${BAREBOX_IPL_BIN_SYMLINK}
+}
diff --git a/meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb 
b/meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb
new file mode 100644
index 000..372828a
--- /dev/null
+++ b/meta/recipes-bsp/barebox/barebox-ipl_2014.07.0.bb
@@ -0,0 +1,4 @@
+require barebox-ipl.inc
+
+SRC_URI[md5sum] = f3a93950777916d4b1c2cd366aec8d91
+SRC_URI[sha256sum] = 
6dca5c78de8565a15aaa404edbf1bc4663415f0dc69e094f3b621636876285f2
diff --git a/meta/recipes-bsp/barebox/barebox.inc 
b/meta/recipes-bsp/barebox/barebox.inc
new file mode 100644
index 000..b7885df
--- /dev/null
+++ b/meta/recipes-bsp/barebox/barebox.inc
@@ -0,0 +1,161 @@
+# Copyright (C) 2014 Stefan Mueller-Klieser s.mueller-klie...@phytec.de
+# PHYTEC Messtechnik GmbH
+# Recipe released under the GPL-3.0 license
+#
+# TODO: see if there is some development towards moving all kconfig
+#   funcionality into the kconfig-frontends package and adjust
+#   the recipe accordingly.
+#
+
+SUMMARY = The Barebox Bootloader
+DESCRIPTION = Barebox (formerly known as u-boot-v2) is a bootloader that \
+   inherits the best of U-Boot and the Linux kernel: The size and \
+  look-and-feel of u-boot, with driver model and lots of design \
+  concepts from the kernel. \
+  The bb recipe adds config fragment support and kconfig \
+  command handling using the yocto kern-tools-native.
+HOMEPAGE = http://www.barebox.org;
+SECTION = bootloaders
+PROVIDES = virtual/bootloader barebox
+LICENSE = GPL-2.0
+LIC_FILES_CHKSUM = file://COPYING;md5=057bf9e50e1ca857d0eb97bfe4ba8e5d
+SRC_URI = http://barebox.org/download/barebox-${PV}.tar.bz2;
+
+inherit kernel-arch cml1 deploy
+DEPENDS += kern-tools-native
+PACKAGE_ARCH = ${MACHINE_ARCH}
+# verbose build 0/1
+VERBOSE_BUILD ?= 0
+EXTRA_OEMAKE = 'V=${VERBOSE_BUILD} CROSS_COMPILE=${TARGET_PREFIX} 
CC=${TARGET_PREFIX}gcc ${TOOLCHAIN_OPTIONS}'
+# target rootfs boot directory
+base_bootdir = /boot
+
+# these variables need to be assigned in the machine.conf 
+# name of the image created by barebox Makefile
+BAREBOX_BIN ?= barebox.bin
+# filename of file to be deployed by this recipe
+BAREBOX_BIN_DEPLOY ?= barebox-${MACHINE}-${PV}-${PR}.bin
+BAREBOX_BIN_SYMLINK ?= barebox.bin
+BAREBOX_DEFAULT_DEFCONFIG ?= 
+# barebox environment
+BAREBOX_ENV ?= defaultenv/barebox_default_env
+BAREBOX_ENV_DEPLOY ?= bareboxenv-${MACHINE}-${PV}-${PR

[OE-core] [PATCH] cml1: correct diffconfig output format

[Stefan Müller-Klieser]

If used with some packages using kconfig mechanism, the diffconfig
command generates wrong output format. Diff provides all options to
format the output correctly. This method formats as intended, is more
robust and works with the merge_config.h script from yocto-kernel-tools.

Signed-off-by: Stefan Müller-Klieser s.mueller-klie...@phytec.de
---
 meta/classes/cml1.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/cml1.bbclass b/meta/classes/cml1.bbclass
index 34c0c4e..0d4b10b 100644
--- a/meta/classes/cml1.bbclass
+++ b/meta/classes/cml1.bbclass
@@ -58,7 +58,7 @@ python do_diffconfig() {
 bb.fatal(No config files found. Did you do menuconfig ?\n%s % e)
 
 if isdiff:
-statement = 'diff -Nurp ' + configorig + ' ' + config + '| sed -n 
s/^\+//p ' + fragment
+statement = 'diff --unchanged-line-format= --old-line-format= 
--new-line-format=%L' + configorig + ' ' + config + '' + fragment
 subprocess.call(statement, shell=True)
 
 shutil.copy(configorig, config)
-- 
1.9.1

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