Re: [OE-core] [PATCH 0/1] [denzil] linux-yocto: make validate_branches handle non BSP branches

2012-08-21 Thread Bruce Ashfield

On 12-08-22 1:27 AM, Tom Zanussi wrote:

On Mon, 2012-08-13 at 13:10 -0400, Bruce Ashfield wrote:

Scott,

Here's a fix for validate_branches that comes from a pending master
change that I've been running for 3 weeks, and one that TomZ has tested
in his yocto BSP work on denzil.

It fixes some BSP use cases that involve setting a specific SRCREV
on a base branch, so that a BSP branch that doesn't yet exist can
be created off of that known base. Previously only an existing BSP
branch could have it's SRCREV restricted.



Hi Scott,

I don't see this in denzil - do you have it queued for the next pull
request, or did it get missed?


I was wondering the same thing, in particular, since I just made a minor
tweak to this in master (which I'll send out tomorrow). The attached
patch should also be merged to denzil.

Bruce



Tom


cc: Tom Zanussi

Cheers,

Bruce

The following changes since commit 73cdebf60df225ee10f2eb215935be3b61e1b831:

   documentation/dev-manual/dev-manual-kernel-appendix.xml: Add note about 
conflict (2012-06-29 15:54:26 +0100)

are available in the git repository at:
   git://git.pokylinux.org/poky-contrib zedd/kernel-denzil
   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=zedd/kernel-denzil

Bruce Ashfield (1):
   linux-yocto: allow do_validate_branches to handle all branches

  meta/classes/kernel-yocto.bbclass |  119 +
  1 files changed, 80 insertions(+), 39 deletions(-)








0001-kernel-yocto-set-master-branch-to-a-defined-SRCREV.patch
Description: application/mbox
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [discussion] perf: specify SLANG_INC dir for perf

2012-08-21 Thread Bruce Ashfield

On 12-08-21 11:01 PM, Liang Li wrote:

On 2012-08-21 21:07, Bruce Ashfield  wrote:

On 12-08-21 01:08 AM, Liang Li wrote:

On 2012-08-20 22:48, Bruce Ashfield   wrote:

On 12-08-17 09:05 AM, Liang Li wrote:

On 2012-08-17 21:01, Liang Liwrote:

On 2012-08-17 18:53, Richard Purdie
wrote:

On Fri, 2012-08-17 at 18:00 +0800, Liang Li wrote:

I am totally confused, you mentioned 'general kernel do_install', I
assume it's oe-core kernel.bbclass concept. Then you mentioned 'get
the fix upstream in the mainline kernel', how could that happen?

We are discussing about the solution to 'fix the compile warning to
error' stuff that triggered by the '-I/usr/include/slang', right?


Yes.


We do not necessarily have to change recipe to fix it since the issue
is not introduced by the recipe, the hard coded '-I/usr/include/slang'
in the Makefile cause the issue, we can fix the root cause by kernel
patch(other than just comment the line out). I see your previous patch
to kernel, by comment out the '-I/usr/include/slang' line in the
Makefile, is the same behavior, but we won't have the change(comment
out -I.. in Makefile) upstream to mainline, right?


I am suggesting that firstly, someone send a patch to the mainline
kernel which changes -I/usr/include/slang to -I=/usr/include/slang in
that Makefile.

Secondly, I'm suggesting that we add a line to kernel_do_install() in
kernel.bbclass which does a sed on the Makefile as installed into
$kerneldir which changes -I/usr/include/slang to -I=/usr/include/slang.

We can then drop the patch I added to the linux-yocto kernels.

This is all that should be needed, it should fix all the issues people
have reported in a way that is acceptable to everyone.



Ah, I see what you mean now. But we have push acceptable kernel patch




One final (I hope) follow up on this.

Liang: were you going to put together (and test) the 'sed fix' for
kernel.bbclass ?



No problem, the patch for kernel.bbclass:

commit 60a0b06
Author: Liang Li
Date:   Tue Aug 21 11:06:01 2012 +0800

  kernel.bbclass: fix INC directory for SLANG

  The change is intend to fix the hardcoded '-I/usr/include/slang' in
  the Makefile to be able to aware of SYSROOT if its specified.

  A planned kernel patch almost did the same change, but the change here
  won't conflict with it so this change could work for all kernels.

  Signed-off-by: Liang Li

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 1afb9ab..282194d 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -190,6 +190,9 @@ kernel_do_install() {
for entry in $bin_files; do
rm -f $kerneldir/$entry
done
+
+   # Fix SLNAG_INC for slang.h


s/SLNAG_INC/SLANG_INC/


+   sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' 
$kerneldir/tools/perf/Makefile


It looks like your baseline for this patch is the denzil branch. We'd
want a version for master, which we could backport as required.



Yes, the change for the master branch is almost the same, except line
number context:

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index f34e632..fdef1be 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -204,6 +204,9 @@ kernel_do_install() {
for entry in $bin_files; do
rm -f $kerneldir/$entry
done
+
+   # Fix SLANG_INC for slang.h
+   sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' 
$kerneldir/tools/perf/Makefile
  }

  sysroot_stage_all_append() {

---


   }

   PACKAGE_PREPROCESS_FUNCS += "kernel_package_preprocess"

---

The patch for kernel tree:

commit 6b72896
Author: Liang Li
Date:   Wed Aug 1 14:31:24 2012 +0800

  perf: add SLANG_INC for slang.h

  Previously we hard code '-I/usr/include/slang' to CFLAGS to works with
  some hosts that has /usr/include/slang/slang.h other than
  /usr/include/slang.h like Fedora. This will cause compiling warnings
  in some cases.


I'd rephrase this slightly as:

CFLAGS was previously hard coded to contain "-I/usr/include/slang" to
work with hosts that have "/usr/include/slang/slang.h" as well as hosts
that have "/usr/include/slang.h". This path can cause compile warnings
in some cases:

   

.. and indicate that these warnings can actually cause build errors if
WERROR is enabled.



  We could downgrade the priority of the default hard coded path, and
  provide user a chance to specify correct location of slang.h then user
  could specify SLANG_INC to avoid compile warnings like the
  '/usr/include/slang' is not exists etc.


Another minor rephrase:

To fix this issue, we can use -idirafter to downgrade the priority of the
default hard coded path. We can also make the slang include directory
a variable, to allow the user to specify SLANG_INC and set their own
include location.



No problem, rephrased commit message:

Subject: [PATCH] perf: add SLANG_INC for slang.h

CFLA

Re: [OE-core] [PATCH 0/1] [denzil] linux-yocto: make validate_branches handle non BSP branches

2012-08-21 Thread Tom Zanussi
On Mon, 2012-08-13 at 13:10 -0400, Bruce Ashfield wrote:
> Scott,
> 
> Here's a fix for validate_branches that comes from a pending master
> change that I've been running for 3 weeks, and one that TomZ has tested
> in his yocto BSP work on denzil.
> 
> It fixes some BSP use cases that involve setting a specific SRCREV
> on a base branch, so that a BSP branch that doesn't yet exist can 
> be created off of that known base. Previously only an existing BSP
> branch could have it's SRCREV restricted.
> 

Hi Scott,

I don't see this in denzil - do you have it queued for the next pull
request, or did it get missed?

Tom

> cc: Tom Zanussi 
> 
> Cheers,
> 
> Bruce
> 
> The following changes since commit 73cdebf60df225ee10f2eb215935be3b61e1b831:
> 
>   documentation/dev-manual/dev-manual-kernel-appendix.xml: Add note about 
> conflict (2012-06-29 15:54:26 +0100)
> 
> are available in the git repository at:
>   git://git.pokylinux.org/poky-contrib zedd/kernel-denzil
>   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=zedd/kernel-denzil
> 
> Bruce Ashfield (1):
>   linux-yocto: allow do_validate_branches to handle all branches
> 
>  meta/classes/kernel-yocto.bbclass |  119 
> +
>  1 files changed, 80 insertions(+), 39 deletions(-)
> 



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


Re: [OE-core] [discussion] perf: specify SLANG_INC dir for perf

2012-08-21 Thread Liang Li
On 2012-08-21 21:07, Bruce Ashfield  wrote:
> On 12-08-21 01:08 AM, Liang Li wrote:
> > On 2012-08-20 22:48, Bruce Ashfield  wrote:
> >> On 12-08-17 09:05 AM, Liang Li wrote:
> >>> On 2012-08-17 21:01, Liang Li   wrote:
>  On 2012-08-17 18:53, Richard Purdie  
>   wrote:
> > On Fri, 2012-08-17 at 18:00 +0800, Liang Li wrote:
> >> I am totally confused, you mentioned 'general kernel do_install', I
> >> assume it's oe-core kernel.bbclass concept. Then you mentioned 'get
> >> the fix upstream in the mainline kernel', how could that happen?
> >>
> >> We are discussing about the solution to 'fix the compile warning to
> >> error' stuff that triggered by the '-I/usr/include/slang', right?
> >
> > Yes.
> >
> >>We do not necessarily have to change recipe to fix it since the 
> >> issue
> >> is not introduced by the recipe, the hard coded '-I/usr/include/slang'
> >> in the Makefile cause the issue, we can fix the root cause by kernel
> >> patch(other than just comment the line out). I see your previous patch
> >> to kernel, by comment out the '-I/usr/include/slang' line in the
> >> Makefile, is the same behavior, but we won't have the change(comment
> >> out -I.. in Makefile) upstream to mainline, right?
> >
> > I am suggesting that firstly, someone send a patch to the mainline
> > kernel which changes -I/usr/include/slang to -I=/usr/include/slang in
> > that Makefile.
> >
> > Secondly, I'm suggesting that we add a line to kernel_do_install() in
> > kernel.bbclass which does a sed on the Makefile as installed into
> > $kerneldir which changes -I/usr/include/slang to -I=/usr/include/slang.
> >
> > We can then drop the patch I added to the linux-yocto kernels.
> >
> > This is all that should be needed, it should fix all the issues people
> > have reported in a way that is acceptable to everyone.
> >
> 
>  Ah, I see what you mean now. But we have push acceptable kernel patch
> >>>
> >>
> >> One final (I hope) follow up on this.
> >>
> >> Liang: were you going to put together (and test) the 'sed fix' for
> >> kernel.bbclass ?
> >>
> >
> > No problem, the patch for kernel.bbclass:
> >
> > commit 60a0b06
> > Author: Liang Li
> > Date:   Tue Aug 21 11:06:01 2012 +0800
> >
> >  kernel.bbclass: fix INC directory for SLANG
> >
> >  The change is intend to fix the hardcoded '-I/usr/include/slang' in
> >  the Makefile to be able to aware of SYSROOT if its specified.
> >
> >  A planned kernel patch almost did the same change, but the change here
> >  won't conflict with it so this change could work for all kernels.
> >
> >  Signed-off-by: Liang Li
> >
> > diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
> > index 1afb9ab..282194d 100644
> > --- a/meta/classes/kernel.bbclass
> > +++ b/meta/classes/kernel.bbclass
> > @@ -190,6 +190,9 @@ kernel_do_install() {
> > for entry in $bin_files; do
> > rm -f $kerneldir/$entry
> > done
> > +
> > +   # Fix SLNAG_INC for slang.h
> 
> s/SLNAG_INC/SLANG_INC/
> 
> > +   sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' 
> > $kerneldir/tools/perf/Makefile
> 
> It looks like your baseline for this patch is the denzil branch. We'd
> want a version for master, which we could backport as required.
> 

Yes, the change for the master branch is almost the same, except line
number context:

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index f34e632..fdef1be 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -204,6 +204,9 @@ kernel_do_install() {
for entry in $bin_files; do
rm -f $kerneldir/$entry
done
+
+   # Fix SLANG_INC for slang.h
+   sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' 
$kerneldir/tools/perf/Makefile
 }
 
 sysroot_stage_all_append() {

---

> >   }
> >
> >   PACKAGE_PREPROCESS_FUNCS += "kernel_package_preprocess"
> >
> > ---
> >
> > The patch for kernel tree:
> >
> > commit 6b72896
> > Author: Liang Li
> > Date:   Wed Aug 1 14:31:24 2012 +0800
> >
> >  perf: add SLANG_INC for slang.h
> >
> >  Previously we hard code '-I/usr/include/slang' to CFLAGS to works with
> >  some hosts that has /usr/include/slang/slang.h other than
> >  /usr/include/slang.h like Fedora. This will cause compiling warnings
> >  in some cases.
> 
> I'd rephrase this slightly as:
> 
> CFLAGS was previously hard coded to contain "-I/usr/include/slang" to
> work with hosts that have "/usr/include/slang/slang.h" as well as hosts
> that have "/usr/include/slang.h". This path can cause compile warnings
> in some cases:
> 
>   
> 
> .. and indicate that these warnings can actually cause build errors if
> WERROR is enabled.
> 
> >
> >  We could downgrade the priority of the default hard coded path, and
> >  provide user a chance to specify correct location of slang.h then user
> 

Re: [OE-core] qemu gl changes are broken.

2012-08-21 Thread Paul Eggleton
On Tuesday 21 August 2012 16:46:31 Scott Garman wrote:
> On 08/21/2012 10:45 AM, Scott Garman wrote:
> > On 08/21/2012 09:21 AM, Scott Garman wrote:
> >> Hello,
> >> 
> >> I tried building some images overnight with the latest master, and could
> >> not get them to run this morning with qemu - I got an odd error that a
> >> VNC server was started instead?
> >> 
> >> Just on a hunch, I reverted the following commits:
> >> 
> >> qemu: Fix broken accidental path move
> >> qemu-git: Drop GL passthrough support
> >> qemu-0.15: Drop GL passthrough support
> >> 
> >> rebuilt qemu-native, and now qemu works correctly.
> >> 
> >> I have no idea why qemu is trying to fallback on a VNC server, as I did
> >> not have one running on my machine. I know we use VNC for our
> >> autobuilder sanity tests, but those haven't been integrated into the
> >> runqemu scripts as far as I can tell.
> > 
> > This may need further research, I seem to have it working on another
> > machine. I'll report back once I've learned more.
> 
> Ok, I can no longer reproduce this issue anymore, even on the original
> machine. Figures. :-/

Is this perhaps what happens when DISPLAY is not set? Just a guess...

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

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


Re: [OE-core] [RFC] Replacing chrpath with patchelf

2012-08-21 Thread Khem Raj
On Sat, Aug 18, 2012 at 6:27 PM, Phil Blundell  wrote:
> On Sat, 2012-08-18 at 10:15 -0700, Khem Raj wrote:
>> so I figured. which now means you need to have patchelf on build host
>> preinstalled instead of chrpath.
>
> That'd be a bit of a pain, since (afaict) patchelf is not so widely
> available in host distributions as chrpath.
>
>> you never know. It just lifts this limitation. We ship
>> chrpath-nativesdk as well and people can use it in different ways
>
> I agree that removing the weird and arbitrary-seeming (from the user
> perspective) restriction on rpath length would be a good thing.  But
> it's less clear that swapping a weird restriction that few people will
> ever run into for a requirement to pre-install a relatively obscure host
> binary on all systems is a good tradeoff.

patchelf does have packages for all familiar linux distros but its not in
their default feeds. May be it will be nicer to create one meta
package which installs
all prerequisites not only chrpath/patchelf and maintain that ourselves
this could be nicer since then user only installs

apt-get install make-my-host-compile-oe package on ubuntu and similar
for others.

>
> How about writing a python function, or a trivial standalone program
> (which can be compiled -native without requiring anything other than
> ${BUILD_CC}) to preprocess the binaries before chrpath sees them, or
> indeed just to replace chrpath altogether?  Or, as an alternative, how
> about fixing chrpath to suck less and then just waiting for the improved
> version to trickle down into deployed host distributions?
>
> p.
>
>
>
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

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


Re: [OE-core] qemu gl changes are broken.

2012-08-21 Thread Scott Garman

On 08/21/2012 10:45 AM, Scott Garman wrote:

On 08/21/2012 09:21 AM, Scott Garman wrote:

Hello,

I tried building some images overnight with the latest master, and could
not get them to run this morning with qemu - I got an odd error that a
VNC server was started instead?

Just on a hunch, I reverted the following commits:

qemu: Fix broken accidental path move
qemu-git: Drop GL passthrough support
qemu-0.15: Drop GL passthrough support

rebuilt qemu-native, and now qemu works correctly.

I have no idea why qemu is trying to fallback on a VNC server, as I did
not have one running on my machine. I know we use VNC for our
autobuilder sanity tests, but those haven't been integrated into the
runqemu scripts as far as I can tell.


This may need further research, I seem to have it working on another
machine. I'll report back once I've learned more.


Ok, I can no longer reproduce this issue anymore, even on the original 
machine. Figures. :-/


Scott

--
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center

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


[OE-core] [PATCH v2 4/5] u-boot-mkimage: Update to v2012.07

2012-08-21 Thread Tom Rini
Signed-off-by: Tom Rini 
---
 .../u-boot/u-boot-mkimage_2012.04.01.bb|   35 
 meta/recipes-bsp/u-boot/u-boot-mkimage_2012.07.bb  |   33 ++
 2 files changed, 33 insertions(+), 35 deletions(-)
 delete mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2012.04.01.bb
 create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2012.07.bb

diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.04.01.bb 
b/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.04.01.bb
deleted file mode 100644
index c41a2f1..000
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.04.01.bb
+++ /dev/null
@@ -1,35 +0,0 @@
-DESCRIPTION = "U-boot bootloader mkimage tool"
-LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
-SECTION = "bootloader"
-
-PR = "r1"
-
-# This is needs to be validated among supported BSP's before we can
-# make it default
-DEFAULT_PREFERENCE = "-1"
-
-# This revision corresponds to the tag "v2012.04.01"
-# We use the revision in order to avoid having to fetch it from the
-# repo during parse
-SRCREV = "415d386877df49eb051b85ef74fa59a16dc17c7d"
-
-PV = "v2012.04.01+git${SRCPV}"
-
-SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
-
-S = "${WORKDIR}/git"
-
-EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
-
-do_compile () {
-  oe_runmake tools
-}
-
-do_install () {
-  install -d ${D}${bindir}
-  install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
-  ln -sf uboot-mkimage ${D}${bindir}/mkimage
-}
-
-BBCLASSEXTEND = "native nativesdk"
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.07.bb 
b/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.07.bb
new file mode 100644
index 000..0ba4c03
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.07.bb
@@ -0,0 +1,33 @@
+DESCRIPTION = "U-boot bootloader mkimage tool"
+LICENSE = "GPLv2+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
+SECTION = "bootloader"
+
+# This is needs to be validated among supported BSP's before we can
+# make it default
+DEFAULT_PREFERENCE = "-1"
+
+# This revision corresponds to the tag "v2012.07"
+# We use the revision in order to avoid having to fetch it from the
+# repo during parse
+SRCREV = "190649fb4309d1bc0fe7732fd0f951cb6440f935"
+
+PV = "v2012.07+git${SRCPV}"
+
+SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
+
+S = "${WORKDIR}/git"
+
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
+
+do_compile () {
+  oe_runmake tools
+}
+
+do_install () {
+  install -d ${D}${bindir}
+  install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
+  ln -sf uboot-mkimage ${D}${bindir}/mkimage
+}
+
+BBCLASSEXTEND = "native nativesdk"
-- 
1.7.9.5


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


[OE-core] [PATCH v2 3/5] u-boot-fw-utils: Update to v2012.07

2012-08-21 Thread Tom Rini
Signed-off-by: Tom Rini 
---
 .../u-boot/u-boot-fw-utils_2012.04.01.bb   |   35 
 meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.07.bb |   33 ++
 2 files changed, 33 insertions(+), 35 deletions(-)
 delete mode 100644 meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
 create mode 100644 meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.07.bb

diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb 
b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
deleted file mode 100644
index d217320..000
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
+++ /dev/null
@@ -1,35 +0,0 @@
-DESCRIPTION = "U-boot bootloader fw_printenv/setenv utils"
-LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
-SECTION = "bootloader"
-
-DEPENDS = "mtd-utils"
-
-PR = "r1"
-
-# This is needs to be validated among supported BSP's before we can
-# make it default
-DEFAULT_PREFERENCE = "-1"
-
-# This revision corresponds to the tag "v2012.04.01"
-# We use the revision in order to avoid having to fetch it from the
-# repo during parse
-SRCREV = "415d386877df49eb051b85ef74fa59a16dc17c7d"
-
-PV = "v2012.04.01+git${SRCPV}"
-
-SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
-
-S = "${WORKDIR}/git"
-
-EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
-
-do_compile () {
-  oe_runmake env
-}
-
-do_install () {
-  install -d ${D}${base_sbindir}
-  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_printenv
-  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_setenv
-}
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.07.bb 
b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.07.bb
new file mode 100644
index 000..565beb7
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.07.bb
@@ -0,0 +1,33 @@
+DESCRIPTION = "U-boot bootloader fw_printenv/setenv utils"
+LICENSE = "GPLv2+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
+SECTION = "bootloader"
+
+DEPENDS = "mtd-utils"
+
+# This is needs to be validated among supported BSP's before we can
+# make it default
+DEFAULT_PREFERENCE = "-1"
+
+# This revision corresponds to the tag "v2012.07"
+# We use the revision in order to avoid having to fetch it from the
+# repo during parse
+SRCREV = "190649fb4309d1bc0fe7732fd0f951cb6440f935"
+
+PV = "v2012.07+git${SRCPV}"
+
+SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
+
+S = "${WORKDIR}/git"
+
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
+
+do_compile () {
+  oe_runmake env
+}
+
+do_install () {
+  install -d ${D}${base_sbindir}
+  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_printenv
+  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_setenv
+}
-- 
1.7.9.5


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


[OE-core] [PATCH v2 5/5] u-boot: Update to v2012.07

2012-08-21 Thread Tom Rini
Signed-off-by: Tom Rini 
---
 meta/recipes-bsp/u-boot/u-boot_2012.04.01.bb |   29 --
 meta/recipes-bsp/u-boot/u-boot_2012.07.bb|   29 ++
 2 files changed, 29 insertions(+), 29 deletions(-)
 delete mode 100644 meta/recipes-bsp/u-boot/u-boot_2012.04.01.bb
 create mode 100644 meta/recipes-bsp/u-boot/u-boot_2012.07.bb

diff --git a/meta/recipes-bsp/u-boot/u-boot_2012.04.01.bb 
b/meta/recipes-bsp/u-boot/u-boot_2012.04.01.bb
deleted file mode 100644
index c4ec50d..000
--- a/meta/recipes-bsp/u-boot/u-boot_2012.04.01.bb
+++ /dev/null
@@ -1,29 +0,0 @@
-require u-boot.inc
-
-# This is needs to be validated among supported BSP's before we can
-# make it default
-DEFAULT_PREFERENCE = "-1"
-
-# To build u-boot for your machine, provide the following lines in
-# your machine config, replacing the assignments as appropriate for
-# your machine.
-# UBOOT_MACHINE = "omap3_beagle_config"
-# UBOOT_ENTRYPOINT = "0x80008000"
-# UBOOT_LOADADDRESS = "0x80008000"
-
-LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
-
-# This revision corresponds to the tag "v2012.04.01"
-# We use the revision in order to avoid having to fetch it from the
-# repo during parse
-SRCREV = "415d386877df49eb051b85ef74fa59a16dc17c7d"
-
-PV = "v2012.04.01+git${SRCPV}"
-PR = "r1"
-
-SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
-
-S = "${WORKDIR}/git"
-
-PACKAGE_ARCH = "${MACHINE_ARCH}"
diff --git a/meta/recipes-bsp/u-boot/u-boot_2012.07.bb 
b/meta/recipes-bsp/u-boot/u-boot_2012.07.bb
new file mode 100644
index 000..91f32e6
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/u-boot_2012.07.bb
@@ -0,0 +1,29 @@
+require u-boot.inc
+
+# This is needs to be validated among supported BSP's before we can
+# make it default
+DEFAULT_PREFERENCE = "-1"
+
+# To build u-boot for your machine, provide the following lines in
+# your machine config, replacing the assignments as appropriate for
+# your machine.
+# UBOOT_MACHINE = "omap3_beagle_config"
+# UBOOT_ENTRYPOINT = "0x80008000"
+# UBOOT_LOADADDRESS = "0x80008000"
+
+LICENSE = "GPLv2+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
+
+# This revision corresponds to the tag "v2012.07"
+# We use the revision in order to avoid having to fetch it from the
+# repo during parse
+SRCREV = "190649fb4309d1bc0fe7732fd0f951cb6440f935"
+
+PV = "v2012.07+git${SRCPV}"
+PR = "r1"
+
+SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
+
+S = "${WORKDIR}/git"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
-- 
1.7.9.5


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


[OE-core] [PATCH v2 2/5] u-boot-fw-utils: Correct EXTRA_OEMAKE

2012-08-21 Thread Tom Rini
- HOSTCFLAGS / HOSTLDFLAGS have been valid for a long time for tools/env
- HOSTSTRIP will be needed in the future, does no harm to be correct
  now.

Signed-off-by: Tom Rini 
---
 meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb |4 ++--
 .../u-boot/u-boot-fw-utils_2012.04.01.bb   |4 +++-
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb 
b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
index 7c4df1a..ae20042 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
@@ -6,7 +6,7 @@ SECTION = "bootloader"
 
 DEPENDS = "mtd-utils"
 
-PR = "r1"
+PR = "r2"
 
 SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2";
 
@@ -15,7 +15,7 @@ SRC_URI[sha256sum] = 
"362ddb935885da98cf461eba08f31e3e59d0c4ada6cb2fa15596f43af3
 
 S = "${WORKDIR}/u-boot-${PV}"
 
-EXTRA_OEMAKE = 'HOSTCC="${CC}"'
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
 
 do_compile () {
   oe_runmake env
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb 
b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
index fe3422a..d217320 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
@@ -5,6 +5,8 @@ SECTION = "bootloader"
 
 DEPENDS = "mtd-utils"
 
+PR = "r1"
+
 # This is needs to be validated among supported BSP's before we can
 # make it default
 DEFAULT_PREFERENCE = "-1"
@@ -20,7 +22,7 @@ SRC_URI = 
"git://git.denx.de/u-boot.git;branch=master;protocol=git"
 
 S = "${WORKDIR}/git"
 
-EXTRA_OEMAKE = 'HOSTCC="${CC}"'
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
 
 do_compile () {
   oe_runmake env
-- 
1.7.9.5


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


[OE-core] [PATCH v2 1/5] u-boot-mkimage: Correct EXTRA_OEMAKE

2012-08-21 Thread Tom Rini
- HOSTLD has never been used in U-Boot, drop.
- HOSTSTRIP has been valid for a long time for tools
- Same for HOSTCFLAGS / HOSTLDFLAGS

Signed-off-by: Tom Rini 
---
 meta/recipes-bsp/u-boot/u-boot-mkimage_2011.03.bb  |4 ++--
 meta/recipes-bsp/u-boot/u-boot-mkimage_2011.06.bb  |4 ++--
 .../u-boot/u-boot-mkimage_2012.04.01.bb|4 +++-
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2011.03.bb 
b/meta/recipes-bsp/u-boot/u-boot-mkimage_2011.03.bb
index 34bae8e..fdbe7c9 100644
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2011.03.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2011.03.bb
@@ -4,7 +4,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
 
file://README;beginline=1;endline=22;md5=3a00ef51d3fc96e9d6c1bc4708ccd3b5"
 SECTION = "bootloader"
 
-PR = "r1"
+PR = "r2"
 
 SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2 \
file://0001-Drop-config.h-include-in-tools-imximage.h.patch \
@@ -17,7 +17,7 @@ S = "${WORKDIR}/u-boot-${PV}"
 
 BBCLASSEXTEND = "native nativesdk"
 
-EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}" HOSTSTRIP=true'
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
 
 do_compile () {
   oe_runmake tools
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2011.06.bb 
b/meta/recipes-bsp/u-boot/u-boot-mkimage_2011.06.bb
index 3ca0c2c..c4921d6 100644
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2011.06.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2011.06.bb
@@ -4,7 +4,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
 
file://README;beginline=1;endline=22;md5=5ba4218ac89af7846802d0348df3fb90"
 SECTION = "bootloader"
 
-PR = "r0"
+PR = "r1"
 
 SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2";
 
@@ -15,7 +15,7 @@ S = "${WORKDIR}/u-boot-${PV}"
 
 BBCLASSEXTEND = "native nativesdk"
 
-EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
 
 do_compile () {
   oe_runmake tools
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.04.01.bb 
b/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.04.01.bb
index aa107fe..c41a2f1 100644
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.04.01.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2012.04.01.bb
@@ -3,6 +3,8 @@ LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
 SECTION = "bootloader"
 
+PR = "r1"
+
 # This is needs to be validated among supported BSP's before we can
 # make it default
 DEFAULT_PREFERENCE = "-1"
@@ -18,7 +20,7 @@ SRC_URI = 
"git://git.denx.de/u-boot.git;branch=master;protocol=git"
 
 S = "${WORKDIR}/git"
 
-EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS}" HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'
 
 do_compile () {
   oe_runmake tools
-- 
1.7.9.5


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


Re: [OE-core] [PATCH 2/5] u-boot-fw-utils: Correct EXTRA_OEMAKE

2012-08-21 Thread Tom Rini
On Tue, Aug 21, 2012 at 01:56:46PM -0700, Khem Raj wrote:
> On Tue, Aug 21, 2012 at 12:28 PM, Saul Wold  wrote:
> > HOSTCFLAGS="${CFLAGS}
> 
> needs a closing quote

Good spotting, both of you.  But I can't test a v2 as the problem of
nothing being buildable on just oe-core still remains.  Of course, I'll
happily don another brown paper bag atop my current one, just point me
at what I've got wrong :)

-- 
Tom

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


Re: [OE-core] [PATCH 2/5] u-boot-fw-utils: Correct EXTRA_OEMAKE

2012-08-21 Thread Khem Raj
On Tue, Aug 21, 2012 at 12:28 PM, Saul Wold  wrote:
> HOSTCFLAGS="${CFLAGS}

needs a closing quote

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


[OE-core] pseudo diagnostics: please do not ignore these completely!

2012-08-21 Thread Peter Seebach
If you have really weird build problems:

Check the pseudo.log file for a given package. A lot of packages are
generating a fair number of diagnostics and warnings some of which
indicate that stuff is non-trivially busted for some packages. A quick
scan of a small build turned up 16 packages with no diagnostics, and 21
which had diagnostics of some sort. Of those 21, 18 had at least one
case in which the database showed an inode as being a directory, and a
request showed that it wasn't anymore, or vice versa.

Pseudo is fairly paranoid about trying to catch stuff like this, but it
tends to do its best to fail quietly and just log the errors -- these
logs don't do much if we don't follow up on them, though.

The basic rule is:

If pseudo is aware of or using a file or directory, that file or
directory should not be moved/renamed, or replaced, except under
pseudo.

In many cases, things will recover. If a file is created under pseudo,
then renamed outside pseudo, then statted under pseudo again, the
database entry for the inode will be found and pseudo will probably do
the right thing. If there's a mismatch where an inode's basic type
(file/directory/link) mismatches between the database and an incoming
request, pseudo will nuke the bogus database entry.

But this won't catch everything.

So if you have a strange problem that you can't reproduce, check the
pseudo.log file. It may not mean much to you, but if it says anything
about "dir err" or "ino mismatch", something has gone pretty badly
wrong. If you can reproduce "this file is full of weird messages" with
a given package, that's probably a bug in the package, and I will try
to look into some of these when I have a minute or three and see
whether I can find any patterns.

And yes, I'm aware that running things under pseudo is slow. I have
plans to work on that too. Maybe I should do a kickstarter. :)

-s
-- 
Listen, get this.  Nobody with a good compiler needs to be justified.

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


Re: [OE-core] [PATCH 2/5] u-boot-fw-utils: Correct EXTRA_OEMAKE

2012-08-21 Thread Saul Wold

On 08/20/2012 10:57 AM, Tom Rini wrote:

On Fri, Aug 17, 2012 at 07:46:19PM -0700, Saul Wold wrote:

On 08/17/2012 09:48 AM, Tom Rini wrote:

- HOSTCFLAGS / HOSTLDFLAGS have been valid for a long time for tools/env
- HOSTSTRIP will be needed in the future, does no harm to be correct
   now.

Signed-off-by: Tom Rini 
---
  meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb |4 ++--
  .../u-boot/u-boot-fw-utils_2012.04.01.bb   |4 +++-
  2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb 
b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
index 7c4df1a..99eef15 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
@@ -6,7 +6,7 @@ SECTION = "bootloader"

  DEPENDS = "mtd-utils"

-PR = "r1"
+PR = "r2"

  SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2";

@@ -15,7 +15,7 @@ SRC_URI[sha256sum] = 
"362ddb935885da98cf461eba08f31e3e59d0c4ada6cb2fa15596f43af3

  S = "${WORKDIR}/u-boot-${PV}"

-EXTRA_OEMAKE = 'HOSTCC="${CC}"'
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS} HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'


Ah, finally here it is --^

Missing a "!

Sau!



  do_compile () {
oe_runmake env
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb 
b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
index fe3422a..364f8a5 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2012.04.01.bb
@@ -5,6 +5,8 @@ SECTION = "bootloader"

  DEPENDS = "mtd-utils"

+PR = "r1"
+
  # This is needs to be validated among supported BSP's before we can
  # make it default
  DEFAULT_PREFERENCE = "-1"
@@ -20,7 +22,7 @@ SRC_URI = 
"git://git.denx.de/u-boot.git;branch=master;protocol=git"

  S = "${WORKDIR}/git"

-EXTRA_OEMAKE = 'HOSTCC="${CC}"'
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTCFLAGS="${CFLAGS} HOSTLDFLAGS="${LDFLAGS}" 
HOSTSTRIP=true'

  do_compile () {
oe_runmake env




This had a failure also:


ERROR: Function failed: do_install (see 
/srv/ssd/sgw_ab/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/i586-poky-linux/u-boot-fw-utils-2011.06-r2/temp/log.do_install.69452
 for further information)
ERROR: Logfile of failure stored in: 
/srv/ssd/sgw_ab/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/i586-poky-linux/u-boot-fw-utils-2011.06-r2/temp/log.do_install.69452
Log data follows:
| DEBUG: Executing shell function do_install
| install: cannot stat 
`/srv/ssd/sgw_ab/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/i586-poky-linux/u-boot-fw-utils-2011.06-r2/u-boot-2011.06/tools/env/fw_printenv':
 No such file or directory
| ERROR: Function failed: do_install (see 
/srv/ssd/sgw_ab/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/i586-poky-linux/u-boot-fw-utils-2011.06-r2/temp/log.do_install.69452
 for further information)
NOTE: package u-boot-fw-utils-2011.06-r2: task do_install: Failed
ERROR: Task 2755 
(/srv/ssd/sgw_ab/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb,
 do_install) failed with exit code '1'


I have to admit that since these were trivial changes I didn't try and
build them.  But oe-core doesn't build anymore.  Doing:
git pull
git reset --hard origin/master
rm -rf build/conf build/tmp*
. oe-init-build-env
bitbake core-image-sato

Fails and can't give me a log.  Can you pastebin the compile logs?


Finally the compile log, sorry had other fires I was fighting, seems the 
make is doing nothing!



DEBUG: Executing shell function do_compile
NOTE: make -j 6 HOSTCC=arm-poky-linux-gnueabi-gcc  -march=armv7-a 
-fno-tree-vectorize -mthumb-interwork -mfloat-abi=softfp -mfpu=neon 
-mtune=cortex-a8 --sysroot=/intel/poky/builds/world/tmp/sysroots/beagleboard 
HOSTCFLAGS= -O2 -pipe -g -feliminate-unused-debug-types HOSTLDFLAGS=-Wl,-O1 
-Wl,--hash-style=gnu -Wl,--as-needed HOSTSTRIP=true tools
make -j 6 HOSTCC=arm-poky-linux-gnueabi-gcc -march=armv7-a 
-fno-tree-vectorize -mthumb-interwork -mfloat-abi=softfp -mfpu=neon 
-mtune=cortex-a8 --sysroot=/intel/poky/builds/world/tmp/sysroots/beagleboard 
HOSTCFLAGS= -O2 -pipe -g -feliminate-unused-debug-types HOSTLDFLAGS=-Wl,-O1 
-Wl,--hash-style=gnu -Wl,--as-needed HOSTSTRIP=true tools
DEBUG: Shell function do_compile finished




Thanks!



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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread McClintock Matthew-B29882
On Tue, Aug 21, 2012 at 2:13 PM, Chris Larson  wrote:
> On Tue, Aug 21, 2012 at 11:47 AM, McClintock Matthew-B29882
>  wrote:
>>> #ifndef O_CLOEXEC
>>> # define O_CLOEXEC 0
>>> #endif
>>>
>>> and be done with it
>>
>> Well this is seemingly the same way of doing it, just looks like you
>> always want it applied? I don't think it should always be applied.
>>
>> If this was it takes to get the build fix in, then I will do it...
>> please confirm what will be accepted.
>
> No, on newer systems O_CLOEXEC will be defined, so that #ifndef block
> will never be entered, and there'll be no change.

Right, I was not thinking...

-M

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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Chris Larson
On Tue, Aug 21, 2012 at 11:47 AM, McClintock Matthew-B29882
 wrote:
>> #ifndef O_CLOEXEC
>> # define O_CLOEXEC 0
>> #endif
>>
>> and be done with it
>
> Well this is seemingly the same way of doing it, just looks like you
> always want it applied? I don't think it should always be applied.
>
> If this was it takes to get the build fix in, then I will do it...
> please confirm what will be accepted.

No, on newer systems O_CLOEXEC will be defined, so that #ifndef block
will never be entered, and there'll be no change.
-- 
Christopher Larson

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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread McClintock Matthew-B29882
On Tue, Aug 21, 2012 at 2:02 PM, Chris Larson  wrote:
> On Tue, Aug 21, 2012 at 11:48 AM, Khem Raj  wrote:
>> On Tue, Aug 21, 2012 at 11:41 AM, Saul Wold  wrote:
>>> On 08/21/2012 11:30 AM, Khem Raj wrote:

 On Tue, Aug 21, 2012 at 11:10 AM, McClintock Matthew-B29882
  wrote:
>
> On Tue, Aug 21, 2012 at 1:06 PM, Khem Raj  wrote:
>>
>> On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
>>  wrote:
>>>
>>> On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:

 On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock
  wrote:
>
> +
> +do_configure_prepend (){
> +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h;
> then
> +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
> +   fi
> +}



 IMO It would be safer to create a patch for kmod itself where you
 define O_CLOEXEC if it
 was not defined before. The above seems a bit risky
>>>
>>>
>>> Why is it risky? I only wanted to do this for affected systems. There
>>> is not an easy way to do this with a patch, unless of course I apply
>>> the patch manually.
>>
>>
>> manually gripping at the host installation and then if O_CLOEXEC might
>> be in comments
>
>
> How about grep define.*O_CLOEXEC -r ${includedir_native}/bits/fcntl.h
>
>> and furthermore it if it comes from fcntl.h which is not where you are
>> looking for
>
>
> I am grepping this file though?


 I would go into the specific file where its asking for O_CLOEXEC

 and add

 #ifndef O_CLOEXEC
 # define O_CLOEXEC 0
 #endif

 and be done with it
>
> This does seem like a nicer approach.

OK - 2v1 ;) I'll respin as a patch.

-M

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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Chris Larson
On Tue, Aug 21, 2012 at 11:48 AM, Khem Raj  wrote:
> On Tue, Aug 21, 2012 at 11:41 AM, Saul Wold  wrote:
>> On 08/21/2012 11:30 AM, Khem Raj wrote:
>>>
>>> On Tue, Aug 21, 2012 at 11:10 AM, McClintock Matthew-B29882
>>>  wrote:

 On Tue, Aug 21, 2012 at 1:06 PM, Khem Raj  wrote:
>
> On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
>  wrote:
>>
>> On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:
>>>
>>> On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock
>>>  wrote:

 +
 +do_configure_prepend (){
 +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h;
 then
 +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
 +   fi
 +}
>>>
>>>
>>>
>>> IMO It would be safer to create a patch for kmod itself where you
>>> define O_CLOEXEC if it
>>> was not defined before. The above seems a bit risky
>>
>>
>> Why is it risky? I only wanted to do this for affected systems. There
>> is not an easy way to do this with a patch, unless of course I apply
>> the patch manually.
>
>
> manually gripping at the host installation and then if O_CLOEXEC might
> be in comments


 How about grep define.*O_CLOEXEC -r ${includedir_native}/bits/fcntl.h

> and furthermore it if it comes from fcntl.h which is not where you are
> looking for


 I am grepping this file though?
>>>
>>>
>>> I would go into the specific file where its asking for O_CLOEXEC
>>>
>>> and add
>>>
>>> #ifndef O_CLOEXEC
>>> # define O_CLOEXEC 0
>>> #endif
>>>
>>> and be done with it

This does seem like a nicer approach.
-- 
Christopher Larson

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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread McClintock Matthew-B29882
On Tue, Aug 21, 2012 at 1:30 PM, Khem Raj  wrote:
> On Tue, Aug 21, 2012 at 11:10 AM, McClintock Matthew-B29882
>  wrote:
>> On Tue, Aug 21, 2012 at 1:06 PM, Khem Raj  wrote:
>>> On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
>>>  wrote:
 On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:
> On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock  
> wrote:
>> +
>> +do_configure_prepend (){
>> +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
>> +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
>> +   fi
>> +}
>
>
> IMO It would be safer to create a patch for kmod itself where you
> define O_CLOEXEC if it
> was not defined before. The above seems a bit risky

 Why is it risky? I only wanted to do this for affected systems. There
 is not an easy way to do this with a patch, unless of course I apply
 the patch manually.
>>>
>>> manually gripping at the host installation and then if O_CLOEXEC might
>>> be in comments
>>
>> How about grep define.*O_CLOEXEC -r ${includedir_native}/bits/fcntl.h
>>
>>> and furthermore it if it comes from fcntl.h which is not where you are
>>> looking for
>>
>> I am grepping this file though?
>
> I would go into the specific file where its asking for O_CLOEXEC
>
> and add
>
> #ifndef O_CLOEXEC
> # define O_CLOEXEC 0
> #endif
>
> and be done with it

Well this is seemingly the same way of doing it, just looks like you
always want it applied? I don't think it should always be applied.

If this was it takes to get the build fix in, then I will do it...
please confirm what will be accepted.

-M

>
>>
>>> there are few variables like that where its impacting more than
>>> affected systems.
>>
>> I don't follow...
>>
>> -M
>
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

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


[OE-core] [PATCH 1/1 denzil] recipes-core/eglibc-2.13: Patch for locale-base-tt-ru packaging

2012-08-21 Thread Mark Hatle
From: Jeff Polk 

The eglibc-2.13 build can fail because locale-base-tt-ru is in
PACKAGES twice. This is because the SUPPORTED list and the i18n
directories are out of sync with each other; the SUPPORTED list
expects a directory named "tt_RU.UTF8", but the directory is
actually named "tt_RU", and likewise for the @iqtelif variants.

Signed-off-by: Peter Seebach 
Signed-off-by: Jeff Polk 
Signed-off-by: Mark Hatle 
---
 .../eglibc/eglibc-2.13/supported-tt-ru.patch   |   13 +
 meta/recipes-core/eglibc/eglibc_2.13.bb|3 ++-
 2 files changed, 15 insertions(+), 1 deletions(-)
 create mode 100644 meta/recipes-core/eglibc/eglibc-2.13/supported-tt-ru.patch

diff --git a/meta/recipes-core/eglibc/eglibc-2.13/supported-tt-ru.patch 
b/meta/recipes-core/eglibc/eglibc-2.13/supported-tt-ru.patch
new file mode 100644
index 000..c65e38f
--- /dev/null
+++ b/meta/recipes-core/eglibc/eglibc-2.13/supported-tt-ru.patch
@@ -0,0 +1,13 @@
+--- a/localedata/SUPPORTED
 b/localedata/SUPPORTED
+@@ -385,8 +385,8 @@
+ tr_TR.UTF-8/UTF-8 \
+ tr_TR/ISO-8859-9 \
+ ts_ZA/UTF-8 \
+-tt_RU.UTF-8/UTF-8 \
+-tt_RU.UTF-8@iqtelif/UTF-8 \
++tt_RU/UTF-8 \
++tt_RU@iqtelif/UTF-8 \
+ ug_CN/UTF-8 \
+ uk_UA.UTF-8/UTF-8 \
+ uk_UA/KOI8-U \
diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bb 
b/meta/recipes-core/eglibc/eglibc_2.13.bb
index 690f99d..d0bac6b 100644
--- a/meta/recipes-core/eglibc/eglibc_2.13.bb
+++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
@@ -3,7 +3,7 @@ require eglibc.inc
 SRCREV = "15508"
 
 DEPENDS += "gperf-native"
-PR = "r28"
+PR = "r29"
 PR_append = "+svnr${SRCPV}"
 
 EGLIBC_BRANCH="eglibc-2_13"
@@ -14,6 +14,7 @@ SRC_URI = 
"svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};proto=http
file://stack-protector-test.patch \
file://armv4-eabi-compile-fix.patch \
file://etc/ld.so.conf \
+   file://supported-tt-ru.patch \
file://generate-supported.mk \
file://ppc-sqrt.patch \
file://multilib_readlib.patch \
-- 
1.7.3.4


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


[OE-core] [PATCH 0/1 denzil] Fix for generation of russian locale

2012-08-21 Thread Mark Hatle
The following changes since commit 95756cfbb7a9348b23cb46a49a5509e57e973faf:

  valgrind: fix default.supp missing issue (2012-08-21 11:19:26 +0100)

are available in the git repository at:
  git://git.yoctoproject.org/poky-contrib mhatle/denzil-eglibc
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=mhatle/denzil-eglibc

Jeff Polk (1):
  recipes-core/eglibc-2.13: Patch for locale-base-tt-ru packaging

 .../eglibc/eglibc-2.13/supported-tt-ru.patch   |   13 +
 meta/recipes-core/eglibc/eglibc_2.13.bb|3 ++-
 2 files changed, 15 insertions(+), 1 deletions(-)
 create mode 100644 meta/recipes-core/eglibc/eglibc-2.13/supported-tt-ru.patch

-- 
1.7.3.4


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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Khem Raj
On Tue, Aug 21, 2012 at 11:41 AM, Saul Wold  wrote:
> On 08/21/2012 11:30 AM, Khem Raj wrote:
>>
>> On Tue, Aug 21, 2012 at 11:10 AM, McClintock Matthew-B29882
>>  wrote:
>>>
>>> On Tue, Aug 21, 2012 at 1:06 PM, Khem Raj  wrote:

 On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
  wrote:
>
> On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:
>>
>> On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock
>>  wrote:
>>>
>>> +
>>> +do_configure_prepend (){
>>> +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h;
>>> then
>>> +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
>>> +   fi
>>> +}
>>
>>
>>
>> IMO It would be safer to create a patch for kmod itself where you
>> define O_CLOEXEC if it
>> was not defined before. The above seems a bit risky
>
>
> Why is it risky? I only wanted to do this for affected systems. There
> is not an easy way to do this with a patch, unless of course I apply
> the patch manually.


 manually gripping at the host installation and then if O_CLOEXEC might
 be in comments
>>>
>>>
>>> How about grep define.*O_CLOEXEC -r ${includedir_native}/bits/fcntl.h
>>>
 and furthermore it if it comes from fcntl.h which is not where you are
 looking for
>>>
>>>
>>> I am grepping this file though?
>>
>>
>> I would go into the specific file where its asking for O_CLOEXEC
>>
>> and add
>>
>> #ifndef O_CLOEXEC
>> # define O_CLOEXEC 0
>> #endif
>>
>> and be done with it
>>
> Wasn't this proposed back a month ago:
>
> http://lists.linuxtogo.org/pipermail/openembedded-core/2012-July/026343.html
>
> And there was discussion about that approach then? I think it was rejected
> due to lack of testing.

I think if it worked fine on a system like cents 5.8 then that would
cover the case
as well try it on newer systems where O_CLOEXEC is available that should do it.

>
> Sau!
>
>>>
 there are few variables like that where its impacting more than
 affected systems.
>>>
>>>
>>> I don't follow...
>>>
>>> -M
>>
>>
>> ___
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>>
>

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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread McClintock Matthew-B29882
On Tue, Aug 21, 2012 at 1:41 PM, Saul Wold  wrote:
> On 08/21/2012 11:30 AM, Khem Raj wrote:
>>
>> On Tue, Aug 21, 2012 at 11:10 AM, McClintock Matthew-B29882
>>  wrote:
>>>
>>> On Tue, Aug 21, 2012 at 1:06 PM, Khem Raj  wrote:

 On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
  wrote:
>
> On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:
>>
>> On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock
>>  wrote:
>>>
>>> +
>>> +do_configure_prepend (){
>>> +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h;
>>> then
>>> +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
>>> +   fi
>>> +}
>>
>>
>>
>> IMO It would be safer to create a patch for kmod itself where you
>> define O_CLOEXEC if it
>> was not defined before. The above seems a bit risky
>
>
> Why is it risky? I only wanted to do this for affected systems. There
> is not an easy way to do this with a patch, unless of course I apply
> the patch manually.


 manually gripping at the host installation and then if O_CLOEXEC might
 be in comments
>>>
>>>
>>> How about grep define.*O_CLOEXEC -r ${includedir_native}/bits/fcntl.h
>>>
 and furthermore it if it comes from fcntl.h which is not where you are
 looking for
>>>
>>>
>>> I am grepping this file though?
>>
>>
>> I would go into the specific file where its asking for O_CLOEXEC
>>
>> and add
>>
>> #ifndef O_CLOEXEC
>> # define O_CLOEXEC 0
>> #endif
>>
>> and be done with it
>>
> Wasn't this proposed back a month ago:
>
> http://lists.linuxtogo.org/pipermail/openembedded-core/2012-July/026343.html
>
> And there was discussion about that approach then? I think it was rejected
> due to lack of testing.

Then we had a bit more analysis and discussion on the issue and others
chimed in they had implemented similar work arounds... it's all in
that thread.

-M

>
>
>>>
 there are few variables like that where its impacting more than
 affected systems.
>>>
>>>
>>> I don't follow...
>>>
>>> -M
>>
>>
>> ___
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>>
>
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

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


[OE-core] [PATCH] distutils.bblass: change order of args to install step

2012-08-21 Thread Matthew McClintock
This let's the user override install-lib argument again if it needs
to be something else, otherwise things like python-setuptools
won't be able to modify the install-lib dir

This fixes a new issue exposed by my previous distutils patch
that fixed the python modules default install location. Also,
it removes running the install step twice which was inadvertant

Signed-off-by: Matthew McClintock 
---
 meta/classes/distutils.bbclass |3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/classes/distutils.bbclass b/meta/classes/distutils.bbclass
index 7588b81..52a1aa8 100644
--- a/meta/classes/distutils.bbclass
+++ b/meta/classes/distutils.bbclass
@@ -38,8 +38,7 @@ distutils_do_install() {
 STAGING_LIBDIR=${STAGING_LIBDIR} \
 PYTHONPATH=${D}/${PYTHON_SITEPACKAGES_DIR} \
 BUILD_SYS=${BUILD_SYS} HOST_SYS=${HOST_SYS} \
-${STAGING_BINDIR_NATIVE}/python-native/python setup.py install 
${DISTUTILS_INSTALL_ARGS} || \
-${STAGING_BINDIR_NATIVE}/python-native/python setup.py install 
${DISTUTILS_INSTALL_ARGS} --install-lib=${D}${libdir}/${PYTHON_DIR} || \
+${STAGING_BINDIR_NATIVE}/python-native/python setup.py install 
--install-lib=${D}${libdir}/${PYTHON_DIR} ${DISTUTILS_INSTALL_ARGS} || \
 bbfatal "python setup.py install execution failed."
 
 for i in `find ${D} -name "*.py"` ; do \
-- 
1.7.9.7



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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Saul Wold

On 08/21/2012 11:30 AM, Khem Raj wrote:

On Tue, Aug 21, 2012 at 11:10 AM, McClintock Matthew-B29882
 wrote:

On Tue, Aug 21, 2012 at 1:06 PM, Khem Raj  wrote:

On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
 wrote:

On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:

On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock  wrote:

+
+do_configure_prepend (){
+   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
+   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
+   fi
+}



IMO It would be safer to create a patch for kmod itself where you
define O_CLOEXEC if it
was not defined before. The above seems a bit risky


Why is it risky? I only wanted to do this for affected systems. There
is not an easy way to do this with a patch, unless of course I apply
the patch manually.


manually gripping at the host installation and then if O_CLOEXEC might
be in comments


How about grep define.*O_CLOEXEC -r ${includedir_native}/bits/fcntl.h


and furthermore it if it comes from fcntl.h which is not where you are
looking for


I am grepping this file though?


I would go into the specific file where its asking for O_CLOEXEC

and add

#ifndef O_CLOEXEC
# define O_CLOEXEC 0
#endif

and be done with it


Wasn't this proposed back a month ago:

http://lists.linuxtogo.org/pipermail/openembedded-core/2012-July/026343.html

And there was discussion about that approach then? I think it was 
rejected due to lack of testing.


Sau!




there are few variables like that where its impacting more than
affected systems.


I don't follow...

-M


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




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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread McClintock Matthew-B29882
On Tue, Aug 21, 2012 at 1:06 PM, Khem Raj  wrote:
> On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
>  wrote:
>> On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:
>>> On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock  
>>> wrote:
 +
 +do_configure_prepend (){
 +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
 +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
 +   fi
 +}
>>>
>>>
>>> IMO It would be safer to create a patch for kmod itself where you
>>> define O_CLOEXEC if it
>>> was not defined before. The above seems a bit risky
>>
>> Why is it risky? I only wanted to do this for affected systems. There
>> is not an easy way to do this with a patch, unless of course I apply
>> the patch manually.
>
> manually gripping at the host installation and then if O_CLOEXEC might
> be in comments

How about grep define.*O_CLOEXEC -r ${includedir_native}/bits/fcntl.h

> and furthermore it if it comes from fcntl.h which is not where you are
> looking for

I am grepping this file though?

> there are few variables like that where its impacting more than
> affected systems.

I don't follow...

-M

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


Re: [OE-core] Toolchain rework, call for testing

2012-08-21 Thread Khem Raj
On Tue, Aug 21, 2012 at 4:55 AM, Marcin Juszkiewicz
 wrote:
> W dniu 17.08.2012 06:47, Khem Raj pisze:
>> Hi All
>>
>> Recently glibc build has been simplified upstream. It has dropped the
>> dependency on libgcc_s and libgcc_eh for building glibc itself.
>> This means that we can simplify our toolchain bootstrap a bit by
>> dropping 1 of the 3 cross gcc build stages. We do not need
>> gcc-cross-intermediate anymore.
>
> Please adapt also toolchain-layer in meta-oe for those stuck with gcc 4.6.

Patch for toolchain-layer is sent for meta-linaro The patch is here

https://github.com/kraj/meta-linaro/commits/master

please merge into the meta-linaro layer.

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

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


[OE-core] [PATCH] gcc: bump PR

2012-08-21 Thread Martin Jansa
* missing in 30617bde61a3b0a0944b49a0c9fb7159dacbb19f, eglibc-initial fails to 
compile in incremental build with OEBasic

Signed-off-by: Martin Jansa 
---
 meta/recipes-devtools/gcc/gcc-4.7.inc |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-devtools/gcc/gcc-4.7.inc 
b/meta/recipes-devtools/gcc/gcc-4.7.inc
index 3f13385..36ce1be 100644
--- a/meta/recipes-devtools/gcc/gcc-4.7.inc
+++ b/meta/recipes-devtools/gcc/gcc-4.7.inc
@@ -1,6 +1,6 @@
 require gcc-common.inc
 
-PR = "r9"
+PR = "r10"
 
 # Third digit in PV should be incremented after a minor release
 # happens from this branch on gcc e.g. currently its 4.7.1
-- 
1.7.8.6


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


[OE-core] [PATCH 1/1] runqemu-export-rootfs and friends: don't put pseudo db in target fs

2012-08-21 Thread Peter Seebach
In a few places, we have scripts which use /var/pseudo for
the pseudo state directory controlling a given filesystem. This
seems possibly risky because it means that stuff running under
qemu or whatnot could wipe out the data being used to handle that
rootfs.

Signed-off-by: Peter Seebach 
---
 .../installer/adt-installer/scripts/extract_rootfs |7 +++
 scripts/runqemu-export-rootfs  |2 +-
 scripts/runqemu-extract-sdk|   13 +++--
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git 
a/meta/recipes-devtools/installer/adt-installer/scripts/extract_rootfs 
b/meta/recipes-devtools/installer/adt-installer/scripts/extract_rootfs
index 62dc170..0bbbeef 100755
--- a/meta/recipes-devtools/installer/adt-installer/scripts/extract_rootfs
+++ b/meta/recipes-devtools/installer/adt-installer/scripts/extract_rootfs
@@ -28,7 +28,6 @@ extract_rootfs()
   native_sysroot=$3
   target_sysroot=$2
   PSEUDO_COMMAND="$native_sysroot/usr/bin/pseudo"
-  PSEUDO_OPTS="-P $natvie_sysroot/usr"
   TAR_OPTS="-xjf"
   PSEUDO_OPTS="-P $native_sysroot/usr"
 
@@ -46,9 +45,9 @@ extract_rootfs()
 mkdir -p "$target_sysroot"
   fi
 
-  mkdir -p "$target_sysroot/var/pseudo"
-  touch "$target_sysroot/var/pseudo/pseudo.pid"
-  PSEUDO_LOCALSTATEDIR="$target_sysroot/var/pseudo"
+  mkdir -p "$target_sysroot/../pseudo_state"
+  touch "$target_sysroot/../pseudo_state/pseudo.pid"
+  PSEUDO_LOCALSTATEDIR="$target_sysroot/../pseudo_state"
   export PSEUDO_LOCALSTATEDIR
 
   echo_info "Extracting rootfs: $1, using pseudo..."
diff --git a/scripts/runqemu-export-rootfs b/scripts/runqemu-export-rootfs
index f8213ba..9b7941b 100755
--- a/scripts/runqemu-export-rootfs
+++ b/scripts/runqemu-export-rootfs
@@ -68,7 +68,7 @@ NFSPID=~/.runqemu-sdk/nfs$NFS_INSTANCE.pid
 MOUNTPID=~/.runqemu-sdk/mount$NFS_INSTANCE.pid
 
 PSEUDO_OPTS="-P $OECORE_NATIVE_SYSROOT/usr"
-PSEUDO_LOCALSTATEDIR="$NFS_EXPORT_DIR/var/pseudo"
+PSEUDO_LOCALSTATEDIR="$NFS_EXPORT_DIR/../pseudo_state"
 export PSEUDO_LOCALSTATEDIR
 
 if [ ! -d "$PSEUDO_LOCALSTATEDIR" ]; then
diff --git a/scripts/runqemu-extract-sdk b/scripts/runqemu-extract-sdk
index 4b52475..8bab333 100755
--- a/scripts/runqemu-extract-sdk
+++ b/scripts/runqemu-extract-sdk
@@ -73,15 +73,16 @@ if [ ! -d "$SDK_ROOTFS_DIR" ]; then
mkdir -p "$SDK_ROOTFS_DIR"
 fi
 
-if [ -e "$SDK_ROOTFS_DIR/var/pseudo" ]; then
-   echo "Error: $SDK_ROOTFS_DIR/var/pseudo already exists!"
-   echo "Please delete the entire rootfs tree manually if this is really 
what you want"
+if [ -e "$SDK_ROOTFS_DIR/../pseudo_state" ]; then
+   echo "Error: $SDK_ROOTFS_DIR/../pseudo_state already exists!"
+   echo "Please delete the rootfs tree and pseudo directory manually"
+echo "if this is really what you want."
exit 1
 fi
 
-mkdir -p "$SDK_ROOTFS_DIR/var/pseudo"
-touch "$SDK_ROOTFS_DIR/var/pseudo/pseudo.pid"
-PSEUDO_LOCALSTATEDIR="$SDK_ROOTFS_DIR/var/pseudo"
+mkdir -p "$SDK_ROOTFS_DIR/../pseudo_state"
+touch "$SDK_ROOTFS_DIR/../pseudo_state/pseudo.pid"
+PSEUDO_LOCALSTATEDIR="$SDK_ROOTFS_DIR/../pseudo_state"
 export PSEUDO_LOCALSTATEDIR
 
 echo "Extracting rootfs tarball using pseudo..."
-- 
1.7.0.4


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


[OE-core] [PATCH 0/1] Move PSEUDO_LOCALSTATEDIR outside rootfs

2012-08-21 Thread Peter Seebach
There's a race condition in one of our (non-poky) builds which involves
a rootfs getting deleted while a pseudo process has a db in it open.
This caused me to realize that there existed a pseudo database in a
target filesystem, which made me panic. And we have a few really strange
reports I've never been able to reproduce where target filesystems
under qemu went pear-shaped in ways that look like pseudo suddenly
got really, really, confused.

Like say what would happen if a target could wipe out its own filesystem
permissions, because they were in its rootfs. :P

I haven't tested this super thoroughly, but I can still complete builds
and SDK builds with it, and it makes sense.

The following changes since commit 4ef8960c8d2876914bb78cbdce5fae3c5297e942:
  Khem Raj (1):
gcc-cross-intermediate, gcc-crosssdk-intermediate: Remove

are available in the git repository at:

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

Peter Seebach (1):
  runqemu-export-rootfs and friends: don't put pseudo db in target fs

 .../installer/adt-installer/scripts/extract_rootfs |7 +++
 scripts/runqemu-export-rootfs  |2 +-
 scripts/runqemu-extract-sdk|   13 +++--
 3 files changed, 11 insertions(+), 11 deletions(-)


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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Khem Raj
On Tue, Aug 21, 2012 at 11:10 AM, McClintock Matthew-B29882
 wrote:
> On Tue, Aug 21, 2012 at 1:06 PM, Khem Raj  wrote:
>> On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
>>  wrote:
>>> On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:
 On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock  
 wrote:
> +
> +do_configure_prepend (){
> +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
> +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
> +   fi
> +}


 IMO It would be safer to create a patch for kmod itself where you
 define O_CLOEXEC if it
 was not defined before. The above seems a bit risky
>>>
>>> Why is it risky? I only wanted to do this for affected systems. There
>>> is not an easy way to do this with a patch, unless of course I apply
>>> the patch manually.
>>
>> manually gripping at the host installation and then if O_CLOEXEC might
>> be in comments
>
> How about grep define.*O_CLOEXEC -r ${includedir_native}/bits/fcntl.h
>
>> and furthermore it if it comes from fcntl.h which is not where you are
>> looking for
>
> I am grepping this file though?

I would go into the specific file where its asking for O_CLOEXEC

and add

#ifndef O_CLOEXEC
# define O_CLOEXEC 0
#endif

and be done with it

>
>> there are few variables like that where its impacting more than
>> affected systems.
>
> I don't follow...
>
> -M

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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread McClintock Matthew-B29882
On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:
> On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock  
> wrote:
>> +
>> +do_configure_prepend (){
>> +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
>> +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
>> +   fi
>> +}
>
>
> IMO It would be safer to create a patch for kmod itself where you
> define O_CLOEXEC if it
> was not defined before. The above seems a bit risky

Why is it risky? I only wanted to do this for affected systems. There
is not an easy way to do this with a patch, unless of course I apply
the patch manually.

-M

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


Re: [OE-core] [yocto] Toolchain rework, call for testing

2012-08-21 Thread Martin Jansa
On Tue, Aug 21, 2012 at 10:15:47AM -0700, Khem Raj wrote:
> On Tue, Aug 21, 2012 at 5:18 AM, Martin Jansa  wrote:
> >
> > Maybe it's because
> > http://git.openembedded.org/openembedded-core/commit/?id=30617bde61a3b0a0944b49a0c9fb7159dacbb19f
> > is missing PR bump and gcc wasn't rebuilt before eglibc upgrade (OEBasic 
> > not OEBasicHash here).
> 
> yes seems so. I have stopped using OEBasic long ago, you should stop
> using it too.

I'm using OEBasicHash on one jenkins setup, but that one is still
building changes from 2 days ago.. (building 24/7.. )

> But feel free to send a PR bump patchlet.

-- 
Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com


signature.asc
Description: Digital signature
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] [PATCH] task-base: pull in the bluez asound module when appropriate

2012-08-21 Thread Christopher Larson
From: Christopher Larson 

If the machine and distro both want alsa, then have task-base-bluetooth pull
in the libasound-module-bluez package.

Signed-off-by: Christopher Larson 
---
 meta/recipes-core/tasks/task-base.bb |5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/tasks/task-base.bb 
b/meta/recipes-core/tasks/task-base.bb
index 2325dbf..d5dd140 100644
--- a/meta/recipes-core/tasks/task-base.bb
+++ b/meta/recipes-core/tasks/task-base.bb
@@ -2,7 +2,7 @@ DESCRIPTION = "Merge machine and distro options to create a 
basic machine task/p
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = 
"file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58 \
 
file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
-PR = "r74"
+PR = "r75"
 
 inherit task
 
@@ -205,8 +205,9 @@ RRECOMMENDS_task-base-pcmcia = "\
 
 # Provide bluez-utils-compat utils for the time being, the binaries in that 
package will vanish soon from upstream releases, so beware! 
 
-RDEPENDS_task-base-bluetooth = "\ 
+RDEPENDS_task-base-bluetooth = "\
 bluez4 \
+${@base_contains('COMBINED_FEATURES', 'alsa', 'libasound-module-bluez', 
'',d)} \
 "
 
 RRECOMMENDS_task-base-bluetooth = "\
-- 
1.7.9.5


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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Khem Raj
On Tue, Aug 21, 2012 at 10:59 AM, McClintock Matthew-B29882
 wrote:
> On Tue, Aug 21, 2012 at 12:54 PM, Khem Raj  wrote:
>> On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock  
>> wrote:
>>> +
>>> +do_configure_prepend (){
>>> +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
>>> +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
>>> +   fi
>>> +}
>>
>>
>> IMO It would be safer to create a patch for kmod itself where you
>> define O_CLOEXEC if it
>> was not defined before. The above seems a bit risky
>
> Why is it risky? I only wanted to do this for affected systems. There
> is not an easy way to do this with a patch, unless of course I apply
> the patch manually.

manually gripping at the host installation and then if O_CLOEXEC might
be in comments
and furthermore it if it comes from fcntl.h which is not where you are
looking for
there are few variables like that where its impacting more than
affected systems.


>
> -M

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


Re: [OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Khem Raj
On Tue, Aug 21, 2012 at 9:20 AM, Matthew McClintock  wrote:
> +
> +do_configure_prepend (){
> +   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
> +   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
> +   fi
> +}


IMO It would be safer to create a patch for kmod itself where you
define O_CLOEXEC if it
was not defined before. The above seems a bit risky

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


Re: [OE-core] qemu gl changes are broken.

2012-08-21 Thread Scott Garman

On 08/21/2012 09:21 AM, Scott Garman wrote:

Hello,

I tried building some images overnight with the latest master, and could
not get them to run this morning with qemu - I got an odd error that a
VNC server was started instead?

Just on a hunch, I reverted the following commits:

qemu: Fix broken accidental path move
qemu-git: Drop GL passthrough support
qemu-0.15: Drop GL passthrough support

rebuilt qemu-native, and now qemu works correctly.

I have no idea why qemu is trying to fallback on a VNC server, as I did
not have one running on my machine. I know we use VNC for our
autobuilder sanity tests, but those haven't been integrated into the
runqemu scripts as far as I can tell.


This may need further research, I seem to have it working on another 
machine. I'll report back once I've learned more.


Scott

--
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center

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


Re: [OE-core] [yocto] RFC: OE-Core task rework

2012-08-21 Thread Philip Balister

On 08/21/2012 01:49 AM, Paul Eggleton wrote:

On Monday 20 August 2012 15:45:07 Mark Hatle wrote:

...



Yup, there is a logical grouping for the lsb specific tasks, as for others.
The naming needs to be made clear as to why it's there, and what it
represents. Same with the summary and descriptions -- they should list the
functionality provided by this group of components.


The main concern with LSB is that we have something called task-basic, which
seems to be intended for LSB but does not state as much, and I know at least
one person has tried to use it and then been a little puzzled as to why rpm
has been pulled in when ipk packaging has been selected. Just naming some of
these tasks more appropriately would help avoid such situations. In the
specific case of task-basic there may be a case for creating a similar but not
LSB-focused task that pulls in real shell utilities in preference to the light
versions provided by busybox.


I was seriously stumped by the presence of rpm in what I felt should 
have been a very basic image. If a task/group is targeted at lsb, it 
needs to have lsb plastered all over the name so the rest of us can 
avoid it :)


Philip




4) Consider the usefulness of the existing PACKAGE_GROUP_ structure which
converts some IMAGE_FEATURES into the addition of tasks to be installed
(see classes/core-image.bbclass). At the very least we should probably
rename this if we rename tasks to package groups, and there's a question
as to how IMAGE_FEATURES scales - should every task be represented in
there or only a select list? Would we be better off not trying to bring
any tasks in via IMAGE_FEATURES at all and mostly leave that to control
post-processing (e.g. package-management, debug-tweaks, etc.)?


I'd certainly prefer that images were limited to selecting software from
task (group) recipes only, and not providing their own.  An image should be
able to change the selection based on an "image feature" or similar
configuration, but the underlying tasks/groups, recipes, etc should all be
'generic' to a given distribution configuration.


The first part seems reasonable to me; but I'm still short of an answer as to
what to do with the existing IMAGE_FEATURES code as mentioned above. At the
moment aside from the special features such as debug-tweaks, these are just a
thin layer on top of task recipes which doesn't add very much at all other
than extra maintenance.


I think if you go through the images today, a lot of that stuff is either
old, or could be simplified by creating appropriate tasks/groups.


OK, that's a good point. I have another task on my list to clean up our image
recipes and that would be a good segue into that.

Cheers,
Paul



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


[OE-core] [PATCH] conf/bitbake.conf: allow cache reuse between bitbake, bitbake-layers and hob

2012-08-21 Thread Paul Eggleton
BB_ENV_EXTRAWHITE is set by the bitbake wrapper script but not by
bitbake-layers, and thus it was affecting the data hash and thus
invalidating the cache. Any variables passed through from the
environment by being added to this variable should themselves
influence the hash already if they are not already excluded from,
the data hash, so BB_ENV_EXTRAWHITE can be excluded.

Similarly, DISABLE_SANITY_CHECKS is set by the hob script in order to
have sanity checks run later, but not set by the bitbake wrapper script.
It doesn't actually affect the build itself and so its value can also
be excluded from the data hash.

These changes prevent unnecessary reparsing between executions of
bitbake, bitbake-layers, and the initial pseudo build within the hob
script.

Fixes [YOCTO #2680].

Signed-off-by: Paul Eggleton 
---
 meta/conf/bitbake.conf |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index bc2d47d..ad98756 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -762,7 +762,7 @@ BB_CONSOLELOG ?= 
"${LOG_DIR}/cooker/${MACHINE}/${DATETIME}.log"
 # Setup our default hash policy
 BB_SIGNATURE_HANDLER ?= "OEBasicHash"
 BB_HASHBASE_WHITELIST ?= "TMPDIR FILE PATH PWD BB_TASKHASH BBPATH DL_DIR 
SSTATE_DIR THISDIR FILESEXTRAPATHS FILE_DIRNAME HOME LOGNAME SHELL TERM USER 
FILESPATH STAGING_DIR_HOST STAGING_DIR_TARGET COREBASE PRSERV_HOST PRSERV_PORT 
PRSERV_DUMPDIR PRSERV_DUMPFILE PRSERV_LOCKDOWN PARALLEL_MAKE CCACHE_DIR 
EXTERNAL_TOOLCHAIN CCACHE"
-BB_HASHCONFIG_WHITELIST ?= "${BB_HASHBASE_WHITELIST} DATE TIME SESSION_MANAGER 
DBUS_SESSION_BUS_ADDRESS SSH_AGENT_PID XDG_SESSION_COOKIE SSH_AUTH_SOCK 
XAUTHORITY PSEUDO_BUILD"
+BB_HASHCONFIG_WHITELIST ?= "${BB_HASHBASE_WHITELIST} DATE TIME SESSION_MANAGER 
DBUS_SESSION_BUS_ADDRESS SSH_AGENT_PID XDG_SESSION_COOKIE SSH_AUTH_SOCK 
XAUTHORITY PSEUDO_BUILD BB_ENV_EXTRAWHITE DISABLE_SANITY_CHECKS"
 BB_SIGNATURE_EXCLUDE_FLAGS ?= "doc defaultval _append _prepend deps depends 
lockfiles type vardepsexclude \
vardeps vardepvalue file-checksums python func 
task export unexport noexec \
nostamp dirs cleandirs sstate-lockfile-shared 
prefuncs postfuncs export_func \
-- 
1.7.9.5


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


Re: [OE-core] [yocto] Toolchain rework, call for testing

2012-08-21 Thread Khem Raj
On Tue, Aug 21, 2012 at 5:18 AM, Martin Jansa  wrote:
>
> Maybe it's because
> http://git.openembedded.org/openembedded-core/commit/?id=30617bde61a3b0a0944b49a0c9fb7159dacbb19f
> is missing PR bump and gcc wasn't rebuilt before eglibc upgrade (OEBasic not 
> OEBasicHash here).

yes seems so. I have stopped using OEBasic long ago, you should stop
using it too.
But feel free to send a PR bump patchlet.

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


Re: [OE-core] Toolchain rework, call for testing

2012-08-21 Thread Khem Raj
On Tue, Aug 21, 2012 at 4:55 AM, Marcin Juszkiewicz
 wrote:
> W dniu 17.08.2012 06:47, Khem Raj pisze:
>> Hi All
>>
>> Recently glibc build has been simplified upstream. It has dropped the
>> dependency on libgcc_s and libgcc_eh for building glibc itself.
>> This means that we can simplify our toolchain bootstrap a bit by
>> dropping 1 of the 3 cross gcc build stages. We do not need
>> gcc-cross-intermediate anymore.
>
> Please adapt also toolchain-layer in meta-oe for those stuck with gcc 4.6.
>

Yes. I am testing a patch for it. For tries just delete the *intermediate* files
and see if it helps, I believe you will need it for meta-linaro as well.

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

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


[OE-core] qemu gl changes are broken.

2012-08-21 Thread Scott Garman

Hello,

I tried building some images overnight with the latest master, and could 
not get them to run this morning with qemu - I got an odd error that a 
VNC server was started instead?


Just on a hunch, I reverted the following commits:

qemu: Fix broken accidental path move
qemu-git: Drop GL passthrough support
qemu-0.15: Drop GL passthrough support

rebuilt qemu-native, and now qemu works correctly.

I have no idea why qemu is trying to fallback on a VNC server, as I did 
not have one running on my machine. I know we use VNC for our 
autobuilder sanity tests, but those haven't been integrated into the 
runqemu scripts as far as I can tell.


Scott

--
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center

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


Re: [OE-core] [PATCH] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread McClintock Matthew-B29882
On Tue, Aug 21, 2012 at 5:49 AM, Richard Purdie
 wrote:
> On Mon, 2012-08-20 at 13:49 -0500, Matthew McClintock wrote:
>> kmod will fail to build with the following error because O_CLOEXEC is
>> not defined:
>>
>> | libkmod/libkmod-module.c: In function 'kmod_module_get_initstate':
>> | libkmod/libkmod-module.c:1640: error: 'O_CLOEXEC' undeclared (first use in 
>> this function)
>> | libkmod/libkmod-module.c:1640: error: (Each undeclared identifier is 
>> reported only once
>> | libkmod/libkmod-module.c:1640: error: for each function it appears in.)
>> | libkmod/libkmod-module.c: In function 'kmod_module_get_refcnt':
>> | libkmod/libkmod-module.c:1754: error: 'O_CLOEXEC' undeclared (first use in 
>> this function)
>> | libkmod/libkmod-module.c: In function 'kmod_module_get_sections':
>> | libkmod/libkmod-module.c:1913: error: 'O_CLOEXEC' undeclared (first use in 
>> this function)
>> | libkmod/libkmod-file.c: In function 'kmod_file_open':
>> | libkmod/libkmod-file.c:282: error: 'O_CLOEXEC' undeclared (first use in 
>> this function)
>> | libkmod/libkmod-file.c:282: error: (Each undeclared identifier is reported 
>> only once
>> | libkmod/libkmod-file.c:282: error: for each function it appears in.)
>>
>> Since we are only using kmod-native for depmod, and it's a non-threaded
>> user of this libary being built this should be safe to override O_CLOEXEC.
>>
>> Keep in mind this is ONLY effecting the native builds and not what is
>> being shipped in the root file system.
>>
>> Signed-off-by: Matthew McClintock 
>> ---
>>  meta/recipes-kernel/kmod/kmod-native_git.bb |8 +++-
>>  1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-kernel/kmod/kmod-native_git.bb 
>> b/meta/recipes-kernel/kmod/kmod-native_git.bb
>> index 96de8b8..3c5e3c3 100644
>> --- a/meta/recipes-kernel/kmod/kmod-native_git.bb
>> +++ b/meta/recipes-kernel/kmod/kmod-native_git.bb
>> @@ -4,7 +4,7 @@
>>  require kmod.inc
>>  inherit native
>>
>> -PR = "${INC_PR}.0"
>> +PR = "${INC_PR}.1"
>>
>>  do_install_append (){
>>   for tool in depmod insmod lsmod modinfo modprobe rmmod
>> @@ -12,3 +12,9 @@ do_install_append (){
>>   ln -s kmod ${D}${bindir}/$tool
>>   done
>>  }
>> +
>> +do_compile_append (){
>> + if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
>> + export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
>> + fi
>> +}
>
>
> A compile append would execute after the compile. Isn't this after the
> point you need the change?

Yes, and this makes me wonder what I tested...

Anyways, v2 fixed up and sent. I also needed to export the CFLAGS even
earlier so they got picked up correctly.

-M

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


[OE-core] [PATCH v2] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Matthew McClintock
kmod will fail to build with the following error because O_CLOEXEC is
not defined:

| libkmod/libkmod-module.c: In function 'kmod_module_get_initstate':
| libkmod/libkmod-module.c:1640: error: 'O_CLOEXEC' undeclared (first use in 
this function)
| libkmod/libkmod-module.c:1640: error: (Each undeclared identifier is reported 
only once
| libkmod/libkmod-module.c:1640: error: for each function it appears in.)
| libkmod/libkmod-module.c: In function 'kmod_module_get_refcnt':
| libkmod/libkmod-module.c:1754: error: 'O_CLOEXEC' undeclared (first use in 
this function)
| libkmod/libkmod-module.c: In function 'kmod_module_get_sections':
| libkmod/libkmod-module.c:1913: error: 'O_CLOEXEC' undeclared (first use in 
this function)
| libkmod/libkmod-file.c: In function 'kmod_file_open':
| libkmod/libkmod-file.c:282: error: 'O_CLOEXEC' undeclared (first use in this 
function)
| libkmod/libkmod-file.c:282: error: (Each undeclared identifier is reported 
only once
| libkmod/libkmod-file.c:282: error: for each function it appears in.)

Since we are only using kmod-native for depmod, and it's a non-threaded
user of this libary being built this should be safe to override O_CLOEXEC.

Keep in mind this is ONLY effecting the native builds and not what is
being shipped in the root file system.

Signed-off-by: Matthew McClintock 
---
 meta/recipes-kernel/kmod/kmod-native_git.bb |8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/kmod/kmod-native_git.bb 
b/meta/recipes-kernel/kmod/kmod-native_git.bb
index 96de8b8..4558c7b 100644
--- a/meta/recipes-kernel/kmod/kmod-native_git.bb
+++ b/meta/recipes-kernel/kmod/kmod-native_git.bb
@@ -4,7 +4,7 @@
 require kmod.inc
 inherit native
 
-PR = "${INC_PR}.0"
+PR = "${INC_PR}.1"
 
 do_install_append (){
for tool in depmod insmod lsmod modinfo modprobe rmmod
@@ -12,3 +12,9 @@ do_install_append (){
ln -s kmod ${D}${bindir}/$tool
done
 }
+
+do_configure_prepend (){
+   if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
+   export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
+   fi
+}
-- 
1.7.9.7



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


Re: [OE-core] [PATCH v4] busybox: Upgrade to upstream 1.20.2

2012-08-21 Thread Saul Wold

On 08/21/2012 01:23 AM, Radu Moisan wrote:

Updated defconfig file from generated .config
Disabled CONFIG_LSOF,CONFIG_CROSS_COMPILER_PREFIX, and CONFIG_EXTRA_CFLAGS
for backwards compatibility.
Added busybox-1.20.2-kernel_ver.patch from upstream.

Signed-off-by: Radu Moisan 
---
  .../poky-tiny/defconfig|0
  ...box_1.19.4.bbappend => busybox_1.20.2.bbappend} |0
  .../B921600.patch  |0
  .../busybox-1.20.2/busybox-1.20.2-kernel_ver.patch |   25 +
  .../busybox-appletlib-dependency.patch |0
  .../busybox-mkfs-minix-tests_bigendian.patch   |0
  .../busybox-udhcpc-no_deconfig.patch   |   37 +--
  .../{busybox-1.19.4 => busybox-1.20.2}/defconfig   |   39 +---
  .../fix-for-spurious-testsuite-failure.patch   |0
  .../get_header_tar.patch   |0
  .../run-parts.in.usr-bin.patch |0
  .../sys_resource.patch |0
  .../watch.in.usr-bin.patch |0
  .../wget_dl_dir_fix.patch  |0
  .../{busybox_1.19.4.bb => busybox_1.20.2.bb}   |9 +++--
  15 files changed, 72 insertions(+), 38 deletions(-)
  rename meta-yocto/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/poky-tiny/defconfig (100%)
  rename meta-yocto/recipes-core/busybox/{busybox_1.19.4.bbappend => 
busybox_1.20.2.bbappend} (100%)


This needs to be split into 2 patches, one for meta-yocto (sent to 
poky@yoctoproject,org and the rest of them to the OE-Core list.


Otherwise this looks good, I think you have addressed the other issues.

Sau!



  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/B921600.patch (100%)
  create mode 100644 
meta/recipes-core/busybox/busybox-1.20.2/busybox-1.20.2-kernel_ver.patch
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/busybox-appletlib-dependency.patch (100%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/busybox-mkfs-minix-tests_bigendian.patch (100%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/busybox-udhcpc-no_deconfig.patch (78%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => busybox-1.20.2}/defconfig 
(98%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/fix-for-spurious-testsuite-failure.patch (100%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/get_header_tar.patch (100%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/run-parts.in.usr-bin.patch (100%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/sys_resource.patch (100%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/watch.in.usr-bin.patch (100%)
  rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/wget_dl_dir_fix.patch (100%)
  rename meta/recipes-core/busybox/{busybox_1.19.4.bb => busybox_1.20.2.bb} 
(79%)

diff --git a/meta-yocto/recipes-core/busybox/busybox-1.19.4/poky-tiny/defconfig 
b/meta-yocto/recipes-core/busybox/busybox-1.20.2/poky-tiny/defconfig
similarity index 100%
rename from meta-yocto/recipes-core/busybox/busybox-1.19.4/poky-tiny/defconfig
rename to meta-yocto/recipes-core/busybox/busybox-1.20.2/poky-tiny/defconfig
diff --git a/meta-yocto/recipes-core/busybox/busybox_1.19.4.bbappend 
b/meta-yocto/recipes-core/busybox/busybox_1.20.2.bbappend
similarity index 100%
rename from meta-yocto/recipes-core/busybox/busybox_1.19.4.bbappend
rename to meta-yocto/recipes-core/busybox/busybox_1.20.2.bbappend
diff --git a/meta/recipes-core/busybox/busybox-1.19.4/B921600.patch 
b/meta/recipes-core/busybox/busybox-1.20.2/B921600.patch
similarity index 100%
rename from meta/recipes-core/busybox/busybox-1.19.4/B921600.patch
rename to meta/recipes-core/busybox/busybox-1.20.2/B921600.patch
diff --git 
a/meta/recipes-core/busybox/busybox-1.20.2/busybox-1.20.2-kernel_ver.patch 
b/meta/recipes-core/busybox/busybox-1.20.2/busybox-1.20.2-kernel_ver.patch
new file mode 100644
index 000..456fb08
--- /dev/null
+++ b/meta/recipes-core/busybox/busybox-1.20.2/busybox-1.20.2-kernel_ver.patch
@@ -0,0 +1,25 @@
+--- busybox-1.20.2/libbb/kernel_version.c
 busybox-1.20.2-kernel_ver/libbb/kernel_version.c
+@@ -20,18 +20,15 @@
+ int FAST_FUNC get_linux_version_code(void)
+ {
+   struct utsname name;
+-  char *s;
++  char *s, *t;
+   int i, r;
+
+-  if (uname(&name) == -1) {
+-  bb_perror_msg("can't get system information");
+-  return 0;
+-  }
+-
++  uname(&name); /* never fails */
+   s = name.release;
+   r = 0;
+   for (i = 0; i < 3; i++) {
+-  r = r * 256 + atoi(strtok(s, "."));
++  t = strtok(s, ".");
++  r = r * 256 + (t ? atoi(t) : 0);
+   s = NULL;
+   }
+   return r;
diff --git 
a/meta/recipes-core/busybox/busybox-1.19.4

Re: [OE-core] [PATCH 0/1] package_rpm.bbclass: fix and enhance the incremental rpm generation

2012-08-21 Thread Saul Wold

On 08/20/2012 11:48 PM, Robert Yang wrote:

* Test info:
$ bitbake core-image-sato core-image-sato-sdk meta-toolchain meta-toolchain-sdk
$ bitbake package-index

Tested the incremental rpm generation based on core-image-minimal and
core-image-sato

* Note: it doesn't have any conflict with the "Install external binary packages"
   patches which are in Patch Review.

// Robert

The following changes since commit 7c40daab587306da25c8d8f566392ee1f57bbef4:

   qemu: Fix broken accidental path move (2012-08-20 17:25:28 +0100)

are available in the git repository at:
   git://git.pokylinux.org/poky-contrib robert/inc_rpm
   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/inc_rpm

Robert Yang (1):
   package_rpm.bbclass: fix and enhance the incremental rpm generation

  meta/classes/package_rpm.bbclass |   68 --
  1 files changed, 43 insertions(+), 25 deletions(-)


Merged into OE-Core

Thanks
Sau!




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




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


Re: [OE-core] [PATCH 0/9] Rework cross toolchain bootstrap

2012-08-21 Thread Saul Wold

On 08/20/2012 08:07 PM, Khem Raj wrote:

This patchset drops one gcc bootstrap phase. It has been possible because of
some work upstream glibc to drop build time dependency on libgcc_s and libgcc_eh

I have built all possible images in OE-Core for all qemu machines
all of them built fine except *-rt and *-initramfs which failed due to unrelated
issues. I have built from scratch as well as built from sstate. Seems to go
ok.


The following changes since commit 04568d1f18250d531aad5d286157d2d559083520:

   qemu: Fix broken accidental path move (2012-08-20 17:25:21 +0100)

are available in the git repository at:

   git://git.openembedded.org/openembedded-core-contrib kraj/toolchain-rework
   
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=kraj/toolchain-rework

Khem Raj (9):
   eglibc-2.16: Remove build dependency on libgcc_s and libgcc_eh
   eglibc-2.16: Update to top of 2.16 branch
   gcc-cross: Make gcc-cross-initial as the only intermediate gcc stage
   eglibc: Fix eglibc-initial and let eglibc depend on it
   uclibc.inc,uclibc-config.inc: Untabify python snippets
   uclibc: Use gcc-cross-initial as staging compiler
   gcc-crosssdk-initial_4.7.bb: Add --with-native-system-header-dir to
 EXTRA_OECONF
   gcc-cross-intermediate, gcc-crosssdk-intermediate: Remove

Merged the above set of 8 into OE-Core

Thanks
Sau!


   gcc: Switch SRC_URI to use svn

  .../0001-Add-ARM-specific-static-stubs.c.patch |   27 ++
  ...f-libgcc_s-and-libgcc_eh-when-building-gl.patch |  343 
  meta/recipes-core/eglibc/eglibc-initial.inc|   16 +-
  meta/recipes-core/eglibc/eglibc-testing.inc|2 +-
  meta/recipes-core/eglibc/eglibc.inc|   11 +-
  meta/recipes-core/eglibc/eglibc_2.16.bb|6 +-
  meta/recipes-core/uclibc/uclibc-config.inc |  128 
  meta/recipes-core/uclibc/uclibc-initial_0.9.33.bb  |   10 +-
  meta/recipes-core/uclibc/uclibc-initial_git.bb |   10 +-
  meta/recipes-core/uclibc/uclibc.inc|   28 +-
  meta/recipes-core/uclibc/uclibc_0.9.33.bb  |7 +-
  meta/recipes-core/uclibc/uclibc_git.bb |7 +-
  meta/recipes-devtools/gcc/gcc-4.7.inc  |   11 +-
  .../gcc/gcc-4.7/arm-nolibfloat.patch   |   40 ---
  meta/recipes-devtools/gcc/gcc-cross-initial.inc|   50 ++-
  .../gcc/gcc-cross-intermediate.inc |   68 
  .../gcc/gcc-cross-intermediate_4.7.bb  |3 -
  .../gcc/gcc-crosssdk-initial_4.7.bb|2 +
  .../gcc/gcc-crosssdk-intermediate.inc  |9 -
  .../gcc/gcc-crosssdk-intermediate_4.7.bb   |3 -
  20 files changed, 534 insertions(+), 247 deletions(-)
  create mode 100644 
meta/recipes-core/eglibc/eglibc-2.16/0001-Add-ARM-specific-static-stubs.c.patch
  create mode 100644 
meta/recipes-core/eglibc/eglibc-2.16/0001-Avoid-use-of-libgcc_s-and-libgcc_eh-when-building-gl.patch
  delete mode 100644 meta/recipes-devtools/gcc/gcc-4.7/arm-nolibfloat.patch
  delete mode 100644 meta/recipes-devtools/gcc/gcc-cross-intermediate.inc
  delete mode 100644 meta/recipes-devtools/gcc/gcc-cross-intermediate_4.7.bb
  delete mode 100644 meta/recipes-devtools/gcc/gcc-crosssdk-intermediate.inc
  delete mode 100644 meta/recipes-devtools/gcc/gcc-crosssdk-intermediate_4.7.bb



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


Re: [OE-core] [PATCH] insane.bbclass: Fix RPATH warning in the face of funny path strings

2012-08-21 Thread Saul Wold

On 08/20/2012 02:05 PM, Andy Ross wrote:

In toolchain edge cases it's possible for the RPATH of a library to be
set to something like "/usr/lib/../lib".  This should be detected as
"/usr/lib" and generate a warning.

Signed-off-by: Andy Ross 
---
  meta/classes/insane.bbclass | 5 -
  1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 556a176..9d085a4 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -166,6 +166,9 @@ def package_qa_check_useless_rpaths(file, name, d, elf, 
messages):
  """
  Check for RPATHs that are useless but not dangerous
  """
+def rpath_eq(a, b):
+return os.path.normpath(a) == os.path.normpath(b)
+
  if not elf:
  return

@@ -181,7 +184,7 @@ def package_qa_check_useless_rpaths(file, name, d, elf, 
messages):
m = rpath_re.match(line)
if m:
   rpath = m.group(1)
-  if rpath == libdir or rpath == base_libdir:
+  if rpath_eq(rpath, libdir) or rpath_eq(rpath, base_libdir):
  # The dynamic linker searches both these places anyway.  There is 
no point in
  # looking there again.
  messages.append("%s: %s contains probably-redundant RPATH %s" % 
(name, package_qa_clean_path(file, d), rpath))



Merged this to OE-Core

Thanks
Sau!


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


Re: [OE-core] [PATCH 0/1] coreutils: fix to build with acl support

2012-08-21 Thread Saul Wold

On 08/21/2012 12:42 AM, jackie.hu...@windriver.com wrote:

From: Jackie Huang 

Fix the issue that coreutils was built without acl support:

configure: WARNING: libacl development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without ACL support.

which will cause ls and cp commands don't have the ACL related behaviors:
ls -l: no plus sign(+) after the permission string for files that have ACL
cp -p: can not preserves the ACLs.

[YOCTO #2959]

Signed-off-by: Jackie Huang 
---
The following changes since commit 04568d1f18250d531aad5d286157d2d559083520:

   qemu: Fix broken accidental path move (2012-08-20 17:25:21 +0100)

are available in the git repository at:
   git://git.pokylinux.org/poky-contrib jhuang0/bug2959_acl_0821
   
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=jhuang0/bug2959_acl_0821

Jackie Huang (1):
   coreutils: fix to build with acl support

  .../coreutils-6.9/coreutils-build-with-acl.patch   |   28 
  .../coreutils-8.14/coreutils-build-with-acl.patch  |   28 
  meta/recipes-core/coreutils/coreutils_6.9.bb   |1 +
  meta/recipes-core/coreutils/coreutils_8.14.bb  |1 +
  4 files changed, 58 insertions(+), 0 deletions(-)
  create mode 100644 
meta/recipes-core/coreutils/coreutils-6.9/coreutils-build-with-acl.patch
  create mode 100644 
meta/recipes-core/coreutils/coreutils-8.14/coreutils-build-with-acl.patch


Merged into OE-Core

Please remember to add PR Bumps for recipe changes.

Thanks
Sau!


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


Re: [OE-core] [PATCH] libxpm-native: add dependency on gettext-native

2012-08-21 Thread Saul Wold

On 08/20/2012 06:35 PM, Jesse Zhang wrote:

Hi Saul,

On 08/21/2012 05:16 AM, Saul Wold wrote:

On 08/02/2012 06:57 PM, Jesse Zhang wrote:

libxpm needs xgettext to build po files. Add explicit dependency on
gettext-native.

Signed-off-by: Jesse Zhang 
---
   meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb |3 ++-
   1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb 
b/meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb
index 9eaa212..6830216 100644
--- a/meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb
+++ b/meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb
@@ -10,7 +10,8 @@ protocol."
   LICENSE = "BSD"
   LIC_FILES_CHKSUM = "file://COPYING;md5=3e07763d16963c3af12db271a31abaa5"
   DEPENDS += "libxext libsm libxt"
-PR = "r0"
+DEPENDS_virtclass-native = "gettext-native"


Is there a reason you don't do an inherit gettext here to ensure other bits are 
set?


Actually I don't know what's the preferred way to do this. What is
required by libxpm is the xgettext command; using DEPENDS will be enough
to make it available, but I didn't know the other solution.

Ok, well I am telling you there is another, more correct, way to do this 
using the inherit gettext, so please try using inherit gettext and see 
if that works correctly.  The meta/classes/gettext.bbclass in 
conjunction with native.bbclass correctly set the DEPENDS to use 
gettext-native.


Also, on another note, please modify your patch subject to just say 
libxpm, this initial confused a couple of us thinking that libxpm was a 
native specific recipe.


Sau!



Also for the native, you maybe able to use the gettext-minimal-native if there 
is reason you can't use the inherit.


gettext-minimal-native is already pulled in through the implicit
dependency chain. But clearly it doesn't provide xgettext?

jesse




+PR = "r1"
   PE = "1"

   XORG_PN = "libXpm"









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


Re: [OE-core] [PATCH 0/1] apr-util: fix the rules.mk with path to sysroot instead of the workdir of apr

2012-08-21 Thread Saul Wold

On 08/16/2012 08:11 PM, jackie.hu...@windriver.com wrote:

From: Jackie Huang 

The copying for rules.mk was happened in do_configure_prepend, but it will be
replaced by the one generated by configure, in which APR_MKEXPORT points to
the workdir of apr and cause compile filure when the workdir of apr is removed.
So change the copying in _prepend to _append.

[YOCTO #2947]

Signed-off-by: Jackie Huang 
---
The following changes since commit 24babf9316da50c8a4d2f328c4336cb8cd6cf667:

   classes/chrpath: trigger an error if chrpath fails (2012-08-15 17:51:04 
+0100)

are available in the git repository at:
   git://git.pokylinux.org/poky-contrib jhuang0/bug2947_apr_0817
   
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=jhuang0/bug2947_apr_0817

Jackie Huang (1):
   apr-util: fix the rules.mk with path to sysroot instead of the
 workdir of apr

  meta/recipes-support/apr/apr-util_1.4.1.bb |6 --
  1 files changed, 4 insertions(+), 2 deletions(-)


Merged into OE-Core

Thanks
Sau!


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


Re: [OE-core] [OE-Core] error expanding variable do_*

2012-08-21 Thread Paul Eggleton
On Tuesday 21 August 2012 15:52:17 Giuseppe Condorelli wrote:
> > > []$ bitbake core-image-minimal
> > > Pseudo is not present but is required, building this first before the
> > 
> > main
> > 
> > > build
> > > NOTE: Error expanding variable do_unpack| ETA:
> > > 00:00:04
> > > NOTE: Error during finalise of
> > > /oelinux/condorg/openembedded-core/meta/recipes-sato/web/web_git.bb
> > > ERROR: Unable to parse
> > > /oelinux/condorg/openembedded-core/meta/recipes-sato/web/web_git.bb
> > > ERROR: Command execution failed: Exited with 1
> > > 
> > > I tried with bitbake version 1.15.2 and 1.15.3, the latest two versions.
> > > What about the "Error expanding variable do_unpack?" I've read something
> > 
> > in
> > 
> > > previous message on mailing list
> > > but with no understanding.
> > 
> > This is almost certainly because of the whitespace changes in python
> > functions.
> 
> What type of whitespace changes you are talking about?

Please see:
http://lists.linuxtogo.org/pipermail/openembedded-core/2012-July/025962.html

> > If you use master of OE-Core, you must use BitBake 1.15.3+; if you
> > use denzil, you need to use 1.15.2. I'm a bit confused as to how you could
> > have still got this error with 1.15.2 though since that change was not in
> > that version (unless you are using master of OE-Core with BitBake 1.15.2).
>  
> I've pulled last two months updates for master OE-Core branch and I tried
> with both bitbake 1.15.2 and 1.15.3 versions.

I just tried BitBake 1.15.3 with OE-Core latest master and I do not see that 
error. Are you using latest master of OE-Core or some specific revision? Can 
you provide me with any other information that would help me to reproduce the 
issue? If you check out OE-Core in a separate new directory and use BitBake 
1.15.3 do you still get the same issue?

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

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


Re: [OE-core] [OE-Core] error expanding variable do_*

2012-08-21 Thread Giuseppe Condorelli
2012/8/21 Paul Eggleton 

> On Tuesday 21 August 2012 15:15:42 Giuseppe Condorelli wrote:
> > Hi,
> >
>
Hi Paul,

many thanks for your quick reply.


> > today I restarted to work on my own OE meta layer and, after having
> pulled
> > last two months changes is OE-Core, I was able to
> > build nothing.
> > This is the error I met
> >
> > []$ bitbake core-image-minimal
> > Pseudo is not present but is required, building this first before the
> main
> > build
> > NOTE: Error expanding variable do_unpack| ETA:
> > 00:00:04
> > NOTE: Error during finalise of
> > /oelinux/condorg/openembedded-core/meta/recipes-sato/web/web_git.bb
> > ERROR: Unable to parse
> > /oelinux/condorg/openembedded-core/meta/recipes-sato/web/web_git.bb
> > ERROR: Command execution failed: Exited with 1
> >
> > I tried with bitbake version 1.15.2 and 1.15.3, the latest two versions.
> > What about the "Error expanding variable do_unpack?" I've read something
> in
> > previous message on mailing list
> > but with no understanding.
>
> This is almost certainly because of the whitespace changes in python
> functions.


What type of whitespace changes you are talking about?


> If you use master of OE-Core, you must use BitBake 1.15.3+; if you
> use denzil, you need to use 1.15.2. I'm a bit confused as to how you could
> have still got this error with 1.15.2 though since that change was not in
> that
> version (unless you are using master of OE-Core with BitBake 1.15.2).
>
>  I've pulled last two months updates for master OE-Core branch and I tried
with both bitbake 1.15.2 and 1.15.3 versions.

> Cheers,
> Paul
>
> --
>
> Paul Eggleton
> Intel Open Source Technology Centre
>
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [OE-Core] error expanding variable do_*

2012-08-21 Thread Paul Eggleton
On Tuesday 21 August 2012 15:15:42 Giuseppe Condorelli wrote:
> Hi,
> 
> today I restarted to work on my own OE meta layer and, after having pulled
> last two months changes is OE-Core, I was able to
> build nothing.
> This is the error I met
> 
> []$ bitbake core-image-minimal
> Pseudo is not present but is required, building this first before the main
> build
> NOTE: Error expanding variable do_unpack| ETA:
> 00:00:04
> NOTE: Error during finalise of
> /oelinux/condorg/openembedded-core/meta/recipes-sato/web/web_git.bb
> ERROR: Unable to parse
> /oelinux/condorg/openembedded-core/meta/recipes-sato/web/web_git.bb
> ERROR: Command execution failed: Exited with 1
> 
> I tried with bitbake version 1.15.2 and 1.15.3, the latest two versions.
> What about the "Error expanding variable do_unpack?" I've read something in
> previous message on mailing list
> but with no understanding.

This is almost certainly because of the whitespace changes in python 
functions. If you use master of OE-Core, you must use BitBake 1.15.3+; if you 
use denzil, you need to use 1.15.2. I'm a bit confused as to how you could 
have still got this error with 1.15.2 though since that change was not in that 
version (unless you are using master of OE-Core with BitBake 1.15.2).

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

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


[OE-core] [PATCH] scripts/combo-layer: specify branch when getting current revision

2012-08-21 Thread Paul Eggleton
Handle the case during update where the configured branch is not
currently checked out in the component repository by just specifying
it in the places where it was not previously.

Signed-off-by: Paul Eggleton 
---
 scripts/combo-layer |5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/scripts/combo-layer b/scripts/combo-layer
index 448fe71..330faca 100755
--- a/scripts/combo-layer
+++ b/scripts/combo-layer
@@ -191,7 +191,7 @@ def action_init(conf, args):
 branch = repo.get('branch', "master")
 file_filter = repo.get('file_filter', "")
 runcmd("git archive %s | tar -x -C %s %s" % (branch, extract_dir, 
file_filter), ldir)
-lastrev = runcmd("git rev-parse HEAD", ldir).strip()
+lastrev = runcmd("git rev-parse %s" % branch, ldir).strip()
 conf.update(name, "last_revision", lastrev, initmode=True)
 runcmd("git add .")
 if conf.localconffile:
@@ -475,7 +475,8 @@ def apply_patchlist(conf, repos):
 else:
 logger.info("No patches to apply from %s" % name)
 ldir = conf.repos[name]['local_repo_dir']
-lastrev = runcmd("git rev-parse HEAD", ldir).strip()
+branch = conf.repos[name].get('branch', "master")
+lastrev = runcmd("git rev-parse %s" % branch, ldir).strip()
 
 if lastrev != repo['last_revision']:
 conf.update(name, "last_revision", lastrev)
-- 
1.7.9.5


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


Re: [OE-core] [PATCH] scripts/combo-layer: checkout branch in update when pull is disabled

2012-08-21 Thread Paul Eggleton
On Tuesday 21 August 2012 11:40:55 Paul Eggleton wrote:
> If the user selects not to pull the component repos, at least ensure
> that the correct branch is checked out before proceeding.
> 
> Signed-off-by: Paul Eggleton 
> ---
>  scripts/combo-layer |4 
>  1 file changed, 4 insertions(+)
> 
> diff --git a/scripts/combo-layer b/scripts/combo-layer
> index 448fe71..6134e81 100755
> --- a/scripts/combo-layer
> +++ b/scripts/combo-layer
> @@ -356,6 +356,10 @@ def action_update(conf, args):
>  branch = repo.get('branch', "master")
>  repo_patch_dir = os.path.join(os.getcwd(), patch_dir, name)
> 
> +if conf.nopull:
> +# Need to do this here as we didn't call action_pull that would
> do it otherwise +runcmd("git checkout %s" % branch, ldir)
> +
>  # Step 2: generate the patch list and store to patch dir
>  logger.info("Generating patches from %s..." % name)
>  if dest_dir != ".":

Actually as Richard points out we don't need to be doing the checkout here - 
we can just specify the branch in the one or two places where it isn't already 
specified. A different patch is on its way.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

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


Re: [OE-core] [discussion] perf: specify SLANG_INC dir for perf

2012-08-21 Thread Bruce Ashfield

On 12-08-21 01:08 AM, Liang Li wrote:

On 2012-08-20 22:48, Bruce Ashfield  wrote:

On 12-08-17 09:05 AM, Liang Li wrote:

On 2012-08-17 21:01, Liang Li   wrote:

On 2012-08-17 18:53, Richard Purdie   wrote:

On Fri, 2012-08-17 at 18:00 +0800, Liang Li wrote:

I am totally confused, you mentioned 'general kernel do_install', I
assume it's oe-core kernel.bbclass concept. Then you mentioned 'get
the fix upstream in the mainline kernel', how could that happen?

We are discussing about the solution to 'fix the compile warning to
error' stuff that triggered by the '-I/usr/include/slang', right?


Yes.


   We do not necessarily have to change recipe to fix it since the issue
is not introduced by the recipe, the hard coded '-I/usr/include/slang'
in the Makefile cause the issue, we can fix the root cause by kernel
patch(other than just comment the line out). I see your previous patch
to kernel, by comment out the '-I/usr/include/slang' line in the
Makefile, is the same behavior, but we won't have the change(comment
out -I.. in Makefile) upstream to mainline, right?


I am suggesting that firstly, someone send a patch to the mainline
kernel which changes -I/usr/include/slang to -I=/usr/include/slang in
that Makefile.

Secondly, I'm suggesting that we add a line to kernel_do_install() in
kernel.bbclass which does a sed on the Makefile as installed into
$kerneldir which changes -I/usr/include/slang to -I=/usr/include/slang.

We can then drop the patch I added to the linux-yocto kernels.

This is all that should be needed, it should fix all the issues people
have reported in a way that is acceptable to everyone.



Ah, I see what you mean now. But we have push acceptable kernel patch




One final (I hope) follow up on this.

Liang: were you going to put together (and test) the 'sed fix' for
kernel.bbclass ?



No problem, the patch for kernel.bbclass:

commit 60a0b06
Author: Liang Li
Date:   Tue Aug 21 11:06:01 2012 +0800

 kernel.bbclass: fix INC directory for SLANG

 The change is intend to fix the hardcoded '-I/usr/include/slang' in
 the Makefile to be able to aware of SYSROOT if its specified.

 A planned kernel patch almost did the same change, but the change here
 won't conflict with it so this change could work for all kernels.

 Signed-off-by: Liang Li

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 1afb9ab..282194d 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -190,6 +190,9 @@ kernel_do_install() {
for entry in $bin_files; do
rm -f $kerneldir/$entry
done
+
+   # Fix SLNAG_INC for slang.h


s/SLNAG_INC/SLANG_INC/


+   sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' 
$kerneldir/tools/perf/Makefile


It looks like your baseline for this patch is the denzil branch. We'd
want a version for master, which we could backport as required.


  }

  PACKAGE_PREPROCESS_FUNCS += "kernel_package_preprocess"

---

The patch for kernel tree:

commit 6b72896
Author: Liang Li
Date:   Wed Aug 1 14:31:24 2012 +0800

 perf: add SLANG_INC for slang.h

 Previously we hard code '-I/usr/include/slang' to CFLAGS to works with
 some hosts that has /usr/include/slang/slang.h other than
 /usr/include/slang.h like Fedora. This will cause compiling warnings
 in some cases.


I'd rephrase this slightly as:

CFLAGS was previously hard coded to contain "-I/usr/include/slang" to
work with hosts that have "/usr/include/slang/slang.h" as well as hosts
that have "/usr/include/slang.h". This path can cause compile warnings
in some cases:

 

.. and indicate that these warnings can actually cause build errors if
WERROR is enabled.



 We could downgrade the priority of the default hard coded path, and
 provide user a chance to specify correct location of slang.h then user
 could specify SLANG_INC to avoid compile warnings like the
 '/usr/include/slang' is not exists etc.


Another minor rephrase:

To fix this issue, we can use -idirafter to downgrade the priority of the
default hard coded path. We can also make the slang include directory
a variable, to allow the user to specify SLANG_INC and set their own
include location.



 Signed-off-by: Liang Li

diff --git a/tools/perf/Makefile b/tools/perf/Makefile
index b7a7a87..e403c36 100644
--- a/tools/perf/Makefile
+++ b/tools/perf/Makefile
@@ -496,8 +496,10 @@ else
msg := $(warning newt not found, disables TUI support. Please 
install newt-devel or libnewt-dev);
BASIC_CFLAGS += -DNO_NEWT_SUPPORT
else
-   # Fedora has /usr/include/slang/slang.h, but ubuntu 
/usr/include/slang.h
-   BASIC_CFLAGS += -I/usr/include/slang
+   # Some releases like Fedora has /usr/include/slang/slang.h 
other than /usr/include/slang.h
+   SLANG_INC ?= -idirafter =/usr/include/slang


One more question, have you confirmed that gcc is fine with this being
in sysroot 

Re: [OE-core] Toolchain rework, call for testing

2012-08-21 Thread Gary Thomas

On 2012-08-21 06:00, Phil Blundell wrote:

On Tue, 2012-08-21 at 13:55 +0200, Marcin Juszkiewicz wrote:

W dniu 17.08.2012 06:47, Khem Raj pisze:

Hi All

Recently glibc build has been simplified upstream. It has dropped the
dependency on libgcc_s and libgcc_eh for building glibc itself.
This means that we can simplify our toolchain bootstrap a bit by
dropping 1 of the 3 cross gcc build stages. We do not need
gcc-cross-intermediate anymore.


Please adapt also toolchain-layer in meta-oe for those stuck with gcc 4.6.


Why is it that gcc-4.7 doesn't work for those people?


I have troubles with older versions of the Linux kernel on ARM.  Anything
older than 3.2 either doesn't boot or is unstable.  Kernel only, user code
built with 4.7+ seems to be fine.  Since I need to use those older kernels
and I don't have the time/budget to figure out why they don't work with 4.7+,
I'm sticking with 4.6+ for now.

--

Gary Thomas |  Consulting for the
MLB Associates  |Embedded world


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


Re: [OE-core] [yocto] Toolchain rework, call for testing

2012-08-21 Thread Martin Jansa
On Thu, Aug 16, 2012 at 09:47:37PM -0700, Khem Raj wrote:
> Hi All
> 
> Recently glibc build has been simplified upstream. It has dropped the
> dependency on libgcc_s and libgcc_eh for building glibc itself.
> This means that we can simplify our toolchain bootstrap a bit by
> dropping 1 of the 3 cross gcc build stages. We do not need
> gcc-cross-intermediate
> anymore. This should bring some build time reduction and simplify the
> bootstrap. I have a series of patches which I have tested
> by building core-image-minimal and meta-toolchain for all supported
> qemu architectures and also uclibc/eglibc both
> but it needs a lot more testing therefore I am calling out wider
> audience for help in testing it out.
> 
> The branch is
> 
> http://git.openembedded.org/openembedded-core-contrib/log/?h=kraj/toolchain-rework

eglibc fails to build in incremental build

| arm-oe-linux-gnueabi-gcc  -march=armv4t -marm -mthumb-interwork 
-mtune=arm920t --sysroot=/OE/shr-core/tmp-eglibc/sysroots/om-gta02-tcbootstrap  
 -nostdlib -nostartfiles -r -o 
/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/build-arm-oe-linux-gnueabi/libc_pic.os
 \
|  -Wl,-d -Wl,--whole-archive 
/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/build-arm-oe-linux-gnueabi/libc_pic.a
 -o 
/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/build-arm-oe-linux-gnueabi/libc_pic.os
| arm-oe-linux-gnueabi-gcc  -march=armv4t -marm -mthumb-interwork 
-mtune=arm920t --sysroot=/OE/shr-core/tmp-eglibc/sysroots/om-gta02-tcbootstrap  
 -nostdlib -nostartfiles -r -o 
/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/build-arm-oe-linux-gnueabi/elf/librtld.map.o
 '-Wl,-(' 
/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/build-arm-oe-linux-gnueabi/elf/dl-allobjs.os
 
/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/build-arm-oe-linux-gnueabi/libc_pic.a
 -lgcc '-Wl,-)' 
-Wl,-Map,/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/build-arm-oe-linux-gnueabi/elf/librtld.mapT
| 
/OE/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/libexec/armv4t-oe-linux-gnueabi.gcc-cross-initial/gcc/arm-oe-linux-gnueabi/4.7.2/ld:
 cannot find -lgcc
| collect2: error: ld returned 1 exit status
| make[2]: *** 
[/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/build-arm-oe-linux-gnueabi/elf/librtld.map]
 Error 1
| make[2]: *** Waiting for unfinished jobs
| make[2]: Leaving directory 
`/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/eglibc-2_16/libc/elf'
| make[1]: *** [elf/subdir_lib] Error 2
| make[1]: Leaving directory 
`/OE/shr-core/tmp-eglibc/work/armv4t-oe-linux-gnueabi/eglibc-2.16-r6+svnr19922/eglibc-2_16/libc'

Maybe it's because 
http://git.openembedded.org/openembedded-core/commit/?id=30617bde61a3b0a0944b49a0c9fb7159dacbb19f
is missing PR bump and gcc wasn't rebuilt before eglibc upgrade (OEBasic not 
OEBasicHash here).

Cheers,
-- 
Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com


signature.asc
Description: Digital signature
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Toolchain rework, call for testing

2012-08-21 Thread Martin Jansa
On Tue, Aug 21, 2012 at 01:00:03PM +0100, Phil Blundell wrote:
> On Tue, 2012-08-21 at 13:55 +0200, Marcin Juszkiewicz wrote:
> > W dniu 17.08.2012 06:47, Khem Raj pisze:
> > > Hi All
> > > 
> > > Recently glibc build has been simplified upstream. It has dropped the
> > > dependency on libgcc_s and libgcc_eh for building glibc itself.
> > > This means that we can simplify our toolchain bootstrap a bit by
> > > dropping 1 of the 3 cross gcc build stages. We do not need
> > > gcc-cross-intermediate anymore. 
> > 
> > Please adapt also toolchain-layer in meta-oe for those stuck with gcc 4.6.
> 
> Why is it that gcc-4.7 doesn't work for those people?

At least it should parse for people who have toolchain-layer enabled..

Parsing recipes...ERROR: ParseError at
/OE/shr-core/meta-openembedded/toolchain-layer/recipes-devtools/gcc/gcc-cross-intermediate_4.6.bb:2:
Could not include required file
recipes-devtools/gcc/gcc-cross-intermediate.inc

Cheers,
-- 
Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com


signature.asc
Description: Digital signature
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Toolchain rework, call for testing

2012-08-21 Thread Phil Blundell
On Tue, 2012-08-21 at 13:55 +0200, Marcin Juszkiewicz wrote:
> W dniu 17.08.2012 06:47, Khem Raj pisze:
> > Hi All
> > 
> > Recently glibc build has been simplified upstream. It has dropped the
> > dependency on libgcc_s and libgcc_eh for building glibc itself.
> > This means that we can simplify our toolchain bootstrap a bit by
> > dropping 1 of the 3 cross gcc build stages. We do not need
> > gcc-cross-intermediate anymore. 
> 
> Please adapt also toolchain-layer in meta-oe for those stuck with gcc 4.6.

Why is it that gcc-4.7 doesn't work for those people?

p.



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


Re: [OE-core] Toolchain rework, call for testing

2012-08-21 Thread Marcin Juszkiewicz
W dniu 17.08.2012 06:47, Khem Raj pisze:
> Hi All
> 
> Recently glibc build has been simplified upstream. It has dropped the
> dependency on libgcc_s and libgcc_eh for building glibc itself.
> This means that we can simplify our toolchain bootstrap a bit by
> dropping 1 of the 3 cross gcc build stages. We do not need
> gcc-cross-intermediate anymore. 

Please adapt also toolchain-layer in meta-oe for those stuck with gcc 4.6.

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


[OE-core] [PATCH v3] grub-efi-native: Update to version 2.00

2012-08-21 Thread Radu Moisan
Added -Wno-unused-result because of a read() with no result, and causing
error due to -Werror
Updated grub recipe because of a patch name change
Removed unnecessary patches (merged upstream).

Signed-off-by: Radu Moisan 
---
 ...2man.patch => grub-2.00-disable-help2man.patch} |  117 +++-
 ...grub-2.00-fix-enable_execute_stack-check.patch} |8 +-
 ...ix.patch => grub-2.00-fpmath-sse-387-fix.patch} |0
 .../grub/files/grub-no-unused-result.patch |   18 +++
 ...-efi-native_1.99.bb => grub-efi-native_2.00.bb} |   18 ++-
 meta/recipes-bsp/grub/grub_2.00.bb |2 +-
 6 files changed, 47 insertions(+), 116 deletions(-)
 rename meta/recipes-bsp/grub/files/{grub-1.99-disable-help2man.patch => 
grub-2.00-disable-help2man.patch} (54%)
 rename 
meta/recipes-bsp/grub/files/{grub-1.99-fix-enable_execute_stack-check.patch => 
grub-2.00-fix-enable_execute_stack-check.patch} (92%)
 rename meta/recipes-bsp/grub/files/{grub-1.99-fpmath-sse-387-fix.patch => 
grub-2.00-fpmath-sse-387-fix.patch} (100%)
 create mode 100644 meta/recipes-bsp/grub/files/grub-no-unused-result.patch
 rename meta/recipes-bsp/grub/{grub-efi-native_1.99.bb => 
grub-efi-native_2.00.bb} (80%)

diff --git a/meta/recipes-bsp/grub/files/grub-1.99-disable-help2man.patch 
b/meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch
similarity index 54%
rename from meta/recipes-bsp/grub/files/grub-1.99-disable-help2man.patch
rename to meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch
index 3ffefdc..a2e42c2 100644
--- a/meta/recipes-bsp/grub/files/grub-1.99-disable-help2man.patch
+++ b/meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch
@@ -10,14 +10,14 @@ unnecessarily.
 
 Signed-off-by: Darren Hart 
 
-Index: grub-1.99/configure.ac
+Index: grub-2.00/configure.ac
 ===
 grub-1.99.orig/configure.ac
-+++ grub-1.99/configure.ac
-@@ -275,11 +275,12 @@ if test x$grub_cv_apple_cc = xyes ; then
-   HOST_CFLAGS="$HOST_CFLAGS -fnested-functions"
+--- grub-2.00.orig/configure.ac
 grub-2.00/configure.ac
+@@ -310,11 +310,12 @@ if test x$USE_NLS = xno; then
+   HOST_CFLAGS="$HOST_CFLAGS -fno-builtin-gettext"
  fi
-
+ 
 -if test "x$cross_compiling" = xyes; then
 -  AC_MSG_WARN([cannot generate manual pages while cross compiling])
 -else
@@ -32,11 +32,11 @@ Index: grub-1.99/configure.ac
  
  # Check for functions and headers.
  AC_CHECK_FUNCS(posix_memalign memalign asprintf vasprintf getextmntent)
-Index: grub-1.99/configure
+Index: grub-2.00/configure
 ===
 grub-1.99.orig/configure
-+++ grub-1.99/configure
-@@ -1172,7 +1172,6 @@ GL_COND_LIBTOOL_FALSE
+--- grub-2.00.orig/configure
 grub-2.00/configure
+@@ -1215,7 +1215,6 @@ GL_COND_LIBTOOL_FALSE
  GL_COND_LIBTOOL_TRUE
  BUILD_CC
  LIBUTIL
@@ -44,15 +44,7 @@ Index: grub-1.99/configure
  POSUB
  LTLIBINTL
  LIBINTL
-@@ -1204,6 +1203,7 @@ LEXLIB
- am__fastdepCC_FALSE
- am__fastdepCC_TRUE
- CCDEPMODE
-+am__nodep
- AMDEPBACKSLASH
- AMDEP_FALSE
- AMDEP_TRUE
-@@ -3011,7 +3011,6 @@ _ACEOF
+@@ -3069,7 +3068,6 @@ _ACEOF
  # Let the site file select an alternate cache file if it wants to.
  # Prefer an explicitly selected file to automatically selected ones.
  ac_site_file1=NONE
@@ -60,7 +52,7 @@ Index: grub-1.99/configure
  if test -n "$CONFIG_SITE"; then
# We do not want a PATH search for config.site.
case $CONFIG_SITE in #((
-@@ -3019,14 +3018,8 @@ if test -n "$CONFIG_SITE"; then
+@@ -3077,14 +3075,8 @@ if test -n "$CONFIG_SITE"; then
  */*) ac_site_file1=$CONFIG_SITE;;
  *)   ac_site_file1=./$CONFIG_SITE;;
esac
@@ -76,7 +68,7 @@ Index: grub-1.99/configure
  do
test "x$ac_site_file" = xNONE && continue
if test /dev/null != "$ac_site_file" && test -r "$ac_site_file"; then
-@@ -3313,8 +3306,9 @@ case $target_os in *\ *) target_os=`echo
+@@ -3371,8 +3363,9 @@ case $target_os in *\ *) target_os=`echo
  # The aliases save the names the user supplied, while $host etc.
  # will get canonicalized.
  test -n "$target_alias" &&
@@ -88,83 +80,8 @@ Index: grub-1.99/configure
program_prefix=${target_alias}-
  
  am__api_version='1.11'
-@@ -3786,11 +3780,11 @@ MAKEINFO=${MAKEINFO-"${am_missing_run}ma
- 
- # We need awk for the "check" target.  The system "awk" is bad on
- # some platforms.
--# Always define AMTAR for backward compatibility.
--
--AMTAR=${AMTAR-"${am_missing_run}tar"}
-+# Always define AMTAR for backward compatibility.  Yes, it's still used
-+# in the wild :-(  We should find a proper way to deprecate it ...
-+AMTAR='$${TAR-tar}'
- 
--am__tar='${AMTAR} chof - "$$tardir"'; am__untar='${AMTAR} xf -'
-+am__tar='$${TAR-tar} chof - "$$tardir"' am__untar='$${TAR-tar} xf -'
- 
- 
- 
-@@ -4256,6 +4250,7 @@ fi
- if test "x$enable_dependency_tracking" != xno; then
-   am_depcomp="$ac_aux_dir/depcomp"
-   AMDEPBACKSLASH='\'
-+  am__nodep='_no'
- fi
-  if test "x$enable_dependency_tracking" != xno; then

[OE-core] [PATCH v2] grub-efi-native: Update to version 2.00

2012-08-21 Thread Radu Moisan
Added -Wno-unused-result because of a read() with no result, and causing
error due to -Werror
Updated grub recipe because of a patch name change
Removed unnecessary patches (merged upstream).

Signed-off-by: Radu Moisan 
---
 ...2man.patch => grub-2.00-disable-help2man.patch} |  117 +++-
 ...grub-2.00-fix-enable_execute_stack-check.patch} |8 +-
 ...ix.patch => grub-2.00-fpmath-sse-387-fix.patch} |0
 .../grub/files/grub-no-unused-result.patch |   13 +++
 ...-efi-native_1.99.bb => grub-efi-native_2.00.bb} |   18 ++-
 meta/recipes-bsp/grub/grub_2.00.bb |2 +-
 6 files changed, 42 insertions(+), 116 deletions(-)
 rename meta/recipes-bsp/grub/files/{grub-1.99-disable-help2man.patch => 
grub-2.00-disable-help2man.patch} (54%)
 rename 
meta/recipes-bsp/grub/files/{grub-1.99-fix-enable_execute_stack-check.patch => 
grub-2.00-fix-enable_execute_stack-check.patch} (92%)
 rename meta/recipes-bsp/grub/files/{grub-1.99-fpmath-sse-387-fix.patch => 
grub-2.00-fpmath-sse-387-fix.patch} (100%)
 create mode 100644 meta/recipes-bsp/grub/files/grub-no-unused-result.patch
 rename meta/recipes-bsp/grub/{grub-efi-native_1.99.bb => 
grub-efi-native_2.00.bb} (80%)

diff --git a/meta/recipes-bsp/grub/files/grub-1.99-disable-help2man.patch 
b/meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch
similarity index 54%
rename from meta/recipes-bsp/grub/files/grub-1.99-disable-help2man.patch
rename to meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch
index 3ffefdc..a2e42c2 100644
--- a/meta/recipes-bsp/grub/files/grub-1.99-disable-help2man.patch
+++ b/meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch
@@ -10,14 +10,14 @@ unnecessarily.
 
 Signed-off-by: Darren Hart 
 
-Index: grub-1.99/configure.ac
+Index: grub-2.00/configure.ac
 ===
 grub-1.99.orig/configure.ac
-+++ grub-1.99/configure.ac
-@@ -275,11 +275,12 @@ if test x$grub_cv_apple_cc = xyes ; then
-   HOST_CFLAGS="$HOST_CFLAGS -fnested-functions"
+--- grub-2.00.orig/configure.ac
 grub-2.00/configure.ac
+@@ -310,11 +310,12 @@ if test x$USE_NLS = xno; then
+   HOST_CFLAGS="$HOST_CFLAGS -fno-builtin-gettext"
  fi
-
+ 
 -if test "x$cross_compiling" = xyes; then
 -  AC_MSG_WARN([cannot generate manual pages while cross compiling])
 -else
@@ -32,11 +32,11 @@ Index: grub-1.99/configure.ac
  
  # Check for functions and headers.
  AC_CHECK_FUNCS(posix_memalign memalign asprintf vasprintf getextmntent)
-Index: grub-1.99/configure
+Index: grub-2.00/configure
 ===
 grub-1.99.orig/configure
-+++ grub-1.99/configure
-@@ -1172,7 +1172,6 @@ GL_COND_LIBTOOL_FALSE
+--- grub-2.00.orig/configure
 grub-2.00/configure
+@@ -1215,7 +1215,6 @@ GL_COND_LIBTOOL_FALSE
  GL_COND_LIBTOOL_TRUE
  BUILD_CC
  LIBUTIL
@@ -44,15 +44,7 @@ Index: grub-1.99/configure
  POSUB
  LTLIBINTL
  LIBINTL
-@@ -1204,6 +1203,7 @@ LEXLIB
- am__fastdepCC_FALSE
- am__fastdepCC_TRUE
- CCDEPMODE
-+am__nodep
- AMDEPBACKSLASH
- AMDEP_FALSE
- AMDEP_TRUE
-@@ -3011,7 +3011,6 @@ _ACEOF
+@@ -3069,7 +3068,6 @@ _ACEOF
  # Let the site file select an alternate cache file if it wants to.
  # Prefer an explicitly selected file to automatically selected ones.
  ac_site_file1=NONE
@@ -60,7 +52,7 @@ Index: grub-1.99/configure
  if test -n "$CONFIG_SITE"; then
# We do not want a PATH search for config.site.
case $CONFIG_SITE in #((
-@@ -3019,14 +3018,8 @@ if test -n "$CONFIG_SITE"; then
+@@ -3077,14 +3075,8 @@ if test -n "$CONFIG_SITE"; then
  */*) ac_site_file1=$CONFIG_SITE;;
  *)   ac_site_file1=./$CONFIG_SITE;;
esac
@@ -76,7 +68,7 @@ Index: grub-1.99/configure
  do
test "x$ac_site_file" = xNONE && continue
if test /dev/null != "$ac_site_file" && test -r "$ac_site_file"; then
-@@ -3313,8 +3306,9 @@ case $target_os in *\ *) target_os=`echo
+@@ -3371,8 +3363,9 @@ case $target_os in *\ *) target_os=`echo
  # The aliases save the names the user supplied, while $host etc.
  # will get canonicalized.
  test -n "$target_alias" &&
@@ -88,83 +80,8 @@ Index: grub-1.99/configure
program_prefix=${target_alias}-
  
  am__api_version='1.11'
-@@ -3786,11 +3780,11 @@ MAKEINFO=${MAKEINFO-"${am_missing_run}ma
- 
- # We need awk for the "check" target.  The system "awk" is bad on
- # some platforms.
--# Always define AMTAR for backward compatibility.
--
--AMTAR=${AMTAR-"${am_missing_run}tar"}
-+# Always define AMTAR for backward compatibility.  Yes, it's still used
-+# in the wild :-(  We should find a proper way to deprecate it ...
-+AMTAR='$${TAR-tar}'
- 
--am__tar='${AMTAR} chof - "$$tardir"'; am__untar='${AMTAR} xf -'
-+am__tar='$${TAR-tar} chof - "$$tardir"' am__untar='$${TAR-tar} xf -'
- 
- 
- 
-@@ -4256,6 +4250,7 @@ fi
- if test "x$enable_dependency_tracking" != xno; then
-   am_depcomp="$ac_aux_dir/depcomp"
-   AMDEPBACKSLASH='\'
-+  am__nodep='_no'
- fi
-  if test "x$enable_dependency_tracking" != xno; then

[OE-core] [PATCH v2] grub: Update to upstream stable 2.00

2012-08-21 Thread Radu Moisan
Removed unnecessary patches (merged upstream).

Signed-off-by: Radu Moisan 
---
 ...rub-1.99-gcc-4.7.0-strict-aliasing-errors.patch |  147 
 ...b-1.99-gcc-4.7.0-uninitialized-var-errors.patch |   41 -
 .../grub/files/grub-1.99-gcc-4.7.0.patch   |   34 
 .../files/grub-1.99_fix_for_automake_1.11.2.patch  |  179 
 .../grub/grub-1.99/grub-1.99-lazy_stat.patch   |   64 ---
 .../grub/{grub-1.99 => grub-2.00}/40_custom|0
 .../{grub-1.99 => grub-2.00}/grub-install.in.patch |0
 .../{grub-1.99 => grub-2.00}/remove-gets.patch |0
 .../grub/{grub_1.99.bb => grub_2.00.bb}|   16 +-
 9 files changed, 7 insertions(+), 474 deletions(-)
 delete mode 100644 
meta/recipes-bsp/grub/files/grub-1.99-gcc-4.7.0-strict-aliasing-errors.patch
 delete mode 100644 
meta/recipes-bsp/grub/files/grub-1.99-gcc-4.7.0-uninitialized-var-errors.patch
 delete mode 100644 meta/recipes-bsp/grub/files/grub-1.99-gcc-4.7.0.patch
 delete mode 100644 
meta/recipes-bsp/grub/files/grub-1.99_fix_for_automake_1.11.2.patch
 delete mode 100644 meta/recipes-bsp/grub/grub-1.99/grub-1.99-lazy_stat.patch
 rename meta/recipes-bsp/grub/{grub-1.99 => grub-2.00}/40_custom (100%)
 rename meta/recipes-bsp/grub/{grub-1.99 => grub-2.00}/grub-install.in.patch 
(100%)
 rename meta/recipes-bsp/grub/{grub-1.99 => grub-2.00}/remove-gets.patch (100%)
 rename meta/recipes-bsp/grub/{grub_1.99.bb => grub_2.00.bb} (69%)

diff --git 
a/meta/recipes-bsp/grub/files/grub-1.99-gcc-4.7.0-strict-aliasing-errors.patch 
b/meta/recipes-bsp/grub/files/grub-1.99-gcc-4.7.0-strict-aliasing-errors.patch
deleted file mode 100644
index 9d81050..000
--- 
a/meta/recipes-bsp/grub/files/grub-1.99-gcc-4.7.0-strict-aliasing-errors.patch
+++ /dev/null
@@ -1,147 +0,0 @@
-Upstream-Status: Pending
-
-This patch fixes these strick-aliasing errors with gcc 4.7
-
-
-| gcc -DHAVE_CONFIG_H -I. -I..  -Wall -W -I../include -I../include  
-DGRUB_MACHINE_EFI=1 -DGRUB_MACHINE=I386_EFI -nostdinc -isystem 
/usr/lib/gcc/x86_64-redhat-linux/4.7.0/include -DGRUB_FILE=\"fs/zfs/zfs.c\" -I. 
-I. -I.. -I.. -I../include -I../include
-isystem/home/nitin/builds/build0/tmp/sysroots/x86_64-linux/usr/include -O2 
-pipe -g -feliminate-unused-debug-types -Wall -W -Wshadow -Wpointer-arith 
-Wmissing-prototypes -Wundef -Wstrict-prototypes -g -falign-jumps=1 
-falign-loops=1 -falign-functions=1 -mno-mmx -mno-sse -mno-sse2 -mno-3dnow 
-fno-dwarf2-cfi-asm -m32 -fno-stack-protector -mno-stack-arg-probe -Werror 
-Wno-trampolines  -ffreestanding  
-isystem/home/nitin/builds/build0/tmp/sysroots/x86_64-linux/usr/include -O2 
-pipe -c -o fs/zfs/zfs_module-zfs.o `test -f 'fs/zfs/zfs.c' || echo 
'./'`fs/zfs/zfs.c
-| fs/zfs/zfs.c: In function 'get_filesystem_dnode':
-| fs/zfs/zfs.c:1449:7: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:1449:7: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c: In function 'make_mdn':
-| fs/zfs/zfs.c:1478:3: error: dereferencing type-punned pointer will break 
strict-alERROR: Function failed: do_compile (see 
/home/nitin/builds/build0/tmp/work/x86_64-linux/grub-efi-i586-native-1.99-r7/temp/log.do_compile.9293
 for further information)
-| iasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c: In function 'dnode_get_fullpath':
-| fs/zfs/zfs.c:1554:3: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:1554:3: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:1571:7: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:1571:7: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c: In function 'grub_zfs_open':
-| fs/zfs/zfs.c:2234:7: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:2234:7: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c: In function 'fill_fs_info':
-| fs/zfs/zfs.c:2362:7: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:2362:7: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:2395:3: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:2395:3: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c: In function 'iterate_zap':
-| fs/zfs/zfs.c:2420:5: error: dereferencing type-punned pointer will break 
strict-aliasing rules [-Werror=strict-aliasing]
-| fs/zfs/zfs.c:2420:5: error: dereferencing type-punned pointe

Re: [OE-core] [PATCH] [Multilib] Complete recipe enablement

2012-08-21 Thread Richard Purdie
On Tue, 2012-08-21 at 11:40 +0300, Marinescu, Bogdan A wrote:
> Resending.
> 
> On Thu, Aug 2, 2012 at 10:49 AM, Bogdan Marinescu
>  wrote:
> RP: The list of recipes in multilib.conf needs to go away and
> we need
> to just be able to extend all recipes with the multilib class.
> Tested by building and running lib32-core-image-sato-sdk.
> 
> [YOCTO #1563]
> 
> Signed-off-by: Bogdan Marinescu 
> ---
>  meta/classes/multilib.bbclass |3 +
>  meta/conf/multilib.conf   |  745 

Already merged:

http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=c4a7517f8ae6e176a0b8e59b1176cf65bb45aa78

As a sidenote, please put a prefix to the subject line showing which
area the patch touches (missing a multilib: in this case) as per the
commit message guidelines.

Cheers,

Richard



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


Re: [OE-core] [PATCH] kmod-native_git.bb: fix builds for hosts with older libc

2012-08-21 Thread Richard Purdie
On Mon, 2012-08-20 at 13:49 -0500, Matthew McClintock wrote:
> kmod will fail to build with the following error because O_CLOEXEC is
> not defined:
> 
> | libkmod/libkmod-module.c: In function 'kmod_module_get_initstate':
> | libkmod/libkmod-module.c:1640: error: 'O_CLOEXEC' undeclared (first use in 
> this function)
> | libkmod/libkmod-module.c:1640: error: (Each undeclared identifier is 
> reported only once
> | libkmod/libkmod-module.c:1640: error: for each function it appears in.)
> | libkmod/libkmod-module.c: In function 'kmod_module_get_refcnt':
> | libkmod/libkmod-module.c:1754: error: 'O_CLOEXEC' undeclared (first use in 
> this function)
> | libkmod/libkmod-module.c: In function 'kmod_module_get_sections':
> | libkmod/libkmod-module.c:1913: error: 'O_CLOEXEC' undeclared (first use in 
> this function)
> | libkmod/libkmod-file.c: In function 'kmod_file_open':
> | libkmod/libkmod-file.c:282: error: 'O_CLOEXEC' undeclared (first use in 
> this function)
> | libkmod/libkmod-file.c:282: error: (Each undeclared identifier is reported 
> only once
> | libkmod/libkmod-file.c:282: error: for each function it appears in.)
> 
> Since we are only using kmod-native for depmod, and it's a non-threaded
> user of this libary being built this should be safe to override O_CLOEXEC.
> 
> Keep in mind this is ONLY effecting the native builds and not what is
> being shipped in the root file system.
> 
> Signed-off-by: Matthew McClintock 
> ---
>  meta/recipes-kernel/kmod/kmod-native_git.bb |8 +++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-kernel/kmod/kmod-native_git.bb 
> b/meta/recipes-kernel/kmod/kmod-native_git.bb
> index 96de8b8..3c5e3c3 100644
> --- a/meta/recipes-kernel/kmod/kmod-native_git.bb
> +++ b/meta/recipes-kernel/kmod/kmod-native_git.bb
> @@ -4,7 +4,7 @@
>  require kmod.inc
>  inherit native
>  
> -PR = "${INC_PR}.0"
> +PR = "${INC_PR}.1"
>  
>  do_install_append (){
>   for tool in depmod insmod lsmod modinfo modprobe rmmod
> @@ -12,3 +12,9 @@ do_install_append (){
>   ln -s kmod ${D}${bindir}/$tool
>   done
>  }
> +
> +do_compile_append (){
> + if ! grep O_CLOEXEC -r ${includedir_native}/bits/fcntl.h; then
> + export CFLAGS="$CFLAGS -D O_CLOEXEC=0"
> + fi
> +}


A compile append would execute after the compile. Isn't this after the
point you need the change?

Cheers,

Richard



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


Re: [OE-core] [PATCH] python-dbus: updated to 1.1.1

2012-08-21 Thread Richard Purdie
On Tue, 2012-08-21 at 11:40 +0300, Marinescu, Bogdan A wrote:
> Resending.
> 
> On Wed, Aug 1, 2012 at 6:16 PM, Bogdan Marinescu
>  wrote:
> Signed-off-by: Bogdan Marinescu 
> ---
>  .../{python-dbus_1.1.0.bb => python-dbus_1.1.1.bb} |4
> ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>  rename meta/recipes-devtools/python/{python-dbus_1.1.0.bb =>
> python-dbus_1.1.1.bb} (83%)
> 
> diff --git a/meta/recipes-devtools/python/python-dbus_1.1.0.bb
> b/meta/recipes-devtools/python/python-dbus_1.1.1.bb
> similarity index 83%
> rename from meta/recipes-devtools/python/python-dbus_1.1.0.bb
> rename to meta/recipes-devtools/python/python-dbus_1.1.1.bb
> index a10fda3..51d5bc2 100644
> --- a/meta/recipes-devtools/python/python-dbus_1.1.0.bb
> +++ b/meta/recipes-devtools/python/python-dbus_1.1.1.bb
> @@ -8,8 +8,8 @@ PR = "r0"
> 
>  SRC_URI =
> 
> "http://dbus.freedesktop.org/releases/dbus-python/dbus-python-${PV}.tar.gz";
> 
> -SRC_URI[md5sum] = "41a4595fd92eeebd632623af0bd7"
> -SRC_URI[sha256sum] =
> "2dc30b53cdb2e20f8d0265af3be878e9da68745cf9ccc72f75de869094c4fc60"
> +SRC_URI[md5sum] = "742c7432ad0f7c3f98291d58fa2e35dc"
> +SRC_URI[sha256sum] =
> "8917ca4fb8f4d693aee18d200cbad166b2c938dfb88c03bb4ab3d90a7c915e88"
>  S = "${WORKDIR}/dbus-python-${PV}"

This is in master:
http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=516ce2fc8d2a6a330e8662921929aaf947daecb1
 

?

Cheers,

Richard




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


[OE-core] [PATCH] scripts/combo-layer: checkout branch in update when pull is disabled

2012-08-21 Thread Paul Eggleton
If the user selects not to pull the component repos, at least ensure
that the correct branch is checked out before proceeding.

Signed-off-by: Paul Eggleton 
---
 scripts/combo-layer |4 
 1 file changed, 4 insertions(+)

diff --git a/scripts/combo-layer b/scripts/combo-layer
index 448fe71..6134e81 100755
--- a/scripts/combo-layer
+++ b/scripts/combo-layer
@@ -356,6 +356,10 @@ def action_update(conf, args):
 branch = repo.get('branch', "master")
 repo_patch_dir = os.path.join(os.getcwd(), patch_dir, name)
 
+if conf.nopull:
+# Need to do this here as we didn't call action_pull that would do 
it otherwise
+runcmd("git checkout %s" % branch, ldir)
+
 # Step 2: generate the patch list and store to patch dir
 logger.info("Generating patches from %s..." % name)
 if dest_dir != ".":
-- 
1.7.9.5


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


Re: [OE-core] [discussion] perf: specify SLANG_INC dir for perf

2012-08-21 Thread Richard Purdie
On Tue, 2012-08-21 at 10:51 +0200, Henning Heinold wrote:
> On Tue, Aug 21, 2012 at 01:08:25PM +0800, Liang Li wrote:
> > --- a/meta/classes/kernel.bbclass
> > +++ b/meta/classes/kernel.bbclass
> > @@ -190,6 +190,9 @@ kernel_do_install() {
> > for entry in $bin_files; do
> > rm -f $kerneldir/$entry
> > done
> > +
> > +   # Fix SLNAG_INC for slang.h
> > +   sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' 
> > $kerneldir/tools/perf/Makefile
> Hm,
> 
> is this "-I=" a new special feature of gcc?
> 
> -I/usr/include/slang
> -I=/usr/include/slang

It means the path is relative to any sysroot. It wasn't around for the
first implementations of sysroot but came shortly thereafter. As long as
its gcc 3.4 onwards it should be fine iirc.

Cheers,

Richard


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


Re: [OE-core] [PATCH 00/32] denzil pull request1

2012-08-21 Thread Richard Purdie
On Mon, 2012-08-13 at 14:14 -0700, Scott Garman wrote:
> Hello,
> 
> This is my first pull request for denzil since the 1.2.1 release was made.
> 
> It has been run through the autobuilder, which is having some issues with
> running sanity tests and buildhistory, so if the build succeeded for
> everything except for those steps, I note it as "Green with known AB
> issues" below.
> 
> Note that these tests were run with a poky-based branch.
> 
[...]
> 
> The following changes since commit 741146fa90f28f7ce8d82ee7f7e254872d519724:
> 
>   gconf.bbclass: don't register schemas in the install stage (2012-06-29 
> 15:03:01 +0100)
> 
> are available in the git repository at:
> 
>   git://git.openembedded.org/openembedded-core-contrib 
> sgarman/denzil-next-pull1
>   
> http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgarman/denzil-next-pull1

Merged to denzil. Thanks for putting this together.

Cheers,

Richard



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


Re: [OE-core] [discussion] perf: specify SLANG_INC dir for perf

2012-08-21 Thread Liang Li
On 2012-08-21 16:51, Henning Heinold  wrote:
> On Tue, Aug 21, 2012 at 01:08:25PM +0800, Liang Li wrote:
> > --- a/meta/classes/kernel.bbclass
> > +++ b/meta/classes/kernel.bbclass
> > @@ -190,6 +190,9 @@ kernel_do_install() {
> > for entry in $bin_files; do
> > rm -f $kerneldir/$entry
> > done
> > +
> > +   # Fix SLNAG_INC for slang.h
> > +   sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' 
> > $kerneldir/tools/perf/Makefile
> Hm,
> 
> is this "-I=" a new special feature of gcc?
> 
> -I/usr/include/slang
> -I=/usr/include/slang
> 

Not something new/special feature of gcc, should be as old as
--sysroot, maybe more than 10 years elder I guess.

Cheers,
Liang Li

> Bye Henning

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


Re: [OE-core] [yocto] RFC: OE-Core task rework

2012-08-21 Thread Paul Eggleton
On Tuesday 21 August 2012 09:49:37 Paul Eggleton wrote:
> The main concern with LSB is that we have something called task-basic,
> which  seems to be intended for LSB but does not state as much, and I know
> at least one person has tried to use it and then been a little puzzled as
> to why rpm has been pulled in when ipk packaging has been selected. Just
> naming some of these tasks more appropriately would help avoid such
> situations. In the specific case of task-basic there may be a case for
> creating a similar but not LSB-focused task that pulls in real shell
> utilities in preference to the light versions provided by busybox.

Ugh, sorry. I meant task-core-basic here - not to be confused with task-basic 
in meta-oe [1].

Cheers,
Paul

[1] http://www.openembedded.org/wiki/Meta-oe_tasks

-- 

Paul Eggleton
Intel Open Source Technology Centre

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


Re: [OE-core] [discussion] perf: specify SLANG_INC dir for perf

2012-08-21 Thread Henning Heinold
On Tue, Aug 21, 2012 at 01:08:25PM +0800, Liang Li wrote:
> --- a/meta/classes/kernel.bbclass
> +++ b/meta/classes/kernel.bbclass
> @@ -190,6 +190,9 @@ kernel_do_install() {
>   for entry in $bin_files; do
>   rm -f $kerneldir/$entry
>   done
> +
> + # Fix SLNAG_INC for slang.h
> + sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' 
> $kerneldir/tools/perf/Makefile
Hm,

is this "-I=" a new special feature of gcc?

-I/usr/include/slang
-I=/usr/include/slang

Bye Henning

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


Re: [OE-core] [yocto] RFC: OE-Core task rework

2012-08-21 Thread Paul Eggleton
On Monday 20 August 2012 15:45:07 Mark Hatle wrote:
> > 1) Do we rename "task" to something a little more understandable to the
> > uninitiated, such as "package group"? The word "task" is already used in a
> > much more natural sense within bitbake as a unit of work. Historically I
> > believe we picked up this term from Debian but I'm not aware of
> > significant use by other mainstream distributions.
> 
> "group" or "packagegroup" or "package-group" is my suggestion for the
> existing 'task' recipes.  From what I've seen, it should be a rename
> opportunity -- we can even provide/rprovide the old names

Indeed, and I think we would for compatibility purposes.
 
> > 2) Look at the existing tasks and:
> >   * evaluate their usefulness
> >   * remove any that are obsolete
> >   * adjust existing contents if needed
> >   * look for useful groups of packages that might be added
> >   
> >   We need to pay particular attention to task-core-boot and task-base as
> >   these> 
> > are pulled in by default in any image that inherits from
> > core-image.bbclass - if these are not generally working for people that
> > are creating their own images, we need to change them such that they are.
> 
> During the pre-oe-core Yocto Project development, a design was generated to
> roughly group the packages into functional areas.  For the most part, this
> design (as well as legacy elements) still exist.
> 
> I think the boot, base and others need to be evaluated for usefulness... but
> my feeling is most of them are close to being correct.

I think so as well. We may be pulling in one or two packages unconditionally 
where they should be optional (e.g. modutils-initscripts?) but most of it is 
pretty sensible.
 
> >   *  Try to make each task have a reasonable name - there are some current
> >   uses of "core" and "base" that don't actually convey any meaning; LSB
> >   specific tasks should be named as such, etc.
> 
> Yup, there is a logical grouping for the lsb specific tasks, as for others. 
> The naming needs to be made clear as to why it's there, and what it
> represents. Same with the summary and descriptions -- they should list the
> functionality provided by this group of components.

The main concern with LSB is that we have something called task-basic, which 
seems to be intended for LSB but does not state as much, and I know at least 
one person has tried to use it and then been a little puzzled as to why rpm 
has been pulled in when ipk packaging has been selected. Just naming some of 
these tasks more appropriately would help avoid such situations. In the 
specific case of task-basic there may be a case for creating a similar but not 
LSB-focused task that pulls in real shell utilities in preference to the light 
versions provided by busybox.

> > 4) Consider the usefulness of the existing PACKAGE_GROUP_ structure which
> > converts some IMAGE_FEATURES into the addition of tasks to be installed
> > (see classes/core-image.bbclass). At the very least we should probably
> > rename this if we rename tasks to package groups, and there's a question
> > as to how IMAGE_FEATURES scales - should every task be represented in
> > there or only a select list? Would we be better off not trying to bring
> > any tasks in via IMAGE_FEATURES at all and mostly leave that to control
> > post-processing (e.g. package-management, debug-tweaks, etc.)?
> 
> I'd certainly prefer that images were limited to selecting software from
> task (group) recipes only, and not providing their own.  An image should be
> able to change the selection based on an "image feature" or similar
> configuration, but the underlying tasks/groups, recipes, etc should all be
> 'generic' to a given distribution configuration.

The first part seems reasonable to me; but I'm still short of an answer as to 
what to do with the existing IMAGE_FEATURES code as mentioned above. At the 
moment aside from the special features such as debug-tweaks, these are just a 
thin layer on top of task recipes which doesn't add very much at all other 
than extra maintenance.

> I think if you go through the images today, a lot of that stuff is either
> old, or could be simplified by creating appropriate tasks/groups.

OK, that's a good point. I have another task on my list to clean up our image 
recipes and that would be a good segue into that.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

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


Re: [OE-core] [PATCH][RFC] coreutils: Upgrade to upstream version 8.17

2012-08-21 Thread Radu Moisan



Your new patch needs a header, explaining why, and adding
Upstream-Status and Signed-off-by tags.


I meant this only as an RFC, to get feedback for my patch, from a
functional point of view, it is not intended to be the final patch.
I have a Signed-off-by tag, are you referring to something else?
Can you give me more details about Upstream-Status? I've google'd this
problem for a bit but could not find an existing patch for it, nor a bug
filled for this matter. Since I don't know yet if what I'm fixing it's
really a bug (or a misconfiguration on my side) I send this RFC to get a
hold of whether I'm on the good track here or not. If this patch turns
out to be valid, I'll get in touch with the guys from coreutils' mailing
lists, and try to push it upstream.


Radu,

For patches included in a given recipe we also have Signed-off-by as 
well as the Upstream-Status tag as defined by:


http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines

Please review this again.

Soul,
I understood your point, and in the *actual* patch I will update the 
Sign-off-by and a short description, and an Upstream Status. However, I 
reviewed again  Commit_Patch_Message_Guidelines and there is nothing in 
there mentioned about RFCs. This is an RFC, and it was intended to get a 
quick feedback from people more familiar/experienced with 
coreutils/autotools. I need feedback about the functional change of this 
patch. In my first reply I described as extensively as I could my 
problem and my question.


As far as the RFCs go, how long should I wait on an RFC? It's been more 
then a few days and nobody commented. Would it be appropriate to assume 
that if nobody had any comments, the patch is valid?


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


Re: [OE-core] [PATCH] python-dbus: updated to 1.1.1

2012-08-21 Thread Marinescu, Bogdan A
Resending.

On Wed, Aug 1, 2012 at 6:16 PM, Bogdan Marinescu <
bogdan.a.marine...@intel.com> wrote:

> Signed-off-by: Bogdan Marinescu 
> ---
>  .../{python-dbus_1.1.0.bb => python-dbus_1.1.1.bb} |4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>  rename meta/recipes-devtools/python/{python-dbus_1.1.0.bb =>
> python-dbus_1.1.1.bb} (83%)
>
> diff --git 
> a/meta/recipes-devtools/python/python-dbus_1.1.0.bbb/meta/recipes-devtools/python/
> python-dbus_1.1.1.bb
> similarity index 83%
> rename from meta/recipes-devtools/python/python-dbus_1.1.0.bb
> rename to meta/recipes-devtools/python/python-dbus_1.1.1.bb
> index a10fda3..51d5bc2 100644
> --- a/meta/recipes-devtools/python/python-dbus_1.1.0.bb
> +++ b/meta/recipes-devtools/python/python-dbus_1.1.1.bb
> @@ -8,8 +8,8 @@ PR = "r0"
>
>  SRC_URI = "
> http://dbus.freedesktop.org/releases/dbus-python/dbus-python-${PV}.tar.gz";
>
> -SRC_URI[md5sum] = "41a4595fd92eeebd632623af0bd7"
> -SRC_URI[sha256sum] =
> "2dc30b53cdb2e20f8d0265af3be878e9da68745cf9ccc72f75de869094c4fc60"
> +SRC_URI[md5sum] = "742c7432ad0f7c3f98291d58fa2e35dc"
> +SRC_URI[sha256sum] =
> "8917ca4fb8f4d693aee18d200cbad166b2c938dfb88c03bb4ab3d90a7c915e88"
>  S = "${WORKDIR}/dbus-python-${PV}"
>
>  inherit distutils-base autotools pkgconfig
> --
> 1.7.9.5
>
>
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] [PATCH v4] busybox: Upgrade to upstream 1.20.2

2012-08-21 Thread Radu Moisan
Updated defconfig file from generated .config
Disabled CONFIG_LSOF,CONFIG_CROSS_COMPILER_PREFIX, and CONFIG_EXTRA_CFLAGS
for backwards compatibility.
Added busybox-1.20.2-kernel_ver.patch from upstream.

Signed-off-by: Radu Moisan 
---
 .../poky-tiny/defconfig|0
 ...box_1.19.4.bbappend => busybox_1.20.2.bbappend} |0
 .../B921600.patch  |0
 .../busybox-1.20.2/busybox-1.20.2-kernel_ver.patch |   25 +
 .../busybox-appletlib-dependency.patch |0
 .../busybox-mkfs-minix-tests_bigendian.patch   |0
 .../busybox-udhcpc-no_deconfig.patch   |   37 +--
 .../{busybox-1.19.4 => busybox-1.20.2}/defconfig   |   39 +---
 .../fix-for-spurious-testsuite-failure.patch   |0
 .../get_header_tar.patch   |0
 .../run-parts.in.usr-bin.patch |0
 .../sys_resource.patch |0
 .../watch.in.usr-bin.patch |0
 .../wget_dl_dir_fix.patch  |0
 .../{busybox_1.19.4.bb => busybox_1.20.2.bb}   |9 +++--
 15 files changed, 72 insertions(+), 38 deletions(-)
 rename meta-yocto/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/poky-tiny/defconfig (100%)
 rename meta-yocto/recipes-core/busybox/{busybox_1.19.4.bbappend => 
busybox_1.20.2.bbappend} (100%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/B921600.patch (100%)
 create mode 100644 
meta/recipes-core/busybox/busybox-1.20.2/busybox-1.20.2-kernel_ver.patch
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/busybox-appletlib-dependency.patch (100%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/busybox-mkfs-minix-tests_bigendian.patch (100%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/busybox-udhcpc-no_deconfig.patch (78%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => busybox-1.20.2}/defconfig 
(98%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/fix-for-spurious-testsuite-failure.patch (100%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/get_header_tar.patch (100%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/run-parts.in.usr-bin.patch (100%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/sys_resource.patch (100%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/watch.in.usr-bin.patch (100%)
 rename meta/recipes-core/busybox/{busybox-1.19.4 => 
busybox-1.20.2}/wget_dl_dir_fix.patch (100%)
 rename meta/recipes-core/busybox/{busybox_1.19.4.bb => busybox_1.20.2.bb} (79%)

diff --git a/meta-yocto/recipes-core/busybox/busybox-1.19.4/poky-tiny/defconfig 
b/meta-yocto/recipes-core/busybox/busybox-1.20.2/poky-tiny/defconfig
similarity index 100%
rename from meta-yocto/recipes-core/busybox/busybox-1.19.4/poky-tiny/defconfig
rename to meta-yocto/recipes-core/busybox/busybox-1.20.2/poky-tiny/defconfig
diff --git a/meta-yocto/recipes-core/busybox/busybox_1.19.4.bbappend 
b/meta-yocto/recipes-core/busybox/busybox_1.20.2.bbappend
similarity index 100%
rename from meta-yocto/recipes-core/busybox/busybox_1.19.4.bbappend
rename to meta-yocto/recipes-core/busybox/busybox_1.20.2.bbappend
diff --git a/meta/recipes-core/busybox/busybox-1.19.4/B921600.patch 
b/meta/recipes-core/busybox/busybox-1.20.2/B921600.patch
similarity index 100%
rename from meta/recipes-core/busybox/busybox-1.19.4/B921600.patch
rename to meta/recipes-core/busybox/busybox-1.20.2/B921600.patch
diff --git 
a/meta/recipes-core/busybox/busybox-1.20.2/busybox-1.20.2-kernel_ver.patch 
b/meta/recipes-core/busybox/busybox-1.20.2/busybox-1.20.2-kernel_ver.patch
new file mode 100644
index 000..456fb08
--- /dev/null
+++ b/meta/recipes-core/busybox/busybox-1.20.2/busybox-1.20.2-kernel_ver.patch
@@ -0,0 +1,25 @@
+--- busybox-1.20.2/libbb/kernel_version.c
 busybox-1.20.2-kernel_ver/libbb/kernel_version.c
+@@ -20,18 +20,15 @@
+ int FAST_FUNC get_linux_version_code(void)
+ {
+   struct utsname name;
+-  char *s;
++  char *s, *t;
+   int i, r;
+ 
+-  if (uname(&name) == -1) {
+-  bb_perror_msg("can't get system information");
+-  return 0;
+-  }
+-
++  uname(&name); /* never fails */
+   s = name.release;
+   r = 0;
+   for (i = 0; i < 3; i++) {
+-  r = r * 256 + atoi(strtok(s, "."));
++  t = strtok(s, ".");
++  r = r * 256 + (t ? atoi(t) : 0);
+   s = NULL;
+   }
+   return r;
diff --git 
a/meta/recipes-core/busybox/busybox-1.19.4/busybox-appletlib-dependency.patch 
b/meta/recipes-core/busybox/busybox-1.20.2/busybox-appletlib-dependency.patch
similarity index 100%
rename from 
meta/recipes-core/busybox/busybox-1.19.4/busybox-appletlib-dependency.patch
rename to 
meta/recipes-core/busybox/busybox-1.20.2/busybox-appletl

[OE-core] [PATCH 1/1] coreutils: fix to build with acl support

2012-08-21 Thread jackie.huang
From: Jackie Huang 

Fix the issue that coreutils was built without acl support:

configure: WARNING: libacl development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without ACL support.

which will cause ls and cp commands don't have the ACL related behaviors:
ls -l: no plus sign(+) after the permission string for files that have ACL
cp -p: can not preserves the ACLs.

[YOCTO #2959]

Signed-off-by: Jackie Huang 
---
 .../coreutils-6.9/coreutils-build-with-acl.patch   |   28 
 .../coreutils-8.14/coreutils-build-with-acl.patch  |   28 
 meta/recipes-core/coreutils/coreutils_6.9.bb   |1 +
 meta/recipes-core/coreutils/coreutils_8.14.bb  |1 +
 4 files changed, 58 insertions(+), 0 deletions(-)
 create mode 100644 
meta/recipes-core/coreutils/coreutils-6.9/coreutils-build-with-acl.patch
 create mode 100644 
meta/recipes-core/coreutils/coreutils-8.14/coreutils-build-with-acl.patch

diff --git 
a/meta/recipes-core/coreutils/coreutils-6.9/coreutils-build-with-acl.patch 
b/meta/recipes-core/coreutils/coreutils-6.9/coreutils-build-with-acl.patch
new file mode 100644
index 000..8273d78
--- /dev/null
+++ b/meta/recipes-core/coreutils/coreutils-6.9/coreutils-build-with-acl.patch
@@ -0,0 +1,28 @@
+Upstream-Status: Inappropriate [embedded specific]
+
+Fix the following issue so that coreutils can build with ACL:
+
+configure: WARNING: libacl development library was not found or not usable.
+configure: WARNING: GNU coreutils will be built without ACL support.
+
+Signed-off-by: Jackie Huang 
+---
+ m4/acl.m4 |2 +-
+ 1 files changed, 1 insertions(+), 1 deletions(-)
+
+diff --git a/m4/acl.m4 b/m4/acl.m4
+index d6a448a..a9d4836 100644
+--- a/m4/acl.m4
 b/m4/acl.m4
+@@ -159,7 +159,7 @@ AC_DEFUN([gl_ACL_GET_FILE],
+   ]])],
+[gl_cv_func_working_acl_get_file=yes],
+[gl_cv_func_working_acl_get_file=no],
+-   [gl_cv_func_working_acl_get_file=cross-compiling])])
++   [gl_cv_func_working_acl_get_file=yes])])
+ 
+   AS_IF([test $gl_cv_func_working_acl_get_file = yes], [$1], [$2])
+ ])
+-- 
+1.7.7
+
diff --git 
a/meta/recipes-core/coreutils/coreutils-8.14/coreutils-build-with-acl.patch 
b/meta/recipes-core/coreutils/coreutils-8.14/coreutils-build-with-acl.patch
new file mode 100644
index 000..8273d78
--- /dev/null
+++ b/meta/recipes-core/coreutils/coreutils-8.14/coreutils-build-with-acl.patch
@@ -0,0 +1,28 @@
+Upstream-Status: Inappropriate [embedded specific]
+
+Fix the following issue so that coreutils can build with ACL:
+
+configure: WARNING: libacl development library was not found or not usable.
+configure: WARNING: GNU coreutils will be built without ACL support.
+
+Signed-off-by: Jackie Huang 
+---
+ m4/acl.m4 |2 +-
+ 1 files changed, 1 insertions(+), 1 deletions(-)
+
+diff --git a/m4/acl.m4 b/m4/acl.m4
+index d6a448a..a9d4836 100644
+--- a/m4/acl.m4
 b/m4/acl.m4
+@@ -159,7 +159,7 @@ AC_DEFUN([gl_ACL_GET_FILE],
+   ]])],
+[gl_cv_func_working_acl_get_file=yes],
+[gl_cv_func_working_acl_get_file=no],
+-   [gl_cv_func_working_acl_get_file=cross-compiling])])
++   [gl_cv_func_working_acl_get_file=yes])])
+ 
+   AS_IF([test $gl_cv_func_working_acl_get_file = yes], [$1], [$2])
+ ])
+-- 
+1.7.7
+
diff --git a/meta/recipes-core/coreutils/coreutils_6.9.bb 
b/meta/recipes-core/coreutils/coreutils_6.9.bb
index 5bfab16..20d8120 100644
--- a/meta/recipes-core/coreutils/coreutils_6.9.bb
+++ b/meta/recipes-core/coreutils/coreutils_6.9.bb
@@ -24,6 +24,7 @@ SRC_URI_BASE = "${GNU_MIRROR}/coreutils/${BP}.tar.bz2 \
file://coreutils-fix-install.patch \
file://man-touch.patch \
file://coreutils_fix_for_automake-1.12.patch \
+   file://coreutils-build-with-acl.patch \
"
 
 SRC_URI = "${SRC_URI_BASE} file://fix_for_manpage_building.patch"
diff --git a/meta/recipes-core/coreutils/coreutils_8.14.bb 
b/meta/recipes-core/coreutils/coreutils_8.14.bb
index 9a714a9..178bd67 100644
--- a/meta/recipes-core/coreutils/coreutils_8.14.bb
+++ b/meta/recipes-core/coreutils/coreutils_8.14.bb
@@ -16,6 +16,7 @@ inherit autotools gettext
 SRC_URI = "${GNU_MIRROR}/coreutils/${BP}.tar.xz \
file://remove-usr-local-lib-from-m4.patch \
file://remove-gets.patch \
+   file://coreutils-build-with-acl.patch \
   "
 SRC_URI[md5sum] = "bcb135ce553493a45aba01b39eb3920a"
 SRC_URI[sha256sum] = 
"0d120817c19292edb19e92ae6b8eac9020e03d51e0af9cb116cf82b65d18b02d"
-- 
1.7.4


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


[OE-core] [PATCH 0/1] coreutils: fix to build with acl support

2012-08-21 Thread jackie.huang
From: Jackie Huang 

Fix the issue that coreutils was built without acl support:

configure: WARNING: libacl development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without ACL support.

which will cause ls and cp commands don't have the ACL related behaviors:
ls -l: no plus sign(+) after the permission string for files that have ACL
cp -p: can not preserves the ACLs.

[YOCTO #2959]

Signed-off-by: Jackie Huang 
---
The following changes since commit 04568d1f18250d531aad5d286157d2d559083520:

  qemu: Fix broken accidental path move (2012-08-20 17:25:21 +0100)

are available in the git repository at:
  git://git.pokylinux.org/poky-contrib jhuang0/bug2959_acl_0821
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=jhuang0/bug2959_acl_0821

Jackie Huang (1):
  coreutils: fix to build with acl support

 .../coreutils-6.9/coreutils-build-with-acl.patch   |   28 
 .../coreutils-8.14/coreutils-build-with-acl.patch  |   28 
 meta/recipes-core/coreutils/coreutils_6.9.bb   |1 +
 meta/recipes-core/coreutils/coreutils_8.14.bb  |1 +
 4 files changed, 58 insertions(+), 0 deletions(-)
 create mode 100644 
meta/recipes-core/coreutils/coreutils-6.9/coreutils-build-with-acl.patch
 create mode 100644 
meta/recipes-core/coreutils/coreutils-8.14/coreutils-build-with-acl.patch

-- 
1.7.4


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