[OE-core] [PATCH] avahi: avoid depending on skipped package

2018-11-28 Thread Jens Rehsack
When built without D-Bus, libavahi-client is not build: Building libavahi-client: no (You need avahi-daemon and D-Bus!) which causes avahi-dev RDEPENDS failing when creating an image containing development-tools: * - nothing provides libavahi-client = 0.7-r0 needed by

Re: [OE-core] [PATCH 1/1] initrdscripts/init-live.sh: fix mounts w/ spaces fail to move to real rootfs

2018-11-28 Thread Awan, Arsalan
Hi, This was fixed in Thud (& master), but can we backport this 6f8f984ba363f764e83290b972ec31a90aad1603 to [sumo ... morty]? Link: http://git.openembedded.org/openembedded-core/commit/meta/recipes-core/initrdscripts/files/init-live.sh?h=thud=6f8f984ba363f764e83290b972ec31a90aad1603 Thanks -

Re: [OE-core] [PATCH 1/1] netbase: add entry to /etc/hosts according to /etc/hostname

2018-11-28 Thread Richard Purdie
On Wed, 2018-11-28 at 10:26 +, Burton, Ross wrote: > What if the hostname entry already exists? $sysconfdir instead of > hard-coding etc. > > I can't help but think a better solution would be to simply move > /etc/hosts to base-files, so it can be updated at the same time as > hostname. I'm

[OE-core] [PATCH] arch-armv7*.inc: Add Cortex vfpv4-d16 support

2018-11-28 Thread Phil Edworthy
Some ARM Cortex devices have the VFPv4-D16, but no NEON. Signed-off-by: Phil Edworthy --- The additional lines for vpfv4d16 are basically copies of the vpfv3d16 lines. Tested with thud on Renesas RZ/N1 (Cortex A7 with VFPv4-D16, no NEON), also applies to sumo. ---

Re: [OE-core] [PATCH] licence: Add license file CC-BY-SA-4.0

2018-11-28 Thread Burton, Ross
Your mail client has mangled the patch, can you resend this using git-send-email? Applying: licence: Add license file CC-BY-SA-4.0 error: git apply: bad git-diff - expected /dev/null on line 23 error: could not build fake ancestor Patch failed at 0001 licence: Add license file CC-BY-SA-4.0 Ross

Re: [OE-core] [PATCH 5/8] acl: Use install instead of cp

2018-11-28 Thread Peter Kjellerstedt
> -Original Message- > From: openembedded-core-boun...@lists.openembedded.org core-boun...@lists.openembedded.org> On Behalf Of Douglas Royds > Sent: den 28 november 2018 05:53 > To: openembedded-core@lists.openembedded.org > Subject: [OE-core] [PATCH 5/8] acl: Use install instead of cp >

Re: [OE-core] [PATCH] u-boot-tools: Improve build preparation

2018-11-28 Thread Otavio Salvador
On Tue, Nov 27, 2018 at 6:49 PM Alexey Brodkin wrote: > On Tue, 2018-11-27 at 17:52 -0200, Otavio Salvador wrote: > > On Tue, Nov 27, 2018 at 5:30 PM Otavio Salvador > > wrote: > > > On Tue, Nov 27, 2018 at 5:13 PM Alexey Brodkin > > > wrote: > > > > Even though we're not going to build here

Re: [OE-core] [PATCH] u-boot-tools: Improve build preparation

2018-11-28 Thread Otavio Salvador
On Wed, Nov 28, 2018 at 8:58 AM Otavio Salvador wrote: > On Tue, Nov 27, 2018 at 6:49 PM Alexey Brodkin > wrote: > > On Tue, 2018-11-27 at 17:52 -0200, Otavio Salvador wrote: > > > On Tue, Nov 27, 2018 at 5:30 PM Otavio Salvador > > > wrote: > > > > On Tue, Nov 27, 2018 at 5:13 PM Alexey

Re: [OE-core] [PATCH] avahi: avoid depending on skipped package

