Re: [OE-core] [PATCH] glibc: apply complocaledir while localedir changed

2016-08-09 Thread Hongxu Jia

On 08/10/2016 01:49 PM, Hongxu Jia wrote:

Since localedir changed, we should apply complocaledir=/usr/lib/locale

Build image with IMAGE_LINGUAS="en-us en-gb"

Boot target and display all locales:
...
root@localhost:~# locale -a
C
POSIX
en_GB
en_US
...

Signed-off-by: Hongxu Jia 
---
  meta/recipes-core/glibc/glibc_2.24.bb | 4 +---
  1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta/recipes-core/glibc/glibc_2.24.bb 
b/meta/recipes-core/glibc/glibc_2.24.bb
index 88bcbef2..219370c 100644
--- a/meta/recipes-core/glibc/glibc_2.24.bb
+++ b/meta/recipes-core/glibc/glibc_2.24.bb
@@ -128,9 +128,7 @@ do_compile () {
  
  }
  
-# Use the host locale archive when built for nativesdk so that we don't need to

-# ship a complete (100MB) locale set.
-do_compile_prepend_class-nativesdk() {
+do_compile_prepend() {
  echo "complocaledir=/usr/lib/locale" >> ${S}/configparms


Please drop this, It should be "complocaledir=${localedir}" better.

//Hongxu


  }
  



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


[OE-core] [PATCH] glibc: apply complocaledir while localedir changed

2016-08-09 Thread Hongxu Jia
Since localedir changed, we should apply complocaledir=/usr/lib/locale

Build image with IMAGE_LINGUAS="en-us en-gb"

Boot target and display all locales:
...
root@localhost:~# locale -a
C
POSIX
en_GB
en_US
...

Signed-off-by: Hongxu Jia 
---
 meta/recipes-core/glibc/glibc_2.24.bb | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta/recipes-core/glibc/glibc_2.24.bb 
b/meta/recipes-core/glibc/glibc_2.24.bb
index 88bcbef2..219370c 100644
--- a/meta/recipes-core/glibc/glibc_2.24.bb
+++ b/meta/recipes-core/glibc/glibc_2.24.bb
@@ -128,9 +128,7 @@ do_compile () {
 
 }
 
-# Use the host locale archive when built for nativesdk so that we don't need to
-# ship a complete (100MB) locale set.
-do_compile_prepend_class-nativesdk() {
+do_compile_prepend() {
 echo "complocaledir=/usr/lib/locale" >> ${S}/configparms
 }
 
-- 
2.8.1

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


Re: [OE-core] [PATCH] cross-canadian.bbclass: Add BASECANADIANEXTRAOS to specify main extraos

2016-08-09 Thread Huang, Jie (Jackie)
Ping!

I don't see any objection on this but this is not merged yet, is it ignored?

Thanks,
Jackie

> -Original Message-
> From: openembedded-core-boun...@lists.openembedded.org 
> [mailto:openembedded-core-
> boun...@lists.openembedded.org] On Behalf Of Mark Hatle
> Sent: Tuesday, December 08, 2015 5:36 AM
> To: openembedded-core@lists.openembedded.org
> Subject: [OE-core] [PATCH] cross-canadian.bbclass: Add BASECANADIANEXTRAOS to 
> specify main
> extraos
> 
> By default the system will expand the extra os entries for uclibc and musl
> even if they are not enabled in the build.  There was no way to prevent this
> behavior while still getting the expansion for things like x32 or spe.
> 
> The change adds a new setting which a distribution creator can override
> easily, setting the base set of canadianextraos components.  The other
> expansions are then based on this setting.
> 
> Signed-off-by: Mark Hatle 
> ---
>  meta/classes/cross-canadian.bbclass | 22 +-
>  1 file changed, 17 insertions(+), 5 deletions(-)
> 
> diff --git a/meta/classes/cross-canadian.bbclass 
> b/meta/classes/cross-canadian.bbclass
> index ea17f09..799844b 100644
> --- a/meta/classes/cross-canadian.bbclass
> +++ b/meta/classes/cross-canadian.bbclass
> @@ -15,7 +15,8 @@ STAGING_BINDIR_TOOLCHAIN =
> "${STAGING_DIR_NATIVE}${bindir_native}/${SDK_ARCH}${S
>  # Update BASE_PACKAGE_ARCH and PACKAGE_ARCHS
>  #
>  PACKAGE_ARCH = "${SDK_ARCH}-${SDKPKGSUFFIX}"
> -CANADIANEXTRAOS = "linux-uclibc linux-musl"
> +BASECANADIANEXTRAOS ?= "linux-uclibc linux-musl"
> +CANADIANEXTRAOS = "${BASECANADIANEXTRAOS}"
>  CANADIANEXTRAVENDOR = ""
>  MODIFYTOS ??= "1"
>  python () {
> @@ -34,8 +35,13 @@ python () {
> 
>  tos = d.getVar("TARGET_OS", True)
>  whitelist = []
> +extralibcs = [""]
> +if "uclibc" in d.getVar("BASECANADIANEXTRAOS", True):
> +extralibcs.append("uclibc")
> +if "musl" in d.getVar("BASECANADIANEXTRAOS", True):
> +extralibcs.append("musl")
>  for variant in ["", "spe", "x32", "eabi", "n32"]:
> -for libc in ["", "uclibc", "musl"]:
> +for libc in extralibcs:
>  entry = "linux"
>  if variant and libc:
>  entry = entry + "-" + libc + variant
> @@ -59,14 +65,20 @@ python () {
>  if tarch == "x86_64":
>  d.setVar("LIBCEXTENSION", "")
>  d.setVar("ABIEXTENSION", "")
> -d.appendVar("CANADIANEXTRAOS", " linux-gnux32 linux-uclibcx32 
> linux-muslx32")
> +d.appendVar("CANADIANEXTRAOS", " linux-gnux32")
> +for extraos in d.getVar("BASECANADIANEXTRAOS", True).split():
> +d.appendVar("CANADIANEXTRAOS", " " + extraos + "x32")
>  elif tarch == "powerpc":
>  # PowerPC can build "linux" and "linux-gnuspe"
>  d.setVar("LIBCEXTENSION", "")
>  d.setVar("ABIEXTENSION", "")
> -d.appendVar("CANADIANEXTRAOS", " linux-gnuspe linux-uclibcspe 
> linux-muslspe")
> +d.appendVar("CANADIANEXTRAOS", " linux-gnuspe")
> +for extraos in d.getVar("BASECANADIANEXTRAOS", True).split():
> +d.appendVar("CANADIANEXTRAOS", " " + extraos + "spe")
>  elif tarch == "mips64":
> -d.appendVar("CANADIANEXTRAOS", " linux-gnun32 linux-uclibcn32 
> linux-musln32")
> +d.appendVar("CANADIANEXTRAOS", " linux-gnun32")
> +for extraos in d.getVar("BASECANADIANEXTRAOS", True).split():
> +d.appendVar("CANADIANEXTRAOS", " " + extraos + "n32")
>  if tarch == "arm" or tarch == "armeb":
>  d.setVar("TARGET_OS", "linux-gnueabi")
>  else:
> --
> 1.9.3
> 
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] e2fsprogs: Fix missing check for permission denied.

2016-08-09 Thread jackie.huang
From: Jackie Huang 

If the path to "ROOT_SYSCONFDIR /mke2fs.conf" has a permission denied problem,
then the get_dirlist() call will return EACCES. But the code in profile_init
will treat that as a fatal error and all executions will fail with:
  Couldn't init profile successfully (error: 13).

But the problem should not really be visible for the target package as the path
then will be "/etc/mke2fs.conf", and it is not likely that a user have no
permission to read /etc.

Signed-off-by: Jian Liu 
Signed-off-by: Jackie Huang 
---
 ...s-fix-missing-check-for-permission-denied.patch | 32 ++
 meta/recipes-devtools/e2fsprogs/e2fsprogs_1.43.bb  |  2 ++
 2 files changed, 34 insertions(+)
 create mode 100644 
meta/recipes-devtools/e2fsprogs/e2fsprogs/e2fsprogs-fix-missing-check-for-permission-denied.patch

diff --git 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/e2fsprogs-fix-missing-check-for-permission-denied.patch
 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs/e2fsprogs-fix-missing-check-for-permission-denied.patch
new file mode 100644
index 000..33054c6
--- /dev/null
+++ 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs/e2fsprogs-fix-missing-check-for-permission-denied.patch
@@ -0,0 +1,32 @@
+Subject: [PATCH] Fix missing check for permission denied.
+
+If the path to "ROOT_SYSCONFDIR/mke2fs.conf" has a permission denied problem,
+then the get_dirlist() call will return EACCES. But the code in profile_init
+will treat that as a fatal error and all executions will fail with:
+  Couldn't init profile successfully (error: 13).
+
+Upstream-Status: Pending
+
+Written-by: Henrik Wallin 
+
+Signed-off-by: Jackie Huang 
+---
+ lib/support/profile.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/lib/support/profile.c b/lib/support/profile.c
+index 51a3314..1c1039f 100644
+--- a/lib/support/profile.c
 b/lib/support/profile.c
+@@ -335,7 +335,7 @@ profile_init(const char **files, profile_t *ret_profile)
+   *last = new_file;
+   last = _file->next;
+   }
+-  } else if ((retval != ENOTDIR) &&
++  } else if ((retval != ENOTDIR) && (retval != EACCES) &&
+  strcmp(*fs, default_filename))
+   goto errout;
+ 
+-- 
+2.7.4
+
diff --git a/meta/recipes-devtools/e2fsprogs/e2fsprogs_1.43.bb 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs_1.43.bb
index e82124b..837224c 100644
--- a/meta/recipes-devtools/e2fsprogs/e2fsprogs_1.43.bb
+++ b/meta/recipes-devtools/e2fsprogs/e2fsprogs_1.43.bb
@@ -11,6 +11,8 @@ SRC_URI += "file://acinclude.m4 \
 file://Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch \
 "
 
+SRC_URI_append_class-native = " 
file://e2fsprogs-fix-missing-check-for-permission-denied.patch"
+
 SRCREV = "d6adf070b0e85f209c0d7f310188b134b5cb7180"
 UPSTREAM_CHECK_GITTAGREGEX = "v(?P\d+\.\d+(\.\d+)*)$"
 
-- 
2.7.4

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


[OE-core] [PATCH][v4] watchdog: enable systemd support

2016-08-09 Thread jackie.huang
From: Roy Li 

1. inherit systemd, and add two unit files which are from Fedora 23
2. auto load soft dog kernel module

Signed-off-by: Roy Li 
Signed-off-by: Jackie Huang 
---
 .../watchdog/watchdog/watchdog-ping.service  | 11 +++
 meta/recipes-extended/watchdog/watchdog/watchdog.service | 11 +++
 meta/recipes-extended/watchdog/watchdog_5.15.bb  | 16 +++-
 3 files changed, 37 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-extended/watchdog/watchdog/watchdog-ping.service
 create mode 100644 meta/recipes-extended/watchdog/watchdog/watchdog.service

diff --git a/meta/recipes-extended/watchdog/watchdog/watchdog-ping.service 
b/meta/recipes-extended/watchdog/watchdog/watchdog-ping.service
new file mode 100644
index 000..44bac9d
--- /dev/null
+++ b/meta/recipes-extended/watchdog/watchdog/watchdog-ping.service
@@ -0,0 +1,11 @@
+[Unit]
+Description=watchdog daemon for use with ping test / network dependency
+After=network.target
+Conflicts=watchdog.service
+
+[Service]
+Type=forking
+ExecStart=@SBINDIR@/watchdog
+
+[Install]
+WantedBy=multi-user.target
diff --git a/meta/recipes-extended/watchdog/watchdog/watchdog.service 
b/meta/recipes-extended/watchdog/watchdog/watchdog.service
new file mode 100644
index 000..c5faa4e
--- /dev/null
+++ b/meta/recipes-extended/watchdog/watchdog/watchdog.service
@@ -0,0 +1,11 @@
+[Unit]
+Description=watchdog daemon
+# man systemd.special
+# auto added After=basic.target
+
+[Service]
+Type=forking
+ExecStart=@SBINDIR@/watchdog
+
+[Install]
+WantedBy=multi-user.target
diff --git a/meta/recipes-extended/watchdog/watchdog_5.15.bb 
b/meta/recipes-extended/watchdog/watchdog_5.15.bb
index ee1a893..7ff151c 100644
--- a/meta/recipes-extended/watchdog/watchdog_5.15.bb
+++ b/meta/recipes-extended/watchdog/watchdog_5.15.bb
@@ -12,6 +12,8 @@ SRC_URI = 
"${SOURCEFORGE_MIRROR}/watchdog/watchdog-${PV}.tar.gz \

file://0001-Include-linux-param.h-for-EXEC_PAGESIZE-definition.patch \
file://watchdog-init.patch \
file://watchdog-conf.patch \
+   file://watchdog-ping.service \
+   file://watchdog.service \
 "
 
 SRC_URI[md5sum] = "678c32f6f35a0492c9c1b76b4aa88828"
@@ -21,7 +23,7 @@ UPSTREAM_CHECK_URI = 
"http://sourceforge.net/projects/watchdog/files/watchdog/;
 UPSTREAM_CHECK_REGEX = "/watchdog/(?P(\d+[\.\-_]*)+)/"
 
 inherit autotools
-inherit update-rc.d
+inherit update-rc.d systemd
 
 DEPENDS_append_libc-musl = " libtirpc "
 CFLAGS_append_libc-musl = " -I${STAGING_INCDIR}/tirpc "
@@ -33,6 +35,18 @@ INITSCRIPT_PARAMS = "start 15 1 2 3 4 5 . stop 85 0 6 ."
 
 RRECOMMENDS_${PN} = "kernel-module-softdog"
 
+SYSTEMD_SERVICE_${PN} = "watchdog.service"
+
 do_install_append() {
install -D ${S}/redhat/watchdog.init 
${D}/${sysconfdir}/init.d/watchdog.sh
+   install -d ${D}${systemd_system_unitdir}
+   install -m 0644 ${WORKDIR}/watchdog*.service 
${D}${systemd_system_unitdir}
+
+   if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; 
then
+   install -d ${D}${sysconfdir}/modules-load.d
+   echo "softdog" > ${D}${sysconfdir}/modules-load.d/softdog.conf
+   sed -i -e 's,@SBINDIR@,${sbindir},g' 
${D}${systemd_system_unitdir}/*.service
+   fi
 }
+
+FILES_${PN} += "${systemd_system_unitdir}/*"
-- 
2.7.4

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


Re: [OE-core] [PATCH][v3] watchdog: enable systemd support

2016-08-09 Thread Huang, Jie (Jackie)


> -Original Message-
> From: openembedded-core-boun...@lists.openembedded.org 
> [mailto:openembedded-core-
> boun...@lists.openembedded.org] On Behalf Of jackie.hu...@windriver.com
> Sent: Friday, July 15, 2016 2:08 PM
> To: openembedded-core@lists.openembedded.org
> Subject: [OE-core] [PATCH][v3] watchdog: enable systemd support
> 
> From: Roy Li 
> 
> 1. inherit systemd, and add two unit files which are from Fedora 23
> 2. auto load soft dog kernel module
> 
> Signed-off-by: Roy Li 
> Signed-off-by: Jackie Huang 
> ---
>  .../watchdog/watchdog/watchdog-ping.service  | 11 +++
>  meta/recipes-extended/watchdog/watchdog/watchdog.service | 11 +++
>  meta/recipes-extended/watchdog/watchdog_5.15.bb  | 16 
> +++-
>  3 files changed, 37 insertions(+), 1 deletion(-)
>  create mode 100644 
> meta/recipes-extended/watchdog/watchdog/watchdog-ping.service
>  create mode 100644 meta/recipes-extended/watchdog/watchdog/watchdog.service
> 
> diff --git a/meta/recipes-extended/watchdog/watchdog/watchdog-ping.service 
> b/meta/recipes-
> extended/watchdog/watchdog/watchdog-ping.service
> new file mode 100644
> index 000..44bac9d
> --- /dev/null
> +++ b/meta/recipes-extended/watchdog/watchdog/watchdog-ping.service
> @@ -0,0 +1,11 @@
> +[Unit]
> +Description=watchdog daemon for use with ping test / network dependency
> +After=network.target
> +Conflicts=watchdog.service
> +
> +[Service]
> +Type=forking
> +ExecStart=@SBINDIR@/watchdog
> +
> +[Install]
> +WantedBy=multi-user.target
> diff --git a/meta/recipes-extended/watchdog/watchdog/watchdog.service 
> b/meta/recipes-
> extended/watchdog/watchdog/watchdog.service
> new file mode 100644
> index 000..c5faa4e
> --- /dev/null
> +++ b/meta/recipes-extended/watchdog/watchdog/watchdog.service
> @@ -0,0 +1,11 @@
> +[Unit]
> +Description=watchdog daemon
> +# man systemd.special
> +# auto added After=basic.target
> +
> +[Service]
> +Type=forking
> +ExecStart=@SBINDIR@/watchdog
> +
> +[Install]
> +WantedBy=multi-user.target
> diff --git a/meta/recipes-extended/watchdog/watchdog_5.15.bb b/meta/recipes-
> extended/watchdog/watchdog_5.15.bb
> index ee1a893..ac2ee5c 100644
> --- a/meta/recipes-extended/watchdog/watchdog_5.15.bb
> +++ b/meta/recipes-extended/watchdog/watchdog_5.15.bb
> @@ -12,6 +12,8 @@ SRC_URI = 
> "${SOURCEFORGE_MIRROR}/watchdog/watchdog-${PV}.tar.gz \
> 
> file://0001-Include-linux-param.h-for-EXEC_PAGESIZE-definition.patch \
> file://watchdog-init.patch \
> file://watchdog-conf.patch \
> +   file://watchdog-ping.service \
> +   file://watchdog.service \
>  "
> 
>  SRC_URI[md5sum] = "678c32f6f35a0492c9c1b76b4aa88828"
> @@ -21,7 +23,7 @@ UPSTREAM_CHECK_URI =
> "http://sourceforge.net/projects/watchdog/files/watchdog/;
>  UPSTREAM_CHECK_REGEX = "/watchdog/(?P(\d+[\.\-_]*)+)/"
> 
>  inherit autotools
> -inherit update-rc.d
> +inherit update-rc.d systemd
> 
>  DEPENDS_append_libc-musl = " libtirpc "
>  CFLAGS_append_libc-musl = " -I${STAGING_INCDIR}/tirpc "
> @@ -33,6 +35,18 @@ INITSCRIPT_PARAMS = "start 15 1 2 3 4 5 . stop 85 0 6 ."
> 
>  RRECOMMENDS_${PN} = "kernel-module-softdog"
> 
> +SYSTEMD_SERVICE_${PN} = "watchdog.service"
> +
>  do_install_append() {
>   install -D ${S}/redhat/watchdog.init 
> ${D}/${sysconfdir}/init.d/watchdog.sh
> + install -d ${D}${systemd_system_unitdir}
> + install -m 0644 ${WORKDIR}/watchdog*.service 
> ${D}${systemd_system_unitdir}
> +
> + if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; 
> then
> + install -d ${D}${sysconfdir}/modules-load.d
> + echo "softdog" > ${D}${sysconfdir}/modules-load.d/softdog.conf
> + sed -i -e 's,@SBINDIR@,${sbindir},g' 
> ${D}${systemd_unitdir}/system/*.service

This should be ${ systemd_system_unitdir} as well, I will change and send v4.

Thanks,
Jackie

> + fi
>  }
> +
> +FILES_${PN} += "${systemd_system_unitdir}/*"
> --
> 2.8.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/1] xmlto: fix depends for native

2016-08-09 Thread Robert Yang

After talked with Ming, we should use his patch:
xmlto: adds libxslt to DEPENDS


My patch moved libxslt-bin from RDEPENDS_${PN}_append_class-target to
RDEPENDS_${PN} but renamed to libxslt, which will cause libxslt to
be installed on target but not libxslt-bin.

// Robert

On 08/08/2016 01:46 PM, Robert Yang wrote:

xmlto-native requires xsltproc which is provided by libxslt to run.
Fixed:
xmlto: Can't continue, xsltproc tool not found or not executable.

Signed-off-by: Robert Yang 
---
  meta/recipes-devtools/xmlto/xmlto_0.0.28.bb | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb 
b/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
index 886e7b4..d6f2e92 100644
--- a/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
+++ b/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
@@ -20,9 +20,9 @@ RDEPENDS_${PN} = "docbook-xml-dtd4 \
util-linux \
libxml2 \
bash \
+  libxslt \
  "
  RDEPENDS_${PN}_append_class-target = " \
-  libxslt-bin \
coreutils \
  "
  CACHED_CONFIGUREVARS += "ac_cv_path_TAIL=${bindir}/tail"


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


[OE-core] [PATCH][master] curl: Update to 7.50.1

2016-08-09 Thread Armin Kuster
This fixes 3 CVES:

CVE-2016-5419
CVE-2016-5420
CVE-2016-5421

Affects curl <= 7.50.0

Signed-off-by: Armin Kuster 
---
 meta/recipes-support/curl/curl_7.49.1.bb | 71 
 meta/recipes-support/curl/curl_7.50.1.bb | 71 
 2 files changed, 71 insertions(+), 71 deletions(-)
 delete mode 100644 meta/recipes-support/curl/curl_7.49.1.bb
 create mode 100644 meta/recipes-support/curl/curl_7.50.1.bb

diff --git a/meta/recipes-support/curl/curl_7.49.1.bb 
b/meta/recipes-support/curl/curl_7.49.1.bb
deleted file mode 100644
index db9dc1a..000
--- a/meta/recipes-support/curl/curl_7.49.1.bb
+++ /dev/null
@@ -1,71 +0,0 @@
-SUMMARY = "Command line tool and library for client-side URL transfers"
-HOMEPAGE = "http://curl.haxx.se/;
-BUGTRACKER = "http://curl.haxx.se/mail/list.cgi?list=curl-tracker;
-SECTION = "console/network"
-LICENSE = "MIT"
-LIC_FILES_CHKSUM = 
"file://COPYING;beginline=8;md5=3a34942f4ae3fbf1a303160714e664ac"
-
-SRC_URI = "http://curl.haxx.se/download/curl-${PV}.tar.bz2 \
-   file://0001-replace-krb5-config-with-pkg-config.patch \
-"
-
-# curl likes to set -g0 in CFLAGS, so we stop it
-# from mucking around with debug options
-#
-SRC_URI += " file://configure_ac.patch"
-
-SRC_URI[md5sum] = "6bb1f7af5b58b30e4e6414b8c1abccab"
-SRC_URI[sha256sum] = 
"eb63cec4bef692eab9db459033f409533e6d10e20942f4b060b32819e81885f1"
-
-inherit autotools pkgconfig binconfig multilib_header
-
-PACKAGECONFIG ??= "${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6", "", 
d)} gnutls proxy zlib"
-PACKAGECONFIG_class-native = "ipv6 proxy ssl zlib"
-PACKAGECONFIG_class-nativesdk = "ipv6 proxy ssl zlib"
-
-PACKAGECONFIG[dict] = "--enable-dict,--disable-dict,"
-PACKAGECONFIG[gnutls] = "--with-gnutls,--without-gnutls,gnutls"
-PACKAGECONFIG[gopher] = "--enable-gopher,--disable-gopher,"
-PACKAGECONFIG[imap] = "--enable-imap,--disable-imap,"
-PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6,"
-PACKAGECONFIG[ldap] = "--enable-ldap,--disable-ldap,"
-PACKAGECONFIG[ldaps] = "--enable-ldaps,--disable-ldaps,"
-PACKAGECONFIG[libidn] = "--with-libidn,--without-libidn,libidn"
-PACKAGECONFIG[libssh2] = "--with-libssh2,--without-libssh2,libssh2"
-PACKAGECONFIG[pop3] = "--enable-pop3,--disable-pop3,"
-PACKAGECONFIG[proxy] = "--enable-proxy,--disable-proxy,"
-PACKAGECONFIG[rtmpdump] = "--with-librtmp,--without-librtmp,rtmpdump"
-PACKAGECONFIG[rtsp] = "--enable-rtsp,--disable-rtsp,"
-PACKAGECONFIG[smb] = "--enable-smb,--disable-smb,"
-PACKAGECONFIG[smtp] = "--enable-smtp,--disable-smtp,"
-PACKAGECONFIG[ssl] = "--with-ssl 
--with-random=/dev/urandom,--without-ssl,openssl"
-PACKAGECONFIG[telnet] = "--enable-telnet,--disable-telnet,"
-PACKAGECONFIG[tftp] = "--enable-tftp,--disable-tftp,"
-PACKAGECONFIG[zlib] = "--with-zlib=${STAGING_LIBDIR}/../,--without-zlib,zlib"
-PACKAGECONFIG[krb5] = "--with-gssapi,--without-gssapi,krb5"
-
-EXTRA_OECONF = " \
---enable-crypto-auth \
---with-ca-bundle=${sysconfdir}/ssl/certs/ca-certificates.crt \
---without-libmetalink \
---without-libpsl \
---without-nghttp2 \
-"
-
-do_install_append() {
-   oe_multilib_header curl/curlbuild.h
-}
-
-do_install_append_class-target() {
-   # cleanup buildpaths from curl-config
-   sed -i -e 's,${STAGING_DIR_HOST},,g' ${D}${bindir}/curl-config
-}
-
-PACKAGES =+ "lib${BPN}"
-
-FILES_lib${BPN} = "${libdir}/lib*.so.*"
-RRECOMMENDS_lib${BPN} += "ca-certificates"
-
-FILES_${PN} += "${datadir}/zsh"
-
-BBCLASSEXTEND = "native nativesdk"
diff --git a/meta/recipes-support/curl/curl_7.50.1.bb 
b/meta/recipes-support/curl/curl_7.50.1.bb
new file mode 100644
index 000..a21419a
--- /dev/null
+++ b/meta/recipes-support/curl/curl_7.50.1.bb
@@ -0,0 +1,71 @@
+SUMMARY = "Command line tool and library for client-side URL transfers"
+HOMEPAGE = "http://curl.haxx.se/;
+BUGTRACKER = "http://curl.haxx.se/mail/list.cgi?list=curl-tracker;
+SECTION = "console/network"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = 
"file://COPYING;beginline=8;md5=3a34942f4ae3fbf1a303160714e664ac"
+
+SRC_URI = "http://curl.haxx.se/download/curl-${PV}.tar.bz2 \
+   file://0001-replace-krb5-config-with-pkg-config.patch \
+"
+
+# curl likes to set -g0 in CFLAGS, so we stop it
+# from mucking around with debug options
+#
+SRC_URI += " file://configure_ac.patch"
+
+SRC_URI[md5sum] = "015f6a0217ca6f2c5442ca406476920b"
+SRC_URI[sha256sum] = 
"3c12c5f54ccaa1d40abc65d672107dcc75d3e1fcb38c267484334280096e5156"
+
+inherit autotools pkgconfig binconfig multilib_header
+
+PACKAGECONFIG ??= "${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6", "", 
d)} gnutls proxy zlib"
+PACKAGECONFIG_class-native = "ipv6 proxy ssl zlib"
+PACKAGECONFIG_class-nativesdk = "ipv6 proxy ssl zlib"
+
+PACKAGECONFIG[dict] = "--enable-dict,--disable-dict,"
+PACKAGECONFIG[gnutls] = "--with-gnutls,--without-gnutls,gnutls"
+PACKAGECONFIG[gopher] = "--enable-gopher,--disable-gopher,"
+PACKAGECONFIG[imap] = "--enable-imap,--disable-imap,"

Re: [OE-core] [PATCH] kernel.bbclass: add lzop dependency

2016-08-09 Thread Khem Raj

> On Aug 9, 2016, at 5:08 AM, Mike Looijmans  wrote:
> 
> I've been adding "lzop-native" to a lot of kernel recipes because you also 
> need it when generating an LZO compressed kernel (I like my boot time to be 
> under 1 second).
> 
> It's a small dependency which is hard to automatically figure out, since 
> there are a number of ways to trigger it (we just found two). I would welcome 
> this patch very much, and I don't think many will suffer…

From linux.inc days there is a python snippet which can do this conditionally 
e.g. see

https://github.com/kraj/meta-raspberrypi/blob/master/recipes-kernel/linux/linux-rpi.inc#L121

> 
> 
> On 01-08-16 14:32, Trevor Woerner wrote:
>> If the initramfs image is type lzo, then a native lzop is needed.
>> 
>> Signed-off-by: Trevor Woerner 
>> ---
>>  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 fa76a14..db42744 100644
>> --- a/meta/classes/kernel.bbclass
>> +++ b/meta/classes/kernel.bbclass
>> @@ -1,7 +1,7 @@
>>  inherit linux-kernel-base kernel-module-split
>> 
>>  PROVIDES += "virtual/kernel"
>> -DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc 
>> kmod-native depmodwrapper-cross bc-native"
>> +DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc 
>> kmod-native depmodwrapper-cross bc-native lzop-native"
>> 
>>  S = "${STAGING_KERNEL_DIR}"
>>  B = "${WORKDIR}/build"
>> 
> 
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



signature.asc
Description: Message signed with OpenPGP using GPGMail
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 1/1] initrdscripts/init-install*: Add rootwait when installing to USB devices

2016-08-09 Thread California Sullivan
It can take a bit for USB devices to be detected, so if a USB device is
your rootfs and you don't set rootwait you will most likely get a kernel
panic. Fix this by adding rootwait to the kernel command line on
installation.

Fixes [YOCTO #9462].

Signed-off-by: California Sullivan 
---
 meta/recipes-core/initrdscripts/files/init-install-efi.sh | 6 ++
 meta/recipes-core/initrdscripts/files/init-install.sh | 5 +
 2 files changed, 11 insertions(+)

diff --git a/meta/recipes-core/initrdscripts/files/init-install-efi.sh 
b/meta/recipes-core/initrdscripts/files/init-install-efi.sh
index f564f4e..bf8a76b 100644
--- a/meta/recipes-core/initrdscripts/files/init-install-efi.sh
+++ b/meta/recipes-core/initrdscripts/files/init-install-efi.sh
@@ -144,6 +144,12 @@ if [ ! "${device#/dev/mmcblk}" = "${device}" ]; then
 part_prefix="p"
 rootwait="rootwait"
 fi
+
+# USB devices also require rootwait
+if [ -n `readlink /dev/disk/by-id/usb* | grep $TARGET_DEVICE_NAME` ]; then
+rootwait="rootwait"
+fi
+
 bootfs=${device}${part_prefix}1
 rootfs=${device}${part_prefix}2
 swap=${device}${part_prefix}3
diff --git a/meta/recipes-core/initrdscripts/files/init-install.sh 
b/meta/recipes-core/initrdscripts/files/init-install.sh
index 72ce92b..5f2599e 100644
--- a/meta/recipes-core/initrdscripts/files/init-install.sh
+++ b/meta/recipes-core/initrdscripts/files/init-install.sh
@@ -152,6 +152,11 @@ if [ ! "${device#/dev/mmcblk}" = "${device}" ]; then
 rootwait="rootwait"
 fi
 
+# USB devices also require rootwait
+if [ -n `readlink /dev/disk/by-id/usb* | grep $TARGET_DEVICE_NAME` ]; then
+rootwait="rootwait"
+fi
+
 if [ $grub_version -eq 0 ] ; then
 bios_boot=''
 bootfs=${device}${part_prefix}1
-- 
2.5.5

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


[OE-core] [PATCH v2] u-boot.inc: Enable out-of-tree builds

2016-08-09 Thread Manjukumar Matha
This patch enabled out-of-tree builds for u-boot. This also helps building
u-boot using EXTERNALSRC flow

Signed-off-by: Manjukumar Matha 
---
 meta/recipes-bsp/u-boot/u-boot.inc | 35 ++-
 1 file changed, 18 insertions(+), 17 deletions(-)

diff --git a/meta/recipes-bsp/u-boot/u-boot.inc 
b/meta/recipes-bsp/u-boot/u-boot.inc
index 1f64faa..9e19c4f 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -9,6 +9,7 @@ LIC_FILES_CHKSUM = 
"file://Licenses/README;md5=a2c678cfd4a4d97135585cad908541c6"
 SRC_URI = "git://git.denx.de/u-boot.git;branch=master"
 
 S = "${WORKDIR}/git"
+B = "${WORKDIR}/build"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 
@@ -88,12 +89,12 @@ do_compile () {
 j=$(expr $j + 1);
 if [ $j -eq $i ]
 then
-oe_runmake O=${config} ${config}
-oe_runmake O=${config} ${UBOOT_MAKE_TARGET}
+oe_runmake -C ${S} O=${B}/${config} ${config}
+oe_runmake -C ${S} O=${B}/${config} ${UBOOT_MAKE_TARGET}
 for binary in ${UBOOT_BINARIES}; do
 k=$(expr $k + 1);
 if [ $k -eq $i ]; then
-cp ${S}/${config}/${binary} 
${S}/${config}/u-boot-${type}.${UBOOT_SUFFIX}
+cp ${B}/${config}/${binary} 
${B}/${config}/u-boot-${type}.${UBOOT_SUFFIX}
 fi
 done
 unset k
@@ -103,8 +104,8 @@ do_compile () {
 done
 unset  i
 else
-oe_runmake ${UBOOT_MACHINE}
-oe_runmake ${UBOOT_MAKE_TARGET}
+oe_runmake -C ${S} O=${B} ${UBOOT_MACHINE}
+oe_runmake -C ${S} O=${B} ${UBOOT_MAKE_TARGET}
 fi
 
 }
@@ -119,7 +120,7 @@ do_install () {
 if [ $j -eq $i ]
 then
 install -d ${D}/boot
-install ${S}/${config}/u-boot-${type}.${UBOOT_SUFFIX} 
${D}/boot/u-boot-${type}-${PV}-${PR}.${UBOOT_SUFFIX}
+install ${B}/${config}/u-boot-${type}.${UBOOT_SUFFIX} 
${D}/boot/u-boot-${type}-${PV}-${PR}.${UBOOT_SUFFIX}
 ln -sf u-boot-${type}-${PV}-${PR}.${UBOOT_SUFFIX} 
${D}/boot/${UBOOT_BINARY}-${type}
 ln -sf u-boot-${type}-${PV}-${PR}.${UBOOT_SUFFIX} 
${D}/boot/${UBOOT_BINARY}
 fi
@@ -129,7 +130,7 @@ do_install () {
 unset  i
 else
 install -d ${D}/boot
-install ${S}/${UBOOT_BINARY} ${D}/boot/${UBOOT_IMAGE}
+install ${B}/${UBOOT_BINARY} ${D}/boot/${UBOOT_IMAGE}
 ln -sf ${UBOOT_IMAGE} ${D}/boot/${UBOOT_BINARY}
 fi
 
@@ -143,7 +144,7 @@ do_install () {
 j=$(expr $j + 1);
 if [ $j -eq $i ]
 then
-install ${S}/${config}/${UBOOT_ELF} 
${D}/boot/u-boot-${type}-${PV}-${PR}.${UBOOT_ELF_SUFFIX}
+install ${B}/${config}/${UBOOT_ELF} 
${D}/boot/u-boot-${type}-${PV}-${PR}.${UBOOT_ELF_SUFFIX}
 ln -sf u-boot-${type}-${PV}-${PR}.${UBOOT_ELF_SUFFIX} 
${D}/boot/${UBOOT_BINARY}-${type}
 ln -sf u-boot-${type}-${PV}-${PR}.${UBOOT_ELF_SUFFIX} 
${D}/boot/${UBOOT_BINARY}
 fi
@@ -152,7 +153,7 @@ do_install () {
 done
 unset i
 else
-install ${S}/${UBOOT_ELF} ${D}/boot/${UBOOT_ELF_IMAGE}
+install ${B}/${UBOOT_ELF} ${D}/boot/${UBOOT_ELF_IMAGE}
 ln -sf ${UBOOT_ELF_IMAGE} ${D}/boot/${UBOOT_ELF_BINARY}
 fi
 fi
@@ -172,7 +173,7 @@ do_install () {
 j=$(expr $j + 1);
 if [ $j -eq $i ]
 then
- install ${S}/${config}/${SPL_BINARY} 
${D}/boot/${SPL_IMAGE}-${type}-${PV}-${PR}
+ install ${B}/${config}/${SPL_BINARY} 
${D}/boot/${SPL_IMAGE}-${type}-${PV}-${PR}
  ln -sf ${SPL_IMAGE}-${type}-${PV}-${PR} 
${D}/boot/${SPL_BINARYNAME}-${type}
  ln -sf ${SPL_IMAGE}-${type}-${PV}-${PR} 
${D}/boot/${SPL_BINARYNAME}
 fi
@@ -181,7 +182,7 @@ do_install () {
 done
 unset  i
 else
-install ${S}/${SPL_BINARY} ${D}/boot/${SPL_IMAGE}
+install ${B}/${SPL_BINARY} ${D}/boot/${SPL_IMAGE}
 ln -sf ${SPL_IMAGE} ${D}/boot/${SPL_BINARYNAME}
 fi
 fi
@@ -205,7 +206,7 @@ do_deploy () {
 if [ $j -eq $i ]
 then
 install -d ${DEPLOYDIR}
-install ${S}/${config}/u-boot-${type}.${UBOOT_SUFFIX} 
${DEPLOYDIR}/u-boot-${type}-${PV}-${PR}.${UBOOT_SUFFIX}
+install ${B}/${config}/u-boot-${type}.${UBOOT_SUFFIX} 
${DEPLOYDIR}/u-boot-${type}-${PV}-${PR}.${UBOOT_SUFFIX}
 cd 

[OE-core] [PATCH 2/2] connman: clean up musl fixes

2016-08-09 Thread Ross Burton
The upstreamable include fixes have been sent upstream.  The patch set adds
AC_USE_SYSTEM_EXTENSIONS so we don't need to explictly define _GNU_SOURCE
anymore.

Signed-off-by: Ross Burton 
---
 meta/recipes-connectivity/connman/connman.inc  |   1 -
 .../0003-Fix-header-inclusions-for-musl.patch  | 118 --
 .../connman/connman/includes.patch | 423 +
 meta/recipes-connectivity/connman/connman_1.33.bb  |   5 +-
 4 files changed, 425 insertions(+), 122 deletions(-)
 delete mode 100644 
meta/recipes-connectivity/connman/connman/0003-Fix-header-inclusions-for-musl.patch
 create mode 100644 meta/recipes-connectivity/connman/connman/includes.patch

diff --git a/meta/recipes-connectivity/connman/connman.inc 
b/meta/recipes-connectivity/connman/connman.inc
index f09a2de..35a7eed 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -29,7 +29,6 @@ EXTRA_OECONF += "\
 --disable-polkit \
 --enable-client \
 "
-CFLAGS += "-D_GNU_SOURCE"
 
 PACKAGECONFIG ??= "wispr \
${@bb.utils.contains('DISTRO_FEATURES', 
'systemd','systemd', '', d)} \
diff --git 
a/meta/recipes-connectivity/connman/connman/0003-Fix-header-inclusions-for-musl.patch
 
b/meta/recipes-connectivity/connman/connman/0003-Fix-header-inclusions-for-musl.patch
deleted file mode 100644
index eefc683..000
--- 
a/meta/recipes-connectivity/connman/connman/0003-Fix-header-inclusions-for-musl.patch
+++ /dev/null
@@ -1,118 +0,0 @@
-From 67645a01a2f3f52625d8dd77f2811a9e213e1b7d Mon Sep 17 00:00:00 2001
-From: Khem Raj 
-Date: Sun, 13 Sep 2015 13:28:20 -0700
-Subject: [PATCH] Fix header inclusions for musl
-
-Signed-off-by: Khem Raj 

-Upstream-Status: Pending
-
- gweb/gresolv.c| 1 +
- plugins/wifi.c| 3 +--
- src/tethering.c   | 2 --
- tools/dhcp-test.c | 1 -
- tools/dnsproxy-test.c | 1 +
- 5 files changed, 3 insertions(+), 5 deletions(-)
-
-Index: connman-1.30/gweb/gresolv.c
-===
 connman-1.30.orig/gweb/gresolv.c
-+++ connman-1.30/gweb/gresolv.c
-@@ -28,6 +28,7 @@
- #include 
- #include 
- #include 
-+#include 
- #include 
- #include 
- #include 
-Index: connman-1.30/plugins/wifi.c
-===
 connman-1.30.orig/plugins/wifi.c
-+++ connman-1.30/plugins/wifi.c
-@@ -30,9 +30,8 @@
- #include 
- #include 
- #include 
--#include 
--#include 
- #include 
-+#include 
- 
- #ifndef IFF_LOWER_UP
- #define IFF_LOWER_UP  0x1
-Index: connman-1.30/src/tethering.c
-===
 connman-1.30.orig/src/tethering.c
-+++ connman-1.30/src/tethering.c
-@@ -31,10 +31,8 @@
- #include 
- #include 
- #include 
--#include 
- #include 
- #include 
--#include 
- #include 
- #include 
- 
-Index: connman-1.30/tools/dhcp-test.c
-===
 connman-1.30.orig/tools/dhcp-test.c
-+++ connman-1.30/tools/dhcp-test.c
-@@ -33,7 +33,6 @@
- #include 
- #include 
- #include 
--#include 
- 
- #include 
- 
-Index: connman-1.30/tools/dnsproxy-test.c
-===
 connman-1.30.orig/tools/dnsproxy-test.c
-+++ connman-1.30/tools/dnsproxy-test.c
-@@ -27,6 +27,7 @@
- #include 
- #include 
- #include 
-+#include 
- #include 
- #include 
- #include 
-Index: connman-1.30/configure.ac
-===
 connman-1.30.orig/configure.ac
-+++ connman-1.30/configure.ac
-@@ -173,6 +173,8 @@ AM_CONDITIONAL(PPTP_BUILTIN, test "${ena
- 
- AC_CHECK_HEADERS([execinfo.h])
- 
-+AC_CHECK_MEMBERS([struct in6_pktinfo.ipi6_addr], [], [], [[#include 
]])
-+
- AC_CHECK_HEADERS(resolv.h, dummy=yes,
-   AC_MSG_ERROR(resolver header files are required))
- AC_CHECK_LIB(resolv, ns_initparse, dummy=yes, [
-Index: connman-1.30/gdhcp/common.h
-===
 connman-1.30.orig/gdhcp/common.h
-+++ connman-1.30/gdhcp/common.h
-@@ -19,6 +19,7 @@
-  *
-  */
- 
-+#include 
- #include 
- #include 
- 
-@@ -170,8 +171,8 @@ static const uint8_t dhcp_option_lengths
-   [OPTION_U32]= 4,
- };
- 
--/* already defined within netinet/in.h if using GNU compiler */
--#ifndef __USE_GNU
-+/* already defined within netinet/in.h if using GNU or musl libc */
-+#ifndef HAVE_STRUCT_IN6_PKTINFO_IPI6_ADDR
- struct in6_pktinfo {
-   struct in6_addr ipi6_addr;  /* src/dst IPv6 address */
-   unsigned int ipi6_ifindex;  /* send/recv interface index */
diff --git a/meta/recipes-connectivity/connman/connman/includes.patch 
b/meta/recipes-connectivity/connman/connman/includes.patch
new file mode 100644
index 000..55cb187
--- /dev/null
+++ b/meta/recipes-connectivity/connman/connman/includes.patch
@@ -0,0 +1,423 

[OE-core] [PATCH 1/2] connman: disable version-scripts to fix crashes at startup

2016-08-09 Thread Ross Burton
With binutils 2.27 on at least MIPS, connmand will crash on startup.  This
appears to be due to the symbol visibilty scripts hiding symbols that stdio
looks up at runtime, resulting in it segfaulting.

This certainly appears to be a bug in binutils 2.27 although the problem has
been known about for some time:

https://sourceware.org/bugzilla/show_bug.cgi?id=17908

As the version scripts are only used to hide symbols from plugins we can safely
remove the scripts to work around the problem until binutils is fixed.

Signed-off-by: Ross Burton 
---
 .../connman/connman/no-version-scripts.patch   | 27 ++
 meta/recipes-connectivity/connman/connman_1.33.bb  |  2 +-
 2 files changed, 28 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-connectivity/connman/connman/no-version-scripts.patch

diff --git a/meta/recipes-connectivity/connman/connman/no-version-scripts.patch 
b/meta/recipes-connectivity/connman/connman/no-version-scripts.patch
new file mode 100644
index 000..e96e38b
--- /dev/null
+++ b/meta/recipes-connectivity/connman/connman/no-version-scripts.patch
@@ -0,0 +1,27 @@
+With binutils 2.27 on at least MIPS, connmand will crash on startup.  This
+appears to be due to the symbol visibilty scripts hiding symbols that stdio
+looks up at runtime, resulting in it segfaulting.
+
+This certainly appears to be a bug in binutils 2.27 although the problem has
+been known about for some time:
+
+https://sourceware.org/bugzilla/show_bug.cgi?id=17908
+
+As the version scripts are only used to hide symbols from plugins we can safely
+remove the scripts to work around the problem until binutils is fixed.
+
+Upstream-Status: Inappropriate
+Signed-off-by: Ross Burton 
+
+diff --git a/Makefile.am b/Makefile.am
+index d70725c..76ae432 100644
+--- a/Makefile.am
 b/Makefile.am
+@@ -132,2 +132 @@ src_connmand_LDADD = gdbus/libgdbus-internal.la 
$(builtin_libadd) \
+-src_connmand_LDFLAGS = -Wl,--export-dynamic \
+-  -Wl,--version-script=$(srcdir)/src/connman.ver
++src_connmand_LDFLAGS = -Wl,--export-dynamic
+@@ -166,2 +165 @@ vpn_connman_vpnd_LDADD = gdbus/libgdbus-internal.la 
$(builtin_vpn_libadd) \
+-vpn_connman_vpnd_LDFLAGS = -Wl,--export-dynamic \
+-  -Wl,--version-script=$(srcdir)/vpn/vpn.ver
++vpn_connman_vpnd_LDFLAGS = -Wl,--export-dynamic
diff --git a/meta/recipes-connectivity/connman/connman_1.33.bb 
b/meta/recipes-connectivity/connman/connman_1.33.bb
index b31b146..5485c15 100644
--- a/meta/recipes-connectivity/connman/connman_1.33.bb
+++ b/meta/recipes-connectivity/connman/connman_1.33.bb
@@ -3,6 +3,7 @@ require connman.inc
 SRC_URI  = "${KERNELORG_MIRROR}/linux/network/${BPN}/${BP}.tar.xz \
 
file://0001-plugin.h-Change-visibility-to-default-for-debug-symb.patch \
 file://connman \
+file://no-version-scripts.patch \
 "
 SRC_URI_append_libc-musl = 
"file://0002-resolve-musl-does-not-implement-res_ninit.patch \
 file://0003-Fix-header-inclusions-for-musl.patch \
@@ -12,4 +13,3 @@ SRC_URI[md5sum] = "c51903fd3e7a6a371d12ac5d72a1fa01"
 SRC_URI[sha256sum] = 
"bc8946036fa70124d663136f9f6b6238d897ca482782df907b07a428b09df5a0"
 
 RRECOMMENDS_${PN} = "connman-conf"
-
-- 
2.8.1

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


Re: [OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES configurability

2016-08-09 Thread Burton, Ross
On 9 August 2016 at 12:43, Tom Hochstein  wrote:

> No, master does not need this fix. The problematic patch
> 0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch does not
> exist on master because the underlying issue was addressed upstream
> (without introducing this bug).
>
>
Thanks for the clarification Tom.

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


Re: [OE-core] [RFT] binutils 2.27

2016-08-09 Thread Richard Purdie
On Tue, 2016-08-09 at 07:42 -0700, Khem Raj wrote:
> I could also see it on ppc. backtrace, shows the segfault is in exit
> path and happens in libc
> at this point, I think the problem is how libc is compiled with
> binutils 2.27, connman itself
> is ok. 

Its the issue here:

https://sourceware.org/ml/glibc-bugs/2015-01/msg00274.html
https://sourceware.org/bugzilla/show_bug.cgi?id=17908

Basically, if you remove the global _IO_stdin_used symbol, it triggers
compatibility code which crashes.

I've confirmed that if I add that symbol to the version-script in
connman, things work again.

Any idea how we raise the priority of this issue. There are no comments
on the bug despite it having been posted a while ago :(.

Cheers,

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


Re: [OE-core] [PATCH 19/20] oeqa.selftest.wic: Split configuration from code

2016-08-09 Thread Jose Lamego
Hi Ed, thanks for your quick response.

On 08/09/2016 02:26 AM, Ed Bartosh wrote:
> Hi Jose,
> 
> On Mon, Aug 08, 2016 at 09:23:07AM -0700, Jose Lamego wrote:
>> Improve oeqa-selftest capabilities and UX by placing
>> test configuration features and variables into a separate
>> configuration file.
>>
> 
> Frankly I have mixed feelings about these changes. From the first look
> splitting configuration and code is a good idea. However, looking at the
> result I must say I feel like the test became less readable and harder
> to understand and maintain.
> 
> May be the changes make sense for other tests, but for this one they make
> my life as a maintainer of this one less easy I'd say.
I totally understand you point. But before we drop the idea of this
change for wic, would there be a way to keep the code  understandable
while leveraging on the configuration file to be able to easily try
different input/output combinations without touching the test code
(which is the basic purpose of the change)? Maybe renaming the conf
variables (see below proposals)?
If you still believe there would be no benefits from this change, it
will be better to skip it for wic.
> 
>> [Yocto 9389]
>>
>> Signed-off-by: Jose Lamego 
>> ---
>>  meta/lib/oeqa/selftest/conf/wic.conf |  10 +++
>>  meta/lib/oeqa/selftest/wic.py| 150 
>> ---
>>  2 files changed, 98 insertions(+), 62 deletions(-)
>>  create mode 100644 meta/lib/oeqa/selftest/conf/wic.conf
>>
>> diff --git a/meta/lib/oeqa/selftest/conf/wic.conf 
>> b/meta/lib/oeqa/selftest/conf/wic.conf
>> new file mode 100644
>> index 000..e2afb8d
>> --- /dev/null
>> +++ b/meta/lib/oeqa/selftest/conf/wic.conf
>> @@ -0,0 +1,10 @@
>> +[Wic]
>> +setUpClass_image1 = core-image-minimal
>> +setUpClass_image2 = minimal
> setUpClass_image2 is not used in the code.
> 
>> +setUpClass_image3 = qemux86-directdisk
>> +setUpClass_image4 = directdisk
>> +setUpClass_image5 = wic-image-minimal
> setUpClass_image1 is an image produced by bitbake, but setUpClass_image3
> is a name of wic image. They're not the same. Bitbake image is an input
> for wic and wic image is an output. setUpClass_image5 is again the name
> of the bitbake target, but it differs from _image3. Naming doesn't reflect 
> this fact.
setUpClass_image1 could be renamed to setUpClass_bitbakeImage1
setUpClass_image3 could be renamed to setClass_wicImage
setUpClass_image5 to setUpClass_bitbakeImage2
> 
> 
>> +setUpClass_features = IMAGE_FSTYPES += " hddimg"
>> +  MACHINE_FEATURES_append = " efi"
>> +setUpClass_recipes = syslinux syslinux-native parted-native gptfdisk-native 
>> dosfstools-native mtools-native bmap-tools-native
> This list is used only once in the test. Moving it to config file makes
> it longer to understand what's in the list.
> 
>> diff --git a/meta/lib/oeqa/selftest/wic.py b/meta/lib/oeqa/selftest/wic.py
>> index e550785..dc81457 100644
>> --- a/meta/lib/oeqa/selftest/wic.py
>> +++ b/meta/lib/oeqa/selftest/wic.py
>> @@ -31,6 +31,7 @@ from shutil import rmtree
>>  from oeqa.selftest.base import oeSelfTest
>>  from oeqa.utils.commands import runCmd, bitbake, get_bb_var, runqemu
>>  from oeqa.utils.decorators import testcase
>> +from oeqa.utils.readconfig import conffile
>>  
>>  
>>  class Wic(oeSelfTest):
>> @@ -39,6 +40,18 @@ class Wic(oeSelfTest):
>>  resultdir = "/var/tmp/wic/build/"
>>  image_is_ready = False
>>  
>> +@classmethod
>> +def setUpClass(cls):
>> +# Get test configurations from configuration file
>> +cls.config = conffile(__file__)
>> +cls.image1 = cls.config.get('Wic', 'setUpClass_image1')
>> +cls.image2 = cls.config.get('Wic', 'setUpClass_image2')
>> +cls.image3 = cls.config.get('Wic', 'setUpClass_image3')
>> +cls.image4 = cls.config.get('Wic', 'setUpClass_image4')
>> +cls.image5 = cls.config.get('Wic', 'setUpClass_image5')
>> +cls.features = cls.config.get('Wic', 'setUpClass_features')
>> +cls.recipes = cls.config.get('Wic', 'setUpClass_recipes')
> This looks like a duplication of code and configuration and also hides real
> image names and features making test much less readable.
> 
>> +
>>  def setUpLocal(self):
>>  """This code is executed before each test method."""
>>  self.write_config('IMAGE_FSTYPES += " hddimg"\n'
>> @@ -48,9 +61,8 @@ class Wic(oeSelfTest):
>>  # clean up which can result in the native tools built earlier in
>>  # setUpClass being unavailable.
>>  if not Wic.image_is_ready:
>> -bitbake('syslinux syslinux-native parted-native gptfdisk-native 
>> '
>> -'dosfstools-native mtools-native bmap-tools-native')
>> -bitbake('core-image-minimal')
>> +bitbake(self.recipes)
>> +bitbake(self.image1)
> I'd really prefer to see list of dependencies and the image name right
> in the test code without having to 

Re: [OE-core] [RFT] binutils 2.27

2016-08-09 Thread Khem Raj

> On Aug 8, 2016, at 4:16 PM, Burton, Ross  wrote:
> 
> 
> On 8 August 2016 at 16:59, Richard Purdie  > wrote:
> FWIW, I think the connmand segfault on mips and powerpc is hinted at
> with this bug:
> 
> https://bugs.launchpad.net/ubuntu/+source/lua5.3/+bug/1570055 
> 
> 
> connmand is a binary linked with a linker version-script. If I build
> connmand without that option, it doesn't segfault, at least on mips.
> 
> The binutils manuals say you shouldn't build binaries with version
> scripts:
> 
> http://ftp.gnu.org/old-gnu/Manuals/ld-2.9.1/html_node/ld_25.html 
> 
> 
> Its suspected connman might do this to limit the function exposure to
> its plugins.
> 
> Now sure exactly what needs to happen here without more research but it
> certainly hits at where the problem is. I suspect its the same issue on
> ppc.
> 
> I've verified that with binutils 2.26 on x86-64, a minimal test case for an 
> executable using a version script to limit exported symbols to dlopen()d 
> modules does in fact work.  A toolchain is building now to see what happens 
> in different environments.

I could also see it on ppc. backtrace, shows the segfault is in exit path and 
happens in libc
at this point, I think the problem is how libc is compiled with binutils 2.27, 
connman itself
is ok.

> 
> Ross



signature.asc
Description: Message signed with OpenPGP using GPGMail
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] how build but do not install package

2016-08-09 Thread Mike Looijmans


On 09-08-16 14:53, Dvorkin Dmitry wrote:> Dear OE-developers!>> I'm building an image and SDK for my device without any problem. But I'd like> to build some extra packages during the time I'm building my image or SDK, but> I don't want to install this group of packages. Just keep them in /deploy/rpm/> or /deploy/ipk, available for installation by the package manager. Also I'd> like to automatically rebuild them if some dependencies are changed.>> How to do it correctly using bitbake/yocto/oe?Add the recipe to the dependencies of the image recipe:DEPENDS += "package-that-i-want-built-but-not-installed another-package"> How to create group of packages and rebuild them if something is changed, but> do not install into SDK or image.Create a recipe that DEPENDS on the group, and add that recipe to your DEPENDS as above.
 
Kind regards,
 
Mike Looijmans
System Expert
 





  
  

  TOPIC
  Products

   

   
  

  Materiaalweg
  4

   

   
  

  5681
  RJ Best

  T:

  +31 (0) 499 33
  69 69
  

  Postbus
  440

  E:

  mike.looijm...@topicproducts.com
  

  5680 AK
  Best

  W:

  www.topicproducts.com
  
The Netherlands



 Please consider the environment before printing this
e-mailTopic zoekt gedreven (embedded) software specialisten!



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


[OE-core] [PATCH] dbus: backport stdint.h build fix

2016-08-09 Thread Ioan-Adrian Ratiu
This patch fixes an error where dbus configure doesn't detect
stdint.h correctly.

Upstream commit 1bfde222 on branches dbus-1.10 and master

Signed-off-by: Ioan-Adrian Ratiu 
---
 ...1-configure.ac-explicitely-check-stdint.h.patch | 38 ++
 meta/recipes-core/dbus/dbus_1.10.8.bb  |  1 +
 2 files changed, 39 insertions(+)
 create mode 100644 
meta/recipes-core/dbus/dbus/0001-configure.ac-explicitely-check-stdint.h.patch

diff --git 
a/meta/recipes-core/dbus/dbus/0001-configure.ac-explicitely-check-stdint.h.patch
 
b/meta/recipes-core/dbus/dbus/0001-configure.ac-explicitely-check-stdint.h.patch
new file mode 100644
index 000..85acb7b
--- /dev/null
+++ 
b/meta/recipes-core/dbus/dbus/0001-configure.ac-explicitely-check-stdint.h.patch
@@ -0,0 +1,38 @@
+From 1bfde222926be624a30a6e4b2cdc2c5064a36298 Mon Sep 17 00:00:00 2001
+From: Ioan-Adrian Ratiu 
+Date: Fri, 29 Jul 2016 01:19:37 +0300
+Subject: [PATCH] configure.ac: explicitely check stdint.h
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Otherwise HAVE_STDINT_H will not be defined or the var will not be
+picked up from cache so builds could fail with errors like:
+| ../../dbus-1.10.8/dbus/dbus-internals.h:239:8: error: ‘uintptr_t’ undeclared 
(first use in this function)
+
+Signed-off-by: Ioan-Adrian Ratiu 
+[smcv: fix Autoconf underquoting]
+Reviewed-by: Simon McVittie 
+
+Upstream-Status: Backport [from dbus-1.10]
+
+---
+ configure.ac | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/configure.ac b/configure.ac
+index cf5c5b9..a228d63 100644
+--- a/configure.ac
 b/configure.ac
+@@ -699,6 +699,8 @@ AC_CHECK_HEADERS(byteswap.h)
+ 
+ AC_CHECK_HEADERS(unistd.h)
+ 
++AC_CHECK_HEADERS([stdint.h])
++
+ AC_CHECK_HEADERS(ws2tcpip.h)
+ 
+ AC_CHECK_HEADERS(alloca.h)
+-- 
+2.9.2
+
diff --git a/meta/recipes-core/dbus/dbus_1.10.8.bb 
b/meta/recipes-core/dbus/dbus_1.10.8.bb
index bfaf65a..62dd8bb 100644
--- a/meta/recipes-core/dbus/dbus_1.10.8.bb
+++ b/meta/recipes-core/dbus/dbus_1.10.8.bb
@@ -17,6 +17,7 @@ SRC_URI = 
"http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \
file://dbus-1.init \
file://os-test.patch \
file://clear-guid_from_server-if-send_negotiate_unix_f.patch \
+   file://0001-configure.ac-explicitely-check-stdint.h.patch \
 "
 
 SRC_URI[md5sum] = "e912e930f249454752512aa7ac864d43"
-- 
2.9.2

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


Re: [OE-core] "parted" vs "sfdisk"

2016-08-09 Thread Mike Looijmans


On 05-08-16 12:19, Robert P. J. Day wrote:>>    your personal opinions, if you would -- i'm working on some scripts> to do automated installs on a target board, currently based on> parted, but parted seems a bit dense at times, and awkward, and i'm> thinking of switching to sfdisk.>>    for people who have done this sort of thing, do you have any strong> opinions either way of parted versus sfdisk? i realize that's not much> to go on, just curious about personal preferences, and why.I use parted if available, because sfdisk was limited in what it could handle (e.g. >2TB disks). Don't know if that's still the case.
 
Kind regards,
 
Mike Looijmans
System Expert
 





  
  

  TOPIC
  Products

   

   
  

  Materiaalweg
  4

   

   
  

  5681
  RJ Best

  T:

  +31 (0) 499 33
  69 69
  

  Postbus
  440

  E:

  mike.looijm...@topicproducts.com
  

  5680 AK
  Best

  W:

  www.topicproducts.com
  
The Netherlands



 Please consider the environment before printing this
e-mailTopic zoekt gedreven (embedded) software specialisten!



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


Re: [OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES configurability

2016-08-09 Thread Otavio Salvador
On Tue, Aug 9, 2016 at 8:43 AM, Tom Hochstein  wrote:
> No, master does not need this fix. The problematic patch
> 0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch does not
> exist on master because the underlying issue was addressed upstream (without
> introducing this bug).

Acked-by: Otavio Salvador 


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


[OE-core] how build but do not install package

2016-08-09 Thread Dvorkin Dmitry

Dear OE-developers!

I'm building an image and SDK for my device without any problem. But I'd 
like to build some extra packages during the time I'm building my image 
or SDK, but I don't want to install this group of packages. Just keep 
them in /deploy/rpm/ or /deploy/ipk, available for installation by the 
package manager. Also I'd like to automatically rebuild them if some 
dependencies are changed.


How to do it correctly using bitbake/yocto/oe?

How to create group of packages and rebuild them if something is 
changed, but do not install into SDK or image.


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


Re: [OE-core] [PATCH][krogoth] sudo: CVE-2015-8239

2016-08-09 Thread Sona Sarmadi
Hi Ross,

This seems to be fixed in master. I checked the code.

//Sona

From: Burton, Ross [mailto:ross.bur...@intel.com]
Sent: den 9 augusti 2016 13:54
To: Sona Sarmadi 
Cc: OE-core 
Subject: Re: [OE-core] [PATCH][krogoth] sudo: CVE-2015-8239


On 9 August 2016 at 12:04, Sona Sarmadi 
> wrote:
Fixes race condition when checking digests in sudoers.

Reference:
http://seclists.org/oss-sec/2015/q4/327

Reference to upstream fixes:
https://www.sudo.ws/repos/sudo/raw-rev/397722cdd7ec
https://www.sudo.ws/repos/sudo/raw-rev/0cd3cc8fa195

Is this applicable to master too, and should be backport it jethro?

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


[OE-core] [PATCH] dropbear/init: Allow extra arguments for key generation

2016-08-09 Thread Mike Looijmans
This patch adds DROPBEAR_RSAKEY_ARGS and DROPBEAR_DSSKEY_ARGS optional
parameters to /etc/default/dropbear. The contents are simply passed to
the 'dropbearkey' program when generating a host key.

The default keysize for RSA is currently 2048 bits. It takes a CortexA9
running at 700MHz between 4 and 10 seconds to calculate a keypair. The
board boots Linux in about a second, but you have to wait for several
seconds because of the keypair generation. This patch allows one to put
the line DROPBEAR_RSAKEY_ARGS="-s 1024" into /etc/default/dropbear, and
have a host key generated in about 0.2 seconds on the same CPU. This is
particulary useful for read-only rootfs systems which generate a key on
each boot.

Signed-off-by: Mike Looijmans 
---

resubmitted: Added the [OE-core] prefix

 meta/recipes-core/dropbear/dropbear/init | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/dropbear/dropbear/init 
b/meta/recipes-core/dropbear/dropbear/init
index e8fed3f..434bd6b 100755
--- a/meta/recipes-core/dropbear/dropbear/init
+++ b/meta/recipes-core/dropbear/dropbear/init
@@ -62,13 +62,13 @@ for t in $DROPBEAR_KEYTYPES; do
 if [ -f "$DROPBEAR_RSAKEY" -a ! -s "$DROPBEAR_RSAKEY" ]; then
 rm $DROPBEAR_RSAKEY || true
 fi
-test -f $DROPBEAR_RSAKEY || dropbearkey -t rsa -f $DROPBEAR_RSAKEY
+test -f $DROPBEAR_RSAKEY || dropbearkey -t rsa -f $DROPBEAR_RSAKEY 
$DROPBEAR_RSAKEY_ARGS
;;
 dsa)
 if [ -f "$DROPBEAR_DSSKEY" -a ! -s "$DROPBEAR_DSSKEY" ]; then
 rm $DROPBEAR_DSSKEY || true
 fi
-test -f $DROPBEAR_DSSKEY || dropbearkey -t dss -f $DROPBEAR_DSSKEY
+test -f $DROPBEAR_DSSKEY || dropbearkey -t dss -f $DROPBEAR_DSSKEY 
$DROPBEAR_DSSKEY_ARGS
;;
   esac
 done
-- 
1.9.1

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


Re: [OE-core] [PATCH 00/20] oeqa: Split configuration from the code

2016-08-09 Thread Richard Purdie
On Mon, 2016-08-08 at 09:22 -0700, Jose Lamego wrote:
> Improve oeqa-selftest capabilities and UX by placing
> test configuration features and variables into a separate
> configuration file.
> Configuration file reading is done through a new library in
> oeqa.utils and specific configuration files for each test are
> stored in a separate directory.
> 
> [Yocto 9389]

Looking at these changes, I'm not 100% convinced this is an improvement
in readability or experience. Right now, I can look at the test case
definition and get a reasonable idea of what its doing. After the
change, I have to read two files at once, side by side in order to have
a clue what the test does. Is that an improvement?

I'm open to persuasion on why this patchset is a good idea but the
pieces I looked at didn't seem to help much :/

Cheers,

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


Re: [OE-core] [PATCH] kernel.bbclass: add lzop dependency

2016-08-09 Thread Mike Looijmans
I've been adding "lzop-native" to a lot of kernel recipes because you also 
need it when generating an LZO compressed kernel (I like my boot time to be 
under 1 second).


It's a small dependency which is hard to automatically figure out, since there 
are a number of ways to trigger it (we just found two). I would welcome this 
patch very much, and I don't think many will suffer...



On 01-08-16 14:32, Trevor Woerner wrote:

If the initramfs image is type lzo, then a native lzop is needed.

Signed-off-by: Trevor Woerner 
---
  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 fa76a14..db42744 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -1,7 +1,7 @@
  inherit linux-kernel-base kernel-module-split

  PROVIDES += "virtual/kernel"
-DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc 
kmod-native depmodwrapper-cross bc-native"
+DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc 
kmod-native depmodwrapper-cross bc-native lzop-native"

  S = "${STAGING_KERNEL_DIR}"
  B = "${WORKDIR}/build"



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


Re: [OE-core] [PATCH][krogoth] sudo: CVE-2015-8239

2016-08-09 Thread Burton, Ross
On 9 August 2016 at 12:04, Sona Sarmadi  wrote:

> Fixes race condition when checking digests in sudoers.
>
> Reference:
> http://seclists.org/oss-sec/2015/q4/327
>
> Reference to upstream fixes:
> https://www.sudo.ws/repos/sudo/raw-rev/397722cdd7ec
> https://www.sudo.ws/repos/sudo/raw-rev/0cd3cc8fa195
>

Is this applicable to master too, and should be backport it jethro?

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


[OE-core] [krogoth] [PATCH 1/2] systemd: Create missing sysusers offline

2016-08-09 Thread Kristian Amlie
From: Khem Raj 

Some system users which are needed by systemd components were missing
create these users knobbed with relevant packageconfig

(From OE-Core rev: d18957925c6c073b7194e3a233efea24e436f74e)

Signed-off-by: Khem Raj 
Signed-off-by: Richard Purdie 
(cherry picked from commit fd36a447d0da53e713d992b17ce86dd31ea63c67)
Signed-off-by: Kristian Amlie 
---
 meta/recipes-core/systemd/systemd_229.bb | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/systemd/systemd_229.bb 
b/meta/recipes-core/systemd/systemd_229.bb
index c23c749..a2d35f8 100644
--- a/meta/recipes-core/systemd/systemd_229.bb
+++ b/meta/recipes-core/systemd/systemd_229.bb
@@ -319,10 +319,16 @@ PACKAGES =+ "\
 SYSTEMD_PACKAGES = "${@bb.utils.contains('PACKAGECONFIG', 'binfmt', 
'${PN}-binfmt', '', d)}"
 SYSTEMD_SERVICE_${PN}-binfmt = "systemd-binfmt.service"
 
-USERADD_PACKAGES = "${PN}"
+USERADD_PACKAGES = "${PN} ${PN}-extra-utils"
 USERADD_PARAM_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'microhttpd', 
'--system -d / -M --shell /bin/nologin systemd-journal-gateway;', '', d)}"
+USERADD_PARAM_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'microhttpd', 
'--system -d / -M --shell /bin/nologin systemd-journal-remote;', '', d)}"
+USERADD_PARAM_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 
'journal-upload', '--system -d / -M --shell /bin/nologin 
systemd-journal-upload;', '', d)}"
 USERADD_PARAM_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'timesyncd', 
'--system -d / -M --shell /bin/nologin systemd-timesync;', '', d)}"
+USERADD_PARAM_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'networkd', 
'--system -d / -M --shell /bin/nologin systemd-network;', '', d)}"
+USERADD_PARAM_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'coredump', 
'--system -d / -M --shell /bin/nologin systemd-coredump;', '', d)}"
+USERADD_PARAM_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'resolved', 
'--system -d / -M --shell /bin/nologin systemd-resolve;', '', d)}"
 GROUPADD_PARAM_${PN} = "-r lock; -r systemd-journal"
