Re: [OE-core] [PATCH 1/1] openssh: upgrade to 7.4p1

[Dengke Du]

Hi Jussi

The remaining patch are still useful, I just rebase those patchs on 
version 74.p1. I will explain it.


Thanks.


On 2017年01月19日 20:32, Jussi Kukkonen wrote:



On 19 January 2017 at 13:52, Burton, Ross > wrote:


You're removing this patch:

On 19 January 2017 at 09:50, Dengke Du > wrote:

---

a/meta/recipes-connectivity/openssh/openssh/add-test-support-for-busybox.patch
+++ /dev/null
@@ -1,69 +0,0 @@
-Adjust test cases to work with busybox.
-
-- Replace dd parameter "obs" with "bs".
-- Replace "head -" with "head -n ".
-
-Signed-off-by: Maxin B. John >
-Upstream-Status: Pending
-
-Index: openssh-6.8p1/regress/cipher-speed.sh
-===
 openssh-6.8p1.orig/regress/cipher-speed.sh
-+++ openssh-6.8p1/regress/cipher-speed.sh
-@@ -17,7 +17,7 @@ for c in `${SSH} -Q cipher`; do n=0; for
-   printf "%-60s" "$c/$m:"
-   ( ${SSH} -o 'compression no' \
-   -F $OBJ/ssh_proxy -2 -m $m -c $c
somehost \
--  exec sh -c \'"dd of=/dev/null obs=32k"\' \
-+  exec sh -c \'"dd of=/dev/null bs=32k"\' \


But upstream still has exec sh -c \'"dd of=/dev/null obs=32k"\',
so this patch is still needed.

Please always explain why patches are being removed in the commit
message.


There's also lots of changes to the remaining patches. Some of them 
look weird -- possibly they're just whitespace changes? Please mention 
what the patch changes are in the commit message.


Jussi


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

Re: [OE-core] ✗ patchtest: failure for bash 4.3.30 -> 4.4

[Denys Dmytriyenko]

On Thu, Jan 19, 2017 at 03:23:42AM -, Patchwork wrote:
> == Series Details ==
> 
> Series: bash 4.3.30 -> 4.4
> Revision: 1
> URL   : https://patchwork.openembedded.org/series/4836/
> 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:

Wow, nice report from Patchwork!

May I suggest adding corresponding Wiki links to "Suggested fix" field? E.g.
http://openembedded.org/wiki/Commit_Patch_Message_Guidelines


> * Patchbash 4.3.30 -> 4.4
>  Issue Shortlog does not follow expected format 
> [test_shortlog_format] 
>   Suggested fixCommit shortlog (first line of commit message) should 
> follow the format ": "
> 
> * Issue Added patch file is missing Upstream-Status in the header 
> [test_upstream_status_presence] 
>   Suggested fixAdd Upstream-Status:  to the header of 
> meta/recipes-extended/bash/bash/0001-patch-to-obtain-pkgconfig-sanity.patch 
> (possible values: Pending, Submitted, Accepted, Backport, Denied, 
> Inappropriate)
> 
> * 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 a7cb408dd7)
> 
> 
> 
> 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] -> ...).
> 
> ---
> 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] still fighting to get an OE recipe for "libfilesys-smbclient-perl"

[Mike Looijmans]

On 19-01-17 21:58, Robert P. J. Day wrote:


  following up on the whining i did a while back to finish off a few
perl recipes for OE, i have just a few left to resolve, and this is
one of them. i managed to get several dozen perl recipes to work
simply by throwing together recipes based on very little understanding
of perl cpan mechanics and just mindlessly cribbing boilerplate from
other recipes. if it worked, yee ha, move on.

  here's my current recipe for this module (again, slapped together
without really understanding what's going on under the hood):

/ start /

DESCRIPTION = "Filesys::SmbClient"

SECTION = "libs"
LICENSE = "Artisticv1 | GPLv1+"
PR = "r0"

MAINTAINER= "rday"
HOMEPAGE=   "https://metacpan.org/pod/Filesys::SmbClient;

LIC_FILES_CHKSUM = 
"file://${COMMON_LICENSE_DIR}/Artistic-1.0;md5=cda03bbdc3c1951996392b872397b798 
\
file://${COMMON_LICENSE_DIR}/GPL-1.0;md5=e9e36a9de734199567a4d769498f743d"

DEPENDS += "samba"

SRC_URI = 
"https://cpan.metacpan.org/authors/id/A/AL/ALIAN/Filesys-SmbClient-3.2.tar.gz;

SRC_URI[md5sum] = "8a4205fe748aac7f43fb2a0e8ef502b2"
SRC_URI[sha256sum] = 
"a9f2f3d4675de76f9dcae2be40bda93495cd5fafb1bb6326d71833cea451bc6e"

S = "${WORKDIR}/Filesys-SmbClient-${PV}"

inherit cpan

EXTRA_CPANFLAGS = "INC='-I${STAGING_INCDIR}/samba-4.0' 
LIBS='-L${STAGING_LIBDIR}'"

BBCLASSEXTEND = "native"

/ end /

and the error i get when i try to build makes it *clear* that the
configuration is failing to set up properly for cross-compiling for my
powerpc target:

$ bitbake libfilesys-smbclient-perl
... snip ...
| Where can I find libsmbclient.h ? []
*
| ** WHAT 
| ** I Can't find /libsmbclient.h : No such file or directory.
| *
... snip ...
| Where can I find libsmbclient.so ? []
*
| ** WHAT 
| ** I Can't find libsmbclient.so : No such file or directory.
... snip ...
| checking for C compiler default output... a.out
| checking whether the C compiler works... configure: error: cannot
run C compiled programs.
| If you meant to cross compile, use `--host'.
| See `config.log' for more details.


  and sure enough, the config.log for this build contains:

Target: powerpc-poky-linux
Configured with:
../../../../../../work-shared/gcc-6.3.0-r0/gcc-6.3.0/configure
--build=x86_64-linux --host=x86_64-linux --target=powerpc-poky-linux
...


Quick observation: I think host and target are wrong. The "host" is where the 
software will run. The line above builds something that runs on x86_64 and 
that compiles for a powerpc target (target only makes sense if you're 
compiling a compiler). You probably wanted --host=powerpc-poky-linux and no 
"--target" at all.





  beyond this, i'm not sure how to use an OE patch to adjust this
source the right way, given that dozens and dozens of other modules
cross-compiled just fine.

  thoughts? i am in now way a perl expert so i really have no idea
where to go from here.

rday





Kind regards,

Mike Looijmans
System Expert

TOPIC Products
Materiaalweg 4, NL-5681 RJ Best
Postbus 440, NL-5680 AK Best
Telefoon: +31 (0) 499 33 69 79
E-mail: mike.looijm...@topicproducts.com
Website: www.topicproducts.com

Please consider the environment before printing this e-mail





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

Re: [OE-core] [PATCH] bash 4.3.30 -> 4.4

[Zheng, Ruoqin]

Hi Ross

I have try to patch it on the top master branch, it only has whitespace waring:

zhengrq@force:/yocto/work001/fnst/zrq/community/poky$ git apply 
0001-OE-core-bash-4.3.30-4.4.patch0001-OE-core-bash-4.3.30-4.4.patch:52: 
trailing whitespace.
0001-OE-core-bash-4.3.30-4.4.patch:58: trailing whitespace.
--
0001-OE-core-bash-4.3.30-4.4.patch:378: trailing whitespace.
+ rm -f ${BASH_TSTOUT}
0001-OE-core-bash-4.3.30-4.4.patch:383: trailing whitespace.
--
0001-OE-core-bash-4.3.30-4.4.patch:492: trailing whitespace.
SRC_URI[tarball.md5sum] = "14a7c95ac23705559b6f477dfe25"
warning: squelched 2 whitespace errors
warning: 7 lines add whitespace errors.

But the patch really works, I don’know why it doesn't apply to master?

Zheng


From: Burton, Ross [mailto:ross.bur...@intel.com]
Sent: Thursday, January 19, 2017 8:31 PM
To: Zheng, Ruoqin/郑 若钦 
Cc: OE-core 
Subject: Re: [OE-core] [PATCH] bash 4.3.30 -> 4.4

This doesn't apply to master for me, can you check please?

Ross

On 19 January 2017 at 02:52, Zheng Ruoqin 
> wrote:
1) Upgrade bash from 4.3.30 to 4.4
2) Delete 1 patche below, since they are integrated upstream.
   fix-run-intl.patch
3) Modify 2 patches below to make it compatible with new version
   fix-run-coproc-run-heredoc-run-execscript-run-test-f.patch
   test-output.patch
4) Add 1 patche below to solve pkgconfig sanity error
   0001-patch-to-obtain-pkgconfig-sanity.patch

Signed-off-by: Zheng Ruoqin 
>
---
 .../0001-patch-to-obtain-pkgconfig-sanity.patch|  26 
 ...roc-run-heredoc-run-execscript-run-test-f.patch | 137 ++---
 meta/recipes-extended/bash/bash/fix-run-intl.patch | 110 -
 meta/recipes-extended/bash/bash/test-output.patch  |  20 ++-
 meta/recipes-extended/bash/bash_4.3.30.bb  |  
71 ---
 meta/recipes-extended/bash/bash_4.4.bb |  38 
++
 6 files changed, 90 insertions(+), 312 deletions(-)
 create mode 100644 
meta/recipes-extended/bash/bash/0001-patch-to-obtain-pkgconfig-sanity.patch
 delete mode 100644 meta/recipes-extended/bash/bash/fix-run-intl.patch
 delete mode 100644 
meta/recipes-extended/bash/bash_4.3.30.bb
 create mode 100644 meta/recipes-extended/bash/bash_4.4.bb

diff --git 
a/meta/recipes-extended/bash/bash/0001-patch-to-obtain-pkgconfig-sanity.patch 
b/meta/recipes-extended/bash/bash/0001-patch-to-obtain-pkgconfig-sanity.patch
new file mode 100644
index 000..031eb90
--- /dev/null
+++ 
b/meta/recipes-extended/bash/bash/0001-patch-to-obtain-pkgconfig-sanity.patch
@@ -0,0 +1,26 @@
+Subject: [PATCH] patch to obtain pkgconfig sanity
+
+Drop the paths from the pkgconfig file bash.pc.in which 
will cause pkgconfig sanity error below:
+ERROR: QA Issue: bash.pc failed sanity test (tmpdir) in path 
/yocto/work001/fnst/zrq/community/poky/build/tmp/work/i586-poky-linux/bash
/4.4-r0/sysroot-destdir/usr/lib/pkgconfig [pkgconfig]
+
+Signed-off-by: Zheng Ruoqin 
>
+---
+ support/bash.pc.in | 2 --
+ 1 file changed, 2 deletions(-)
+
+diff --git a/support/bash.pc.in 
b/support/bash.pc.in
+index 6009659..871fa9f 100644
+--- a/support/bash.pc.in
 b/support/bash.pc.in
+@@ -12,8 +12,6 @@ LOCAL_CFLAGS = @LOCAL_CFLAGS@
+ LOCAL_DEFS = @LOCAL_DEFS@
+ CCFLAGS = ${LOCAL_DEFS} ${LOCAL_CFLAGS}
+
+-CC = @CC@
+-SHOBJ_CC = @SHOBJ_CC@
+ SHOBJ_CFLAGS = @SHOBJ_CFLAGS@
+ SHOBJ_LD = @SHOBJ_LD@
+ SHOBJ_LDFLAGS = @SHOBJ_LDFLAGS@
+--
+2.7.4
+
diff --git 
a/meta/recipes-extended/bash/bash/fix-run-coproc-run-heredoc-run-execscript-run-test-f.patch
 
b/meta/recipes-extended/bash/bash/fix-run-coproc-run-heredoc-run-execscript-run-test-f.patch
index 7f099ae..d6beaff 100644
--- 
a/meta/recipes-extended/bash/bash/fix-run-coproc-run-heredoc-run-execscript-run-test-f.patch
+++ 
b/meta/recipes-extended/bash/bash/fix-run-coproc-run-heredoc-run-execscript-run-test-f.patch
@@ -20,150 +20,35 @@ the test suite should not be run as root
 Upstream-Status: Pending

 Signed-off-by: Hongxu Jia 
>
+Signed-off-by: Zheng Ruoqin 
>
 ---
- tests/coproc.right   |  5 +
- tests/coproc.tests   | 30 +-
- tests/heredoc.right  |  5 ++---
- tests/heredoc3.sub   |  3 ++-
- tests/run-execscript |  3 ++-
- tests/run-test   |  3 ++-
- 6 files changed, 34 insertions(+), 15 deletions(-)
+ tests/run-execscript | 3 ++-
+ tests/run-test   | 3 ++-
+ 2 files changed, 4 insertions(+), 2 deletions(-)

-diff --git a/tests/coproc.right 

[OE-core] [PATCH v2 0/1] oe-selftest: devtool: remove use of git -C

[Paul Eggleton]

The following changes since commit fc97963bc61bf16112859fe1d7e460a13d34baca:

  oeqa/selftest/devtool: rewrite modify testcase (2017-01-19 22:45:46 +)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib paule/devtool-git-fix1
  
http://cgit.openembedded.org/openembedded-core-contrib/log/?h=paule/devtool-git-fix1

Paul Eggleton (1):
  oe-selftest: devtool: remove use of git -C

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

-- 
2.5.5

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

[OE-core] [PATCH v2 1/1] oe-selftest: devtool: remove use of git -C

[Paul Eggleton]

The -C option isn't available in versions of git older than 1.8.5,
and officially we only require git 1.8.3.1 or newer (and the latter is
the version you'll find on CentOS 7, so the test fails there). In any
case we can simply specify the working directory to runCmd() so just
do that instead.

Signed-off-by: Paul Eggleton 
---
 meta/lib/oeqa/selftest/devtool.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/selftest/devtool.py 
b/meta/lib/oeqa/selftest/devtool.py
index 887c1e6..dc3435f 100644
--- a/meta/lib/oeqa/selftest/devtool.py
+++ b/meta/lib/oeqa/selftest/devtool.py
@@ -461,7 +461,7 @@ class DevtoolTests(DevtoolBase):
 bitbake('mdadm -c package')
 check_line(manfile, 'antique pin sardine', 'man file not modified. man 
searched file path: %s' % manfile)
 
-result = runCmd('git -C %s checkout -- %s' % (tempdir, modfile))
+result = runCmd('git checkout -- %s' % modfile, cwd=tempdir)
 check_line(modfile, 'Linux Software RAID', 'man .in file not restored 
(git failed)')
 
 bitbake('mdadm -c package')
-- 
2.5.5

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

Re: [OE-core] still fighting to get an OE recipe for "libfilesys-smbclient-perl"

[Tim Orling]

On Thu, Jan 19, 2017 at 3:45 PM, Burton, Ross  wrote:

>
> On 19 January 2017 at 20:58, Robert P. J. Day 
> wrote:
>
>>   beyond this, i'm not sure how to use an OE patch to adjust this
>> source the right way, given that dozens and dozens of other modules
>> cross-compiled just fine.
>>
>>   thoughts? i am in now way a perl expert so i really have no idea
>> where to go from here
>>
>
> Is this the module you asked about earlier that I had a quick look at?
> I'd say that basically it can't be cross-compiled easily - or with moderate
> effort - because of various pieces of crazy in the way it builds.
>
> My recommendation is to either find or become a Perl build system expert
> and entirely rewrite the build to use something modern and sane, instead of
> something archaic, hand-coded and crazy.
>
>
I have to agree with Ross here.

The original Perl module is in sorry shape. I would start with getting
somebody in the Perl world to rewrite it first (at least the build
mechanism). It has native code that is just not going to be easy to
cross-compile the way it is without serious effort and hacking. If it was
ANY kind of sane, modern, perl build system I would have it done already
and collected the bounty. The bounty is an order of magnitude too low for
the effort it would take me to get it to work (and that might still be an
underbid by a factor of 2 or more). You might be better off contracting
someone to meet the API in a new perl module. Perhaps the maintainer of
meta-cpan?

--Tim

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

[OE-core] ✗ patchtest: failure for oe-selftest: devtool: remove use of git -C

[Patchwork]

== Series Details ==

Series: oe-selftest: devtool: remove use of git -C
Revision: 1
URL   : https://patchwork.openembedded.org/series/4861/
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 fc97963bc6)



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] -> ...).

---
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] openssh: upgrade to 7.4p1

[Dengke Du]

Thanks for reminding, I use the "devtool modify openssh", so it has 
apply the add-test-


support-for-busybox.patch to the openssh, I am not find this, it's my 
fault. I will do this again.


On 2017年01月19日 19:52, Burton, Ross wrote:

You're removing this patch:

On 19 January 2017 at 09:50, Dengke Du > wrote:


---

a/meta/recipes-connectivity/openssh/openssh/add-test-support-for-busybox.patch
+++ /dev/null
@@ -1,69 +0,0 @@
-Adjust test cases to work with busybox.
-
-- Replace dd parameter "obs" with "bs".
-- Replace "head -" with "head -n ".
-
-Signed-off-by: Maxin B. John >
-Upstream-Status: Pending
-
-Index: openssh-6.8p1/regress/cipher-speed.sh
-===
 openssh-6.8p1.orig/regress/cipher-speed.sh
-+++ openssh-6.8p1/regress/cipher-speed.sh
-@@ -17,7 +17,7 @@ for c in `${SSH} -Q cipher`; do n=0; for
-   printf "%-60s" "$c/$m:"
-   ( ${SSH} -o 'compression no' \
-   -F $OBJ/ssh_proxy -2 -m $m -c $c somehost \
--  exec sh -c \'"dd of=/dev/null obs=32k"\' \
-+  exec sh -c \'"dd of=/dev/null bs=32k"\' \


But upstream still has exec sh -c \'"dd of=/dev/null obs=32k"\', so 
this patch is still needed.


Please always explain why patches are being removed in the commit message.

Ross


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

[OE-core] [PATCH 1/1] oe-selftest: devtool: remove use of git -C

[Paul Eggleton]

The -C option isn't available in versions of git older than 1.8.5,
and officially we only require git 1.8.3.1 or newer (and the latter is
the version you'll find on CentOS 7, so the test fails there). In any
case we can simply specify the working directory to runCmd() so just
do that instead.

Signed-off-by: Paul Eggleton 
---
 meta/lib/oeqa/selftest/devtool.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/selftest/devtool.py 
b/meta/lib/oeqa/selftest/devtool.py
index 94b6b39..face8b8 100644
--- a/meta/lib/oeqa/selftest/devtool.py
+++ b/meta/lib/oeqa/selftest/devtool.py
@@ -473,7 +473,7 @@ class DevtoolTests(DevtoolBase):
 manfile = os.path.join(pkgd, mandir, 'man8', 'mdadm.8')
 check_TH_line(manfile, '.TH MDADM 8 "" v9.999-custom', 'man file not 
modified. man searched file path: %s' % manfile)
 # Test reverting the change
-result = runCmd("git -C %s checkout -- %s" % (tempdir, modfile))
+result = runCmd("git checkout -- %s" % modfile, cwd=tempdir)
 check_TH_line(modfile, '.TH MDADM 8 "" v3.4', 'man .in file not 
restored (git failed)')
 bitbake('mdadm -c package')
 pkgd = get_bb_var('PKGD', 'mdadm')
-- 
2.5.5

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

[OE-core] [PATCH 0/1] oe-selftest: devtool: remove use of git -C

[Paul Eggleton]

The following changes since commit a7cb408dd784178197687a2129e936620bf6a0d3:

  sanity.bbclass: Improved error message (2017-01-17 11:31:31 +)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib paule/devtool-git-fix1
  
http://cgit.openembedded.org/openembedded-core-contrib/log/?h=paule/devtool-git-fix1

Paul Eggleton (1):
  oe-selftest: devtool: remove use of git -C

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

-- 
2.5.5

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

[OE-core] [PATCH] selftest: automatically add the meta-selftest layer

[Benjamin Esquivel]

adding the meta-selftest layer is a small bump you always encounter
when you want to run selftest. Adding an automatic procedure to include
the layer if it is not yet present.

Signed-off-by: Benjamin Esquivel 
---
 scripts/oe-selftest | 12 ++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/scripts/oe-selftest b/scripts/oe-selftest
index 120a6c2..b81e58c 100755
--- a/scripts/oe-selftest
+++ b/scripts/oe-selftest
@@ -131,8 +131,16 @@ def preflight_check():
 os.chdir(builddir)
 
 if not "meta-selftest" in get_bb_var("BBLAYERS"):
-log.error("You don't seem to have the meta-selftest layer in BBLAYERS")
-return False
+log.warn("meta-selftest layer not found in BBLAYERS, adding it")
+meta_selftestdir = os.path.join(
+get_bb_var("BBLAYERS_FETCH_DIR"),
+'meta-selftest')
+if os.path.isdir(meta_selftestdir):
+runCmd("bitbake-layers add-layer %s" %meta_selftestdir)
+else:
+log.error("could not locate meta-selftest in:\n%s"
+%meta_selftestdir)
+return False
 
 log.info("Running bitbake -p")
 runCmd("bitbake -p")
-- 
2.5.1

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

Re: [OE-core] still fighting to get an OE recipe for "libfilesys-smbclient-perl"

[Burton, Ross]

On 19 January 2017 at 20:58, Robert P. J. Day  wrote:

>   beyond this, i'm not sure how to use an OE patch to adjust this
> source the right way, given that dozens and dozens of other modules
> cross-compiled just fine.
>
>   thoughts? i am in now way a perl expert so i really have no idea
> where to go from here
>

Is this the module you asked about earlier that I had a quick look at?  I'd
say that basically it can't be cross-compiled easily - or with moderate
effort - because of various pieces of crazy in the way it builds.

My recommendation is to either find or become a Perl build system expert
and entirely rewrite the build to use something modern and sane, instead of
something archaic, hand-coded and crazy.

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

Re: [OE-core] [PATCH 2/2] Revert "kernel: Modify kernel modules installation path."

[Burton, Ross]

On 19 January 2017 at 21:24, Jason Wessel 
wrote:

> It may be fixable by using, ${nonarch_base_libdir}, but it wasn't quite
> that simple because it didn't fully work for the firmware loader so for now
> it would be best to just revert the two patches and keep oe-core working.
>

I saw the first commit message, slapped my forehead and thought
nonarch_base_libdir.  What was the problem this approach had?

We obviously need to do more real hardware testing of multilib...

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

[OE-core] [PATCH 24/33] update-rc.d: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

use_updatercd() will always return true in rootfs generation so
checking that is not required.

Signed-off-by: Jussi Kukkonen 
---
 meta/classes/update-rc.d.bbclass | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/classes/update-rc.d.bbclass b/meta/classes/update-rc.d.bbclass
index 9d3a7bc..36f9009 100644
--- a/meta/classes/update-rc.d.bbclass
+++ b/meta/classes/update-rc.d.bbclass
@@ -1,6 +1,6 @@
 UPDATERCPN ?= "${PN}"
 
-DEPENDS_append_class-target = "${@bb.utils.contains('DISTRO_FEATURES', 
'sysvinit', ' update-rc.d-native update-rc.d initscripts', '', d)}"
+DEPENDS_append_class-target = "${@bb.utils.contains('DISTRO_FEATURES', 
'sysvinit', ' update-rc.d initscripts', '', d)}"
 
 UPDATERCD = "update-rc.d"
 UPDATERCD_class-cross = ""