2018-11-28 Thread Burton, Ross
Is Avahi actually useful without DBus? This reminds me that I've been wanting to gut that recipe for a long time now... Ross On Wed, 28 Nov 2018 at 08:16, Jens Rehsack wrote: > > When built without D-Bus, libavahi-client is not build: > Building libavahi-client: no (You need

Re: [OE-core] [PATCH 1/1] netbase: add entry to /etc/hosts according to /etc/hostname

2018-11-28 Thread Burton, Ross
What if the hostname entry already exists? $sysconfdir instead of hard-coding etc. I can't help but think a better solution would be to simply move /etc/hosts to base-files, so it can be updated at the same time as hostname. Ross On Mon, 26 Nov 2018 at 06:51, Chen Qi wrote: > > We default

[OE-core] [PATCH 4/4] oeqa/qemurunner: Remove resource python warnings

2018-11-28 Thread Richard Purdie
If runqemu fails it would leak an unclosed socket and file. Ensure we close these in all cases to remove the resource warning. Signed-off-by: Richard Purdie --- meta/lib/oeqa/utils/qemurunner.py | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git

[OE-core] [PATCH 2/4] oeqa/utils/commands: Avoid log message duplication

2018-11-28 Thread Richard Purdie
Each time a runqemu() fails, the log handler would be left behind meaning messages from any subsequent run would be duplicated (or worse/more). This ensures we remove the handler regardless and means we no longer have the duplication. Signed-off-by: Richard Purdie ---

[OE-core] [PATCH 1/4] oeqa/utils/qemurunner: Fix python ResourceWarning for unclosed file

2018-11-28 Thread Richard Purdie
Fixes: Stderr: /media/build1/poky/meta/lib/oeqa/utils/qemurunner.py:381: ResourceWarning: unclosed file <_io.BufferedWriter name=16> self.runqemu = None Signed-off-by: Richard Purdie --- meta/lib/oeqa/utils/qemurunner.py | 2 ++ 1 file changed, 2 insertions(+) diff --git

[OE-core] [PATCH 3/4] scripts/runqemu: Fix logic error causing failures with MACHINE from the environment

2018-11-28 Thread Richard Purdie
MACHINE=qemux86-64 oe-selftest -r runqemu.RunqemuTests.test_boot_machine_slirp_qcow2 fails yet oe-selftest -r runqemu.RunqemuTests.test_boot_machine_slirp_qcow2 with MACHINE in local.conf would work. It turns out that: runqemu slirp wic.qcow2 qemux86-64 works but: MACHINE=qemux86-64

Re: [OE-core] [PATCH] licence: Add license file CC-BY-SA-4.0

2018-11-28 Thread Eric Chanudet
On 28/11/18 at 10:45am, Burton, Ross wrote: >Your mail client has mangled the patch, can you resend this using >git-send-email? I used git-send-email for this, following the wiki[1]'s instructions. I am confused. I tried applying the change from the ml post both with `git am` and `patch` and

[OE-core] [PATCH 2/2] scripts/runqemu: Improve lockfile handling for python with close_fd=True

2018-11-28 Thread Richard Purdie
On python versions with close_fds=True (python 3.2 onwards), the tap device lockfile isn't passed to the child process. Since this guards against use of an active interface, we really want this here, so pass it in pass_fds. This means if the parent exits early, the child still holds the lock,

[OE-core] [PATCH 1/2] scripts/runqemu: Tidy up lock handling code

2018-11-28 Thread Richard Purdie
Various tweaks: - Balance up the aquire/release functions - Use debug messge for both acquiring and release message for consistency in logs - Use None instead of an empty string - Reset the value of the field if we don't have the lock any more Signed-off-by: Richard Purdie --- scripts/runqemu

Re: [OE-core] [PATCH 5/8] acl: Use install instead of cp

2018-11-28 Thread Douglas Royds
On 28/11/18 9:04 PM, Peter Kjellerstedt wrote: -Original Message- From: openembedded-core-boun...@lists.openembedded.org On Behalf Of Douglas Royds Sent: den 28 november 2018 05:53 To: openembedded-core@lists.openembedded.org Subject: [OE-core] [PATCH 5/8] acl: Use install instead of

Re: [OE-core] [PATCH 0/4] recipeutils fixes for devtool