+USERADD_PARAM_${PN}-extra-utils += "--system -d / -M --shell /bin/nologin 
systemd-bus-proxy;"
 
 FILES_${PN}-analyze = "${bindir}/systemd-analyze"
 
-- 
2.7.4

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


[OE-core] [krogoth] [PATCH 2/2] systemd: allow add users as a rootfs postprocess cmd

2016-08-09 Thread Kristian Amlie
From: Stephano Cetola 

Adding all the users / groups to systemd is only available for readonly
file systems. This change allows users to add them to read / write file
systems as well by specifying:

ROOTFS_POSTPROCESS_COMMAND += "systemd_create_users"

Also, add "--shell /sbin/nologin" to each user's add params.

[ YOCTO #9497 ]

(From OE-Core rev: 98a4c642444a524f547f5d978a28814d20c12354)

Signed-off-by: Stephano Cetola 
Signed-off-by: Richard Purdie 
(cherry picked from commit 79be110c1fdfd0affe6a310b96e7107c4549d23c)
Signed-off-by: Kristian Amlie 
---
 meta/classes/rootfs-postcommands.bbclass | 43 +++-
 1 file changed, 20 insertions(+), 23 deletions(-)

diff --git a/meta/classes/rootfs-postcommands.bbclass 
b/meta/classes/rootfs-postcommands.bbclass
index 95d28af..db8b551 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -21,7 +21,7 @@ ROOTFS_POSTUNINSTALL_COMMAND =+ "write_image_manifest ; "
 POSTINST_LOGFILE ?= "${localstatedir}/log/postinstall.log"
 # Set default target for systemd images
 SYSTEMD_DEFAULT_TARGET ?= '${@bb.utils.contains("IMAGE_FEATURES", "x11-base", 
"graphical.target", "multi-user.target", d)}'
-ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("DISTRO_FEATURES", 
"systemd", "set_systemd_default_target; ", "", d)}'
+ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("DISTRO_FEATURES", 
"systemd", "set_systemd_default_target; systemd_create_users;", "", d)}'
 
 ROOTFS_POSTPROCESS_COMMAND += 'empty_var_volatile;'
 