@@ -34,6 +34,8 @@ if ${@use_updatercd(d)} && type update-rc.d >/dev/null 
2>/dev/null; then
 fi
 }
 
+PACKAGE_WRITE_DEPS += "update-rc.d-native"
+
 updatercd_postinst() {
 # Begin section update-rc.d
 if ${@use_updatercd(d)} && type update-rc.d >/dev/null 2>/dev/null; then
-- 
2.7.4

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

[OE-core] [PATCH 23/33] dbus: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

systemctl is needed if both systemd and sysvinit are in distro
features.

Signed-off-by: Jussi Kukkonen 
---
 meta/recipes-core/dbus/dbus_1.10.14.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/dbus/dbus_1.10.14.bb 
b/meta/recipes-core/dbus/dbus_1.10.14.bb
index 2ffd8b0..e4f54ec 100644
--- a/meta/recipes-core/dbus/dbus_1.10.14.bb
+++ b/meta/recipes-core/dbus/dbus_1.10.14.bb
@@ -76,6 +76,7 @@ FILES_${PN}-lib = "${libdir}/lib*.so.*"
 RRECOMMENDS_${PN}-lib = "${PN}"
 FILES_${PN}-dev += "${libdir}/dbus-1.0/include ${bindir}/dbus-test-tool"
 
+PACKAGE_WRITE_DEPS += "${@bb.utils.contains('DISTRO_FEATURES','systemd 
sysvinit','systemd-systemctl-native','',d)}"
 pkg_postinst_dbus() {
# If both systemd and sysvinit are enabled, mask the dbus-1 init script
 if ${@bb.utils.contains('DISTRO_FEATURES','systemd 
sysvinit','true','false',d)}; then
-- 
2.7.4

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

[OE-core] [PATCH 28/33] gsettings: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

glib-compile-schemas is needed during postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/classes/gsettings.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/gsettings.bbclass b/meta/classes/gsettings.bbclass
index e6d1c8a..eae3dc7 100644
--- a/meta/classes/gsettings.bbclass
+++ b/meta/classes/gsettings.bbclass
@@ -7,12 +7,13 @@
 
 # TODO use a trigger so that this runs once per package operation run
 
-DEPENDS += "glib-2.0-native"
 
 RDEPENDS_${PN} += "glib-2.0-utils"
 
 FILES_${PN} += "${datadir}/glib-2.0/schemas"
 
+PACKAGE_WRITE_DEPS += "glib-2.0-native"
+
 gsettings_postinstrm () {
glib-compile-schemas $D${datadir}/glib-2.0/schemas
 }
-- 
2.7.4

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

[OE-core] [PATCH 31/33] fontcache.bbclass: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

Qemu is used to run fc-cache on postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/classes/fontcache.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/fontcache.bbclass b/meta/classes/fontcache.bbclass
index d047a79..e763311 100644
--- a/meta/classes/fontcache.bbclass
+++ b/meta/classes/fontcache.bbclass
@@ -3,7 +3,7 @@
 # packages.
 #
 
-DEPENDS += "qemu-native"
+PACKAGE_WRITE_DEPS += "qemu-native"
 inherit qemu
 
 FONT_PACKAGES ??= "${PN}"
-- 
2.7.4

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

[OE-core] [PATCH 33/33] kernel.bbclass: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

The depmodwrapper dependency is not actually used by the class but
anyone using pkg_postinst_kernel-base() will need it.

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

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 3630042..f462b2f 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -1,7 +1,8 @@
 inherit linux-kernel-base kernel-module-split
 
 PROVIDES += "virtual/kernel"
-DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc 
kmod-native depmodwrapper-cross bc-native lzop-native"
+DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc 
kmod-native bc-native lzop-native"
+PACKAGE_WRITE_DEPS += "depmodwrapper-cross virtual/update-alternatives-native"
 
 S = "${STAGING_KERNEL_DIR}"
 B = "${WORKDIR}/build"
-- 
2.7.4

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

[OE-core] [PATCH 25/33] update-alternatives.bbclass: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

Signed-off-by: Jussi Kukkonen 
---
 meta/classes/update-alternatives.bbclass | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/classes/update-alternatives.bbclass 
b/meta/classes/update-alternatives.bbclass
index a90ef19..ca7fe43 100644
--- a/meta/classes/update-alternatives.bbclass
+++ b/meta/classes/update-alternatives.bbclass
@@ -65,6 +65,8 @@ ALTERNATIVE_PRIORITY = "10"
 # and include that vairable in the set.
 UPDALTVARS  = "ALTERNATIVE ALTERNATIVE_LINK_NAME ALTERNATIVE_TARGET 
ALTERNATIVE_PRIORITY"
 
+PACKAGE_WRITE_DEPS += "virtual/update-alternatives-native"
+
 def gen_updatealternativesvardeps(d):
 pkgs = (d.getVar("PACKAGES") or "").split()
 vars = (d.getVar("UPDALTVARS") or "").split()
-- 
2.7.4

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

[OE-core] [PATCH 26/33] mime.bbclass: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

update-mime-database is used in postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/classes/mime.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/mime.bbclass b/meta/classes/mime.bbclass
index 6cd59af..0df1583 100644
--- a/meta/classes/mime.bbclass
+++ b/meta/classes/mime.bbclass
@@ -1,4 +1,5 @@
-DEPENDS += "shared-mime-info-native shared-mime-info"
+DEPENDS += "shared-mime-info"
+PACKAGE_WRITE_DEPS += "shared-mime-info-native"
 
 mime_postinst() {
 if [ "$1" = configure ]; then
-- 
2.7.4

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

[OE-core] [PATCH 18/33] linux-dtb: Add PACKAGE_WRITE_DEP for postinstall

[Richard Purdie]

From: Jussi Kukkonen 

update-alternatives is used in postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/recipes-kernel/linux/linux-dtb.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-kernel/linux/linux-dtb.inc 
b/meta/recipes-kernel/linux/linux-dtb.inc
index 0e72093..0174c80 100644
--- a/meta/recipes-kernel/linux/linux-dtb.inc
+++ b/meta/recipes-kernel/linux/linux-dtb.inc
@@ -1,6 +1,8 @@
 # Support for device tree generation
 FILES_kernel-devicetree = "/${KERNEL_IMAGEDEST}/devicetree*"
 
+PACKAGE_WRITE_DEPS += "virtual/update-alternatives-native"
+
 python __anonymous () {
 d.appendVar("PACKAGES", " kernel-devicetree")
 }
-- 
2.7.4

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

[OE-core] [PATCH 29/33] gio-module-cache: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

Qemu is used to run gio-querymodules on postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/classes/gio-module-cache.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/gio-module-cache.bbclass 
b/meta/classes/gio-module-cache.bbclass
index 39b7bef..a8190b7 100644
--- a/meta/classes/gio-module-cache.bbclass
+++ b/meta/classes/gio-module-cache.bbclass
@@ -1,4 +1,4 @@
-DEPENDS += "qemu-native"
+PACKAGE_WRITE_DEPS += "qemu-native"
 inherit qemu
 
 GIO_MODULE_PACKAGES ??= "${PN}"
-- 
2.7.4

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

[OE-core] [PATCH 05/33] kernel-module-split: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs kmod-native and depmodwrapper-cross, mark these 
dependencies.

Signed-off-by: Richard Purdie 
---
 meta/classes/kernel-module-split.bbclass | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/classes/kernel-module-split.bbclass 
b/meta/classes/kernel-module-split.bbclass
index 742320c..49cd8e9 100644
--- a/meta/classes/kernel-module-split.bbclass
+++ b/meta/classes/kernel-module-split.bbclass
@@ -22,6 +22,8 @@ if [ x"$D" = "x" ]; then
 fi
 }
 
+PACKAGE_WRITE_DEPS += "kmod-native depmodwrapper-cross"
+
 do_install_append() {
install -d ${D}${sysconfdir}/modules-load.d/ 
${D}${sysconfdir}/modprobe.d/
 }
-- 
2.7.4

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

[OE-core] [PATCH 19/33] xorg-font-common: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

mkfontdir and mkfontscale are used in the postinstall script.

Signed-off-by: Jussi Kukkonen 
---
 meta/recipes-graphics/xorg-font/xorg-font-common.inc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/xorg-font/xorg-font-common.inc 
b/meta/recipes-graphics/xorg-font/xorg-font-common.inc
index d5267f59..cdbebcf 100644
--- a/meta/recipes-graphics/xorg-font/xorg-font-common.inc
+++ b/meta/recipes-graphics/xorg-font/xorg-font-common.inc
@@ -4,7 +4,7 @@ BUGTRACKER = 
"https://bugs.freedesktop.org/enter_bug.cgi?product=xorg;
 SECTION = "x11/fonts"
 LICENSE = "MIT-X"
 
-DEPENDS = " encodings font-alias font-util-native mkfontdir-native 
mkfontscale-native"
+DEPENDS = " encodings font-alias font-util-native"
 RDEPENDS_${PN} = "encodings font-util font-alias"
 
 XORG_PN = "${BPN}"
@@ -30,6 +30,7 @@ do_install_append() {
 
 FILES_${PN} += " ${libdir}/X11/fonts ${datadir}"
 
+PACKAGE_WRITE_DEPS += "mkfontdir-native mkfontscale-native"
 pkg_postinst_${PN} () {
 for fontdir in `find $D/usr/lib/X11/fonts -type d`; do
 mkfontdir $fontdir
-- 
2.7.4

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

[OE-core] [PATCH 27/33] gtk-immodules-cache: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

Qemu is used to run gtk-query-immodules-* on postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/classes/gtk-immodules-cache.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/gtk-immodules-cache.bbclass 
b/meta/classes/gtk-immodules-cache.bbclass
index baea959..3d82dbe 100644
--- a/meta/classes/gtk-immodules-cache.bbclass
+++ b/meta/classes/gtk-immodules-cache.bbclass
@@ -2,7 +2,7 @@
 #
 # Usage: Set GTKIMMODULES_PACKAGES to the packages that needs to update the 
inputmethod modules
 
-DEPENDS =+ "qemu-native"
+PACKAGE_WRITE_DEPS += "qemu-native"
 
 inherit qemu
 
-- 
2.7.4

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

[OE-core] [PATCH 32/33] cantarell-fonts: inherit pkgconfig

[Richard Purdie]

From: Jussi Kukkonen 

configure fails without pkg-config.

Signed-off-by: Jussi Kukkonen 
---
 meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb 
b/meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb
index e3e4cde..c71ab11 100644
--- a/meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb
+++ b/meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb
@@ -18,7 +18,7 @@ UPSTREAM_CHECK_GITTAGREGEX = 
"(?P(?!0\.13)(?!0\.10\.1)\d+\.\d+(\.\d+)+)"
 
 S = "${WORKDIR}/git"
 
-inherit autotools allarch fontcache
+inherit autotools allarch fontcache pkgconfig
 
 PACKAGECONFIG ??= ""
 PACKAGECONFIG[fontforge] = 
"--enable-source-rebuild=yes,--enable-source-rebuild=no,fontforge-native"
-- 
2.7.4

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

[OE-core] [PATCH 21/33] eudev: Add PACKAGE_WRITE_DEPS for postinstall

[Richard Purdie]

From: Jussi Kukkonen 

Qemu is used to run udevadm in postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/recipes-core/udev/eudev_3.2.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/udev/eudev_3.2.bb 
b/meta/recipes-core/udev/eudev_3.2.bb
index 211252c..385ee7a 100644
--- a/meta/recipes-core/udev/eudev_3.2.bb
+++ b/meta/recipes-core/udev/eudev_3.2.bb
@@ -87,6 +87,7 @@ python () {
 raise bb.parse.SkipPackage("'systemd' in DISTRO_FEATURES")
 }
 
+PACKAGE_WRITE_DEPS += "qemu-native"
 pkg_postinst_eudev-hwdb () {
 if test -n "$D"; then
 ${@qemu_run_binary(d, '$D', '${bindir}/udevadm')} hwdb --update --root 
$D
-- 
2.7.4

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

[OE-core] [PATCH 20/33] shadow: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

pwconv and grpconv are used in the postinstall script.

Signed-off-by: Jussi Kukkonen 
---
 meta/recipes-extended/shadow/shadow.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/shadow/shadow.inc 
b/meta/recipes-extended/shadow/shadow.inc
index f79565b..4c4db00 100644
--- a/meta/recipes-extended/shadow/shadow.inc
+++ b/meta/recipes-extended/shadow/shadow.inc
@@ -6,7 +6,6 @@ LICENSE = "BSD | Artistic-1.0"
 LIC_FILES_CHKSUM = "file://COPYING;md5=ed80ff1c2b40843cf5768e5229cf16e5 \
 
file://src/passwd.c;beginline=8;endline=30;md5=d83888ea14ae61951982d77125947661"
 
-DEPENDS = "shadow-native"
 DEPENDS_class-native = ""
 DEPENDS_class-nativesdk = ""
 
@@ -187,6 +186,7 @@ ALTERNATIVE_LINK_NAME[groups.1] = "${mandir}/man1/groups.1"
 ALTERNATIVE_LINK_NAME[su.1] = "${mandir}/man1/su.1"
 ALTERNATIVE_LINK_NAME[nologin.8] = "${mandir}/man8/nologin.8"
 
+PACKAGE_WRITE_DEPS += "shadow-native"
 pkg_postinst_${PN} () {
if [ "x$D" != "x" ]; then
  rootarg="--root $D"
-- 
2.7.4

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

[OE-core] [PATCH 22/33] systemd: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

Qemu is used to run udevadm in postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/recipes-core/systemd/systemd_232.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/systemd/systemd_232.bb 
b/meta/recipes-core/systemd/systemd_232.bb
index 9b76b69..cc8781e 100644
--- a/meta/recipes-core/systemd/systemd_232.bb
+++ b/meta/recipes-core/systemd/systemd_232.bb
@@ -4,7 +4,7 @@ PROVIDES = "udev"
 
 PE = "1"
 
-DEPENDS = "kmod intltool-native gperf-native acl readline libcap libcgroup 
qemu-native util-linux"
+DEPENDS = "kmod intltool-native gperf-native acl readline libcap libcgroup 
util-linux"
 
 SECTION = "base/shell"
 
@@ -571,6 +571,7 @@ pkg_prerm_${PN} () {
-i $D${sysconfdir}/nsswitch.conf
 }
 
+PACKAGE_WRITE_DEPS += "qemu-native"
 pkg_postinst_udev-hwdb () {
if test -n "$D"; then
${@qemu_run_binary(d, '$D', '${base_bindir}/udevadm')} hwdb 
--update \
-- 
2.7.4

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

[OE-core] [PATCH 04/33] gtk-icon-cache: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

gdk-pixbuf-native and gtk-icon-utils-native are needed by the postinstall
scripts so mark the dependency. The utils may be needed at icon build
time too so DEPENDS is unchanged.

Signed-off-by: Richard Purdie 
---
 meta/classes/gtk-icon-cache.bbclass | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/classes/gtk-icon-cache.bbclass 
b/meta/classes/gtk-icon-cache.bbclass
index c5d8d7c..d87167a 100644
--- a/meta/classes/gtk-icon-cache.bbclass
+++ b/meta/classes/gtk-icon-cache.bbclass
@@ -2,6 +2,8 @@ FILES_${PN} += "${datadir}/icons/hicolor"
 
 DEPENDS += "${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} 
gtk-icon-utils-native"
 
+PACKAGE_WRITE_DEPS += "gtk-icon-utils-native gdk-pixbuf-native"
+
 gtk_icon_cache_postinst() {
 if [ "x$D" != "x" ]; then
$INTERCEPT_DIR/postinst_intercept update_icon_cache ${PKG} \
-- 
2.7.4

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

[OE-core] [PATCH 30/33] gconf.bbclass: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

From: Jussi Kukkonen 

gconftool-2 is used during postinstall.

Signed-off-by: Jussi Kukkonen 
---
 meta/classes/gconf.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/gconf.bbclass b/meta/classes/gconf.bbclass
index d07bead..4e0ee2e 100644
--- a/meta/classes/gconf.bbclass
+++ b/meta/classes/gconf.bbclass
@@ -1,4 +1,5 @@
-DEPENDS += "gconf gconf-native"
+DEPENDS += "gconf"
+PACKAGE_WRITE_DEPS += "gconf-native"
 
 # These are for when gconftool is used natively and the prefix isn't 
necessarily
 # the sysroot.  TODO: replicate the postinst logic for -native packages going
-- 
2.7.4

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

[OE-core] [PATCH 12/33] systemd-compat-units: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs systemd-systemctl-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/recipes-core/systemd/systemd-compat-units.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/systemd/systemd-compat-units.bb 
b/meta/recipes-core/systemd/systemd-compat-units.bb
index 0f0876b..fe9a521 100644
--- a/meta/recipes-core/systemd/systemd-compat-units.bb
+++ b/meta/recipes-core/systemd/systemd-compat-units.bb
@@ -4,7 +4,7 @@ LICENSE = "MIT"
 
 PR = "r29"
 
-DEPENDS = "systemd-systemctl-native"
+PACKAGE_WRITE_DEPS += "systemd-systemctl-native"
 
 S = "${WORKDIR}"
 
-- 
2.7.4

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

[OE-core] [PATCH 03/33] sstate: Drop the depchain isPostDep() checks

[Richard Purdie]

The dependencies of do_package_write_* tasks are either going to be packaging
tools needed to build the packages, or, native tools needed at postinst
time. Now we've formalised this dependency pattern, drop the hardcoded
list and work based on the rule. The package creation tools are usually
the same tools needed at rootfs/postinst time anyway so the difference is
moot.

Signed-off-by: Richard Purdie 
---
 meta/classes/sstate.bbclass | 16 +++-
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index b56e95a..f1faf48 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -871,17 +871,14 @@ BB_SETSCENE_DEPVALID = "setscene_depvalid"
 def setscene_depvalid(task, taskdependees, notneeded, d):
 # taskdependees is a dict of tasks which depend on task, each being a 3 
item list of [PN, TASKNAME, FILENAME]
 # task is included in taskdependees too
+# Return - False - We need this dependency
+#- True - We can skip this dependency
 
 bb.debug(2, "Considering setscene task: %s" % (str(taskdependees[task])))
 
 def isNativeCross(x):
 return x.endswith("-native") or "-cross-" in x or "-crosssdk" in x
 
-def isPostInstDep(x):
-if x in ["qemu-native", "gdk-pixbuf-native", "qemuwrapper-cross", 
"depmodwrapper-cross", "systemd-systemctl-native", "gtk-icon-utils-native", 
"ca-certificates-native"]:
-return True
-return False
-
 # We only need to trigger populate_lic through direct dependencies
 if taskdependees[task][1] == "do_populate_lic":
 return True
@@ -903,10 +900,11 @@ def setscene_depvalid(task, taskdependees, notneeded, d):
 # do_package_write_* and do_package doesn't need do_package
 if taskdependees[task][1] == "do_package" and taskdependees[dep][1] in 
['do_package', 'do_package_write_deb', 'do_package_write_ipk', 
'do_package_write_rpm', 'do_packagedata', 'do_package_qa']:
 continue
-# do_package_write_* and do_package doesn't need do_populate_sysroot, 
unless is a postinstall dependency
-if taskdependees[task][1] == "do_populate_sysroot" and 
taskdependees[dep][1] in ['do_package', 'do_package_write_deb', 
'do_package_write_ipk', 'do_package_write_rpm', 'do_packagedata', 
'do_package_qa']:
-if isPostInstDep(taskdependees[task][0]) and taskdependees[dep][1] 
in ['do_package_write_deb', 'do_package_write_ipk', 'do_package_write_rpm']:
-return False
+# do_package_write_* need do_populate_sysroot as they're mainly 
postinstall dependencies
+if taskdependees[task][1] == "do_populate_sysroot" and 
taskdependees[dep][1] in ['do_package_write_deb', 'do_package_write_ipk', 
'do_package_write_rpm']:
+return False
+# do_package/packagedata/package_qa don't need do_populate_sysroot
+if taskdependees[task][1] == "do_populate_sysroot" and 
taskdependees[dep][1] in ['do_package', 'do_packagedata', 'do_package_qa']:
 continue
 # Native/Cross packages don't exist and are noexec anyway
 if isNativeCross(taskdependees[dep][0]) and taskdependees[dep][1] in 
['do_package_write_deb', 'do_package_write_ipk', 'do_package_write_rpm', 
'do_packagedata', 'do_package', 'do_package_qa']:
-- 
2.7.4

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

[OE-core] [PATCH 08/33] keymaps: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs systemd-systemctl-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/recipes-bsp/keymaps/keymaps_1.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-bsp/keymaps/keymaps_1.0.bb 
b/meta/recipes-bsp/keymaps/keymaps_1.0.bb
index 5793a76..34b208c 100644
--- a/meta/recipes-bsp/keymaps/keymaps_1.0.bb
+++ b/meta/recipes-bsp/keymaps/keymaps_1.0.bb
@@ -37,7 +37,7 @@ do_install () {
 fi
 }
 
-DEPENDS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
+PACKAGE_WRITE_DEPS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
 pkg_postinst_${PN} () {
if ${@bb.utils.contains('DISTRO_FEATURES','systemd 
sysvinit','true','false',d)}; then
if [ -n "$D" ]; then
-- 
2.7.4

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

[OE-core] [PATCH 14/33] matchbox-session-sato: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs gconf-native, mark the dependency and drop
the now unneeded DEPENDS.

Signed-off-by: Richard Purdie 
---
 meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb 
b/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
index 42c742f..d146e83 100644
--- a/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
+++ b/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
@@ -6,7 +6,6 @@ LICENSE = "GPLv2.0+"
 LIC_FILES_CHKSUM = 
"file://session;endline=3;md5=f8a5c5b9c279e52dc094d10e11c2be63"
 
 SECTION = "x11"
-DEPENDS = "gconf-native"
 RDEPENDS_${PN} = "formfactor matchbox-theme-sato matchbox-panel-2 
matchbox-desktop matchbox-session gconf"
 PR = "r30"
 
@@ -43,6 +42,7 @@ do_install() {
 chmod +x ${D}/${sysconfdir}/matchbox/session
 }
 
+PACKAGE_WRITE_DEPS += "gconf-native"
 pkg_postinst_${PN} () {
set_value() {
#type, name, value
-- 
2.7.4

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

[OE-core] [PATCH 16/33] useradd: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs shadow-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/classes/useradd.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/useradd.bbclass b/meta/classes/useradd.bbclass
index fd59969..82f4d52 100644
--- a/meta/classes/useradd.bbclass
+++ b/meta/classes/useradd.bbclass
@@ -4,6 +4,7 @@ inherit useradd_base
 # target sysroot, and shadow -native and -sysroot provide the utilities
 # and support files needed to add and modify user and group accounts
 DEPENDS_append_class-target = " base-files shadow-native shadow-sysroot shadow"
+PACKAGE_WRITE_DEPS += "shadow-native"
 
 # This preinstall function can be run in four different contexts:
 #
-- 
2.7.4

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

[OE-core] [PATCH 17/33] nss: Add PACKAGE_WRITE_DEPS

[Richard Purdie]

From: Jussi Kukkonen 

nss-native is required in postinst. It's also needed during
build so not removed from DEPENDS.

Signed-off-by: Jussi Kukkonen 
---
 meta/recipes-support/nss/nss_3.27.1.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-support/nss/nss_3.27.1.bb 
b/meta/recipes-support/nss/nss_3.27.1.bb
index 2a4f4d6..9aad5bd 100644
--- a/meta/recipes-support/nss/nss_3.27.1.bb
+++ b/meta/recipes-support/nss/nss_3.27.1.bb
@@ -207,6 +207,7 @@ do_install_append_class-target() {
 rm ./empty_password
 }
 
+PACKAGE_WRITE_DEPS += "nss-native"
 pkg_postinst_${PN} () {
 if [ -n "$D" ]; then
 for I in $D${libdir}/lib*.chk; do
-- 
2.7.4

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

[OE-core] [PATCH 15/33] ca-certificates: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs ca-certificates-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/recipes-support/ca-certificates/ca-certificates_20161130.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-support/ca-certificates/ca-certificates_20161130.bb 
b/meta/recipes-support/ca-certificates/ca-certificates_20161130.bb
index e0b2e41..e47bad1 100644
--- a/meta/recipes-support/ca-certificates/ca-certificates_20161130.bb
+++ b/meta/recipes-support/ca-certificates/ca-certificates_20161130.bb
@@ -11,6 +11,7 @@ LIC_FILES_CHKSUM = 
"file://debian/copyright;md5=e7358b9541ccf3029e9705ed8de57968
 DEPENDS = "ca-certificates-native"
 DEPENDS_class-native = "openssl-native"
 DEPENDS_class-nativesdk = "ca-certificates-native openssl-native"
+PACKAGE_WRITE_DEPS += "ca-certificates-native"
 
 SRCREV = "61b70a1007dc269d56881a0d480fc841daacc77c"
 
-- 
2.7.4

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

[OE-core] [PATCH 06/33] pixbufcache: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs qemu-native and gdk-pixbuf-native, mark these dependencies

Signed-off-by: Richard Purdie 
---
 meta/classes/pixbufcache.bbclass | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/classes/pixbufcache.bbclass b/meta/classes/pixbufcache.bbclass
index 63bf079..633a875 100644
--- a/meta/classes/pixbufcache.bbclass
+++ b/meta/classes/pixbufcache.bbclass
@@ -8,6 +8,8 @@ inherit qemu
 
 PIXBUF_PACKAGES ??= "${PN}"
 
+PACKAGE_WRITE_DEPS += "qemu-native gdk-pixbuf-native"
+
 pixbufcache_common() {
 if [ "x$D" != "x" ]; then
$INTERCEPT_DIR/postinst_intercept update_pixbuf_cache ${PKG} 
mlprefix=${MLPREFIX} libdir=${libdir} \
-- 
2.7.4

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

[OE-core] [PATCH 02/33] classes/package*: Add support for PACKAGE_WRITE_DEPS

[Richard Purdie]

Add a new variable to allow markup of postinstall (and preinst)
script dependnecies on native/cross tools.

If your postinstall can execute at rootfs creation time rather than on
target but depends on a native tool in order to execute, you need to
list that tool in PACKAGE_WRITE_DEPENDS.

Signed-off-by: Richard Purdie 
---
 meta/classes/package.bbclass | 8 
 meta/classes/package_deb.bbclass | 1 +
 meta/classes/package_ipk.bbclass | 1 +
 meta/classes/package_rpm.bbclass | 1 +
 4 files changed, 11 insertions(+)

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index 568b85c..0068a50 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -54,6 +54,14 @@ ALL_MULTILIB_PACKAGE_ARCHS = "${@all_multilib_tune_values(d, 
'PACKAGE_ARCHS')}"
 # rpm is used for the per-file dependency identification
 PACKAGE_DEPENDS += "rpm-native"
 
+
+# If your postinstall can execute at rootfs creation time rather than on
+# target but depends on a native/cross tool in order to execute, you need to
+# list that tool in PACKAGE_WRITE_DEPENDS. Target package dependencies belong
+# in the package dependencies as normal, this is just for native/cross support
+# tools at rootfs build time.
+PACKAGE_WRITE_DEPS ??= ""
+
 def legitimize_package_name(s):
 """
 Make sure package names are legitimate strings
diff --git a/meta/classes/package_deb.bbclass b/meta/classes/package_deb.bbclass
index 68eca61..6ce008f 100644
--- a/meta/classes/package_deb.bbclass
+++ b/meta/classes/package_deb.bbclass
@@ -351,6 +351,7 @@ python do_package_write_deb () {
 do_package_write_deb[dirs] = "${PKGWRITEDIRDEB}"
 do_package_write_deb[cleandirs] = "${PKGWRITEDIRDEB}"
 do_package_write_deb[umask] = "022"
+do_package_write_deb[depends] += 
"${@oe.utils.build_depends_string(d.getVar('PACKAGE_WRITE_DEPS'), 
'do_populate_sysroot')}"
 addtask package_write_deb after do_packagedata do_package
 
 
diff --git a/meta/classes/package_ipk.bbclass b/meta/classes/package_ipk.bbclass
index 7018a60..039b6ab 100644
--- a/meta/classes/package_ipk.bbclass
+++ b/meta/classes/package_ipk.bbclass
@@ -291,6 +291,7 @@ python do_package_write_ipk () {
 do_package_write_ipk[dirs] = "${PKGWRITEDIRIPK}"
 do_package_write_ipk[cleandirs] = "${PKGWRITEDIRIPK}"
 do_package_write_ipk[umask] = "022"
+do_package_write_ipk[depends] += 
"${@oe.utils.build_depends_string(d.getVar('PACKAGE_WRITE_DEPS'), 
'do_populate_sysroot')}"
 addtask package_write_ipk after do_packagedata do_package
 
 PACKAGEINDEXDEPS += "opkg-utils-native:do_populate_sysroot"
diff --git a/meta/classes/package_rpm.bbclass b/meta/classes/package_rpm.bbclass
index b9f049e..c978ec5 100644
--- a/meta/classes/package_rpm.bbclass
+++ b/meta/classes/package_rpm.bbclass
@@ -772,6 +772,7 @@ python do_package_write_rpm () {
 do_package_write_rpm[dirs] = "${PKGWRITEDIRRPM}"
 do_package_write_rpm[cleandirs] = "${PKGWRITEDIRRPM}"
 do_package_write_rpm[umask] = "022"
+do_package_write_rpm[depends] += 
"${@oe.utils.build_depends_string(d.getVar('PACKAGE_WRITE_DEPS'), 
'do_populate_sysroot')}"
 addtask package_write_rpm after do_packagedata do_package
 
 PACKAGEINDEXDEPS += "rpm-native:do_populate_sysroot"
-- 
2.7.4

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

[OE-core] [PATCH 10/33] initscripts: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs systemd-systemctl-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/recipes-core/initscripts/initscripts_1.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/initscripts/initscripts_1.0.bb 
b/meta/recipes-core/initscripts/initscripts_1.0.bb
index 8f110b0..2e4f7e4 100644
--- a/meta/recipes-core/initscripts/initscripts_1.0.bb
+++ b/meta/recipes-core/initscripts/initscripts_1.0.bb
@@ -44,7 +44,7 @@ KERNEL_VERSION = ""
 
 inherit update-alternatives
 DEPENDS_append = " update-rc.d-native"
-DEPENDS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
+PACKAGE_WRITE_DEPS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
 
 PACKAGES =+ "${PN}-functions"
 RDEPENDS_${PN} = "${PN}-functions \
-- 
2.7.4

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

[OE-core] [PATCH v2 1/3] image-container.bbclass: Add the "container" IMAGE_FSTYPES

[Randy Witt]

The "container" fstype does very little other than pick tar.bz2 as the
actual image type and disable installation of ROOTFS_BOOTSTRAP_INSTALL.

[YOCTO #9502]

Signed-off-by: Randy Witt 
---
 meta/classes/image-container.bbclass | 3 +++
 meta/classes/image.bbclass   | 3 +++
 meta/classes/image_types.bbclass | 1 +
 3 files changed, 7 insertions(+)
 create mode 100644 meta/classes/image-container.bbclass

diff --git a/meta/classes/image-container.bbclass 
b/meta/classes/image-container.bbclass
new file mode 100644
index 000..a10ee32
--- /dev/null
+++ b/meta/classes/image-container.bbclass
@@ -0,0 +1,3 @@
+ROOTFS_BOOTSTRAP_INSTALL = ""
+IMAGE_TYPES_MASKED += "container"
+IMAGE_TYPEDEP_container = "tar.bz2"
diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index 04fd5f9..b2c2a28 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -144,6 +144,9 @@ inherit ${IMAGE_TYPE_live}
 IMAGE_TYPE_vm = '${@bb.utils.contains_any("IMAGE_FSTYPES", ["vmdk", "vdi", 
"qcow2", "hdddirect"], "image-vm", "", d)}'
 inherit ${IMAGE_TYPE_vm}
 
+IMAGE_TYPE_container = '${@bb.utils.contains("IMAGE_FSTYPES", "container", 
"image-container", "", d)}'
+inherit ${IMAGE_TYPE_container}
+
 def build_uboot(d):
 if 'u-boot' in (d.getVar('IMAGE_FSTYPES') or ''):
 return "image_types_uboot"
diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
index 7748dee..727a87e 100644
--- a/meta/classes/image_types.bbclass
+++ b/meta/classes/image_types.bbclass
@@ -328,6 +328,7 @@ IMAGE_TYPES = " \
 hdddirect \
 elf \
 wic wic.gz wic.bz2 wic.lzma \
+container \
 "
 
 # Compression is a special case of conversion. The old variable
-- 
2.9.3

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

[OE-core] [PATCH 07/33] systemd: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs systemd-systemctl-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/classes/systemd.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/systemd.bbclass b/meta/classes/systemd.bbclass
index 0b1c228..1ce0b94 100644
--- a/meta/classes/systemd.bbclass
+++ b/meta/classes/systemd.bbclass
@@ -17,6 +17,7 @@ python __anonymous() {
 # files.
 if bb.utils.contains('DISTRO_FEATURES', 'systemd', True, False, d):
 d.appendVar("DEPENDS", " systemd-systemctl-native")
+d.appendVar("PACKAGE_WRITE_DEPS", " systemd-systemctl-native")
 if not bb.utils.contains('DISTRO_FEATURES', 'sysvinit', True, False, 
d):
 d.setVar("INHIBIT_UPDATERCD_BBCLASS", "1")
 }
-- 
2.7.4

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

[OE-core] [PATCH 01/33] lib/oe/utils: Add build_depends_string function

[Richard Purdie]

This is useful when manipulating depends strings for task [depends]
flags and is slightly easier to parse than some inline python.

Signed-off-by: Richard Purdie 
---
 meta/lib/oe/utils.py | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/lib/oe/utils.py b/meta/lib/oe/utils.py
index 76806b5..330a5ff 100644
--- a/meta/lib/oe/utils.py
+++ b/meta/lib/oe/utils.py
@@ -97,6 +97,10 @@ def param_bool(cfg, field, dflt = None):
 return False
 raise ValueError("invalid value for boolean parameter '%s': '%s'" % 
(field, value))
 
+def build_depends_string(depends, task):
+"""Append a taskname to a string of dependencies as used by the [depends] 
flag"""
+return " ".join(dep + ":" + task for dep in depends.split())
+
 def inherits(d, *classes):
 """Return True if the metadata inherits any of the specified classes"""
 return any(bb.data.inherits_class(cls, d) for cls in classes)
-- 
2.7.4

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

[OE-core] [PATCH 09/33] v86d: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs systemd-systemctl-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/recipes-bsp/v86d/v86d_0.1.10.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-bsp/v86d/v86d_0.1.10.bb 
b/meta/recipes-bsp/v86d/v86d_0.1.10.bb
index 1046d63..eab466d 100644
--- a/meta/recipes-bsp/v86d/v86d_0.1.10.bb
+++ b/meta/recipes-bsp/v86d/v86d_0.1.10.bb
@@ -60,7 +60,7 @@ python __anonymous() {
 
 inherit update-rc.d
 
-DEPENDS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
+PACKAGE_WRITE_DEPS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
 pkg_postinst_${PN} () {
if ${@bb.utils.contains('DISTRO_FEATURES','systemd 
sysvinit','true','false',d)}; then
if [ -n "$D" ]; then
-- 
2.7.4

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

[OE-core] [PATCH 13/33] modutils-initsripts: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs systemd-systemctl-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb 
b/meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb
index db670cf..0f3df55 100644
--- a/meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb
+++ b/meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb
@@ -22,7 +22,7 @@ do_install () {
install -m 0755 ${WORKDIR}/modutils.sh ${D}${sysconfdir}/init.d/
 }
 
-DEPENDS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
+PACKAGE_WRITE_DEPS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
 pkg_postinst_${PN} () {
if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; 
then
if [ -n "$D" ]; then
-- 
2.7.4

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

[OE-core] [PATCH 11/33] psplash: Add PACKAGE_WRITE_DEPS for postinst

[Richard Purdie]

The postinstall needs systemd-systemctl-native, mark the dependency

Signed-off-by: Richard Purdie 
---
 meta/recipes-core/psplash/psplash_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/psplash/psplash_git.bb 
b/meta/recipes-core/psplash/psplash_git.bb
index b0d6bb4..44297e1 100644
--- a/meta/recipes-core/psplash/psplash_git.bb
+++ b/meta/recipes-core/psplash/psplash_git.bb
@@ -110,7 +110,7 @@ FILES_${PN} += "/mnt/.psplash"
 INITSCRIPT_NAME = "psplash.sh"
 INITSCRIPT_PARAMS = "start 0 S . stop 20 0 1 6 ."
 
-DEPENDS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
+PACKAGE_WRITE_DEPS_append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd-systemctl-native','',d)}"
 pkg_postinst_${PN} () {
if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; 
then
if [ -n "$D" ]; then
-- 
2.7.4

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

[OE-core] [PATCH v2 2/3] image-container.bbclass: Error if not using linux-dummy

[Randy Witt]

In the typical case of a container, the user doesn't particularly care
about the kernel version used. Since using linux-dummy prevents building
a kernel in the most common case, alert the user if they choose a
container image, but don't also specify linux-dummy.

In the cases the user actually does want to build against particular
kernel headers, etc, they can override by setting
IMAGE_CONTAINER_NO_DUMMY = "1".

[YOCTO #9502]

Signed-off-by: Randy Witt 
---
 meta/classes/image-container.bbclass | 18 ++
 1 file changed, 18 insertions(+)

diff --git a/meta/classes/image-container.bbclass 
b/meta/classes/image-container.bbclass
index a10ee32..f002858 100644
--- a/meta/classes/image-container.bbclass
+++ b/meta/classes/image-container.bbclass
@@ -1,3 +1,21 @@
 ROOTFS_BOOTSTRAP_INSTALL = ""
 IMAGE_TYPES_MASKED += "container"
 IMAGE_TYPEDEP_container = "tar.bz2"
+
+python __anonymous() {
+if "container" in d.getVar("IMAGE_FSTYPES") and \
+   d.getVar("IMAGE_CONTAINER_NO_DUMMY") != "1" and \
+   "linux-dummy" not in d.getVar("PREFERRED_PROVIDER_virtual/kernel"):
+msg = '"container" is in IMAGE_FSTYPES, but ' \
+  'PREFERRED_PROVIDER_virtual/kernel is not "linux-dummy". ' \
+  'Unless a particular kernel is needed, using linux-dummy will ' \
+  'prevent a kernel from being built, which can reduce ' \
+  'build times. If you don\'t want to use "linux-dummy", set ' \
+  '"IMAGE_CONTAINER_NO_DUMMY" to "1".'
+
+# Raising skip recipe was Paul's clever idea. It causes the error to
+# only be shown for the recipes actually requested to build, rather
+# than bb.fatal which would appear for all recipes inheriting the
+# class.
+raise bb.parse.SkipRecipe(msg)
+}
-- 
2.9.3

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

[OE-core] [PATCH v2 0/3] Add initial container IMAGE_FSTYPES support

[Randy Witt]

Updated with Ross' requests.

The following changes since commit a304b94eae99ade6d99800eeaf588c48d01506f6:

  oeqa/selftest/devtool: rewrite modify testcase (2017-01-19 22:47:22 +)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib rewitt/contwip
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=rewitt/contwip

Randy Witt (3):
  image-container.bbclass: Add the "container" IMAGE_FSTYPES
  image-container.bbclass: Error if not using linux-dummy
  selftest/containerimage.py: Add container IMAGE_FSTYPES test

 .../container-image/container-image-testpkg.bb |  8 +++
 .../container-image/container-test-image.bb|  8 +++
 meta/classes/image-container.bbclass   | 21 ++
 meta/classes/image.bbclass |  3 +
 meta/classes/image_types.bbclass   |  1 +
 meta/lib/oeqa/selftest/containerimage.py   | 82 ++
 6 files changed, 123 insertions(+)
 create mode 100644 
meta-selftest/recipes-test/container-image/container-image-testpkg.bb
 create mode 100644 
meta-selftest/recipes-test/container-image/container-test-image.bb
 create mode 100644 meta/classes/image-container.bbclass
 create mode 100644 meta/lib/oeqa/selftest/containerimage.py

-- 
2.9.3

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

[OE-core] [PATCH v2 3/3] selftest/containerimage.py: Add container IMAGE_FSTYPES test

[Randy Witt]

This test checks to make sure only the files expected exist in a
container image. Currently only ROOTFS_BOOTSTRAP_INSTALL, gets added to
all images without the user specifying it.

But this test should help if a developer in the future ever silently
adds more than just ROOTFS_BOOTSTRAP_INSTALL, and that the developer can
make sure it also gets removed from a container image.

[YOCTO #9502]

Signed-off-by: Randy Witt 
---
 .../container-image/container-image-testpkg.bb |  8 +++
 .../container-image/container-test-image.bb|  8 +++
 meta/lib/oeqa/selftest/containerimage.py   | 82 ++
 3 files changed, 98 insertions(+)
 create mode 100644 
meta-selftest/recipes-test/container-image/container-image-testpkg.bb
 create mode 100644 
meta-selftest/recipes-test/container-image/container-test-image.bb
 create mode 100644 meta/lib/oeqa/selftest/containerimage.py

diff --git 
a/meta-selftest/recipes-test/container-image/container-image-testpkg.bb 
b/meta-selftest/recipes-test/container-image/container-image-testpkg.bb
new file mode 100644
index 000..f8dd229
--- /dev/null
+++ b/meta-selftest/recipes-test/container-image/container-image-testpkg.bb
@@ -0,0 +1,8 @@
+LICENSE = "MIT"
+
+INHIBIT_DEFAULT_DEPS = "1"
+
+do_install_append() {
+install -d ${D}${bindir}
+touch ${D}${bindir}/theapp
+}
diff --git a/meta-selftest/recipes-test/container-image/container-test-image.bb 
b/meta-selftest/recipes-test/container-image/container-test-image.bb
new file mode 100644
index 000..d5f939c
--- /dev/null
+++ b/meta-selftest/recipes-test/container-image/container-test-image.bb
@@ -0,0 +1,8 @@
+IMAGE_INSTALL += "container-image-testpkg"
+
+LICENSE = "MIT"
+
+IMAGE_FSTYPES = "container"
+IMAGE_LINGUAS = ""
+
+inherit core-image
diff --git a/meta/lib/oeqa/selftest/containerimage.py 
b/meta/lib/oeqa/selftest/containerimage.py
new file mode 100644
index 000..6a85222
--- /dev/null
+++ b/meta/lib/oeqa/selftest/containerimage.py
@@ -0,0 +1,82 @@
+import os
+
+from oeqa.selftest.base import oeSelfTest
+from oeqa.utils.commands import bitbake, get_bb_vars, runCmd
+
+# This test builds an image with using the "container" IMAGE_FSTYPE, and
+# ensures that then files in the image are only the ones expected.
+#
+# The only package added to the image is container_image_testpkg, which
+# contains one file. However, due to some other things not cleaning up during
+# rootfs creation, there is some cruft. Ideally bugs will be filed and the
+# cruft removed, but for now we whitelist some known set.
+#
+# Also for performance reasons we're only checking the cruft when using ipk.
+# When using deb, and rpm it is a bit different and we could test all
+# of them, but this test is more to catch if other packages get added by
+# default other than what is in ROOTFS_BOOTSTRAP_INSTALL.
+#
+class ContainerImageTests(oeSelfTest):
+
+# Verify that when specifying a IMAGE_TYPEDEP_ of the form "foo.bar" that
+# the conversion type bar gets added as a dep as well
+def test_expected_files(self):
+
+def get_each_path_part(path):
+if path:
+part = [ '.' + path + '/' ]
+result = get_each_path_part(path.rsplit('/', 1)[0])
+if result:
+return part + result
+else:
+return part
+else:
+return None
+
+bbvars = get_bb_vars(['bindir', 'sysconfdir', 'localstatedir',
+  'DEPLOY_DIR_IMAGE'])
+expected_files = [
+'./',
+'.{bindir}/theapp',
+'.{sysconfdir}/default/',
+'.{sysconfdir}/default/postinst',
+'.{sysconfdir}/ld.so.cache',
+'.{sysconfdir}/timestamp',
+'.{sysconfdir}/version',
+'./run/',
+'.{localstatedir}/cache/',
+'.{localstatedir}/cache/ldconfig/',
+'.{localstatedir}/cache/ldconfig/aux-cache',
+'.{localstatedir}/cache/opkg/',
+'.{localstatedir}/lib/',
+'.{localstatedir}/lib/opkg/'
+]
+
+expected_files = [ x.format(bindir=bbvars['bindir'],
+sysconfdir=bbvars['sysconfdir'],
+localstatedir=bbvars['localstatedir'])
+for x in expected_files ]
+
+# Since tar lists all directories individually, make sure each element
+# from bindir, sysconfdir, etc is added
+expected_files += get_each_path_part(bbvars['bindir'])
+expected_files += get_each_path_part(bbvars['sysconfdir'])
+expected_files += get_each_path_part(bbvars['localstatedir'])
+
+expected_files = sorted(expected_files)
+
+
+self.write_config('PREFERRED_PROVIDER_virtual/kernel 

Re: [OE-core] [PATCH 3/3] selftest/containerimage.py: Add container IMAGE_FSTYPES test

[Randy Witt]

On 01/19/2017 04:01 AM, Burton, Ross wrote:

On 18 January 2017 at 18:31, Randy Witt 
wrote:


+expected_files = [
+'./',
+'./bin/',
+'./bin/theapp',
+'./etc/',
+'./etc/default/',
+'./etc/default/postinst',
+'./etc/ld.so.cache',
+'./etc/timestamp',
+'./etc/version',
+'./run/',
+'./var/',
+'./var/cache/',
+'./var/cache/ldconfig/',
+'./var/cache/ldconfig/aux-cache',
+'./var/cache/opkg/',
+'./var/lib/',
+'./var/lib/opkg/'
+]

These should use ${bindir} etc so they're resilient to distros changing

the paths (please use get_bb_vars() to only invoke bitbake once).

+image = os.path.join(self.builddir, "tmp/deploy/images/qemux86/

container-test-image-qemux86.tar.bz2")



Don't hardcode this path, use ${DEPLOY_DIR_IMAGE} (fetch it with the paths
above for speed) which will get you to tmp/deploy/images/[machine].  Again,
these paths can change (poky sets TMPDIR to tmp, nodistro uses
tmp-${TCLIBC}).

Ross



I just sent a v2, but screwed up the subject. I'll resend the whole series again 
even though only 3/3 changed, to hopefully make it easier on you.


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

Re: [OE-core] [PATCHv2] qemu: Upgrade to 2.8.0

[Aníbal Limón]

On 01/16/2017 03:42 PM, Aníbal Limón wrote:
> 
> 
> On 01/16/2017 10:47 AM, Burton, Ross wrote:
>> On 16 January 2017 at 16:43, Aníbal Limón 
>> wrote:
>>
>>> It's weird, i'll review it, but i don't imagine what is failing
>>> gobject-introspection because a qemu upgrade.
>>>
>>
>> The g-i build runs largely inside a qemu-user instance (the crash is inside
>> qemu).
> 
> Hi,
> 
> I got the stack trace, it's an issue in the PPC TCG some patches in
> upstream looks like could be cause the problem [1][2][3].
> 
> 
> (gdb) bt
> #0  0x0086e204 in static_code_gen_buffer ()
> #1  0x0040e922 in cpu_tb_exec (itb=,
> itb=, cpu=0x29864d0) at
> /home/alimon/repos/poky/build-ppc/tmp/work/x86_64-linux/qemu-native/2.8.0-r0/qemu-2.8.0/cpu-exec.c:164
> #2  cpu_loop_exec_tb (sc=, tb_exit=,
> last_tb=, tb=, cpu=0x29864d0) at
> /home/alimon/repos/poky/build-ppc/tmp/work/x86_64-linux/qemu-native/2.8.0-r0/qemu-2.8.0/cpu-exec.c:544
> #3  cpu_exec (cpu=cpu@entry=0x29864d0) at
> /home/alimon/repos/poky/build-ppc/tmp/work/x86_64-linux/qemu-native/2.8.0-r0/qemu-2.8.0/cpu-exec.c:638
> #4  0x00445fba in cpu_loop (env=env@entry=0x298e750) at
> /home/alimon/repos/poky/build-ppc/tmp/work/x86_64-linux/qemu-native/2.8.0-r0/qemu-2.8.0/linux-user/main.c:1359
> #5  0x00448a95 in clone_func (arg=0x7fffa910) at
> /home/alimon/repos/poky/build-ppc/tmp/work/x86_64-linux/qemu-native/2.8.0-r0/qemu-2.8.0/linux-user/syscall.c:6090
> #6  0x76a750a4 in start_thread (arg=0x77f4d700) at
> pthread_create.c:309
> #7  0x767aa62d in clone () at
> ../sysdeps/unix/sysv/linux/x86_64/clone.S:111
> 
> (gdb) info threds
>   Id   Target Id Frame
>   1Thread 0x77fd0780 (LWP 25006) "qemu-ppc"
> pthread_cond_wait@@GLIBC_2.3.2 () at
> ../nptl/sysdeps/unix/sysv/linux/x86_64/pthread_cond_wait.S:185
>   2Thread 0x7647e700 (LWP 25010) "qemu-ppc" syscall () at
> ../sysdeps/unix/sysv/linux/x86_64/syscall.S:38
> * 3Thread 0x77f4d700 (LWP 25011) "qemu-ppc" 0x0086e204
> in static_code_gen_buffer ()
> 
> I'll send an email to the qemu mailing list.
> 
> Cheers,
>   alimon
> 
> [1]
> http://git.qemu.org/?p=qemu.git;a=commit;h=bd6fefe71cec5a0c7d2be4ac96307f25db56abf9
> [2]
> http://git.qemu.org/?p=qemu.git;a=commit;h=ba4a8df83f474be1ba02510f960e5224f29d229c
> [3]
> http://git.qemu.org/?p=qemu.git;a=commit;h=4f5d326046fa522010c0a66f1bcaac9068253085

I can't get a response from qemu people so i will send a minor upgrade
2.7.1 while i debug the issue.

alimon

> 
>>
>> Ross
>>
> 
> 
> 



signature.asc
Description: OpenPGP digital signature
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] qemu: Upgrade to 2.7.1

[Aníbal Limón]

Minor upgrade contains fixes from 2.7.0.

Removed patches (already in upstream):

- 0001-pci-assign-sync-MSI-MSI-X-cap-and-table-with-PCIDevi.patch
- 0001-virtio-zero-vq-inuse-in-virtio_reset.patch
- 0002-fix-CVE-2016-7423.patch

Signed-off-by: Aníbal Limón 
---
 meta/recipes-devtools/qemu/qemu.inc|  2 -
 ...sync-MSI-MSI-X-cap-and-table-with-PCIDevi.patch | 71 --
 ...0001-virtio-zero-vq-inuse-in-virtio_reset.patch | 57 -
 .../qemu/qemu/0002-fix-CVE-2016-7423.patch | 45 --
 .../qemu/{qemu_2.7.0.bb => qemu_2.7.1.bb}  |  7 +--
 5 files changed, 2 insertions(+), 180 deletions(-)
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0001-pci-assign-sync-MSI-MSI-X-cap-and-table-with-PCIDevi.patch
 delete mode 100644 
meta/recipes-devtools/qemu/qemu/0001-virtio-zero-vq-inuse-in-virtio_reset.patch
 delete mode 100644 meta/recipes-devtools/qemu/qemu/0002-fix-CVE-2016-7423.patch
 rename meta/recipes-devtools/qemu/{qemu_2.7.0.bb => qemu_2.7.1.bb} (72%)

diff --git a/meta/recipes-devtools/qemu/qemu.inc 
b/meta/recipes-devtools/qemu/qemu.inc
index b46e7f7..ac5fcac 100644
--- a/meta/recipes-devtools/qemu/qemu.inc
+++ b/meta/recipes-devtools/qemu/qemu.inc
@@ -9,8 +9,6 @@ require qemu-targets.inc
 inherit autotools ptest
 BBCLASSEXTEND = "native nativesdk"
 
-PR = "r1"
-
 # QEMU_TARGETS is overridable variable
 QEMU_TARGETS ?= "arm aarch64 i386 mips mipsel mips64 mips64el ppc sh4 x86_64"
 
diff --git 
a/meta/recipes-devtools/qemu/qemu/0001-pci-assign-sync-MSI-MSI-X-cap-and-table-with-PCIDevi.patch
 
b/meta/recipes-devtools/qemu/qemu/0001-pci-assign-sync-MSI-MSI-X-cap-and-table-with-PCIDevi.patch
deleted file mode 100644
index 03472dd..000
--- 
a/meta/recipes-devtools/qemu/qemu/0001-pci-assign-sync-MSI-MSI-X-cap-and-table-with-PCIDevi.patch
+++ /dev/null
@@ -1,71 +0,0 @@
-From 6baa545df93253fced4fc0d52b14b98447e00473 Mon Sep 17 00:00:00 2001
-From: Peter Xu 
-Date: Mon, 28 Nov 2016 15:02:44 +0800
-Subject: [PATCH] pci-assign: sync MSI/MSI-X cap and table with PCIDevice
-
-Since commit e1d4fb2d ("kvm-irqchip: x86: add msi route notify fn"),
-kvm_irqchip_add_msi_route() starts to use pci_get_msi_message() to fetch
-MSI info. This requires that we setup MSI related fields in PCIDevice.
-For most devices, that won't be a problem, as long as we are using
-general interfaces like msi_init()/msix_init().
-
-However, for pci-assign devices, MSI/MSI-X is treated differently - PCI
-assign devices are maintaining its own MSI table and cap information in
-AssignedDevice struct. however that's not synced up with PCIDevice's
-fields. That will leads to pci_get_msi_message() failed to find correct
-MSI capability, even with an NULL msix_table.
-
-A quick fix is to sync up the two places: both the capability bits and
-table address for MSI/MSI-X.
-
-Upstream-Status: Backport 
[https://lists.gnu.org/archive/html/qemu-devel/2016-11/msg04649.html]
-
-Reported-by: Changlimin 
-Tested-by: Changlimin 
-Cc: address@hidden
-Fixes: e1d4fb2d ("kvm-irqchip: x86: add msi route notify fn")
-Signed-off-by: Peter Xu 
-Signed-off-by: He Zhe 

- hw/i386/kvm/pci-assign.c | 4 
- 1 file changed, 4 insertions(+)
-
-diff --git a/hw/i386/kvm/pci-assign.c b/hw/i386/kvm/pci-assign.c
-index 8238fbc..87dcbdd 100644
 a/hw/i386/kvm/pci-assign.c
-+++ b/hw/i386/kvm/pci-assign.c
-@@ -1251,6 +1251,7 @@ static int assigned_device_pci_cap_init(PCIDevice 
*pci_dev, Error **errp)
- error_propagate(errp, local_err);
- return -ENOTSUP;
- }
-+dev->dev.cap_present |= QEMU_PCI_CAP_MSI;
- dev->cap.available |= ASSIGNED_DEVICE_CAP_MSI;
- /* Only 32-bit/no-mask currently supported */
- ret = pci_add_capability2(pci_dev, PCI_CAP_ID_MSI, pos, 10,
-@@ -1285,6 +1286,7 @@ static int assigned_device_pci_cap_init(PCIDevice 
*pci_dev, Error **errp)
- error_propagate(errp, local_err);
- return -ENOTSUP;
- }
-+dev->dev.cap_present |= QEMU_PCI_CAP_MSIX;
- dev->cap.available |= ASSIGNED_DEVICE_CAP_MSIX;
- ret = pci_add_capability2(pci_dev, PCI_CAP_ID_MSIX, pos, 12,
-   _err);
-@@ -1648,6 +1650,7 @@ static void 
assigned_dev_register_msix_mmio(AssignedDevice *dev, Error **errp)
- dev->msix_table = NULL;
- return;
- }
-+dev->dev.msix_table = (uint8_t *)dev->msix_table;
- 
- assigned_dev_msix_reset(dev);
- 
-@@ -1665,6 +1668,7 @@ static void 
assigned_dev_unregister_msix_mmio(AssignedDevice *dev)
- error_report("error unmapping msix_table! %s", strerror(errno));
- }
- dev->msix_table = NULL;
-+dev->dev.msix_table = NULL;
- }
- 
- static const VMStateDescription vmstate_assigned_device = {
--- 
-2.8.3
-
diff --git 

[OE-core] [PATCH] selftest/containerimage.py: Add container IMAGE_FSTYPES test

[Randy Witt]

This test checks to make sure only the files expected exist in a
container image. Currently only ROOTFS_BOOTSTRAP_INSTALL, gets added to
all images without the user specifying it.

But this test should help if a developer in the future ever silently
adds more than just ROOTFS_BOOTSTRAP_INSTALL, and that the developer can
make sure it also gets removed from a container image.

[YOCTO #9502]

Signed-off-by: Randy Witt 
---
 .../container-image/container-image-testpkg.bb |  8 +++
 .../container-image/container-test-image.bb|  8 +++
 meta/lib/oeqa/selftest/containerimage.py   | 82 ++
 3 files changed, 98 insertions(+)
 create mode 100644 
meta-selftest/recipes-test/container-image/container-image-testpkg.bb
 create mode 100644 
meta-selftest/recipes-test/container-image/container-test-image.bb
 create mode 100644 meta/lib/oeqa/selftest/containerimage.py

diff --git 
a/meta-selftest/recipes-test/container-image/container-image-testpkg.bb 
b/meta-selftest/recipes-test/container-image/container-image-testpkg.bb
new file mode 100644
index 000..f8dd229
--- /dev/null
+++ b/meta-selftest/recipes-test/container-image/container-image-testpkg.bb
@@ -0,0 +1,8 @@
+LICENSE = "MIT"
+
+INHIBIT_DEFAULT_DEPS = "1"
+
+do_install_append() {
+install -d ${D}${bindir}
+touch ${D}${bindir}/theapp
+}
diff --git a/meta-selftest/recipes-test/container-image/container-test-image.bb 
b/meta-selftest/recipes-test/container-image/container-test-image.bb
new file mode 100644
index 000..d5f939c
--- /dev/null
+++ b/meta-selftest/recipes-test/container-image/container-test-image.bb
@@ -0,0 +1,8 @@
+IMAGE_INSTALL += "container-image-testpkg"
+
+LICENSE = "MIT"
+
+IMAGE_FSTYPES = "container"
+IMAGE_LINGUAS = ""
+
+inherit core-image
diff --git a/meta/lib/oeqa/selftest/containerimage.py 
b/meta/lib/oeqa/selftest/containerimage.py
new file mode 100644
index 000..6a85222
--- /dev/null
+++ b/meta/lib/oeqa/selftest/containerimage.py
@@ -0,0 +1,82 @@
+import os
+
+from oeqa.selftest.base import oeSelfTest
+from oeqa.utils.commands import bitbake, get_bb_vars, runCmd
+
+# This test builds an image with using the "container" IMAGE_FSTYPE, and
+# ensures that then files in the image are only the ones expected.
+#
+# The only package added to the image is container_image_testpkg, which
+# contains one file. However, due to some other things not cleaning up during
+# rootfs creation, there is some cruft. Ideally bugs will be filed and the
+# cruft removed, but for now we whitelist some known set.
+#
+# Also for performance reasons we're only checking the cruft when using ipk.
+# When using deb, and rpm it is a bit different and we could test all
+# of them, but this test is more to catch if other packages get added by
+# default other than what is in ROOTFS_BOOTSTRAP_INSTALL.
+#
+class ContainerImageTests(oeSelfTest):
+
+# Verify that when specifying a IMAGE_TYPEDEP_ of the form "foo.bar" that
+# the conversion type bar gets added as a dep as well
+def test_expected_files(self):
+
+def get_each_path_part(path):
+if path:
+part = [ '.' + path + '/' ]
+result = get_each_path_part(path.rsplit('/', 1)[0])
+if result:
+return part + result
+else:
+return part
+else:
+return None
+
+bbvars = get_bb_vars(['bindir', 'sysconfdir', 'localstatedir',
+  'DEPLOY_DIR_IMAGE'])
+expected_files = [
+'./',
+'.{bindir}/theapp',
+'.{sysconfdir}/default/',
+'.{sysconfdir}/default/postinst',
+'.{sysconfdir}/ld.so.cache',
+'.{sysconfdir}/timestamp',
+'.{sysconfdir}/version',
+'./run/',
+'.{localstatedir}/cache/',
+'.{localstatedir}/cache/ldconfig/',
+'.{localstatedir}/cache/ldconfig/aux-cache',
+'.{localstatedir}/cache/opkg/',
+'.{localstatedir}/lib/',
+'.{localstatedir}/lib/opkg/'
+]
+
+expected_files = [ x.format(bindir=bbvars['bindir'],
+sysconfdir=bbvars['sysconfdir'],
+localstatedir=bbvars['localstatedir'])
+for x in expected_files ]
+
+# Since tar lists all directories individually, make sure each element
+# from bindir, sysconfdir, etc is added
+expected_files += get_each_path_part(bbvars['bindir'])
+expected_files += get_each_path_part(bbvars['sysconfdir'])
+expected_files += get_each_path_part(bbvars['localstatedir'])
+
+expected_files = sorted(expected_files)
+
+
+self.write_config('PREFERRED_PROVIDER_virtual/kernel 

Re: [OE-core] [PATCH 2/2] Revert "kernel: Modify kernel modules installation path."

[Jason Wessel]

The issue here is that in a multilib build the ${base_libdir} is resolving to 
/lib64 which completely breaks some tools that fully expect /lib/modules to be 
a fixed location regardless if the target is 32/64 or something else.

It may be fixable by using, ${nonarch_base_libdir}, but it wasn't quite that 
simple because it didn't fully work for the firmware loader so for now it would 
be best to just revert the two patches and keep oe-core working.

Cheers,
Jason.


On 01/19/2017 03:14 PM, Jason Wessel wrote:

This reverts commit 0ccb2efe9837e2915c093341a662dffc1df00866.

The OVS fails to function and the kernel modules cannot be found by
any of the kernel tools such as depmod because they are installed
into the wrong directory in multilib 64bit/32bit bulids.

Signed-off-by: Jason Wessel 
---
  meta/classes/kernel-module-split.bbclass |  2 +-
  meta/classes/kernel.bbclass  | 12 ++--
  2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/meta/classes/kernel-module-split.bbclass 
b/meta/classes/kernel-module-split.bbclass
index 0e4e110f0c..efe1b42656 100644
--- a/meta/classes/kernel-module-split.bbclass
+++ b/meta/classes/kernel-module-split.bbclass
@@ -127,7 +127,7 @@ python split_kernel_module_packages () {
  postinst = d.getVar('pkg_postinst_modules')
  postrm = d.getVar('pkg_postrm_modules')
  
-modules = do_split_packages(d, root='${base_libdir}/modules', file_regex=module_regex, output_pattern=module_pattern, description='%s kernel module', postinst=postinst, postrm=postrm, recursive=True, hook=frob_metadata, extra_depends='kernel-%s' % (d.getVar("KERNEL_VERSION")))

+modules = do_split_packages(d, root='/lib/modules', file_regex=module_regex, 
output_pattern=module_pattern, description='%s kernel module', postinst=postinst, 
postrm=postrm, recursive=True, hook=frob_metadata, extra_depends='kernel-%s' % 
(d.getVar("KERNEL_VERSION")))
  if modules:
  metapkg = d.getVar('KERNEL_MODULES_META_PACKAGE')
  d.appendVar('RDEPENDS_' + metapkg, ' '+' '.join(modules))
diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 74517e8d92..3630042dbd 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -296,11 +296,11 @@ kernel_do_install() {
#
unset CFLAGS CPPFLAGS CXXFLAGS LDFLAGS MACHINE
if (grep -q -i -e '^CONFIG_MODULES=y$' .config); then
-   oe_runmake DEPMOD=echo 
MODLIB=${D}${base_libdir}/modules/${KERNEL_VERSION} 
INSTALL_FW_PATH=${D}${base_libdir}/firmware modules_install
-   rm "${D}${base_libdir}/modules/${KERNEL_VERSION}/build"
-   rm "${D}${base_libdir}/modules/${KERNEL_VERSION}/source"
+   oe_runmake DEPMOD=echo INSTALL_MOD_PATH="${D}" modules_install
+   rm "${D}/lib/modules/${KERNEL_VERSION}/build"
+   rm "${D}/lib/modules/${KERNEL_VERSION}/source"
# If the kernel/ directory is empty remove it to prevent QA 
issues
-   rmdir --ignore-fail-on-non-empty 
"${D}${base_libdir}/modules/${KERNEL_VERSION}/kernel"
+   rmdir --ignore-fail-on-non-empty 
"${D}/lib/modules/${KERNEL_VERSION}/kernel"
else
bbnote "no modules to install"
fi
@@ -478,7 +478,7 @@ EXPORT_FUNCTIONS do_compile do_install do_configure
  # kernel-image becomes kernel-image-${KERNEL_VERSION}
  PACKAGES = "kernel kernel-base kernel-vmlinux kernel-image kernel-dev 
kernel-modules"
  FILES_${PN} = ""
-FILES_kernel-base = "${base_libdir}/modules/${KERNEL_VERSION}/modules.order 
${base_libdir}/modules/${KERNEL_VERSION}/modules.builtin"
+FILES_kernel-base = "/lib/modules/${KERNEL_VERSION}/modules.order 
/lib/modules/${KERNEL_VERSION}/modules.builtin"
  FILES_kernel-image = ""
  FILES_kernel-dev = "/boot/System.map* /boot/Module.symvers* /boot/config* 
${KERNEL_SRC_PATH} /lib/modules/${KERNEL_VERSION}/build"
  FILES_kernel-vmlinux = "/boot/vmlinux-${KERNEL_VERSION_NAME}"
@@ -511,7 +511,7 @@ pkg_postinst_kernel-base () {
  PACKAGESPLITFUNCS_prepend = "split_kernel_packages "
  
  python split_kernel_packages () {

-do_split_packages(d, root='${base_libdir}/firmware', 
file_regex='^(.*)\.(bin|fw|cis|csp|dsp)$', output_pattern='kernel-firmware-%s', 
description='Firmware for %s', recursive=True, extra_depends='')
+do_split_packages(d, root='/lib/firmware', 
file_regex='^(.*)\.(bin|fw|cis|csp|dsp)$', output_pattern='kernel-firmware-%s', 
description='Firmware for %s', recursive=True, extra_depends='')
  }
  
  # Many scripts want to look in arch/$arch/boot for the bootable



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

[OE-core] [PATCH 2/2] Revert "kernel: Modify kernel modules installation path."

[Jason Wessel]

This reverts commit 0ccb2efe9837e2915c093341a662dffc1df00866.

The OVS fails to function and the kernel modules cannot be found by
any of the kernel tools such as depmod because they are installed
into the wrong directory in multilib 64bit/32bit bulids.

Signed-off-by: Jason Wessel 
---
 meta/classes/kernel-module-split.bbclass |  2 +-
 meta/classes/kernel.bbclass  | 12 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/meta/classes/kernel-module-split.bbclass 
b/meta/classes/kernel-module-split.bbclass
index 0e4e110f0c..efe1b42656 100644
--- a/meta/classes/kernel-module-split.bbclass
+++ b/meta/classes/kernel-module-split.bbclass
@@ -127,7 +127,7 @@ python split_kernel_module_packages () {
 postinst = d.getVar('pkg_postinst_modules')
 postrm = d.getVar('pkg_postrm_modules')
 
-modules = do_split_packages(d, root='${base_libdir}/modules', 
file_regex=module_regex, output_pattern=module_pattern, description='%s kernel 
module', postinst=postinst, postrm=postrm, recursive=True, hook=frob_metadata, 
extra_depends='kernel-%s' % (d.getVar("KERNEL_VERSION")))
+modules = do_split_packages(d, root='/lib/modules', 
file_regex=module_regex, output_pattern=module_pattern, description='%s kernel 
module', postinst=postinst, postrm=postrm, recursive=True, hook=frob_metadata, 
extra_depends='kernel-%s' % (d.getVar("KERNEL_VERSION")))
 if modules:
 metapkg = d.getVar('KERNEL_MODULES_META_PACKAGE')
 d.appendVar('RDEPENDS_' + metapkg, ' '+' '.join(modules))
diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 74517e8d92..3630042dbd 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -296,11 +296,11 @@ kernel_do_install() {
#
unset CFLAGS CPPFLAGS CXXFLAGS LDFLAGS MACHINE
if (grep -q -i -e '^CONFIG_MODULES=y$' .config); then
-   oe_runmake DEPMOD=echo 
MODLIB=${D}${base_libdir}/modules/${KERNEL_VERSION} 
INSTALL_FW_PATH=${D}${base_libdir}/firmware modules_install
-   rm "${D}${base_libdir}/modules/${KERNEL_VERSION}/build"
-   rm "${D}${base_libdir}/modules/${KERNEL_VERSION}/source"
+   oe_runmake DEPMOD=echo INSTALL_MOD_PATH="${D}" modules_install
+   rm "${D}/lib/modules/${KERNEL_VERSION}/build"
+   rm "${D}/lib/modules/${KERNEL_VERSION}/source"
# If the kernel/ directory is empty remove it to prevent QA 
issues
-   rmdir --ignore-fail-on-non-empty 
"${D}${base_libdir}/modules/${KERNEL_VERSION}/kernel"
+   rmdir --ignore-fail-on-non-empty 
"${D}/lib/modules/${KERNEL_VERSION}/kernel"
else
bbnote "no modules to install"
fi
@@ -478,7 +478,7 @@ EXPORT_FUNCTIONS do_compile do_install do_configure
 # kernel-image becomes kernel-image-${KERNEL_VERSION}
 PACKAGES = "kernel kernel-base kernel-vmlinux kernel-image kernel-dev 
kernel-modules"
 FILES_${PN} = ""
-FILES_kernel-base = "${base_libdir}/modules/${KERNEL_VERSION}/modules.order 
${base_libdir}/modules/${KERNEL_VERSION}/modules.builtin"
+FILES_kernel-base = "/lib/modules/${KERNEL_VERSION}/modules.order 
/lib/modules/${KERNEL_VERSION}/modules.builtin"
 FILES_kernel-image = ""
 FILES_kernel-dev = "/boot/System.map* /boot/Module.symvers* /boot/config* 
${KERNEL_SRC_PATH} /lib/modules/${KERNEL_VERSION}/build"
 FILES_kernel-vmlinux = "/boot/vmlinux-${KERNEL_VERSION_NAME}"
@@ -511,7 +511,7 @@ pkg_postinst_kernel-base () {
 PACKAGESPLITFUNCS_prepend = "split_kernel_packages "
 
 python split_kernel_packages () {
-do_split_packages(d, root='${base_libdir}/firmware', 
file_regex='^(.*)\.(bin|fw|cis|csp|dsp)$', output_pattern='kernel-firmware-%s', 
description='Firmware for %s', recursive=True, extra_depends='')
+do_split_packages(d, root='/lib/firmware', 
file_regex='^(.*)\.(bin|fw|cis|csp|dsp)$', output_pattern='kernel-firmware-%s', 
description='Firmware for %s', recursive=True, extra_depends='')
 }
 
 # Many scripts want to look in arch/$arch/boot for the bootable
-- 
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] Revert "linux-firmware: Modify firmware installation path"

[Jason Wessel]

This reverts commit a46a23fed9acd097ecc8ab3985d18688abe7b065.

The 64bit/32bit multilib builds break because the kernel's
firmware loader cannot find the firmware because it
gets placed int the wrong directory.

Signed-off-by: Jason Wessel 
---
 .../linux-firmware/linux-firmware_git.bb   | 168 ++---
 1 file changed, 82 insertions(+), 86 deletions(-)

diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb 
b/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
index 5942997fda..4fedcfbc67 100644
--- a/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
+++ b/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
@@ -195,27 +195,27 @@ do_compile() {
 }
 
 do_install() {
-   install -d  ${D}${base_libdir}/firmware/
-   cp -r * ${D}${base_libdir}/firmware/
+   install -d  ${D}/lib/firmware/
+   cp -r * ${D}/lib/firmware/
 
# Avoid Makefile to be deployed
-   rm ${D}${base_libdir}/firmware/Makefile
+   rm ${D}/lib/firmware/Makefile
 
# Remove unbuild firmware which needs cmake and bash
-   rm ${D}${base_libdir}/firmware/carl9170fw -rf
+   rm ${D}/lib/firmware/carl9170fw -rf
 
# Remove pointless bash script
-   rm ${D}${base_libdir}/firmware/configure
+   rm ${D}/lib/firmware/configure
 
# Libertas sd8686
-   ln -sf libertas/sd8686_v9.bin ${D}${base_libdir}/firmware/sd8686.bin
-   ln -sf libertas/sd8686_v9_helper.bin 
${D}${base_libdir}/firmware/sd8686_helper.bin
+   ln -sf libertas/sd8686_v9.bin ${D}/lib/firmware/sd8686.bin
+   ln -sf libertas/sd8686_v9_helper.bin ${D}/lib/firmware/sd8686_helper.bin
 
# fixup wl12xx location, after 2.6.37 the kernel searches a different 
location for it
-   ( cd ${D}${base_libdir}/firmware ; ln -sf ti-connectivity/* . )
+   ( cd ${D}/lib/firmware ; ln -sf ti-connectivity/* . )
 
 # Copy the iwlwifi ucode
-cp ${WORKDIR}/iwlwifi-8000C-19.ucode ${D}${base_libdir}/firmware/
+cp ${WORKDIR}/iwlwifi-8000C-19.ucode ${D}/lib/firmware/
 }
 
 
@@ -252,21 +252,21 @@ LICENSE_${PN}-ath6k = "Firmware-atheros_firmware"
 LICENSE_${PN}-ath9k = "Firmware-atheros_firmware"
 LICENSE_${PN}-atheros-license = "Firmware-atheros_firmware"
 
-FILES_${PN}-atheros-license = 
"${base_libdir}/firmware/LICENCE.atheros_firmware"
+FILES_${PN}-atheros-license = "/lib/firmware/LICENCE.atheros_firmware"
 FILES_${PN}-ar9170 = " \
-  ${base_libdir}/firmware/ar9170*.fw \
+  /lib/firmware/ar9170*.fw \
 "
 FILES_${PN}-carl9170 = " \
-  ${base_libdir}/firmware/carl9170*.fw \
+  /lib/firmware/carl9170*.fw \
 "
 FILES_${PN}-ath6k = " \
-  ${base_libdir}/firmware/ath6k \
+  /lib/firmware/ath6k \
 "
 FILES_${PN}-ath9k = " \
-  ${base_libdir}/firmware/ar9271.fw \
-  ${base_libdir}/firmware/ar7010*.fw \
-  ${base_libdir}/firmware/htc_9271.fw \
-  ${base_libdir}/firmware/htc_7010.fw \
+  /lib/firmware/ar9271.fw \
+  /lib/firmware/ar7010*.fw \
+  /lib/firmware/htc_9271.fw \
+  /lib/firmware/htc_7010.fw \
 "
 
 RDEPENDS_${PN}-ar9170 += "${PN}-atheros-license"
@@ -280,14 +280,14 @@ LICENSE_${PN}-ar3k-license = "Firmware-qualcommAthos_ar3k"
 LICENSE_${PN}-ath10k = "Firmware-qualcommAthos_ath10k"
 LICENSE_${PN}-ath10k-license = "Firmware-qualcommAthos_ath10k"
 
-FILES_${PN}-ar3k-license = 
"${base_libdir}/firmware/LICENSE.QualcommAtheros_ar3k"
+FILES_${PN}-ar3k-license = "/lib/firmware/LICENSE.QualcommAtheros_ar3k"
 FILES_${PN}-ar3k = " \
-  ${base_libdir}/firmware/ar3k \
+  /lib/firmware/ar3k \
 "
 
-FILES_${PN}-ath10k-license = 
"${base_libdir}/firmware/LICENSE.QualcommAtheros_ath10k"
+FILES_${PN}-ath10k-license = "/lib/firmware/LICENSE.QualcommAtheros_ath10k"
 FILES_${PN}-ath10k = " \
-  ${base_libdir}/firmware/ath10k \
+  /lib/firmware/ath10k \
 "
 
 RDEPENDS_${PN}-ar3k += "${PN}-ar3k-license"
@@ -297,9 +297,9 @@ RDEPENDS_${PN}-ath10k += "${PN}-ath10k-license"
 LICENSE_${PN}-ralink = "Firmware-ralink-firmware"
 LICENSE_${PN}-ralink-license = "Firmware-ralink-firmware"
 
-FILES_${PN}-ralink-license = 
"${base_libdir}/firmware/LICENCE.ralink-firmware.txt"
+FILES_${PN}-ralink-license = "/lib/firmware/LICENCE.ralink-firmware.txt"
 FILES_${PN}-ralink = " \
-  ${base_libdir}/firmware/rt*.bin \
+  /lib/firmware/rt*.bin \
 "
 
 RDEPENDS_${PN}-ralink += "${PN}-ralink-license"
@@ -308,9 +308,9 @@ RDEPENDS_${PN}-ralink += "${PN}-ralink-license"
 LICENSE_${PN}-radeon = "Firmware-radeon"
 LICENSE_${PN}-radeon-license = "Firmware-radeon"
 
-FILES_${PN}-radeon-license = "${base_libdir}/firmware/LICENSE.radeon"
+FILES_${PN}-radeon-license = "/lib/firmware/LICENSE.radeon"
 FILES_${PN}-radeon = " \
-  ${base_libdir}/firmware/radeon \
+  /lib/firmware/radeon \
 "
 
 RDEPENDS_${PN}-radeon += "${PN}-radeon-license"
@@ -322,20 +322,20 @@ LICENSE_${PN}-sd8787 = "Firmware-Marvell"
 LICENSE_${PN}-sd8797 = "Firmware-Marvell"
 LICENSE_${PN}-marvell-license = "Firmware-Marvell"
 
-FILES_${PN}-marvell-license = "${base_libdir}/firmware/LICENCE.Marvell"

[OE-core] still fighting to get an OE recipe for "libfilesys-smbclient-perl"

[Robert P. J. Day]

  following up on the whining i did a while back to finish off a few
perl recipes for OE, i have just a few left to resolve, and this is
one of them. i managed to get several dozen perl recipes to work
simply by throwing together recipes based on very little understanding
of perl cpan mechanics and just mindlessly cribbing boilerplate from
other recipes. if it worked, yee ha, move on.

  here's my current recipe for this module (again, slapped together
without really understanding what's going on under the hood):

/ start /

DESCRIPTION = "Filesys::SmbClient"

SECTION = "libs"
LICENSE = "Artisticv1 | GPLv1+"
PR = "r0"

MAINTAINER= "rday"
HOMEPAGE=   "https://metacpan.org/pod/Filesys::SmbClient;

LIC_FILES_CHKSUM = 
"file://${COMMON_LICENSE_DIR}/Artistic-1.0;md5=cda03bbdc3c1951996392b872397b798 
\
file://${COMMON_LICENSE_DIR}/GPL-1.0;md5=e9e36a9de734199567a4d769498f743d"

DEPENDS += "samba"

SRC_URI = 
"https://cpan.metacpan.org/authors/id/A/AL/ALIAN/Filesys-SmbClient-3.2.tar.gz;

SRC_URI[md5sum] = "8a4205fe748aac7f43fb2a0e8ef502b2"
SRC_URI[sha256sum] = 
"a9f2f3d4675de76f9dcae2be40bda93495cd5fafb1bb6326d71833cea451bc6e"

S = "${WORKDIR}/Filesys-SmbClient-${PV}"

inherit cpan

EXTRA_CPANFLAGS = "INC='-I${STAGING_INCDIR}/samba-4.0' 
LIBS='-L${STAGING_LIBDIR}'"

BBCLASSEXTEND = "native"

/ end /

and the error i get when i try to build makes it *clear* that the
configuration is failing to set up properly for cross-compiling for my
powerpc target:

$ bitbake libfilesys-smbclient-perl
... snip ...
| Where can I find libsmbclient.h ? []
*
| ** WHAT 
| ** I Can't find /libsmbclient.h : No such file or directory.
| *
... snip ...
| Where can I find libsmbclient.so ? []
*
| ** WHAT 
| ** I Can't find libsmbclient.so : No such file or directory.
... snip ...
| checking for C compiler default output... a.out
| checking whether the C compiler works... configure: error: cannot
run C compiled programs.
| If you meant to cross compile, use `--host'.
| See `config.log' for more details.


  and sure enough, the config.log for this build contains:

Target: powerpc-poky-linux
Configured with:
../../../../../../work-shared/gcc-6.3.0-r0/gcc-6.3.0/configure
--build=x86_64-linux --host=x86_64-linux --target=powerpc-poky-linux
...

  beyond this, i'm not sure how to use an OE patch to adjust this
source the right way, given that dozens and dozens of other modules
cross-compiled just fine.

  thoughts? i am in now way a perl expert so i really have no idea
where to go from here.

rday

-- 


Robert P. J. Day Ottawa, Ontario, CANADA
http://crashcourse.ca

Twitter:   http://twitter.com/rpjday
LinkedIn:   http://ca.linkedin.com/in/rpjday






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

[OE-core] [PATCH 0/1] create-pull-request: fix for OE cgit URL change

[Paul Eggleton]

The following changes since commit a7cb408dd784178197687a2129e936620bf6a0d3:

  sanity.bbclass: Improved error message (2017-01-17 11:31:31 +)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib 
paule/create-pull-request-cgit
  
http://cgit.openembedded.org/openembedded-core-contrib/log/?h=paule/create-pull-request-cgit

Paul Eggleton (1):
  create-pull-request: fix for OE cgit URL change

 scripts/create-pull-request | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.5.5

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

[OE-core] [PATCH 1/1] create-pull-request: fix for OE cgit URL change

[Paul Eggleton]

The /cgit.cgi/ part is no longer valid in cgit URLs on
git.openembedded.org as of recent infrastructure changes.

Signed-off-by: Paul Eggleton 
---
 scripts/create-pull-request | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/create-pull-request b/scripts/create-pull-request
index a88f35a..e82858b 100755
--- a/scripts/create-pull-request
+++ b/scripts/create-pull-request
@@ -168,7 +168,7 @@ case "$REMOTE_URL" in

WEB_URL="http://git.pokylinux.org/cgit.cgi/$REMOTE_REPO/log/?h=$BRANCH;
;;
*git.openembedded.org*)
-   
WEB_URL="http://cgit.openembedded.org/cgit.cgi/$REMOTE_REPO/log/?h=$BRANCH;
+   
WEB_URL="http://cgit.openembedded.org/$REMOTE_REPO/log/?h=$BRANCH;
;;
*github.com*)
WEB_URL="https://github.com/$REMOTE_REPO/tree/$BRANCH;
-- 
2.5.5

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

[OE-core] [PATCH 1/1] devtool: check locale and refuse to start if it isn't UTF-8

[Paul Eggleton]

We need to ensure the locale is UTF-8 or otherwise strange errors will
occur later on during execution - the same reason we check this in
BitBake itself. Unfortunately this check has to be before command line
parsing and therefore showing the help text in response to --help, since
that relies upon parsing bitbake's configuration (as we need to load
plugins in other layers).

Fixes [YOCTO #10908].

Signed-off-by: Paul Eggleton 
---
 scripts/devtool | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/scripts/devtool b/scripts/devtool
index 0866be9..c9ad9dd 100755
--- a/scripts/devtool
+++ b/scripts/devtool
@@ -215,6 +215,9 @@ def main():
 global config
 global context
 
+if sys.getfilesystemencoding() != "utf-8":
+sys.exit("Please use a locale setting which supports utf-8.\nPython 
can't change the filesystem locale after loading so we need a utf-8 when python 
starts or things won't work.")
+
 context = Context(fixed_setup=False)
 
 # Default basepath
-- 
2.5.5

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

[OE-core] [PATCH 0/1] devtool: check locale and refuse to start if it isn't UTF-8

[Paul Eggleton]

The following changes since commit a7cb408dd784178197687a2129e936620bf6a0d3:

  sanity.bbclass: Improved error message (2017-01-17 11:31:31 +)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib paule/devtool-utf8
  
http://cgit.openembedded.org/openembedded-core-contrib/log/?h=paule/devtool-utf8

Paul Eggleton (1):
  devtool: check locale and refuse to start if it isn't UTF-8

 scripts/devtool | 3 +++
 1 file changed, 3 insertions(+)

-- 
2.5.5

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

Re: [OE-core] [PATCH 1/3] alsa-lib: update 1.1.2 -> 1.1.3

[Burton, Ross]

Breaks under musl:

| ../../../alsa-lib-1.1.3/src/ucm/parser.c: In function
'is_component_directory':
| ../../../alsa-lib-1.1.3/src/ucm/parser.c:1392:39: error: 'PATH_MAX'
undeclared (first use in this function)
|if (!strncmp(dir, component_dir[i], PATH_MAX))
|^~~~

Can you cherry-pick
http://git.alsa-project.org/?p=alsa-lib.git;a=commit;h=9ed4075f05a4242f32331f7f2c365767970f5003
?

Ross

On 1 January 2017 at 19:53, Andreas Müller 
wrote:

> Signed-off-by: Andreas Müller 
> ---
>  meta/recipes-multimedia/alsa/{alsa-lib_1.1.2.bb => alsa-lib_1.1.3.bb} |
> 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>  rename meta/recipes-multimedia/alsa/{alsa-lib_1.1.2.bb =>
> alsa-lib_1.1.3.bb} (91%)
>
> diff --git a/meta/recipes-multimedia/alsa/alsa-lib_1.1.2.bb
> b/meta/recipes-multimedia/alsa/alsa-lib_1.1.3.bb
> similarity index 91%
> rename from meta/recipes-multimedia/alsa/alsa-lib_1.1.2.bb
> rename to meta/recipes-multimedia/alsa/alsa-lib_1.1.3.bb
> index e313da7..1afac7a 100644
> --- a/meta/recipes-multimedia/alsa/alsa-lib_1.1.2.bb
> +++ b/meta/recipes-multimedia/alsa/alsa-lib_1.1.3.bb
> @@ -12,8 +12,8 @@ SRC_URI = "ftp://ftp.alsa-project.org/
> pub/lib/${BP}.tar.bz2 \
> file://Check-if-wordexp-function-is-supported.patch \
> file://avoid-including-sys-poll.h-directly.patch \
>  "
> -SRC_URI[md5sum] = "1946e6438b8262a7b8fdadacd0e06ba7"
> -SRC_URI[sha256sum] = "d38dacd9892b06b8bff04923c380b3
> 8fb2e379ee5538935ff37e45b395d861d6"
> +SRC_URI[md5sum] = "eefe5992567ba00d6110a540657aaf5c"
> +SRC_URI[sha256sum] = "71282502184c592c1a008e256c22ed
> 0ba5728ca65e05273ceb480c70f515969c"
>
>  inherit autotools pkgconfig
>
> --
> 2.5.5
>
> --
> ___
> 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 v2 1/3] ccache.bbclass: enable max size setup for ccache dir

[Andre McCurdy]

On Thu, Jan 19, 2017 at 7:01 AM, Yannick Gicquel
 wrote:
> ccache directories are limited to 1G by default.

1G has been the default for a long time, so maybe it's time to propose
an upstream change to increase, e.g. to 10G, giving the size of the
webkitgtk build as justification?

Since we build ccache-native within oe-core, any such patch could be
applied to our version of ccache right now (ie we don't need to wait
for it to be accepted and merged upstream).

> This patch enables the configuration of their limits, and as default
> location is TMPDIR, it proposes a size limit to "0" (unlimited).
>
> The setup can be overloaded in local.conf by setting
> CCACHE_MAX_SIZE to a custom value if needed.
>
> Signed-off-by: Yannick Gicquel 
> ---
>  meta/classes/ccache.bbclass | 8 
>  1 file changed, 8 insertions(+)
>
> diff --git a/meta/classes/ccache.bbclass b/meta/classes/ccache.bbclass
> index 93fcaca..b6643a1 100644
> --- a/meta/classes/ccache.bbclass
> +++ b/meta/classes/ccache.bbclass
> @@ -1,6 +1,14 @@
>  CCACHE = "${@bb.utils.which(d.getVar('PATH'), 'ccache') and 'ccache '}"
>  export CCACHE_DIR ?= "${TMPDIR}/ccache/${MULTIMACH_HOST_SYS}/${PN}"
>  CCACHE_DISABLE[unexport] = "1"
> +CCACHE_MAX_SIZE ?= "0"
>
>  do_configure[dirs] =+ "${CCACHE_DIR}"
>  do_kernel_configme[dirs] =+ "${CCACHE_DIR}"
> +
> +ccache_init() {
> +if [ -n "${CCACHE}" ]; then
> +${CCACHE} -M ${CCACHE_MAX_SIZE}
> +fi
> +}
> +do_configure[postfuncs] += "ccache_init"
> --
> 1.9.1
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 1/2] classes/populate_sdk_ext: force a known value for TMPDIR

[Paul Eggleton]

If TMPDIR is configured to be somewhere outside of TOPDIR (a not
uncommon configuration where you have multiple disks and space on /home
is at a premium) then our attempt to find out the location of paths
under TMPDIR by using a relative path led to horribly broken paths
ending up in the eSDK. To save pain, just force a known value for TMPDIR
(i.e. ${TOPDIR}/tmp) and then we can assume that everywhere else.

Fixes [YOCTO #10797].

Signed-off-by: Paul Eggleton 
---
 meta/classes/populate_sdk_ext.bbclass | 5 +++--
 meta/recipes-core/meta/meta-environment-extsdk.bb | 4 ++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/meta/classes/populate_sdk_ext.bbclass 
b/meta/classes/populate_sdk_ext.bbclass
index f21ae8b..d5f6de9 100644
--- a/meta/classes/populate_sdk_ext.bbclass
+++ b/meta/classes/populate_sdk_ext.bbclass
@@ -282,6 +282,7 @@ python copy_buildsystem () {
 # Write a newline just in case there's none at the end of the 
original
 f.write('\n')
 
+f.write('TMPDIR = "${TOPDIR}/tmp"\n')
 f.write('DL_DIR = "${TOPDIR}/downloads"\n')
 
 f.write('INHERIT += "%s"\n' % 'uninative')
@@ -512,8 +513,8 @@ install_tools() {
# (they get populated from sstate on installation)
unfsd_path="${SDK_OUTPUT}/${SDKPATHNATIVE}${bindir_nativesdk}/unfsd"
if [ "${SDK_INCLUDE_TOOLCHAIN}" == "1" -a ! -e $unfsd_path ] ; then
-   
binrelpath=${@os.path.relpath(d.getVar('STAGING_BINDIR_NATIVE'), 
d.getVar('TOPDIR'))}
-   lnr ${SDK_OUTPUT}/${SDKPATH}/$binrelpath/unfsd $unfsd_path
+   
binrelpath=${@os.path.relpath(d.getVar('STAGING_BINDIR_NATIVE'), 
d.getVar('TMPDIR'))}
+   lnr ${SDK_OUTPUT}/${SDKPATH}/tmp/$binrelpath/unfsd $unfsd_path
fi
touch ${SDK_OUTPUT}/${SDKPATH}/.devtoolbase
 
diff --git a/meta/recipes-core/meta/meta-environment-extsdk.bb 
b/meta/recipes-core/meta/meta-environment-extsdk.bb
index 76d2661..757afc1 100644
--- a/meta/recipes-core/meta/meta-environment-extsdk.bb
+++ b/meta/recipes-core/meta/meta-environment-extsdk.bb
@@ -5,8 +5,8 @@ require meta-environment.bb
 PN = "meta-environment-extsdk-${MACHINE}"
 
 create_sdk_files_append() {
-   local 
sysroot=${SDKPATH}/${@os.path.relpath(d.getVar('STAGING_DIR_TARGET'), 
d.getVar('TOPDIR'))}
-   local 
sdkpathnative=${SDKPATH}/${@os.path.relpath(d.getVar('STAGING_DIR_NATIVE'), 
d.getVar('TOPDIR'))}
+   local 
sysroot=${SDKPATH}/tmp/${@os.path.relpath(d.getVar('STAGING_DIR_TARGET'), 
d.getVar('TMPDIR'))}
+   local 
sdkpathnative=${SDKPATH}/tmp/${@os.path.relpath(d.getVar('STAGING_DIR_NATIVE'), 
d.getVar('TMPDIR'))}
 
toolchain_create_sdk_env_script '' '' $sysroot '' ${bindir_native} 
${prefix_native} $sdkpathnative
 }
-- 
2.5.5

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

[OE-core] [PATCH 0/2] eSDK fixes

[Paul Eggleton]

The following changes since commit a7cb408dd784178197687a2129e936620bf6a0d3:

  sanity.bbclass: Improved error message (2017-01-17 11:31:31 +)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib paule/esdk-fixes3
  
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/esdk-fixes3

Paul Eggleton (2):
  classes/populate_sdk_ext: force a known value for TMPDIR
  Revert "classes/sstate: add a mode to error if sstate package unavailable"

 meta/classes/populate_sdk_ext.bbclass |  5 +++--
 meta/classes/sstate.bbclass   | 23 ---
 meta/recipes-core/meta/meta-environment-extsdk.bb |  4 ++--
 3 files changed, 5 insertions(+), 27 deletions(-)

-- 
2.5.5

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

[OE-core] [PATCH 2/2] Revert "classes/sstate: add a mode to error if sstate package unavailable"

[Paul Eggleton]

It turns out that this check cannot work. We don't have the information
to know whether an sstate package is really needed at this point in the
execution, so we check the availability for things that we won't
actually end up needing later on. Thus we can't fail if some of these
aren't found or we'll get needless failures.

This check was intended to give earlier more accurate errors when sstate
artifacts failed to download, but that's not practical so we'll rely
solely on the task execution check that was added within the runqueue.

This reverts most of commit 9e711b54487c3141d7264b8cf0d74f9465020190 (we
still need to allow BB_SETSCENE_ENFORCE through from the external
environment since the eSDK relies upon that.)

Signed-off-by: Paul Eggleton 
---
 meta/classes/sstate.bbclass | 23 ---
 1 file changed, 23 deletions(-)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index afdb4a3..dff530f 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -738,7 +738,6 @@ def sstate_checkhashes(sq_fn, sq_task, sq_hash, sq_hashfn, 
d, siginfo=False):
 
 ret = []
 missed = []
-missing = []
 extension = ".tgz"
 if siginfo:
 extension = extension + ".siginfo"
@@ -760,18 +759,6 @@ def sstate_checkhashes(sq_fn, sq_task, sq_hash, sq_hashfn, 
d, siginfo=False):
 
 return spec, extrapath, tname
 
-def sstate_pkg_to_pn(pkg, d):
-"""
-Translate an sstate filename to a PN value by way of SSTATE_PKGSPEC. 
This is slightly hacky but
-we don't have access to everything in this context.
-"""
-pkgspec = d.getVar('SSTATE_PKGSPEC', False)
-try:
-idx = pkgspec.split(':').index('${PN}')
-except ValueError:
-bb.fatal('Unable to find ${PN} in SSTATE_PKGSPEC')
-return pkg.split(':')[idx]
-
 
 for task in range(len(sq_fn)):
 
@@ -806,8 +793,6 @@ def sstate_checkhashes(sq_fn, sq_task, sq_hash, sq_hashfn, 
d, siginfo=False):
 if localdata.getVar('BB_NO_NETWORK') == "1" and 
localdata.getVar('SSTATE_MIRROR_ALLOW_NETWORK') == "1":
 localdata.delVar('BB_NO_NETWORK')
 
-whitelist = bb.runqueue.get_setscene_enforce_whitelist(d)
-
 from bb.fetch2 import FetchConnectionCache
 def checkstatus_init(thread_worker):
 thread_worker.connection_cache = FetchConnectionCache()
@@ -834,12 +819,6 @@ def sstate_checkhashes(sq_fn, sq_task, sq_hash, sq_hashfn, 
d, siginfo=False):
 except:
 missed.append(task)
 bb.debug(2, "SState: Unsuccessful fetch test for %s" % srcuri)
-if whitelist:
-pn = sstate_pkg_to_pn(sstatefile, d)
-taskname = sq_task[task]
-if not bb.runqueue.check_setscene_enforce_whitelist(pn, 
taskname, whitelist):
-missing.append(task)
-bb.error('Sstate artifact unavailable for %s.%s' % 
(pn, taskname))
 pass
 bb.event.fire(bb.event.ProcessProgress("Checking sstate mirror 
object availability", len(tasklist) - thread_worker.tasks.qsize()), d)
 
@@ -867,8 +846,6 @@ def sstate_checkhashes(sq_fn, sq_task, sq_hash, sq_hashfn, 
d, siginfo=False):
 bb.event.disable_threadlock()
 
 bb.event.fire(bb.event.ProcessFinished("Checking sstate mirror 
object availability"), d)
-if whitelist and missing:
-bb.fatal('Required artifacts were unavailable - exiting')
 
 inheritlist = d.getVar("INHERIT")
 if "toaster" in inheritlist:
-- 
2.5.5

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

Re: [OE-core] [PATCH] kernel-fitimage: Let U-Boot select ramdisk load/entry address

[Burton, Ross]

On 19 January 2017 at 17:45, Denys Dmytriyenko  wrote:

> Yeah, that one too, missed it... Been sitting there since October?
>

Nathan pinged earlier, it's in mut now and will be in master when RP has a
moment.  As you may have noticed, the recipe-specific-sysroots work is
keeping him busy :)

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

Re: [OE-core] [PATCH] kernel-fitimage: Let U-Boot select ramdisk load/entry address

[Denys Dmytriyenko]

On Thu, Jan 19, 2017 at 05:40:15PM +, Burton, Ross wrote:
> On 19 January 2017 at 17:37, Denys Dmytriyenko  wrote:
> 
> > Sometimes it's useful to load ramdisk in a specific location. Your patch
> > unconditionally removes this support, which is not a desired change. I have
> > a patch queued up to do it conditionally, which I'll try to submit shortly.
> >
> 
> Would that be the same as Nathan's "kernel-fitimage.bbclass: Allow unset
> load/entry addresses for ramdisks"?

Yeah, that one too, missed it... Been sitting there since October?

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

[OE-core] [PATCH] linux-firmware: relocate ath9k_htc blobs to correct package

[Saul Wold]

The newer firmware blobs were versioned and landing in the default
package so move them to their appropriately named package.

[YOCTO #10942]

Signed-off-by: Saul Wold 
---
 meta/recipes-kernel/linux-firmware/linux-firmware_git.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb 
b/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
index 5942997..bdad5f2 100644
--- a/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
+++ b/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
@@ -267,6 +267,8 @@ FILES_${PN}-ath9k = " \
   ${base_libdir}/firmware/ar7010*.fw \
   ${base_libdir}/firmware/htc_9271.fw \
   ${base_libdir}/firmware/htc_7010.fw \
+  ${base_libdir}/firmware/ath9k_htc/htc_7010-1.4.0.fw \
+  ${base_libdir}/firmware/ath9k_htc/htc_9271-1.4.0.fw \
 "
 
 RDEPENDS_${PN}-ar9170 += "${PN}-atheros-license"
-- 
2.7.4

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

Re: [OE-core] [PATCH] kernel-fitimage: Let U-Boot select ramdisk load/entry address

[Burton, Ross]

On 19 January 2017 at 17:37, Denys Dmytriyenko  wrote:

> Sometimes it's useful to load ramdisk in a specific location. Your patch
> unconditionally removes this support, which is not a desired change. I have
> a patch queued up to do it conditionally, which I'll try to submit shortly.
>

Would that be the same as Nathan's "kernel-fitimage.bbclass: Allow unset
load/entry addresses for ramdisks"?

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

Re: [OE-core] [PATCH] kernel-fitimage: Let U-Boot select ramdisk load/entry address

[Denys Dmytriyenko]

On Thu, Jan 19, 2017 at 08:58:03AM -0800, Rick Altherr wrote:
> If the ramdisk load and entry parameters are omitted in the FIT image
> specification, U-Boot will determine if the ramdisk needs to be
> relocated and will automatically allocate a location in RAM if so.

Sometimes it's useful to load ramdisk in a specific location. Your patch 
unconditionally removes this support, which is not a desired change. I have 
a patch queued up to do it conditionally, which I'll try to submit shortly.

-- 
Denys


> Signed-off-by: Rick Altherr 
> ---
>  meta/classes/kernel-fitimage.bbclass | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/meta/classes/kernel-fitimage.bbclass 
> b/meta/classes/kernel-fitimage.bbclass
> index 2934b3a31a..ee721d1fff 100644
> --- a/meta/classes/kernel-fitimage.bbclass
> +++ b/meta/classes/kernel-fitimage.bbclass
> @@ -187,8 +187,6 @@ fitimage_emit_section_ramdisk() {
>  arch = "${UBOOT_ARCH}";
>  os = "linux";
>  compression = "none";
> -load = <${UBOOT_RD_LOADADDRESS}>;
> -entry = <${UBOOT_RD_ENTRYPOINT}>;
>  hash@1 {
>  algo = "${ramdisk_csum}";
>  };
> -- 
> 2.11.0.483.g087da7b7c-goog
> 
> -- 
> ___
> 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] flex: upgrade to 2.6.2

[Patrick Ohly]

On Mon, 2017-01-09 at 15:59 +, Ross Burton wrote:
> Patches dropped as they are merged upstream:
> - CVE-2016-6354.patch
> - 0002-avoid-c-comments-in-c-code-fails-with-gcc-6.patch
> - do_not_create_pdf_doc.patch
> 
> Apply a patch from github to simplify cross-compilation and not need a
> flex-native to bootstrap.

After rebasing my pending "UEFI/OVMF" patches I noticed that this
particular version update breaks compilation of acpica-native:

| obj/dtparserlex.o: In function `DtInitLexer':
| dtparserlex.c:(.text+0x10e7): undefined reference to `yy_scan_string'
| obj/prparserlex.o: In function `PrInitLexer':
| prparserlex.c:(.text+0x1327): undefined reference to `yy_scan_string'
| collect2: error: ld returned 1 exit status
| ../Makefile.rules:14: recipe for target 'obj/iasl' failed
| make[1]: *** [obj/iasl] Error 1
| make[1]: Leaving directory
'/fast/build/refkit/intel-corei7-64/tmp-glibc/work/x86_64-linux/acpica-native/20161222-r0/acpica-unix2-20161222/generate/unix/iasl'
| generate/unix/Makefile.common:25: recipe for target 'iasl' failed
| make: *** [iasl] Error 2
| WARNING: exit code 1 from a shell command.
| ERROR: Function failed: do_compile (log file is located
at 
/fast/build/refkit/intel-corei7-64/tmp-glibc/work/x86_64-linux/acpica-native/20161222-r0/temp/log.do_compile.32470)

That still occurs also after updating to the latest upstream acpica.

I searched for the error and found this about yy_scan_string:

In case anyone else is getting symbol is not defined or other
such errors when trying this: remember to include %option
reentrant in the lexer file.

http://stackoverflow.com/questions/1907847/how-to-use-yy-scan-string-in-lex?ref=driverlayer.com/web

I tried that and then just got different compile errors.

Perhaps someone with more knowledge about flex can provide some
guidance?

Fathi, is that something that you or upstream have come across?

Ross, would it be possible to revert this patch so that the UEFI patches
can be merged? Obviously the problem still needs to be resolved.

-- 
Best Regards, Patrick Ohly

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



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

Re: [OE-core] [PATCH] u-boot: Update to 2017.01 release

[Denys Dmytriyenko]

On Thu, Jan 19, 2017 at 04:21:51PM +0100, Marek Vasut wrote:
> Upgrade U-Boot to the latest version.
> 
> Signed-off-by: Marek Vasut 
> Cc: Richard Purdie 
> Cc: Denys Dmytriyenko 

Acked-by: Denys Dmytriyenko 


> ---
>  .../u-boot/{u-boot-common_2016.11.inc => u-boot-common_2017.01.inc} | 2 
> +-
>  .../u-boot/{u-boot-fw-utils_2016.11.bb => u-boot-fw-utils_2017.01.bb}   | 0
>  .../u-boot/{u-boot-mkimage_2016.11.bb => u-boot-mkimage_2017.01.bb} | 0
>  meta/recipes-bsp/u-boot/{u-boot_2016.11.bb => u-boot_2017.01.bb}| 0
>  4 files changed, 1 insertion(+), 1 deletion(-)
>  rename meta/recipes-bsp/u-boot/{u-boot-common_2016.11.inc => 
> u-boot-common_2017.01.inc} (86%)
>  rename meta/recipes-bsp/u-boot/{u-boot-fw-utils_2016.11.bb => 
> u-boot-fw-utils_2017.01.bb} (100%)
>  rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2016.11.bb => 
> u-boot-mkimage_2017.01.bb} (100%)
>  rename meta/recipes-bsp/u-boot/{u-boot_2016.11.bb => u-boot_2017.01.bb} 
> (100%)
> 
> diff --git a/meta/recipes-bsp/u-boot/u-boot-common_2016.11.inc 
> b/meta/recipes-bsp/u-boot/u-boot-common_2017.01.inc
> similarity index 86%
> rename from meta/recipes-bsp/u-boot/u-boot-common_2016.11.inc
> rename to meta/recipes-bsp/u-boot/u-boot-common_2017.01.inc
> index 84adc1c56a..df24c853dd 100644
> --- a/meta/recipes-bsp/u-boot/u-boot-common_2016.11.inc
> +++ b/meta/recipes-bsp/u-boot/u-boot-common_2017.01.inc
> @@ -7,7 +7,7 @@ PE = "1"
>  
>  # We use the revision in order to avoid having to fetch it from the
>  # repo during parse
> -SRCREV = "29e0cfb4f77f7aa369136302cee14a91e22dca71"
> +SRCREV = "a705ebc81b7f91bbd0ef7c634284208342901149"
>  
>  SRC_URI = "git://git.denx.de/u-boot.git"
>  
> diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.11.bb 
> b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2017.01.bb
> similarity index 100%
> rename from meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.11.bb
> rename to meta/recipes-bsp/u-boot/u-boot-fw-utils_2017.01.bb
> diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.11.bb 
> b/meta/recipes-bsp/u-boot/u-boot-mkimage_2017.01.bb
> similarity index 100%
> rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2016.11.bb
> rename to meta/recipes-bsp/u-boot/u-boot-mkimage_2017.01.bb
> diff --git a/meta/recipes-bsp/u-boot/u-boot_2016.11.bb 
> b/meta/recipes-bsp/u-boot/u-boot_2017.01.bb
> similarity index 100%
> rename from meta/recipes-bsp/u-boot/u-boot_2016.11.bb
> rename to meta/recipes-bsp/u-boot/u-boot_2017.01.bb
> -- 
> 2.11.0
> 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] kernel-fitimage: Let U-Boot select ramdisk load/entry address

[Rick Altherr]

If the ramdisk load and entry parameters are omitted in the FIT image
specification, U-Boot will determine if the ramdisk needs to be
relocated and will automatically allocate a location in RAM if so.

Signed-off-by: Rick Altherr 
---
 meta/classes/kernel-fitimage.bbclass | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/classes/kernel-fitimage.bbclass 
b/meta/classes/kernel-fitimage.bbclass
index 2934b3a31a..ee721d1fff 100644
--- a/meta/classes/kernel-fitimage.bbclass
+++ b/meta/classes/kernel-fitimage.bbclass
@@ -187,8 +187,6 @@ fitimage_emit_section_ramdisk() {
 arch = "${UBOOT_ARCH}";
 os = "linux";
 compression = "none";
-load = <${UBOOT_RD_LOADADDRESS}>;
-entry = <${UBOOT_RD_ENTRYPOINT}>;
 hash@1 {
 algo = "${ramdisk_csum}";
 };
-- 
2.11.0.483.g087da7b7c-goog

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

[OE-core] [PATCH] core-image-sato: SDK should contain intltool and glib-2.0 tools for NLS

[Richard Purdie]

The SDK (and its tests) need these tools. We happened to be finding them
from the native tools in PATH but recipe specific sysroots prevents that.

This fixes the SDK to work as originally intended and contain these tools.

We have to clear this in the eSDK case since the _append can't be overridden
from the class. Ugly, but we'll have to come back to this issue.

Signed-off-by: Richard Purdie 
---
 meta/recipes-sato/images/core-image-sato.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-sato/images/core-image-sato.bb 
b/meta/recipes-sato/images/core-image-sato.bb
index e3246d2..99dad11 100644
--- a/meta/recipes-sato/images/core-image-sato.bb
+++ b/meta/recipes-sato/images/core-image-sato.bb
@@ -9,3 +9,6 @@ LICENSE = "MIT"
 inherit core-image
 
 IMAGE_INSTALL += "packagegroup-core-x11-sato-games"
+
+TOOLCHAIN_HOST_TASK_append = " nativesdk-intltool nativesdk-glib-2.0"
+TOOLCHAIN_HOST_TASK_remove_task-populate-sdk-ext = " nativesdk-intltool 
nativesdk-glib-2.0"
-- 
2.7.4

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

[OE-core] [PATCH 08/11] devtool: fix source extraction to work with recipe specific sysroots

[Richard Purdie]

From: Paul Eggleton 

When extracting source for a recipe within devtool (for extract, modify
or upgrade) We need to redirect WORKDIR, STAMPS_DIR etc. under a
temporary directory so that:
 (a) we pick up all files that get unpacked to the WORKDIR, and
 (b) we don't disturb the existing build
However, with recipe-specific sysroots the sysroots for the recipe will
be prepared under WORKDIR, and if we used the system temporary directory
i.e. usually /tmp) as used by mkdtemp by default, then our attempts to
hardlink files into the recipe-specific sysroots will fail on systems
where /tmp is a different filesystem, and we'd have to fall back to
copying the files which is a waste of time. Put the temp directory under
the WORKDIR to prevent that from being a problem.

Signed-off-by: Paul Eggleton 
[RP: Add needed mkdirhier call]
Signed-off-by: Richard Purdie 
---
 scripts/lib/devtool/standard.py | 16 +++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py
index 8944b0a..30b247f 100644
--- a/scripts/lib/devtool/standard.py
+++ b/scripts/lib/devtool/standard.py
@@ -465,7 +465,21 @@ def _extract_source(srctree, keep_temp, devbranch, sync, 
d, tinfoil):
 os.rmdir(srctree)
 
 initial_rev = None
-tempdir = tempfile.mkdtemp(prefix='devtool')
+# We need to redirect WORKDIR, STAMPS_DIR etc. under a temporary
+# directory so that:
+# (a) we pick up all files that get unpacked to the WORKDIR, and
+# (b) we don't disturb the existing build
+# However, with recipe-specific sysroots the sysroots for the recipe
+# will be prepared under WORKDIR, and if we used the system temporary
+# directory (i.e. usually /tmp) as used by mkdtemp by default, then
+# our attempts to hardlink files into the recipe-specific sysroots
+# will fail on systems where /tmp is a different filesystem, and it
+# would have to fall back to copying the files which is a waste of
+# time. Put the temp directory under the WORKDIR to prevent that from
+# being a problem.
+tempbasedir = d.getVar('WORKDIR')
+bb.utils.mkdirhier(tempbasedir)
+tempdir = tempfile.mkdtemp(prefix='devtooltmp-', dir=tempbasedir)
 try:
 tinfoil.logger.setLevel(logging.WARNING)
 
-- 
2.7.4

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

[OE-core] [PATCH 10/11] oe-selftest: Error if the user has buildhistory enabled

[Richard Purdie]

buildhistory interfers with various tests so error if the user has it enabled,
hence avoiding time spent running the tests only for them to fail.

Signed-off-by: Richard Purdie 
---
 scripts/oe-selftest | 4 
 1 file changed, 4 insertions(+)

diff --git a/scripts/oe-selftest b/scripts/oe-selftest
index 120a6c2..124a8dd 100755
--- a/scripts/oe-selftest
+++ b/scripts/oe-selftest
@@ -134,6 +134,10 @@ def preflight_check():
 log.error("You don't seem to have the meta-selftest layer in BBLAYERS")
 return False
 
+if "buildhistory.bbclass" in get_bb_var("BBINCLUDED"):
+log.error("You have buildhistory enabled already and this isn't 
recommended for selftest, please disable it first.")
+return False
+
 log.info("Running bitbake -p")
 runCmd("bitbake -p")
 
-- 
2.7.4

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

[OE-core] [PATCH 02/11] intltool: Allow nativesdk variant to build

[Richard Purdie]

This tool has been missing from the SDKs, the recipe specific sysroot
work highlighted the issues.

Signed-off-by: Richard Purdie 
---
 meta/recipes-devtools/intltool/intltool_0.51.0.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/intltool/intltool_0.51.0.bb 
b/meta/recipes-devtools/intltool/intltool_0.51.0.bb
index a7ea3b7..551bdf0 100644
--- a/meta/recipes-devtools/intltool/intltool_0.51.0.bb
+++ b/meta/recipes-devtools/intltool/intltool_0.51.0.bb
@@ -22,6 +22,7 @@ inherit autotools pkgconfig perlnative
 
 export PERL = "${bindir}/env perl"
 PERL_class-native = "/usr/bin/env nativeperl"
+PERL_class-nativesdk = "/usr/bin/env perl"
 
 # gettext is assumed to exist on the host
 RDEPENDS_${PN}_class-native = "libxml-parser-perl-native"
@@ -33,4 +34,4 @@ FILES_${PN} += "${datadir}/aclocal"
 
 INSANE_SKIP_${PN} += "dev-deps"
 
-BBCLASSEXTEND = "native"
+BBCLASSEXTEND = "native nativesdk"
-- 
2.7.4

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

[OE-core] [PATCH 04/11] vte: Add missing libxml2-native DEPENDS

[Richard Purdie]

Configure searches for xmllint which comes from libxml2-native.

Signed-off-by: Richard Purdie 
---
 meta/recipes-support/vte/vte_0.44.2.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-support/vte/vte_0.44.2.bb 
b/meta/recipes-support/vte/vte_0.44.2.bb
index a0b671e..8cddc98 100644
--- a/meta/recipes-support/vte/vte_0.44.2.bb
+++ b/meta/recipes-support/vte/vte_0.44.2.bb
@@ -1,7 +1,7 @@
 SUMMARY = "Virtual terminal emulator GTK+ widget library"
 BUGTRACKER = "https://bugzilla.gnome.org/buglist.cgi?product=vte;
 LICENSE = "LGPLv2.1+"
-DEPENDS = "glib-2.0 gtk+3 intltool-native"
+DEPENDS = "glib-2.0 gtk+3 intltool-native libxml2-native"
 
 LIC_FILES_CHKSUM = "file://COPYING;md5=4fbd65380cdd255951079008b364516c"
 
-- 
2.7.4

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

[OE-core] [PATCH 01/11] libxml-parser-perl: Allow nativesdk variant to build

[Richard Purdie]

This is needed in order to be able to build nativesdk-intltool.

Signed-off-by: Richard Purdie 
---
 meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb 
b/meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb
index ab9e5fa..d9bbf71 100644
--- a/meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb
+++ b/meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb
@@ -33,4 +33,4 @@ do_compile_class-native() {
cpan_do_compile
 }
 
-BBCLASSEXTEND="native"
+BBCLASSEXTEND="native nativesdk"
-- 
2.7.4

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

[OE-core] [PATCH 06/11] sysprof/gst-player/pcmanfm: Add missing glib-2.0-native DEPENDS

[Richard Purdie]

These recipes use glib-2.0 NLS tools so we need to depend on glib-2.0-native.

Signed-off-by: Richard Purdie 
---
 meta/recipes-kernel/sysprof/sysprof_3.22.3.bb   | 2 +-
 meta/recipes-multimedia/gstreamer/gst-player_git.bb | 2 +-
 meta/recipes-sato/pcmanfm/pcmanfm_1.2.5.bb  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-kernel/sysprof/sysprof_3.22.3.bb 
b/meta/recipes-kernel/sysprof/sysprof_3.22.3.bb
index 05ed8af..e1033ca 100644
--- a/meta/recipes-kernel/sysprof/sysprof_3.22.3.bb
+++ b/meta/recipes-kernel/sysprof/sysprof_3.22.3.bb
@@ -5,7 +5,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=d32239bcb673463ab874e80d47fae504 \
 
 inherit gnomebase gettext systemd
 
-DEPENDS = "glib-2.0 libxml2-native"
+DEPENDS = "glib-2.0 libxml2-native glib-2.0-native"
 
 SRC_URI += " \
file://define-NT_GNU_BUILD_ID.patch \
diff --git a/meta/recipes-multimedia/gstreamer/gst-player_git.bb 
b/meta/recipes-multimedia/gstreamer/gst-player_git.bb
index 2a4c9dc..9a023ab 100644
--- a/meta/recipes-multimedia/gstreamer/gst-player_git.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-player_git.bb
@@ -2,7 +2,7 @@ SUMMARY = "GStreamer playback helper library and examples"
 LICENSE = "LGPL-2.0+"
 LIC_FILES_CHKSUM = 
"file://gtk/gtk-play.c;beginline=1;endline=20;md5=f8c72dae3d36823ec716a9ebcae593b9"
 
-DEPENDS = "glib-2.0 gstreamer1.0 gstreamer1.0-plugins-base 
gstreamer1.0-plugins-bad gtk+3"
+DEPENDS = "glib-2.0 gstreamer1.0 gstreamer1.0-plugins-base 
gstreamer1.0-plugins-bad gtk+3 glib-2.0-native"
 
 SRC_URI = "git://github.com/sdroege/gst-player.git \
file://filechooser.patch;apply=0 \
diff --git a/meta/recipes-sato/pcmanfm/pcmanfm_1.2.5.bb 
b/meta/recipes-sato/pcmanfm/pcmanfm_1.2.5.bb
index aab7911..75ff773 100644
--- a/meta/recipes-sato/pcmanfm/pcmanfm_1.2.5.bb
+++ b/meta/recipes-sato/pcmanfm/pcmanfm_1.2.5.bb
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552 \
 
file://src/gseal-gtk-compat.h;endline=21;md5=46922c8691f58d124f9420fe16149ce2"
 
 SECTION = "x11"
-DEPENDS = "gtk+3 startup-notification libfm intltool-native gettext-native"
+DEPENDS = "gtk+3 startup-notification libfm intltool-native gettext-native 
glib-2.0-native"
 
 
 COMPATIBLE_HOST = 
'(x86_64.*|i.86.*|aarch64.*|arm.*|mips.*|powerpc.*|sh.*)-(linux|freebsd.*)'
-- 
2.7.4

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

[OE-core] [PATCH 05/11] sysprof: Add missing libxml2-native DEPENDS

[Richard Purdie]

Configure searches for xmllint which comes from libxml2-native.

Signed-off-by: Richard Purdie 
---
 meta/recipes-kernel/sysprof/sysprof_3.22.3.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/sysprof/sysprof_3.22.3.bb 
b/meta/recipes-kernel/sysprof/sysprof_3.22.3.bb
index 50be960..05ed8af 100644
--- a/meta/recipes-kernel/sysprof/sysprof_3.22.3.bb
+++ b/meta/recipes-kernel/sysprof/sysprof_3.22.3.bb
@@ -5,7 +5,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=d32239bcb673463ab874e80d47fae504 \
 
 inherit gnomebase gettext systemd
 
-DEPENDS = "glib-2.0"
+DEPENDS = "glib-2.0 libxml2-native"
 
 SRC_URI += " \
file://define-NT_GNU_BUILD_ID.patch \
-- 
2.7.4

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

[OE-core] [PATCH 07/11] sstate: Fix EXTRA_STAGING_FIXMES handling

[Richard Purdie]

The code wasn't working properly, tweak so that it works as expected and
the grep expression includes the right patterns. Not sure this code has ever
worked prior to this.

Signed-off-by: Richard Purdie 
---
 meta/classes/sstate.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index afdb4a3..34069c7 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -518,6 +518,7 @@ python sstate_hardcode_path () {
 for fixmevar in extra_staging_fixmes.split():
 fixme_path = d.getVar(fixmevar)
 sstate_sed_cmd += " -e 's:%s:FIXME_%s:g'" % (fixme_path, fixmevar)
+sstate_grep_cmd += " -e '%s'" % (fixme_path)
 
 fixmefn =  sstate_builddir + "fixmepath"
 
-- 
2.7.4

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

[OE-core] [PATCH 09/11] multilib_global: Drop pointless event mask/code filtering

[Richard Purdie]

This code was pointless so cleanup, drop the unused event and the
filtering is no longer needed.

Signed-off-by: Richard Purdie 
---
 meta/classes/multilib_global.bbclass | 10 --
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/meta/classes/multilib_global.bbclass 
b/meta/classes/multilib_global.bbclass
index aba0371..c9869fd 100644
--- a/meta/classes/multilib_global.bbclass
+++ b/meta/classes/multilib_global.bbclass
@@ -137,13 +137,11 @@ addhandler multilib_virtclass_handler_vendor
 multilib_virtclass_handler_vendor[eventmask] = "bb.event.ConfigParsed"
 
 python multilib_virtclass_handler_global () {
-if not e.data:
-return
-
 variant = e.data.getVar("BBEXTENDVARIANT")
+if variant:
+return
 
-if isinstance(e, bb.event.RecipeParsed) and not variant:
-if bb.data.inherits_class('kernel', e.data) or \
+if bb.data.inherits_class('kernel', e.data) or \
 bb.data.inherits_class('module-base', e.data) or \
 (bb.data.inherits_class('allarch', e.data) and\
  not bb.data.inherits_class('packagegroup', e.data)):
@@ -177,5 +175,5 @@ python multilib_virtclass_handler_global () {
 }
 
 addhandler multilib_virtclass_handler_global
-multilib_virtclass_handler_global[eventmask] = "bb.event.RecipePreFinalise 
bb.event.RecipeParsed"
+multilib_virtclass_handler_global[eventmask] = "bb.event.RecipeParsed"
 
-- 
2.7.4

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

[OE-core] [PATCH 11/11] staging: Add dependency on virtual/binutils for strip binary

[Richard Purdie]

The staging code strips binaries and we need virtual/binutils for that.
Add a specific dependency since the one from do_configure and others
may not be enough to ensure the binaries are in our own sysroot.

Signed-off-by: Richard Purdie 
---
 meta/classes/allarch.bbclass | 1 +
 meta/classes/staging.bbclass | 4 
 2 files changed, 5 insertions(+)

diff --git a/meta/classes/allarch.bbclass b/meta/classes/allarch.bbclass
index 9dce498..026214e 100644
--- a/meta/classes/allarch.bbclass
+++ b/meta/classes/allarch.bbclass
@@ -32,6 +32,7 @@ python () {
 d.setVar("TARGET_CFLAGS", "none")
 d.setVar("TARGET_CXXFLAGS", "none")
 d.setVar("TARGET_LDFLAGS", "none")
+d.setVar("POPULATESYSROOTDEPS", "")
 
 # Avoid this being unnecessarily different due to nuances of
 # the target machine that aren't important for "all" arch
diff --git a/meta/classes/staging.bbclass b/meta/classes/staging.bbclass
index 72a7421..2512ae6 100644
--- a/meta/classes/staging.bbclass
+++ b/meta/classes/staging.bbclass
@@ -228,6 +228,10 @@ python do_populate_sysroot () {
 do_populate_sysroot[vardeps] += "${SYSROOT_PREPROCESS_FUNCS}"
 do_populate_sysroot[vardepsexclude] += "MULTI_PROVIDER_WHITELIST"
 
+POPULATESYSROOTDEPS = ""
+POPULATESYSROOTDEPS_class-target = 
"virtual/${MLPREFIX}${TARGET_PREFIX}binutils:do_populate_sysroot"
+do_populate_sysroot[depends] += "${POPULATESYSROOTDEPS}"
+
 SSTATETASKS += "do_populate_sysroot"
 do_populate_sysroot[cleandirs] = "${SYSROOT_DESTDIR}"
 do_populate_sysroot[sstate-inputdirs] = "${SYSROOT_DESTDIR}"
-- 
2.7.4

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

[OE-core] [PATCH 03/11] systemd-boot: Add missing gperf-native DEPENDS

[Richard Purdie]

Configure searches for this and fails if its not present.

Signed-off-by: Richard Purdie 
---
 meta/recipes-bsp/systemd-boot/systemd-boot_232.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-bsp/systemd-boot/systemd-boot_232.bb 
b/meta/recipes-bsp/systemd-boot/systemd-boot_232.bb
index 602052c..0471ce2 100644
--- a/meta/recipes-bsp/systemd-boot/systemd-boot_232.bb
+++ b/meta/recipes-bsp/systemd-boot/systemd-boot_232.bb
@@ -1,6 +1,6 @@
 require recipes-core/systemd/systemd.inc
 
-DEPENDS = "intltool-native libcap util-linux gnu-efi"
+DEPENDS = "intltool-native libcap util-linux gnu-efi gperf-native"
 
 SRC_URI += 
"file://0001-use-lnr-wrapper-instead-of-looking-for-relative-opti.patch"
 
-- 
2.7.4

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

Re: [OE-core] [PATCH 05/16] gnu-efi: build 64-bit for x32

[Christopher Larson]

On Thu, Jan 19, 2017 at 8:32 AM, Wold, Saul  wrote:

> On Thu, 2017-01-05 at 12:46 -0700, Christopher Larson wrote:
> > From: Christopher Larson 
> >
> > We're targeting the x86_64 EFI ABI.
> >
> Chris,
>
> Firstly, thanks for all the x32 patches!
>
> I tried to build gnu-efi for x32 via genericx86-64 and still got a
> failure about missing the gnu/stubs-64.h. I am building pure x32, is it
> possibly you build x32 in a TMPDIR that already had a 64bit build?
>
> There is a generated stubs-x32.h, but not the required -64 variant.
>
> Can you confirm this builds correctly?


I'm building x32-only, but admittedly 90% of my builds are with an external
toolchain, so any bugs in the build of the toolchain obviously wouldn't be
encountered in that case. I suspect that's what's happening here, so this
would be a new issue to dive into.
-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 05/16] gnu-efi: build 64-bit for x32

[Wold, Saul]

On Thu, 2017-01-05 at 12:46 -0700, Christopher Larson wrote:
> From: Christopher Larson 
> 
> We're targeting the x86_64 EFI ABI.
> 
Chris, 

Firstly, thanks for all the x32 patches!

I tried to build gnu-efi for x32 via genericx86-64 and still got a
failure about missing the gnu/stubs-64.h. I am building pure x32, is it
possibly you build x32 in a TMPDIR that already had a 64bit build?

There is a generated stubs-x32.h, but not the required -64 variant.

Can you confirm this builds correctly?

Thanks

Sau!

> Signed-off-by: Christopher Larson 
> ---
>  meta/recipes-bsp/gnu-efi/gnu-efi_3.0.4.bb | 9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/meta/recipes-bsp/gnu-efi/gnu-efi_3.0.4.bb
> b/meta/recipes-bsp/gnu-efi/gnu-efi_3.0.4.bb
> index 9ddc4ba..e55ab7f 100644
> --- a/meta/recipes-bsp/gnu-efi/gnu-efi_3.0.4.bb
> +++ b/meta/recipes-bsp/gnu-efi/gnu-efi_3.0.4.bb
> @@ -52,3 +52,12 @@ BBCLASSEXTEND = "native"
>  # CFLAGS += -mno-mmx -mno-sse
>  # So also remove -mfpmath=sse from TUNE_CCARGS
>  TUNE_CCARGS_remove = "-mfpmath=sse"
> +
> +python () {
> +ccargs = d.getVar('TUNE_CCARGS', True).split()
> +if '-mx32' in ccargs:
> +# use x86_64 EFI ABI
> +ccargs.remove('-mx32')
> +ccargs.append('-m64')
> +d.setVar('TUNE_CCARGS', ' '.join(ccargs))
> +}
> -- 
> 2.8.0
> 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] u-boot: Update to 2017.01 release

[Marek Vasut]

Upgrade U-Boot to the latest version.

Signed-off-by: Marek Vasut 
Cc: Richard Purdie 
Cc: Denys Dmytriyenko 
---
 .../u-boot/{u-boot-common_2016.11.inc => u-boot-common_2017.01.inc} | 2 +-
 .../u-boot/{u-boot-fw-utils_2016.11.bb => u-boot-fw-utils_2017.01.bb}   | 0
 .../u-boot/{u-boot-mkimage_2016.11.bb => u-boot-mkimage_2017.01.bb} | 0
 meta/recipes-bsp/u-boot/{u-boot_2016.11.bb => u-boot_2017.01.bb}| 0
 4 files changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-bsp/u-boot/{u-boot-common_2016.11.inc => 
u-boot-common_2017.01.inc} (86%)
 rename meta/recipes-bsp/u-boot/{u-boot-fw-utils_2016.11.bb => 
u-boot-fw-utils_2017.01.bb} (100%)
 rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2016.11.bb => 
u-boot-mkimage_2017.01.bb} (100%)
 rename meta/recipes-bsp/u-boot/{u-boot_2016.11.bb => u-boot_2017.01.bb} (100%)

diff --git a/meta/recipes-bsp/u-boot/u-boot-common_2016.11.inc 
b/meta/recipes-bsp/u-boot/u-boot-common_2017.01.inc
similarity index 86%
rename from meta/recipes-bsp/u-boot/u-boot-common_2016.11.inc
rename to meta/recipes-bsp/u-boot/u-boot-common_2017.01.inc
index 84adc1c56a..df24c853dd 100644
--- a/meta/recipes-bsp/u-boot/u-boot-common_2016.11.inc
+++ b/meta/recipes-bsp/u-boot/u-boot-common_2017.01.inc
@@ -7,7 +7,7 @@ PE = "1"
 
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "29e0cfb4f77f7aa369136302cee14a91e22dca71"
+SRCREV = "a705ebc81b7f91bbd0ef7c634284208342901149"
 
 SRC_URI = "git://git.denx.de/u-boot.git"
 
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.11.bb 
b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2017.01.bb
similarity index 100%
rename from meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.11.bb
rename to meta/recipes-bsp/u-boot/u-boot-fw-utils_2017.01.bb
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.11.bb 
b/meta/recipes-bsp/u-boot/u-boot-mkimage_2017.01.bb
similarity index 100%
rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2016.11.bb
rename to meta/recipes-bsp/u-boot/u-boot-mkimage_2017.01.bb
diff --git a/meta/recipes-bsp/u-boot/u-boot_2016.11.bb 
b/meta/recipes-bsp/u-boot/u-boot_2017.01.bb
similarity index 100%
rename from meta/recipes-bsp/u-boot/u-boot_2016.11.bb
rename to meta/recipes-bsp/u-boot/u-boot_2017.01.bb
-- 
2.11.0

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

[OE-core] [PATCH v2 2/3] ccache.bbclass: add trimming tasks

[Yannick Gicquel]

These can be use to trim ccache directories to default cache size (1G)
without deleting the whole dirs.

Signed-off-by: Yannick Gicquel 
---
 meta/classes/ccache.bbclass | 17 +
 1 file changed, 17 insertions(+)

diff --git a/meta/classes/ccache.bbclass b/meta/classes/ccache.bbclass
index b6643a1..0bad4d4 100644
--- a/meta/classes/ccache.bbclass
+++ b/meta/classes/ccache.bbclass
@@ -12,3 +12,20 @@ ccache_init() {
 fi
 }
 do_configure[postfuncs] += "ccache_init"
+
+addtask ccache_trim
+do_ccache_trim[nostamp] = "1"
+do_ccache_trim() {
+if [ -n "${CCACHE}" -a -d "${CCACHE_DIR}" ]; then
+${CCACHE} -M 1G
+${CCACHE} -c
+fi
+}
+
+addtask ccache_trimall after do_ccache_trim
+do_ccache_trimall[recrdeptask] = "do_ccache_trimall do_ccache_trim"
+do_ccache_trimall[recideptask] = "do_${BB_DEFAULT_TASK}"
+do_ccache_trimall[nostamp] = "1"
+do_ccache_trimall() {
+:
+}
-- 
1.9.1

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

[OE-core] [PATCH v2 1/3] ccache.bbclass: enable max size setup for ccache dir

[Yannick Gicquel]

ccache directories are limited to 1G by default.
This patch enables the configuration of their limits, and as default
location is TMPDIR, it proposes a size limit to "0" (unlimited).

The setup can be overloaded in local.conf by setting
CCACHE_MAX_SIZE to a custom value if needed.

Signed-off-by: Yannick Gicquel 
---
 meta/classes/ccache.bbclass | 8 
 1 file changed, 8 insertions(+)

diff --git a/meta/classes/ccache.bbclass b/meta/classes/ccache.bbclass
index 93fcaca..b6643a1 100644
--- a/meta/classes/ccache.bbclass
+++ b/meta/classes/ccache.bbclass
@@ -1,6 +1,14 @@
 CCACHE = "${@bb.utils.which(d.getVar('PATH'), 'ccache') and 'ccache '}"
 export CCACHE_DIR ?= "${TMPDIR}/ccache/${MULTIMACH_HOST_SYS}/${PN}"
 CCACHE_DISABLE[unexport] = "1"
+CCACHE_MAX_SIZE ?= "0"
 
 do_configure[dirs] =+ "${CCACHE_DIR}"
 do_kernel_configme[dirs] =+ "${CCACHE_DIR}"
+
+ccache_init() {
+if [ -n "${CCACHE}" ]; then
+${CCACHE} -M ${CCACHE_MAX_SIZE}
+fi
+}
+do_configure[postfuncs] += "ccache_init"
-- 
1.9.1

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

[OE-core] [PATCH v2 3/3] cmake.bbclass: enable usage of ccache

[Yannick Gicquel]

This allows ccache usage for recipes which inherit cmake.bbclass.
Since cmake v3.4, it can be enabled using some "-D" options.

Here below are some metrics while using it for webkitgtk recipe.
(machine is a 4x core i7 @ 3.4GHz)

$ bitbake -c fetchall webkitgtk
$ time bitbake webkitgtk

real56m25.191s
user359m32.003s
sys 34m49.356s

$ bitbake -c clean webkitgtk
$ bitbake -c cleansstate webkitgtk
$ time bitbake webkitgtk

real25m19.298s
user17m57.861s
sys 6m20.157s

Signed-off-by: Yannick Gicquel 
---
 meta/classes/cmake.bbclass | 14 +-
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/meta/classes/cmake.bbclass b/meta/classes/cmake.bbclass
index 9e74599..737a4e0 100644
--- a/meta/classes/cmake.bbclass
+++ b/meta/classes/cmake.bbclass
@@ -4,12 +4,9 @@ OECMAKE_SOURCEPATH ?= "${S}"
 DEPENDS_prepend = "cmake-native "
 B = "${WORKDIR}/build"
 
-# We need to unset CCACHE otherwise cmake gets too confused
-CCACHE = ""
-
 # C/C++ Compiler (without cpu arch/tune arguments)
-OECMAKE_C_COMPILER ?= "`echo ${CC} | sed 's/^\([^ ]*\).*/\1/'`"
-OECMAKE_CXX_COMPILER ?= "`echo ${CXX} | sed 's/^\([^ ]*\).*/\1/'`"
+OECMAKE_C_COMPILER ?= "${@'${CC}'.replace('${CCACHE}','',1).split(' ')[0]}"
+OECMAKE_CXX_COMPILER ?= "${@'${CXX}'.replace('${CCACHE}','',1).split(' ')[0]}"
 OECMAKE_AR ?= "${AR}"
 
 # Compiler flags
@@ -108,9 +105,16 @@ cmake_do_configure() {
OECMAKE_SITEFILE=""
fi
 
+   if [ -n "${CCACHE}" ]; then
+   OECMAKE_CCACHE="-DCMAKE_C_COMPILER_LAUNCHER=${CCACHE} 
-DCMAKE_CXX_COMPILER_LAUNCHER=${CCACHE}"
+   else
+   OECMAKE_CCACHE=""
+   fi
+
cmake \
  ${OECMAKE_SITEFILE} \
  ${OECMAKE_SOURCEPATH} \
+ ${OECMAKE_CCACHE} \
  -DCMAKE_INSTALL_PREFIX:PATH=${prefix} \
  -DCMAKE_INSTALL_BINDIR:PATH=${@os.path.relpath(d.getVar('bindir'), 
d.getVar('prefix'))} \
  -DCMAKE_INSTALL_SBINDIR:PATH=${@os.path.relpath(d.getVar('sbindir'), 
d.getVar('prefix'))} \
-- 
1.9.1

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

Re: [OE-core] [PATCH v2] elfutils 0.166->0.167

[Burton, Ross]

On 23 December 2016 at 02:16, Zheng Ruoqin 
wrote:

> 2) Delete 5 patches below, since they are integrated upstream.
> elfutils-0.166/0001-elf_getarsym-Silence-Werror-maybe-
> uninitialized-fals.patch
> elfutils-0.166/0001-remove-the-unneed-checking.patch
> elfutils-0.166/aarch64_uio.patch
> elfutils-0.166/fixheadercheck.patch
> elfutils-0.166/shadow.patch
>

Picking on a random patch, fixheadercheck.patch.  The patch is just this:

-if (unlikely (! powerof2 (shdr->sh_addralign))
-|| unlikely ((shdr->sh_addralign ?: 1) < sh_align))
+if (shdr->sh_size && (unlikely (! powerof2
(shdr->sh_addralign))
+|| unlikely ((shdr->sh_addralign ?: 1) < sh_align)))

And upstream the code is:

  if (unlikely (! powerof2 (shdr->sh_addralign))
  || unlikely ((shdr->sh_addralign ?: 1) < sh_align))

Have there been other changes to mean this patch isn't needed any more? Or
should it not be deleted after all?

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

Re: [OE-core] [PATCH 13/14] kern-tools: kconf_check: flag CONFIG options with invalid spaces

[Bruce Ashfield]

On 2017-01-19 09:06 AM, Burton, Ross wrote:


On 18 January 2017 at 13:54, Bruce Ashfield
> wrote:

-SRCREV = "be79004cb2535e79271822b3d74eda5ba12233d2"
+SRCREV = "bf2942f7559d114e96f7c7f1287bf7e5120632a"


Somehow you managed to drop a digit from the SHA, I've a fix locally.


hah. I would have just nak'd it.

That is odd ... must have been during my rebase and edit I did just
before sending.

Thanks though!

Bruce





Ross


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

Re: [OE-core] [PATCH 13/14] kern-tools: kconf_check: flag CONFIG options with invalid spaces

[Burton, Ross]

On 18 January 2017 at 13:54, Bruce Ashfield 
wrote:

> -SRCREV = "be79004cb2535e79271822b3d74eda5ba12233d2"
> +SRCREV = "bf2942f7559d114e96f7c7f1287bf7e5120632a"
>

Somehow you managed to drop a digit from the SHA, I've a fix locally.

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

Re: [OE-core] [PATCH 1/1] openssh: upgrade to 7.4p1

[Jussi Kukkonen]

On 19 January 2017 at 13:52, Burton, Ross  wrote:

> You're removing this patch:
>
> On 19 January 2017 at 09:50, Dengke Du  wrote:
>
>> --- a/meta/recipes-connectivity/openssh/openssh/add-test-support
>> -for-busybox.patch
>> +++ /dev/null
>> @@ -1,69 +0,0 @@
>> -Adjust test cases to work with busybox.
>> -
>> -- Replace dd parameter "obs" with "bs".
>> -- Replace "head -" with "head -n ".
>> -
>> -Signed-off-by: Maxin B. John 
>> -Upstream-Status: Pending
>> -
>> -Index: openssh-6.8p1/regress/cipher-speed.sh
>> -===
>>  openssh-6.8p1.orig/regress/cipher-speed.sh
>> -+++ openssh-6.8p1/regress/cipher-speed.sh
>> -@@ -17,7 +17,7 @@ for c in `${SSH} -Q cipher`; do n=0; for
>> -   printf "%-60s" "$c/$m:"
>> -   ( ${SSH} -o 'compression no' \
>> -   -F $OBJ/ssh_proxy -2 -m $m -c $c somehost \
>> --  exec sh -c \'"dd of=/dev/null obs=32k"\' \
>> -+  exec sh -c \'"dd of=/dev/null bs=32k"\' \
>>
>
> But upstream still has exec sh -c \'"dd of=/dev/null obs=32k"\', so this
> patch is still needed.
>
> Please always explain why patches are being removed in the commit message.
>

There's also lots of changes to the remaining patches. Some of them look
weird -- possibly they're just whitespace changes? Please mention what the
patch changes are in the commit message.

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

Re: [OE-core] [PATCH] bash 4.3.30 -> 4.4

[Burton, Ross]

This doesn't apply to master for me, can you check please?

Ross

On 19 January 2017 at 02:52, Zheng Ruoqin 
wrote:

> 1) Upgrade bash from 4.3.30 to 4.4
> 2) Delete 1 patche below, since they are integrated upstream.
>fix-run-intl.patch
> 3) Modify 2 patches below to make it compatible with new version
>fix-run-coproc-run-heredoc-run-execscript-run-test-f.patch
>test-output.patch
> 4) Add 1 patche below to solve pkgconfig sanity error
>0001-patch-to-obtain-pkgconfig-sanity.patch
>
> Signed-off-by: Zheng Ruoqin 
> ---
>  .../0001-patch-to-obtain-pkgconfig-sanity.patch|  26 
>  ...roc-run-heredoc-run-execscript-run-test-f.patch | 137
> ++---
>  meta/recipes-extended/bash/bash/fix-run-intl.patch | 110
> -
>  meta/recipes-extended/bash/bash/test-output.patch  |  20 ++-
>  meta/recipes-extended/bash/bash_4.3.30.bb  |  71 ---
>  meta/recipes-extended/bash/bash_4.4.bb |  38 ++
>  6 files changed, 90 insertions(+), 312 deletions(-)
>  create mode 100644 meta/recipes-extended/bash/bash/0001-patch-to-obtain-
> pkgconfig-sanity.patch
>  delete mode 100644 meta/recipes-extended/bash/bash/fix-run-intl.patch
>  delete mode 100644 meta/recipes-extended/bash/bash_4.3.30.bb
>  create mode 100644 meta/recipes-extended/bash/bash_4.4.bb
>
> diff --git 
> a/meta/recipes-extended/bash/bash/0001-patch-to-obtain-pkgconfig-sanity.patch
> b/meta/recipes-extended/bash/bash/0001-patch-to-obtain-
> pkgconfig-sanity.patch
> new file mode 100644
> index 000..031eb90
> --- /dev/null
> +++ b/meta/recipes-extended/bash/bash/0001-patch-to-obtain-
> pkgconfig-sanity.patch
> @@ -0,0 +1,26 @@
> +Subject: [PATCH] patch to obtain pkgconfig sanity
> +
> +Drop the paths from the pkgconfig file bash.pc.in which will cause
> pkgconfig sanity error below:
> +ERROR: QA Issue: bash.pc failed sanity test (tmpdir) in path
> /yocto/work001/fnst/zrq/community/poky/build/tmp/work/i586-poky-linux/bash
>   /4.4-r0/sysroot-destdir/usr/lib/pkgconfig [pkgconfig]
> +
> +Signed-off-by: Zheng Ruoqin 
> +---
> + support/bash.pc.in | 2 --
> + 1 file changed, 2 deletions(-)
> +
> +diff --git a/support/bash.pc.in b/support/bash.pc.in
> +index 6009659..871fa9f 100644
> +--- a/support/bash.pc.in
>  b/support/bash.pc.in
> +@@ -12,8 +12,6 @@ LOCAL_CFLAGS = @LOCAL_CFLAGS@
> + LOCAL_DEFS = @LOCAL_DEFS@
> + CCFLAGS = ${LOCAL_DEFS} ${LOCAL_CFLAGS}
> +
> +-CC = @CC@
> +-SHOBJ_CC = @SHOBJ_CC@
> + SHOBJ_CFLAGS = @SHOBJ_CFLAGS@
> + SHOBJ_LD = @SHOBJ_LD@
> + SHOBJ_LDFLAGS = @SHOBJ_LDFLAGS@
> +--
> +2.7.4
> +
> diff --git a/meta/recipes-extended/bash/bash/fix-run-coproc-run-
> heredoc-run-execscript-run-test-f.patch b/meta/recipes-extended/bash/
> bash/fix-run-coproc-run-heredoc-run-execscript-run-test-f.patch
> index 7f099ae..d6beaff 100644
> --- a/meta/recipes-extended/bash/bash/fix-run-coproc-run-
> heredoc-run-execscript-run-test-f.patch
> +++ b/meta/recipes-extended/bash/bash/fix-run-coproc-run-
> heredoc-run-execscript-run-test-f.patch
> @@ -20,150 +20,35 @@ the test suite should not be run as root
>  Upstream-Status: Pending
>
>  Signed-off-by: Hongxu Jia 
> +Signed-off-by: Zheng Ruoqin 
>  ---
> - tests/coproc.right   |  5 +
> - tests/coproc.tests   | 30 +-
> - tests/heredoc.right  |  5 ++---
> - tests/heredoc3.sub   |  3 ++-
> - tests/run-execscript |  3 ++-
> - tests/run-test   |  3 ++-
> - 6 files changed, 34 insertions(+), 15 deletions(-)
> + tests/run-execscript | 3 ++-
> + tests/run-test   | 3 ++-
> + 2 files changed, 4 insertions(+), 2 deletions(-)
>
> -diff --git a/tests/coproc.right b/tests/coproc.right
> -index 6d9deaa..94b001c 100644
>  a/tests/coproc.right
> -+++ b/tests/coproc.right
> -@@ -1,11 +1,8 @@
> --84575
> - 63 60
> - a b c
> --84577
> - 63 60
> - flop
> --./coproc.tests: line 22: 84577 Terminated  coproc REFLECT {
> cat -; }
> --84579
> -+coproc.tests: REFLECT: status 143
> - 63 60
> - FOO
> - 63 60
> -diff --git a/tests/coproc.tests b/tests/coproc.tests
> -index 8be3563..d347eb7 100644
>  a/tests/coproc.tests
> -+++ b/tests/coproc.tests
> -@@ -1,6 +1,13 @@
> -+: ${TMPDIR:=/tmp}
> -+TMPOUT=${TMPDIR}/coproc-wait-$BASHPID
> -+
> - coproc { echo a b c; sleep 2; }
> -
> --echo $COPROC_PID
> -+case $COPROC_PID in
> -+[0-9]*)   ;;
> -+*)echo COPROC_PID not integer ;;
> -+esac
> -+
> - echo ${COPROC[@]}
> -
> - read LINE <&${COPROC[0]}
> -@@ -10,7 +17,11 @@ wait $COPROC_PID
> -
> - coproc REFLECT { cat - ; }
> -
> --echo $REFLECT_PID
> -+case $REFLECT_PID in
> -+[0-9]*)   ;;
> -+*)echo REFLECT_PID not integer ;;
> -+esac
> -+
> - echo ${REFLECT[@]}
> -
> - echo flop >&${REFLECT[1]}
> -@@ -18,12 +29,21 @@ read LINE <&${REFLECT[0]}
> -
> - echo $LINE
> -
> --kill $REFLECT_PID
> --wait $REFLECT_PID
> -+{ sleep 1; kill $REFLECT_PID; } &
> -+wait 

Re: [OE-core] [PATCH 3/3] selftest/containerimage.py: Add container IMAGE_FSTYPES test

[Burton, Ross]

On 18 January 2017 at 18:31, Randy Witt 
wrote:

> +expected_files = [
> +'./',
> +'./bin/',
> +'./bin/theapp',
> +'./etc/',
> +'./etc/default/',
> +'./etc/default/postinst',
> +'./etc/ld.so.cache',
> +'./etc/timestamp',
> +'./etc/version',
> +'./run/',
> +'./var/',
> +'./var/cache/',
> +'./var/cache/ldconfig/',
> +'./var/cache/ldconfig/aux-cache',
> +'./var/cache/opkg/',
> +'./var/lib/',
> +'./var/lib/opkg/'
> +]
>
> These should use ${bindir} etc so they're resilient to distros changing
the paths (please use get_bb_vars() to only invoke bitbake once).

+image = os.path.join(self.builddir, "tmp/deploy/images/qemux86/
> container-test-image-qemux86.tar.bz2")
>

Don't hardcode this path, use ${DEPLOY_DIR_IMAGE} (fetch it with the paths
above for speed) which will get you to tmp/deploy/images/[machine].  Again,
these paths can change (poky sets TMPDIR to tmp, nodistro uses
tmp-${TCLIBC}).

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