Re: [OE-core] [PATCH 1/1] runqemu: replace bashism with working shell idiom

2012-05-14 Thread Scott Garman
On 05/14/2012 03:49 PM, Peter Seebach wrote: The =~ operator is not one of my favorites, not just due to portability issues, but because it's not well known, and a lot of people might not expect a regex operator. The canonical shell idiom for this is to use case with alternation and wildcards.

Re: [OE-core] [PATCH 2/2] rootfs_rpm.bbclass: save rpmlib rather than remove it

2012-05-14 Thread Robert Yang
On 05/14/2012 05:44 PM, Koen Kooi wrote: Op 14 mei 2012, om 10:11 heeft Robert Yang het volgende geschreven: The rpmlib was removed when core-image-minimal, Please replace 'core-image-minimal' with something like: images that add "remove_packaging_data_files ; " to ROOTFS_POSTPRO

[OE-core] [PATCH 0/1] ofono: Add python-pygobject to ofono-tests package DEPENDS

2012-05-14 Thread Darren Hart
The following changes since commit 36d1717e2ad4ca1620ee9f01b524b5ff2f499b26: classes/rootfs_*: fix splitting package dependency strings (2012-05-13 20:07:24 +0100) are available in the git repository at: git://git.yoctoproject.org/user-contrib/dvhart/oe-core dvhart/misc http://git.yoctopr

[OE-core] [PATCH 1/1] ofono: Add python-pygobject to ofono-tests package DEPENDS

2012-05-14 Thread Darren Hart
Some of the ofono-tests python scripts, such as test-modem and receive-sms (22 in total) import gobject. Make the ofono-tests package depend on python-pygobject. Tested on the meta-intel fri2 BSP in a core-image-sato image. Assorted ofono tests run correctly. Signed-off-by: Darren Hart CC: Dongx

Re: [OE-core] [PATCH 1/6] runqemu: Use OE_TMPDIR

2012-05-14 Thread Mark Hatle
On 5/14/12 5:51 PM, Marko Lindqvist wrote: On 8 May 2012 02:56, Scott Garman wrote: I can understand why we're trying to ensure our build system doesn't require /bin/sh to be bash, but I think support scripts like runqemu might be a special case. What do other people in the community think of

Re: [OE-core] [PATCH 1/6] runqemu: Use OE_TMPDIR

2012-05-14 Thread Marko Lindqvist
On 8 May 2012 02:56, Scott Garman wrote: > > I can understand why we're trying to ensure our build system doesn't require > /bin/sh to be bash, but I think support scripts like runqemu might be a > special case. > > What do other people in the community think of this? The runqemu script > isn't tr

[OE-core] [PATCH 1/1] runqemu: replace bashism with working shell idiom

2012-05-14 Thread Peter Seebach
The =~ operator is not one of my favorites, not just due to portability issues, but because it's not well known, and a lot of people might not expect a regex operator. The canonical shell idiom for this is to use case with alternation and wildcards. As a side note, if you are matching anything co

[OE-core] [PATCH 0/1] runqemu: fix the tests for core-image-*

2012-05-14 Thread Peter Seebach
This replaces the =~ tests with a dash-friendly shell idiom which is, IMHO, comparably readable (I actually like it better) and also works on more versions of bash. A side note: Do we really need a more complicated test instead of just "*core-image-*"? The following changes since commit 36d1717e

Re: [OE-core] [PATCH 1/6] runqemu: Use OE_TMPDIR

