Re: [OE-core] [PATCH 1/1] glibc: refresh patch to also fix qemuppc

2018-10-08 Thread ChenQi 

Richard, I'll consider more about maintenance issue next time.
Martin, thanks for taking care of this.

Best Regards,
Chen Qi

On 09/29/2018 08:55 PM, Martin Jansa wrote:
I'm refreshing the patch based on feedback from upstream, I'll 
integrate the changes for ppc as well.


On Sat, Sep 29, 2018 at 2:26 PM Richard Purdie 
> wrote:


On Sat, 2018-09-29 at 10:58 +0800, Chen Qi wrote:
> The patch 0031-sysdeps-ieee754-prevent-maybe-uninitialized-
> errors.patch
> did not take into consideration of qemuppc. Refresh it to also fix
> building for qemuppc.
>
> The related error message is like below:
>
>   ../sysdeps/ieee754/ldbl-128ibm/e_jnl.c:153:20: error: 'temp'
may be
> used uninitialized in this function [-Werror=maybe-uninitialized]
>
> Signed-off-by: Chen Qi mailto:qi.c...@windriver.com>>
> ---
>  ...eee754-prevent-maybe-uninitialized-errors.patch | 41
> +-
>  1 file changed, 32 insertions(+), 9 deletions(-)
>
> diff --git a/meta/recipes-core/glibc/glibc/0031-sysdeps-ieee754-
> prevent-maybe-uninitialized-errors.patch b/meta/recipes-
> core/glibc/glibc/0031-sysdeps-ieee754-prevent-maybe-uninitialized-
> errors.patch
> index b02c4ec..1c5b254 100644
> --- a/meta/recipes-core/glibc/glibc/0031-sysdeps-ieee754-prevent-
> maybe-uninitialized-errors.patch
> +++ b/meta/recipes-core/glibc/glibc/0031-sysdeps-ieee754-prevent-
> maybe-uninitialized-errors.patch
> @@ -1,4 +1,4 @@
> -From c6cc5a6ef46837e341fe271b5ffa6def23810082 Mon Sep 17 00:00:00
> 2001
> +From 113241a738662854256c5e2e415397721b842862 Mon Sep 17 00:00:00
> 2001
>  From: Martin Jansa mailto:martin.ja...@gmail.com>>
>  Date: Fri, 14 Sep 2018 23:23:03 +
>  Subject: [PATCH] sysdeps/ieee754: prevent maybe-uninitialized
errors
> @@ -21,13 +21,14 @@ Upstream-Status: Submitted
[https://www.sourcewar
> e.org/ml/libc-alpha/2018-09/msg


I get nervous when we modify a patch submitted upstream. Was the patch
submitted accepted? Do we need to send this other piece upstream?

Cheers,

Richard

-- 
___

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

http://lists.openembedded.org/mailman/listinfo/openembedded-core



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

Re: [OE-core] [PATCH 1/2] webkitgtk: add opengl to REQUIRED_DISTRO_FEATURES

2018-10-08 Thread Khem Raj 
On Mon, Oct 8, 2018 at 6:47 PM ChenQi  wrote:
>
> ping
>
> I think webkitgtk does require the 'opengl' distro feature.
>
> We have in mesa.inc:
> PACKAGECONFIG ??= " \
> ${@bb.utils.contains('DISTRO_FEATURES', 'opengl',
> 'opengl egl gles gbm dri', '', d)} \
> ..."
>
> PROVIDES = " \
>  ${@bb.utils.contains('PACKAGECONFIG', 'opengl', 'virtual/libgl',
> '', d)} \
>  ${@bb.utils.contains('PACKAGECONFIG', 'gles', 'virtual/libgles1
> virtual/libgles2', '', d)} \
>  ..."
>
> And webkitgtk's source codes are like:
> #if USE(LIBEPOXY)
> // libepoxy headers have to be included before  in
> order to avoid
> // picking up khrplatform.h inclusion that's done in ANGLE.
> #include 
> #endif
>
> #include 
> #include 
>
> #if PLATFORM(IOS)
> #import 
> #elif PLATFORM(MAC)
> #include 
> #elif PLATFORM(WIN)
> #include "OpenGLESShims.h"
> #elif USE(LIBEPOXY)
> //  already included above.
> #elif USE(OPENGL_ES_2)
> #include 
> #else
> #include "OpenGLShims.h"
> #endif
>
> And libepoxy is wrapper around libgl.
>
> So I think we can conclude 'opengl' is required distro feature for
> webkitgkt? At least this is the current situation in OE.

some sort of GL is needed yes. We indicate that with opengl losely
in OE

>
> Best Regards,
> Chen Qi
>
> On 09/03/2018 05:35 PM, Hongxu Jia wrote:
> > They can't be built without opengl in DISTRO_FEATURES.
> > [snip]
> > |webkitgtk-2.20.3/Source/WebCore/platform/graphics/OpenGLShims.h:23:10:
> > fatal error: GL/gl.h: No such file or directory
> > [snip]
> >
> > Signed-off-by: Hongxu Jia 
> > ---
> >   meta/recipes-sato/webkit/webkitgtk_2.20.3.bb | 3 +--
> >   1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb 
> > b/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
> > index a528c5d..93d74bb 100644
> > --- a/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
> > +++ b/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
> > @@ -29,8 +29,7 @@ SRC_URI[sha256sum] = 
> > "579b307d78fa16a73f112790178022166d7ad17d4147492ced9b9fba48
> >
> >   inherit cmake pkgconfig gobject-introspection perlnative 
> > distro_features_check upstream-version-is-even gtk-doc
> >
> > -# depends on libxt
> > -REQUIRED_DISTRO_FEATURES = "x11"
> > +REQUIRED_DISTRO_FEATURES = "x11 opengl"
> >
> >   DEPENDS = "zlib libsoup-2.4 curl libxml2 cairo libxslt libxt libidn 
> > libgcrypt \
> >  gtk+3 gstreamer1.0 gstreamer1.0-plugins-base flex-native 
> > gperf-native sqlite3 \
>
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 2/2] webkitgtk: fix to build without opengl

2018-10-08 Thread ChenQi 

On 10/01/2018 07:11 PM, Burton, Ross wrote:

>From the libepoxy web site:

"Epoxy is a library for handling OpenGL function pointer management for you"

libepoxy is essentially a glorified wrapper around dlopen()/dlsym().
Are you simply moving the failure from build time to run time?
libepoxy should be pulling in a GL implementation because that's what
it does.

Ross


Thanks for pointing out the problem.
I think we can now conclude that 'opengl' is necessary for webkitgtk?
I've pinged a patch which Hongxu sent some time ago.

Best Regards,
Chen Qi


On Sat, 29 Sep 2018 at 10:34, Chen Qi  wrote:

Currently, webkitgtk cannot be built if 'opengl' is not in distro
feature. The error message is like below:

   OpenGLShims.h:23:10: fatal error: GL/gl.h: No such file or directory

The required distro feature for webkitgtk is only 'x11'.

 From its source codes, libepoxy is an alternative option to build webkitgtk.
So fix to use it. In this way, we can successfully build out
webkitgtk if 'opengl' is not in distro features.

Signed-off-by: Chen Qi 
---
  .../0001-fix-to-build-with-libepoxy.patch  | 35 ++
  meta/recipes-sato/webkit/webkitgtk_2.20.5.bb   |  4 ++-
  2 files changed, 38 insertions(+), 1 deletion(-)
  create mode 100644 
meta/recipes-sato/webkit/webkitgtk/0001-fix-to-build-with-libepoxy.patch

diff --git 
a/meta/recipes-sato/webkit/webkitgtk/0001-fix-to-build-with-libepoxy.patch 
b/meta/recipes-sato/webkit/webkitgtk/0001-fix-to-build-with-libepoxy.patch
new file mode 100644
index 000..72ccef4
--- /dev/null
+++ b/meta/recipes-sato/webkit/webkitgtk/0001-fix-to-build-with-libepoxy.patch
@@ -0,0 +1,35 @@
+From 59c04daa31f8083bf4eff8304b13f34dd79d00a3 Mon Sep 17 00:00:00 2001
+From: Chen Qi 
+Date: Sat, 29 Sep 2018 16:32:57 +0800
+Subject: [PATCH] fix to build with libepoxy
+
+In source codes, there are places checking USE(LIBEPOXY), but
+in cmake files, there's nowhere to handle it.
+
+Fix to define USE_LIBEPOXY to 1 if ENABLE_LIBEPOXY is turned on.
+
+Upstream-Status: Pending
+
+Signed-off-by: Chen Qi 
+---
+ Source/cmake/OptionsGTK.cmake | 4 
+ 1 file changed, 4 insertions(+)
+
+diff --git a/Source/cmake/OptionsGTK.cmake b/Source/cmake/OptionsGTK.cmake
+index 386fa1d..6e86257 100644
+--- a/Source/cmake/OptionsGTK.cmake
 b/Source/cmake/OptionsGTK.cmake
+@@ -240,6 +240,10 @@ endif ()
+
+ SET_AND_EXPOSE_TO_BUILD(USE_TEXTURE_MAPPER TRUE)
+
++if (ENABLE_LIBEPOXY)
++SET_AND_EXPOSE_TO_BUILD(USE_LIBEPOXY TRUE)
++endif ()
++
+ if (ENABLE_OPENGL)
+ # ENABLE_OPENGL is true if either USE_OPENGL or ENABLE_GLES2 is true.
+ # But USE_OPENGL is the opposite of ENABLE_GLES2.
+--
+2.7.4
+
diff --git a/meta/recipes-sato/webkit/webkitgtk_2.20.5.bb 
b/meta/recipes-sato/webkit/webkitgtk_2.20.5.bb
index c8d4f94..24d10a2 100644
--- a/meta/recipes-sato/webkit/webkitgtk_2.20.5.bb
+++ b/meta/recipes-sato/webkit/webkitgtk_2.20.5.bb
@@ -23,6 +23,7 @@ SRC_URI = 
"http://www.webkitgtk.org/releases/${BPN}-${PV}.tar.xz \
 file://0012-soup-Forward-declare-URL-class.patch \
 
file://0001-Fix-PaintingData-has-no-member-named-lightVector-on-.patch \
 file://0001-webkitgtk-fix-CVE-2017-17821.patch \
+   file://0001-fix-to-build-with-libepoxy.patch \
 "

  SRC_URI[md5sum] = "72a05f6a4dc1c78b079590a8fd280401"
@@ -42,7 +43,7 @@ DEPENDS = "zlib libsoup-2.4 curl libxml2 cairo libxslt libxt 
libidn libgcrypt \
"

  PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11', 
'wayland' ,d)} \
-   ${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'webgl 
opengl', '' ,d)} \
+   ${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'webgl 
opengl', 'libepoxy' ,d)} \
 enchant \
 libsecret \
"
@@ -52,6 +53,7 @@ PACKAGECONFIG[x11] = 
"-DENABLE_X11_TARGET=ON,-DENABLE_X11_TARGET=OFF,virtual/lib
  PACKAGECONFIG[geoclue] = 
"-DENABLE_GEOLOCATION=ON,-DENABLE_GEOLOCATION=OFF,geoclue"
  PACKAGECONFIG[enchant] = 
"-DENABLE_SPELLCHECK=ON,-DENABLE_SPELLCHECK=OFF,enchant"
  PACKAGECONFIG[gtk2] = 
"-DENABLE_PLUGIN_PROCESS_GTK2=ON,-DENABLE_PLUGIN_PROCESS_GTK2=OFF,gtk+"
+PACKAGECONFIG[libepoxy] = "-DENABLE_LIBEPOXY=ON,-DENABLE_LIBEPOXY=OFF,libepoxy"
  PACKAGECONFIG[gles2] = "-DENABLE_GLES2=ON,-DENABLE_GLES2=OFF,virtual/libgles2"
  PACKAGECONFIG[webgl] = "-DENABLE_WEBGL=ON,-DENABLE_WEBGL=OFF,virtual/libgl"
  PACKAGECONFIG[opengl] = "-DENABLE_OPENGL=ON,-DENABLE_OPENGL=OFF,virtual/libgl"
--
1.9.1

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



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

Re: [OE-core] [PATCH 1/2] webkitgtk: add opengl to REQUIRED_DISTRO_FEATURES

2018-10-08 Thread ChenQi 

ping

I think webkitgtk does require the 'opengl' distro feature.

We have in mesa.inc:
PACKAGECONFIG ??= " \
   ${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 
'opengl egl gles gbm dri', '', d)} \

   ..."