@@ -30,7 +30,25 @@ ROOTFS_POSTPROCESS_COMMAND += 'empty_var_volatile;'
 SSH_DISABLE_DNS_LOOKUP ?= " ssh_disable_dns_lookup ; "
 ROOTFS_POSTPROCESS_COMMAND_append_qemuall = "${SSH_DISABLE_DNS_LOOKUP}"
 
-
+systemd_create_users () {
+   for conffile in ${IMAGE_ROOTFS}/usr/lib/sysusers.d/systemd.conf 
${IMAGE_ROOTFS}/usr/lib/sysusers.d/systemd-remote.conf; do
+   [ -e $conffile ] || continue
+   grep -v "^#" $conffile | sed -e '/^$/d' | while read type name 
id comment; do
+   if [ "$type" = "u" ]; then
+   useradd_params="--shell /sbin/nologin"
+   [ "$id" != "-" ] && useradd_params="$useradd_params 
--uid $id"
+   [ "$comment" != "-" ] && 
useradd_params="$useradd_params --comment $comment"
+   useradd_params="$useradd_params --system $name"
+   eval useradd --root ${IMAGE_ROOTFS} $useradd_params || 
true
+   elif [ "$type" = "g" ]; then
+   groupadd_params=""
+   [ "$id" != "-" ] && groupadd_params="$groupadd_params 
--gid $id"
+   groupadd_params="$groupadd_params --system $name"
+   eval groupadd --root ${IMAGE_ROOTFS} $groupadd_params 
|| true
+   fi
+   done
+   done
+}
 
 #
 # A hook function to support read-only-rootfs IMAGE_FEATURES
