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 insertions(+)
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
b/
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
--- a/meta/recipes-support/attr/a
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
in
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(-)
diff
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
--- a/meta/recipe
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.
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 hos
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
b/meta/recipes-connectivity/openssl/openssl_1.1
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
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
== 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 bee
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 ++--
.../lib/layerindexlib/tests/testdata/layer2/con
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 the
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
== 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. Severa
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 cp
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
b/meta/recipes-de
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
--- a/meta/recipes-support/attr/a
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
b/
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
b/meta/recipes-c
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(-)
diff
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
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
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 deeb865a6f..042e
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 hos
== 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. Severa
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 bzip2
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 |
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(-)
diff
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
--- a/meta/recipes-support/attr/a
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
b/
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, avoi
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 |
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 could
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 runqemu
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
---
meta/lib/oeqa/utils/comman
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 a/meta/lib/oeqa/utils/qe
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 a/meta/lib/oeqa/uti
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 t
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 avahi-
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&id=6f8f984ba363f764e83290b972ec31a90aad1603
Thanks
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 Brodki
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
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
O
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 hostna
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.
---
meta/conf/machine/include/arm/a
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
avahi-dev-0.
> -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
>
49 matches
Mail list logo