[OE-core] [PATCH 1/1] ltp: Reduce local Pending patches

2017-07-03 Thread Dengke Du
Rebase the patch on latest ltp upstream:

0007-fix-__WORDSIZE-undeclared-when-building-with-musl.patch

[Yocto #11548]

Signed-off-by: Dengke Du 
---
 ...TH_MAX-undeclared-when-building-with-musl.patch | 15 +++---
 ...RDSIZE-undeclared-when-building-with-musl.patch | 23 +++---
 ...ition-of-struct-msgbuf-error-building-wit.patch |  9 +
 3 files changed, 25 insertions(+), 22 deletions(-)

diff --git 
a/meta/recipes-extended/ltp/ltp/0006-fix-PATH_MAX-undeclared-when-building-with-musl.patch
 
b/meta/recipes-extended/ltp/ltp/0006-fix-PATH_MAX-undeclared-when-building-with-musl.patch
index 8874b95..020ddfe 100644
--- 
a/meta/recipes-extended/ltp/ltp/0006-fix-PATH_MAX-undeclared-when-building-with-musl.patch
+++ 
b/meta/recipes-extended/ltp/ltp/0006-fix-PATH_MAX-undeclared-when-building-with-musl.patch
@@ -1,22 +1,23 @@
-From 32644bde4d33b677614534ec37030e57883b8e15 Mon Sep 17 00:00:00 2001
+From b906ec2fe4711a727e198cd9259287c042eef8e2 Mon Sep 17 00:00:00 2001
 From: Dengke Du 
 Date: Thu, 9 Feb 2017 16:41:12 +0800
-Subject: [PATCH 1/3] fix PATH_MAX undeclared when building with musl
+Subject: [PATCH] fix PATH_MAX undeclared when building with musl
 
 fix PATH_MAX undeclared when building with musl.
 
+Upstream-Status: Submitted [https://github.com/linux-test-project/ltp/pull/176]
+
 Signed-off-by: Dengke Du 
-Upstream-Status: Pending
 ---
  include/tst_test.h | 3 +++
  1 file changed, 3 insertions(+)
 
 diff --git a/include/tst_test.h b/include/tst_test.h
-index 7ff33b2..9779c0e 100644
+index 335c82684..4dfa1511b 100644
 --- a/include/tst_test.h
 +++ b/include/tst_test.h
-@@ -19,6 +19,9 @@
- #define TST_TEST_H__
+@@ -23,6 +23,9 @@
+ #endif /* __TEST_H__ */
  
  #include 
 +#ifndef __GLIBC__
@@ -26,5 +27,5 @@ index 7ff33b2..9779c0e 100644
  #include "tst_common.h"
  #include "tst_res_flags.h"
 -- 
-2.7.4
+2.11.0
 
diff --git 
a/meta/recipes-extended/ltp/ltp/0007-fix-__WORDSIZE-undeclared-when-building-with-musl.patch
 
b/meta/recipes-extended/ltp/ltp/0007-fix-__WORDSIZE-undeclared-when-building-with-musl.patch
index 8d0e739..2f4ca63 100644
--- 
a/meta/recipes-extended/ltp/ltp/0007-fix-__WORDSIZE-undeclared-when-building-with-musl.patch
+++ 
b/meta/recipes-extended/ltp/ltp/0007-fix-__WORDSIZE-undeclared-when-building-with-musl.patch
@@ -1,30 +1,31 @@
-From a2639bd0f0d3f9f3049ee33e6710fed06225f54f Mon Sep 17 00:00:00 2001
+From d1a27570457fb6e1d6bafe81bfa0f3507b137e32 Mon Sep 17 00:00:00 2001
 From: Dengke Du 
 Date: Thu, 9 Feb 2017 18:20:58 +0800
-Subject: [PATCH 1/2] fix __WORDSIZE undeclared when building with musl
+Subject: [PATCH] fix __WORDSIZE undeclared when building with musl
 
 fix __WORDSIZE undeclared when building with musl.
 
+Upstream-Status: Submitted [https://github.com/linux-test-project/ltp/pull/177]
+
 Signed-off-by: Dengke Du 
-Upstream-Status: Pending
 ---
  include/old/test.h | 3 +++
  1 file changed, 3 insertions(+)
 
 diff --git a/include/old/test.h b/include/old/test.h
-index d492560..263e92e 100644
+index b36764d83..cc6f1b551 100644
 --- a/include/old/test.h
 +++ b/include/old/test.h
-@@ -58,6 +58,9 @@
- #include "tst_clone.h"
- #include "old_device.h"
- #include "old_tmpdir.h"
+@@ -44,6 +44,9 @@
+ #include 
+ #include 
+ #include 
 +#ifndef __GLIBC__
 +#include 
 +#endif
  
- /*
-  * Ensure that NUMSIGS is defined.
+ #include "usctest.h"
+ 
 -- 
-2.7.4
+2.11.0
 
diff --git 
a/meta/recipes-extended/ltp/ltp/0009-fix-redefinition-of-struct-msgbuf-error-building-wit.patch
 
b/meta/recipes-extended/ltp/ltp/0009-fix-redefinition-of-struct-msgbuf-error-building-wit.patch
index 6886c55..dd7d283 100644
--- 
a/meta/recipes-extended/ltp/ltp/0009-fix-redefinition-of-struct-msgbuf-error-building-wit.patch
+++ 
b/meta/recipes-extended/ltp/ltp/0009-fix-redefinition-of-struct-msgbuf-error-building-wit.patch
@@ -1,18 +1,19 @@
-From 2d7ea67e8eaa41cbd6816f7ab654254b40b530a1 Mon Sep 17 00:00:00 2001
+From bf5dd2932200e0199a38f3028d3bef2253f32e38 Mon Sep 17 00:00:00 2001
 From: Dengke Du 
 Date: Thu, 9 Feb 2017 17:17:37 +0800
 Subject: [PATCH] fix redefinition of 'struct msgbuf' error building with musl
 
 When building with musl the file "sys/msg.h" already contain 'struct msgbuf'
 
+Upstream-Status: Submitted [https://github.com/linux-test-project/ltp/pull/177]
+
 Signed-off-by: Dengke Du 
-Upstream-Status: Pending
 ---
  testcases/kernel/syscalls/ipc/msgrcv/msgrcv08.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)
 
 diff --git a/testcases/kernel/syscalls/ipc/msgrcv/msgrcv08.c 
b/testcases/kernel/syscalls/ipc/msgrcv/msgrcv08.c
-index a757c0d..e023114 100644
+index a757c0d18..e023114d2 100644
 --- a/testcases/kernel/syscalls/ipc/msgrcv/msgrcv08.c
 +++ b/testcases/kernel/syscalls/ipc/msgrcv/msgrcv08.c
 @@ -47,11 +47,13 @@ const char *TCID = "msgrcv08";
@@ -31,5 +32,5 @@ index a757c0d..e023114 100644
  static void msr(int msqid)
  {
 -- 
-2.7.4
+2.11.0
 
-- 
2.8.1

-- 
___
Openembedded-core mailing list
Openembedded-

[OE-core] [PATCH 0/1] ltp: Reduce local Pending patches

2017-07-03 Thread Dengke Du
The following changes since commit 98099349e358a9caaec8ec81f0d4abe588909cfe:

  bitbake: npm fetcher: fix unknown variable name. (2017-06-29 14:33:42 +0100)

are available in the git repository at:

  https://github.com/DengkeDu/openembedded-core.git 
dengke/ltp-reduce-local-pending-patches

Dengke Du (1):
  ltp: Reduce local Pending patches

 ...TH_MAX-undeclared-when-building-with-musl.patch | 15 +++---
 ...RDSIZE-undeclared-when-building-with-musl.patch | 23 +++---
 ...ition-of-struct-msgbuf-error-building-wit.patch |  9 +
 3 files changed, 25 insertions(+), 22 deletions(-)

-- 
2.8.1

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


[OE-core] [PATCH 2/2] kernel-yocto/meta: smp configuration changes

2017-07-03 Thread Bruce Ashfield
Integrating the following kernel configuration changes to
clean up the SMP configuration fragments and fix a configuration
audit warning.

  d0e5ea0e199b smp: Separate smp into 32 and 64 bit versions to avoid kernel 
warnings
  f1369c1d817e bsp/mohonpeak: smp gets added by default, remove unnecessary 
include
  6fc22aa1200b bsp/rangeley: smp gets added by default, remove unnecessary 
include

[YOCTO #11743]

Signed-off-by: Alejandro Hernandez 
Signed-off-by: Bruce Ashfield 
---
 meta/recipes-kernel/linux/linux-yocto-rt_4.10.bb   | 2 +-
 meta/recipes-kernel/linux/linux-yocto-rt_4.4.bb| 2 +-
 meta/recipes-kernel/linux/linux-yocto-rt_4.9.bb| 2 +-
 meta/recipes-kernel/linux/linux-yocto-tiny_4.10.bb | 2 +-
 meta/recipes-kernel/linux/linux-yocto-tiny_4.4.bb  | 2 +-
 meta/recipes-kernel/linux/linux-yocto-tiny_4.9.bb  | 2 +-
 meta/recipes-kernel/linux/linux-yocto_4.10.bb  | 2 +-
 meta/recipes-kernel/linux/linux-yocto_4.4.bb   | 2 +-
 meta/recipes-kernel/linux/linux-yocto_4.9.bb   | 2 +-
 9 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.10.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_4.10.bb
index f25974cb6895..e804d8953397 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_4.10.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.10.bb
@@ -12,7 +12,7 @@ python () {
 }
 
 SRCREV_machine ?= "6648a34e00c55a0634b39e661dd6ba14dd106473"
-SRCREV_meta ?= "4b89f331d4220f6f62d20ef60aa59edcd9b44106"
+SRCREV_meta ?= "e92bd55409a6ded89642f5ac22391487364ac798"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto-4.10.git;branch=${KBRANCH};name=machine 
\

git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-4.10;destsuffix=${KMETA}"
diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_4.4.bb
index 4ad799058b9a..f423604f663f 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_4.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.4.bb
@@ -12,7 +12,7 @@ python () {
 }
 
 SRCREV_machine ?= "b0b44854bfb7f7ea10890ab8d8fff936be7e37da"
-SRCREV_meta ?= "c3f8900923a7b56ea6231d31a1a4e81306156dc5"
+SRCREV_meta ?= "d0e5ea0e199b591cc7588dc80b6ec07ea05f74b8"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto-4.4.git;branch=${KBRANCH};name=machine \

git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-4.4;destsuffix=${KMETA}"
diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.9.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_4.9.bb
index 3bfa24b0c98b..f0e9180ac13c 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_4.9.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.9.bb
@@ -12,7 +12,7 @@ python () {
 }
 
 SRCREV_machine ?= "ab749ed8fd7beb6c63ec62a808eac8678db83810"
-SRCREV_meta ?= "60dd89e2423aaa2e7be680db09be43d1f47471e0"
+SRCREV_meta ?= "299f12a06ca1d6fd90b24450dae3b9f257a536be"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto-4.9.git;branch=${KBRANCH};name=machine \

git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-4.9;destsuffix=${KMETA}"
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_4.10.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_4.10.bb
index 2e98914abb7b..f58799be9139 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_4.10.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_4.10.bb
@@ -10,7 +10,7 @@ KMETA = "kernel-meta"
 KCONF_BSP_AUDIT_LEVEL = "2"
 
 SRCREV_machine ?= "6648a34e00c55a0634b39e661dd6ba14dd106473"
-SRCREV_meta ?= "4b89f331d4220f6f62d20ef60aa59edcd9b44106"
+SRCREV_meta ?= "e92bd55409a6ded89642f5ac22391487364ac798"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
 
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_4.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_4.4.bb
index 3f987a905173..868e70edc45a 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_4.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_4.4.bb
@@ -10,7 +10,7 @@ KMETA = "kernel-meta"
 KCONF_BSP_AUDIT_LEVEL = "2"
 
 SRCREV_machine ?= "268676407913f5d496cde6cbf4052eb5acaf6237"
-SRCREV_meta ?= "c3f8900923a7b56ea6231d31a1a4e81306156dc5"
+SRCREV_meta ?= "d0e5ea0e199b591cc7588dc80b6ec07ea05f74b8"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
 
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_4.9.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_4.9.bb
index 8041c5f77130..482e38c8ce8e 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_4.9.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_4.9.bb
@@ -10,7 +10,7 @@ KMETA = "kernel-meta"
 KCONF_BSP_AUDIT_LEVEL = "2"
 
 SRCREV_machine ?= "d2de43e42bb8a270e680aab1c8f2447bff0b3552"
-SRCREV_meta ?= "60dd89e2423aaa2e7be680db09be43d1f47471e0"
+SRCREV_meta ?= "299f12a06ca1d6fd90b24450dae3b9f257a536be"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
 
diff --git a/meta/recipes-kernel/linux/linux-yocto_4.10.bb 
b/meta/recipes-kernel/linux/linux-yocto_4.10.bb
index fa85b21d4bd8..f3966d4fa588 100644
--- a/meta/recipes-kernel/linux

[OE-core] [PATCH 1/2] kernel-yocto: propagate configuration errors to bbclass

2017-07-03 Thread Bruce Ashfield
As pointed out by klapperichp...@johndeere.com, missing configuration
fragments were being picked up twice, once by the tools and once by the
bbclass. Unfortunately, the tools error message was being detected as
configs, and hence no error was reported at all.

Rather than catching the output of the tools, we can instead check the
return code and propagate the error message from the tools directly to
the user.

[YOCTO #11649]

Signed-off-by: Bruce Ashfield 
---
 meta/classes/kernel-yocto.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/kernel-yocto.bbclass 
b/meta/classes/kernel-yocto.bbclass
index 50226f6d492e..1ca0756c4959 100644
--- a/meta/classes/kernel-yocto.bbclass
+++ b/meta/classes/kernel-yocto.bbclass
@@ -281,7 +281,8 @@ do_kernel_configme() {
 
meta_dir=$(kgit --meta)
configs="$(scc --configs -o ${meta_dir})"
-   if [ -z "${configs}" ]; then
+   if [ $? -ne 0 ]; then
+   bberror "${configs}"
bbfatal_log "Could not find configuration queue 
(${meta_dir}/config.queue)"
fi
 
-- 
2.5.0

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


[OE-core] [PATCH 0/2] kernel-yocto: bug fixes

2017-07-03 Thread Bruce Ashfield
Hi all,

I have some pending version updates and new kernels queued, but I'm not
in the office this week, so I'm only sending a couple of bug fixes along
.. in particular the SMP configuration change, since the warning about
that fragment has been popping up on builds for a bit now.

Cheers,

Bruce

The following changes since commit 98099349e358a9caaec8ec81f0d4abe588909cfe:

  bitbake: npm fetcher: fix unknown variable name. (2017-06-29 14:33:42 +0100)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib zedd/kernel
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=zedd/kernel

Bruce Ashfield (2):
  kernel-yocto: propagate configuration errors to bbclass
  kernel-yocto/meta: smp configuration changes

 meta/classes/kernel-yocto.bbclass  | 3 ++-
 meta/recipes-kernel/linux/linux-yocto-rt_4.10.bb   | 2 +-
 meta/recipes-kernel/linux/linux-yocto-rt_4.4.bb| 2 +-
 meta/recipes-kernel/linux/linux-yocto-rt_4.9.bb| 2 +-
 meta/recipes-kernel/linux/linux-yocto-tiny_4.10.bb | 2 +-
 meta/recipes-kernel/linux/linux-yocto-tiny_4.4.bb  | 2 +-
 meta/recipes-kernel/linux/linux-yocto-tiny_4.9.bb  | 2 +-
 meta/recipes-kernel/linux/linux-yocto_4.10.bb  | 2 +-
 meta/recipes-kernel/linux/linux-yocto_4.4.bb   | 2 +-
 meta/recipes-kernel/linux/linux-yocto_4.9.bb   | 2 +-
 10 files changed, 11 insertions(+), 10 deletions(-)

-- 
2.5.0

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


[OE-core] [PATCH v2] openssh: Fix key generation with systemd

2017-07-03 Thread Joshua Watt
106b59d9 broke SSH host key generation when systemd and a read-only root file
system are in use because there isn't a way for systemd to get the optional
weak assigment of SYSCONFDIR from /etc/default/sshd and still provide a default
value if it is not specified. Instead, move the logic for determining if keys
need to be created to a helper script that both the SysV init script and the
systemd unit file can reference.

This does mean that the systemd unit file can't check for file existence to
know if it should start the service, but it wasn't able to do that correctly
anyway anymore. This should be a problem since the serivce is only run once per
power cycle by systemd, and should exit quickly if the keys already exist

Signed-off-by: Joshua Watt 
---
 meta/recipes-connectivity/openssh/openssh/init | 69 +-
 .../openssh/openssh/sshd_check_keys| 64 
 .../openssh/openssh/sshdgenkeys.service| 16 +
 meta/recipes-connectivity/openssh/openssh_7.5p1.bb |  8 +++
 4 files changed, 76 insertions(+), 81 deletions(-)
 create mode 100644 meta/recipes-connectivity/openssh/openssh/sshd_check_keys

diff --git a/meta/recipes-connectivity/openssh/openssh/init 
b/meta/recipes-connectivity/openssh/openssh/init
index 386628a..34ba0f8 100644
--- a/meta/recipes-connectivity/openssh/openssh/init
+++ b/meta/recipes-connectivity/openssh/openssh/init
@@ -19,25 +19,6 @@ fi
 [ -z "$SYSCONFDIR" ] && SYSCONFDIR=/etc/ssh
 mkdir -p $SYSCONFDIR
 
-parse_sshd_opts() {
-set -- ${SSHD_OPTS} --
-sshd_config=/etc/ssh/sshd_config
-while true ; do
-case "$1" in
--f*) if [ "$1" = "-f" ] ; then
- sshd_config="$2"
- shift
- else
- sshd_config="${1#-f}"
- fi
- shift
- ;;
---) shift; break;;
-*) shift;;
-esac
-done
-}
-
 check_for_no_start() {
 # forget it if we're trying to start, and /etc/ssh/sshd_not_to_be_run 
exists
 if [ -e $SYSCONFDIR/sshd_not_to_be_run ]; then
@@ -58,57 +39,13 @@ check_config() {
/usr/sbin/sshd -t $SSHD_OPTS || exit 1
 }
 
-check_keys() {
-   # parse location of keys
-   local HOST_KEY_RSA
-   local HOST_KEY_DSA
-   local HOST_KEY_ECDSA
-   local HOST_KEY_ED25519
-
-   parse_sshd_opts
-   HOST_KEY_RSA=$(grep ^HostKey "${sshd_config}" | grep _rsa_ | tail -1 | 
awk ' { print $2 } ')
-   [ -z "${HOST_KEY_RSA}" ] && HOST_KEY_RSA=$(grep HostKey 
"${sshd_config}" | grep _rsa_ | tail -1 | awk ' { print $2 } ')
-   [ -z "${HOST_KEY_RSA}" ] && HOST_KEY_RSA=$SYSCONFDIR/ssh_host_rsa_key
-   HOST_KEY_DSA=$(grep ^HostKey "${sshd_config}" | grep _dsa_ | tail -1 | 
awk ' { print $2 } ')
-   [ -z "${HOST_KEY_DSA}" ] && HOST_KEY_DSA=$(grep HostKey 
"${sshd_config}" | grep _dsa_ | tail -1 | awk ' { print $2 } ')
-   [ -z "${HOST_KEY_DSA}" ] && HOST_KEY_DSA=$SYSCONFDIR/ssh_host_dsa_key
-   HOST_KEY_ECDSA=$(grep ^HostKey "${sshd_config}" | grep _ecdsa_ | tail 
-1 | awk ' { print $2 } ')
-   [ -z "${HOST_KEY_ECDSA}" ] && HOST_KEY_ECDSA=$(grep HostKey 
"${sshd_config}" | grep _ecdsa_ | tail -1 | awk ' { print $2 } ')
-   [ -z "${HOST_KEY_ECDSA}" ] && 
HOST_KEY_ECDSA=$SYSCONFDIR/ssh_host_ecdsa_key
-   HOST_KEY_ED25519=$(grep ^HostKey "${sshd_config}" | grep _ed25519_ | 
tail -1 | awk ' { print $2 } ')
-   [ -z "${HOST_KEY_ED25519}" ] && HOST_KEY_ED25519=$(grep HostKey 
"${sshd_config}" | grep _ed25519_ | tail -1 | awk ' { print $2 } ')
-   [ -z "${HOST_KEY_ED25519}" ] && 
HOST_KEY_ED25519=$SYSCONFDIR/ssh_host_ed25519_key
-
-   # create keys if necessary
-   if [ ! -f $HOST_KEY_RSA ]; then
-   echo "  generating ssh RSA key..."
-   mkdir -p $(dirname $HOST_KEY_RSA)
-   ssh-keygen -q -f $HOST_KEY_RSA -N '' -t rsa
-   fi
-   if [ ! -f $HOST_KEY_ECDSA ]; then
-   echo "  generating ssh ECDSA key..."
-   mkdir -p $(dirname $HOST_KEY_ECDSA)
-   ssh-keygen -q -f $HOST_KEY_ECDSA -N '' -t ecdsa
-   fi
-   if [ ! -f $HOST_KEY_DSA ]; then
-   echo "  generating ssh DSA key..."
-   mkdir -p $(dirname $HOST_KEY_DSA)
-   ssh-keygen -q -f $HOST_KEY_DSA -N '' -t dsa
-   fi
-   if [ ! -f $HOST_KEY_ED25519 ]; then
-   echo "  generating ssh ED25519 key..."
-   mkdir -p $(dirname $HOST_KEY_ED25519)
-   ssh-keygen -q -f $HOST_KEY_ED25519 -N '' -t ed25519
-   fi
-}
-
 export PATH="${PATH:+$PATH:}/usr/sbin:/sbin"
 
 case "$1" in
   start)
check_for_no_start
echo "Starting OpenBSD Secure Shell server: sshd"
-   check_keys
+   @LIBEXECDIR@/sshd_check_keys
check_privsep_dir
start-stop-daemon -S -p $PIDFILE -x /usr/sbin/sshd -- $SSHD_OPTS
 echo "done."
@@ -121,7 +58,7 @@ case "$1" in
 
   reload|force-reload)
check_for_no_start
-

Re: [OE-core] [PATCH v2 1/3] initramfs-framework: udev: Add a runtime dependency on udev-extraconf

2017-07-03 Thread Ng, Wei Tee
Yes, it's mainly to support the live boot image. Therefore the udev-extraconf 
is being added in the rdepends.
The wrong that you mentioned is because of the commit message was not convey in 
the correct way?
The initramfs-framework currently is not handling live boot support. Perhaps, 
"In order to implement initramfs-framework for live boot support" will be 
better?

Regards,
Wei Tee

-Original Message-
From: Otavio Salvador [mailto:otavio.salva...@ossystems.com.br] 
Sent: Tuesday, July 4, 2017 4:17 AM
To: Ng, Wei Tee 
Cc: Patches and discussions about the oe-core layer 

Subject: Re: [OE-core] [PATCH v2 1/3] initramfs-framework: udev: Add a runtime 
dependency on udev-extraconf

On Thu, Jun 29, 2017 at 9:30 PM,   wrote:
> From: "Ng, Wei Tee" 
>
> In order to implement initramfs-framework for boot up process, it 
> depends on udev performing automounting, which happens in udev-extraconf.
> Explicitly depend on it so that we always have it installed.
>
> Signed-off-by: Ng, Wei Tee 

Wrong; this is need for the live support so the rdepends should be there.


-- 
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


Re: [OE-core] [PATCH v3 1/4] gcc-6.3: Fix libgfortran build

2017-07-03 Thread Khem Raj

Whole series looks ok to me.

On 7/3/17 2:57 PM, Ricardo Ribalda Delgado wrote:

Without this patch libbacktrace cannot be included/link during
libgfortran build.

libbtool: link: cannot find the library `../libbacktrace/libbacktrace.la'
or unhandled argument `../libbacktrace/libbacktrace.la'

Signed-off-by: Ricardo Ribalda Delgado 
---
  meta/recipes-devtools/gcc/gcc-6.3.inc  |  1 +
  .../gcc/gcc-6.3/0055-Fix-gfortran-build.patch  | 53 ++
  2 files changed, 54 insertions(+)
  create mode 100644 
meta/recipes-devtools/gcc/gcc-6.3/0055-Fix-gfortran-build.patch

diff --git a/meta/recipes-devtools/gcc/gcc-6.3.inc 
b/meta/recipes-devtools/gcc/gcc-6.3.inc
index 86b3e9ff19a4..2595dbd0ee8b 100644
--- a/meta/recipes-devtools/gcc/gcc-6.3.inc
+++ b/meta/recipes-devtools/gcc/gcc-6.3.inc
@@ -74,6 +74,7 @@ SRC_URI = "\
 
file://0047-libgcc_s-Use-alias-for-__cpu_indicator_init-instead-.patch \
 file://0048-sync-gcc-stddef.h-with-musl.patch \
 file://0054_all_nopie-all-flags.patch \
+   file://0055-Fix-gfortran-build.patch \
 ${BACKPORTS} \
  "
  BACKPORTS = "\
diff --git a/meta/recipes-devtools/gcc/gcc-6.3/0055-Fix-gfortran-build.patch 
b/meta/recipes-devtools/gcc/gcc-6.3/0055-Fix-gfortran-build.patch
new file mode 100644
index ..30b135b9780f
--- /dev/null
+++ b/meta/recipes-devtools/gcc/gcc-6.3/0055-Fix-gfortran-build.patch
@@ -0,0 +1,53 @@
+Without this patch libbacktrace cannot be included/link during
+libgfortran build.
+
+libbtool: link: cannot find the library `../libbacktrace/libbacktrace.la'
+or unhandled argument `../libbacktrace/libbacktrace.la'
+
+Upstream-Status: Inappropriate [OE specific]
+Signed-off-by: Ricardo Ribalda Delgado 
+
+diff --git a/libgfortran/Makefile.am b/libgfortran/Makefile.am
+index 39d3e11d223c..b5efe6dfe195 100644
+--- a/libgfortran/Makefile.am
 b/libgfortran/Makefile.am
+@@ -37,7 +37,7 @@ toolexeclib_LTLIBRARIES = libgfortran.la
+ toolexeclib_DATA = libgfortran.spec
+ libgfortran_la_LINK = $(LINK) $(libgfortran_la_LDFLAGS)
+ libgfortran_la_LDFLAGS = -version-info `grep -v '^\#' 
$(srcdir)/libtool-version` \
+-  $(LTLDFLAGS) $(LIBQUADLIB) ../libbacktrace/libbacktrace.la \
++  $(LTLDFLAGS) $(LIBQUADLIB) ../../libbacktrace/libbacktrace.la \
+   -lm $(extra_ldflags_libgfortran) \
+   $(version_arg) -Wc,-shared-libgcc
+ libgfortran_la_DEPENDENCIES = $(version_dep) libgfortran.spec 
$(LIBQUADLIB_DEP)
+@@ -63,7 +63,7 @@ AM_CPPFLAGS = -iquote$(srcdir)/io 
-I$(srcdir)/$(MULTISRCTOP)../gcc \
+ -I$(MULTIBUILDTOP)../libgcc \
+ -I$(srcdir)/$(MULTISRCTOP)../libbacktrace \
+ -I$(MULTIBUILDTOP)../libbacktrace \
+--I../libbacktrace
++-I../../libbacktrace
+
+ # Fortran rules for complex multiplication and division
+ AM_CFLAGS += -fcx-fortran-rules
+diff --git a/libgfortran/Makefile.in b/libgfortran/Makefile.in
+index 7ed080cf7b0d..d197fbd6b683 100644
+--- a/libgfortran/Makefile.in
 b/libgfortran/Makefile.in
+@@ -598,7 +598,7 @@ toolexeclib_LTLIBRARIES = libgfortran.la
+ toolexeclib_DATA = libgfortran.spec
+ libgfortran_la_LINK = $(LINK) $(libgfortran_la_LDFLAGS)
+ libgfortran_la_LDFLAGS = -version-info `grep -v '^\#' 
$(srcdir)/libtool-version` \
+-  $(LTLDFLAGS) $(LIBQUADLIB) ../libbacktrace/libbacktrace.la \
++  $(LTLDFLAGS) $(LIBQUADLIB) ../../libbacktrace/libbacktrace.la \
+   -lm $(extra_ldflags_libgfortran) \
+   $(version_arg) -Wc,-shared-libgcc
+
+@@ -618,7 +618,7 @@ AM_CPPFLAGS = -iquote$(srcdir)/io 
-I$(srcdir)/$(MULTISRCTOP)../gcc \
+ -I$(MULTIBUILDTOP)../libgcc \
+ -I$(srcdir)/$(MULTISRCTOP)../libbacktrace \
+ -I$(MULTIBUILDTOP)../libbacktrace \
+--I../libbacktrace
++-I../../libbacktrace
+
+ gfor_io_src = io/size_from_kind.c $(am__append_2)
+ gfor_io_headers = \


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


[OE-core] [PATCHv2] gtk+3: Update the patches to work with old versions of patch

2017-07-03 Thread Peter Kjellerstedt
The patch "0003-Add-disable-opengl-configure-option.patch" used Git
rename syntax, which is not supported by patch 2.6.x and older.

Signed-off-by: Peter Kjellerstedt 
---

PATCHv2: Updated the commit message to clarify why this is needed.

 .../gtk+/gtk+3/0001-Hardcoded-libtool.patch|  12 +-
 ...Do-not-try-to-initialize-GL-without-libGL.patch |   8 +-
 .../0003-Add-disable-opengl-configure-option.patch | 268 +++--
 ...4-configure.ac-Fix-wayland-protocols-path.patch |   8 +-
 4 files changed, 207 insertions(+), 89 deletions(-)

diff --git a/meta/recipes-gnome/gtk+/gtk+3/0001-Hardcoded-libtool.patch 
b/meta/recipes-gnome/gtk+/gtk+3/0001-Hardcoded-libtool.patch
index 0bda1f1074..c210bbc7d5 100644
--- a/meta/recipes-gnome/gtk+/gtk+3/0001-Hardcoded-libtool.patch
+++ b/meta/recipes-gnome/gtk+/gtk+3/0001-Hardcoded-libtool.patch
@@ -1,4 +1,4 @@
-From 5a73a5b92566e314bbc8a1bd40f1ec204837c111 Mon Sep 17 00:00:00 2001
+From 0ecaa5bab162abf0cb2057d77beeb7b89d5873b4 Mon Sep 17 00:00:00 2001
 From: Jussi Kukkonen 
 Date: Tue, 21 Jun 2016 14:53:56 +0300
 Subject: [PATCH 1/4] Hardcoded libtool
@@ -12,10 +12,10 @@ Signed-off-by: Jussi Kukkonen 
  1 file changed, 3 insertions(+), 3 deletions(-)
 
 diff --git a/configure.ac b/configure.ac
-index 23d3077..b6931d6 100644
+index 6628e21..f43ac09 100644
 --- a/configure.ac
 +++ b/configure.ac
-@@ -610,7 +610,7 @@ AC_MSG_CHECKING([whether to write dependencies into .pc 
files])
+@@ -617,7 +617,7 @@ AC_MSG_CHECKING([whether to write dependencies into .pc 
files])
  case $enable_explicit_deps in
auto)
  export SED
@@ -24,7 +24,7 @@ index 23d3077..b6931d6 100644
  if test "x$deplibs_check_method" != xpass_all || test "x$enable_static" = 
xyes ; then
enable_explicit_deps=yes
  else
-@@ -876,7 +876,7 @@ else
+@@ -895,7 +895,7 @@ else
  dnl Now we check to see if our libtool supports shared lib deps
  dnl (in a rather ugly way even)
  if $dynworks; then
@@ -33,7 +33,7 @@ index 23d3077..b6931d6 100644
  module_deplibs_check=`$module_libtool_config | \
  grep '^[[a-z_]]*check[[a-z_]]*_method=[['\''"]]' | \
  sed 's/.*[['\''"]]\(.*\)[['\''"]]$/\1/'`
-@@ -1630,7 +1630,7 @@ AC_SUBST(GTK_PRINT_BACKENDS)
+@@ -1649,7 +1649,7 @@ AC_SUBST(GTK_PRINT_BACKENDS)
  # We are using gmodule-no-export now, but I'm leaving the stripping
  # code in place for now, since pango and atk still require gmodule.
  export SED
@@ -43,5 +43,5 @@ index 23d3077..b6931d6 100644
GDK_DEP_LIBS=`echo $GDK_DEP_LIBS | sed -e "s/$export_dynamic//"`
GTK_DEP_LIBS=`echo $GTK_DEP_LIBS | sed -e "s/$export_dynamic//"`
 -- 
-2.1.4
+2.12.0
 
diff --git 
a/meta/recipes-gnome/gtk+/gtk+3/0002-Do-not-try-to-initialize-GL-without-libGL.patch
 
b/meta/recipes-gnome/gtk+/gtk+3/0002-Do-not-try-to-initialize-GL-without-libGL.patch
index 989716e513..ab7b659414 100644
--- 
a/meta/recipes-gnome/gtk+/gtk+3/0002-Do-not-try-to-initialize-GL-without-libGL.patch
+++ 
b/meta/recipes-gnome/gtk+/gtk+3/0002-Do-not-try-to-initialize-GL-without-libGL.patch
@@ -1,4 +1,4 @@
-From d63b926f8c8d8b5c9f9ec33d078b775f0409d88a Mon Sep 17 00:00:00 2001
+From 2d2361f24be1e51201063dbbda570debe8702b6f Mon Sep 17 00:00:00 2001
 From: Jussi Kukkonen 
 Date: Fri, 16 Oct 2015 16:35:16 +0300
 Subject: [PATCH 2/4] Do not try to initialize GL without libGL
@@ -21,10 +21,10 @@ Signed-off-by: Jussi Kukkonen 
  2 files changed, 12 insertions(+)
 
 diff --git a/configure.ac b/configure.ac
-index b6931d6..e27da49 100644
+index f43ac09..965eec9 100644
 --- a/configure.ac
 +++ b/configure.ac
-@@ -342,6 +342,13 @@ AC_ARG_ENABLE(mir-backend,
+@@ -346,6 +346,13 @@ AC_ARG_ENABLE(mir-backend,
[enable the Mir gdk backend])],
  [backend_set=yes])
  
@@ -56,5 +56,5 @@ index 81479d8..3c8c5c0 100644
  
  gint
 -- 
-2.1.4
+2.12.0
 
diff --git 
a/meta/recipes-gnome/gtk+/gtk+3/0003-Add-disable-opengl-configure-option.patch 
b/meta/recipes-gnome/gtk+/gtk+3/0003-Add-disable-opengl-configure-option.patch
index e6ee21b64f..eaf6aec2aa 100644
--- 
a/meta/recipes-gnome/gtk+/gtk+3/0003-Add-disable-opengl-configure-option.patch
+++ 
b/meta/recipes-gnome/gtk+/gtk+3/0003-Add-disable-opengl-configure-option.patch
@@ -1,7 +1,7 @@
-From cdfd6216ac49fc7622d563d671a5b6511492827d Mon Sep 17 00:00:00 2001
+From a2e6b7cbbd6d741fed38d91b3742aa4a08395aba Mon Sep 17 00:00:00 2001
 From: Jussi Kukkonen 
 Date: Tue, 21 Jun 2016 15:11:39 +0300
-Subject: [PATCH] Add --disable-opengl configure option
+Subject: [PATCH 3/4] Add --disable-opengl configure option
 
 --disable-opengl will remove the dependency on libepoxy and on the
 OpenGL APIs. This is useful for those who want to keep using gtk+3
@@ -20,32 +20,34 @@ as it actually will depend on OpenGL.
 Upstream-Status: Inappropriate [Evil eye expected from upstream]
 Signed-off-by: Jussi Kukkonen 
 ---
- configure.ac   | 13 +--
- demos/gtk-demo/glarea.c| 13 

Re: [OE-core] [PATCH 01/11] gtk+3: Update the patches to work without PATCHTOOL = "git"

2017-07-03 Thread Peter Kjellerstedt
> -Original Message-
> From: Alexander Kanavin [mailto:alexander.kana...@linux.intel.com]
> Sent: den 15 juni 2017 22:44
> To: Peter Kjellerstedt ; Burton, Ross
> 
> Cc: OE-core ; Kukkonen, Jussi
> 
> Subject: Re: [OE-core] [PATCH 01/11] gtk+3: Update the patches to work
> without PATCHTOOL = "git"
> 
> On 06/15/2017 06:33 PM, Peter Kjellerstedt wrote:
> > It turns out the problem is only seen if opengl is enabled in
> > DISTRO_FEATURES. However, since that is the default for poky, that
> > still does not explain why others do not see the problem
> >
> > I have included the log.do_compile file where one can see that the
> > file gdkx-with-gl-context.h is missing (that is the file that was
> > supposed to be renamed from gdkx.h by the patch). When I look in
> > tmp/work/i586-poky-linux/gtk+3/3.22.15-r0/gtk+-3.22.15/gdk/x11 I see
> > gdkx.h and gdkx-without-gl-context.h, instead of the expected
> > gdkx-with-gl-context.h and gdkx-without-gl-context.h...
> 
> I guess Jussi needs to look at this, as he's maintaining the patches
> and the recipe.
> 
> Alex

Ok, I have finally figured out what is going on. The problem is the 
version of patch that is installed on the host. Support for Git style 
patches was added to patch in version 2.7, but on the host where I 
failed to build gtk+3, the installed version of patch is 2.6.1...

However, since this is the only patch in all of Poky and OpenEmbedded 
that uses the Git style format, I still think the correct thing to do is 
to apply my change. It does not change the contents of the patch, only 
the format.

I will send an updated change with an updated commit message to clarify 
what is going on.

//Peter

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


[OE-core] [PATCH v3 4/4] gcc-7.1: Fix libgfortran build

2017-07-03 Thread Ricardo Ribalda Delgado
Without this patch libbacktrace cannot be included/link during
libgfortran build.

libbtool: link: cannot find the library
`../libbacktrace/libbacktrace.la'
or unhandled argument `../libbacktrace/libbacktrace.la

Suggested-by: Khem Raj 
Signed-off-by: Ricardo Ribalda Delgado 
---
 meta/recipes-devtools/gcc/gcc-7.1.inc  |  1 +
 .../gcc/gcc-7.1/0048-Fix-gfortran-build.patch  | 53 ++
 2 files changed, 54 insertions(+)
 create mode 100644 
meta/recipes-devtools/gcc/gcc-7.1/0048-Fix-gfortran-build.patch

diff --git a/meta/recipes-devtools/gcc/gcc-7.1.inc 
b/meta/recipes-devtools/gcc/gcc-7.1.inc
index 4098d6a2c1e0..e142cdf79433 100644
--- a/meta/recipes-devtools/gcc/gcc-7.1.inc
+++ b/meta/recipes-devtools/gcc/gcc-7.1.inc
@@ -72,6 +72,7 @@ SRC_URI = "\
file://0045-Link-libgcc-using-LDFLAGS-not-just-SHLIB_LDFLAGS.patch \

file://0046-libgcc_s-Use-alias-for-__cpu_indicator_init-instead-.patch \
file://0047-sync-gcc-stddef.h-with-musl.patch \
+   file://0048-Fix-gfortran-build.patch \
${BACKPORTS} \
 "
 BACKPORTS = "\
diff --git a/meta/recipes-devtools/gcc/gcc-7.1/0048-Fix-gfortran-build.patch 
b/meta/recipes-devtools/gcc/gcc-7.1/0048-Fix-gfortran-build.patch
new file mode 100644
index ..a71b6b0bee0e
--- /dev/null
+++ b/meta/recipes-devtools/gcc/gcc-7.1/0048-Fix-gfortran-build.patch
@@ -0,0 +1,53 @@
+Without this patch libbacktrace cannot be included/link during
+libgfortran build.
+
+libbtool: link: cannot find the library `../libbacktrace/libbacktrace.la'
+or unhandled argument `../libbacktrace/libbacktrace.la'
+
+Upstream-Status: Inappropriate [OE specific]
+Signed-off-by: Ricardo Ribalda Delgado 
+
+diff --git a/libgfortran/Makefile.am b/libgfortran/Makefile.am
+index 2857af5943f9..e07760069a3c 100644
+--- a/libgfortran/Makefile.am
 b/libgfortran/Makefile.am
+@@ -37,7 +37,7 @@ toolexeclib_LTLIBRARIES = libgfortran.la
+ toolexeclib_DATA = libgfortran.spec
+ libgfortran_la_LINK = $(LINK) $(libgfortran_la_LDFLAGS)
+ libgfortran_la_LDFLAGS = -version-info `grep -v '^\#' 
$(srcdir)/libtool-version` \
+-  $(LTLDFLAGS) $(LIBQUADLIB) ../libbacktrace/libbacktrace.la \
++  $(LTLDFLAGS) $(LIBQUADLIB) ../../libbacktrace/libbacktrace.la \
+   $(HWCAP_LDFLAGS) \
+   -lm $(extra_ldflags_libgfortran) \
+   $(version_arg) -Wc,-shared-libgcc
+@@ -64,7 +64,7 @@ AM_CPPFLAGS = -iquote$(srcdir)/io 
-I$(srcdir)/$(MULTISRCTOP)../gcc \
+ -I$(MULTIBUILDTOP)../libgcc \
+ -I$(srcdir)/$(MULTISRCTOP)../libbacktrace \
+ -I$(MULTIBUILDTOP)../libbacktrace \
+--I../libbacktrace
++-I../../libbacktrace
+ 
+ # Fortran rules for complex multiplication and division
+ AM_CFLAGS += -fcx-fortran-rules
+diff --git a/libgfortran/Makefile.in b/libgfortran/Makefile.in
+index 05b183dcadb0..c9e0280e412c 100644
+--- a/libgfortran/Makefile.in
 b/libgfortran/Makefile.in
+@@ -581,7 +581,7 @@ toolexeclib_LTLIBRARIES = libgfortran.la
+ toolexeclib_DATA = libgfortran.spec
+ libgfortran_la_LINK = $(LINK) $(libgfortran_la_LDFLAGS)
+ libgfortran_la_LDFLAGS = -version-info `grep -v '^\#' 
$(srcdir)/libtool-version` \
+-  $(LTLDFLAGS) $(LIBQUADLIB) ../libbacktrace/libbacktrace.la \
++  $(LTLDFLAGS) $(LIBQUADLIB) ../../libbacktrace/libbacktrace.la \
+   $(HWCAP_LDFLAGS) \
+   -lm $(extra_ldflags_libgfortran) \
+   $(version_arg) -Wc,-shared-libgcc
+@@ -602,7 +602,7 @@ AM_CPPFLAGS = -iquote$(srcdir)/io 
-I$(srcdir)/$(MULTISRCTOP)../gcc \
+ -I$(MULTIBUILDTOP)../libgcc \
+ -I$(srcdir)/$(MULTISRCTOP)../libbacktrace \
+ -I$(MULTIBUILDTOP)../libbacktrace \
+--I../libbacktrace
++-I../../libbacktrace
+ 
+ gfor_io_src = io/size_from_kind.c $(am__append_2)
+ gfor_io_headers = \
-- 
2.11.0

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


[OE-core] [PATCH v3 3/4] libgfortran: Add missing fincludes

2017-07-03 Thread Ricardo Ribalda Delgado
ERROR: libgfortran-6.3.0-r0 do_package: QA Issue: libgfortran:
Files/directories were installed but not shipped in any package:
/usr/lib/gcc/x86_64-poky-linux/6.3.0/finclude
/usr/lib/gcc/x86_64-poky-linux/6.3.0/finclude/ieee_features.mod
/usr/lib/gcc/x86_64-poky-linux/6.3.0/finclude/ieee_arithmetic.mod
/usr/lib/gcc/x86_64-poky-linux/6.3.0/finclude/ieee_exceptions.mod

Signed-off-by: Ricardo Ribalda Delgado 
---
 meta/recipes-devtools/gcc/libgfortran.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-devtools/gcc/libgfortran.inc 
b/meta/recipes-devtools/gcc/libgfortran.inc
index 3b94c1b85b70..5f5d4af7589d 100644
--- a/meta/recipes-devtools/gcc/libgfortran.inc
+++ b/meta/recipes-devtools/gcc/libgfortran.inc
@@ -54,6 +54,7 @@ FILES_${PN}-dev = "\
 ${libdir}/libgfortran.la \
 ${libdir}/gcc/${TARGET_SYS}/${BINV}/libgfortranbegin.* \
 ${libdir}/gcc/${TARGET_SYS}/${BINV}/libcaf_single* \
+${libdir}/gcc/${TARGET_SYS}/${BINV}/finclude/ \
 "
 FILES_${PN}-staticdev = "${libdir}/libgfortran.a"
 
-- 
2.11.0

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


[OE-core] [PATCH v3 2/4] libgfortran: Add missing dependency gcc-cross

2017-07-03 Thread Ricardo Ribalda Delgado
Due to the fact that the recipe uses INHIBIT_DEFAULT_DEPS, we need to
manually add the cross compiler as a dependency.

Signed-off-by: Ricardo Ribalda Delgado 
---
 meta/recipes-devtools/gcc/libgfortran.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/gcc/libgfortran.inc 
b/meta/recipes-devtools/gcc/libgfortran.inc
index 4846decbb834..3b94c1b85b70 100644
--- a/meta/recipes-devtools/gcc/libgfortran.inc
+++ b/meta/recipes-devtools/gcc/libgfortran.inc
@@ -37,7 +37,7 @@ do_install () {
 }
 
 INHIBIT_DEFAULT_DEPS = "1"
-DEPENDS = "gcc-runtime"
+DEPENDS = "gcc-runtime gcc-cross-${TARGET_ARCH}"
 
 BBCLASSEXTEND = "nativesdk"
 
-- 
2.11.0

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


[OE-core] [PATCH v3 1/4] gcc-6.3: Fix libgfortran build

2017-07-03 Thread Ricardo Ribalda Delgado
Without this patch libbacktrace cannot be included/link during
libgfortran build.

libbtool: link: cannot find the library `../libbacktrace/libbacktrace.la'
or unhandled argument `../libbacktrace/libbacktrace.la'

Signed-off-by: Ricardo Ribalda Delgado 
---
 meta/recipes-devtools/gcc/gcc-6.3.inc  |  1 +
 .../gcc/gcc-6.3/0055-Fix-gfortran-build.patch  | 53 ++
 2 files changed, 54 insertions(+)
 create mode 100644 
meta/recipes-devtools/gcc/gcc-6.3/0055-Fix-gfortran-build.patch

diff --git a/meta/recipes-devtools/gcc/gcc-6.3.inc 
b/meta/recipes-devtools/gcc/gcc-6.3.inc
index 86b3e9ff19a4..2595dbd0ee8b 100644
--- a/meta/recipes-devtools/gcc/gcc-6.3.inc
+++ b/meta/recipes-devtools/gcc/gcc-6.3.inc
@@ -74,6 +74,7 @@ SRC_URI = "\

file://0047-libgcc_s-Use-alias-for-__cpu_indicator_init-instead-.patch \
file://0048-sync-gcc-stddef.h-with-musl.patch \
file://0054_all_nopie-all-flags.patch \
+   file://0055-Fix-gfortran-build.patch \
${BACKPORTS} \
 "
 BACKPORTS = "\
diff --git a/meta/recipes-devtools/gcc/gcc-6.3/0055-Fix-gfortran-build.patch 
b/meta/recipes-devtools/gcc/gcc-6.3/0055-Fix-gfortran-build.patch
new file mode 100644
index ..30b135b9780f
--- /dev/null
+++ b/meta/recipes-devtools/gcc/gcc-6.3/0055-Fix-gfortran-build.patch
@@ -0,0 +1,53 @@
+Without this patch libbacktrace cannot be included/link during
+libgfortran build.
+
+libbtool: link: cannot find the library `../libbacktrace/libbacktrace.la'
+or unhandled argument `../libbacktrace/libbacktrace.la'
+
+Upstream-Status: Inappropriate [OE specific]
+Signed-off-by: Ricardo Ribalda Delgado 
+
+diff --git a/libgfortran/Makefile.am b/libgfortran/Makefile.am
+index 39d3e11d223c..b5efe6dfe195 100644
+--- a/libgfortran/Makefile.am
 b/libgfortran/Makefile.am
+@@ -37,7 +37,7 @@ toolexeclib_LTLIBRARIES = libgfortran.la
+ toolexeclib_DATA = libgfortran.spec
+ libgfortran_la_LINK = $(LINK) $(libgfortran_la_LDFLAGS)
+ libgfortran_la_LDFLAGS = -version-info `grep -v '^\#' 
$(srcdir)/libtool-version` \
+-  $(LTLDFLAGS) $(LIBQUADLIB) ../libbacktrace/libbacktrace.la \
++  $(LTLDFLAGS) $(LIBQUADLIB) ../../libbacktrace/libbacktrace.la \
+   -lm $(extra_ldflags_libgfortran) \
+   $(version_arg) -Wc,-shared-libgcc
+ libgfortran_la_DEPENDENCIES = $(version_dep) libgfortran.spec 
$(LIBQUADLIB_DEP)
+@@ -63,7 +63,7 @@ AM_CPPFLAGS = -iquote$(srcdir)/io 
-I$(srcdir)/$(MULTISRCTOP)../gcc \
+ -I$(MULTIBUILDTOP)../libgcc \
+ -I$(srcdir)/$(MULTISRCTOP)../libbacktrace \
+ -I$(MULTIBUILDTOP)../libbacktrace \
+--I../libbacktrace
++-I../../libbacktrace
+ 
+ # Fortran rules for complex multiplication and division
+ AM_CFLAGS += -fcx-fortran-rules
+diff --git a/libgfortran/Makefile.in b/libgfortran/Makefile.in
+index 7ed080cf7b0d..d197fbd6b683 100644
+--- a/libgfortran/Makefile.in
 b/libgfortran/Makefile.in
+@@ -598,7 +598,7 @@ toolexeclib_LTLIBRARIES = libgfortran.la
+ toolexeclib_DATA = libgfortran.spec
+ libgfortran_la_LINK = $(LINK) $(libgfortran_la_LDFLAGS)
+ libgfortran_la_LDFLAGS = -version-info `grep -v '^\#' 
$(srcdir)/libtool-version` \
+-  $(LTLDFLAGS) $(LIBQUADLIB) ../libbacktrace/libbacktrace.la \
++  $(LTLDFLAGS) $(LIBQUADLIB) ../../libbacktrace/libbacktrace.la \
+   -lm $(extra_ldflags_libgfortran) \
+   $(version_arg) -Wc,-shared-libgcc
+ 
+@@ -618,7 +618,7 @@ AM_CPPFLAGS = -iquote$(srcdir)/io 
-I$(srcdir)/$(MULTISRCTOP)../gcc \
+ -I$(MULTIBUILDTOP)../libgcc \
+ -I$(srcdir)/$(MULTISRCTOP)../libbacktrace \
+ -I$(MULTIBUILDTOP)../libbacktrace \
+--I../libbacktrace
++-I../../libbacktrace
+ 
+ gfor_io_src = io/size_from_kind.c $(am__append_2)
+ gfor_io_headers = \
-- 
2.11.0

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


Re: [OE-core] [PATCH v2 1/2] mesa: Avoid platform probing when building without EGL

2017-07-03 Thread Nicolas Dechesne
On Mon, Jul 3, 2017 at 10:02 PM, Otavio Salvador
 wrote:
> The issue has been exposed by meta-freescale BSP. Fix tested with
> imx6qsabresd machine.
>
> Signed-off-by: Otavio Salvador 

this looks correct to me, and this is due to this upstream commit:
1d89787a19 configure: loosen --with-platforms heuristics

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


Re: [OE-core] [PATCH v2 1/3] initramfs-framework: udev: Add a runtime dependency on udev-extraconf

2017-07-03 Thread Otavio Salvador
On Thu, Jun 29, 2017 at 9:30 PM,   wrote:
> From: "Ng, Wei Tee" 
>
> In order to implement initramfs-framework for boot up process, it
> depends on udev performing automounting, which happens in udev-extraconf.
> Explicitly depend on it so that we always have it installed.
>
> Signed-off-by: Ng, Wei Tee 

Wrong; this is need for the live support so the rdepends should be there.


-- 
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] [PATCH v2] oeqa/sdk: Replace buildiptables for buildlzip tests

2017-07-03 Thread jose . perez . carranza
From: Jose Perez Carranza 

Buildiptables test cases are conflicting with images built with “musl”
as standard C library, in order to avoid those issues lzip package was
selected to be used on the tests as this does not have any "musl"
dependency.

[YOCTO #11713]

Signed-off-by: Jose Perez Carranza 
---
 meta/lib/oeqa/sdk/cases/{buildiptables.py => buildlzip.py} | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)
 rename meta/lib/oeqa/sdk/cases/{buildiptables.py => buildlzip.py} (84%)

diff --git a/meta/lib/oeqa/sdk/cases/buildiptables.py 
b/meta/lib/oeqa/sdk/cases/buildlzip.py
similarity index 84%
rename from meta/lib/oeqa/sdk/cases/buildiptables.py
rename to meta/lib/oeqa/sdk/cases/buildlzip.py
index 419c7ebd09b..3a89ce86276 100644
--- a/meta/lib/oeqa/sdk/cases/buildiptables.py
+++ b/meta/lib/oeqa/sdk/cases/buildlzip.py
@@ -3,15 +3,15 @@ from oeqa.sdk.case import OESDKTestCase
 from oeqa.sdk.utils.sdkbuildproject import SDKBuildProject
 
 
-class BuildIptablesTest(OESDKTestCase):
+class BuildLzipTest(OESDKTestCase):
 td_vars = ['DATETIME']
 
 @classmethod
 def setUpClass(self):
 dl_dir = self.td.get('DL_DIR', None)
 
-self.project = SDKBuildProject(self.tc.sdk_dir + "/iptables/", 
self.tc.sdk_env, 
-
"http://downloads.yoctoproject.org/mirror/sources/iptables-1.4.13.tar.bz2";,
+self.project = SDKBuildProject(self.tc.sdk_dir + "/lzip/", 
self.tc.sdk_env,
+
"http://downloads.yoctoproject.org/mirror/sources/lzip-1.19.tar.gz";,
 self.tc.sdk_dir, self.td['DATETIME'], dl_dir=dl_dir)
 self.project.download_archive()
 
@@ -21,7 +21,7 @@ class BuildIptablesTest(OESDKTestCase):
 self.tc.hasTargetPackage("gcc")):
 raise unittest.SkipTest("SDK doesn't contain a cross-canadian 
toolchain")
 
-def test_iptables(self):
+def test_lzip(self):
 self.assertEqual(self.project.run_configure(), 0,
 msg="Running configure failed")
 
-- 
2.12.0

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


[OE-core] [PATCH v2 2/2] mesa: Upgrade to 17.1.4 release

2017-07-03 Thread Otavio Salvador
This includes following upstream bug fixes:

Bug 77240 - khrplatform.h not installed if EGL is disabled
Bug 95530 - Stellaris - colored overlay of sectors doesn't render on i965
Bug 96958 - [SKL] Improper rendering in Europa Universalis IV
Bug 99467 - [radv] DOOM 2016 + wine. Green screen everywhere (but can be 
started)
Bug 101071 - compiling glsl fails with undefined reference to `pthread_create'
Bug 101252 - eglGetDisplay() is not thread safe
Bug 101294 - radeonsi minecraft forge splash freeze since 17.1
Bug 101451 - [G33] ES2-CTS.functional.clipping.polygon regression

Signed-off-by: Otavio Salvador 
---

Changes in v2: None

 meta/recipes-graphics/mesa/{mesa-gl_17.1.3.bb => mesa-gl_17.1.4.bb} | 0
 meta/recipes-graphics/mesa/{mesa_17.1.3.bb => mesa_17.1.4.bb}   | 4 ++--
 2 files changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-graphics/mesa/{mesa-gl_17.1.3.bb => mesa-gl_17.1.4.bb} 
(100%)
 rename meta/recipes-graphics/mesa/{mesa_17.1.3.bb => mesa_17.1.4.bb} (84%)

diff --git a/meta/recipes-graphics/mesa/mesa-gl_17.1.3.bb 
b/meta/recipes-graphics/mesa/mesa-gl_17.1.4.bb
similarity index 100%
rename from meta/recipes-graphics/mesa/mesa-gl_17.1.3.bb
rename to meta/recipes-graphics/mesa/mesa-gl_17.1.4.bb
diff --git a/meta/recipes-graphics/mesa/mesa_17.1.3.bb 
b/meta/recipes-graphics/mesa/mesa_17.1.4.bb
similarity index 84%
rename from meta/recipes-graphics/mesa/mesa_17.1.3.bb
rename to meta/recipes-graphics/mesa/mesa_17.1.4.bb
index d9860213f2..2a1ecd3124 100644
--- a/meta/recipes-graphics/mesa/mesa_17.1.3.bb
+++ b/meta/recipes-graphics/mesa/mesa_17.1.4.bb
@@ -7,8 +7,8 @@ SRC_URI = 
"https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \
file://0002-hardware-gloat.patch \
 "
 
-SRC_URI[md5sum] = "1946a93d543bc219427e2bebe2ac4752"
-SRC_URI[sha256sum] = 
"5f1ee9a8aea2880f887884df2dea0c16dd1b13eb42fd2e52265db0dc1b380e8c"
+SRC_URI[md5sum] = "be2ef7c9edec23b07f74f6512a6a6fa5"
+SRC_URI[sha256sum] = 
"06f3b0e6a28f0d20b7f3391cf67fe89ae98ecd0a686cd545da76557b6cec9cad"
 
 #because we cannot rely on the fact that all apps will use pkgconfig,
 #make eglplatform.h independent of MESA_EGL_NO_X11_HEADER
-- 
2.13.2

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


[OE-core] [PATCH v2 1/2] mesa: Avoid platform probing when building without EGL

2017-07-03 Thread Otavio Salvador
The 17.1.2 release has changed the platform setting and when not
explicitly disabled it assumes x11 support.

Fixes:

| checking for x11-xcb xcb xcb-dri2 >= 1.8 xcb-xfixes... no
| configure: error: Package requirements (x11-xcb xcb xcb-dri2 >= 1.8 
xcb-xfixes) were not met:
|
| No package 'x11-xcb' found
| No package 'xcb' found
| No package 'xcb-dri2' found
| No package 'xcb-xfixes' found
|
| Consider adjusting the PKG_CONFIG_PATH environment variable if you
| installed software in a non-standard prefix.
|
| Alternatively, you may set the environment variables XCB_DRI2_CFLAGS
| and XCB_DRI2_LIBS to avoid the need to call pkg-config.
| See the pkg-config man page for more details.

The issue has been exposed by meta-freescale BSP. Fix tested with
imx6qsabresd machine.

Signed-off-by: Otavio Salvador 
---

Changes in v2:
- new patch

 meta/recipes-graphics/mesa/mesa.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/mesa/mesa.inc 
b/meta/recipes-graphics/mesa/mesa.inc
index 41747cffc8..a12ab7ab5b 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -49,7 +49,7 @@ PACKAGECONFIG[gles] = "--enable-gles1 --enable-gles2, 
--disable-gles1 --disable-
 EGL_PLATFORMS  = "drm"
 EGL_PLATFORMS .="${@bb.utils.contains('PACKAGECONFIG', 'x11', ',x11', '', d)}"
 EGL_PLATFORMS .="${@bb.utils.contains('PACKAGECONFIG', 'wayland', ',wayland', 
'', d)}"
-PACKAGECONFIG[egl] = "--enable-egl --with-platforms=${EGL_PLATFORMS}, 
--disable-egl"
+PACKAGECONFIG[egl] = "--enable-egl --with-platforms=${EGL_PLATFORMS}, 
--disable-egl --with-platforms=''"
 
 PACKAGECONFIG[etnaviv] = ""
 PACKAGECONFIG[imx] = ""
-- 
2.13.2

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


Re: [OE-core] [PATCH] oeqa/sdk: Replace buildiptables for buildgzip tests

2017-07-03 Thread Khem Raj
On Mon, Jul 3, 2017 at 12:25 PM Jose Perez Carranza <
jose.perez.carra...@linux.intel.com> wrote:

> On 07/03/2017 11:59 AM, Khem Raj wrote:
> > On Mon, Jul 3, 2017 at 9:26 AM,  
> wrote:
> >> From: Jose Perez Carranza 
> >>
> >> Buildiptables test cases are conflicting with images built with “musl”
> >> as standard C library, in order to avoid those issues gzip package was
> >> selected to be used on the tests as this does not conflicts with “musl”.
> >>
> >> [YOCTO #11713]
> > I see that gzip recipes also have musl appends for do_configure, I
> > understand, that this test case will compile on device so it will get
> > hosted env and it might be working fine. I would think it will be good
> > to use a package which we dont patch even in OE
> I did some test also suing "Lzip", I did not see any reference to "musl"
> on that recipe, did you see any problem using "lzip"instead "gzip"?


Lzip should be ok


>
> Regards
> José
> >
> >> Signed-off-by: Jose Perez Carranza  >
> >> ---
> >>   meta/lib/oeqa/sdk/cases/{buildiptables.py => buildgzip.py} | 8
> 
> >>   1 file changed, 4 insertions(+), 4 deletions(-)
> >>   rename meta/lib/oeqa/sdk/cases/{buildiptables.py => buildgzip.py}
> (84%)
> >>
> >> diff --git a/meta/lib/oeqa/sdk/cases/buildiptables.py
> b/meta/lib/oeqa/sdk/cases/buildgzip.py
> >> similarity index 84%
> >> rename from meta/lib/oeqa/sdk/cases/buildiptables.py
> >> rename to meta/lib/oeqa/sdk/cases/buildgzip.py
> >> index 419c7ebd09b..a92df65c28b 100644
> >> --- a/meta/lib/oeqa/sdk/cases/buildiptables.py
> >> +++ b/meta/lib/oeqa/sdk/cases/buildgzip.py
> >> @@ -3,15 +3,15 @@ from oeqa.sdk.case import OESDKTestCase
> >>   from oeqa.sdk.utils.sdkbuildproject import SDKBuildProject
> >>
> >>
> >> -class BuildIptablesTest(OESDKTestCase):
> >> +class BuildGzipTest(OESDKTestCase):
> >>   td_vars = ['DATETIME']
> >>
> >>   @classmethod
> >>   def setUpClass(self):
> >>   dl_dir = self.td.get('DL_DIR', None)
> >>
> >> -self.project = SDKBuildProject(self.tc.sdk_dir + "/iptables/",
> self.tc.sdk_env,
> >> -"
> http://downloads.yoctoproject.org/mirror/sources/iptables-1.4.13.tar.bz2";,
> >> +self.project = SDKBuildProject(self.tc.sdk_dir + "/gzip/",
> self.tc.sdk_env,
> >> +"
> http://downloads.yoctoproject.org/mirror/sources/gzip-1.8.tar.gz";,
> >>   self.tc.sdk_dir, self.td['DATETIME'],
> dl_dir=dl_dir)
> >>   self.project.download_archive()
> >>
> >> @@ -21,7 +21,7 @@ class BuildIptablesTest(OESDKTestCase):
> >>   self.tc.hasTargetPackage("gcc")):
> >>   raise unittest.SkipTest("SDK doesn't contain a
> cross-canadian toolchain")
> >>
> >> -def test_iptables(self):
> >> +def test_gzip(self):
> >>   self.assertEqual(self.project.run_configure(), 0,
> >>   msg="Running configure failed")
> >>
> >> --
> >> 2.12.0
> >>
>
> --
> Saludos
> José
>
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] oeqa/sdk: Replace buildiptables for buildgzip tests

2017-07-03 Thread Jose Perez Carranza

On 07/03/2017 11:59 AM, Khem Raj wrote:

On Mon, Jul 3, 2017 at 9:26 AM,   wrote:

From: Jose Perez Carranza 

Buildiptables test cases are conflicting with images built with “musl”
as standard C library, in order to avoid those issues gzip package was
selected to be used on the tests as this does not conflicts with “musl”.

[YOCTO #11713]

I see that gzip recipes also have musl appends for do_configure, I
understand, that this test case will compile on device so it will get
hosted env and it might be working fine. I would think it will be good
to use a package which we dont patch even in OE
I did some test also suing "Lzip", I did not see any reference to "musl" 
on that recipe, did you see any problem using "lzip"instead "gzip"?


Regards
José



Signed-off-by: Jose Perez Carranza 
---
  meta/lib/oeqa/sdk/cases/{buildiptables.py => buildgzip.py} | 8 
  1 file changed, 4 insertions(+), 4 deletions(-)
  rename meta/lib/oeqa/sdk/cases/{buildiptables.py => buildgzip.py} (84%)

diff --git a/meta/lib/oeqa/sdk/cases/buildiptables.py 
b/meta/lib/oeqa/sdk/cases/buildgzip.py
similarity index 84%
rename from meta/lib/oeqa/sdk/cases/buildiptables.py
rename to meta/lib/oeqa/sdk/cases/buildgzip.py
index 419c7ebd09b..a92df65c28b 100644
--- a/meta/lib/oeqa/sdk/cases/buildiptables.py
+++ b/meta/lib/oeqa/sdk/cases/buildgzip.py
@@ -3,15 +3,15 @@ from oeqa.sdk.case import OESDKTestCase
  from oeqa.sdk.utils.sdkbuildproject import SDKBuildProject


-class BuildIptablesTest(OESDKTestCase):
+class BuildGzipTest(OESDKTestCase):
  td_vars = ['DATETIME']

  @classmethod
  def setUpClass(self):
  dl_dir = self.td.get('DL_DIR', None)

-self.project = SDKBuildProject(self.tc.sdk_dir + "/iptables/", 
self.tc.sdk_env,
-
"http://downloads.yoctoproject.org/mirror/sources/iptables-1.4.13.tar.bz2";,
+self.project = SDKBuildProject(self.tc.sdk_dir + "/gzip/", 
self.tc.sdk_env,
+
"http://downloads.yoctoproject.org/mirror/sources/gzip-1.8.tar.gz";,
  self.tc.sdk_dir, self.td['DATETIME'], dl_dir=dl_dir)
  self.project.download_archive()

@@ -21,7 +21,7 @@ class BuildIptablesTest(OESDKTestCase):
  self.tc.hasTargetPackage("gcc")):
  raise unittest.SkipTest("SDK doesn't contain a cross-canadian 
toolchain")

-def test_iptables(self):
+def test_gzip(self):
  self.assertEqual(self.project.run_configure(), 0,
  msg="Running configure failed")

--
2.12.0



--
Saludos
José

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


Re: [OE-core] [PATCH] oeqa/sdk: Replace buildiptables for buildgzip tests

2017-07-03 Thread Khem Raj
On Mon, Jul 3, 2017 at 9:26 AM,   wrote:
> From: Jose Perez Carranza 
>
> Buildiptables test cases are conflicting with images built with “musl”
> as standard C library, in order to avoid those issues gzip package was
> selected to be used on the tests as this does not conflicts with “musl”.
>
> [YOCTO #11713]

I see that gzip recipes also have musl appends for do_configure, I
understand, that this test case will compile on device so it will get
hosted env and it might be working fine. I would think it will be good
to use a package which we dont patch even in OE

>
> Signed-off-by: Jose Perez Carranza 
> ---
>  meta/lib/oeqa/sdk/cases/{buildiptables.py => buildgzip.py} | 8 
>  1 file changed, 4 insertions(+), 4 deletions(-)
>  rename meta/lib/oeqa/sdk/cases/{buildiptables.py => buildgzip.py} (84%)
>
> diff --git a/meta/lib/oeqa/sdk/cases/buildiptables.py 
> b/meta/lib/oeqa/sdk/cases/buildgzip.py
> similarity index 84%
> rename from meta/lib/oeqa/sdk/cases/buildiptables.py
> rename to meta/lib/oeqa/sdk/cases/buildgzip.py
> index 419c7ebd09b..a92df65c28b 100644
> --- a/meta/lib/oeqa/sdk/cases/buildiptables.py
> +++ b/meta/lib/oeqa/sdk/cases/buildgzip.py
> @@ -3,15 +3,15 @@ from oeqa.sdk.case import OESDKTestCase
>  from oeqa.sdk.utils.sdkbuildproject import SDKBuildProject
>
>
> -class BuildIptablesTest(OESDKTestCase):
> +class BuildGzipTest(OESDKTestCase):
>  td_vars = ['DATETIME']
>
>  @classmethod
>  def setUpClass(self):
>  dl_dir = self.td.get('DL_DIR', None)
>
> -self.project = SDKBuildProject(self.tc.sdk_dir + "/iptables/", 
> self.tc.sdk_env,
> -
> "http://downloads.yoctoproject.org/mirror/sources/iptables-1.4.13.tar.bz2";,
> +self.project = SDKBuildProject(self.tc.sdk_dir + "/gzip/", 
> self.tc.sdk_env,
> +
> "http://downloads.yoctoproject.org/mirror/sources/gzip-1.8.tar.gz";,
>  self.tc.sdk_dir, self.td['DATETIME'], dl_dir=dl_dir)
>  self.project.download_archive()
>
> @@ -21,7 +21,7 @@ class BuildIptablesTest(OESDKTestCase):
>  self.tc.hasTargetPackage("gcc")):
>  raise unittest.SkipTest("SDK doesn't contain a cross-canadian 
> toolchain")
>
> -def test_iptables(self):
> +def test_gzip(self):
>  self.assertEqual(self.project.run_configure(), 0,
>  msg="Running configure failed")
>
> --
> 2.12.0
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] oeqa/sdk: Replace buildiptables for buildgzip tests

2017-07-03 Thread jose . perez . carranza
From: Jose Perez Carranza 

Buildiptables test cases are conflicting with images built with “musl”
as standard C library, in order to avoid those issues gzip package was
selected to be used on the tests as this does not conflicts with “musl”.

[YOCTO #11713]

Signed-off-by: Jose Perez Carranza 
---
 meta/lib/oeqa/sdk/cases/{buildiptables.py => buildgzip.py} | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)
 rename meta/lib/oeqa/sdk/cases/{buildiptables.py => buildgzip.py} (84%)

diff --git a/meta/lib/oeqa/sdk/cases/buildiptables.py 
b/meta/lib/oeqa/sdk/cases/buildgzip.py
similarity index 84%
rename from meta/lib/oeqa/sdk/cases/buildiptables.py
rename to meta/lib/oeqa/sdk/cases/buildgzip.py
index 419c7ebd09b..a92df65c28b 100644
--- a/meta/lib/oeqa/sdk/cases/buildiptables.py
+++ b/meta/lib/oeqa/sdk/cases/buildgzip.py
@@ -3,15 +3,15 @@ from oeqa.sdk.case import OESDKTestCase
 from oeqa.sdk.utils.sdkbuildproject import SDKBuildProject
 
 
-class BuildIptablesTest(OESDKTestCase):
+class BuildGzipTest(OESDKTestCase):
 td_vars = ['DATETIME']
 
 @classmethod
 def setUpClass(self):
 dl_dir = self.td.get('DL_DIR', None)
 
-self.project = SDKBuildProject(self.tc.sdk_dir + "/iptables/", 
self.tc.sdk_env, 
-
"http://downloads.yoctoproject.org/mirror/sources/iptables-1.4.13.tar.bz2";,
+self.project = SDKBuildProject(self.tc.sdk_dir + "/gzip/", 
self.tc.sdk_env,
+
"http://downloads.yoctoproject.org/mirror/sources/gzip-1.8.tar.gz";,
 self.tc.sdk_dir, self.td['DATETIME'], dl_dir=dl_dir)
 self.project.download_archive()
 
@@ -21,7 +21,7 @@ class BuildIptablesTest(OESDKTestCase):
 self.tc.hasTargetPackage("gcc")):
 raise unittest.SkipTest("SDK doesn't contain a cross-canadian 
toolchain")
 
-def test_iptables(self):
+def test_gzip(self):
 self.assertEqual(self.project.run_configure(), 0,
 msg="Running configure failed")
 
-- 
2.12.0

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


Re: [OE-core] [PATCH v2 1/3] gcc-6.3: Fix libgfortran build

2017-07-03 Thread Khem Raj
On Mon, Jul 3, 2017 at 8:22 AM, Burton, Ross  wrote:
>
> On 30 June 2017 at 10:25, Ricardo Ribalda Delgado
>  wrote:
>>
>> --- a/meta/recipes-devtools/gcc/gcc-6.3.inc
>> +++ b/meta/recipes-devtools/gcc/gcc-6.3.inc
>
>
> Is this fixed in gcc 7 (in which case the upstream-status is backport), or
> is gcc 7.1 still broken?

this in needed on gcc 7.1 as well.

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


Re: [OE-core] [PATCH 1/3] gcc-6.3: Fix libgfortran build

2017-07-03 Thread Khem Raj
On Fri, Jun 30, 2017 at 1:51 AM, Ricardo Ribalda Delgado
 wrote:
> Without this patch libbacktrace cannot be included/link during
> libgfortran build.
>
> libbtool: link: cannot find the library `../libbacktrace/libbacktrace.la'
> or unhandled argument `../libbacktrace/libbacktrace.la'
>
> Signed-off-by: Ricardo Ribalda Delgado 
> ---
>  meta/recipes-devtools/gcc/gcc-6.3.inc  |  1 +
>  .../gcc/gcc-6.3/0099-Fix-gfortran-build.patch  | 44 
> ++
>  2 files changed, 45 insertions(+)
>  create mode 100644 
> meta/recipes-devtools/gcc/gcc-6.3/0099-Fix-gfortran-build.patch
>
> diff --git a/meta/recipes-devtools/gcc/gcc-6.3.inc 
> b/meta/recipes-devtools/gcc/gcc-6.3.inc
> index 86b3e9ff19a4..36a116d48fb4 100644
> --- a/meta/recipes-devtools/gcc/gcc-6.3.inc
> +++ b/meta/recipes-devtools/gcc/gcc-6.3.inc
> @@ -74,6 +74,7 @@ SRC_URI = "\
> 
> file://0047-libgcc_s-Use-alias-for-__cpu_indicator_init-instead-.patch \
> file://0048-sync-gcc-stddef.h-with-musl.patch \
> file://0054_all_nopie-all-flags.patch \
> +   file://0099-Fix-gfortran-build.patch \

call it 0055 for legibility sake.

> ${BACKPORTS} \
>  "
>  BACKPORTS = "\
> diff --git a/meta/recipes-devtools/gcc/gcc-6.3/0099-Fix-gfortran-build.patch 
> b/meta/recipes-devtools/gcc/gcc-6.3/0099-Fix-gfortran-build.patch
> new file mode 100644
> index ..32b2f9a0567f
> --- /dev/null
> +++ b/meta/recipes-devtools/gcc/gcc-6.3/0099-Fix-gfortran-build.patch
> @@ -0,0 +1,44 @@
> +diff --git a/libgfortran/Makefile.am b/libgfortran/Makefile.am
> +index 39d3e11d223c..b5efe6dfe195 100644
> +--- a/libgfortran/Makefile.am
>  b/libgfortran/Makefile.am
> +@@ -37,7 +37,7 @@ toolexeclib_LTLIBRARIES = libgfortran.la
> + toolexeclib_DATA = libgfortran.spec
> + libgfortran_la_LINK = $(LINK) $(libgfortran_la_LDFLAGS)
> + libgfortran_la_LDFLAGS = -version-info `grep -v '^\#' 
> $(srcdir)/libtool-version` \
> +-  $(LTLDFLAGS) $(LIBQUADLIB) ../libbacktrace/libbacktrace.la \
> ++  $(LTLDFLAGS) $(LIBQUADLIB) ../../libbacktrace/libbacktrace.la \
> +   -lm $(extra_ldflags_libgfortran) \
> +   $(version_arg) -Wc,-shared-libgcc
> + libgfortran_la_DEPENDENCIES = $(version_dep) libgfortran.spec 
> $(LIBQUADLIB_DEP)
> +@@ -63,7 +63,7 @@ AM_CPPFLAGS = -iquote$(srcdir)/io 
> -I$(srcdir)/$(MULTISRCTOP)../gcc \
> + -I$(MULTIBUILDTOP)../libgcc \
> + -I$(srcdir)/$(MULTISRCTOP)../libbacktrace \
> + -I$(MULTIBUILDTOP)../libbacktrace \
> +--I../libbacktrace
> ++-I../../libbacktrace
> +
> + # Fortran rules for complex multiplication and division
> + AM_CFLAGS += -fcx-fortran-rules
> +diff --git a/libgfortran/Makefile.in b/libgfortran/Makefile.in
> +index 7ed080cf7b0d..d197fbd6b683 100644
> +--- a/libgfortran/Makefile.in
>  b/libgfortran/Makefile.in
> +@@ -598,7 +598,7 @@ toolexeclib_LTLIBRARIES = libgfortran.la
> + toolexeclib_DATA = libgfortran.spec
> + libgfortran_la_LINK = $(LINK) $(libgfortran_la_LDFLAGS)
> + libgfortran_la_LDFLAGS = -version-info `grep -v '^\#' 
> $(srcdir)/libtool-version` \
> +-  $(LTLDFLAGS) $(LIBQUADLIB) ../libbacktrace/libbacktrace.la \
> ++  $(LTLDFLAGS) $(LIBQUADLIB) ../../libbacktrace/libbacktrace.la \
> +   -lm $(extra_ldflags_libgfortran) \
> +   $(version_arg) -Wc,-shared-libgcc
> +
> +@@ -618,7 +618,7 @@ AM_CPPFLAGS = -iquote$(srcdir)/io 
> -I$(srcdir)/$(MULTISRCTOP)../gcc \
> + -I$(MULTIBUILDTOP)../libgcc \
> + -I$(srcdir)/$(MULTISRCTOP)../libbacktrace \
> + -I$(MULTIBUILDTOP)../libbacktrace \
> +--I../libbacktrace
> ++-I../../libbacktrace
> +
> + gfor_io_src = io/size_from_kind.c $(am__append_2)
> + gfor_io_headers = \
> --
> 2.11.0
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] State of bitbake world, Failed tasks 2017-07-02

2017-07-03 Thread Khem Raj
On Mon, Jul 3, 2017 at 4:20 AM, Martin Jansa  wrote:
> http://www.openembedded.org/wiki/Bitbake_World_Status
>
> == Number of issues - stats ==
> {| class='wikitable'
> !|Date !!colspan='3'|Failed tasks!!|Signatures
> !!colspan='14'|QA !!Comment
> |-
> || ||qemuarm ||qemux86 ||qemux86_64 ||all
> ||already-stripped ||libdir ||textrel ||build-deps ||file-rdeps 
> ||version-going-backwards ||host-user-contaminated ||installed-vs-shipped 
> ||unknown-configure-option ||symlink-to-sysroot ||invalid-pkgconfig ||pkgname 
> ||ldflags ||compile-host-path ||
> |-
> ||2017-07-02 ||7 ||8 ||5 ||0 ||0 ||1 ||4 ||0 ||0 ||1238 ||1 ||0 ||0 ||0 ||0 
> ||0 ||0 ||0 ||
> |}
>
> == Failed tasks 2017-07-02 ==
>
> INFO: jenkins-job.sh-1.8.24 Complete log available at
> http://logs.nslu2-linux.org/buildlogs/oe/world/rocko/log.report.20170702_225940.log
>
> === common (3) ===
> *
> meta-openembedded/meta-multimedia/recipes-mediacenter/kodi/kodi_17.bb:do_configure
> *
> meta-openembedded/meta-oe/recipes-extended/fluentbit/fluentbit_0.11.11.bb:do_package
> *
> virtual:native:meta-qt5/recipes-qt/qt5/qtxmlpatterns_git.bb:do_populate_sysroot
>
> === common-x86 (2) ===
> * meta-qt5/recipes-qt/qt5/qtwebengine_git.bb:do_compile
> *
> openembedded-core/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb:do_compile_ptest_base
>
> === qemuarm (4) ===
> *
> meta-openembedded/meta-oe/recipes-graphics/gegl/gegl_0.3.18.bb:do_fetch
> *
> meta-openembedded/meta-oe/recipes-support/synergy/synergy_git.bb:do_fetch
> *

these worked ok for me. Seems to be transitory failures.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] systemd: Do not use xlocale.h

2017-07-03 Thread Khem Raj
glibc specific header which has been removed from glibc 2.26+

Signed-off-by: Khem Raj 
---
 ...se-util-Do-not-include-unneeded-xlocale.h.patch | 27 ++
 meta/recipes-core/systemd/systemd_232.bb   |  3 ++-
 2 files changed, 29 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-core/systemd/systemd/0022-parse-util-Do-not-include-unneeded-xlocale.h.patch

diff --git 
a/meta/recipes-core/systemd/systemd/0022-parse-util-Do-not-include-unneeded-xlocale.h.patch
 
b/meta/recipes-core/systemd/systemd/0022-parse-util-Do-not-include-unneeded-xlocale.h.patch
new file mode 100644
index 00..cad1867a4c
--- /dev/null
+++ 
b/meta/recipes-core/systemd/systemd/0022-parse-util-Do-not-include-unneeded-xlocale.h.patch
@@ -0,0 +1,27 @@
+From 0c50b8332092178378257c2de800fb6b6b4a8706 Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Mon, 3 Jul 2017 08:45:04 -0700
+Subject: [PATCH] parse-util: Do not include unneeded xlocale.h
+
+Signed-off-by: Khem Raj 
+---
+Upstream-Status: Backport [ partial 
https://github.com/systemd/systemd/commit/284d1cd0a12cad96a5ea61d1afb0dd677dbd147e]
+
+ src/basic/parse-util.c | 1 -
+ 1 file changed, 1 deletion(-)
+
+diff --git a/src/basic/parse-util.c b/src/basic/parse-util.c
+index c98815b9b..a0eb45805 100644
+--- a/src/basic/parse-util.c
 b/src/basic/parse-util.c
+@@ -23,7 +23,6 @@
+ #include 
+ #include 
+ #include 
+-#include 
+ 
+ #include "alloc-util.h"
+ #include "extract-word.h"
+-- 
+2.13.2
+
diff --git a/meta/recipes-core/systemd/systemd_232.bb 
b/meta/recipes-core/systemd/systemd_232.bb
index d54e92707f..287e07b153 100644
--- a/meta/recipes-core/systemd/systemd_232.bb
+++ b/meta/recipes-core/systemd/systemd_232.bb
@@ -31,7 +31,8 @@ SRC_URI += " \

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

file://0020-back-port-233-don-t-use-the-unified-hierarchy-for-the-systemd.patch 
\

file://0021-build-sys-check-for-lz4-in-the-old-and-new-numbering.patch \
-"
+   file://0022-parse-util-Do-not-include-unneeded-xlocale.h.patch \
+   "
 SRC_URI_append_qemuall = " 
file://0001-core-device.c-Change-the-default-device-timeout-to-2.patch"
 
 PACKAGECONFIG ??= "xz \
-- 
2.13.2

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


Re: [OE-core] [PATCH v2 1/3] gcc-6.3: Fix libgfortran build

2017-07-03 Thread Ricardo Ribalda Delgado
Hi Ross


On 3 Jul 2017 5:22 pm, "Burton, Ross"  wrote:

>
> On 30 June 2017 at 10:25, Ricardo Ribalda Delgado <
> ricardo.riba...@gmail.com> wrote:
>
>> --- a/meta/recipes-devtools/gcc/gcc-6.3.inc
>> +++ b/meta/recipes-devtools/gcc/gcc-6.3.inc
>>
>
> Is this fixed in gcc 7 (in which case the upstream-status is backport), or
> is gcc 7.1 still broken?
>

I do not know, at qtec we are following yocto, and currently it is at
version 6.3.

 I could try to build it with gcc7 during this week. and send another patch
if needed.

Regards!


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


Re: [OE-core] [PATCH v2 1/3] gcc-6.3: Fix libgfortran build

2017-07-03 Thread Burton, Ross
On 30 June 2017 at 10:25, Ricardo Ribalda Delgado  wrote:

> --- a/meta/recipes-devtools/gcc/gcc-6.3.inc
> +++ b/meta/recipes-devtools/gcc/gcc-6.3.inc
>

Is this fixed in gcc 7 (in which case the upstream-status is backport), or
is gcc 7.1 still broken?

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


Re: [OE-core] [PATCH] openssh: Fix key generation with systemd

2017-07-03 Thread Burton, Ross
On 29 June 2017 at 13:30, Joshua Watt  wrote:

> +   install -D -m 0755 ${WORKDIR}/sshd_check_keys
> ${D}${libexecdir}/${BPN}/sshd_check_keys
>

packages/corei7-64-poky-linux/openssh/openssh:
  PKGSIZE changed from 0 to 2423 (+100%)
  FILELIST: added "/usr/libexec/openssh/sshd_check_keys"

Can you pthat binary in the openssh-sshd package instead of the previously
empty openssh?

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


Re: [OE-core] [PATCH 2/2] kernel: user defined KERNEL_VERSION_PKG_NAME

2017-07-03 Thread Razvan Heghedus



On 06/29/2017 04:06 PM, Bruce Ashfield wrote:



On Thu, Jun 29, 2017 at 5:10 AM, Razvan Heghedus 
mailto:razvan.heghe...@ni.com>> wrote:




On 06/28/2017 04:29 AM, Bruce Ashfield wrote:



On Tue, Jun 27, 2017 at 5:15 AM, Razvan Heghedus
mailto:razvan.heghe...@ni.com>> wrote:



On 06/26/2017 06:52 PM, Bruce Ashfield wrote:



On Wed, Jun 21, 2017 at 8:00 AM, Heghedus Razvan
mailto:razvan.heghe...@ni.com>> wrote:

Add possibility to set KERNEL_VERSION_PKG_NAME to a user
defined value.

Signed-off-by: Heghedus Razvan mailto:razvan.heghe...@ni.com>>
---
 meta/classes/kernel.bbclass | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/meta/classes/kernel.bbclass
b/meta/classes/kernel.bbclass
index 605c101e62..02728d5a86 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -28,12 +28,16 @@ INITRAMFS_IMAGE_BUNDLE ?= ""
 # LINUX_VERSION which is a constant.
 KERNEL_VERSION_NAME = "${@d.getVar('KERNEL_VERSION') or
" "}"
 KERNEL_VERSION_NAME[vardepvalue] = "${LINUX_VERSION}"
-KERNEL_VERSION_PKG_NAME =
"${@legitimize_package_name(d.getVar('KERNEL_VERSION'))}"
-KERNEL_VERSION_PKG_NAME[vardepvalue] = "${LINUX_VERSION}"

 python __anonymous () {
 import re

+if d.getVar('USER_KERNEL_VERSION_PKG') is None :
+ d.setVar('KERNEL_VERSION_PKG_NAME',
"${@legitimize_package_name(d.getVar('KERNEL_VERSION'))}")
+ d.setVar('KERNEL_VERSION_PKG_NAME[vardepvalue]',
"${LINUX_VERSION}")
+else:
+ d.setVar('KERNEL_VERSION_PKG_NAME',
"${@legitimize_package_name(d.getVar('USER_KERNEL_VERSION_PKG'))}")


This is introducing yet another variable that tweaks the
already complex setting of
the kernel version. Not to mention this code is already
touchy with respect to
parse time and rebuilding of the kernel.

My concern is that if this is set, we are completely
disassociated with the source
code of the kernel.

Where did you think this would be set ? local.conf ? distro
config ? somewhere else ?

If we had a way to simply override KERNEL_VERSION, we
wouldn't need any extra
variables.

Bruce

+
 # Merge KERNEL_IMAGETYPE and KERNEL_ALT_IMAGETYPE
into KERNEL_IMAGETYPES
 type = d.getVar('KERNEL_IMAGETYPE') or ""
 alttype = d.getVar('KERNEL_ALT_IMAGETYPE') or ""
--
2.13.1

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

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





-- 
"Thou shalt not follow the NULL pointer, for chaos and

madness await thee at its end"

I have setting the variable in the kernel recipe. I need a
way to override the KERNEL_VERSION because I want the kernel
packages name to contain only a part of the version or
nothing at all.
I need this for the the kernel upgrade stuff, because if the
package name is something like: kernel-4.9.8-{static_string}
then I couldn't upgrade to a version like:
kernel-4.9.10-{static_string}, because they are two different
packages. I wanted a simple way to be able to have the
package name : kernel-4.9-{static_string}, then I could do
the upgrade for the new minor updates of the kernel.


I could have sworn this (upgrading) was already possible via the
version string we
are currently using.  i.e. the PV is already picked up from the
kernel source, and
that should be doing the job.

i.e. when I unpack my

kernel-image-bzimage-4.10.15-yocto-standard_4.10.15+git0+4d929fac34_d2c1ed3c0c-r0_qemux86_64.ipk
package, I see that it has:

 Version: 4.10.15+git0+4d929fac34_d2c1ed3c0c-r0
 but obviously has the general provides: Provides:
kernel-image-bzimage

So that should be upgradable based on the version ... sure they
have different names, but the provides
and versions take care of things.

The versioning, ability to install multiple kernels, upgrades,
etc, have really churned
these variables making them a mess to read.

I'm probably misunderstanding your use case and error, can you
elaborate for me
and/or provide a log ? I'm more of a ker

Re: [OE-core] [PATCH] bootimg-efi.py: Use IMGDEPLOYDIR instead of DEPLOY_DIR_IMAGE for initrd

2017-07-03 Thread Patrick Ohly
On Mon, 2017-07-03 at 15:44 +0300, Ed Bartosh wrote:
> On Mon, Jul 03, 2017 at 11:27:53AM +0200, Patrick Ohly wrote:
> > On Mon, 2017-07-03 at 11:36 +0300, Ed Bartosh wrote:
> > > On Fri, Jun 30, 2017 at 10:53:30AM -0700, Alejandro Hernandez wrote:
> > > > When using wic to create an image from a certain build, wic is expecting
> > > > to find initrd at the final destination of our images 
> > > > (DEPLOY_DIR_IMAGE),
> > > > which is wrong, since the initrd file has not been copied to the final
> > > > directory yet,
> > > 
> > > Is it possible to ensure that initrd is deployed before wic is run by
> > > making do_image_wic depend on initrd deploy task?
> > 
> > Not in this case, because both do_image_wic and do_image_cpio share the
> > same deploy task.
> 
> Now I understand what's going on here. Basically Alejandro is trying to
> use one format of the same image in another. It's like trying to put
> .wic image inside ext2 image or something similar.
> 
> I believe it doesn't fit current design of oe image creation system.
> I'd suggest to use different image recipes to achieve this.

Yes, that is also my preferred solution (see other email from this
morning).

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.



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


Re: [OE-core] [PATCH] cairo: fix QA Issue

2017-07-03 Thread Burton, Ross
On 3 July 2017 at 13:34, Gordan Markuš  wrote:

> -FILES_${PN}-dev += "${libdir}/cairo/*.so"
> +FILES_${PN}-dev += "${libdir}/cairo ${libdir}/cairo/*.so"
>

I just spent five minutes working out *why* this fixes the QA issue you
were getting.

A better fix would be to delete ${libdir}/cairo using:

  rmdir ${libdir}/cairo || :

or

  rmdir --ignore-fail-on-non-empty ${libdir}/cairo

(former is POSIX compliant, but I can't think of a situation where that is
only a hypothetical problem)

Then in the commit message you can say that the directory is always created
but it only has content if you disable cairo-trace.  At least, I hope you
are explicitly disabling cairo-trace as there's no other way I can see for
you go get this.

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


Re: [OE-core] [PATCH] bootimg-efi.py: Use IMGDEPLOYDIR instead of DEPLOY_DIR_IMAGE for initrd

2017-07-03 Thread Ed Bartosh
On Mon, Jul 03, 2017 at 11:27:53AM +0200, Patrick Ohly wrote:
> On Mon, 2017-07-03 at 11:36 +0300, Ed Bartosh wrote:
> > On Fri, Jun 30, 2017 at 10:53:30AM -0700, Alejandro Hernandez wrote:
> > > When using wic to create an image from a certain build, wic is expecting
> > > to find initrd at the final destination of our images (DEPLOY_DIR_IMAGE),
> > > which is wrong, since the initrd file has not been copied to the final
> > > directory yet,
> > 
> > Is it possible to ensure that initrd is deployed before wic is run by
> > making do_image_wic depend on initrd deploy task?
> 
> Not in this case, because both do_image_wic and do_image_cpio share the
> same deploy task.

Now I understand what's going on here. Basically Alejandro is trying to
use one format of the same image in another. It's like trying to put
.wic image inside ext2 image or something similar.

I believe it doesn't fit current design of oe image creation system.
I'd suggest to use different image recipes to achieve this.

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


[OE-core] [PATCH] cairo: fix QA Issue

2017-07-03 Thread Gordan Markuš
QA Issue: cairo: Files/directories were installed but not shipped in any
package: /usr/lib/cairo

Signed-off-by: Gordan Markuš 
---
 meta/recipes-graphics/cairo/cairo_1.14.8.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/cairo/cairo_1.14.8.bb 
b/meta/recipes-graphics/cairo/cairo_1.14.8.bb
index af6be56..9460191 100644
--- a/meta/recipes-graphics/cairo/cairo_1.14.8.bb
+++ b/meta/recipes-graphics/cairo/cairo_1.14.8.bb
@@ -31,7 +31,7 @@ to replay rendering."
 DESCRIPTION_cairo-perf-utils = "The Cairo library performance utilities"
 
 FILES_${PN} = "${libdir}/libcairo.so.*"
-FILES_${PN}-dev += "${libdir}/cairo/*.so"
+FILES_${PN}-dev += "${libdir}/cairo ${libdir}/cairo/*.so"
 FILES_${PN}-gobject = "${libdir}/libcairo-gobject.so.*"
 FILES_${PN}-script-interpreter = "${libdir}/libcairo-script-interpreter.so.*"
 FILES_${PN}-perf-utils = "${bindir}/cairo-trace ${libdir}/cairo/*.la 
${libdir}/cairo/libcairo-trace.so.*"
-- 
2.9.3

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


[OE-core] [PATCH v2] sanity.bbclass: fix AttributeError in mirror format checks

2017-07-03 Thread Mikko Ylinen
mirrors is a list after split() and results in:

AttributeError: 'list' object has no attribute 'strip'

when the 'mirror values are pairs' check fails.

Signed-off-by: Mikko Ylinen 
---

v2: drop redundant .strip() because it's guaranteed str(mirrors)
does not have leading or trailing spaces.

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

diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index e8064ac483..9e0f57ddfe 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -839,7 +839,7 @@ def check_sanity_everybuild(status, d):
 
 # Split into pairs
 if len(mirrors) % 2 != 0:
-bb.warn('Invalid mirror variable value for %s: %s, should contain 
paired members.' % (mirror_var, mirrors.strip()))
+bb.warn('Invalid mirror variable value for %s: %s, should contain 
paired members.' % (mirror_var, str(mirrors)))
 continue
 mirrors = list(zip(*[iter(mirrors)]*2))
 
-- 
2.13.2

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


[OE-core] [PATCH] ltp: syscalls/add_key02: fix for nonempty NULL payload

2017-07-03 Thread Fathi Boudra
Add upstream patch to fix syscalls/add_key02 test for nonempty
NULL payload:
https://github.com/linux-test-project/ltp/commit/25045624e941ee76a13febd36187e23c6c435507

Signed-off-by: Milosz Wasilewski 
Signed-off-by: Fathi Boudra 
---
 ...d_key02-update-to-test-fix-for-nonempty-N.patch | 136 +
 meta/recipes-extended/ltp/ltp_20170516.bb  |   1 +
 2 files changed, 137 insertions(+)
 create mode 100644 
meta/recipes-extended/ltp/ltp/0038-syscalls-add_key02-update-to-test-fix-for-nonempty-N.patch

diff --git 
a/meta/recipes-extended/ltp/ltp/0038-syscalls-add_key02-update-to-test-fix-for-nonempty-N.patch
 
b/meta/recipes-extended/ltp/ltp/0038-syscalls-add_key02-update-to-test-fix-for-nonempty-N.patch
new file mode 100644
index 00..3759e1fa9f
--- /dev/null
+++ 
b/meta/recipes-extended/ltp/ltp/0038-syscalls-add_key02-update-to-test-fix-for-nonempty-N.patch
@@ -0,0 +1,136 @@
+From 25045624e941ee76a13febd36187e23c6c435507 Mon Sep 17 00:00:00 2001
+From: Eric Biggers 
+Date: Mon, 12 Jun 2017 11:55:21 -0700
+Subject: [PATCH 01/39] syscalls/add_key02: update to test fix for nonempty
+ NULL payload
+
+add_key02 was supposed to be a "Basic test for the add_key() syscall",
+but it actually happened to test the obscure case of passing a NULL
+payload with nonzero length.  This case was mishandled by the kernel,
+which either returned EINVAL or crashed with a NULL pointer dereference,
+depending on the key type.  (The former applied to the test, as it used
+the "user" key type.)  The expected behavior in this case is that the
+syscall fail with EFAULT.
+
+Update the test to expect the fixed behavior from v4.12-rc5, and make
+the test more thorough by testing additional key types, including ones
+that caused a NULL pointer dereference in unfixed kernels.
+
+Signed-off-by: Eric Biggers 
+Upstream-Status: Accepted 
[https://github.com/linux-test-project/ltp/commit/25045624e941ee76a13febd36187e23c6c435507]
+---
+ testcases/kernel/syscalls/add_key/add_key02.c | 69 ---
+ 1 file changed, 51 insertions(+), 18 deletions(-)
+
+diff --git a/testcases/kernel/syscalls/add_key/add_key02.c 
b/testcases/kernel/syscalls/add_key/add_key02.c
+index 866800d6f..2e3308dcd 100644
+--- a/testcases/kernel/syscalls/add_key/add_key02.c
 b/testcases/kernel/syscalls/add_key/add_key02.c
+@@ -1,5 +1,6 @@
+ 
/**
+  * Copyright (c) Crackerjack Project., 2007 *
++ * Copyright (c) 2017 Google, Inc.
*
+  *  *
+  * This program is free software;  you can redistribute it and/or modify  
*
+  * it under the terms of the GNU General Public License as published by   
*
+@@ -18,10 +19,17 @@
+  
**/
+ 
+ /*
+- * Basic test for the add_key() syscall.
++ * Test that the add_key() syscall correctly handles a NULL payload with 
nonzero
++ * length.  Specifically, it should fail with EFAULT rather than oopsing the
++ * kernel with a NULL pointer dereference or failing with EINVAL, as it did
++ * before (depending on the key type).  This is a regression test for commit
++ * 5649645d725c ("KEYS: fix dereferencing NULL payload with nonzero length").
+  *
+- * History:   Porting from Crackerjack to LTP is done by
+- *  Manas Kumar Nayak makna...@in.ibm.com>
++ * Note that none of the key types that exhibited the NULL pointer dereference
++ * are guaranteed to be built into the kernel, so we just test as many as we
++ * can, in the hope of catching one.  We also test with the "user" key type 
for
++ * good measure, although it was one of the types that failed with EINVAL 
rather
++ * than dereferencing NULL.
+  */
+ 
+ #include "config.h"
+@@ -33,36 +41,61 @@
+ 
+ #ifdef HAVE_LINUX_KEYCTL_H
+ struct tcase {
+-  char *type;
+-  char *desc;
+-  void *payload;
+-  int plen;
+-  int exp_errno;
++  const char *type;
++  size_t plen;
+ } tcases[] = {
+-  {"user", "firstkey", NULL, 1, EINVAL}
++  /*
++   * The payload length we test for each key type needs to pass initial
++   * validation but is otherwise arbitrary.  Note: the "rxrpc_s" key type
++   * requires a payload of exactly 8 bytes.
++   */
++  { "asymmetric", 64 },
++  { "cifs.idmap", 64 },
++  { "cifs.spnego",64 },
++  { "pkcs7_test", 64 },
++  { "rxrpc",  64 },
++  { "rxrpc_s", 8 },
++  { "user",   64 },
+ };
+ #endif /* HAVE_LINUX_KEYCTL_H */
+ 
+ static void verify_add_key(unsigned int i)
+ {
+ #ifdef HAVE_LINUX_KEYCTL_H
+-  TEST(tst_syscall(__NR_add_key, tcases[i].type, tcases[i].desc,
+-   tcases[i].payload, tcases[i].plen,
+-   KEY_SPEC_USER_KEYRING));
+

Re: [OE-core] New Recipe for ser2net with IPv6 Support (v3.0 & v3.1)

2017-07-03 Thread Burton, Ross
On 30 June 2017 at 08:30, Thomas A. F. Thorne MEng AUS MIET <
thomas.tho...@net2edge.com> wrote:

> I am creating recipes for ser2net_3.0.bb and ser2net_3.1.bb as part of
> my testing.  I would be happy to submit the files as patches.  I am
> currently working on krogoth but could submit the patch against another
> version if preferred.
>
> I have CCed the meta-oe maintainers (on my first message attempt but not
> my second)
> as that the is the layer that holds the ser2net recipe at present.  I hope
> that was not a mistake, please
> for give the extra email if it was.
>

If the existing recipes are in meta-oe then submitting an upgrade to 3.1
(assuming thats the latest version) to meta-oe would be great.  This would
have to be against the master branch, and in general we don't like to
maintain many versions of each recipe as only one of them will be tested.

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


Re: [OE-core] New Recipe for ser2net with IPv6 Support (v3.0 & v3.1)

2017-07-03 Thread Maxin B. John
Hi,

On Fri, Jun 30, 2017 at 08:30:20AM +0100, Thomas A. F. Thorne MEng AUS MIET 
wrote:
> Good afternoon,
> 
> Would submitting a patch to this list that adds a recipe for ser2net
> v3.0 and ser2net v3.1 be the right thing to do to get it one day
> included in the layer?  Would the new versions beneficial to the wider
> community? 

The chances of a package upgrade, just for new features, in a released branch
like "krogoth" will be close to impossible (There may be exceptions, if changes
are security related) 

> I am working on an application that will use the ser2net application to
> access some serial ports over IPv6.  The current version of ser2net
> included with the meta-openembedded/meta-oe layer is 2.9.1 which does
> not include IPv6 support.  I believe that v3.0 onwards may include IPv6
> support and I am testing that out now. 
> 
> I am creating recipes for ser2net_3.0.bb and ser2net_3.1.bb as part of
> my testing.  I would be happy to submit the files as patches.  I am
> currently working on krogoth but could submit the patch against another
> version if preferred. 
> 
> I have CCed the meta-oe maintainers (on my first message attempt but not my 
> second) 
> as that the is the layer that holds the ser2net recipe at present.  I hope 
> that was not a mistake, please
> for give the extra email if it was. 

Sent a patch to upgrade ser2net to version 3.4 (for master branch):
http://lists.openembedded.org/pipermail/openembedded-devel/2017-July/113484.html

> Regards,
> Thomas Thorne  Software Engineer  Net2Edge Limited

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


[OE-core] State of bitbake world, Failed tasks 2017-07-02

2017-07-03 Thread Martin Jansa
http://www.openembedded.org/wiki/Bitbake_World_Status

== Number of issues - stats ==
{| class='wikitable'
!|Date   !!colspan='3'|Failed tasks 
!!|Signatures   
!!colspan='14'|QA !!Comment
|-
||  ||qemuarm   ||qemux86   ||qemux86_64||all
||already-stripped  ||libdir||textrel   ||build-deps
||file-rdeps||version-going-backwards   ||host-user-contaminated
||installed-vs-shipped  ||unknown-configure-option  ||symlink-to-sysroot
||invalid-pkgconfig ||pkgname   ||ldflags   ||compile-host-path 
||
|-
||2017-07-02||7 ||8 ||5 ||0 ||0 ||1 
||4 ||0 ||0 ||1238  ||1 
||0 ||0 ||0 ||0 ||0 
||0 ||0 ||
|}

== Failed tasks 2017-07-02 ==

INFO: jenkins-job.sh-1.8.24 Complete log available at
http://logs.nslu2-linux.org/buildlogs/oe/world/rocko/log.report.20170702_225940.log

=== common (3) ===
* 
meta-openembedded/meta-multimedia/recipes-mediacenter/kodi/kodi_17.bb:do_configure
* 
meta-openembedded/meta-oe/recipes-extended/fluentbit/fluentbit_0.11.11.bb:do_package
* 
virtual:native:meta-qt5/recipes-qt/qt5/qtxmlpatterns_git.bb:do_populate_sysroot

=== common-x86 (2) ===
* meta-qt5/recipes-qt/qt5/qtwebengine_git.bb:do_compile
* 
openembedded-core/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb:do_compile_ptest_base

=== qemuarm (4) ===
* meta-openembedded/meta-oe/recipes-graphics/gegl/gegl_0.3.18.bb:do_fetch
* meta-openembedded/meta-oe/recipes-support/synergy/synergy_git.bb:do_fetch
* 
meta-openembedded/meta-python/recipes-devtools/python/python3-jsonschema_2.6.0.bb:do_compile
* 
meta-openembedded/meta-python/recipes-devtools/python/python3-pytest_3.1.2.bb:do_compile

=== qemux86 (1) ===
* 
meta-openembedded/meta-python/recipes-devtools/python/python-pytest_3.1.2.bb:do_compile

=== qemux86_64 (0) ===

=== Number of failed tasks (20) ===
{| class=wikitable
|-
|| qemuarm  || 7 ||
http://logs.nslu2-linux.org/buildlogs/oe/world/rocko/log.world.qemuarm.20170701_001009.log/
|| http://errors.yoctoproject.org/Errors/Build/40492/
|-
|| qemux86  || 8 ||
http://logs.nslu2-linux.org/buildlogs/oe/world/rocko/log.world.qemux86.20170701_001007.log/
|| http://errors.yoctoproject.org/Errors/Build/40495/
|-
|| qemux86_64   || 5 ||
http://logs.nslu2-linux.org/buildlogs/oe/world/rocko/log.world.qemux86-64.20170702_004648.log/
|| http://errors.yoctoproject.org/Errors/Build/40496/
|}

=== PNBLACKLISTs (15) ===

=== QA issues (1244) ===
{| class=wikitable
!| Count||Issue
|-
||0 ||already-stripped
|-
||0 ||build-deps
|-
||0 ||compile-host-path
|-
||0 ||file-rdeps
|-
||0 ||installed-vs-shipped
|-
||0 ||invalid-pkgconfig
|-
||0 ||ldflags
|-
||0 ||pkgname
|-
||0 ||symlink-to-sysroot
|-
||0 ||unknown-configure-option
|-
||1238  ||version-going-backwards
|-
||1 ||host-user-contaminated
|-
||1 ||libdir
|-
||4 ||textrel
|}



=== Incorrect PACKAGE_ARCH or sstate signatures (0) ===

Complete log: 
http://logs.nslu2-linux.org/buildlogs/oe/world/rocko/log.signatures.20170701_233319.log/

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


Re: [OE-core] [PATCH] ltp: add acl, attr, curl and util-linux runtime dependencies

2017-07-03 Thread Fathi Boudra
On 3 July 2017 at 11:59, Maxin B. John  wrote:
> Hi,
>
> On Mon, Jul 03, 2017 at 11:03:31AM +0300, Fathi Boudra wrote:
>> * Use a list and re-order alphabetically to make it easier to read and
>>   update.
>> * Add missing runtime dependencies:
>>   - acl (getfacl/setfacl commands are required)
>>   - attr (getfattr/setfattr commands are required)
>>   - curl (curl command is required)
>>   - util-linux (ipcs command is required)
>>
>> Reported-by: Naresh Kamboju 
>> Signed-off-by: Fathi Boudra 
>> ---
>>  meta/recipes-extended/ltp/ltp_20170516.bb | 23 +--
>>  1 file changed, 21 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb 
>> b/meta/recipes-extended/ltp/ltp_20170516.bb
>> index f537d60c15..564fdcf8c2 100644
>> --- a/meta/recipes-extended/ltp/ltp_20170516.bb
>> +++ b/meta/recipes-extended/ltp/ltp_20170516.bb
>> @@ -95,7 +95,27 @@ do_install(){
>>  cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases
>>  }
>>
>> -RDEPENDS_${PN} = "perl e2fsprogs-mke2fs python-core libaio bash gawk expect 
>> ldd unzip gzip cpio cronie logrotate which at"
>> +RDEPENDS_${PN} = "
>> +acl \
>> +at \
>> +attr \
>> +bash \
>> +cpio \
>> +cronie \
>> +curl \
>> +e2fsprogs-mke2fs \
>> +expect \
>> +gawk \
>> +gzip \
>> +ldd \
>> +libaio \
>> +logrotate \
>> +perl \
>> +python-core \
>> +unzip \
>> +util-linux \
>> +which \
>> +"
>
> These changes look good. However, this patch is causing a parser error:
> ERROR: ParseError at 
> /home/maxin/poky/meta/recipes-extended/ltp/ltp_20170516.bb:98: unparsed line: 
> 'RDEPENDS_${PN} = "
>
> Please fix it.

fixed in v2: https://patchwork.openembedded.org/patch/141483/

>>  FILES_${PN}-staticdev += "/opt/ltp/lib/libmem.a 
>> /opt/ltp/testcases/data/nm01/lib.a"
>>
>> @@ -106,4 +126,3 @@ INHIBIT_PACKAGE_STRIP = "1"
>>  INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
>>  # However, test_arch_stripped is already stripped, so...
>>  INSANE_SKIP_${PN} += "already-stripped"
>
> Best Regards,
> Maxin
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2] ltp: add acl, attr, curl and util-linux runtime dependencies

2017-07-03 Thread Fathi Boudra
* Use a list and re-order alphabetically to make it easier to read and
  update.
* Add missing runtime dependencies:
  - acl (getfacl/setfacl commands are required)
  - attr (getfattr/setfattr commands are required)
  - curl (curl command is required)
  - util-linux (ipcs command is required)

Reported-by: Naresh Kamboju 
Signed-off-by: Fathi Boudra 
---
Changes in v2:
 * add missing backslash

 meta/recipes-extended/ltp/ltp_20170516.bb | 23 +--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb 
b/meta/recipes-extended/ltp/ltp_20170516.bb
index f537d60c15..564fdcf8c2 100644
--- a/meta/recipes-extended/ltp/ltp_20170516.bb
+++ b/meta/recipes-extended/ltp/ltp_20170516.bb
@@ -95,7 +95,27 @@ do_install(){
 cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases
 }
 
-RDEPENDS_${PN} = "perl e2fsprogs-mke2fs python-core libaio bash gawk expect 
ldd unzip gzip cpio cronie logrotate which at"
+RDEPENDS_${PN} = "\
+acl \
+at \
+attr \
+bash \
+cpio \
+cronie \
+curl \
+e2fsprogs-mke2fs \
+expect \
+gawk \
+gzip \
+ldd \
+libaio \
+logrotate \
+perl \
+python-core \
+unzip \
+util-linux \
+which \
+"
 
 FILES_${PN}-staticdev += "/opt/ltp/lib/libmem.a 
/opt/ltp/testcases/data/nm01/lib.a"
 
@@ -106,4 +126,3 @@ INHIBIT_PACKAGE_STRIP = "1"
 INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
 # However, test_arch_stripped is already stripped, so...
 INSANE_SKIP_${PN} += "already-stripped"
-
-- 
2.13.0

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


[OE-core] [PATCHv4] oe-pkgdata-util: package-info: Allow extra variables to be displayed

2017-07-03 Thread Peter Kjellerstedt
By specifying the -e  option one or multiple times, extra
variables available in the pkgdata can be displayed, e.,g,
`oe-pkgdata-util package-info -e SUMMARY -e LICENSE ...`. The extra
variables displayed are quoted since some of them may contain
whitespace.

Signed-off-by: Peter Kjellerstedt 
---
PATCHv2: Changed references to SRC_URI into LICENSE, since SRC_URI is
not an available variable in the pkgdata.

PATCHv3: Avoid a "TypeError: 'NoneType' object is not iterable" if
no --extra argument is specified. Duh!

PATCHv4: Removed a remaining reference to src_uri in a variable name.

 scripts/oe-pkgdata-util | 12 ++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/scripts/oe-pkgdata-util b/scripts/oe-pkgdata-util
index 6255662a4b..677effeef6 100755
--- a/scripts/oe-pkgdata-util
+++ b/scripts/oe-pkgdata-util
@@ -325,8 +325,15 @@ def package_info(args):
 recipe_version = recipe_version + "-" + mappings[pkg]['PR']
 pkg_size = mappings[pkg]['PKGSIZE']
 
-items.append("%s %s %s %s %s" %
- (pkg, pkg_version, recipe, recipe_version, pkg_size))
+line = "%s %s %s %s %s" % (pkg, pkg_version, recipe, recipe_version, 
pkg_size)
+
+if args.extra:
+for var in args.extra:
+val = mappings[pkg][var].strip()
+val = re.sub(r'\s+', ' ', val)
+line += ' "%s"' % val
+
+items.append(line)
 print('\n'.join(items))
 
 def get_recipe_pkgs(pkgdata_dir, recipe, unpackaged):
@@ -530,6 +537,7 @@ def main():
   description='Looks up the specified 
runtime package(s) and display information')
 parser_package_info.add_argument('pkg', nargs='*', help='Runtime package 
name to look up')
 parser_package_info.add_argument('-f', '--file', help='Read package names 
from the specified file (one per line, first field only)')
+parser_package_info.add_argument('-e', '--extra', help='Extra variables to 
display, e.g., LICENSE (can be specified multiple times)', action='append')
 parser_package_info.set_defaults(func=package_info)
 
 parser_find_path = subparsers.add_parser('find-path',
-- 
2.12.0

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


Re: [OE-core] [PATCH] sanity.bbclass: fix AttributeError in mirror format checks

2017-07-03 Thread Patrick Ohly
On Fri, 2017-06-30 at 14:24 +0300, Mikko Ylinen wrote:
> mirrors is a list after split() and results in:
> 
> AttributeError: 'list' object has no attribute 'strip'
> 
> when the 'mirror values are pairs' check fails.
> 
> Signed-off-by: Mikko Ylinen 
> ---
>  meta/classes/sanity.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
> index e8064ac483..1f74026e13 100644
> --- a/meta/classes/sanity.bbclass
> +++ b/meta/classes/sanity.bbclass
> @@ -839,7 +839,7 @@ def check_sanity_everybuild(status, d):
>  
>  # Split into pairs
>  if len(mirrors) % 2 != 0:
> -bb.warn('Invalid mirror variable value for %s: %s, should 
> contain paired members.' % (mirror_var, mirrors.strip()))
> +bb.warn('Invalid mirror variable value for %s: %s, should 
> contain paired members.' % (mirror_var, str(mirrors).strip()))

.strip() is redundant here, because str(mirrors) will produce [...],
i.e. the string will never have leading or trailing spaces.

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.



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


[OE-core] [PATCH V2 0/1] expat: upgrade to 2.2.1

2017-07-03 Thread Dengke Du
The following changes since commit 98099349e358a9caaec8ec81f0d4abe588909cfe:

  bitbake: npm fetcher: fix unknown variable name. (2017-06-29 14:33:42 +0100)

are available in the git repository at:

  https://github.com/DengkeDu/openembedded-core.git 
dengke/expat-upgrade-to-2.2.1
  https://github.com//tree/dengke/expat-upgrade-to-2.2.1

Dengke Du (1):
  expat: upgrade to 2.2.1

 meta/recipes-core/expat/expat_2.2.0.bb | 5 -
 meta/recipes-core/expat/expat_2.2.1.bb | 5 +
 2 files changed, 5 insertions(+), 5 deletions(-)
 delete mode 100644 meta/recipes-core/expat/expat_2.2.0.bb
 create mode 100644 meta/recipes-core/expat/expat_2.2.1.bb

-- 
2.8.1

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


[OE-core] [PATCH V2 1/1] expat: upgrade to 2.2.1

2017-07-03 Thread Dengke Du
The COPYING file in expat has the following changes:

2001-20016 to 2001-2017

Signed-off-by: Dengke Du 
---
 meta/recipes-core/expat/expat_2.2.0.bb | 5 -
 meta/recipes-core/expat/expat_2.2.1.bb | 5 +
 2 files changed, 5 insertions(+), 5 deletions(-)
 delete mode 100644 meta/recipes-core/expat/expat_2.2.0.bb
 create mode 100644 meta/recipes-core/expat/expat_2.2.1.bb

diff --git a/meta/recipes-core/expat/expat_2.2.0.bb 
b/meta/recipes-core/expat/expat_2.2.0.bb
deleted file mode 100644
index ef21a11..000
--- a/meta/recipes-core/expat/expat_2.2.0.bb
+++ /dev/null
@@ -1,5 +0,0 @@
-require expat.inc
-LIC_FILES_CHKSUM = "file://COPYING;md5=9c3ee559c6f9dcee1043ead112139f4f"
-
-SRC_URI[md5sum] = "2f47841c829facb346eb6e3fab5212e2"
-SRC_URI[sha256sum] = 
"d9e50ff2d19b3538bd2127902a89987474e1a4db8e43a66a4d1a712ab9a504ff"
diff --git a/meta/recipes-core/expat/expat_2.2.1.bb 
b/meta/recipes-core/expat/expat_2.2.1.bb
new file mode 100644
index 000..47827c5
--- /dev/null
+++ b/meta/recipes-core/expat/expat_2.2.1.bb
@@ -0,0 +1,5 @@
+require expat.inc
+LIC_FILES_CHKSUM = "file://COPYING;md5=5b8620d98e49772d95fc1d291c26aa79"
+
+SRC_URI[md5sum] = "d9c3baeab58774cefc2f04faf29f2cf8"
+SRC_URI[sha256sum] = 
"1868cadae4c82a018e361e2b2091de103cd820aaacb0d6cfa49bd2cd83978885"
-- 
2.8.1

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


Re: [OE-core] [PATCH] bootimg-efi.py: Use IMGDEPLOYDIR instead of DEPLOY_DIR_IMAGE for initrd

2017-07-03 Thread Patrick Ohly
On Mon, 2017-07-03 at 11:36 +0300, Ed Bartosh wrote:
> On Fri, Jun 30, 2017 at 10:53:30AM -0700, Alejandro Hernandez wrote:
> > When using wic to create an image from a certain build, wic is expecting
> > to find initrd at the final destination of our images (DEPLOY_DIR_IMAGE),
> > which is wrong, since the initrd file has not been copied to the final
> > directory yet,
> 
> Is it possible to ensure that initrd is deployed before wic is run by
> making do_image_wic depend on initrd deploy task?

Not in this case, because both do_image_wic and do_image_cpio share the
same deploy task.

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.



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


Re: [OE-core] [PATCH] bootimg-efi.py: Use IMGDEPLOYDIR instead of DEPLOY_DIR_IMAGE for initrd

2017-07-03 Thread Ed Bartosh
On Sat, Jul 01, 2017 at 04:48:41PM -0500, Alejandro Hernandez wrote:
> 
> 
> > From the DESCRIPTION:
> >
> > core-image-tiny-initramfs doesn't actually generate an image but
> > rather generates boot and rootfs artifacts into a common
> > location that can subsequently be picked up by external image
> > generation tools such as wic.
> This is the old description, when it was though that we would have a common
> artifacts directory, which never happened due to some changes in wic
> last year

Can you elaborate on this please?


I'm still thinking that having a common artifacts directory is what we
should aim for. And DEPLOY_DIR_IMAGE is a good candidate for it as most
of artifacts are already there.

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


Re: [OE-core] [PATCH v2 0/5] #11662 - wic should mount /boot

2017-07-03 Thread Ed Bartosh
On Mon, Jul 03, 2017 at 09:53:32AM +0200, Patrick Ohly wrote:
> On Mon, 2017-07-03 at 10:31 +0300, Ed Bartosh wrote:
> > On Fri, Jun 30, 2017 at 08:34:30PM +0200, Patrick Ohly wrote:
> > > then I don't see a need for any additional flags. Just
> > > don't use the features which result in a rootfs modification.
> > 
> > I also didn't see it till last message from Otavio. Now I do - they
> > don't want to change .wks files. They're using standard wks from
> > scripts/lib/wic/canned-wks or from standard layers and they don't want
> > to duplicate them when they don't want rootfs modifications.
> > 
> > It could be a valid reason to have --no-fstab-update option I think.
> > However, I'm still not 100% convinced I'm ok with this if nobody else
> > objects.
> 
> Okay, now I see what the purpose is.
> 
> I prefer a --no-fstab-update over a general --no-rootfs-update because
> for each case where wic would normally modify the rootfs, some other
> mechanism must be in place which makes that modification redundant (like
> using PARTUUID). Having separate parameters forces the developers to
> think about it. Just my 2 cents...
>

Tha makes sense to me.

>From other point of view if the goal is to have rootfs unmodified
--no-rootfs-update would make it easier to achive. Moreover it will
guarantee that rootfs is unmodified even if wic introduces new
functionality that modifies rootfs.

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


Re: [OE-core] [PATCH] ltp: add acl, attr, curl and util-linux runtime dependencies

2017-07-03 Thread Dengke Du



On 2017年07月03日 16:59, Maxin B. John wrote:

Hi,

On Mon, Jul 03, 2017 at 11:03:31AM +0300, Fathi Boudra wrote:

* Use a list and re-order alphabetically to make it easier to read and
   update.
* Add missing runtime dependencies:
   - acl (getfacl/setfacl commands are required)
   - attr (getfattr/setfattr commands are required)
   - curl (curl command is required)
   - util-linux (ipcs command is required)

Reported-by: Naresh Kamboju 
Signed-off-by: Fathi Boudra 
---
  meta/recipes-extended/ltp/ltp_20170516.bb | 23 +--
  1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb 
b/meta/recipes-extended/ltp/ltp_20170516.bb
index f537d60c15..564fdcf8c2 100644
--- a/meta/recipes-extended/ltp/ltp_20170516.bb
+++ b/meta/recipes-extended/ltp/ltp_20170516.bb
@@ -95,7 +95,27 @@ do_install(){
  cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases
  }
  
-RDEPENDS_${PN} = "perl e2fsprogs-mke2fs python-core libaio bash gawk expect ldd unzip gzip cpio cronie logrotate which at"

+RDEPENDS_${PN} = "

should be:

RDEPENDS_${PN} = " \

+acl \
+at \
+attr \
+bash \
+cpio \
+cronie \
+curl \
+e2fsprogs-mke2fs \
+expect \
+gawk \
+gzip \
+ldd \
+libaio \
+logrotate \
+perl \
+python-core \
+unzip \
+util-linux \
+which \
+"

These changes look good. However, this patch is causing a parser error:
ERROR: ParseError at /home/maxin/poky/meta/recipes-extended/ltp/ltp_20170516.bb:98: 
unparsed line: 'RDEPENDS_${PN} = "

Please fix it.


  FILES_${PN}-staticdev += "/opt/ltp/lib/libmem.a 
/opt/ltp/testcases/data/nm01/lib.a"
  
@@ -106,4 +126,3 @@ INHIBIT_PACKAGE_STRIP = "1"

  INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
  # However, test_arch_stripped is already stripped, so...
  INSANE_SKIP_${PN} += "already-stripped"

Best Regards,
Maxin


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


Re: [OE-core] [PATCH] ltp: add acl, attr, curl and util-linux runtime dependencies

2017-07-03 Thread Maxin B. John
Hi,

On Mon, Jul 03, 2017 at 11:03:31AM +0300, Fathi Boudra wrote:
> * Use a list and re-order alphabetically to make it easier to read and
>   update.
> * Add missing runtime dependencies:
>   - acl (getfacl/setfacl commands are required)
>   - attr (getfattr/setfattr commands are required)
>   - curl (curl command is required)
>   - util-linux (ipcs command is required)
> 
> Reported-by: Naresh Kamboju 
> Signed-off-by: Fathi Boudra 
> ---
>  meta/recipes-extended/ltp/ltp_20170516.bb | 23 +--
>  1 file changed, 21 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb 
> b/meta/recipes-extended/ltp/ltp_20170516.bb
> index f537d60c15..564fdcf8c2 100644
> --- a/meta/recipes-extended/ltp/ltp_20170516.bb
> +++ b/meta/recipes-extended/ltp/ltp_20170516.bb
> @@ -95,7 +95,27 @@ do_install(){
>  cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases
>  }
>  
> -RDEPENDS_${PN} = "perl e2fsprogs-mke2fs python-core libaio bash gawk expect 
> ldd unzip gzip cpio cronie logrotate which at"
> +RDEPENDS_${PN} = "
> +acl \
> +at \
> +attr \
> +bash \
> +cpio \
> +cronie \
> +curl \
> +e2fsprogs-mke2fs \
> +expect \
> +gawk \
> +gzip \
> +ldd \
> +libaio \
> +logrotate \
> +perl \
> +python-core \
> +unzip \
> +util-linux \
> +which \
> +"

These changes look good. However, this patch is causing a parser error:
ERROR: ParseError at 
/home/maxin/poky/meta/recipes-extended/ltp/ltp_20170516.bb:98: unparsed line: 
'RDEPENDS_${PN} = "

Please fix it.

>  FILES_${PN}-staticdev += "/opt/ltp/lib/libmem.a 
> /opt/ltp/testcases/data/nm01/lib.a"
>  
> @@ -106,4 +126,3 @@ INHIBIT_PACKAGE_STRIP = "1"
>  INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
>  # However, test_arch_stripped is already stripped, so...
>  INSANE_SKIP_${PN} += "already-stripped"

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


Re: [OE-core] [PATCH] bootimg-efi.py: Use IMGDEPLOYDIR instead of DEPLOY_DIR_IMAGE for initrd

2017-07-03 Thread Ed Bartosh
On Fri, Jun 30, 2017 at 10:53:30AM -0700, Alejandro Hernandez wrote:
> When using wic to create an image from a certain build, wic is expecting
> to find initrd at the final destination of our images (DEPLOY_DIR_IMAGE),
> which is wrong, since the initrd file has not been copied to the final
> directory yet,

Is it possible to ensure that initrd is deployed before wic is run by
making do_image_wic depend on initrd deploy task?

> so instead of trying to use an initrd file from 
> DEPLOY_DIR_IMAGE we get it from IMGDEPLOYDIR, which is the directory 
> where the resulting images are placed before their final destination, 
> and its where we can find the correct initrd file for our image.
> 
> Signed-off-by: Alejandro Hernandez 
> ---
>  scripts/lib/wic/plugins/source/bootimg-efi.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/scripts/lib/wic/plugins/source/bootimg-efi.py 
> b/scripts/lib/wic/plugins/source/bootimg-efi.py
> index 9879cb9fceb..d1d445ecfe1 100644
> --- a/scripts/lib/wic/plugins/source/bootimg-efi.py
> +++ b/scripts/lib/wic/plugins/source/bootimg-efi.py
> @@ -105,9 +105,9 @@ class BootimgEFIPlugin(SourcePlugin):
>  
>  if initrd:
>  # obviously we need to have a common common deploy var
> -bootimg_dir = get_bitbake_var("DEPLOY_DIR_IMAGE")
> +bootimg_dir = get_bitbake_var("IMGDEPLOYDIR")
>  if not bootimg_dir:
> -raise WicError("Couldn't find DEPLOY_DIR_IMAGE, exiting")
> +raise WicError("Couldn't find IMGDEPLOYDIR, exiting")
>  
>  cp_cmd = "cp %s/%s %s" % (bootimg_dir, initrd, hdddir)
>  exec_cmd(cp_cmd, True)
> -- 
> 2.12.0
> 
> -- 
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

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


[OE-core] [PATCH 1/1] grep: upgrade to 3.1

2017-07-03 Thread Dengke Du
Signed-off-by: Dengke Du 
---
 meta/recipes-extended/grep/{grep_3.0.bb => grep_3.1.bb} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-extended/grep/{grep_3.0.bb => grep_3.1.bb} (89%)

diff --git a/meta/recipes-extended/grep/grep_3.0.bb 
b/meta/recipes-extended/grep/grep_3.1.bb
similarity index 89%
rename from meta/recipes-extended/grep/grep_3.0.bb
rename to meta/recipes-extended/grep/grep_3.1.bb
index b2940d5429..05b6b93949 100644
--- a/meta/recipes-extended/grep/grep_3.0.bb
+++ b/meta/recipes-extended/grep/grep_3.1.bb
@@ -9,8 +9,8 @@ SRC_URI = "${GNU_MIRROR}/grep/grep-${PV}.tar.xz \
file://0001-Unset-need_charset_alias-when-building-for-musl.patch \
   "
 
-SRC_URI[md5sum] = "fa07c1616adeb9c3262be5177d10ad4a"
-SRC_URI[sha256sum] = 
"e2c81db5056e3e8c5995f0bb5d0d0e1cad1f6f45c3b2fc77b6e81435aed48ab5"
+SRC_URI[md5sum] = "feca7b3e7c7f4aab2b42ecbfc513b070"
+SRC_URI[sha256sum] = 
"db625c7ab3bb3ee757b3926a5cfa8d9e1c3991ad24707a83dde8a5ef2bf7a07e"
 
 inherit autotools gettext texinfo pkgconfig
 
-- 
2.11.0

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


[OE-core] [PATCH 0/1] grep: upgrade to 3.1

2017-07-03 Thread Dengke Du
The following changes since commit 98099349e358a9caaec8ec81f0d4abe588909cfe:

  bitbake: npm fetcher: fix unknown variable name. (2017-06-29 14:33:42 +0100)

are available in the git repository at:

  https://github.com/DengkeDu/openembedded-core.git dengke/grep-upgrade-to-3.1

Dengke Du (1):
  grep: upgrade to 3.1

 meta/recipes-extended/grep/{grep_3.0.bb => grep_3.1.bb} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-extended/grep/{grep_3.0.bb => grep_3.1.bb} (89%)

-- 
2.11.0

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


Re: [OE-core] [PATCH 05/11] gstreamer1.0-plugins-good: upgrade 1.10.4 -> 1.12.1

2017-07-03 Thread Maxin B. John
Hi Jussi,

On Mon, Jul 03, 2017 at 09:33:09AM +0300, Jussi Kukkonen wrote:
>I'm guessing 12.1 has the same issue with renamed plugins that I reported
>on the previous patchset? 
>
> See 
> https://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/?id=ca0ed8a134f1d26083e6f89636493df89ff29cff
>  
> : this changes packages names so we need to add special case RPROVIDES
> for these three (and probably should also change gstreamer1.0-meta-base
> to use the real name).
>
>  - Jussi

Sorry, haven't noticed it before. Thanks for sharing this here. I will update
the patches and send a v2.

>
>>  On 30 June 2017 at 17:56, Maxin B. John  wrote:
>>
>>  Remove backported patch:
>>  
>> 1.0001-v4l2object-Also-add-videometa-if-there-is-padding-to.patch

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


[OE-core] [PATCH] ltp: add acl, attr, curl and util-linux runtime dependencies

2017-07-03 Thread Fathi Boudra
* Use a list and re-order alphabetically to make it easier to read and
  update.
* Add missing runtime dependencies:
  - acl (getfacl/setfacl commands are required)
  - attr (getfattr/setfattr commands are required)
  - curl (curl command is required)
  - util-linux (ipcs command is required)

Reported-by: Naresh Kamboju 
Signed-off-by: Fathi Boudra 
---
 meta/recipes-extended/ltp/ltp_20170516.bb | 23 +--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-extended/ltp/ltp_20170516.bb 
b/meta/recipes-extended/ltp/ltp_20170516.bb
index f537d60c15..564fdcf8c2 100644
--- a/meta/recipes-extended/ltp/ltp_20170516.bb
+++ b/meta/recipes-extended/ltp/ltp_20170516.bb
@@ -95,7 +95,27 @@ do_install(){
 cp -r testcases/open_posix_testsuite ${D}/opt/ltp/testcases
 }
 
-RDEPENDS_${PN} = "perl e2fsprogs-mke2fs python-core libaio bash gawk expect 
ldd unzip gzip cpio cronie logrotate which at"
+RDEPENDS_${PN} = "
+acl \
+at \
+attr \
+bash \
+cpio \
+cronie \
+curl \
+e2fsprogs-mke2fs \
+expect \
+gawk \
+gzip \
+ldd \
+libaio \
+logrotate \
+perl \
+python-core \
+unzip \
+util-linux \
+which \
+"
 
 FILES_${PN}-staticdev += "/opt/ltp/lib/libmem.a 
/opt/ltp/testcases/data/nm01/lib.a"
 
@@ -106,4 +126,3 @@ INHIBIT_PACKAGE_STRIP = "1"
 INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
 # However, test_arch_stripped is already stripped, so...
 INSANE_SKIP_${PN} += "already-stripped"
-
-- 
2.13.0

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


Re: [OE-core] [PATCH v2 0/5] #11662 - wic should mount /boot

2017-07-03 Thread Patrick Ohly
On Mon, 2017-07-03 at 10:31 +0300, Ed Bartosh wrote:
> On Fri, Jun 30, 2017 at 08:34:30PM +0200, Patrick Ohly wrote:
> > then I don't see a need for any additional flags. Just
> > don't use the features which result in a rootfs modification.
> 
> I also didn't see it till last message from Otavio. Now I do - they
> don't want to change .wks files. They're using standard wks from
> scripts/lib/wic/canned-wks or from standard layers and they don't want
> to duplicate them when they don't want rootfs modifications.
> 
> It could be a valid reason to have --no-fstab-update option I think.
> However, I'm still not 100% convinced I'm ok with this if nobody else
> objects.

Okay, now I see what the purpose is.

I prefer a --no-fstab-update over a general --no-rootfs-update because
for each case where wic would normally modify the rootfs, some other
mechanism must be in place which makes that modification redundant (like
using PARTUUID). Having separate parameters forces the developers to
think about it. Just my 2 cents...

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.



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


Re: [OE-core] [PATCH v2 0/5] #11662 - wic should mount /boot

2017-07-03 Thread Ed Bartosh
On Fri, Jun 30, 2017 at 08:34:30PM +0200, Patrick Ohly wrote:
> On Fri, 2017-06-30 at 14:33 -0300, Otavio Salvador wrote:
> > Other possible rootfs changes also should be possible to be disabled
> > but IMO it should be per-feature (one for fstab, one for exclude,
> > ...).
> 
> I also think it should be per-feature, it necessary at all.
> 
> I still do not fully understand under which circumstances wic modifies
> the rootfs. If that happens only when explicitly requested in the wks
> file as Ed said, then I don't see a need for any additional flags.

Yes, that happens only if it explicitly requested, i.e. if there are valid
mount points in .wks file

> then I don't see a need for any additional flags. Just
> don't use the features which result in a rootfs modification.

I also didn't see it till last message from Otavio. Now I do - they
don't want to change .wks files. They're using standard wks from
scripts/lib/wic/canned-wks or from standard layers and they don't want
to duplicate them when they don't want rootfs modifications.

It could be a valid reason to have --no-fstab-update option I think.
However, I'm still not 100% convinced I'm ok with this if nobody else
objects.

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