@@ -73,27 +91,6 @@ read_only_rootfs_hook () {
${IMAGE_ROOTFS}/etc/init.d/populate-volatile.sh
fi
fi
-
-   if ${@bb.utils.contains("DISTRO_FEATURES", "systemd", "true", "false", 
d)}; then
-   # Update user database files so that services don't fail for a 
read-only systemd system
-   for conffile in ${IMAGE_ROOTFS}/usr/lib/sysusers.d/systemd.conf 
${IMAGE_ROOTFS}/usr/lib/sysusers.d/systemd-remote.conf; do
-   [ -e $conffile ] || continue
-   grep -v "^#" $conffile | sed -e '/^$/d' | while read type name 
id comment; do
-   if [ "$type" = "u" ]; then
-   useradd_params=""
-   [ "$id" != "-" ] && useradd_params="$useradd_params 
--uid $id"
-   [ "$comment" != "-" ] && 
useradd_params="$useradd_params --comment $comment"
-   useradd_params="$useradd_params --system $name"
-   eval useradd --root ${IMAGE_ROOTFS} $useradd_params || 
true
-   elif [ "$type" = "g" ]; then
-   groupadd_params=""
-   [ "$id" != "-" ] && groupadd_params="$groupadd_params 
--gid $id"
-   groupadd_params="$groupadd_params --system $name"
-   eval groupadd --root ${IMAGE_ROOTFS} $groupadd_params 
|| true
-   fi
-   done
-   done
-   fi
 }
 
 #