2012-05-14 Thread Khem Raj
On Mon, May 14, 2012 at 3:34 PM, Mark Hatle wrote: > --- a/scripts/runqemu > +++ b/scripts/runqemu > @@ -300,14 +300,16 @@ findimage() { >     # recently created one is the one we most likely want to boot. >     filenames=`ls -t $where/*-image*$machine.$extension 2>/dev/null | xargs` >     for nam

Re: [OE-core] [PATCH 0/2] RFC - CCACHE_DIR to not impact sstate

2012-05-14 Thread Khem Raj
On Mon, May 14, 2012 at 2:50 PM, Jason Wessel wrote: > > Regardless of the state of the CCACHE_DIR the .o will be the same. and .o are part of checksum right ? ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.lin

Re: [OE-core] [PATCH 1/6] runqemu: Use OE_TMPDIR

2012-05-14 Thread Mark Hatle
On 5/8/12 2:07 AM, Peter Seebach wrote: On Mon, 7 May 2012 16:56:11 -0700 Scott Garman wrote: From what I can tell, the =~ regex operator is a bashism. It's also one that helps a lot with the code readability. So now that we're faced with re-writing the script to avoid using that operator,

Re: [OE-core] [PATCH 0/2] bitbake.conf: gracefully work with ccache environment variables

2012-05-14 Thread Jason Wessel
On 05/14/2012 05:00 PM, Marko Lindqvist wrote: > As you can see from my answer to last part I quote, I'm not against > your patch, so don't read first parts of this email with that mindset. > They are more generic discussion about ccache caches. No worries. > > On 14 May 2012 23:58, Jason Wesse

Re: [OE-core] [PATCH 0/2] bitbake.conf: gracefully work with ccache environment variables

2012-05-14 Thread Marko Lindqvist
As you can see from my answer to last part I quote, I'm not against your patch, so don't read first parts of this email with that mindset. They are more generic discussion about ccache caches. On 14 May 2012 23:58, Jason Wessel wrote: > > This is only an issue if the compiler has changed.  In my

Re: [OE-core] [PATCH 0/2] RFC - CCACHE_DIR to not impact sstate

2012-05-14 Thread Jason Wessel
On 05/14/2012 04:33 PM, Khem Raj wrote: > On Sun, May 13, 2012 at 7:28 PM, Jason Wessel > wrote: >> I am not exactly sure how to fix this, so I thought I might ask in the >> form of a working patch. The problem is that I want to use an >> external CCACHE_DIR on some build servers, but use the def

Re: [OE-core] [PATCH 0/2] RFC - CCACHE_DIR to not impact sstate

2012-05-14 Thread Khem Raj
On Sun, May 13, 2012 at 7:28 PM, Jason Wessel wrote: > I am not exactly sure how to fix this, so I thought I might ask in the > form of a working patch.  The problem is that I want to use an > external CCACHE_DIR on some build servers, but use the defaults on > others.  Ultimately the sstate sums

[OE-core] [PATCH v2 1/1] bitbake.conf: Add CCACHE_DIR to BB_HASHBASE_WHITELIST

2012-05-14 Thread Jason Wessel
The location of the ccache should not affect the sum in the sstate files. The end user should be free to use an external ccache via defining CCACHE_DIR to a new value in local.conf or using the default per package local ccache in the temp directory. Signed-off-by: Jason Wessel --- meta/conf/bit

Re: [OE-core] [PATCH 0/2] bitbake.conf: gracefully work with ccache environment variables

2012-05-14 Thread Jason Wessel
On 05/14/2012 12:18 PM, Marko Lindqvist wrote: > On 14 May 2012 19:09, Jason Wessel wrote: >> The end user of oe-core should be free to turn off ccache, or use an >> external ccache without impacting the sstate sums. > > ... > > External cache makes sense for anything being built by system > c

Re: [OE-core] [PATCH 1/1] kmod: fix installation path of pkgconfig files

2012-05-14 Thread Otavio Salvador
On Mon, May 14, 2012 at 3:30 PM, Otavio Salvador wrote: > The pkgconfig files need to be installed in ${libdir} instead of > ${base_libdir}. > > Signed-off-by: Otavio Salvador This patch is near but still doesn't work as expected. I am working on that. -- Otavio Salvador                     

[OE-core] [PATCH 1/1] kmod: fix installation path of pkgconfig files

2012-05-14 Thread Otavio Salvador
The pkgconfig files need to be installed in ${libdir} instead of ${base_libdir}. Signed-off-by: Otavio Salvador --- meta/recipes-kernel/kmod/kmod_git.bb |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/meta/recipes-kernel/kmod/kmod_git.bb b/meta/recipes-kernel/kmod/

Re: [OE-core] [PATCH 0/2] bitbake.conf: gracefully work with ccache environment variables

2012-05-14 Thread Chris Larson
On Mon, May 14, 2012 at 10:18 AM, Marko Lindqvist wrote: >> I'll send a patch just to add CCACHE_DIR to the BB_HASHBASE_WHITELIST. > >  External cache makes sense for anything being built by system > compiler. Caching results of anything produced by OE-built compilers > to user's normal cache is p

Re: [OE-core] [PATCH 0/2] bitbake.conf: gracefully work with ccache environment variables

2012-05-14 Thread Marko Lindqvist
On 14 May 2012 19:09, Jason Wessel wrote: > The end user of oe-core should be free to turn off ccache, or use an > external ccache without impacting the sstate sums. ... > I'll send a patch just to add CCACHE_DIR to the BB_HASHBASE_WHITELIST. External cache makes sense for anything being buil

Re: [OE-core] [PATCH 0/2] bitbake.conf: gracefully work with ccache environment variables

2012-05-14 Thread Jason Wessel
On 05/14/2012 11:09 AM, Jason Wessel wrote: > The end user of oe-core should be free to turn off ccache, or use an > external ccache without impacting the sstate sums. > > Example of using a different ccache directory via local.conf > > CCACHE_DIR = "/tmp/ccache" > > Example of turning off ccach

[OE-core] [PATCH 2/3] tcmode-external-csl: avoid hardcoding a list of ia32 archs

2012-05-14 Thread Christopher Larson
From: Christopher Larson Signed-off-by: Christopher Larson --- meta/conf/distro/include/tcmode-external-csl.inc |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/meta/conf/distro/include/tcmode-external-csl.inc b/meta/conf/distro/include/tcmode-external-csl.inc inde

[OE-core] [PATCH 3/3] tcmode-external-csl: handle TUNE_PKGARCH=core2

2012-05-14 Thread Christopher Larson
From: Christopher Larson Signed-off-by: Christopher Larson --- meta/conf/distro/include/tcmode-external-csl.inc |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/meta/conf/distro/include/tcmode-external-csl.inc b/meta/conf/distro/include/tcmode-external-csl.inc index dea

[OE-core] [PATCH 0/3] Minor tcmode-external-csl fixes

2012-05-14 Thread Christopher Larson
From: Christopher Larson The following changes since commit 36d1717e2ad4ca1620ee9f01b524b5ff2f499b26: classes/rootfs_*: fix splitting package dependency strings (2012-05-13 20:07:24 +0100) are available in the git repository at: https://github.com/kergoth/oe-core external-csl-minor-fixes

[OE-core] [PATCH 1/3] tcmode-external-csl: drop unnecessary cppflags/ldflags

2012-05-14 Thread Christopher Larson
From: Christopher Larson Signed-off-by: Christopher Larson --- meta/conf/distro/include/tcmode-external-csl.inc |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/meta/conf/distro/include/tcmode-external-csl.inc b/meta/conf/distro/include/tcmode-external-csl.inc index 3

[OE-core] [PATCH 1/2] bitbake.conf: Move default hash policy initialization before conf file inclusion

2012-05-14 Thread Jason Wessel
It is desirable to be able to augment the default hash policy from local.conf if adding new exported variables that do not impact the end result of a build, and in order to do this the default initialization must occur prior to the conf file inclusion. This is an example of a change that should no

[OE-core] [PATCH 2/2] bitbake.conf: Add CCACHE_DIR to BB_HASHBASE_WHITELIST

2012-05-14 Thread Jason Wessel
The location of the ccache should not affect the sum in the sstate files. The end user should be free to use an external ccache via defining CCACHE_DIR to a new value in local.conf or the default per package local ccache in the temp directory. Signed-off-by: Jason Wessel --- meta/conf/bitbake.c

[OE-core] [PATCH 0/2] bitbake.conf: gracefully work with ccache environment variables

2012-05-14 Thread Jason Wessel
The end user of oe-core should be free to turn off ccache, or use an external ccache without impacting the sstate sums. Example of using a different ccache directory via local.conf CCACHE_DIR = "/tmp/ccache" Example of turning off ccache: export CCACHE_DISABLE = "1" BB_HASHBASE_WHITELIST += "CC

Re: [OE-core] [PATCH 1/1] kmod: fix installation path of pkgconfig files

2012-05-14 Thread Otavio Salvador
On Mon, May 14, 2012 at 11:06 AM, Koen Kooi wrote: > > Op 14 mei 2012, om 15:47 heeft Otavio Salvador het volgende geschreven: > >> The pkgconfig files need to be installed in ${libdir} instead of >> ${base_libdir}. > > Thanks for the patch, I was working on something similar :) I talked with the