2018-11-28 Thread Paul Eggleton
On Wednesday, 28 November 2018 5:16:13 PM NZDT Paul Eggleton wrote: > A couple of fixes for issues in the recipeutils module (which affect > devtool), a cleanup patch, and some additional related tests for > oe-selftest. Oops, patch 4/4 missed out the actual meta-selftest recipes that a few of

[OE-core] ✗ patchtest: failure for ptest: Strip build host paths from any installed Makefile (rev4)

2018-11-28 Thread Patchwork
== Series Details == Series: ptest: Strip build host paths from any installed Makefile (rev4) Revision: 4 URL : https://patchwork.openembedded.org/series/15114/ State : failure == Summary == Thank you for submitting this patch series to OpenEmbedded Core. This is an automated response.

[OE-core] [PATCH v2 4/4] oe-selftest: add some tests for recipeutils module

2018-11-28 Thread Paul Eggleton
Add some tests for functions in meta/lib/oe/recipeutils.py, in particular for a few issues I've just fixed. I haven't added tests for all of the functions - some of them are already being tested via devtool in any case. Signed-off-by: Paul Eggleton --- .../python/python-async-test.inc

[OE-core] [PATCH v2 0/3] ptest: Strip build host paths from any installed Makefile

2018-11-28 Thread Douglas Royds
Only one change from v1: Simplified commit acl: Use install instead of cp Douglas Royds (3): acl: Use install instead of cp acl: Strip build host tool details from installed ptest file attr: Strip build host paths and tool versions from ptest files meta/recipes-support/attr/acl_2.2.52.bb

[OE-core] [PATCH v2 1/3] acl: Use install instead of cp

2018-11-28 Thread Douglas Royds
Signed-off-by: Douglas Royds --- meta/recipes-support/attr/acl_2.2.52.bb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/meta/recipes-support/attr/acl_2.2.52.bb b/meta/recipes-support/attr/acl_2.2.52.bb index 8b89de9b42..318308a009 100644 ---

[OE-core] [PATCH v2 2/3] acl: Strip build host tool details from installed ptest file

2018-11-28 Thread Douglas Royds
Adopt the PTEST_BUILD_HOST_FILES mechanism to strip build host paths from the installed builddefs file. Also strip the record of the build host rpm version. Signed-off-by: Douglas Royds --- meta/recipes-support/attr/acl_2.2.52.bb | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-)

[OE-core] [PATCH v2 3/3] attr: Strip build host paths and tool versions from ptest files

2018-11-28 Thread Douglas Royds
Using the new PTEST_BUILD_HOST_FILES mechanism. Specifically remove reference to build host rpm version, if any. Signed-off-by: Douglas Royds --- meta/recipes-support/attr/attr.inc | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/meta/recipes-support/attr/attr.inc

[OE-core] Is python2 native broken in current eSDK?

2018-11-28 Thread Adrian Freihofer
Having baked an eSDK. Now I'm facing a problem with devtool and python2 native. The error occurs at least on the thud and on the master branch. Also the host distro seams not relevant. The same happens on Ubuntu 16.04, Ubuntu 18.04 and on Fedora 29 hosts. For example modifying and building

[OE-core] [PATCH v3 6/8] acl: Use install instead of cp

2018-11-28 Thread Douglas Royds
Signed-off-by: Douglas Royds --- meta/recipes-support/attr/acl_2.2.52.bb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/meta/recipes-support/attr/acl_2.2.52.bb b/meta/recipes-support/attr/acl_2.2.52.bb index 8b89de9b42..318308a009 100644 ---

[OE-core] [PATCH v3 5/8] strace: Strip build host tool details from installed ptest Makefile

2018-11-28 Thread Douglas Royds
Adopt new PTEST_BUILD_HOST_PATTERN mechanism. Signed-off-by: Douglas Royds Signed-off-by: Richard Purdie --- meta/recipes-devtools/strace/strace_4.25.bb | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/meta/recipes-devtools/strace/strace_4.25.bb

[OE-core] [PATCH v3 8/8] attr: Strip build host paths and tool versions from ptest files