-- 
2.7.4

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


[OE-core] [PATCH] dropbear/init: Allow extra arguments for key generation

2016-08-09 Thread Mike Looijmans
This patch adds DROPBEAR_RSAKEY_ARGS and DROPBEAR_DSSKEY_ARGS optional
parameters to /etc/default/dropbear. The contents are simply passed to
the 'dropbearkey' program when generating a host key.

The default keysize for RSA is currently 2048 bits. It takes a CortexA9
running at 700MHz between 4 and 10 seconds to calculate a keypair. The
board boots Linux in about a second, but you have to wait for several
seconds because of the keypair generation. This patch allows one to put
the line DROPBEAR_RSAKEY_ARGS="-s 1024" into /etc/default/dropbear, and
have a host key generated in about 0.2 seconds on the same CPU. This is
particulary useful for read-only rootfs systems which generate a key on
each boot.

Signed-off-by: Mike Looijmans 
---
 meta/recipes-core/dropbear/dropbear/init | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/dropbear/dropbear/init 
b/meta/recipes-core/dropbear/dropbear/init
index e8fed3f..434bd6b 100755
--- a/meta/recipes-core/dropbear/dropbear/init
+++ b/meta/recipes-core/dropbear/dropbear/init
@@ -62,13 +62,13 @@ for t in $DROPBEAR_KEYTYPES; do
 if [ -f "$DROPBEAR_RSAKEY" -a ! -s "$DROPBEAR_RSAKEY" ]; then
 rm $DROPBEAR_RSAKEY || true
 fi