PROVIDES = " \
${@bb.utils.contains('PACKAGECONFIG', 'opengl', 'virtual/libgl', 
'', d)} \
${@bb.utils.contains('PACKAGECONFIG', 'gles', 'virtual/libgles1 
virtual/libgles2', '', d)} \

..."

And webkitgtk's source codes are like:
#if USE(LIBEPOXY)
// libepoxy headers have to be included before  in 
order to avoid

// picking up khrplatform.h inclusion that's done in ANGLE.
#include 
#endif

#include 
#include 

#if PLATFORM(IOS)
#import 
#elif PLATFORM(MAC)
#include 
#elif PLATFORM(WIN)
#include "OpenGLESShims.h"
#elif USE(LIBEPOXY)
//  already included above.
#elif USE(OPENGL_ES_2)
#include 
#else
#include "OpenGLShims.h"
#endif

And libepoxy is wrapper around libgl.

So I think we can conclude 'opengl' is required distro feature for 
webkitgkt? At least this is the current situation in OE.


Best Regards,
Chen Qi

On 09/03/2018 05:35 PM, Hongxu Jia wrote:

They can't be built without opengl in DISTRO_FEATURES.
[snip]
|webkitgtk-2.20.3/Source/WebCore/platform/graphics/OpenGLShims.h:23:10:
fatal error: GL/gl.h: No such file or directory
[snip]

Signed-off-by: Hongxu Jia 
---
  meta/recipes-sato/webkit/webkitgtk_2.20.3.bb | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb 
b/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
index a528c5d..93d74bb 100644
--- a/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
+++ b/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
@@ -29,8 +29,7 @@ SRC_URI[sha256sum] = 
"579b307d78fa16a73f112790178022166d7ad17d4147492ced9b9fba48
  
  inherit cmake pkgconfig gobject-introspection perlnative distro_features_check upstream-version-is-even gtk-doc
  
-# depends on libxt

-REQUIRED_DISTRO_FEATURES = "x11"
+REQUIRED_DISTRO_FEATURES = "x11 opengl"
  
  DEPENDS = "zlib libsoup-2.4 curl libxml2 cairo libxslt libxt libidn libgcrypt \

 gtk+3 gstreamer1.0 gstreamer1.0-plugins-base flex-native 
gperf-native sqlite3 \



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

Re: [OE-core] [PATCH 1/2] webkitgtk: add opengl to REQUIRED_DISTRO_FEATURES

2018-10-08 Thread ChenQi 

ping

I think webkitgtk does require the 'opengl' distro feature.

Best Regards,
Chen Qi

On 09/03/2018 05:35 PM, Hongxu Jia wrote:

They can't be built without opengl in DISTRO_FEATURES.
[snip]
|webkitgtk-2.20.3/Source/WebCore/platform/graphics/OpenGLShims.h:23:10:
fatal error: GL/gl.h: No such file or directory
[snip]

Signed-off-by: Hongxu Jia 
---
  meta/recipes-sato/webkit/webkitgtk_2.20.3.bb | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb 
b/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
index a528c5d..93d74bb 100644
--- a/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
+++ b/meta/recipes-sato/webkit/webkitgtk_2.20.3.bb
@@ -29,8 +29,7 @@ SRC_URI[sha256sum] = 
"579b307d78fa16a73f112790178022166d7ad17d4147492ced9b9fba48
  
  inherit cmake pkgconfig gobject-introspection perlnative distro_features_check upstream-version-is-even gtk-doc
  
-# depends on libxt

-REQUIRED_DISTRO_FEATURES = "x11"
+REQUIRED_DISTRO_FEATURES = "x11 opengl"
  
  DEPENDS = "zlib libsoup-2.4 curl libxml2 cairo libxslt libxt libidn libgcrypt \

 gtk+3 gstreamer1.0 gstreamer1.0-plugins-base flex-native 
gperf-native sqlite3 \



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

[OE-core] [PATCH v2] connman: update to 1.36

2018-10-08 Thread Oleksandr Kravchuk 
From: Olekandr Kravchuk 

- updated connman to v1.36
- removed mainstreamed patches
- includes.patch has been rabased and transformed into git format

Signed-off-by: Oleksandr Kravchuk 
---
 meta/recipes-connectivity/connman/connman.inc |   2 +-
 ...ues-which-cause-problems-under-musl.patch} | 437 --
 ...tls-Fix-a-crash-using-wispr-over-TLS.patch |  41 --
 ...eep-track-of-addr-in-fw_snat-session.patch | 112 -
 ...ubnet-route-creation-deletion-in-ipr.patch |  69 ---
 ...PIs-for-creating-and-deleting-subnet.patch |  68 ---
 ...net-route-creation-and-deletion-APIs.patch |  77 ---
 .../connman/connman_1.35.bb   |  22 -
 .../connman/connman_1.36.bb   |  16 +
 9 files changed, 208 insertions(+), 636 deletions(-)
 rename meta/recipes-connectivity/connman/connman/{includes.patch => 
0001-Fix-various-issues-which-cause-problems-under-musl.patch} (68%)
 delete mode 100644 
meta/recipes-connectivity/connman/connman/0001-giognutls-Fix-a-crash-using-wispr-over-TLS.patch
 delete mode 100644 
meta/recipes-connectivity/connman/connman/0001-session-Keep-track-of-addr-in-fw_snat-session.patch
 delete mode 100644 
meta/recipes-connectivity/connman/connman/0002-inet-Implement-subnet-route-creation-deletion-in-ipr.patch
 delete mode 100644 
meta/recipes-connectivity/connman/connman/0003-inet-Implement-APIs-for-creating-and-deleting-subnet.patch
 delete mode 100644 
meta/recipes-connectivity/connman/connman/0004-session-Use-subnet-route-creation-and-deletion-APIs.patch
 delete mode 100644 meta/recipes-connectivity/connman/connman_1.35.bb
 create mode 100644 meta/recipes-connectivity/connman/connman_1.36.bb

diff --git a/meta/recipes-connectivity/connman/connman.inc 
b/meta/recipes-connectivity/connman/connman.inc
index 2b03f9cb06..0ba375137d 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -156,7 +156,7 @@ RDEPENDS_${PN}-client ="${PN}"
 
 FILES_${PN} = "${bindir}/* ${sbindir}/* ${libexecdir}/* ${libdir}/lib*.so.* \
 ${libdir}/connman/plugins \
-${sysconfdir} ${sharedstatedir} ${localstatedir} \
+${sysconfdir} ${sharedstatedir} ${localstatedir} ${datadir} \
 ${base_bindir}/* ${base_sbindir}/* ${base_libdir}/*.so* 
${datadir}/${PN} \
 ${datadir}/dbus-1/system-services/* \
 ${sysconfdir}/tmpfiles.d/connman_resolvconf.conf"
diff --git a/meta/recipes-connectivity/connman/connman/includes.patch 
b/meta/recipes-connectivity/connman/connman/0001-Fix-various-issues-which-cause-problems-under-musl.patch
similarity index 68%
rename from meta/recipes-connectivity/connman/connman/includes.patch
rename to 
meta/recipes-connectivity/connman/connman/0001-Fix-various-issues-which-cause-problems-under-musl.patch
index 9f7395cbbb..f344fea109 100644
--- a/meta/recipes-connectivity/connman/connman/includes.patch
+++ 
b/meta/recipes-connectivity/connman/connman/0001-Fix-various-issues-which-cause-problems-under-musl.patch
@@ -1,36 +1,43 @@
-Fix various issues which cause problems under musl.
-
-Upstream-Status: Backport [bd1326ba7d68df38c5ccaafd2403a5fb30bd452b]
-Signed-off-by: Ross Burton 
-
-From 630516bcc0233b047f65665c003201ba6e77453d Mon Sep 17 00:00:00 2001
+From 181ff3439783c6920f5211730672685a210c318f Mon Sep 17 00:00:00 2001
 From: Ross Burton 
-Date: Tue, 9 Aug 2016 16:22:36 +0100
-Subject: [PATCH 1/3] Use AC_USE_SYSTEM_EXTENSIONS
+Date: Mon, 8 Oct 2018 22:12:56 +0200
+Subject: [PATCH] Fix various issues which cause problems under musl
+
+Instead of using #define _GNU_SOURCE in some source files which causes
+problems when building with musl as more files need the define, simply
+use AC_USE_SYSTEM_EXTENSIONS in configure.ac to get it defined globally.
 
-Instead of using #define _GNU_SOURCE in some source files which causes problems
-when building with musl as more files need the define, simply use
-AC_USE_SYSTEM_EXTENSIONS in configure.ac to get it defined globally.
+Signed-off-by: Ross Burton 
+Upstream-Status: Backport [bd1326ba7d68df38c5ccaafd2403a5fb30bd452b]
 ---
- configure.ac   | 1 +
- gdhcp/client.c | 1 -
- plugins/tist.c | 1 -
- src/backtrace.c| 1 -
- src/inet.c | 1 -
- src/log.c  | 1 -
- src/ntp.c  | 1 -
- src/resolver.c | 1 -
- src/rfkill.c   | 1 -
- src/stats.c| 1 -
- src/timezone.c | 1 -
- tools/stats-tool.c | 1 -
- tools/tap-test.c   | 1 -
- tools/wispr.c  | 1 -
- vpn/plugins/vpn.c  | 1 -
- 15 files changed, 1 insertion(+), 14 deletions(-)
+ configure.ac | 3 +++
+ gdhcp/client.c   | 1 -
+ gdhcp/common.h   | 5 +++--
+ gweb/gresolv.c   | 1 +
+ plugins/tist.c   | 1 -
+ plugins/wifi.c   | 3 +--
+ src/backtrace.c  | 1 -
+ src/inet.c   | 1 -
+ src/ippool.c | 1 -
+ src/iptables.c   | 2 +-
+ src/log.c| 1 -
+ src/ntp.c

[OE-core] [OE-CORE][PATCH] wic: Support for changing the imager.

2018-10-08 Thread Davis, Michael 
Signed-off-by: Michael Davis 
---
 scripts/lib/wic/engine.py | 2 +-
 scripts/wic   | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/scripts/lib/wic/engine.py b/scripts/lib/wic/engine.py
index 850cec30e55..e6c830ce789 100644
--- a/scripts/lib/wic/engine.py
+++ b/scripts/lib/wic/engine.py
@@ -191,7 +191,7 @@ def wic_create(wks_file, rootfs_dir, bootimg_dir, 
kernel_dir,
 if not os.path.exists(options.outdir):
 os.makedirs(options.outdir)
 
-pname = 'direct'
+pname = options.imager
 plugin_class = PluginMgr.get_plugins('imager').get(pname)
 if not plugin_class:
 raise WicError('Unknown plugin: %s' % pname)
diff --git a/scripts/wic b/scripts/wic
index 7392bc4e7f4..37dfe2dc589 100755
--- a/scripts/wic
+++ b/scripts/wic
@@ -362,6 +362,8 @@ def wic_init_parser_create(subparser):
"bitbake variables")
 subparser.add_argument("-D", "--debug", dest="debug", action="store_true",
   default=False, help="output debug information")
+subparser.add_argument("-i", "--imager", dest="imager",
+  default="direct", help="the wic imager plugin")
 return
 
 
-- 
2.17.1
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] YP 2.7 Planning

2018-10-08 Thread Jolley, Stephen K 
All,

We have begun planning for YP 2.7.   In prior versions, items that we didn't 
have the bandwidth to do were assigned the target milestone of 2.99.  Since YP 
2.7 planning has just begun, I would like each of you to review the bugs and 
enhancements in YP 2.99 and determine if you will have the bandwidth to own any 
of them in YP 2.7.

In addition to reviewing the bugs under your name, also review any under the 
unassigned name to see if you have both an interest and the time to own any of 
those in YP 2.7.

Link for bugs in YP 2.99 by owner: 
https://wiki.yoctoproject.org/wiki/Bug_Triage#2.99_Bugs_by_Owner
Link for enhancements in YP 2.99 by owner: 
https://wiki.yoctoproject.org/wiki/Bug_Triage#2.99_Enhancements_by_Owner

Thanks,

Stephen K. Jolley
Yocto Project Program Manager
INTEL, MS JF1-255, 2111 N.E. 25th Avenue, Hillsboro, OR 97124
*Cell:(208) 244-4460
* Email: stephen.k.jol...@intel.com

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

[OE-core] Yocto Project Unassigned Bugs - Help Needed

2018-10-08 Thread Jolley, Stephen K 
All,



The triage team meets weekly and does its best to handle the bugs reported into 
the bugzilla. The number of people attending that meeting has fallen, as have 
the number of people available to help fix bugs. One of the things we hear 
users report is they don't know how to help. We (the triage team) are therefore 
going to start reporting out the currently 300 unassigned bugs.



We're hoping people may be able to spare some time now and again to help out 
with these.



Bugs are split into two types, "true bugs" where things don't work as they 
should and "enhancements" which are features we'd want to add to the system.



There are also roughly five different "priority" classes right now, "2.6", 
"2.7", "2.8", "2.99" and "Future", the more pressing/urgent issues being in 
"2.6" and then "2.7".



Please review this link and if a bug is something you would be able to help 
with either take ownership of the bug, or send me 
(stephen.k.jol...@intel.com) an e-mail with 
the bug number you would like and I will assign it to you (please make sure you 
have a bugzilla account).



The list is at: https://wiki.yoctoproject.org/wiki/Bug_Triage#Unassigned_Bugs


Thanks,

Stephen K. Jolley
Yocto Project Program Manager
INTEL, MS JF1-255, 2111 N.E. 25th Avenue, Hillsboro, OR 97124
*Cell:(208) 244-4460
* Email: stephen.k.jol...@intel.com

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

Re: [OE-core] [PATCH] busybox: fix conflict with runlevel applet

2018-10-08 Thread Khem Raj 
On Mon, Oct 8, 2018 at 8:39 AM Burton, Ross  wrote:
>
> On Thu, 4 Oct 2018 at 12:28, Martin Jansa  wrote:
>>
>> On Thu, Oct 04, 2018 at 12:59:37PM +0200, Andrej Valek wrote:
>> > This init feature was introduced via
>> > http://lists.openembedded.org/pipermail/openembedded-core/2016-January/116109.html
>> > which is from 2016. Busybox has this runlevel applet available from 2005.
>> >
>> > I think, Raj was trying to make this system really tiny. So, I am not
>> > sure, what would be the impact, if we dropped this runlevel script.
>>
>> OK, fair enough. I don't use busybox init anyway.
>>
>> I was just curious, because there is also runlevel from upstart which we were
>> using (and I have u-a for runlevel in busybox.bbappend because of that).
>> So I was surprised that there is actually 3rd implementation as OE
>> specific shell script.
>
>
> Khem, do you have an opinion?  I suspect that shell script including comment 
> is longer than the compiled size of the runlevel applet in busybox...  We 
> could simplify by just having the applet and removing that runlevel stub.

we should enable runlevel unconditionally when selecting busybox init
and delete the file that I added for runlevel, that way we will be
inline with busybox for this applet.

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

Re: [OE-core] [PATCH] Support for changing the imager used in wic.

2018-10-08 Thread Burton, Ross 
That's one messed up patch: can you resend using git-send-email please?

Ross

On Mon, 8 Oct 2018 at 14:59, Davis, Michael 
wrote:

> >*From 136d6214b6fc2c1118dac5b7f3e029ed006fab8a Mon Sep 17 00:00:00 2001*
>
> From: Paul Knopf  >
>
> Date: Mon, 21 May 2018 16:56:51 -0400
>
> Subject: [PATCH] Support for changing the imager used in wic.
>
>
>
> ---
>
>  scripts/lib/wic/engine.py | 2 +-
>
>  scripts/wic   | 1 +
>
>  2 files changed, 2 insertions(+), 1 deletion(-)
>
>
>
> diff --git a/scripts/lib/wic/engine.py b/scripts/lib/wic/engine.py
>
> index edcfab3..d7c5c11 100644
>
> --- a/scripts/lib/wic/engine.py
>
> +++ b/scripts/lib/wic/engine.py
>
> @@ -191,7 +191,7 @@ def wic_create(wks_file, rootfs_dir, bootimg_dir,
>
> kernel_dir,
>
>  if not os.path.exists(options.outdir):
>
>  os.makedirs(options.outdir)
>
>
>
> -pname = 'direct'
>
> +pname = options.imager
>
>  plugin_class = PluginMgr.get_plugins('imager').get(pname)
>
>  if not plugin_class:
>
>  raise WicError('Unknown plugin: %s' % pname)
>
> diff --git a/scripts/wic b/scripts/wic
>
> index 7392bc4..255e524 100755
>
> --- a/scripts/wic
>
> +++ b/scripts/wic
>
> @@ -360,6 +360,7 @@ def wic_init_parser_create(subparser):
>
>  subparser.add_argument("-v", "--vars", dest='vars_dir',
>
>help="directory with .env files that store "
>
> "bitbake variables")
>
> +subparser.add_argument("-i", "--imager", dest="imager",
>
> default="direct", help="the wic imager plugin")
>
>  subparser.add_argument("-D", "--debug", dest="debug",
>
> action="store_true",
>
>default=False, help="output debug information")
>
>  return
>
> --
>
> 2.7.4
>
>
>
>
>
> I found this patch that got sent to the yocto mailing list, but never made
> it to the correct mailing list.
>
> I would really like to have the functionality if it is possible to get it
> pulled in.
>
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] busybox: fix conflict with runlevel applet

2018-10-08 Thread Burton, Ross 
On Thu, 4 Oct 2018 at 12:28, Martin Jansa  wrote:

> On Thu, Oct 04, 2018 at 12:59:37PM +0200, Andrej Valek wrote:
> > This init feature was introduced via
> >
> http://lists.openembedded.org/pipermail/openembedded-core/2016-January/116109.html
> > which is from 2016. Busybox has this runlevel applet available from 2005.
> >
> > I think, Raj was trying to make this system really tiny. So, I am not
> > sure, what would be the impact, if we dropped this runlevel script.
>
> OK, fair enough. I don't use busybox init anyway.
>
> I was just curious, because there is also runlevel from upstart which we
> were
> using (and I have u-a for runlevel in busybox.bbappend because of that).
> So I was surprised that there is actually 3rd implementation as OE
> specific shell script.


Khem, do you have an opinion?  I suspect that shell script including
comment is longer than the compiled size of the runlevel applet in
busybox...  We could simplify by just having the applet and removing that
runlevel stub.

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

[OE-core] [PATCH] lib11: fix patch headers

2018-10-08 Thread Ross Burton 
devtool doesn't like patches without subjects, so add meaningful subjects.

Signed-off-by: Ross Burton 
---
 meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch  | 2 +-
 meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch   | 2 +-
 meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch 
b/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
index c4c9396675a..c1172147a6e 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
@@ -1,7 +1,7 @@
 From 62005ad019d6afb81da100b3113292a16472d241 Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:08:27 +0800
-Subject: [PATCH] 
+Subject: [PATCH] Stub out x18n
 
 Update patch to version 1.6.6
 Signed-off-by: Changqing Li 
diff --git a/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch 
b/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
index 559250f9576..09cf74fff4c 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
@@ -1,7 +1,7 @@
 From 5c1693028ef12ab767d7cdbc349c27efa1ada87a Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:13:38 +0800
-Subject: [PATCH]
+Subject: [PATCH] Fix build when xlocale disabled
 
 Signed-off-by: Xiaofeng Yan 
 
diff --git a/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch 
b/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
index c077639f895..0b57e0b068c 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
@@ -1,7 +1,7 @@
 From 046703373055c7029c19d171e415ff3caaa87372 Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:16:33 +0800
-Subject: [PATCH]
+Subject: [PATCH] Disable unicode
 
 Signed-off-by: Xiaofeng Yan 
 
-- 
2.11.0

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

[OE-core] [PATCH] lib11: fix patch headers

2018-10-08 Thread Ross Burton 
Signed-off-by: Ross Burton 
---
 meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch  | 3 ++-
 meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch   | 3 ++-
 meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch 
b/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
index 4ca2f42d4c7..c1172147a6e 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
@@ -1,10 +1,11 @@
 From 62005ad019d6afb81da100b3113292a16472d241 Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:08:27 +0800
-Subject: [PATCH] Upstream-Status: Inappropriate [configuration]
+Subject: [PATCH] Stub out x18n
 
 Update patch to version 1.6.6
 Signed-off-by: Changqing Li 
+Upstream-Status: Inappropriate [configuration]
 ---
  src/Makefile.am|   5 +-
  src/X18NCMSstubs.c | 426 +
diff --git a/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch 
b/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
index 3677fb1c5ca..09cf74fff4c 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
@@ -1,12 +1,13 @@
 From 5c1693028ef12ab767d7cdbc349c27efa1ada87a Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:13:38 +0800
-Subject: [PATCH] Upstream-Status: Pending
+Subject: [PATCH] Fix build when xlocale disabled
 
 Signed-off-by: Xiaofeng Yan 
 
 Update patch to version 1.6.6
 Signed-off-by: Changqing Li 
+Upstream-Status: Pending
 ---
  src/Font.c | 4 
  1 file changed, 4 insertions(+)
diff --git a/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch 
b/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
index 1a3069a9ad6..0b57e0b068c 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
@@ -1,13 +1,14 @@
 From 046703373055c7029c19d171e415ff3caaa87372 Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:16:33 +0800
-Subject: [PATCH] Upstream-Status: Pending
+Subject: [PATCH] Disable unicode
 
 Signed-off-by: Xiaofeng Yan 
 
 Update patch to version 1.6.6
 
 Signed-off-by: Changqing Li 
+Upstream-Status: Pending
 ---
  include/X11/Xlib.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
-- 
2.11.0

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

Re: [OE-core] [yocto] [yocto-docs][PATCH] kernel-dev: document the change detection of kernel feature files

2018-10-08 Thread Bruce Ashfield 
On Mon, Oct 8, 2018 at 5:18 AM Urs Fässler  wrote:
>
> On Tue, 2018-10-02 at 10:44 -0400, bruce.ashfielda wrote:
> > On 2018-10-01 8:42 AM, Urs F?ssler wrote:
> > > Recommend to add recipe-space features to SRC_URI so that changes
> > > to them
> > > are detected automatically.
> > >
> > > Signed-off-by: Urs F?ssler 
> > > Signed-off-by: Pascal Bach 
> > > ---
> > >   documentation/kernel-dev/kernel-dev-common.xml | 7 +--
> > >   1 file changed, 5 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/documentation/kernel-dev/kernel-dev-common.xml
> > > b/documentation/kernel-dev/kernel-dev-common.xml
> > > index 83b02b1c1..6289ce8d4 100644
> > > --- a/documentation/kernel-dev/kernel-dev-common.xml
> > > +++ b/documentation/kernel-dev/kernel-dev-common.xml
> > > @@ -2623,6 +2623,9 @@
> > >   .scc file in the
> > >   SRC_URI statement to reference
> > > multiple kernel
> > >   features.
> > > +Since BitBake only detects changes on files listed in
> > > +SRC_URI, it is best to add all
> >
> > This is not necessarily true. It would be better stated that if you
> > are modifying the .cfg files in the layer directly, you should list
> > them on the SRC_URI to have the changes picked up by bitbake.
> >
> > It is not a matter of "best", so it should be stated that way.
>
> Thank you for the feedback. I am not sure if I don't understand the
> problem or your answer correctly.

I meant to say "not stated that way".

>
> We faced the problem, that our CI server did not picked up the changes
> made in the .cfg files. I investigated the problem and found that
> BitBake only detects changes on files listed in SRC_URI. This is
> supported in the chapter "Working with Advanced Metadata" that states:

I'm aware of how the items are currently written, having provided much
of the information for those sections.

>
> "BitBake parses [the .scc files] and fetches any files referenced in
> the .scc files by the include, patch, or kconf commands. Because of
> this, it is necessary to bump the recipe PR value when changing the
> content of files not explicitly listed in the SRC_URI."
> https://www.yoctoproject.org/docs/latest/mega-manual/mega-manual.html#recipe-space-metadata
>
> The behavior is imho not what you expect. But fixing this is quite a
> big change since it involves changes in the interaction of BitBake
> (change detection on files) with Yocto classes (recursive parsing of
> scc files). Therefore I try to solve it with a better description.

Yep. I've had a bug/feature to do just that for some time, but it hasn't
made it to the top of the list yet.

My point is that the description should not say "best", it isn't a matter
of best or worst. It is a matter of stating the behaviour, knowing how
you plan to interact with the elements and then using them appropriately.

>
> You stated that it is only necessary to add the .cfg files when you
> change the .cfg files in the layer directly. I don't see how you would
> change the .cfg files otherwise or what use case you have in mind.

By using a kernel-cache repository. Just like all the main linux-yocto
builds. Alternatively, you could make sure the gather meta-data task
is always run, but then that typically triggers more compiles/builds
than you want.

>
> > > @+.cfg to
> > > SRC_URI.
> > >   
> > >
> > >   
> > > @@ -2674,10 +2677,10 @@
> > >   
> > >   
> > >   Add the Feature File to
> > > SRC_URI:
> > > -Add the .scc file to the
> > > +Add the .cfg and
> > > .scc file to the
> > >   recipe's SRC_URI
> > > statement:
> > >   
> > > - SRC_URI_append = " file://test.scc"
> > > + SRC_URI_append = " file://test.cfg file://test.scc"
> >
> > This is wrong. You should not have both on the SRC_URI
>
> I concluded this also from the description in the "Working with
> Advanced Metadata" chapter (see above). But I would be happy to know
> about another solution for our problem.

If all that test.scc does is include test.cfg, you don't put both on
the SRC_URI.
All you've done is include it twice. If you are worried about modifying the
.cfg's and having sstate be invalided so the tasks re-run, then you'd just
put the .cfg on the SRC_URI.

Bruce

>
> Urs
>
> > >   
> > >   The leading space before the path is
> > > important as the
> > >   path is appended to the existing path.
>
>
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end"
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org

[OE-core] [PATCH 4/4] lib11: fix patch headers

2018-10-08 Thread Ross Burton 
Signed-off-by: Ross Burton 
---
 meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch  | 3 ++-
 meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch   | 3 ++-
 meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch 
b/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
index 4ca2f42d4c7..c4c9396675a 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/X18NCMSstubs.patch
@@ -1,10 +1,11 @@
 From 62005ad019d6afb81da100b3113292a16472d241 Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:08:27 +0800
-Subject: [PATCH] Upstream-Status: Inappropriate [configuration]
+Subject: [PATCH] 
 
 Update patch to version 1.6.6
 Signed-off-by: Changqing Li 
+Upstream-Status: Inappropriate [configuration]
 ---
  src/Makefile.am|   5 +-
  src/X18NCMSstubs.c | 426 +
diff --git a/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch 
b/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
index 3677fb1c5ca..559250f9576 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/fix-disable-xlocale.patch
@@ -1,12 +1,13 @@
 From 5c1693028ef12ab767d7cdbc349c27efa1ada87a Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:13:38 +0800
-Subject: [PATCH] Upstream-Status: Pending
+Subject: [PATCH]
 
 Signed-off-by: Xiaofeng Yan 
 
 Update patch to version 1.6.6
 Signed-off-by: Changqing Li 
+Upstream-Status: Pending
 ---
  src/Font.c | 4 
  1 file changed, 4 insertions(+)
diff --git a/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch 
b/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
index 1a3069a9ad6..c077639f895 100644
--- a/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
+++ b/meta/recipes-graphics/xorg-lib/libx11/fix-utf8-wrong-define.patch
@@ -1,13 +1,14 @@
 From 046703373055c7029c19d171e415ff3caaa87372 Mon Sep 17 00:00:00 2001
 From: Changqing Li 
 Date: Mon, 10 Sep 2018 09:16:33 +0800
-Subject: [PATCH] Upstream-Status: Pending
+Subject: [PATCH]
 
 Signed-off-by: Xiaofeng Yan 
 
 Update patch to version 1.6.6
 
 Signed-off-by: Changqing Li 
+Upstream-Status: Pending
 ---
  include/X11/Xlib.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
-- 
2.11.0

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

[OE-core] [PATCH 3/4] python3: fix patch headers

2018-10-08 Thread Ross Burton 
Signed-off-by: Ross Burton 
---
 meta/recipes-devtools/python/python3/ftplib.patch | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-devtools/python/python3/ftplib.patch 
b/meta/recipes-devtools/python/python3/ftplib.patch
index 8bb3cd02196..49c5b2736b9 100644
--- a/meta/recipes-devtools/python/python3/ftplib.patch
+++ b/meta/recipes-devtools/python/python3/ftplib.patch
@@ -1,3 +1,6 @@
+Upstream-Status: Backport
+Signed-off-by: Ross Burton 
+
 From cabe916dc694997d4892b58986e73a713d5a2f8d Mon Sep 17 00:00:00 2001
 From: "Miss Islington (bot)"
  <31488909+miss-isling...@users.noreply.github.com>
-- 
2.11.0

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

[OE-core] [PATCH 2/4] python: fix patch headers

2018-10-08 Thread Ross Burton 
Signed-off-by: Ross Burton 
---
 .../0001-bpo-33354-Fix-test_ssl-when-a-filename-cannot-be-enc.patch   | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git 
a/meta/recipes-devtools/python/python/0001-bpo-33354-Fix-test_ssl-when-a-filename-cannot-be-enc.patch
 
b/meta/recipes-devtools/python/python/0001-bpo-33354-Fix-test_ssl-when-a-filename-cannot-be-enc.patch
index 776bbdcf7a4..4c0b3577b24 100644
--- 
a/meta/recipes-devtools/python/python/0001-bpo-33354-Fix-test_ssl-when-a-filename-cannot-be-enc.patch
+++ 
b/meta/recipes-devtools/python/python/0001-bpo-33354-Fix-test_ssl-when-a-filename-cannot-be-enc.patch
@@ -7,9 +7,7 @@ Subject: [PATCH] bpo-33354: Fix test_ssl when a filename cannot 
be encoded
 Skip test_load_dh_params() of test_ssl when Python filesystem encoding
 cannot encode the provided path.
 
-Upstream-Status:
-Backport[https://github.com/python/cpython/commit/19f6bd06af3c7fc0db5f96878aaa68f5589ff13e]
-
+Upstream-Status: Backport 
[https://github.com/python/cpython/commit/19f6bd06af3c7fc0db5f96878aaa68f5589ff13e]
 Signed-off-by: Anuj Mittal 
 ---
  Lib/test/test_ssl.py | 9 -
-- 
2.11.0

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

[OE-core] [PATCH 1/4] gdb: fix patch headers

2018-10-08 Thread Ross Burton 
Signed-off-by: Ross Burton 
---
 .../gdb/gdb/gdbserver-ctrl-c-handling.patch| 22 +++---
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/meta/recipes-devtools/gdb/gdb/gdbserver-ctrl-c-handling.patch 
b/meta/recipes-devtools/gdb/gdb/gdbserver-ctrl-c-handling.patch
index dffb0dd81fc..71fe7f83b65 100644
--- a/meta/recipes-devtools/gdb/gdb/gdbserver-ctrl-c-handling.patch
+++ b/meta/recipes-devtools/gdb/gdb/gdbserver-ctrl-c-handling.patch
@@ -1,14 +1,14 @@
-+This problem was created by the upstream commit 78708b7c8c
-+After applying the commit, it will send SIGINT to the process 
group(-signal_pid).
-+But if we use gdbserver send SIGINT, and the attached process is not a process
-+group leader, then the "kill (-signal_pid, SIGINT)" returns error and fails  
to
-+interrupt the attached process.
-+
-+Upstream-Status: Submitted
-+[https://sourceware.org/bugzilla/show_bug.cgi?id=18945]
-+
-+Author: Josh Gao
-+Signed-off-by: Zhixiong Chi 
+This problem was created by the upstream commit 78708b7c8c
+After applying the commit, it will send SIGINT to the process 
group(-signal_pid).
+But if we use gdbserver send SIGINT, and the attached process is not a process
+group leader, then the "kill (-signal_pid, SIGINT)" returns error and fails  to
+interrupt the attached process.
+
+Upstream-Status: Submitted 
[https://sourceware.org/bugzilla/show_bug.cgi?id=18945]
+
+Author: Josh Gao
+Signed-off-by: Zhixiong Chi 
+
 Index: gdb-8.2/gdb/gdbserver/linux-low.c
 ===
 --- gdb-8.2.orig/gdb/gdbserver/linux-low.c
-- 
2.11.0

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

[OE-core] [PATCH] Support for changing the imager used in wic.

2018-10-08 Thread Davis, Michael 
>From 136d6214b6fc2c1118dac5b7f3e029ed006fab8a Mon Sep 17 00:00:00 2001

From: Paul Knopf https://lists.yoctoproject.org/listinfo/yocto>>

Date: Mon, 21 May 2018 16:56:51 -0400

Subject: [PATCH] Support for changing the imager used in wic.



---

 scripts/lib/wic/engine.py | 2 +-

 scripts/wic   | 1 +

 2 files changed, 2 insertions(+), 1 deletion(-)



diff --git a/scripts/lib/wic/engine.py b/scripts/lib/wic/engine.py

index edcfab3..d7c5c11 100644

--- a/scripts/lib/wic/engine.py

+++ b/scripts/lib/wic/engine.py

@@ -191,7 +191,7 @@ def wic_create(wks_file, rootfs_dir, bootimg_dir,

kernel_dir,

 if not os.path.exists(options.outdir):

 os.makedirs(options.outdir)



-pname = 'direct'

+pname = options.imager

 plugin_class = PluginMgr.get_plugins('imager').get(pname)

 if not plugin_class:

 raise WicError('Unknown plugin: %s' % pname)

diff --git a/scripts/wic b/scripts/wic

index 7392bc4..255e524 100755

--- a/scripts/wic

+++ b/scripts/wic

@@ -360,6 +360,7 @@ def wic_init_parser_create(subparser):

 subparser.add_argument("-v", "--vars", dest='vars_dir',

   help="directory with .env files that store "

"bitbake variables")

+subparser.add_argument("-i", "--imager", dest="imager",

default="direct", help="the wic imager plugin")

 subparser.add_argument("-D", "--debug", dest="debug",

action="store_true",

   default=False, help="output debug information")

 return

--

2.7.4


I found this patch that got sent to the yocto mailing list, but never made it 
to the correct mailing list.
I would really like to have the functionality if it is possible to get it 
pulled in.

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

Re: [OE-core] Python3 and HTTPS issue

2018-10-08 Thread Burton, Ross 
Had a change of heart and just sent a patch to add ca-certificates to the
recommends for python3-crypt.

Ross

On Mon, 8 Oct 2018 at 12:52, Burton, Ross  wrote:

> Python3 would be overkill, but the SSL libraries probably should yes.
>
> Ross
>
> On Sat, 6 Oct 2018 at 02:54, Marc Ferland  wrote:
>
>> On Fri, Oct 5, 2018 at 5:22 PM Burton, Ross 
>> wrote:
>> >
>> > Did you install ca-certificates?
>> >
>>
>> Looks like it did the trick, thanks.
>>
>> Just wondering: why isn't ca-certificates not RRECOMMENDed by openssl
>> or maybe python3?
>>
>> Marc
>>
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] python3: add ca-certificates to python3-crypt RRECOMMENDS

2018-10-08 Thread Ross Burton 
It's likely that the certificate root chain is needed if python3-crypt is
installed, so recommend it.

Also remove the redundant nativesdk-python3-crypt assignment, as the general
form is sufficient when class-extended.

Signed-off-by: Ross Burton 
---
 meta/recipes-devtools/python/python3_3.5.6.bb | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/recipes-devtools/python/python3_3.5.6.bb 
b/meta/recipes-devtools/python/python3_3.5.6.bb
index 6085e36b7db..e1796581342 100644
--- a/meta/recipes-devtools/python/python3_3.5.6.bb
+++ b/meta/recipes-devtools/python/python3_3.5.6.bb
@@ -217,8 +217,7 @@ py_package_preprocess () {
 # manual dependency additions
 RPROVIDES_${PN}-modules = "${PN}"
 RRECOMMENDS_${PN}-core_append_class-nativesdk = " nativesdk-python3-modules"
-RRECOMMENDS_${PN}-crypt = "openssl"
-RRECOMMENDS_${PN}-crypt_class-nativesdk = "nativesdk-openssl"
+RRECOMMENDS_${PN}-crypt = "openssl ca-certificates"
 
 FILES_${PN}-2to3 += "${bindir}/2to3-${PYTHON_MAJMIN}"
 FILES_${PN}-pydoc += "${bindir}/pydoc${PYTHON_MAJMIN} ${bindir}/pydoc3"
-- 
2.11.0

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

Re: [OE-core] Python3 and HTTPS issue

2018-10-08 Thread Burton, Ross 
Python3 would be overkill, but the SSL libraries probably should yes.

Ross

On Sat, 6 Oct 2018 at 02:54, Marc Ferland  wrote:

> On Fri, Oct 5, 2018 at 5:22 PM Burton, Ross  wrote:
> >
> > Did you install ca-certificates?
> >
>
> Looks like it did the trick, thanks.
>
> Just wondering: why isn't ca-certificates not RRECOMMENDed by openssl
> or maybe python3?
>
> Marc
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] connman: update patch status

2018-10-08 Thread Ross Burton 
These patches are all upstream now, so mark as Backport.

Signed-off-by: Ross Burton 
---
 meta/recipes-connectivity/connman/connman/includes.patch | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/connman/connman/includes.patch 
b/meta/recipes-connectivity/connman/connman/includes.patch
index dcce5bea122..9f7395c 100644
--- a/meta/recipes-connectivity/connman/connman/includes.patch
+++ b/meta/recipes-connectivity/connman/connman/includes.patch
@@ -1,6 +1,6 @@
 Fix various issues which cause problems under musl.
 
-Upstream-Status: Submitted
+Upstream-Status: Backport [bd1326ba7d68df38c5ccaafd2403a5fb30bd452b]
 Signed-off-by: Ross Burton 
 
 From 630516bcc0233b047f65665c003201ba6e77453d Mon Sep 17 00:00:00 2001
-- 
2.11.0

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

[OE-core] [PATCH] libsoup: upgrade to 2.62.3

2018-10-08 Thread Ross Burton 
Freeze-breaking upgrade for a security bug fix involving cookie URLs, and a
number of static analysis fixes.

Drop CVE-2018-12910.patch as this is merged in 2.62.3.

Signed-off-by: Ross Burton 
---
 .../libsoup/libsoup-2.4/CVE-2018-12910.patch   | 32 --
 ...libsoup-2.4_2.62.2.bb => libsoup-2.4_2.62.3.bb} |  7 ++---
 2 files changed, 3 insertions(+), 36 deletions(-)
 delete mode 100644 
meta/recipes-support/libsoup/libsoup-2.4/CVE-2018-12910.patch
 rename meta/recipes-support/libsoup/{libsoup-2.4_2.62.2.bb => 
libsoup-2.4_2.62.3.bb} (87%)

diff --git a/meta/recipes-support/libsoup/libsoup-2.4/CVE-2018-12910.patch 
b/meta/recipes-support/libsoup/libsoup-2.4/CVE-2018-12910.patch
deleted file mode 100644
index 158bf930934..000
--- a/meta/recipes-support/libsoup/libsoup-2.4/CVE-2018-12910.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-CVE: CVE-2018-12910
-Upstream-Status: Backport
-Signed-off-by: Ross Burton 
-
-From db2b0d5809d5f8226d47312b40992cadbcde439f Mon Sep 17 00:00:00 2001
-From: Michael Catanzaro 
-Date: Sun, 24 Jun 2018 19:46:19 -0500
-Subject: [PATCH] cookie-jar: bail if hostname is an empty string
-
-There are several other ways to fix the problem with this function, but
-skipping over all of the code is probably the simplest.
-
-Fixes #3

- libsoup/soup-cookie-jar.c | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/libsoup/soup-cookie-jar.c b/libsoup/soup-cookie-jar.c
-index 2369c8a7..b2b78909 100644
 a/libsoup/soup-cookie-jar.c
-+++ b/libsoup/soup-cookie-jar.c
-@@ -307,7 +307,7 @@ get_cookies (SoupCookieJar *jar, SoupURI *uri, gboolean 
for_http, gboolean copy_
- 
-   priv = soup_cookie_jar_get_instance_private (jar);
- 
--  if (!uri->host)
-+  if (!uri->host || !uri->host[0])
-   return NULL;
- 
-   /* The logic here is a little weird, but the plan is that if
--- 
-2.17.1
diff --git a/meta/recipes-support/libsoup/libsoup-2.4_2.62.2.bb 
b/meta/recipes-support/libsoup/libsoup-2.4_2.62.3.bb
similarity index 87%
rename from meta/recipes-support/libsoup/libsoup-2.4_2.62.2.bb
rename to meta/recipes-support/libsoup/libsoup-2.4_2.62.3.bb
index ba3e86a03ca..aaa3cc51036 100644
--- a/meta/recipes-support/libsoup/libsoup-2.4_2.62.2.bb
+++ b/meta/recipes-support/libsoup/libsoup-2.4_2.62.3.bb
@@ -9,10 +9,9 @@ DEPENDS = "glib-2.0 glib-2.0-native libxml2 sqlite3 
intltool-native"
 
 SHRT_VER = "${@d.getVar('PV').split('.')[0]}.${@d.getVar('PV').split('.')[1]}"
 
-SRC_URI = "${GNOME_MIRROR}/libsoup/${SHRT_VER}/libsoup-${PV}.tar.xz \
-   file://CVE-2018-12910.patch"
-SRC_URI[md5sum] = "eaf99b04ac8968ed2b26f2509ba75584"
-SRC_URI[sha256sum] = 
"9e536fe3da60b25d2c63addb84a9d5072d00b0d8b8cbeabc629a6bcd63f879b6"
+SRC_URI = "${GNOME_MIRROR}/libsoup/${SHRT_VER}/libsoup-${PV}.tar.xz"
+SRC_URI[md5sum] = "dfbf30af5fb6190bfafc5aa6abcc9dce"
+SRC_URI[sha256sum] = 
"d312ade547495c2093ff8bda61f9b9727a98cfdae339f3263277dd39c0451172"
 
 S = "${WORKDIR}/libsoup-${PV}"
 
-- 
2.11.0

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

Re: [OE-core] [yocto] [yocto-docs][PATCH] kernel-dev: document the change detection of kernel feature files

2018-10-08 Thread Urs Fässler 
On Tue, 2018-10-02 at 10:44 -0400, bruce.ashfielda wrote:
> On 2018-10-01 8:42 AM, Urs F?ssler wrote:
> > Recommend to add recipe-space features to SRC_URI so that changes
> > to them
> > are detected automatically.
> > 
> > Signed-off-by: Urs F?ssler 
> > Signed-off-by: Pascal Bach 
> > ---
> >   documentation/kernel-dev/kernel-dev-common.xml | 7 +--
> >   1 file changed, 5 insertions(+), 2 deletions(-)
> > 
> > diff --git a/documentation/kernel-dev/kernel-dev-common.xml
> > b/documentation/kernel-dev/kernel-dev-common.xml
> > index 83b02b1c1..6289ce8d4 100644
> > --- a/documentation/kernel-dev/kernel-dev-common.xml
> > +++ b/documentation/kernel-dev/kernel-dev-common.xml
> > @@ -2623,6 +2623,9 @@
> >   .scc file in the
> >   SRC_URI statement to reference
> > multiple kernel
> >   features.
> > +Since BitBake only detects changes on files listed in
> > +SRC_URI, it is best to add all
> 
> This is not necessarily true. It would be better stated that if you
> are modifying the .cfg files in the layer directly, you should list
> them on the SRC_URI to have the changes picked up by bitbake.
> 
> It is not a matter of "best", so it should be stated that way.

Thank you for the feedback. I am not sure if I don't understand the
problem or your answer correctly.

We faced the problem, that our CI server did not picked up the changes
made in the .cfg files. I investigated the problem and found that
BitBake only detects changes on files listed in SRC_URI. This is
supported in the chapter "Working with Advanced Metadata" that states:

"BitBake parses [the .scc files] and fetches any files referenced in
the .scc files by the include, patch, or kconf commands. Because of
this, it is necessary to bump the recipe PR value when changing the
content of files not explicitly listed in the SRC_URI."
https://www.yoctoproject.org/docs/latest/mega-manual/mega-manual.html#recipe-space-metadata

The behavior is imho not what you expect. But fixing this is quite a
big change since it involves changes in the interaction of BitBake
(change detection on files) with Yocto classes (recursive parsing of
scc files). Therefore I try to solve it with a better description.

You stated that it is only necessary to add the .cfg files when you
change the .cfg files in the layer directly. I don't see how you would
change the .cfg files otherwise or what use case you have in mind.

> > @+.cfg to
> > SRC_URI.
> >   
> >   
> >   
> > @@ -2674,10 +2677,10 @@
> >   
> >   
> >   Add the Feature File to
> > SRC_URI:
> > -Add the .scc file to the
> > +Add the .cfg and
> > .scc file to the
> >   recipe's SRC_URI
> > statement:
> >   
> > - SRC_URI_append = " file://test.scc"
> > + SRC_URI_append = " file://test.cfg file://test.scc"
> 
> This is wrong. You should not have both on the SRC_URI

I concluded this also from the description in the "Working with
Advanced Metadata" chapter (see above). But I would be happy to know
about another solution for our problem.

Urs

> >   
> >   The leading space before the path is
> > important as the
> >   path is appended to the existing path.



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