2018-11-28 Thread Douglas Royds
Using the new PTEST_BUILD_HOST_FILES mechanism. Specifically remove reference to build host rpm version, if any. Signed-off-by: Douglas Royds --- meta/recipes-support/attr/attr.inc | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/meta/recipes-support/attr/attr.inc

[OE-core] [PATCH v3 2/8] ptest: Optionally strip build host tool info from installed files

2018-11-28 Thread Douglas Royds
Several packages (eg. acl, attr, openssl) install Makefiles or other build host configuration files for ptest so that they can run `make check`, for instance. These build host files commonly include paths and versions of build host tools, (eg. rpm or perl) whose presence and version depends on the

[OE-core] [PATCH v3 0/8] ptest: Strip build host paths from any installed Makefile

2018-11-28 Thread Douglas Royds
Only one change from v1: Simplified commit acl: Use install instead of cp Douglas Royds (8): ptest: Reproducibility: Strip build host paths from any installed Makefile ptest: Optionally strip build host tool info from installed files openssl ptest: Strip build host paths from configdata.pm

[OE-core] [PATCH v3 7/8] acl: Strip build host tool details from installed ptest file

2018-11-28 Thread Douglas Royds
Adopt the PTEST_BUILD_HOST_FILES mechanism to strip build host paths from the installed builddefs file. Also strip the record of the build host rpm version. Signed-off-by: Douglas Royds --- meta/recipes-support/attr/acl_2.2.52.bb | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-)

[OE-core] [PATCH v3 3/8] openssl ptest: Strip build host paths from configdata.pm

2018-11-28 Thread Douglas Royds
This file contains CC, CPP, CFLAGS, CXXFLAGS and the like. Signed-off-by: Douglas Royds Signed-off-by: Richard Purdie --- meta/recipes-connectivity/openssl/openssl_1.1.1a.bb | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb

[OE-core] ✗ patchtest: failure for ptest: Strip build host paths from any installed Makefile (rev3)

2018-11-28 Thread Patchwork
== Series Details == Series: ptest: Strip build host paths from any installed Makefile (rev3) Revision: 3 URL : https://patchwork.openembedded.org/series/15114/ State : failure == Summary == Thank you for submitting this patch series to OpenEmbedded Core. This is an automated response.

[OE-core] [PATCH v3 1/8] ptest: Reproducibility: Strip build host paths from any installed Makefile

2018-11-28 Thread Douglas Royds
A common pattern for ptest is to install a Makefile that provides a make check target. These generated Makefiles are normally full of build host paths, to hosttools, and to files and scripts within the source or build tree. They also commonly include the CFLAGS and CPPFLAGS. None of these build

[OE-core] [PATCH v3 4/8] openssl: Strip perl version from installed ptest configdata.pm file

2018-11-28 Thread Douglas Royds
Signed-off-by: Douglas Royds Signed-off-by: Richard Purdie --- meta/recipes-connectivity/openssl/openssl_1.1.1a.bb | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb b/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb index

[OE-core] [meta-oe][thud][PATCH] layerindexlib: change LAYERSERIES_COMPAT to thud

2018-11-28 Thread Oleksandr Kravchuk
Signed-off-by: Oleksandr Kravchuk --- bitbake/lib/layerindexlib/tests/restapi.py| 2 +- .../lib/layerindexlib/tests/testdata/build/conf/bblayers.conf | 2 +- .../lib/layerindexlib/tests/testdata/layer1/conf/layer.conf | 4 ++--

[OE-core] ✗ patchtest: failure for layerindexlib: change LAYERSERIES_COMPAT to thud

2018-11-28 Thread Patchwork
== Series Details == Series: layerindexlib: change LAYERSERIES_COMPAT to thud Revision: 1 URL : https://patchwork.openembedded.org/series/15124/ State : failure == Summary == Thank you for submitting this patch series to OpenEmbedded Core. This is an automated response. Several tests have

Re: [OE-core] [meta-oe][thud][PATCH] layerindexlib: change LAYERSERIES_COMPAT to thud