-test -f $DROPBEAR_RSAKEY || dropbearkey -t rsa -f $DROPBEAR_RSAKEY
+test -f $DROPBEAR_RSAKEY || dropbearkey -t rsa -f $DROPBEAR_RSAKEY 
$DROPBEAR_RSAKEY_ARGS
;;
 dsa)
 if [ -f "$DROPBEAR_DSSKEY" -a ! -s "$DROPBEAR_DSSKEY" ]; then
 rm $DROPBEAR_DSSKEY || true
 fi
-test -f $DROPBEAR_DSSKEY || dropbearkey -t dss -f $DROPBEAR_DSSKEY
+test -f $DROPBEAR_DSSKEY || dropbearkey -t dss -f $DROPBEAR_DSSKEY 
$DROPBEAR_DSSKEY_ARGS
;;
   esac
 done
-- 
1.9.1

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


Re: [OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES configurability

2016-08-09 Thread Tom Hochstein
No, master does not need this fix. The problematic patch 
0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch does not exist 
on master because the underlying issue was addressed upstream (without 
introducing this bug).

Tom

From: Burton, Ross [mailto:ross.bur...@intel.com]
Sent: Tuesday, August 09, 2016 5:10 AM
To: Tom Hochstein 
Cc: OE-core 
Subject: Re: [OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES 
configurability


On 8 August 2016 at 16:45, Tom Hochstein 
> wrote:
The most recent patch 
0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch
incorrectly removed the configuration constructs that allowed the
package to be configured without OpenGL ES support.

Does this also need to be applied to master?

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


[OE-core] [PATCH][krogoth] sudo: CVE-2015-8239

2016-08-09 Thread Sona Sarmadi
Fixes race condition when checking digests in sudoers.

Reference:
http://seclists.org/oss-sec/2015/q4/327

Reference to upstream fixes:
https://www.sudo.ws/repos/sudo/raw-rev/397722cdd7ec
https://www.sudo.ws/repos/sudo/raw-rev/0cd3cc8fa195

Signed-off-by: Sona Sarmadi 
---
 .../sudo/sudo/CVE-2015-8239-1.patch| 699 +
 .../sudo/sudo/CVE-2015-8239-2.patch|  45 ++
 meta/recipes-extended/sudo/sudo_1.8.15.bb  |   2 +
 3 files changed, 746 insertions(+)
 create mode 100644 meta/recipes-extended/sudo/sudo/CVE-2015-8239-1.patch
 create mode 100644 meta/recipes-extended/sudo/sudo/CVE-2015-8239-2.patch

diff --git a/meta/recipes-extended/sudo/sudo/CVE-2015-8239-1.patch 
b/meta/recipes-extended/sudo/sudo/CVE-2015-8239-1.patch
new file mode 100644
index 000..582c0bf
--- /dev/null
+++ b/meta/recipes-extended/sudo/sudo/CVE-2015-8239-1.patch
@@ -0,0 +1,699 @@
+# HG changeset patch
+# User Todd C. Miller 
+# Date 1451928918 25200
+# Node ID 397722cdd7eceef0aec561909418215e275ccd44
+# Parent  33272418bb10ee780184dbd2d966a4e5c3bc597e
+Add support for using fexecve() if supported on commands that are
+checksummed.
+
+Reference to upstream patch:
+https://www.sudo.ws/repos/sudo/rev/397722cdd7ec
+
+CVE: CVE-2015-8239
+Upstream-Status: Backport
+Signed-off-by: Sona Sarmadi 
+---
+diff -ruN a/configure b/configure
+--- a/configure2015-11-01 00:35:24.0 +0100
 b/configure2016-08-08 12:56:03.441681854 +0200
+@@ -2650,6 +2650,7 @@
+ as_fn_append ac_header_list " sys/select.h"
+ as_fn_append ac_header_list " sys/stropts.h"
+ as_fn_append ac_header_list " sys/sysmacros.h"
++as_fn_append ac_func_list " fexecve"
+ as_fn_append ac_func_list " killpg"
+ as_fn_append ac_func_list " nl_langinfo"
+ as_fn_append ac_func_list " strftime"
+@@ -18078,6 +18079,8 @@
+ 
+ 
+ 
++
++
+ for ac_func in getgrouplist
+ do :
+   ac_fn_c_check_func "$LINENO" "getgrouplist" "ac_cv_func_getgrouplist"
+@@ -19903,8 +19906,8 @@
+ fi
+ done
+ 
+-# Check for fexecve, posix_spawn, and posix_spawnp
+-for ac_func in fexecve posix_spawn posix_spawnp
++# Check for posix_spawn, and posix_spawnp
++for ac_func in posix_spawn posix_spawnp
+ do :
+   as_ac_var=`$as_echo "ac_cv_func_$ac_func" | $as_tr_sh`
+ ac_fn_c_check_func "$LINENO" "$ac_func" "$as_ac_var"
+diff -ruN a/configure.ac b/configure.ac
+--- a/configure.ac 2016-08-08 12:55:08.78102 +0200
 b/configure.ac 2016-08-08 12:56:03.445681547 +0200
+@@ -1,7 +1,7 @@
+ dnl
+ dnl Use the top-level autogen.sh script to generate configure and config.h.in
+ dnl
+-dnl Copyright (c) 1994-1996,1998-2015 Todd C. Miller 

++dnl Copyright (c) 1994-1996,1998-2016 Todd C. Miller 

+ dnl
+ AC_PREREQ([2.59])
+ AC_INIT([sudo], [1.8.15], [http://www.sudo.ws/bugs/], [sudo])
+@@ -2384,7 +2384,7 @@
+ dnl Function checks
+ dnl
+ AC_FUNC_GETGROUPS
+-AC_CHECK_FUNCS_ONCE([killpg nl_langinfo strftime pread pwrite openat])
++AC_CHECK_FUNCS_ONCE([fexecve killpg nl_langinfo strftime pread pwrite openat])
+ AC_CHECK_FUNCS([getgrouplist], [], [
+ case "$host_os" in
+ aix*)
+@@ -2676,8 +2676,8 @@
+ if test X"$with_noexec" != X"no"; then
+ # Check for non-standard exec functions
+ AC_CHECK_FUNCS([exect execvP execvpe])
+-# Check for fexecve, posix_spawn, and posix_spawnp
+-AC_CHECK_FUNCS([fexecve posix_spawn posix_spawnp])
++# Check for posix_spawn, and posix_spawnp
++AC_CHECK_FUNCS([posix_spawn posix_spawnp])
+ fi
+ 
+ dnl
+diff -ruN a/doc/sudoers.cat b/doc/sudoers.cat
+--- a/doc/sudoers.cat  2016-08-08 12:55:08.78102 +0200
 b/doc/sudoers.cat  2016-08-08 12:56:03.445681547 +0200
+@@ -410,6 +410,13 @@
+  $ openssl dgst -binary -sha224 /bin/ls | openssl base64
+  EYGH2oNk1JC0p9679IMATo8+BT7JVDCd4sQaJQ==
+ 
++ Warning, if the user has write access to the command itself (directly or
++ via a sudo command), it may be possible for the user to replace the
++ command after the digest check has been performed but before the command
++ is executed.  A similar race condition exists on systems that lack the
++ fexecve(2) system call when the directory in which the command is located
++ is writable by the user.
++
+  Command digests are only supported by version 1.8.7 or higher.
+ 
+DDeeffaauullttss
+diff -ruN a/doc/sudoers.man.in b/doc/sudoers.man.in
+--- a/doc/sudoers.man.in   2016-08-08 12:55:08.78102 +0200
 b/doc/sudoers.man.in   2016-08-08 12:56:03.445681547 +0200
+@@ -1,7 +1,7 @@
+ .\" DO NOT EDIT THIS FILE, IT IS NOT THE MASTER!
+ .\" IT IS GENERATED AUTOMATICALLY FROM sudoers.mdoc.in
+ .\"
+-.\" Copyright (c) 1994-1996, 1998-2005, 2007-2015
++.\" Copyright (c) 1994-1996, 1998-2005, 2007-2016
+ .\"   Todd C. Miller 
+ .\"
+ .\" Permission to use, copy, modify, and distribute this 

Re: [OE-core] [PATCH 1/3] linux-dtb.inc: Support for .dtbo files for dtb overlays

2016-08-09 Thread Andreas Müller
On Tue, Aug 9, 2016 at 12:00 PM, Burton, Ross  wrote:
>
> On 9 August 2016 at 10:22, Andreas Müller 
> wrote:
>>
>> Is there anything wrong with this patch? The original patch was sent
>> end of May, there were pings / a resend but it was never applied. Why
>> I insist: It is necessary for RaspberryPi's open source (VC4) patch
>> series pending for very long time.
>
>
> Just slipped through the net, now in my staging branch for master.
>
> Ross
Thanks
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES configurability

2016-08-09 Thread Burton, Ross
On 8 August 2016 at 16:45, Tom Hochstein  wrote:

> The most recent patch 0011-drop-demos-dependant-on-
> obsolete-MESA_screen_surface.patch
> incorrectly removed the configuration constructs that allowed the
> package to be configured without OpenGL ES support.
>

Does this also need to be applied to master?

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


Re: [OE-core] [krogoth] systemd network service fails to start on boot

2016-08-09 Thread Burton, Ross
On 20 July 2016 at 08:53, Kristian Amlie  wrote:

> on our krogoth build the systemd network service does not start, and it
> seems to be because of missing users, which has already been fixed on
> master. I've cherry-picked the two patches that fixed the problem for
> us. Note that they are not written by me, but by other people on this
> list. I hope they can be applied to krogoth.
>

Can you send the two patches to this list, with the [krogoth] tag in the
subject, and your signed-off-by to confirm that you've tested them?

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


Re: [OE-core] [PATCH 1/3] linux-dtb.inc: Support for .dtbo files for dtb overlays

2016-08-09 Thread Burton, Ross
On 9 August 2016 at 10:22, Andreas Müller 
wrote:

> Is there anything wrong with this patch? The original patch was sent
> end of May, there were pings / a resend but it was never applied. Why
> I insist: It is necessary for RaspberryPi's open source (VC4) patch
> series pending for very long time.
>

Just slipped through the net, now in my staging branch for master.

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


Re: [OE-core] [PATCH] oe-selftest: reorganize syntax to use subcommands

2016-08-09 Thread Burton, Ross
On 2 August 2016 at 21:45, Humberto Ibarra 
wrote:

> The current syntax of oe-selftest is messy. This patch reorganizes it in a
> clearer way by grouping all functions in a 'run' subcommand and a 'list'
> subcommand.
>

This breaks the AB as the commands it invokes don't work anymore.  Is there
a plan to migrate the autobuilder to support both the old and new commands?

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


Re: [OE-core] [PATCH 1/3] linux-dtb.inc: Support for .dtbo files for dtb overlays

2016-08-09 Thread Andreas Müller
On Mon, Aug 8, 2016 at 9:31 AM, Andreas Müller
 wrote:
> On Sun, Jul 31, 2016 at 11:17 PM, Paul Eggleton
>  wrote:
>> On Sat, 30 Jul 2016 08:00:45 Otavio Salvador wrote:
>>> On Fri, Jul 29, 2016 at 7:27 PM, Khem Raj  wrote:
>>> > From: Herve Jourdain 
>>> >
>>> > Signed-off-by: Herve Jourdain 
>>> > Signed-off-by: Khem Raj 
>>> > ---
>>> >
>>> >  meta/recipes-kernel/linux/linux-dtb.inc | 18 +++---
>>> >  1 file changed, 11 insertions(+), 7 deletions(-)
>>> >
>>> > diff --git a/meta/recipes-kernel/linux/linux-dtb.inc
>>> > b/meta/recipes-kernel/linux/linux-dtb.inc index 74f5ef8..8528d64 100644
>>> > --- a/meta/recipes-kernel/linux/linux-dtb.inc
>>> > +++ b/meta/recipes-kernel/linux/linux-dtb.inc
>>> > @@ -33,12 +33,13 @@ do_compile_append() {
>>> >
>>> >  do_install_append() {
>>> >
>>> > for DTB in ${KERNEL_DEVICETREE}; do
>>> >
>>> > DTB=`normalize_dtb "${DTB}"`
>>> >
>>> > -   DTB_BASE_NAME=`basename ${DTB} .dtb`
>>> > +   DTB_EXT=${DTB##*.}
>>>
>>> Bashism, no?
>>
>> No:
>>
>>   http://pubs.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html
>>
> Anything wrong with this patch. The orinal patch was sent end of May!
>
Should read before sending... What I meant:

Is there anything wrong with this patch? The original patch was sent
end of May, there were pings / a resend but it was never applied. Why
I insist: It is necessary for RaspberryPi's open source (VC4) patch
series pending for very long time.

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


Re: [OE-core] [PATCH 20/20] oeqa.utils.readconfig: Read self-test configuration file

2016-08-09 Thread Joshua G Lock
On Mon, 2016-08-08 at 09:23 -0700, Jose Lamego wrote:
> Improve oeqa-selftest capabilities and UX by placing
> test configuration features and variables into a separate
> configuration file.
> This library reads a configuration file located at the
> oeqa/selftest/conf directory with same name as the caller
> file and the conf extension.

This patch to add the oeqa.utils.readconfig module should really be
applied before all of the changes which make use it. It's a relatively
minor niggle, but this should really be 1/20.

> 
> [Yocto 9389]
> 
> Signed-off-by: Jose Lamego 
> ---
>  meta/lib/oeqa/utils/readconfig.py | 18 ++
>  1 file changed, 18 insertions(+)
>  create mode 100644 meta/lib/oeqa/utils/readconfig.py
> 
> diff --git a/meta/lib/oeqa/utils/readconfig.py
> b/meta/lib/oeqa/utils/readconfig.py
> new file mode 100644
> index 000..1534ce7
> --- /dev/null
> +++ b/meta/lib/oeqa/utils/readconfig.py
> @@ -0,0 +1,18 @@
> +import os
> +import re
> +import configparser
> +
> +
> +def conffile(codefile):
> +# Get test configurations from configuration file
> +f = re.compile(r'(pyc?$)')
> +fn = os.path.basename(codefile)
> +configfile = os.path.join(os.path.dirname(codefile), 'conf',
> +  f.sub('conf', fn))

This seems like it would be simpler with os.path.splitext()[1] ?

> +config = configparser.SafeConfigParser()

Since Python 3.2 that's just ConfigParser()[2]. As we require Python
3.4 or newer for BitBake you can just use configparser.ConfigParser()
here.

> +read_result = config.read(configfile)
> +if read_result == []:
> +raise configparser.Error("Failed to get test configuration
> from \
> +\"%s\". Verify that file exists and format is correct."
> + % configfile)
> +return config
> -- 
> 1.8.3.1
> 

Regards,

Joshua

1. https://docs.python.org/3/library/os.path.html#os.path.splitext
2. https://docs.python.org/3/whatsnew/3.2.html
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [krogoth] systemd network service fails to start on boot

2016-08-09 Thread Kristian Amlie
Ping!

Is there no interest in fixing network services under systemd?

-- 
Kristian

On 20/07/16 09:53, Kristian Amlie wrote:
> Hey,
> 
> on our krogoth build the systemd network service does not start, and it
> seems to be because of missing users, which has already been fixed on
> master. I've cherry-picked the two patches that fixed the problem for
> us. Note that they are not written by me, but by other people on this
> list. I hope they can be applied to krogoth.
> 
> Regards
> Kristian Amlie
> 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/2] webkitgtk: Disable gold for mips/mips64

2016-08-09 Thread Khem Raj

> On Aug 9, 2016, at 1:29 AM, André Draszik  wrote:
> 
> On Mon, 2016-08-08 at 15:51 -0700, Khem Raj wrote:
>> with binutils 2.27, mips has got the gold support
>> but it doesnt work for webkitgtk _yet_ therefore
>> disable it for now.
>> 
>> Signed-off-by: Khem Raj 
>> ---
>>  meta/recipes-sato/webkit/webkitgtk_2.12.3.bb | 2 ++
>>  1 file changed, 2 insertions(+)
>> 
>> diff --git a/meta/recipes-sato/webkit/webkitgtk_2.12.3.bb b/meta/recipes-
>> sato/webkit/webkitgtk_2.12.3.bb
>> index f2ba8b6..d9f1c30 100644
>> --- a/meta/recipes-sato/webkit/webkitgtk_2.12.3.bb
>> +++ b/meta/recipes-sato/webkit/webkitgtk_2.12.3.bb
>> @@ -75,6 +75,8 @@ EXTRA_OECMAKE_append_armv4 = " -DENABLE_JIT=OFF "
>>  # binutils 2.25.1 has a bug on aarch64:
>>  # https://sourceware.org/bugzilla/show_bug.cgi?id=18430
>>  EXTRA_OECMAKE_append_aarch64 = " -DUSE_LD_GOLD=OFF "
>> +EXTRA_OECMAKE_append_mips = " -DUSE_LD_GOLD=OFF "
>> +EXTRA_OECMAKE_append_mips64 = " -DUSE_LD_GOLD=OFF "
> 
> What about mipsel and mips64el? Should need the same I would think…

yes, I would think so. I do not test the el parts.


signature.asc
Description: Message signed with OpenPGP using GPGMail
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] xmlto: adds libxslt to DEPENDS

2016-08-09 Thread Liu Ming


> 在 2016年8月9日,上午9:58,Robert Yang  写道:
> 
> Hi Ming,
> 
> I sent a similar patch yesterday:
> 
> [OE-core] [PATCH 0/1] xmlto: fix depends for native
> 
> http://cgit.openembedded.org/openembedded-core-contrib/commit/?h=rbt/xml=0b7f903f8c74204bb48cc74ce1a6957534ac9ede

Arha, sorry, I have not checked OE mails(for a long time), so we can ignore my 
commit.

//Ming Liu
> 
> // Robert
> 
>> On 08/09/2016 03:53 PM, Ming Liu wrote:
>> From: Ming Liu 
>> 
>> The native libxslt-bin is needed by xmlto-native, but it's not in either
>> DEPENDS or RDEPENDS of native xmlto.
>> 
>> This fixes a following error per test:
>> | xmlto: Can't continue, xsltproc tool not found or not executable.
>> | make[2]: *** 
>> [/mnt/OE/build/tmp-glibc/work/qemux86-oe-linux/perf/1.0-r9/perf-1.0/perf-annotate.1]
>>  Error 3
>> | make[2]: *** Waiting for unfinished jobs
>> 
>> Signed-off-by: Ming Liu 
>> ---
>>  meta/recipes-devtools/xmlto/xmlto_0.0.28.bb | 3 +++
>>  1 file changed, 3 insertions(+)
>> 
>> diff --git a/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb 
>> b/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
>> index 886e7b4..cc4bccd 100644
>> --- a/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
>> +++ b/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
>> @@ -14,6 +14,9 @@ SRC_URI[sha256sum] = 
>> "2f986b7c9a0e9ac6728147668e776d405465284e13c74d4146c9cbc51f
>> 
>>  inherit autotools
>> 
>> +# xmlto-native needs libxslt-native
>> +DEPENDS = "libxslt"
>> +
>>  # xmlto needs getopt/xmllint/xsltproc/bash/tail at runtime
>>  RDEPENDS_${PN} = "docbook-xml-dtd4 \
>>docbook-xsl-stylesheets \
>> 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] systemd: fix useradd parameter generation

2016-08-09 Thread Enrico Joerns

On 08/09/2016 10:24 AM, Enrico Joerns wrote:

On 08/09/2016 07:17 AM, Anders Darander wrote:

* Enrico Jorns  [160808 15:55]:

Constructing the USERADD_PARAM_${PN} value with += (as done before)
resulted in having a whitespace added for each entry not having its
PACKAGECONFIG parameter activated.



This is not only a cosmetical issue. The useradd class concatenates all
useradd commands from all subpackages of a package by doing a
";".join(). When having a useradd command that ends with ";" and some
extra spaces, concatenation will result in an empty useradd argument
("commandA;;commandB"). The useradd command will be called with this
and make the Yocto build break by returning an error code.



Fortunately, there is no problem when having a "commandA;;commandB" in
the useradd string, this will be ignored. Thus using "_append"
instead of
"+=" is sufficient here.


NAK

Fix useradd.bbclass instead. Otherwise, the same issue will happen with
other recipes. Having a single variable, that you're not allowed to use
the normal way to add to, would be more confusing. Personally, I would
find it hard to rember that I need to use .= for USERADD_PARAM...


Yes, I did not have .= in mind when creating this solution, but this
would be the better approach.
Nevertheless, I also prefer a more general solution, but was not sure
about how it could look like. Maybe I've found one now.

So, forget about this patch, I will post a useradd class patch in a
separate thread instead.


Just saw yocto patch 553fa35147adb996d515c3d27f62555d9c969cd8 (oe-core: 
e8d4356c38e3c2aacd6dc49231c73bcb7d597308) seems to address and fix this 
problem. Did not find that before... :-|


--
Pengutronix e.K.   | Enrico Jörns|
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-5080 |
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

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


Re: [OE-core] [PATCH 1/2] webkitgtk: Disable gold for mips/mips64

2016-08-09 Thread André Draszik
On Mon, 2016-08-08 at 15:51 -0700, Khem Raj wrote:
> with binutils 2.27, mips has got the gold support
> but it doesnt work for webkitgtk _yet_ therefore
> disable it for now.
> 
> Signed-off-by: Khem Raj 
> ---
>  meta/recipes-sato/webkit/webkitgtk_2.12.3.bb | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta/recipes-sato/webkit/webkitgtk_2.12.3.bb b/meta/recipes-
> sato/webkit/webkitgtk_2.12.3.bb
> index f2ba8b6..d9f1c30 100644
> --- a/meta/recipes-sato/webkit/webkitgtk_2.12.3.bb
> +++ b/meta/recipes-sato/webkit/webkitgtk_2.12.3.bb
> @@ -75,6 +75,8 @@ EXTRA_OECMAKE_append_armv4 = " -DENABLE_JIT=OFF "
>  # binutils 2.25.1 has a bug on aarch64:
>  # https://sourceware.org/bugzilla/show_bug.cgi?id=18430
>  EXTRA_OECMAKE_append_aarch64 = " -DUSE_LD_GOLD=OFF "
> +EXTRA_OECMAKE_append_mips = " -DUSE_LD_GOLD=OFF "
> +EXTRA_OECMAKE_append_mips64 = " -DUSE_LD_GOLD=OFF "

What about mipsel and mips64el? Should need the same I would think...

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


Re: [OE-core] [PATCH] systemd: fix useradd parameter generation

2016-08-09 Thread Enrico Joerns

On 08/09/2016 07:17 AM, Anders Darander wrote:

* Enrico Jorns  [160808 15:55]:

Constructing the USERADD_PARAM_${PN} value with += (as done before)
resulted in having a whitespace added for each entry not having its
PACKAGECONFIG parameter activated.



This is not only a cosmetical issue. The useradd class concatenates all
useradd commands from all subpackages of a package by doing a
";".join(). When having a useradd command that ends with ";" and some
extra spaces, concatenation will result in an empty useradd argument
("commandA;;commandB"). The useradd command will be called with this
and make the Yocto build break by returning an error code.



Fortunately, there is no problem when having a "commandA;;commandB" in
the useradd string, this will be ignored. Thus using "_append" instead of
"+=" is sufficient here.


NAK

Fix useradd.bbclass instead. Otherwise, the same issue will happen with
other recipes. Having a single variable, that you're not allowed to use
the normal way to add to, would be more confusing. Personally, I would
find it hard to rember that I need to use .= for USERADD_PARAM...


Yes, I did not have .= in mind when creating this solution, but this 
would be the better approach.
Nevertheless, I also prefer a more general solution, but was not sure 
about how it could look like. Maybe I've found one now.


So, forget about this patch, I will post a useradd class patch in a 
separate thread instead.


Regards, Enrico

--
Pengutronix e.K.   | Enrico Jörns|
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-5080 |
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

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


Re: [OE-core] [PATCH 19/20] oeqa.selftest.wic: Split configuration from code

2016-08-09 Thread Ed Bartosh
Hi Jose,

On Mon, Aug 08, 2016 at 09:23:07AM -0700, Jose Lamego wrote:
> Improve oeqa-selftest capabilities and UX by placing
> test configuration features and variables into a separate
> configuration file.
> 

Frankly I have mixed feelings about these changes. From the first look
splitting configuration and code is a good idea. However, looking at the
result I must say I feel like the test became less readable and harder
to understand and maintain.

May be the changes make sense for other tests, but for this one they make
my life as a maintainer of this one less easy I'd say.

> [Yocto 9389]
> 
> Signed-off-by: Jose Lamego 
> ---
>  meta/lib/oeqa/selftest/conf/wic.conf |  10 +++
>  meta/lib/oeqa/selftest/wic.py| 150 
> ---
>  2 files changed, 98 insertions(+), 62 deletions(-)
>  create mode 100644 meta/lib/oeqa/selftest/conf/wic.conf
> 
> diff --git a/meta/lib/oeqa/selftest/conf/wic.conf 
> b/meta/lib/oeqa/selftest/conf/wic.conf
> new file mode 100644
> index 000..e2afb8d
> --- /dev/null
> +++ b/meta/lib/oeqa/selftest/conf/wic.conf
> @@ -0,0 +1,10 @@
> +[Wic]
> +setUpClass_image1 = core-image-minimal
> +setUpClass_image2 = minimal
setUpClass_image2 is not used in the code.

> +setUpClass_image3 = qemux86-directdisk
> +setUpClass_image4 = directdisk
> +setUpClass_image5 = wic-image-minimal
setUpClass_image1 is an image produced by bitbake, but setUpClass_image3
is a name of wic image. They're not the same. Bitbake image is an input
for wic and wic image is an output. setUpClass_image5 is again the name
of the bitbake target, but it differs from _image3. Naming doesn't reflect this 
fact.


> +setUpClass_features = IMAGE_FSTYPES += " hddimg"
> +  MACHINE_FEATURES_append = " efi"
> +setUpClass_recipes = syslinux syslinux-native parted-native gptfdisk-native 
> dosfstools-native mtools-native bmap-tools-native
This list is used only once in the test. Moving it to config file makes
it longer to understand what's in the list.

> diff --git a/meta/lib/oeqa/selftest/wic.py b/meta/lib/oeqa/selftest/wic.py
> index e550785..dc81457 100644
> --- a/meta/lib/oeqa/selftest/wic.py
> +++ b/meta/lib/oeqa/selftest/wic.py
> @@ -31,6 +31,7 @@ from shutil import rmtree
>  from oeqa.selftest.base import oeSelfTest
>  from oeqa.utils.commands import runCmd, bitbake, get_bb_var, runqemu
>  from oeqa.utils.decorators import testcase
> +from oeqa.utils.readconfig import conffile
>  
>  
>  class Wic(oeSelfTest):
> @@ -39,6 +40,18 @@ class Wic(oeSelfTest):
>  resultdir = "/var/tmp/wic/build/"
>  image_is_ready = False
>  
> +@classmethod
> +def setUpClass(cls):
> +# Get test configurations from configuration file
> +cls.config = conffile(__file__)
> +cls.image1 = cls.config.get('Wic', 'setUpClass_image1')
> +cls.image2 = cls.config.get('Wic', 'setUpClass_image2')
> +cls.image3 = cls.config.get('Wic', 'setUpClass_image3')
> +cls.image4 = cls.config.get('Wic', 'setUpClass_image4')
> +cls.image5 = cls.config.get('Wic', 'setUpClass_image5')
> +cls.features = cls.config.get('Wic', 'setUpClass_features')
> +cls.recipes = cls.config.get('Wic', 'setUpClass_recipes')
This looks like a duplication of code and configuration and also hides real
image names and features making test much less readable.

> +
>  def setUpLocal(self):
>  """This code is executed before each test method."""
>  self.write_config('IMAGE_FSTYPES += " hddimg"\n'
> @@ -48,9 +61,8 @@ class Wic(oeSelfTest):
>  # clean up which can result in the native tools built earlier in
>  # setUpClass being unavailable.
>  if not Wic.image_is_ready:
> -bitbake('syslinux syslinux-native parted-native gptfdisk-native '
> -'dosfstools-native mtools-native bmap-tools-native')
> -bitbake('core-image-minimal')
> +bitbake(self.recipes)
> +bitbake(self.image1)
I'd really prefer to see list of dependencies and the image name right
in the test code without having to look into configuration file.

The same is true for pretty much all the changes.

>  Wic.image_is_ready = True
>  
>  rmtree(self.resultdir, ignore_errors=True)
> @@ -73,30 +85,35 @@ class Wic(oeSelfTest):
>  @testcase(1211)
>  def test_build_image_name(self):
>  """Test wic create directdisk --image-name core-image-minimal"""
> -self.assertEqual(0, runCmd("wic create directdisk "
> -   "--image-name core-image-minimal").status)
> -self.assertEqual(1, len(glob(self.resultdir + 
> "directdisk-*.direct")))
> +self.assertEqual(
> +0, runCmd("wic create %s --image-name %s"
> +  % (self.image4, self.image1)).status)
> +self.assertEqual(
> +1, len(glob(self.resultdir + "%s-*.direct" % self.image4)))
>  
>   

[OE-core] [PATCH] Use PYTHON_SITEPACKAGES_DIR insted of hard-coded *site-packages*

2016-08-09 Thread Ming Liu
From: Ming Liu 

For thoese recipes that are inheriting python*-dir.bbclass, there is
already a PYTHON_SITEPACKAGES_DIR present, use that definition replacing
redundant "${libdir}/python*/site-packages".

Signed-off-by: Ming Liu 
---
 .../bootchart2/bootchart2_0.14.8.bb|  2 +-
 meta/recipes-devtools/python/python-setuptools.inc |  2 +-
 .../python/python-setuptools_22.0.5.bb |  2 +-
 meta/recipes-devtools/python/python-smartpm_git.bb | 36 +++---
 meta/recipes-devtools/python/python3-pip_8.1.2.bb  |  8 ++---
 .../python/python3-setuptools_22.0.5.bb|  2 +-
 meta/recipes-devtools/rpm/rpm_5.4.16.bb| 18 +--
 meta/recipes-kernel/lttng/lttng-tools_git.bb   | 10 +++---
 meta/recipes-kernel/perf/perf.bb   |  2 +-
 9 files changed, 41 insertions(+), 41 deletions(-)

diff --git a/meta/recipes-devtools/bootchart2/bootchart2_0.14.8.bb 
b/meta/recipes-devtools/bootchart2/bootchart2_0.14.8.bb
index d0555c3..d039b1b 100644
--- a/meta/recipes-devtools/bootchart2/bootchart2_0.14.8.bb
+++ b/meta/recipes-devtools/bootchart2/bootchart2_0.14.8.bb
@@ -134,7 +134,7 @@ do_install () {
 }
 
 PACKAGES =+ "pybootchartgui"
-FILES_pybootchartgui += "${libdir}/python*/site-packages/pybootchartgui 
${bindir}/pybootchartgui"
+FILES_pybootchartgui += "${PYTHON_SITEPACKAGES_DIR}/pybootchartgui 
${bindir}/pybootchartgui"
 RDEPENDS_pybootchartgui = "python3-pycairo python3-compression python3-image 
python3-textutils python3-shell python3-compression python3-codecs"
 RDEPENDS_${PN}_class-target += "${@bb.utils.contains('DISTRO_FEATURES', 
'sysvinit', 'sysvinit-pidof', 'procps', d)}"
 RDEPENDS_${PN}_class-target += "lsb"
diff --git a/meta/recipes-devtools/python/python-setuptools.inc 
b/meta/recipes-devtools/python/python-setuptools.inc
index 53a514b..92ca9a0 100644
--- a/meta/recipes-devtools/python/python-setuptools.inc
+++ b/meta/recipes-devtools/python/python-setuptools.inc
@@ -17,5 +17,5 @@ UPSTREAM_CHECK_URI = "https://pypi.python.org/pypi/setuptools;
 S = "${WORKDIR}/${SRCNAME}-${PV}"
 
 do_install_prepend() {
-install -d ${D}/${libdir}/${PYTHON_DIR}/site-packages
+install -d ${D}${PYTHON_SITEPACKAGES_DIR}
 }
diff --git a/meta/recipes-devtools/python/python-setuptools_22.0.5.bb 
b/meta/recipes-devtools/python/python-setuptools_22.0.5.bb
index c172039..526474c 100644
--- a/meta/recipes-devtools/python/python-setuptools_22.0.5.bb
+++ b/meta/recipes-devtools/python/python-setuptools_22.0.5.bb
@@ -7,7 +7,7 @@ DEPENDS_class-native += "python-native"
 
 inherit distutils
 
-DISTUTILS_INSTALL_ARGS += 
"--install-lib=${D}${libdir}/${PYTHON_DIR}/site-packages"
+DISTUTILS_INSTALL_ARGS += "--install-lib=${D}${PYTHON_SITEPACKAGES_DIR}"
 
 RDEPENDS_${PN} = "\
   python-stringold \
diff --git a/meta/recipes-devtools/python/python-smartpm_git.bb 
b/meta/recipes-devtools/python/python-smartpm_git.bb
index 81e45b7..0f0cfaa 100644
--- a/meta/recipes-devtools/python/python-smartpm_git.bb
+++ b/meta/recipes-devtools/python/python-smartpm_git.bb
@@ -55,41 +55,41 @@ inherit distutils
 
 do_install_append() {
# We don't support the following items
-   rm -rf ${D}${libdir}/python*/site-packages/smart/backends/slack
-   rm -rf ${D}${libdir}/python*/site-packages/smart/backends/arch
-   rm -rf ${D}${libdir}/python*/site-packages/smart/interfaces/qt
+   rm -rf ${D}${PYTHON_SITEPACKAGES_DIR}/smart/backends/slack
+   rm -rf ${D}${PYTHON_SITEPACKAGES_DIR}/smart/backends/arch
+   rm -rf ${D}${PYTHON_SITEPACKAGES_DIR}/smart/interfaces/qt
 
# Temporary, debian support in OE is missing the python module
-   rm -f ${D}${libdir}/python*/site-packages/smart/plugins/aptchannelsync.py*
-   rm -f ${D}${libdir}/python*/site-packages/smart/plugins/debdir.py*
-   rm -rf ${D}${libdir}/python*/site-packages/smart/backends/deb
+   rm -f ${D}${PYTHON_SITEPACKAGES_DIR}/smart/plugins/aptchannelsync.py*
+   rm -f ${D}${PYTHON_SITEPACKAGES_DIR}/smart/plugins/debdir.py*
+   rm -rf ${D}${PYTHON_SITEPACKAGES_DIR}/smart/backends/deb
 
# Disable automatic channel detection
-   rm -f ${D}${libdir}/python*/site-packages/smart/plugins/detectsys.py*
+   rm -f ${D}${PYTHON_SITEPACKAGES_DIR}/smart/plugins/detectsys.py*
 
# Disable landscape support
-   rm -f ${D}${libdir}/python*/site-packages/smart/plugins/landscape.py*
+   rm -f ${D}${PYTHON_SITEPACKAGES_DIR}/smart/plugins/landscape.py*
 
# Disable urpmi channel support
-   rm -f ${D}${libdir}/python*/site-packages/smart/plugins/urpmichannelsync.py*
+   rm -f ${D}${PYTHON_SITEPACKAGES_DIR}/smart/plugins/urpmichannelsync.py*
 
# Disable yum channel support
-   rm -f ${D}${libdir}/python*/site-packages/smart/plugins/yumchannelsync.py*
+   rm -f ${D}${PYTHON_SITEPACKAGES_DIR}/smart/plugins/yumchannelsync.py*
 
# Disable zypper channel support
-   rm -f ${D}${libdir}/python*/site-packages/smart/plugins/zyppchannelsync.py*
+   rm -f 

Re: [OE-core] [PATCH] xmlto: adds libxslt to DEPENDS

2016-08-09 Thread Robert Yang

Hi Ming,

I sent a similar patch yesterday:

[OE-core] [PATCH 0/1] xmlto: fix depends for native

http://cgit.openembedded.org/openembedded-core-contrib/commit/?h=rbt/xml=0b7f903f8c74204bb48cc74ce1a6957534ac9ede

// Robert

On 08/09/2016 03:53 PM, Ming Liu wrote:

From: Ming Liu 

The native libxslt-bin is needed by xmlto-native, but it's not in either
DEPENDS or RDEPENDS of native xmlto.

This fixes a following error per test:
| xmlto: Can't continue, xsltproc tool not found or not executable.
| make[2]: *** 
[/mnt/OE/build/tmp-glibc/work/qemux86-oe-linux/perf/1.0-r9/perf-1.0/perf-annotate.1]
 Error 3
| make[2]: *** Waiting for unfinished jobs

Signed-off-by: Ming Liu 
---
  meta/recipes-devtools/xmlto/xmlto_0.0.28.bb | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb 
b/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
index 886e7b4..cc4bccd 100644
--- a/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
+++ b/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
@@ -14,6 +14,9 @@ SRC_URI[sha256sum] = 
"2f986b7c9a0e9ac6728147668e776d405465284e13c74d4146c9cbc51f

  inherit autotools

+# xmlto-native needs libxslt-native
+DEPENDS = "libxslt"
+
  # xmlto needs getopt/xmllint/xsltproc/bash/tail at runtime
  RDEPENDS_${PN} = "docbook-xml-dtd4 \
docbook-xsl-stylesheets \


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


[OE-core] [PATCH] xmlto: adds libxslt to DEPENDS

2016-08-09 Thread Ming Liu
From: Ming Liu 

The native libxslt-bin is needed by xmlto-native, but it's not in either
DEPENDS or RDEPENDS of native xmlto.

This fixes a following error per test:
| xmlto: Can't continue, xsltproc tool not found or not executable.
| make[2]: *** 
[/mnt/OE/build/tmp-glibc/work/qemux86-oe-linux/perf/1.0-r9/perf-1.0/perf-annotate.1]
 Error 3
| make[2]: *** Waiting for unfinished jobs

Signed-off-by: Ming Liu 
---
 meta/recipes-devtools/xmlto/xmlto_0.0.28.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb 
b/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
index 886e7b4..cc4bccd 100644
--- a/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
+++ b/meta/recipes-devtools/xmlto/xmlto_0.0.28.bb
@@ -14,6 +14,9 @@ SRC_URI[sha256sum] = 
"2f986b7c9a0e9ac6728147668e776d405465284e13c74d4146c9cbc51f
 
 inherit autotools
 
+# xmlto-native needs libxslt-native
+DEPENDS = "libxslt"
+
 # xmlto needs getopt/xmllint/xsltproc/bash/tail at runtime
 RDEPENDS_${PN} = "docbook-xml-dtd4 \
   docbook-xsl-stylesheets \
-- 
1.9.1

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


[OE-core] [krogoth][PATCH] mesa-demos: Fix OpenGL ES configurability

2016-08-09 Thread Tom Hochstein
The most recent patch 
0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch
incorrectly removed the configuration constructs that allowed the
package to be configured without OpenGL ES support.

Signed-off-by: Tom Hochstein 
---
 .../mesa-demos/0012-Fix-gles-configurability.patch | 39 ++
 meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb |  1 +
 2 files changed, 40 insertions(+)
 create mode 100644 
meta/recipes-graphics/mesa/mesa-demos/0012-Fix-gles-configurability.patch

diff --git 
a/meta/recipes-graphics/mesa/mesa-demos/0012-Fix-gles-configurability.patch 
b/meta/recipes-graphics/mesa/mesa-demos/0012-Fix-gles-configurability.patch
new file mode 100644
index 000..0b437a4
--- /dev/null
+++ b/meta/recipes-graphics/mesa/mesa-demos/0012-Fix-gles-configurability.patch
@@ -0,0 +1,39 @@
+Index: mesa-demos-8.2.0/src/egl/opengles1/Makefile.am
+===
+--- mesa-demos-8.2.0.orig/src/egl/opengles1/Makefile.am2016-07-29 
10:52:50.131729547 -0500
 mesa-demos-8.2.0/src/egl/opengles1/Makefile.am 2016-07-29 
10:59:26.0 -0500
+@@ -36,6 +36,8 @@
+   $(EGL_LIBS) \
+   -lm
+ 
++if HAVE_EGL
++if HAVE_GLESV1
+ bin_PROGRAMS =
+ 
+ if HAVE_X11
+@@ -76,3 +78,5 @@
+ torus_x11_LDADD = ../eglut/libeglut_x11.la
+ tri_x11_LDADD = ../eglut/libeglut_x11.la
+ endif
++endif
++endif
+Index: mesa-demos-8.2.0/src/egl/opengles2/Makefile.am
+===
+--- mesa-demos-8.2.0.orig/src/egl/opengles2/Makefile.am2016-07-29 
10:52:50.131729547 -0500
 mesa-demos-8.2.0/src/egl/opengles2/Makefile.am 2016-07-29 
10:59:03.0 -0500
+@@ -33,6 +33,8 @@
+   $(EGL_LIBS) \
+   -lm
+ 
++if HAVE_EGL
++if HAVE_GLESV2
+ bin_PROGRAMS =
+ 
+ if HAVE_WAYLAND
+@@ -53,4 +55,5 @@
+ es2gears_x11_LDADD = ../eglut/libeglut_x11.la
+ es2tri_LDADD = $(X11_LIBS)
+ endif
+-
++endif
++endif
diff --git a/meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb 
b/meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb
index 2df9bba..8e12d82 100644
--- a/meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb
+++ b/meta/recipes-graphics/mesa/mesa-demos_8.2.0.bb
@@ -21,6 +21,7 @@ SRC_URI = 
"ftp://ftp.freedesktop.org/pub/mesa/demos/${PV}/${BPN}-${PV}.tar.bz2 \
 file://0009-glsl-perf-Install-.glsl-.vert-.frag-files.patch \
 file://0010-sharedtex_mt-fix-rendering-thread-hang.patch \
 file://0011-drop-demos-dependant-on-obsolete-MESA_screen_surface.patch \
+file://0012-Fix-gles-configurability.patch \
 "
 SRC_URI[md5sum] = "72613a2c8c013716db02e3ff59d29061"
 SRC_URI[sha256sum] = 
"e4bfecb5816ddd4b7b37c1bc876b63f1f7f06fda5879221a9774d0952f90ba92"
-- 
1.9.1

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