Re: [OE-core] [PATCH] image.bbclass: Add additional bb.debug to help track 12304

[Andre McCurdy]

On Wed, Dec 6, 2017 at 8:28 PM, Saul Wold  wrote:
> We actually caught the ext4 size issue in the wild with the debug
> output in the oe_mkext234fs() code, but it did not help.  What that
> showed was that the get_rootfs_size was returning a default size of
> 8192, where as the actual rootfs was more like 10572, thus too large
> to fit in the created sparse file.
>
> This additional temporary debug code should help us determine where
> the failure might be.

It might also be useful to save away the output of "ls -lR" for the
rootfs dir as seen by both get_rootfs_size() and oe_mkext234fs(), so
that it the apparent size is changing between the times these two
functions run we'll get some clues as to why.

> More debug for
> [YOCTO #12304]
>
> Signed-off-by: Saul Wold 
> ---
>  meta/classes/image.bbclass | 28 
>  1 file changed, 24 insertions(+), 4 deletions(-)
>
> diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
> index d93de02b759..9964393e211 100644
> --- a/meta/classes/image.bbclass
> +++ b/meta/classes/image.bbclass
> @@ -519,7 +519,7 @@ python () {
>  #
>  # Compute the rootfs size
>  #
> -def get_rootfs_size(d):
> +def get_rootfs_size(d, force_size=0):
>  import subprocess
>
>  rootfs_alignment = int(d.getVar('IMAGE_ROOTFS_ALIGNMENT'))
> @@ -531,24 +531,35 @@ def get_rootfs_size(d):
>  initramfs_fstypes = d.getVar('INITRAMFS_FSTYPES') or ''
>  initramfs_maxsize = d.getVar('INITRAMFS_MAXSIZE')
>
> -output = subprocess.check_output(['du', '-ks',
> +if (force_size != 0):
> +size_kb = force_size
> +else:
> +output = subprocess.check_output(['du', '-ks',
>d.getVar('IMAGE_ROOTFS')])
> -size_kb = int(output.split()[0])
> +size_kb = int(output.split()[0])
> +
>  base_size = size_kb * overhead_factor
> -base_size = max(base_size, rootfs_req_size) + rootfs_extra_space
> +bb.debug(1, '%f = %d * %f' % (base_size, size_kb, overhead_factor))
> +base_size2 = max(base_size, rootfs_req_size) + rootfs_extra_space
> +bb.debug(1, '%f = max(%f, %d)[%f] + %d' % (base_size2, base_size, 
> rootfs_req_size, max(base_size, rootfs_req_size), overhead_factor))
>
> +base_size = base_size2
>  if base_size != int(base_size):
>  base_size = int(base_size + 1)
>  else:
>  base_size = int(base_size)
> +bb.debug(1, '%f = int(%f)' % (base_size, base_size2))
>
> +base_size_saved = base_size
>  base_size += rootfs_alignment - 1
>  base_size -= base_size % rootfs_alignment
> +bb.debug(1, '%d = aligned(%d)' % (base_size, base_size_saved))
>
>  # Do not check image size of the debugfs image. This is not supposed
>  # to be deployed, etc. so it doesn't make sense to limit the size
>  # of the debug.
>  if (d.getVar('IMAGE_BUILDING_DEBUGFS') or "") == "true":
> +bb.debug(1, 'returning debugfs size %d' % (base_size))
>  return base_size
>
>  # Check the rootfs size against IMAGE_ROOTFS_MAXSIZE (if set)
> @@ -566,6 +577,8 @@ def get_rootfs_size(d):
>  (base_size, initramfs_maxsize_int))
>  bb.error("You can set INITRAMFS_MAXSIZE a larger value. Usually, 
> it should")
>  bb.fatal("be less than 1/2 of ram size, or you may fail to boot 
> it.\n")
> +
> +bb.debug(1, 'returning %d' % (base_size))
>  return base_size
>
>  python set_image_size () {
> @@ -574,6 +587,13 @@ python set_image_size () {
>  d.setVarFlag('ROOTFS_SIZE', 'export', '1')
>  }
>
> +do_testsize[nostamp] = "1"
> +python do_testsize() {
> +sz = get_rootfs_size(d, 10572)
> +bb.warn('size is %d' % sz)
> +}
> +addtask do_testsize
> +
>  #
>  # Create symlinks to the newly created image
>  #
> --
> 2.13.6
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] image.bbclass: Add additional bb.debug to help track 12304

[Saul Wold]

We actually caught the ext4 size issue in the wild with the debug
output in the oe_mkext234fs() code, but it did not help.  What that
showed was that the get_rootfs_size was returning a default size of
8192, where as the actual rootfs was more like 10572, thus too large
to fit in the created sparse file.

This additional temporary debug code should help us determine where
the failure might be. 

More debug for
[YOCTO #12304]

Signed-off-by: Saul Wold 
---
 meta/classes/image.bbclass | 28 
 1 file changed, 24 insertions(+), 4 deletions(-)

diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index d93de02b759..9964393e211 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -519,7 +519,7 @@ python () {
 #
 # Compute the rootfs size
 #
-def get_rootfs_size(d):
+def get_rootfs_size(d, force_size=0):
 import subprocess
 
 rootfs_alignment = int(d.getVar('IMAGE_ROOTFS_ALIGNMENT'))
@@ -531,24 +531,35 @@ def get_rootfs_size(d):
 initramfs_fstypes = d.getVar('INITRAMFS_FSTYPES') or ''
 initramfs_maxsize = d.getVar('INITRAMFS_MAXSIZE')
 
-output = subprocess.check_output(['du', '-ks',
+if (force_size != 0):
+size_kb = force_size
+else:
+output = subprocess.check_output(['du', '-ks',
   d.getVar('IMAGE_ROOTFS')])
-size_kb = int(output.split()[0])
+size_kb = int(output.split()[0])
+
 base_size = size_kb * overhead_factor
-base_size = max(base_size, rootfs_req_size) + rootfs_extra_space
+bb.debug(1, '%f = %d * %f' % (base_size, size_kb, overhead_factor))
+base_size2 = max(base_size, rootfs_req_size) + rootfs_extra_space
+bb.debug(1, '%f = max(%f, %d)[%f] + %d' % (base_size2, base_size, 
rootfs_req_size, max(base_size, rootfs_req_size), overhead_factor))
 
+base_size = base_size2
 if base_size != int(base_size):
 base_size = int(base_size + 1)
 else:
 base_size = int(base_size)
+bb.debug(1, '%f = int(%f)' % (base_size, base_size2))
 
+base_size_saved = base_size
 base_size += rootfs_alignment - 1
 base_size -= base_size % rootfs_alignment
+bb.debug(1, '%d = aligned(%d)' % (base_size, base_size_saved))
 
 # Do not check image size of the debugfs image. This is not supposed
 # to be deployed, etc. so it doesn't make sense to limit the size
 # of the debug.
 if (d.getVar('IMAGE_BUILDING_DEBUGFS') or "") == "true":
+bb.debug(1, 'returning debugfs size %d' % (base_size))
 return base_size
 
 # Check the rootfs size against IMAGE_ROOTFS_MAXSIZE (if set)
@@ -566,6 +577,8 @@ def get_rootfs_size(d):
 (base_size, initramfs_maxsize_int))
 bb.error("You can set INITRAMFS_MAXSIZE a larger value. Usually, 
it should")
 bb.fatal("be less than 1/2 of ram size, or you may fail to boot 
it.\n")
+
+bb.debug(1, 'returning %d' % (base_size))
 return base_size
 
 python set_image_size () {
@@ -574,6 +587,13 @@ python set_image_size () {
 d.setVarFlag('ROOTFS_SIZE', 'export', '1')
 }
 
+do_testsize[nostamp] = "1"
+python do_testsize() {
+sz = get_rootfs_size(d, 10572)
+bb.warn('size is %d' % sz)
+}
+addtask do_testsize
+
 #
 # Create symlinks to the newly created image
 #
-- 
2.13.6

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

[OE-core] [PATCH] image_types: btrfs use sparse file creation

[Saul Wold]

This will speed up file creation and still allow the btrfs tools to
create a full btrfs image.  This is similar to what we do for ext234
FS types.

Signed-off-by: Saul Wold 
---
 meta/classes/image_types.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
index aaba1073a08..ce7699d9b60 100644
--- a/meta/classes/image_types.bbclass
+++ b/meta/classes/image_types.bbclass
@@ -104,7 +104,7 @@ IMAGE_CMD_btrfs () {
size=${MIN_BTRFS_SIZE}
bbwarn "Rootfs size is too small for BTRFS. Filesystem will be 
extended to ${size}K"
fi
-   dd if=/dev/zero 
of=${IMGDEPLOYDIR}/${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.btrfs count=${size} bs=1024
+   dd if=/dev/zero 
of=${IMGDEPLOYDIR}/${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.btrfs seek=${size} count=0 
bs=1024
mkfs.btrfs ${EXTRA_IMAGECMD} -r ${IMAGE_ROOTFS} 
${IMGDEPLOYDIR}/${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.btrfs
 }
 
-- 
2.13.6

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

[OE-core] [PATCH] image_types: btrfs use sparse file creation

[Saul Wold]

This will speed up file creation and still allow the btrfs tools to
create a full btrfs image.  This is similar to what we do for ext234
FS types.

Signed-off-by: Saul Wold 
---
 meta/classes/image_types.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
index aaba1073a08..ce7699d9b60 100644
--- a/meta/classes/image_types.bbclass
+++ b/meta/classes/image_types.bbclass
@@ -104,7 +104,7 @@ IMAGE_CMD_btrfs () {
size=${MIN_BTRFS_SIZE}
bbwarn "Rootfs size is too small for BTRFS. Filesystem will be 
extended to ${size}K"
fi
-   dd if=/dev/zero 
of=${IMGDEPLOYDIR}/${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.btrfs count=${size} bs=1024
+   dd if=/dev/zero 
of=${IMGDEPLOYDIR}/${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.btrfs seek=${size} count=0 
bs=1024
mkfs.btrfs ${EXTRA_IMAGECMD} -r ${IMAGE_ROOTFS} 
${IMGDEPLOYDIR}/${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.btrfs
 }
 
-- 
2.13.6

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

[OE-core] [PATCH 1/1] features_backfill: fix for multilib

[jackie.huang]

From: Jackie Huang 

The backfilling feature doesn't work for multilib,

e.g. build with:
MACHINE = "qemumips64"
MULTILIB_GLOBAL_VARIANTS_append = " libn32"
MULTILIBS = "multilib:libn32"
DEFAULTTUNE_virtclass-multilib-libn32 ?= "mips64-n32"
require conf/multilib.conf

And we have backfill_considered in machine/include/mips/arch-mips.inc:
MACHINE_FEATURES_BACKFILL_CONSIDERED_append = 
"${@bb.utils.contains('TUNE_FEATURES', 'n32', 'qemu-usermode', '', d)}"

For libn32 builds, 'qemu-usermode' is not expected but it still
presents in MACHINE_FEATURES.

To fix the issue:
* Change the oe.utils.features_backfill to always compare with the
  original features(before backfilled), so we can run the function
  multiple times when needed.

* run oe.utils.features_backfill at the end of multilib_virtclass_handler
  to update the backfilled features to ensure it's correct for multilib.

[YOCTO #12373]

Signed-off-by: Jackie Huang 
---
 meta/classes/multilib.bbclass | 4 
 meta/lib/oe/utils.py  | 9 +++--
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/meta/classes/multilib.bbclass b/meta/classes/multilib.bbclass
index 816f54e..589ee37 100644
--- a/meta/classes/multilib.bbclass
+++ b/meta/classes/multilib.bbclass
@@ -77,6 +77,10 @@ python multilib_virtclass_handler () {
 if newtune:
 e.data.setVar("DEFAULTTUNE", newtune)
 e.data.setVar('DEFAULTTUNE_ML_%s' % variant, newtune)
+
+# Update the backfilled features after DEFAULTTUNE changed
+oe.utils.features_backfill("DISTRO_FEATURES", e.data)
+oe.utils.features_backfill("MACHINE_FEATURES", e.data)
 }
 
 addhandler multilib_virtclass_handler
diff --git a/meta/lib/oe/utils.py b/meta/lib/oe/utils.py
index 1897c5f..371acd0 100644
--- a/meta/lib/oe/utils.py
+++ b/meta/lib/oe/utils.py
@@ -107,13 +107,18 @@ def features_backfill(var,d):
 backfill = (d.getVar(var+"_BACKFILL") or "").split()
 considered = (d.getVar(var+"_BACKFILL_CONSIDERED") or "").split()
 
+features_original = (d.getVar(var + "_ORIGINAL") or "").split()
+if not features_original:
+features_original = features
+d.setVar(var + "_ORIGINAL", " ".join(features_original))
+
 addfeatures = []
 for feature in backfill:
-if feature not in features and feature not in considered:
+if feature not in features_original and feature not in considered:
 addfeatures.append(feature)
 
 if addfeatures:
-d.appendVar(var, " " + " ".join(addfeatures))
+d.setVar(var, " ".join(features_original) + " " + " 
".join(addfeatures))
 
 def all_distro_features(d, features, truevalue="1", falsevalue=""):
 """
-- 
1.9.1

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

[OE-core] [PATCH 0/1] features_backfill: fix for multilib

[jackie.huang]

From: Jackie Huang 

Tested with:

1) local.conf:
MACHINE = "qemumips64"
MULTILIB_GLOBAL_VARIANTS_append = " libn32"
MULTILIBS = "multilib:libn32"
DEFAULTTUNE_virtclass-multilib-libn32 ?= "mips64-n32"
require conf/multilib.conf

2) run bitbake -e to check the features:

$ bitbake -e libn32-gobject-introspection | grep '^MACHINE_FEATURES'

(before the fix)
MACHINE_FEATURES="alsa bluetooth usbgadget screen rtc qemu-usermode"
MACHINE_FEATURES_BACKFILL="rtc qemu-usermode"
MACHINE_FEATURES_BACKFILL_CONSIDERED=" qemu-usermode"

(after the fix)
MACHINE_FEATURES="alsa bluetooth usbgadget screen rtc"
MACHINE_FEATURES_BACKFILL="rtc qemu-usermode"
MACHINE_FEATURES_BACKFILL_CONSIDERED=" qemu-usermode"
MACHINE_FEATURES_ORIGINAL="alsa bluetooth usbgadget screen"

3) world build:

$ bitbake world

builds without error.


--
The following changes since commit 4469acdf1d0338220f3fe2ecb5e079eea6fda375:

  lib/oe/utils: remove param_bool() (2017-12-02 11:25:34 +)

are available in the git repository at:

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

Jackie Huang (1):
  features_backfill: fix for multilib

 meta/classes/multilib.bbclass | 4 
 meta/lib/oe/utils.py  | 9 +++--
 2 files changed, 11 insertions(+), 2 deletions(-)

-- 
1.9.1

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

Re: [OE-core] [PATCH] libyui

[Khem Raj]

Meta-oe

On Wed, Dec 6, 2017 at 5:26 PM Zheng, Ruoqin 
wrote:

> Hi Ross!
>
>  Where should I put this recipe?
>
> --
>
> Zheng Ruoqin
>
> Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST)
>
> ADDR.: No.6 Wenzhu Road, Software Avenue,
>
>Nanjing, 210012, China
>
> MAIL : zhengrq.f...@cn.fujistu.com
>
>
>
> *From:* Burton, Ross [mailto:ross.bur...@intel.com]
> *Sent:* Monday, December 04, 2017 6:17 PM
> *To:* Zheng, Ruoqin/郑 若钦 
> *Cc:* Alexander Kanavin ;
> openembedded-core@lists.openembedded.org
>
>
> *Subject:* Re: [OE-core] [PATCH] libyui
>
>
>
> Again, it sounds like this should be in another layer.
>
>
>
> Ross
>
>
>
> On 4 December 2017 at 02:01, Zheng, Ruoqin 
> wrote:
>
> Hi Alex:
>I attend to make dnfdragora(which is a gui for dnf) used in oe-core,
> and libyui is the depend of dnfdragora.
>
> --
> Zheng Ruoqin
> Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST)
> ADDR.: No.6 Wenzhu Road, Software Avenue,
>Nanjing, 210012, China
> MAIL : zhengrq.f...@cn.fujistu.com
>
>
>
> -Original Message-
> From: Alexander Kanavin [mailto:alexander.kana...@linux.intel.com]
> Sent: Wednesday, November 29, 2017 2:48 PM
> To: Zheng, Ruoqin/郑 若钦 ;
> openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH] libyui
>
> On 11/29/2017 07:10 AM, zhengrq wrote:
> > Add a new recipe libyui.
>
>
> Add why? What does it do? Why should it be in oe-core?
>
> Alex
>
>
>
>
> --
> ___
> 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
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] libyui

[Zheng, Ruoqin]

Hi Ross!
 Where should I put this recipe?
--
Zheng Ruoqin
Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST)
ADDR.: No.6 Wenzhu Road, Software Avenue,
   Nanjing, 210012, China
MAIL : zhengrq.f...@cn.fujistu.com

From: Burton, Ross [mailto:ross.bur...@intel.com]
Sent: Monday, December 04, 2017 6:17 PM
To: Zheng, Ruoqin/郑 若钦 
Cc: Alexander Kanavin ; 
openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] libyui

Again, it sounds like this should be in another layer.

Ross

On 4 December 2017 at 02:01, Zheng, Ruoqin 
mailto:zhengrq.f...@cn.fujitsu.com>> wrote:
Hi Alex:
   I attend to make dnfdragora(which is a gui for dnf) used in oe-core, and 
libyui is the depend of dnfdragora.

--
Zheng Ruoqin
Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST)
ADDR.: No.6 Wenzhu Road, Software Avenue,
   Nanjing, 210012, China
MAIL : zhengrq.f...@cn.fujistu.com


-Original Message-
From: Alexander Kanavin 
[mailto:alexander.kana...@linux.intel.com]
Sent: Wednesday, November 29, 2017 2:48 PM
To: Zheng, Ruoqin/郑 若钦 
mailto:zhengrq.f...@cn.fujitsu.com>>; 
openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] libyui

On 11/29/2017 07:10 AM, zhengrq wrote:
> Add a new recipe libyui.


Add why? What does it do? Why should it be in oe-core?

Alex




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



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

[OE-core] [pyro][PATCH] staging.bbclass: handle postinst-useradd-* fixmes

[Matthew McClintock]

From: Mikko Ylinen 

After 02457ef7f600ce954874e2d11e74b1c6daaa3bfc, PSEUDO for
postinst-useradd-* scripts get to use only one PSEUDO_LOCALSTATEDIR
which is set under recipes ${WORKDIR}.

When the those scripts are run in a clean build environment that
is built from the sstate (populate_sysroot_setscene run for
postinst-useradd-* providers), pseudo fails to run because it cannot
access the PSEUDO_LOCALSTATEDIR (recipe ${WORKDIR}s do not exist).
This triggers a sysroot staging error.

Previously, the PSEUDO_LOCALSTATEDIR setting in useradd.bbclass
worked because the RSS sstate/staging logic automagically processed
${STAGING_DIR_TARGET} in postinst-useradd-* scripts to point under
the sysroot being built.

The fix uses the same fixme processing by adding PSEUDO_LOCALSTATEDIR
variable to it. Furthermore, LOGFIFO is added to be able to use
the logging fifo of the recipe that actually runs postinst-useradd-*.

Signed-off-by: Mikko Ylinen 
Signed-off-by: Ross Burton 
---
 meta/classes/staging.bbclass | 2 +-
 meta/classes/useradd.bbclass | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/meta/classes/staging.bbclass b/meta/classes/staging.bbclass
index a90cf43c94..984051d6aa 100644
--- a/meta/classes/staging.bbclass
+++ b/meta/classes/staging.bbclass
@@ -249,7 +249,7 @@ def staging_processfixme(fixme, target, recipesysroot, 
recipesysrootnative, d):
 if not fixme:
 return
 cmd = "sed -e 's:^[^/]*/:%s/:g' %s | xargs sed -i -e 
's:FIXMESTAGINGDIRTARGET:%s:g; s:FIXMESTAGINGDIRHOST:%s:g'" % (target, " 
".join(fixme), recipesysroot, recipesysrootnative)
-for fixmevar in ['COMPONENTS_DIR', 'HOSTTOOLS_DIR', 'PKGDATA_DIR']:
+for fixmevar in ['COMPONENTS_DIR', 'HOSTTOOLS_DIR', 'PKGDATA_DIR', 
'PSEUDO_LOCALSTATEDIR', 'LOGFIFO']:
 fixme_path = d.getVar(fixmevar)
 cmd += " -e 's:FIXME_%s:%s:g'" % (fixmevar, fixme_path)
 bb.debug(2, cmd)
diff --git a/meta/classes/useradd.bbclass b/meta/classes/useradd.bbclass
index 0f51e5522b..0f551b50f3 100644
--- a/meta/classes/useradd.bbclass
+++ b/meta/classes/useradd.bbclass
@@ -133,9 +133,10 @@ useradd_sysroot () {
 }
 
 # The export of PSEUDO in useradd_sysroot() above contains references to
-# ${COMPONENTS_DIR}. These need to be handled when restoring
+# ${COMPONENTS_DIR} and ${PSEUDO_LOCALSTATEDIR}. Additionally, the logging
+# shell functions use ${LOGFIFO}. These need to be handled when restoring
 # postinst-useradd-${PN} from the sstate cache.
-EXTRA_STAGING_FIXMES += "COMPONENTS_DIR"
+EXTRA_STAGING_FIXMES += "COMPONENTS_DIR PSEUDO_LOCALSTATEDIR LOGFIFO"
 
 python useradd_sysroot_sstate () {
 task = d.getVar("BB_CURRENTTASK")
-- 
2.14.1

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

[OE-core] [PATCHv2] recipes-multimedia/gstreamer: Add gst-validate recipe

[Aníbal Limón]

From: Aníbal Limón 

The gst-validate is a tool to run integration tests of Gstreamer
components [1].

This tool can be used along with gst-integration-testsuites (scenarios
and media) [2] to test Gstreamer components on the target device.

An example of test using gst-integration-testsuites:

$ gst-validate-launcher --sync # get [2] uses git and git-annex
$ gst-validate-launcher

or

$ gst-validate-launcher -nd # needs xserver-xorg-xvfb to run wo DISPLAY

[1]
https://blogs.gnome.org/tsaunier/2014/04/21/gst-validate-a-suite-of-tools-to-run-integration-tests-for-gstreamer-2/
[2] https://cgit.freedesktop.org/gstreamer/gst-integration-testsuites

Signed-off-by: Aníbal Limón 
---
 .../gstreamer/gst-validate_1.12.3.bb | 20 
 1 file changed, 20 insertions(+)
 create mode 100644 meta/recipes-multimedia/gstreamer/gst-validate_1.12.3.bb

diff --git a/meta/recipes-multimedia/gstreamer/gst-validate_1.12.3.bb 
b/meta/recipes-multimedia/gstreamer/gst-validate_1.12.3.bb
new file mode 100644
index 00..47f4a260f8
--- /dev/null
+++ b/meta/recipes-multimedia/gstreamer/gst-validate_1.12.3.bb
@@ -0,0 +1,20 @@
+SUMMARY = "Gstreamer validation tool"
+DESCRIPTION = "A Tool to test GStreamer components"
+HOMEPAGE = 
"https://gstreamer.freedesktop.org/releases/gst-validate/1.12.3.html";
+SECTION = "multimedia"
+
+LICENSE = "LGPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343"
+
+SRC_URI = "https://gstreamer.freedesktop.org/src/${PN}/${PN}-${PV}.tar.xz";
+SRC_URI[md5sum] = "623edc479a1e5c1e76bd7e1cf8393253"
+SRC_URI[sha256sum] = 
"5139949d20274fdd702492438eeab2c9e55aa82f60aca17db27ebd3faf08489e"
+
+DEPENDS = "json-glib glib-2.0 gstreamer1.0 gstreamer1.0-plugins-base"
+
+# to use non display mode in gst-validate-launcher
+RDEPENDS_${PN} = "xserver-xorg-xvfb" 
+
+FILES_${PN} += "${datadir}/gstreamer-1.0/* ${libdir}/gst-validate-launcher/* 
${libdir}/gstreamer-1.0/*"
+
+inherit pkgconfig gettext autotools gobject-introspection gtk-doc
-- 
2.11.0

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

Re: [OE-core] [PATCH 5/5] lib/oe/package.py: is_elf: Make it less prone to false positives

[Burton, Ross]

If you are looking at this then you'll want to pull my branch again as I
just pushed a few fixes to make ARM binaries actually work.

Also, a prototype "is this binary stripped" function would be:

elf = oe.elf.Elf.from_file(sys.argv[1])
for h in elf.header.section_headers:
if h.type == oe.elf.Elf.ShType.progbits and h.name == ".debug_info":
return False
return True

If you're not working on this yet, please say, as I'll probably have a look
this week.

Ross


On 4 December 2017 at 15:33, Burton, Ross  wrote:

> On 4 December 2017 at 15:30, Olof Johansson 
> wrote:
>
>> On 17-12-04 12:36 +, Burton, Ross wrote:
>> > You might be interested in some of the patches I've got sitting in
>> > http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=ross/mutb
>> > specifically around "Add new ELF parser".  This adds a fully-featured
>> > Python ELF parser to lib/oe which could be used to inspect the binaries
>> the
>> > way file does, but without having to call and parse the output of file.
>>
>> Oh, that's awesome. Please disregard v1 of my patch series, and
>> I'll try to adapt package.bbclass and lib/oe/package.py to make
>> use of your changes and see what happens. Thanks!
>>
>
> Hopefully to get what you want out of it doesn't involve too much pain...
>
> I still need to finish the series of to get it ready for merging, but I've
> just pushed a new branch ross/elf to poky-contrib which has just the
> patches.  As I clean it up I'll force push there, so feel free to
> track/rebase/send improvements.
>
> Ross
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] ✗ patchtest: failure for cross-localedef-native: Include locale_t.h

[Patchwork]

== Series Details ==

Series: cross-localedef-native: Include locale_t.h
Revision: 1
URL   : https://patchwork.openembedded.org/series/10101/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue A patch file has been added, but does not have a 
Signed-off-by tag [test_signed_off_by_presence] 
  Suggested fixSign off the added patch file 
(meta/recipes-core/glibc/glibc/0001-Include-locale_t.h-compatibility-header.patch)

* Issue Upstream-Status is Inappropriate, but no reason was 
provided [test_upstream_status_presence_format] 
  Suggested fixInclude a brief reason why 
0001-Include-locale_t.h-compatibility-header.patch is inappropriate
  Current  Upstream-Status: Inappropriate
  Standard format  Upstream-Status: Inappropriate [reason]



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

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

[OE-core] [morty][PATCH] cross-localedef-native: Include locale_t.h

[Joshua Watt]

Newer versions of glibc (2.26) moved the struct locale definition from
xlocale.h to bits/types/locale_t.h. For compatibility with build hosts
using this version of glibc, include this header.

See f0be25b6336db7492e47d2e8e72eb8af53b5506d in glibc

Signed-off-by: Joshua Watt 
---
 .../glibc/cross-localedef-native_2.24.bb   |  1 +
 ...1-Include-locale_t.h-compatibility-header.patch | 27 ++
 2 files changed, 28 insertions(+)
 create mode 100644 
meta/recipes-core/glibc/glibc/0001-Include-locale_t.h-compatibility-header.patch

diff --git a/meta/recipes-core/glibc/cross-localedef-native_2.24.bb 
b/meta/recipes-core/glibc/cross-localedef-native_2.24.bb
index d4cccedb43a..4a77eee2c9b 100644
--- a/meta/recipes-core/glibc/cross-localedef-native_2.24.bb
+++ b/meta/recipes-core/glibc/cross-localedef-native_2.24.bb
@@ -36,6 +36,7 @@ SRC_URI = "${GLIBC_GIT_URI};branch=${SRCBRANCH};name=glibc \
file://0023-eglibc-Install-PIC-archives.patch \

file://0024-eglibc-Forward-port-cross-locale-generation-support.patch \
file://0025-Define-DUMMY_LOCALE_T-if-not-defined.patch \
+   file://0001-Include-locale_t.h-compatibility-header.patch \
 "
 # Makes for a rather long rev (22 characters), but...
 #
diff --git 
a/meta/recipes-core/glibc/glibc/0001-Include-locale_t.h-compatibility-header.patch
 
b/meta/recipes-core/glibc/glibc/0001-Include-locale_t.h-compatibility-header.patch
new file mode 100644
index 000..634f8d86441
--- /dev/null
+++ 
b/meta/recipes-core/glibc/glibc/0001-Include-locale_t.h-compatibility-header.patch
@@ -0,0 +1,27 @@
+From abfeb0cf4e3261a66a7a23abc9aed33c034c850d Mon Sep 17 00:00:00 2001
+From: Joshua Watt 
+Date: Wed, 6 Dec 2017 13:26:19 -0600
+Subject: [PATCH] Include locale_t.h compatibility header
+
+Newer versions of glibc (since 2.26) moved the locale typedefs from
+xlocale.h to bits/types/locale_t.h. Create a compatibility header for
+these newer versions of glibc
+
+See f0be25b6336db7492e47d2e8e72eb8af53b5506d in glibc
+
+Upstream-Status: Inappropriate
+---
+ locale/bits/types/locale_t.h | 1 +
+ 1 file changed, 1 insertion(+)
+ create mode 100644 locale/bits/types/locale_t.h
+
+diff --git a/locale/bits/types/locale_t.h b/locale/bits/types/locale_t.h
+new file mode 100644
+index 00..b519a6c5f8
+--- /dev/null
 b/locale/bits/types/locale_t.h
+@@ -0,0 +1 @@
++#include 
+-- 
+2.14.3
+
-- 
2.14.3

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

[OE-core] [oe-core][PATCH 1/1] systemd: fix segfault when terminating systemd --test

[Joe Slater]

Currently, if "systemd --test" is not allowed to complete
sending output, it will segfault.

Signed-off-by: Joe Slater 
---
 ...te-presets-after-generators-have-run-6526.patch |  69 +
 ...any-initialization-steps-when-running-in-.patch | 163 +
 meta/recipes-core/systemd/systemd_234.bb   |   2 +
 3 files changed, 234 insertions(+)
 create mode 100644 
meta/recipes-core/systemd/systemd/0001-core-evaluate-presets-after-generators-have-run-6526.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/0001-main-skip-many-initialization-steps-when-running-in-.patch

diff --git 
a/meta/recipes-core/systemd/systemd/0001-core-evaluate-presets-after-generators-have-run-6526.patch
 
b/meta/recipes-core/systemd/systemd/0001-core-evaluate-presets-after-generators-have-run-6526.patch
new file mode 100644
index 000..df100e5
--- /dev/null
+++ 
b/meta/recipes-core/systemd/systemd/0001-core-evaluate-presets-after-generators-have-run-6526.patch
@@ -0,0 +1,69 @@
+From 28dd66ecfce743b1ea9046c7bb501e0fcaeff724 Mon Sep 17 00:00:00 2001
+From: Luca Bruno 
+Date: Sun, 6 Aug 2017 13:24:24 +
+Subject: [PATCH] core: evaluate presets after generators have run (#6526)
+
+This commit moves the first-boot system preset-settings evaluation out
+of main and into the manager startup logic itself. Notably, it reverses
+the order between generators and presets evaluation, so that any changes
+performed by first-boot generators are taken into the account by presets
+logic.
+
+After this change, units created by a generator can be enabled as part
+of a preset.
+
+Upstream-Status: Backport
+
+Signed-off-by: Catalin Enache 
+---
+ src/core/main.c| 12 ++--
+ src/core/manager.c |  8 
+ 2 files changed, 10 insertions(+), 10 deletions(-)
+
+diff --git a/src/core/main.c b/src/core/main.c
+index dfedc3d..11ac9cf 100644
+--- a/src/core/main.c
 b/src/core/main.c
+@@ -1809,18 +1809,10 @@ int main(int argc, char *argv[]) {
+ if (prctl(PR_SET_CHILD_SUBREAPER, 1) < 0)
+ log_warning_errno(errno, "Failed to make us a 
subreaper: %m");
+ 
+-if (arg_system) {
++if (arg_system)
++/* Bump up RLIMIT_NOFILE for systemd itself */
+ (void) bump_rlimit_nofile(&saved_rlimit_nofile);
+ 
+-if (empty_etc) {
+-r = unit_file_preset_all(UNIT_FILE_SYSTEM, 0, NULL, 
UNIT_FILE_PRESET_ENABLE_ONLY, NULL, 0);
+-if (r < 0)
+-log_full_errno(r == -EEXIST ? LOG_NOTICE : 
LOG_WARNING, r, "Failed to populate /etc with preset unit settings, ignoring: 
%m");
+-else
+-log_info("Populated /etc with preset unit 
settings.");
+-}
+-}
+-
+ r = manager_new(arg_system ? UNIT_FILE_SYSTEM : UNIT_FILE_USER, 
arg_action == ACTION_TEST, &m);
+ if (r < 0) {
+ log_emergency_errno(r, "Failed to allocate manager object: 
%m");
+diff --git a/src/core/manager.c b/src/core/manager.c
+index 1aadb70..fb5e2b5 100644
+--- a/src/core/manager.c
 b/src/core/manager.c
+@@ -1328,6 +1328,14 @@ int manager_startup(Manager *m, FILE *serialization, 
FDSet *fds) {
+ if (r < 0)
+ return r;
+ 
++if (m->first_boot && m->unit_file_scope == UNIT_FILE_SYSTEM) {
++q = unit_file_preset_all(UNIT_FILE_SYSTEM, 0, NULL, 
UNIT_FILE_PRESET_ENABLE_ONLY, NULL, 0);
++if (q < 0)
++log_full_errno(q == -EEXIST ? LOG_NOTICE : 
LOG_WARNING, q, "Failed to populate /etc with preset unit settings, ignoring: 
%m");
++else
++log_info("Populated /etc with preset unit settings.");
++}
++
+ lookup_paths_reduce(&m->lookup_paths);
+ manager_build_unit_path_cache(m);
+ 
+-- 
+2.10.2
+
diff --git 
a/meta/recipes-core/systemd/systemd/0001-main-skip-many-initialization-steps-when-running-in-.patch
 
b/meta/recipes-core/systemd/systemd/0001-main-skip-many-initialization-steps-when-running-in-.patch
new file mode 100644
index 000..a033b04
--- /dev/null
+++ 
b/meta/recipes-core/systemd/systemd/0001-main-skip-many-initialization-steps-when-running-in-.patch
@@ -0,0 +1,163 @@
+From dea374e898a749a0474b72b2015cca9009b1432b Mon Sep 17 00:00:00 2001
+From: Lennart Poettering 
+Date: Wed, 13 Sep 2017 10:31:40 +0200
+Subject: [PATCH] main: skip many initialization steps when running in --test
+ mode
+
+Most importantly, don't collect open socket activation fds when in
+--test mode. This specifically created a problem because we invoke
+pager_open() beforehand (which these days makes copies of the original
+stdout/stderr in order to be able to restore them when the pager goes
+away) and we might mistakenly the fd copies it creates as socket
+activation fds.
+
+Fixes: #6383
+
+Upstream-Status: Backport
+
+Signed-off-by: Catalin Enache 
+---
+ src/core

Re: [OE-core] [rocko PATCH 1/3] Revert "go: Fix build with PIE on musl"

[Anibal Limón]

On Wed, Dec 6, 2017 at 5:17 AM, Otavio Salvador 
wrote:

> This reverts commit d6fcf91c06a3d118e8741273fac6903100141db4.
>
> This commit was included on the rocko update by mistake. It ended
> being dropped from master merge queue but forgotten in rocko one.
>

Also causes a build failure on sd-600 [1] when linking containerd-docker
(meta-virtualization) to libgcc due to pass -shared and -pie by default.

...
08:51:05 | cd ctr && go build -ldflags "-w -extldflags -static -X
github.com/containerd/containerd.GitCommit=3addd840653146c90a254301d6c3a663c7fd6429
" -tags "" -o ../bin/ctr
08:51:05 | cd containerd && go build -ldflags "-w -extldflags -static -X
github.com/containerd/containerd.GitCommit=3addd840653146c90a254301d6c3a663c7fd6429
" -tags "" -o ../bin/containerd
08:51:05 | cd containerd-shim && go build -ldflags "-w -extldflags -static
-X
github.com/containerd/containerd.GitCommit=3addd840653146c90a254301d6c3a663c7fd6429
" -tags "" -o ../bin/containerd-shim
08:51:05 | # github.com/containerd/containerd/containerd-shim
08:51:05 |
/srv/oe/build/tmp-rpb_wayland-glibc/work/armv7ahf-neon-linaro-linux-gnueabi/containerd-docker/v0.2.x+git3addd840653146c90a254301d6c3a663c7fd6429-r0/recipe-sysroot-native/usr/lib/arm-linaro-linux-gnueabi/go/pkg/tool/linux_amd64/link:
running arm-linaro-linux-gnueabi-gcc failed: exit status 1
08:51:05 |
/srv/oe/build/tmp-rpb_wayland-glibc/work/armv7ahf-neon-linaro-linux-gnueabi/containerd-docker/v0.2.x+git3addd840653146c90a254301d6c3a663c7fd6429-r0/recipe-sysroot-native/usr/bin/arm-linaro-linux-gnueabi/../../libexec/arm-linaro-linux-gnueabi/gcc/arm-linaro-linux-gnueabi/7.1.1/ld:
/srv/oe/build/tmp-rpb_wayland-glibc/work/armv7ahf-neon-linaro-linux-gnueabi/containerd-docker/v0.2.x+git3addd840653146c90a254301d6c3a663c7fd6429-r0/recipe-sysroot/usr/lib/arm-linaro-linux-gnueabi/7.1.1/crtbeginT.o:
relocation R_ARM_MOVW_ABS_NC against `a local symbol' can not be used when
making a shared object; recompile with -fPIC
...

[1]
https://ci.linaro.org/job/lt-qcom-openembedded-rpb-rocko/6/DISTRO=rpb-wayland,MACHINE=sd-600eval,label=docker-stretch-amd64/console

Cheers,
Anibal


>
> Signed-off-by: Otavio Salvador 
> ---
>
>  meta/recipes-devtools/go/go-1.9.inc|   3 -
>  .../go/go-1.9/default-buildmode-pie.patch  |  18 
>  .../go/go-1.9/set-external-linker.patch| 111
> -
>  3 files changed, 132 deletions(-)
>  delete mode 100644 meta/recipes-devtools/go/go-1.
> 9/default-buildmode-pie.patch
>  delete mode 100644 meta/recipes-devtools/go/go-1.
> 9/set-external-linker.patch
>
> diff --git a/meta/recipes-devtools/go/go-1.9.inc
> b/meta/recipes-devtools/go/go-1.9.inc
> index f52abb5735..65adaa8d72 100644
> --- a/meta/recipes-devtools/go/go-1.9.inc
> +++ b/meta/recipes-devtools/go/go-1.9.inc
> @@ -15,9 +15,6 @@ SRC_URI += "\
>  file://0007-ld-add-soname-to-shareable-objects.patch \
>  
> file://0008-make.bash-add-GOHOSTxx-indirection-for-cross-canadia.patch
> \
>  
> file://0009-cmd-go-buildmode-pie-forces-external-linking-mode-on.patch
> \
> -file://default-buildmode-pie.patch \
>  "
> -SRC_URI_append_libc-musl = " file://set-external-linker.patch"
> -
>  SRC_URI[main.md5sum] = "da2d44ea384076efec43ee1f8b7d45d2"
>  SRC_URI[main.sha256sum] = "a4ab229028ed167ba1986825751463
> 605264e44868362ca8e7accc8be057e993"
> diff --git a/meta/recipes-devtools/go/go-1.9/default-buildmode-pie.patch
> b/meta/recipes-devtools/go/go-1.9/default-buildmode-pie.patch
> deleted file mode 100644
> index a7933bd39e..00
> --- a/meta/recipes-devtools/go/go-1.9/default-buildmode-pie.patch
> +++ /dev/null
> @@ -1,18 +0,0 @@
> -Default to PIE on linux platforms
> -
> -Upstream-Status: Pending
> -Signed-off-by: Khem Raj 
> -
> -diff -upr src/go.orig/src/cmd/go/internal/work/build.go
> src/go/src/cmd/go/internal/work/build.go
>  go.orig/src/cmd/go/internal/work/build.go  2017-08-27
> 17:38:26.354750979 +0200
> -+++ go/src/cmd/go/internal/work/build.go   2017-08-27
> 17:40:27.555130105 +0200
> -@@ -304,7 +304,8 @@ func BuildModeInit() {
> -   ldBuildmode = "c-shared"
> -   case "default":
> -   switch platform {
> --  case "android/arm", "android/arm64", "android/amd64",
> "android/386":
> -+  case "linux/386", "linux/amd64", "linux/arm",
> "linux/arm64", "linux/ppc64le", "linux/s390x",
> -+  "android/arm", "android/arm64", "android/amd64",
> "android/386":
> -   codegenArg = "-shared"
> -   ldBuildmode = "pie"
> -   case "darwin/arm", "darwin/arm64":
> diff --git a/meta/recipes-devtools/go/go-1.9/set-external-linker.patch
> b/meta/recipes-devtools/go/go-1.9/set-external-linker.patch
> deleted file mode 100644
> index d6bd7fa39c..00
> --- a/meta/recipes-devtools/go/go-1.9/set-external-linker.patch
> +++ /dev/null
> @@ -1,111 +0,0 @@
> -Change the dynamic linker hardcoding to use musl when not usin

Re: [OE-core] [PATCH] sudo: provide sudo paths through EXTRA_OECONF

[Alexander Kanavin]

On 12/06/2017 12:28 PM, Nikolay Merinov wrote:
]> +  --with-vardir=/var/lib/sudo \

+--with-iologdir=/var/log/sudo-io \
+--with-rundir=/var/run/sudo \
+--with-logpath=/var/log/sudo.log \



Don't hardcode /var here, use ${localstatedir}.

Alex

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

Re: [OE-core] [PATCH 8/8] linux-yocto-dev: update to v4.15+

[Burton, Ross]

The amended commit has mismatching commit log and patch (target elfutils is
in the commit log but not the patch), and doesn't mention installing target
elfutils if they want it.

Ross

On 4 December 2017 at 17:43, Bruce Ashfield 
wrote:

> On 2017-12-04 12:39 PM, Richard Purdie wrote:
>
>> On Mon, 2017-12-04 at 12:24 -0500, Bruce Ashfield wrote:
>>
>>> On 2017-12-04 11:38 AM, Khem Raj wrote:
>>>

 On Mon, Dec 4, 2017 at 7:39 AM, Bruce Ashfield
  wrote:

>
> Outside of the normal patch refreshes and boot issues, there are
> new
> build time tools within the kernel that required the following
> dependencies:
>
> For ORC_UNWINDER support in x86-64:
>
>DEPENDS += "${@bb.utils.contains('ARCH', 'x86', 'elfutils-
> native elfutils', '', d)}"
>
 do we need both target and host elfutils

>>> Yup. There were references to both. Some had to run for hostcc
>>> and others in the target arch.
>>>
>>
>> Just for reference this is pretty bad for performance as it delays the
>> kernel compile until some substantial parts of userspace build.
>>
>>
> On a second look, I can likely turn off the target part, if someone
> wants it, they can always install the package (or it could be a
> rdepends).
>
> I'll amend the commit and leave it on that branch with just the
> DEPENDS on the -native version.
>
> Is ORC_UNWINDER useful and commonly used?
>>
>
> The upstream kernel commit turned it on by default, I turned it off in
> the kernel-cache, but I wanted to make sure the dependency was
> in place.
>
> The commit series from Josh Poimboeuf all lead to it being on as
> the default choice (even with a slight overhead).
>
> Bruce
>
>
>
>> Cheers,
>>
>> Richard
>>
>>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] ✗ patchtest: failure for "libxslt: use HTTP instead of F..." and 1 more

[Leonardo Sandoval]

On Wed, 6 Dec 2017 16:03:13 +
"Burton, Ross"  wrote:

> On 6 December 2017 at 16:00, Leonardo Sandoval <
> leonardo.sandoval.gonza...@linux.intel.com> wrote:  
> 
> > On Wed, 06 Dec 2017 15:02:57 -
> > Patchwork  wrote:
> >  
> > > == Series Details ==
> > >
> > > Series: "libxslt: use HTTP instead of F..." and 1 more
> > > Revision: 1
> > > URL   : https://patchwork.openembedded.org/series/10099/
> > > State : failure
> > >
> > > == Summary ==
> > >
> > >
> > > Thank you for submitting this patch series to OpenEmbedded Core. This is
> > > an automated response. Several tests have been executed on the proposed
> > > series by patchtest resulting in the following failures:
> > >
> > >
> > >
> > > * Issue SRC_URI changed but checksums are the same  
> > [test_src_uri_checksums_not_changed]  
> > >   Suggested fixInclude the SRC_URI's checksums changes into your  
> > patch
> >
> >
> > I believe we need to improve this check. checksums do not necessary change
> > if URL changes.
> >  
> 
> Not sure how to improve it.  Maybe making the wording vague, as that is a
> common mistake and the URL changing but checksums not changing is  not
> common.

Rigth. Also bitbake fails if downloaded tarball does not match its checksums 
with the ones defined on recipe. Time to disable this check.

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

Re: [OE-core] ✗ patchtest: failure for "libxslt: use HTTP instead of F..." and 1 more

[Burton, Ross]

On 6 December 2017 at 16:00, Leonardo Sandoval <
leonardo.sandoval.gonza...@linux.intel.com> wrote:

> On Wed, 06 Dec 2017 15:02:57 -
> Patchwork  wrote:
>
> > == Series Details ==
> >
> > Series: "libxslt: use HTTP instead of F..." and 1 more
> > Revision: 1
> > URL   : https://patchwork.openembedded.org/series/10099/
> > State : failure
> >
> > == Summary ==
> >
> >
> > Thank you for submitting this patch series to OpenEmbedded Core. This is
> > an automated response. Several tests have been executed on the proposed
> > series by patchtest resulting in the following failures:
> >
> >
> >
> > * Issue SRC_URI changed but checksums are the same
> [test_src_uri_checksums_not_changed]
> >   Suggested fixInclude the SRC_URI's checksums changes into your
> patch
>
>
> I believe we need to improve this check. checksums do not necessary change
> if URL changes.
>

Not sure how to improve it.  Maybe making the wording vague, as that is a
common mistake and the URL changing but checksums not changing is  not
common.

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

Re: [OE-core] ✗ patchtest: failure for "libxslt: use HTTP instead of F..." and 1 more

[Leonardo Sandoval]

On Wed, 06 Dec 2017 15:02:57 -
Patchwork  wrote:

> == Series Details ==
> 
> Series: "libxslt: use HTTP instead of F..." and 1 more
> Revision: 1
> URL   : https://patchwork.openembedded.org/series/10099/
> State : failure
> 
> == Summary ==
> 
> 
> Thank you for submitting this patch series to OpenEmbedded Core. This is
> an automated response. Several tests have been executed on the proposed
> series by patchtest resulting in the following failures:
> 
> 
> 
> * Issue SRC_URI changed but checksums are the same 
> [test_src_uri_checksums_not_changed] 
>   Suggested fixInclude the SRC_URI's checksums changes into your patch


I believe we need to improve this check. checksums do not necessary change if 
URL changes.

> 
> 
> 
> If you believe any of these test results are incorrect, please reply to the
> mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
> Otherwise we would appreciate you correcting the issues and submitting a new
> version of the patchset if applicable. Please ensure you add/increment the
> version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
> [PATCH v3] -> ...).
> 
> ---
> Guidelines: 
> https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
> Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
> Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe
> 
> -- 
> ___
> 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] core/loader.py: fix regex to include all available test cases

[Leonardo Sandoval]

On Wed, 6 Dec 2017 11:50:58 +
Joshua Lock  wrote:

> On 05/12/17 15:51, leonardo.sandoval.gonza...@linux.intel.com wrote:
> > From: Leonardo Sandoval 
> > 
> > Some test cases (i.e. eSDK.oeSDKExtSelfTest*) does not match with current 
> > regex,
> > fix it to accept these (and previous ones).
> > 
> > Without it, the following runtime exception is observed when executing eSDK
> > selftest (oe-selftest -r eSDK)
> > 
> >  Traceback (most recent call last):
> >File "/openembedded-core/scripts/oe-selftest", line 70, in 
> >  ret = main()
> >File "/openembedded-core/scripts/oe-selftest", line 57, in main
> >  results = args.func(logger, args)
> >File "/openembedded-core/meta/lib/oeqa/selftest/context.py", line 
> > 253, in run
> >  rc = self._internal_run(logger, args)
> >File "/openembedded-core/meta/lib/oeqa/selftest/context.py", line 
> > 205, in _internal_run
> >  self.tc.loadTests(self.module_paths, **self.tc_kwargs['load'])
> >File "/openembedded-core/meta/lib/oeqa/core/context.py", line 58, in 
> > loadTests
> >  modules_required, filters)
> >File "/openembedded-core/meta/lib/oeqa/core/loader.py", line 69, in 
> > __init__
> >  self.modules = _built_modules_dict(modules)
> >File "/openembedded-core/meta/lib/oeqa/core/loader.py", line 48, in 
> > _built_modules_dict
> >  module_name, class_name, test_name = m.groups()
> >  AttributeError: 'NoneType' object has no attribute 'groups'
> > 
> > Signed-off-by: Leonardo Sandoval 
> >   
> 
> LGTM and fixes [YOCTO #12385], thanks!
> 

unfortunately, I found another way that this breaks ('bitbake  -c testimage 
core-image-full-cmdline', which also uses the core loader) so there is a need 
for a V2. 

> Acked-by: Joshua Lock 
> 
> > ---
> >   meta/lib/oeqa/core/loader.py | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta/lib/oeqa/core/loader.py b/meta/lib/oeqa/core/loader.py
> > index 975a081ba4..9ab4df0a60 100644
> > --- a/meta/lib/oeqa/core/loader.py
> > +++ b/meta/lib/oeqa/core/loader.py
> > @@ -43,7 +43,7 @@ def _built_modules_dict(modules):
> >   for module in modules:
> >   # Assumption: package and module names do not contain upper case
> >   # characters, whereas class names do
> > -m = re.match(r'^([^A-Z]+)(?:\.([A-Z][^.]*)(?:\.([^.]+))?)?$', 
> > module)
> > +m = re.match(r'^([a-zA-Z]+)(?:\.([a-zA-Z][^.]*)(?:\.([^.]+))?)?$', 
> > module)
> >   
> >   module_name, class_name, test_name = m.groups()
> >   
> >   
> -- 
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] ✗ patchtest: failure for "libxslt: use HTTP instead of F..." and 1 more

[Patchwork]

== Series Details ==

Series: "libxslt: use HTTP instead of F..." and 1 more
Revision: 1
URL   : https://patchwork.openembedded.org/series/10099/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue SRC_URI changed but checksums are the same 
[test_src_uri_checksums_not_changed] 
  Suggested fixInclude the SRC_URI's checksums changes into your patch



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

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

[OE-core] [PATCH 2/2] libxslt: remove inappropriate patch

[Ross Burton]

This was patching -lxslt directly into the pkgconfig file, but XSLT_LIBS already
contains this so the patch is redundant.

Signed-off-by: Ross Burton 
---
 .../libxslt/libxslt/pkgconfig_fix.patch| 24 --
 meta/recipes-support/libxslt/libxslt_1.1.32.bb |  5 +
 2 files changed, 1 insertion(+), 28 deletions(-)
 delete mode 100644 meta/recipes-support/libxslt/libxslt/pkgconfig_fix.patch

diff --git a/meta/recipes-support/libxslt/libxslt/pkgconfig_fix.patch 
b/meta/recipes-support/libxslt/libxslt/pkgconfig_fix.patch
deleted file mode 100644
index 16a801010ca..000
--- a/meta/recipes-support/libxslt/libxslt/pkgconfig_fix.patch
+++ /dev/null
@@ -1,24 +0,0 @@
-Upstream-Status: Inappropriate [configuration]
-
-Index: libxslt-1.1.27/libexslt.pc.in
-===
 libxslt-1.1.27.orig/libexslt.pc.in
-+++ libxslt-1.1.27/libexslt.pc.in
-@@ -8,5 +8,5 @@ Name: libexslt
- Version: @LIBEXSLT_VERSION@
- Description: EXSLT Extension library
- Requires: libxml-2.0
--Libs: @EXSLT_LIBDIR@ @EXSLT_LIBS@
-+Libs: -lexslt @EXSLT_LIBDIR@ @EXSLT_LIBS@
- Cflags: @EXSLT_INCLUDEDIR@
-Index: libxslt-1.1.27/libxslt.pc.in
-===
 libxslt-1.1.27.orig/libxslt.pc.in
-+++ libxslt-1.1.27/libxslt.pc.in
-@@ -8,5 +8,5 @@ Name: libxslt
- Version: @VERSION@
- Description: XSLT library version 2.
- Requires: libxml-2.0
--Libs: @XSLT_LIBDIR@ @XSLT_LIBS@ @EXTRA_LIBS@
-+Libs: -lxslt @XSLT_LIBDIR@ @XSLT_LIBS@ @EXTRA_LIBS@
- Cflags: @XSLT_INCLUDEDIR@
diff --git a/meta/recipes-support/libxslt/libxslt_1.1.32.bb 
b/meta/recipes-support/libxslt/libxslt_1.1.32.bb
index b8951ab3a8b..6a03f776992 100644
--- a/meta/recipes-support/libxslt/libxslt_1.1.32.bb
+++ b/meta/recipes-support/libxslt/libxslt_1.1.32.bb
@@ -8,10 +8,7 @@ LIC_FILES_CHKSUM = 
"file://Copyright;md5=0cd9a07afbeb24026c9b03aecfeba458"
 SECTION = "libs"
 DEPENDS = "libxml2"
 
-SRC_URI = "http://xmlsoft.org/sources/libxslt-${PV}.tar.gz \
-   file://pkgconfig_fix.patch \
-   "
-
+SRC_URI = "http://xmlsoft.org/sources/libxslt-${PV}.tar.gz";
 SRC_URI[md5sum] = "1fc72f98e98bf4443f1651165f3aa146"
 SRC_URI[sha256sum] = 
"526ecd0abaf4a7789041622c3950c0e7f2c4c8835471515fd77eec684a355460"
 
-- 
2.11.0

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

[OE-core] [PATCH 1/2] libxslt: use HTTP instead of FTP in SRC_URI

[Ross Burton]

FTP isn't as reliable as HTTP.

[ YOCTO #12398 ]

Signed-off-by: Ross Burton 
---
 meta/recipes-support/libxslt/libxslt_1.1.32.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-support/libxslt/libxslt_1.1.32.bb 
b/meta/recipes-support/libxslt/libxslt_1.1.32.bb
index e8b1409d79b..b8951ab3a8b 100644
--- a/meta/recipes-support/libxslt/libxslt_1.1.32.bb
+++ b/meta/recipes-support/libxslt/libxslt_1.1.32.bb
@@ -8,7 +8,7 @@ LIC_FILES_CHKSUM = 
"file://Copyright;md5=0cd9a07afbeb24026c9b03aecfeba458"
 SECTION = "libs"
 DEPENDS = "libxml2"
 
-SRC_URI = "ftp://xmlsoft.org/libxslt/libxslt-${PV}.tar.gz \
+SRC_URI = "http://xmlsoft.org/sources/libxslt-${PV}.tar.gz \
file://pkgconfig_fix.patch \
"
 
-- 
2.11.0

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

Re: [OE-core] [rocko] Go fixups

[Martin Jansa]

Thanks,

this will most likely fix the issue I've reported earlier in:
http://lists.openembedded.org/pipermail/openembedded-core/2017-December/145278.html
as well.

On Wed, Dec 6, 2017 at 12:28 PM, Otavio Salvador <
otavio.salva...@ossystems.com.br> wrote:

> Hello Armin,
>
> There was one patch which was mistakenly forgotten on rocko-next, and
> ended being merged. It ended being reworked in master and I did the
> needed backports/revert in our repo[1].
>
> 1. https://github.com/OSSystems/oe-core/compare/rocko...rocko-next
>
> Please consider applying those soon as rocko is currently with a
> broken Go toolchain.
>
> --
> 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
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] Error caused by kernel-yocto: ensure that only valid BSPs are built

[Bruce Ashfield]

On Tue, Dec 5, 2017 at 6:29 PM, Davis, Michael 
wrote:

> After some more digging I found it is because the KTYPE is defined as
> standard/base and the spp script can only find standard.
>
> Is this expected behavior and yocto-bsp should be fixed or a bug and spp
> should be fixed?
>
>
>

That would be expected from the spp point of view, but if a tool is
generating that invalid KTYPE, that is a bug in the tool.

yocto-bsp was recently depreciated and removed in master recently, so
fixing it may not make sense at this point, but logging it in the bug
tracker (against the tool) wouldn't hurt.

Bruce


>
>
> *From:* Davis, Michael
> *Sent:* Tuesday, December 05, 2017 4:45 PM
> *To:* 'openembedded-core@lists.openembedded.org'
> *Subject:* Error caused by kernel-yocto: ensure that only valid BSPs are
> built
>
>
>
> After upgrading rocko all my builds are failing with the following message.
>
> *ERROR*: linux-yocto-4.12.14+gitAUTOINC+e3c9041f48_1d685baca1-r0.1
> do_kernel_metadata: Could not locate BSP definition for test/standard and
> no defconfig was provided
>
> The source of the issue seems to be commit 44aea7b87307795fe4e089c51d45af
> ccaa2f6525.
>
> This includes a fresh BSP I created with yocto-bsp create, so the helper
> tools are generating examples that don’t work.
>
> bsp_definition=$(spp ${includes} --find -DKMACHINE=${KMACHINE}
> -DKTYPE=${LINUX_KERNEL_TYPE}) always returns nothing even though I have
> .scc files defined.
>
> What is the new expected format to avoid this error?
>
>
>
> Thanks,
>
> Mike
>
>
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>


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

Re: [OE-core] [PATCH] core/loader.py: fix regex to include all available test cases

[Joshua Lock]

On 05/12/17 15:51, leonardo.sandoval.gonza...@linux.intel.com wrote:

From: Leonardo Sandoval 

Some test cases (i.e. eSDK.oeSDKExtSelfTest*) does not match with current regex,
fix it to accept these (and previous ones).

Without it, the following runtime exception is observed when executing eSDK
selftest (oe-selftest -r eSDK)

 Traceback (most recent call last):
   File "/openembedded-core/scripts/oe-selftest", line 70, in 
 ret = main()
   File "/openembedded-core/scripts/oe-selftest", line 57, in main
 results = args.func(logger, args)
   File "/openembedded-core/meta/lib/oeqa/selftest/context.py", line 253, 
in run
 rc = self._internal_run(logger, args)
   File "/openembedded-core/meta/lib/oeqa/selftest/context.py", line 205, 
in _internal_run
 self.tc.loadTests(self.module_paths, **self.tc_kwargs['load'])
   File "/openembedded-core/meta/lib/oeqa/core/context.py", line 58, in 
loadTests
 modules_required, filters)
   File "/openembedded-core/meta/lib/oeqa/core/loader.py", line 69, in 
__init__
 self.modules = _built_modules_dict(modules)
   File "/openembedded-core/meta/lib/oeqa/core/loader.py", line 48, in 
_built_modules_dict
 module_name, class_name, test_name = m.groups()
 AttributeError: 'NoneType' object has no attribute 'groups'

Signed-off-by: Leonardo Sandoval 


LGTM and fixes [YOCTO #12385], thanks!

Acked-by: Joshua Lock 


---
  meta/lib/oeqa/core/loader.py | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/core/loader.py b/meta/lib/oeqa/core/loader.py
index 975a081ba4..9ab4df0a60 100644
--- a/meta/lib/oeqa/core/loader.py
+++ b/meta/lib/oeqa/core/loader.py
@@ -43,7 +43,7 @@ def _built_modules_dict(modules):
  for module in modules:
  # Assumption: package and module names do not contain upper case
  # characters, whereas class names do
-m = re.match(r'^([^A-Z]+)(?:\.([A-Z][^.]*)(?:\.([^.]+))?)?$', module)
+m = re.match(r'^([a-zA-Z]+)(?:\.([a-zA-Z][^.]*)(?:\.([^.]+))?)?$', 
module)
  
  module_name, class_name, test_name = m.groups()
  


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

[OE-core] [rocko] Go fixups

[Otavio Salvador]

Hello Armin,

There was one patch which was mistakenly forgotten on rocko-next, and
ended being merged. It ended being reworked in master and I did the
needed backports/revert in our repo[1].

1. https://github.com/OSSystems/oe-core/compare/rocko...rocko-next

Please consider applying those soon as rocko is currently with a
broken Go toolchain.

-- 
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] [rocko PATCH 3/3] go: ensure use of BUILD_CC when building bootstrap tools

[Otavio Salvador]

From: Matt Madison 

For cross-canadian builds, we were accidentally using
the crosssdk C compiler when building the Go compiler
bootstrap.  Add a patch to the make script to let us
use BUILD_CC, and prepend do_compile to set it in
the local environment to ensure that the trailing
blank gets stripped, since that confuses Go.

[YOCTO #12341]

Signed-off-by: Matt Madison 
Signed-off-by: Ross Burton 
(cherry picked from commit 0dbb860924fc157880b52d8e08bad3c6c6b019b8)
Signed-off-by: Otavio Salvador 
---

 meta/recipes-devtools/go/go-1.9.inc|  1 +
 ...verride-CC-when-building-dist-and-go_boot.patch | 43 ++
 meta/recipes-devtools/go/go-common.inc |  4 ++
 3 files changed, 48 insertions(+)
 create mode 100644 
meta/recipes-devtools/go/go-1.9/0010-make.bash-override-CC-when-building-dist-and-go_boot.patch

diff --git a/meta/recipes-devtools/go/go-1.9.inc 
b/meta/recipes-devtools/go/go-1.9.inc
index 60292651df..7f12241dc1 100644
--- a/meta/recipes-devtools/go/go-1.9.inc
+++ b/meta/recipes-devtools/go/go-1.9.inc
@@ -15,6 +15,7 @@ SRC_URI += "\
 file://0007-ld-add-soname-to-shareable-objects.patch \
 file://0008-make.bash-add-GOHOSTxx-indirection-for-cross-canadia.patch 
\
 file://0009-cmd-go-buildmode-pie-forces-external-linking-mode-on.patch 
\
+file://0010-make.bash-override-CC-when-building-dist-and-go_boot.patch 
\
 "
 SRC_URI_append_libc-musl = " file://set-external-linker.patch"
 
diff --git 
a/meta/recipes-devtools/go/go-1.9/0010-make.bash-override-CC-when-building-dist-and-go_boot.patch
 
b/meta/recipes-devtools/go/go-1.9/0010-make.bash-override-CC-when-building-dist-and-go_boot.patch
new file mode 100644
index 00..83fd78c3d7
--- /dev/null
+++ 
b/meta/recipes-devtools/go/go-1.9/0010-make.bash-override-CC-when-building-dist-and-go_boot.patch
@@ -0,0 +1,43 @@
+From 21d83dd9499e5be30eea28dd7034d1ea2a01c838 Mon Sep 17 00:00:00 2001
+From: Matt Madison 
+Date: Tue, 14 Nov 2017 07:38:42 -0800
+Subject: [PATCH 10/10] make.bash: override CC when building dist and
+ go_bootstrap
+
+For cross-canadian builds, dist and go_bootstrap
+run on the build host, so CC needs to point to the
+build host's C compiler.  Add a BUILD_CC environment
+for this, falling back to $CC if not present.
+
+Upstream-Status: Pending
+
+Signed-off-by: Matt Madison 
+---
+ src/make.bash | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/src/make.bash b/src/make.bash
+index 0bdadc6..f199349 100755
+--- a/src/make.bash
 b/src/make.bash
+@@ -131,7 +131,7 @@ if [ "$GOROOT_BOOTSTRAP" = "$GOROOT" ]; then
+   exit 1
+ fi
+ rm -f cmd/dist/dist
+-GOROOT="$GOROOT_BOOTSTRAP" GOOS="" GOARCH="" "$GOROOT_BOOTSTRAP/bin/go" build 
-o cmd/dist/dist ./cmd/dist
++CC=${BUILD_CC:-${CC}} GOROOT="$GOROOT_BOOTSTRAP" GOOS="" GOARCH="" 
"$GOROOT_BOOTSTRAP/bin/go" build -o cmd/dist/dist ./cmd/dist
+ 
+ # -e doesn't propagate out of eval, so check success by hand.
+ eval $(./cmd/dist/dist env -p || echo FAIL=true)
+@@ -167,7 +167,7 @@ elif [ "$1" = "--host-only" ]; then
+ fi
+ 
+ if [ "$do_host_build" = "yes" ]; then
+-  ./cmd/dist/dist bootstrap $buildall $GO_DISTFLAGS -v # builds 
go_bootstrap
++  CC=${BUILD_CC:-${CC}} ./cmd/dist/dist bootstrap $buildall $GO_DISTFLAGS 
-v # builds go_bootstrap
+   # Delay move of dist tool to now, because bootstrap may clear tool 
directory.
+   mv cmd/dist/dist "$GOTOOLDIR"/dist
+   echo
+-- 
+2.7.4
+
diff --git a/meta/recipes-devtools/go/go-common.inc 
b/meta/recipes-devtools/go/go-common.inc
index ce1eb86812..9af68738a6 100644
--- a/meta/recipes-devtools/go/go-common.inc
+++ b/meta/recipes-devtools/go/go-common.inc
@@ -20,3 +20,7 @@ B = "${S}"
 
 INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
 SSTATE_SCAN_CMD = "true"
+
+do_compile_prepend() {
+   BUILD_CC=${BUILD_CC}
+}
-- 
2.15.1

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

[OE-core] [rocko PATCH 2/3] go: Use right dynamic linker on musl

[Otavio Salvador]

From: Khem Raj 

Signed-off-by: Khem Raj 
Signed-off-by: Ross Burton 
(cherry picked from commit 0d6e83757fc26d3e88bfe3c2437b5c7c9be09118)
Signed-off-by: Otavio Salvador 
---

 meta/recipes-devtools/go/go-1.9.inc|   2 +
 .../go/go-1.9/set-external-linker.patch| 111 +
 2 files changed, 113 insertions(+)
 create mode 100644 meta/recipes-devtools/go/go-1.9/set-external-linker.patch

diff --git a/meta/recipes-devtools/go/go-1.9.inc 
b/meta/recipes-devtools/go/go-1.9.inc
index 65adaa8d72..60292651df 100644
--- a/meta/recipes-devtools/go/go-1.9.inc
+++ b/meta/recipes-devtools/go/go-1.9.inc
@@ -16,5 +16,7 @@ SRC_URI += "\
 file://0008-make.bash-add-GOHOSTxx-indirection-for-cross-canadia.patch 
\
 file://0009-cmd-go-buildmode-pie-forces-external-linking-mode-on.patch 
\
 "
+SRC_URI_append_libc-musl = " file://set-external-linker.patch"
+
 SRC_URI[main.md5sum] = "da2d44ea384076efec43ee1f8b7d45d2"
 SRC_URI[main.sha256sum] = 
"a4ab229028ed167ba1986825751463605264e44868362ca8e7accc8be057e993"
diff --git a/meta/recipes-devtools/go/go-1.9/set-external-linker.patch 
b/meta/recipes-devtools/go/go-1.9/set-external-linker.patch
new file mode 100644
index 00..d6bd7fa39c
--- /dev/null
+++ b/meta/recipes-devtools/go/go-1.9/set-external-linker.patch
@@ -0,0 +1,111 @@
+Change the dynamic linker hardcoding to use musl when not using glibc
+this should be applied conditional to musl being the system C library
+
+Upstream-Status: Inappropriate [Real Fix should be portable across libcs]
+
+Signed-off-by: Khem Raj 
+
+Index: go/src/cmd/link/internal/amd64/obj.go
+===
+--- go.orig/src/cmd/link/internal/amd64/obj.go
 go/src/cmd/link/internal/amd64/obj.go
+@@ -67,7 +67,7 @@ func Init() {
+   ld.Thearch.Append64 = ld.Append64l
+   ld.Thearch.TLSIEtoLE = tlsIEtoLE
+ 
+-  ld.Thearch.Linuxdynld = "/lib64/ld-linux-x86-64.so.2"
++  ld.Thearch.Linuxdynld = "/lib/ld-musl-x86_64.so.1"
+   ld.Thearch.Freebsddynld = "/libexec/ld-elf.so.1"
+   ld.Thearch.Openbsddynld = "/usr/libexec/ld.so"
+   ld.Thearch.Netbsddynld = "/libexec/ld.elf_so"
+Index: go/src/cmd/link/internal/arm/obj.go
+===
+--- go.orig/src/cmd/link/internal/arm/obj.go
 go/src/cmd/link/internal/arm/obj.go
+@@ -63,7 +63,7 @@ func Init() {
+   ld.Thearch.Append32 = ld.Append32l
+   ld.Thearch.Append64 = ld.Append64l
+ 
+-  ld.Thearch.Linuxdynld = "/lib/ld-linux.so.3" // 2 for OABI, 3 for EABI
++  ld.Thearch.Linuxdynld = "/lib/ld-musl-armhf.so.1"
+   ld.Thearch.Freebsddynld = "/usr/libexec/ld-elf.so.1"
+   ld.Thearch.Openbsddynld = "/usr/libexec/ld.so"
+   ld.Thearch.Netbsddynld = "/libexec/ld.elf_so"
+Index: go/src/cmd/link/internal/arm64/obj.go
+===
+--- go.orig/src/cmd/link/internal/arm64/obj.go
 go/src/cmd/link/internal/arm64/obj.go
+@@ -62,7 +62,7 @@ func Init() {
+   ld.Thearch.Append32 = ld.Append32l
+   ld.Thearch.Append64 = ld.Append64l
+ 
+-  ld.Thearch.Linuxdynld = "/lib/ld-linux-aarch64.so.1"
++  ld.Thearch.Linuxdynld = "/lib/ld-musl-aarch64.so.1"
+ 
+   ld.Thearch.Freebsddynld = "XXX"
+   ld.Thearch.Openbsddynld = "XXX"
+Index: go/src/cmd/link/internal/mips/obj.go
+===
+--- go.orig/src/cmd/link/internal/mips/obj.go
 go/src/cmd/link/internal/mips/obj.go
+@@ -77,7 +77,7 @@ func Init() {
+   ld.Thearch.Append64 = ld.Append64b
+   }
+ 
+-  ld.Thearch.Linuxdynld = "/lib/ld.so.1"
++  ld.Thearch.Linuxdynld = "/lib/ld-musl-mipsle.so.1"
+ 
+   ld.Thearch.Freebsddynld = "XXX"
+   ld.Thearch.Openbsddynld = "XXX"
+Index: go/src/cmd/link/internal/mips64/obj.go
+===
+--- go.orig/src/cmd/link/internal/mips64/obj.go
 go/src/cmd/link/internal/mips64/obj.go
+@@ -75,7 +75,7 @@ func Init() {
+   ld.Thearch.Append64 = ld.Append64b
+   }
+ 
+-  ld.Thearch.Linuxdynld = "/lib64/ld64.so.1"
++  ld.Thearch.Linuxdynld = "/lib64/ld-musl-mips64le.so.1"
+ 
+   ld.Thearch.Freebsddynld = "XXX"
+   ld.Thearch.Openbsddynld = "XXX"
+Index: go/src/cmd/link/internal/ppc64/obj.go
+===
+--- go.orig/src/cmd/link/internal/ppc64/obj.go
 go/src/cmd/link/internal/ppc64/obj.go
+@@ -77,7 +77,7 @@ func Init() {
+   }
+ 
+   // TODO(austin): ABI v1 uses /usr/lib/ld.so.1
+-  ld.Thearch.Linuxdynld = "/lib64/ld64.so.1"
++  ld.Thearch.Linuxdynld = "/lib/ld-musl-powerpc64le.so.1"
+ 
+   ld.Thearch.Freebsddynld = "XXX"
+   ld.Thearch.Openbsddynld = "XXX"
+Index: go/src/cmd/link/internal/s390x/obj.go
+===
+--- go.orig/src/cmd/link/i

[OE-core] [rocko PATCH 1/3] Revert "go: Fix build with PIE on musl"

[Otavio Salvador]

This reverts commit d6fcf91c06a3d118e8741273fac6903100141db4.

This commit was included on the rocko update by mistake. It ended
being dropped from master merge queue but forgotten in rocko one.

Signed-off-by: Otavio Salvador 
---

 meta/recipes-devtools/go/go-1.9.inc|   3 -
 .../go/go-1.9/default-buildmode-pie.patch  |  18 
 .../go/go-1.9/set-external-linker.patch| 111 -
 3 files changed, 132 deletions(-)
 delete mode 100644 meta/recipes-devtools/go/go-1.9/default-buildmode-pie.patch
 delete mode 100644 meta/recipes-devtools/go/go-1.9/set-external-linker.patch

diff --git a/meta/recipes-devtools/go/go-1.9.inc 
b/meta/recipes-devtools/go/go-1.9.inc
index f52abb5735..65adaa8d72 100644
--- a/meta/recipes-devtools/go/go-1.9.inc
+++ b/meta/recipes-devtools/go/go-1.9.inc
@@ -15,9 +15,6 @@ SRC_URI += "\
 file://0007-ld-add-soname-to-shareable-objects.patch \
 file://0008-make.bash-add-GOHOSTxx-indirection-for-cross-canadia.patch 
\
 file://0009-cmd-go-buildmode-pie-forces-external-linking-mode-on.patch 
\
-file://default-buildmode-pie.patch \
 "
-SRC_URI_append_libc-musl = " file://set-external-linker.patch"
-
 SRC_URI[main.md5sum] = "da2d44ea384076efec43ee1f8b7d45d2"
 SRC_URI[main.sha256sum] = 
"a4ab229028ed167ba1986825751463605264e44868362ca8e7accc8be057e993"
diff --git a/meta/recipes-devtools/go/go-1.9/default-buildmode-pie.patch 
b/meta/recipes-devtools/go/go-1.9/default-buildmode-pie.patch
deleted file mode 100644
index a7933bd39e..00
--- a/meta/recipes-devtools/go/go-1.9/default-buildmode-pie.patch
+++ /dev/null
@@ -1,18 +0,0 @@
-Default to PIE on linux platforms
-
-Upstream-Status: Pending
-Signed-off-by: Khem Raj 
-
-diff -upr src/go.orig/src/cmd/go/internal/work/build.go 
src/go/src/cmd/go/internal/work/build.go
 go.orig/src/cmd/go/internal/work/build.go  2017-08-27 17:38:26.354750979 
+0200
-+++ go/src/cmd/go/internal/work/build.go   2017-08-27 17:40:27.555130105 
+0200
-@@ -304,7 +304,8 @@ func BuildModeInit() {
-   ldBuildmode = "c-shared"
-   case "default":
-   switch platform {
--  case "android/arm", "android/arm64", "android/amd64", 
"android/386":
-+  case "linux/386", "linux/amd64", "linux/arm", "linux/arm64", 
"linux/ppc64le", "linux/s390x",
-+  "android/arm", "android/arm64", "android/amd64", 
"android/386":
-   codegenArg = "-shared"
-   ldBuildmode = "pie"
-   case "darwin/arm", "darwin/arm64":
diff --git a/meta/recipes-devtools/go/go-1.9/set-external-linker.patch 
b/meta/recipes-devtools/go/go-1.9/set-external-linker.patch
deleted file mode 100644
index d6bd7fa39c..00
--- a/meta/recipes-devtools/go/go-1.9/set-external-linker.patch
+++ /dev/null
@@ -1,111 +0,0 @@
-Change the dynamic linker hardcoding to use musl when not using glibc
-this should be applied conditional to musl being the system C library
-
-Upstream-Status: Inappropriate [Real Fix should be portable across libcs]
-
-Signed-off-by: Khem Raj 
-
-Index: go/src/cmd/link/internal/amd64/obj.go
-===
 go.orig/src/cmd/link/internal/amd64/obj.go
-+++ go/src/cmd/link/internal/amd64/obj.go
-@@ -67,7 +67,7 @@ func Init() {
-   ld.Thearch.Append64 = ld.Append64l
-   ld.Thearch.TLSIEtoLE = tlsIEtoLE
- 
--  ld.Thearch.Linuxdynld = "/lib64/ld-linux-x86-64.so.2"
-+  ld.Thearch.Linuxdynld = "/lib/ld-musl-x86_64.so.1"
-   ld.Thearch.Freebsddynld = "/libexec/ld-elf.so.1"
-   ld.Thearch.Openbsddynld = "/usr/libexec/ld.so"
-   ld.Thearch.Netbsddynld = "/libexec/ld.elf_so"
-Index: go/src/cmd/link/internal/arm/obj.go
-===
 go.orig/src/cmd/link/internal/arm/obj.go
-+++ go/src/cmd/link/internal/arm/obj.go
-@@ -63,7 +63,7 @@ func Init() {
-   ld.Thearch.Append32 = ld.Append32l
-   ld.Thearch.Append64 = ld.Append64l
- 
--  ld.Thearch.Linuxdynld = "/lib/ld-linux.so.3" // 2 for OABI, 3 for EABI
-+  ld.Thearch.Linuxdynld = "/lib/ld-musl-armhf.so.1"
-   ld.Thearch.Freebsddynld = "/usr/libexec/ld-elf.so.1"
-   ld.Thearch.Openbsddynld = "/usr/libexec/ld.so"
-   ld.Thearch.Netbsddynld = "/libexec/ld.elf_so"
-Index: go/src/cmd/link/internal/arm64/obj.go
-===
 go.orig/src/cmd/link/internal/arm64/obj.go
-+++ go/src/cmd/link/internal/arm64/obj.go
-@@ -62,7 +62,7 @@ func Init() {
-   ld.Thearch.Append32 = ld.Append32l
-   ld.Thearch.Append64 = ld.Append64l
- 
--  ld.Thearch.Linuxdynld = "/lib/ld-linux-aarch64.so.1"
-+  ld.Thearch.Linuxdynld = "/lib/ld-musl-aarch64.so.1"
- 
-   ld.Thearch.Freebsddynld = "XXX"
-   ld.Thearch.Openbsddynld = "XXX"
-Index: go/src/cmd/link/internal/mips/obj.go
-==

[OE-core] [PATCH 0/1] qemu: use upstream swtpm support

[Patrick Ohly]

This aligns TPM support in OE with the approach accepted and merged
upstream.

An update for swtpm in meta-security was already sent
("[meta-security][PATCH 1/1] swtpm/libtpm: update to latest master").

Changes:
  -v2: rebased

Patrick Ohly (1):
  qemu: use upstream swtpm support

 meta/recipes-devtools/qemu/qemu/0001-Provide-support-for-the-CUSE-TPM.patch
 |  870 
+---
 
meta/recipes-devtools/qemu/qemu/0001-tpm-Clean-up-driver-registration-lookup.patch
  |  154 ++-
 
meta/recipes-devtools/qemu/qemu/0002-Introduce-condition-to-notify-waiters-of-completed-c.patch
 |   86 +--
 
meta/recipes-devtools/qemu/qemu/0002-tpm-Clean-up-model-registration-lookup.patch
   |  121 -
 
meta/recipes-devtools/qemu/qemu/0003-Introduce-condition-in-TPM-backend-for-notification.patch
  |   79 +-
 
meta/recipes-devtools/qemu/qemu/0003-tpm-backend-Remove-unneeded-member-variable-from-bac.patch
 |   75 +-
 
meta/recipes-devtools/qemu/qemu/0004-Add-support-for-VM-suspend-resume-for-TPM-TIS-v2.9.patch
   |  719 +-
 
meta/recipes-devtools/qemu/qemu/0004-tpm-backend-Move-thread-handling-inside-TPMBackend.patch
   |  417 -
 
meta/recipes-devtools/qemu/qemu/0005-tpm-backend-Initialize-and-free-data-members-in-it-s.patch
 |  185 +-
 
meta/recipes-devtools/qemu/qemu/0006-tpm-backend-Made-few-interface-methods-optional.patch
  |  284 +++-
 
meta/recipes-devtools/qemu/qemu/0007-tpm-backend-Add-new-api-to-read-backend-TpmInfo.patch
  |  293 -
 
meta/recipes-devtools/qemu/qemu/0008-tpm-backend-Move-realloc_buffer-implementation-to-tp.patch
 |  140 ++-
 
meta/recipes-devtools/qemu/qemu/0009-tpm-passthrough-move-reusable-code-to-utils.patch
  |  182 -
 meta/recipes-devtools/qemu/qemu/0010-tpm-Added-support-for-TPM-emulator.patch  
 | 1059 
-
 meta/recipes-devtools/qemu/qemu/0011-tpm-Move-tpm_cleanup-to-right-place.patch 
 |   43 +++-
 
meta/recipes-devtools/qemu/qemu/0012-tpm-Use-EMSGSIZE-instead-of-EBADMSG-to-compile-on-Op.patch
 |   67 +-
 
meta/recipes-devtools/qemu/qemu/chardev-connect-socket-to-a-spawned-command.patch
   |  227 +++-
 meta/recipes-devtools/qemu/qemu_2.10.1.bb  
 |   17 +-
 18 files changed, 3260 insertions(+), 1758 deletions(-)
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0001-Provide-support-for-the-CUSE-TPM.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0001-tpm-Clean-up-driver-registration-lookup.patch
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0002-Introduce-condition-to-notify-waiters-of-completed-c.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0002-tpm-Clean-up-model-registration-lookup.patch
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0003-Introduce-condition-in-TPM-backend-for-notification.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0003-tpm-backend-Remove-unneeded-member-variable-from-bac.patch
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0004-Add-support-for-VM-suspend-resume-for-TPM-TIS-v2.9.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0004-tpm-backend-Move-thread-handling-inside-TPMBackend.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0005-tpm-backend-Initialize-and-free-data-members-in-it-s.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0006-tpm-backend-Made-few-interface-methods-optional.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0007-tpm-backend-Add-new-api-to-read-backend-TpmInfo.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0008-tpm-backend-Move-realloc_buffer-implementation-to-tp.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0009-tpm-passthrough-move-reusable-code-to-utils.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0010-tpm-Added-support-for-TPM-emulator.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0011-tpm-Move-tpm_cleanup-to-right-place.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0012-tpm-Use-EMSGSIZE-instead-of-EBADMSG-to-compile-on-Op.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/chardev-connect-socket-to-a-spawned-command.patch

base-commit: a7cd9d1183be603777fc9c8c448281fe01224f7b
-- 
git-series 0.9.1
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] sudo: provide sudo paths through EXTRA_OECONF

[Nikolay Merinov]

Configure script for a sudo utility trying to figure out "vardir",
"iologdir", "rundir" and "logpath" from directory structure on build
machine. If none of a standart paths present on a build machine then
variables will be set to empty string and sudo build will fail with
"Installed but not shipped" error.

Signed-off-by: Nikolay Merinov 
---
 meta/recipes-extended/sudo/sudo_1.8.21p2.bb | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-extended/sudo/sudo_1.8.21p2.bb 
b/meta/recipes-extended/sudo/sudo_1.8.21p2.bb
index acd0025abc..7a50f1e522 100644
--- a/meta/recipes-extended/sudo/sudo_1.8.21p2.bb
+++ b/meta/recipes-extended/sudo/sudo_1.8.21p2.bb
@@ -17,6 +17,10 @@ EXTRA_OECONF += " \
  ac_cv_type_rsize_t=no \
  ${@bb.utils.contains('DISTRO_FEATURES', 'pam', '--with-pam', 
'--without-pam', d)} \
  ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 
'--enable-tmpfiles.d=${libdir}/tmpfiles.d', '--disable-tmpfiles.d', d)} \
+--with-vardir=/var/lib/sudo \
+--with-iologdir=/var/log/sudo-io \
+--with-rundir=/var/run/sudo \
+--with-logpath=/var/log/sudo.log \
  "
 
 do_install_append () {
-- 
2.14.1

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

[OE-core] ✗ patchtest: failure for qemu: use upstream swtpm support

[Patchwork]

== Series Details ==

Series: qemu: use upstream swtpm support
Revision: 1
URL   : https://patchwork.openembedded.org/series/10092/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue Series does not apply on top of target branch 
[test_series_merge_on_head] 
  Suggested fixRebase your series on top of targeted branch
  Targeted branch  master (currently at a7cd9d1183)



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

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

Re: [OE-core] [PATCH 1/1 V2] targetcontrol.py: use oe.types.boolean for QEMU_USE_KVM

[Robert Yang]

Hi Ross,

On 12/06/2017 11:11 AM, Robert Yang wrote:



On 12/06/2017 09:37 AM, Robert Yang wrote:


On 12/06/2017 12:10 AM, Burton, Ross wrote:
On 1 December 2017 at 05:13, Robert Yang > wrote:


    So that both QEMU_USE_KVM = "True" and "1" will work.

    [YOCTO #12343]

    Signed-off-by: Robert Yang mailto:liezhi.y...@windriver.com>>
    ---
  meta/classes/testimage.bbclass | 2 +-
  meta/lib/oeqa/targetcontrol.py | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

    diff --git a/meta/classes/testimage.bbclass b/meta/classes/testimage.bbclass
    index 45bb2bd..7260ad4 100644
    --- a/meta/classes/testimage.bbclass
    +++ b/meta/classes/testimage.bbclass
    @@ -215,7 +215,7 @@ def testimage_main(d):
      # Get use_kvm
      qemu_use_kvm = d.getVar("QEMU_USE_KVM")
      if qemu_use_kvm and \
    -       (qemu_use_kvm == 'True' and 'x86' in machine or \
    +       (oe.types.boolean(qemu_use_kvm) and 'x86' in machine or \
          d.getVar('MACHINE') in qemu_use_kvm.split()):
          kvm = True
      else:
    diff --git a/meta/lib/oeqa/targetcontrol.py b/meta/lib/oeqa/targetcontrol.py
    index f63936c..3fc3870 100644
    --- a/meta/lib/oeqa/targetcontrol.py
    +++ b/meta/lib/oeqa/targetcontrol.py
    @@ -107,7 +107,7 @@ class QemuTarget(BaseTarget):
          dump_dir = d.getVar("TESTIMAGE_DUMP_DIR")
          qemu_use_kvm = d.getVar("QEMU_USE_KVM")
          if qemu_use_kvm and \
    -           (qemu_use_kvm == "True" and "x86" in d.getVar("MACHINE") or \
    +           (oe.types.boolean(qemu_use_kvm) and "x86" in d.getVar("MACHINE")
    or \
              d.getVar("MACHINE") in qemu_use_kvm.split()):
              use_kvm = True
          else:


https://autobuilder.yocto.io/builders/nightly-oe-selftest/builds/665/steps/Running%20oe-selftest/logs/stdio 



   File 
"/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-oe-selftest/build/meta/lib/oeqa/targetcontrol.py", 
line 110, in __init__

     (oe.types.boolean(qemu_use_kvm) and "x86" in d.getVar("MACHINE") or \
UnboundLocalError: local variable 'oe' referenced before assignment


Sorry, I had tried testimage and oe-selftest before and just now locally,
didn't see this error, need more investigations.


I can reproduce it now:

Add QEMU_USE_KVM = "1" to con/local.conf
$ oe-selftest  -r devtool.DevtoolTests.test_devtool_deploy_target


I fixed it in the repo:


  git://git.openembedded.org/openembedded-core-contrib rbt/use_qemu
  http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/use_qemu

Robert Yang (1):
  targetcontrol.py: use oe.types.boolean for QEMU_USE_KVM


diff --git a/meta/classes/testimage.bbclass b/meta/classes/testimage.bbclass
index 45bb2bd..7260ad4 100644
--- a/meta/classes/testimage.bbclass
+++ b/meta/classes/testimage.bbclass
@@ -215,7 +215,7 @@ def testimage_main(d):
 # Get use_kvm
 qemu_use_kvm = d.getVar("QEMU_USE_KVM")
 if qemu_use_kvm and \
-   (qemu_use_kvm == 'True' and 'x86' in machine or \
+   (oe.types.boolean(qemu_use_kvm) and 'x86' in machine or \
 d.getVar('MACHINE') in qemu_use_kvm.split()):
 kvm = True
 else:
diff --git a/meta/lib/oeqa/targetcontrol.py b/meta/lib/oeqa/targetcontrol.py
index f63936c..59a9c35 100644
--- a/meta/lib/oeqa/targetcontrol.py
+++ b/meta/lib/oeqa/targetcontrol.py
@@ -91,6 +91,8 @@ class QemuTarget(BaseTarget):

 def __init__(self, d, logger, image_fstype=None):

+import oe.types
+
 super(QemuTarget, self).__init__(d, logger)

 self.rootfs = ''
@@ -107,7 +109,7 @@ class QemuTarget(BaseTarget):
 dump_dir = d.getVar("TESTIMAGE_DUMP_DIR")
 qemu_use_kvm = d.getVar("QEMU_USE_KVM")
 if qemu_use_kvm and \
-   (qemu_use_kvm == "True" and "x86" in d.getVar("MACHINE") or \
+   (oe.types.boolean(qemu_use_kvm) and "x86" in d.getVar("MACHINE") or 
\
 d.getVar("MACHINE") in qemu_use_kvm.split()):
 use_kvm = True
 else:

// Robert



// Robert



// Robert



Ross

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

[OE-core] [PATCH] iptables: Split the iptables modules into separate packages

[Peter Kjellerstedt]

By splitting the iptables modules into separate packages it is
possible to pick and choose the modules to install and thereby reduce
the total size of the installed modules.

Backwards compatibility is maintained by adding a recommendation of
iptables-modules, which is a meta package that depends on all the
generated packages.

Signed-off-by: Peter Kjellerstedt 
---

In our case we could save more than 1 MB by removing modules we do not
use.

 meta/recipes-extended/iptables/iptables_1.6.1.bb | 43 +---
 1 file changed, 30 insertions(+), 13 deletions(-)

diff --git a/meta/recipes-extended/iptables/iptables_1.6.1.bb 
b/meta/recipes-extended/iptables/iptables_1.6.1.bb
index b37c55a64e..b6e6cbe8a3 100644
--- a/meta/recipes-extended/iptables/iptables_1.6.1.bb
+++ b/meta/recipes-extended/iptables/iptables_1.6.1.bb
@@ -7,17 +7,6 @@ LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263\
 
file://iptables/iptables.c;beginline=13;endline=25;md5=c5cffd09974558cf27d0f763df2a12dc"
 
-RRECOMMENDS_${PN} = "kernel-module-x-tables \
- kernel-module-ip-tables \
- kernel-module-iptable-filter \
- kernel-module-iptable-nat \
- kernel-module-nf-defrag-ipv4 \
- kernel-module-nf-conntrack \
- kernel-module-nf-conntrack-ipv4 \
- kernel-module-nf-nat \
- kernel-module-ipt-masquerade"
-FILES_${PN} =+ "${libdir}/xtables/ ${datadir}/xtables"
-
 SRC_URI = "http://netfilter.org/projects/iptables/files/iptables-${PV}.tar.bz2 
\
file://types.h-add-defines-that-are-required-for-if_packet.patch \

file://0001-configure-Add-option-to-enable-disable-libnfnetlink.patch \
@@ -30,8 +19,8 @@ SRC_URI[sha256sum] = 
"0fc2d7bd5d7be11311726466789d4c65fb4c8e096c9182b56ce9744086
 
 inherit autotools pkgconfig
 
-EXTRA_OECONF = "--with-kernel=${STAGING_INCDIR} \
-   "
+EXTRA_OECONF = "--with-kernel=${STAGING_INCDIR}"
+
 PACKAGECONFIG ?= "${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)}"
 
 PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6,"
@@ -47,3 +36,31 @@ do_configure_prepend() {
# Keep ax_check_linker_flags.m4 which belongs to autoconf-archive.
rm -f libtool.m4 lt~obsolete.m4 ltoptions.m4 ltsugar.m4 ltversion.m4
 }
+
+PACKAGES += "${PN}-modules"
+PACKAGES_DYNAMIC += "^${PN}-module-.*"
+
+python populate_packages_prepend() {
+modules = do_split_packages(d, '${libdir}/xtables', 'lib(.*)\.so$', 
'${PN}-module-%s', '${PN} module %s', extra_depends='')
+if modules:
+metapkg = d.getVar('PN') + '-modules'
+d.appendVar('RDEPENDS_' + metapkg, ' ' + ' '.join(modules))
+}
+
+FILES_${PN} += "${datadir}/xtables"
+
+ALLOW_EMPTY_${PN}-modules = "1"
+
+RDEPENDS_${PN} = "iptables-module-xt-standard"
+RRECOMMENDS_${PN} = " \
+${PN}-modules \
+kernel-module-x-tables \
+kernel-module-ip-tables \
+kernel-module-iptable-filter \
+kernel-module-iptable-nat \
+kernel-module-nf-defrag-ipv4 \
+kernel-module-nf-conntrack \
+kernel-module-nf-conntrack-ipv4 \
+kernel-module-nf-nat \
+kernel-module-ipt-masquerade \
+"
-- 
2.12.0

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

[OE-core] [RFC PATCH] autoconf: Override config.guess/.sub in source tree

[Nathan Rossi]

autoconf has not been released in a number of years. However many
changes have been made to the gnu-config config.guess/.sub since 2.69,
including new architectures, OS variants, etc. In order to enable these
targets without creating patches for the source itself populate the
gnu-config files from the sysroot as is done with autotools recipes.
Whilst it is not possible for the autoconf recipe to bootstrap its
configure task (using the autotools_do_configure), the files can be
manually copied into the target location.

Signed-off-by: Nathan Rossi 
---
 meta/recipes-devtools/autoconf/autoconf.inc | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-devtools/autoconf/autoconf.inc 
b/meta/recipes-devtools/autoconf/autoconf.inc
index ea62df8fb6..df81bc6107 100644
--- a/meta/recipes-devtools/autoconf/autoconf.inc
+++ b/meta/recipes-devtools/autoconf/autoconf.inc
@@ -41,6 +41,10 @@ PERL_class-nativesdk = "/usr/bin/env perl"
 CACHED_CONFIGUREVARS += "ac_cv_path_PERL='${PERL}'"
 
 do_configure() {
+   # manually install a newer config.guess/.sub
+   install -m 0755 ${STAGING_DATADIR_NATIVE}/gnu-config/config.guess 
${S}/build-aux
+   install -m 0755 ${STAGING_DATADIR_NATIVE}/gnu-config/config.sub 
${S}/build-aux
+
oe_runconf
 }
 
-- 
2.15.0


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

[OE-core] [PATCH] packagegroup-core-tools-testapps: kexec does not support MicroBlaze

[Nathan Rossi]

The kexec-tools do not currently support MicroBlaze, override the
KEXECTOOLS variable to disable the inclusion of these tools in the
packagegroup.

Signed-off-by: Nathan Rossi 
---
 meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb 
b/meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb
index 317097854f..0d2da86613 100644
--- a/meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb
@@ -10,9 +10,10 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
 
 inherit packagegroup
 
-# kexec-tools doesn't work on e5500-64b and nios2 yet
+# kexec-tools doesn't work on e5500-64b, microblaze and nios2 yet
 KEXECTOOLS ?= "kexec"
 KEXECTOOLS_e5500-64b ?= ""
+KEXECTOOLS_microblaze ?= ""
 KEXECTOOLS_nios2 ?= ""
 
 X11GLTOOLS = "\
-- 
2.15.0


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

[OE-core] [PATCH] packagegroup-core-sdk: Disable SANITIZERS for MicroBlaze

[Nathan Rossi]

The libasan and libubsan sanitizers are not available for MicroBlaze.
Follow the overriding of the SANITIZERS variable as done for other
architectures to remove these from the packagegroup when targeting
MicroBlaze.

Signed-off-by: Nathan Rossi 
---
 meta/recipes-core/packagegroups/packagegroup-core-sdk.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb 
b/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
index af0ce2013e..393f0d3d13 100644
--- a/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
@@ -28,6 +28,7 @@ RDEPENDS_packagegroup-core-sdk = "\
 
 SANITIZERS = "libasan-dev libubsan-dev"
 SANITIZERS_aarch64 = ""
+SANITIZERS_microblaze = ""
 SANITIZERS_mipsarch = ""
 SANITIZERS_nios2 = ""
 SANITIZERS_powerpc64 = ""
-- 
2.15.0


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

[OE-core] [PATCH] prelink_git.bb: Update recipe to newest commit

[Nathan Rossi]

Update prelink to the newest commit on the cross_prelink ref. This
includes the following changes:

  05aeafd053 Disable automatic generation of prelink.pdf
  aa2985eefa src/rtld: Add MicroBlaze support based on glibc-2.24
  62f80843f8 README: update information on reloc8/reloc9 failures

The primary purpose of this update is to enable the added MicroBlaze
rtld support so as to enable gobject-introspection for MicroBlaze.

Signed-off-by: Nathan Rossi 
---
 meta/recipes-devtools/prelink/prelink_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/prelink/prelink_git.bb 
b/meta/recipes-devtools/prelink/prelink_git.bb
index cc93bcc384..0f6d16e005 100644
--- a/meta/recipes-devtools/prelink/prelink_git.bb
+++ b/meta/recipes-devtools/prelink/prelink_git.bb
@@ -9,7 +9,7 @@ and executables, so that far fewer relocations need to be 
resolved at \
 runtime and thus programs come up faster."
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=c93c0550bd3173f4504b2cbd8991e50b"
-SRCREV = "ef20628dd78b92e1a3123afc67b64cf010bdd9e4"
+SRCREV = "05aeafd053e56356ec8c62f4bb8f7b95bae192f3"
 PV = "1.0+git${SRCPV}"
 
 #
-- 
2.15.0


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

[OE-core] [PATCH 0/1] qemu: use upstream swtpm support

[Patrick Ohly]

This aligns TPM support in OE with the approach accepted and merged
upstream.

An update for swtpm in meta-security was already sent
("[meta-security][PATCH 1/1] swtpm/libtpm: update to latest master").

Patrick Ohly (1):
  qemu: use upstream swtpm support

 meta/recipes-devtools/qemu/qemu/0001-Provide-support-for-the-CUSE-TPM.patch
 |  870 
+---
 
meta/recipes-devtools/qemu/qemu/0001-tpm-Clean-up-driver-registration-lookup.patch
  |  154 ++-
 
meta/recipes-devtools/qemu/qemu/0002-Introduce-condition-to-notify-waiters-of-completed-c.patch
 |   86 +--
 
meta/recipes-devtools/qemu/qemu/0002-tpm-Clean-up-model-registration-lookup.patch
   |  121 -
 
meta/recipes-devtools/qemu/qemu/0003-Introduce-condition-in-TPM-backend-for-notification.patch
  |   79 +-
 
meta/recipes-devtools/qemu/qemu/0003-tpm-backend-Remove-unneeded-member-variable-from-bac.patch
 |   75 +-
 
meta/recipes-devtools/qemu/qemu/0004-Add-support-for-VM-suspend-resume-for-TPM-TIS-v2.9.patch
   |  719 +-
 
meta/recipes-devtools/qemu/qemu/0004-tpm-backend-Move-thread-handling-inside-TPMBackend.patch
   |  417 -
 
meta/recipes-devtools/qemu/qemu/0005-tpm-backend-Initialize-and-free-data-members-in-it-s.patch
 |  185 +-
 
meta/recipes-devtools/qemu/qemu/0006-tpm-backend-Made-few-interface-methods-optional.patch
  |  284 +++-
 
meta/recipes-devtools/qemu/qemu/0007-tpm-backend-Add-new-api-to-read-backend-TpmInfo.patch
  |  293 -
 
meta/recipes-devtools/qemu/qemu/0008-tpm-backend-Move-realloc_buffer-implementation-to-tp.patch
 |  140 ++-
 
meta/recipes-devtools/qemu/qemu/0009-tpm-passthrough-move-reusable-code-to-utils.patch
  |  182 -
 meta/recipes-devtools/qemu/qemu/0010-tpm-Added-support-for-TPM-emulator.patch  
 | 1059 
-
 meta/recipes-devtools/qemu/qemu/0011-tpm-Move-tpm_cleanup-to-right-place.patch 
 |   43 +++-
 
meta/recipes-devtools/qemu/qemu/0012-tpm-Use-EMSGSIZE-instead-of-EBADMSG-to-compile-on-Op.patch
 |   67 +-
 
meta/recipes-devtools/qemu/qemu/chardev-connect-socket-to-a-spawned-command.patch
   |  227 +++-
 meta/recipes-devtools/qemu/qemu_2.10.0.bb  
 |   17 +-
 18 files changed, 3260 insertions(+), 1758 deletions(-)
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0001-Provide-support-for-the-CUSE-TPM.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0001-tpm-Clean-up-driver-registration-lookup.patch
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0002-Introduce-condition-to-notify-waiters-of-completed-c.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0002-tpm-Clean-up-model-registration-lookup.patch
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0003-Introduce-condition-in-TPM-backend-for-notification.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0003-tpm-backend-Remove-unneeded-member-variable-from-bac.patch
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0004-Add-support-for-VM-suspend-resume-for-TPM-TIS-v2.9.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0004-tpm-backend-Move-thread-handling-inside-TPMBackend.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0005-tpm-backend-Initialize-and-free-data-members-in-it-s.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0006-tpm-backend-Made-few-interface-methods-optional.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0007-tpm-backend-Add-new-api-to-read-backend-TpmInfo.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0008-tpm-backend-Move-realloc_buffer-implementation-to-tp.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0009-tpm-passthrough-move-reusable-code-to-utils.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0010-tpm-Added-support-for-TPM-emulator.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0011-tpm-Move-tpm_cleanup-to-right-place.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/0012-tpm-Use-EMSGSIZE-instead-of-EBADMSG-to-compile-on-Op.patch
 create mode 100644 
meta/recipes-devtools/qemu/qemu/chardev-connect-socket-to-a-spawned-command.patch

base-commit: 3b413a80578caacd9a7f405f3c51a3921d78a60d
-- 
git-series 0.9.1
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] How to add group for multiple recipes

[Robert Yang]

Hi Ben,

On 12/06/2017 04:12 PM, GUAN Ben (ST-FIR/ENG1-Zhu) wrote:

Hello,

In oe-core, there are some default groups provided by recipe "base-passwd". Any 
other recipes could refer to these groups when they want to create a new user.

Now we have a requirement: there are several recipes which create their own 
users, and they shall belongs to a common group. How could we add this group? 
It seems that groups or users created in a recipe are only visible to the same 
recipe. Other recipes cannot refer to them (e.g. using chown/chgrp).

We already tried to add a recipe which only generates the group and added this 
to DEPENDS of the other recipes. But it seem as if the group was not generated 
in time. When the other recipes ran, the group was not there. But the final 
image contained the group.


What's the error, please ? Where did you check the group ? If you have added the
the recipe to another's DEPENDS, then the group should be in another recipe's
recipe-sysroot.

// Robert



Best regards

Ben GUAN
ST-FIR/ENG1-Zhu


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

[OE-core] How to add group for multiple recipes

[GUAN Ben (ST-FIR/ENG1-Zhu)]

Hello,

In oe-core, there are some default groups provided by recipe "base-passwd". Any 
other recipes could refer to these groups when they want to create a new user. 

Now we have a requirement: there are several recipes which create their own 
users, and they shall belongs to a common group. How could we add this group? 
It seems that groups or users created in a recipe are only visible to the same 
recipe. Other recipes cannot refer to them (e.g. using chown/chgrp).

We already tried to add a recipe which only generates the group and added this 
to DEPENDS of the other recipes. But it seem as if the group was not generated 
in time. When the other recipes ran, the group was not there. But the final 
image contained the group. 

Best regards

Ben GUAN
ST-FIR/ENG1-Zhu  

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