2018-11-28 Thread Mark Hatle
Does this actually matter? The test cases, just care that the layer branch has valid contents -- but they don't care what branch it's on. --Mark On 11/28/18 4:51 PM, Oleksandr Kravchuk wrote: > Signed-off-by: Oleksandr Kravchuk > --- > bitbake/lib/layerindexlib/tests/restapi.py

[OE-core] [PATCH v4 2/8] ptest: Optionally strip build host tool info from installed files

2018-11-28 Thread Douglas Royds
Several packages (eg. acl, attr, openssl) install Makefiles or other build host configuration files for ptest so that they can run `make check`, for instance. These build host files commonly include paths and versions of build host tools, (eg. rpm or perl) whose presence and version depends on the

[OE-core] [PATCH v4 0/8] ptest: Strip build host paths from any installed Makefile

2018-11-28 Thread Douglas Royds
Only one change from v1: Simplified commit acl: Use install instead of cp Douglas Royds (8): ptest: Reproducibility: Strip build host paths from any installed Makefile ptest: Optionally strip build host tool info from installed files openssl ptest: Strip build host paths from

[OE-core] [PATCH v4 1/8] ptest: Reproducibility: Strip build host paths from any installed Makefile

2018-11-28 Thread Douglas Royds
A common pattern for ptest is to install a Makefile that provides a make check target. These generated Makefiles are normally full of build host paths, to hosttools, and to files and scripts within the source or build tree. They also commonly include the CFLAGS and CPPFLAGS. None of these build

[OE-core] [PATCH v4 5/8] strace: Strip build host tool details from installed ptest Makefile

2018-11-28 Thread Douglas Royds
Adopt new PTEST_BUILD_HOST_PATTERN mechanism. Signed-off-by: Douglas Royds --- meta/recipes-devtools/strace/strace_4.25.bb | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/meta/recipes-devtools/strace/strace_4.25.bb b/meta/recipes-devtools/strace/strace_4.25.bb

[OE-core] [PATCH v4 7/8] acl: Strip build host tool details from installed ptest file

2018-11-28 Thread Douglas Royds
Adopt the PTEST_BUILD_HOST_FILES mechanism to strip build host paths from the installed builddefs file. Also strip the record of the build host rpm version. Signed-off-by: Douglas Royds --- meta/recipes-support/attr/acl_2.2.52.bb | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-)

[OE-core] [PATCH v4 8/8] attr: Strip build host paths and tool versions from ptest files

2018-11-28 Thread Douglas Royds
Using the new PTEST_BUILD_HOST_FILES mechanism. Specifically remove reference to build host rpm version, if any. Signed-off-by: Douglas Royds --- meta/recipes-support/attr/attr.inc | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/meta/recipes-support/attr/attr.inc

[OE-core] [PATCH v4 4/8] openssl: Strip perl version from installed ptest configdata.pm file

2018-11-28 Thread Douglas Royds
Signed-off-by: Douglas Royds --- meta/recipes-connectivity/openssl/openssl_1.1.1a.bb | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb b/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb index deeb865a6f..042e023de5 100644 ---

[OE-core] [PATCH v4 6/8] acl: Use install instead of cp

2018-11-28 Thread Douglas Royds
Signed-off-by: Douglas Royds --- meta/recipes-support/attr/acl_2.2.52.bb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/meta/recipes-support/attr/acl_2.2.52.bb b/meta/recipes-support/attr/acl_2.2.52.bb index 8b89de9b42..318308a009 100644 ---

[OE-core] [PATCH v4 3/8] openssl ptest: Strip build host paths from configdata.pm

2018-11-28 Thread Douglas Royds
This file contains CC, CPP, CFLAGS, CXXFLAGS and the like. Signed-off-by: Douglas Royds --- meta/recipes-connectivity/openssl/openssl_1.1.1a.bb | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb

Re: [OE-core] [PATCH v4 6/8] acl: Use install instead of cp

2018-11-28 Thread Mark Hatle
You should explain in the commit message -why- you made the change. It's already clear WHAT the change is, but not why you did it. On 11/28/18 5:43 PM, Douglas Royds wrote: > Signed-off-by: Douglas Royds > --- > meta/recipes-support/attr/acl_2.2.52.bb | 4 ++-- > 1 file changed, 2