Re: [OE-core] [PATCH 2/2] gdk-pixbuf: Add --with-x11 when building lsb image

2012-05-14 Thread Mark Hatle
On 5/12/12 12:41 AM, Saul Wold wrote: On 05/11/2012 12:23 AM, Xiaofeng Yan wrote: From: Xiaofeng Yan LSB Test Suite complains "No library libgdk_pixbuf_xlib-2.0.so.0" \ because of having "--without-x11" for gdk-pixbuf_2.24.1.bb. Use the linuxstdbase override and also make it conditional on x11

Re: [OE-core] [PATCH 1/1] kmod: fix installation path of pkgconfig files

2012-05-14 Thread Koen Kooi
Op 14 mei 2012, om 15:47 heeft Otavio Salvador het volgende geschreven: > The pkgconfig files need to be installed in ${libdir} instead of > ${base_libdir}. Thanks for the patch, I was working on something similar :) I talked with the kmod and udev people and they asked why we want kmod libs in

[OE-core] [PATCH 0/1] qemu: Add option to disable GL acceleration, Edwin, May14, 2012

2012-05-14 Thread edwin . zhai
From: Zhai Edwin All, This patch add an PACKAGECONFIG in qemu to disable GL acceleration: * By default configure try best to enable GL acceleration and fail when missing host dependency(libSDL and libGL). * End user can also choose to turn off GL capability, thus remove the host dependence(li

[OE-core] [PATCH 1/1] qemu: Add an option to remove host sdl/gl checking

2012-05-14 Thread edwin . zhai
From: Zhai Edwin Add an PACKAGECONFIG in qemu to disable GL acceleration: * By default configure try best to enable GL acceleration and fail when missing host dependency(libSDL and libGL). * End user can also choose to turn off GL capability, thus remove the host dependence in building. [YOC

[OE-core] [PATCH 1/1] kmod: fix installation path of pkgconfig files

2012-05-14 Thread Otavio Salvador
The pkgconfig files need to be installed in ${libdir} instead of ${base_libdir}. Signed-off-by: Otavio Salvador --- meta/recipes-kernel/kmod/kmod_git.bb |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/meta/recipes-kernel/kmod/kmod_git.bb b/meta/recipes-kernel/kmod/

Re: [OE-core] [PATCH 0/2] RFC - CCACHE_DIR to not impact sstate

2012-05-14 Thread Jason Wessel
On 05/13/2012 09:47 PM, Chris Larson wrote: > On Sun, May 13, 2012 at 7:28 PM, Jason Wessel > wrote: >> I am not exactly sure how to fix this, so I thought I might ask in the >> form of a working patch. The problem is that I want to use an >> external CCACHE_DIR on some build servers, but use the

[OE-core] [PATCH] wpa-supplicant: don't reset ${S}

2012-05-14 Thread Jesse Zhang
If ${S} is set to the subdir wpa_supplicant, patches with changes in other subdirs can't be applied. Fix ${S} so patches can be applied in the normal way. Signed-off-by: Jesse Zhang --- I was adding a patch to wpa-supplicant using bbappend in our layer, but it was impossible since ${S} was overr

[OE-core] [PATCH] wpa-supplicant: don't reset ${S}

2012-05-14 Thread Jesse Zhang
If ${S} is set to the subdir wpa_supplicant, patches with changes in other subdirs can't be applied. Fix ${S} so patches can be applied in the normal way. Signed-off-by: Jesse Zhang --- [Resending. The last copy didn't appear after two hours. Sorry for the inconvenience.] I was adding a patch t

Re: [OE-core] [PATCH 2/2] rootfs_rpm.bbclass: save rpmlib rather than remove it

2012-05-14 Thread Koen Kooi
Op 14 mei 2012, om 10:11 heeft Robert Yang het volgende geschreven: > The rpmlib was removed when core-image-minimal, Please replace 'core-image-minimal' with something like: images that add "remove_packaging_data_files ; " to ROOTFS_POSTPROCESS_COMMAND That points out the actual prob

[OE-core] [PATCH 2/2] rootfs_rpm.bbclass: save rpmlib rather than remove it

2012-05-14 Thread Robert Yang
The rpmlib was removed when core-image-minimal, which would make the increment rpm image generation doesn't work in the second build, since list_installed_packages would get incorrect value in the second build, move the rpmlib to ${T} rather than remove it, and move it back when INC_RPM_IMAGE_GEN =

[OE-core] [PATCH 0/2] V2 Fix incremental rpm image generation

2012-05-14 Thread Robert Yang
Changes of V2: * Save rpmlib for core-image-minimal rather than remove it // Robert The following changes since commit 12b4543ac9f54c0db0ee4a7ee546a71946a051f8: classes/rootfs_*: fix splitting package dependency strings (2012-05-13 20:09:56 +0100) are available in the git repository at: gi

[OE-core] [PATCH 1/2] package_rpm.bbclass: Fix incremental rpm image generation

2012-05-14 Thread Robert Yang
Fix the incremental rpm image generation, it didn't work since the code has been changed. The btmanifest should have a ".manifest" suffix, so that it can be moved to ${T} by rootfs_rpm.bbclass: mv ${IMAGE_ROOTFS}/install/*.manifest ${T}/ Note: The locale pkgs would always be re-installed. [YOCTO

Re: [OE-core] [PATCH 1/1] archiver.bbclass: Add default set

2012-05-14 Thread Xiaofeng Yan
On 2012年05月11日 22:14, Chris Larson wrote: On Fri, May 11, 2012 at 3:13 AM, wrote: diff --git a/meta/classes/archiver.bbclass b/meta/classes/archiver.bbclass index 59b58f4..a720db2 100644 --- a/meta/classes/archiver.bbclass +++ b/meta/classes/archiver.bbclass @@ -6,6 +6,10 @@ ARCHIVE_EXCLUDE_FRO