[OE-core] [PATCH] insane.bbclass: Updated MicroBlaze machine definitions

2013-04-08 Thread Nathan Rossi
* Removed existing definition with machine 47787, this
  definition is outdated, a sanity error should occur if an ELF uses this
  value.
* Added new definition with machine 189. This value replaces the existing
  value since August 2009. See binutils thread for more information.
  (http://sourceware.org/ml/binutils/2009-08/msg00127.html)

Signed-off-by: Nathan Rossi nathan.ro...@xilinx.com
---
 meta/classes/insane.bbclass |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 2f10688..75db7a2 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -60,6 +60,8 @@ def package_qa_get_machine_dict():
 s390:   (22, 0,0,  False,
 32),
 sh4:(42, 0,0,  True, 
 32),
 sparc:  ( 2, 0,0,  False,
 32),
+microblaze:  (189,   0,0,  False,
 32),
+microblazeel:(189,   0,0,  True, 
 32),
   },
 linux-uclibc : { 
 arm :   (  40,97,0,  True,   
   32),
@@ -97,8 +99,6 @@ def package_qa_get_machine_dict():
   },
 linux-gnu :   {
 powerpc:(20, 0,0,  False,
 32),
-microblaze:   (47787,  0,0,  False,  
   32),
-microblazeel: (47787,  0,0,  True,   
   32),
 sh4:(42, 0,0,  True, 
 32),
   },
 linux-gnux32 :   {
-- 
1.7.5.4



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


Re: [OE-core] [PATCH] systemd: Set the default firmware path to enable firmware loading in udev

2013-04-08 Thread Koen Kooi

Op 7 apr. 2013, om 11:20 heeft Holger Hans Peter Freyther 
hol...@moiji-mobile.com het volgende geschreven:

 From: Holger Hans Peter Freyther ze...@selfish.org
 
 After some breakage in udev the kernel gained direct firmware loading.
 For older kernels (e.g. 3.2 in my case) udev still needs to load the
 firmware. Firmware loading is enabled once a default firmware path is
 set. Apply a compile fix from the upstream project.
 ---
 .../systemd/systemd/199-firmware.patch |   97 
 meta/recipes-core/systemd/systemd_199.bb   |3 +
 2 files changed, 100 insertions(+)
 create mode 100644 meta/recipes-core/systemd/systemd/199-firmware.patch
 
 diff --git a/meta/recipes-core/systemd/systemd/199-firmware.patch 
 b/meta/recipes-core/systemd/systemd/199-firmware.patch
 new file mode 100644
 index 000..1f4540f
 --- /dev/null
 +++ b/meta/recipes-core/systemd/systemd/199-firmware.patch
 @@ -0,0 +1,97 @@
 +http://cgit.freedesktop.org/systemd/systemd/patch/?id=d8d4bee76cf3b40ea923bc57d44aa0815ca9b5ff
 +
 +From d8d4bee76cf3b40ea923bc57d44aa0815ca9b5ff Mon Sep 17 00:00:00 2001
 +From: Kay Sievers k...@vrfy.org
 +Date: Thu, 28 Mar 2013 14:28:10 +
 +Subject: build-sys: fix HAVE/ENABLE_FIRMWARE

The above makes it clear that it's a backport, but I'd still like to see a 
Upstream-status: Backport in there :)

regards,

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


Re: [OE-core] [PATCH v2] ptest bugfix: Make all ptest files go into -ptest package

2013-04-08 Thread Björn Stenberg
Richard Purdie wrote:
 Why are we putting all the debug files into the -ptest package now? Does
 that make sense?

(Sorry, I missed this comment.)

My reasoning is that we want all ptest data to be contained in -ptest packages 
and not leak into other packages. 

If we keep debug data in the normal -dbg packages, you will always get ptest 
debug data installed in your image when defining distro-feature ptest, even 
if it doesn't include any -ptest packages. That feels wrong to me.

-- 
Björn

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


Re: [OE-core] gcc-cross-initial Failure

2013-04-08 Thread Jack Mitchell

On 05/04/13 18:03, Khem Raj wrote:

On Apr 5, 2013, at 9:36 AM, Jack Mitchell m...@communistcode.co.uk wrote:


I build for the beaglebone and I changed them in line with your default 
beaglebone build patch you posted a week or so ago. I think it moved it form 
soft to hard float possibly...

yes do a clean build if possible


Hi Khem,

Clean build results in the same failure.

Host GCC:

Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-unknown-linux-gnu/4.8.0/lto-wrapper
Target: x86_64-unknown-linux-gnu
Configured with: /build/src/gcc-4.8.0/configure --prefix=/usr 
--libdir=/usr/lib --libexecdir=/usr/lib --mandir=/usr/share/man 
--infodir=/usr/share/info --with-bugurl=https://bugs.archlinux.org/ 
--enable-languages=c,c++,ada,fortran,go,lto,objc,obj-c++ --enable-shared 
--enable-threads=posix --with-system-zlib --enable-__cxa_atexit 
--disable-libunwind-exceptions --enable-clocale=gnu 
--disable-libstdcxx-pch --enable-gnu-unique-object 
--enable-linker-build-id --enable-cloog-backend=isl 
--disable-cloog-version-check --enable-lto --enable-gold 
--enable-ld=default --enable-plugin --with-plugin-ld=ld.gold 
--with-linker-hash-style=gnu --disable-install-libiberty 
--disable-multilib --disable-libssp --disable-werror 
--enable-checking=release

Thread model: posix
gcc version 4.8.0 (GCC)


Tune:

DEFAULTTUNE_beaglebone = cortexa8hf-neon

Cheers,

--

  Jack Mitchell (j...@embed.me.uk)
  Embedded Systems Engineer
  http://www.embed.me.uk

--


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


[OE-core] [PATCH 0/2] rename libpng_1.2.50 to libpng12

2013-04-08 Thread Kang Kai
Rename libpng_1.2.50 to libpng12 that multi-versions libpng could coexist.

The following changes since commit 6d4d42d63db4c3fcffd831ce359599f3ee1d710e:

  qemuimage-tests/sanity/boot: Increase timeout (2013-04-06 17:22:21 +0100)

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

Kang Kai (2):
  libpng12: rename libpng_1.2.50 to libpng12
  libpng12: remove prefer version and add it to lsb packagegroup

 meta/conf/distro/include/default-versions.inc  |3 ---
 .../packagegroups/packagegroup-core-lsb.bb |1 +
 .../{libpng_1.2.50.bb = libpng12_1.2.50.bb}   |   18 +++---
 3 files changed, 16 insertions(+), 6 deletions(-)
 rename meta/recipes-lsb4/libpng/{libpng_1.2.50.bb = libpng12_1.2.50.bb} (62%)

-- 
1.7.5.4


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


[OE-core] [PATCH 1/2] libpng12: rename libpng_1.2.50 to libpng12

2013-04-08 Thread Kang Kai
As Mark's suggestion, rename libpng_1.2.50 to libpng12 that
multi-versions libpng could coexist. And drop files that conflict with
higher version.

We want to make sure we have both the old and new versions to meet LSB
compliance (for people who have that enabled) as well as the new version
for newer applications.

CC: Mark Hatle mark.ha...@windriver.com

Signed-off-by: Kang Kai kai.k...@windriver.com
---
 .../{libpng_1.2.50.bb = libpng12_1.2.50.bb}   |   18 +++---
 1 files changed, 15 insertions(+), 3 deletions(-)
 rename meta/recipes-lsb4/libpng/{libpng_1.2.50.bb = libpng12_1.2.50.bb} (62%)

diff --git a/meta/recipes-lsb4/libpng/libpng_1.2.50.bb 
b/meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
similarity index 62%
rename from meta/recipes-lsb4/libpng/libpng_1.2.50.bb
rename to meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
index 8fdc41b..43ff75a 100644
--- a/meta/recipes-lsb4/libpng/libpng_1.2.50.bb
+++ b/meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
@@ -8,14 +8,26 @@ LIC_FILES_CHKSUM = 
file://LICENSE;md5=c3d807a85c09ebdff087f18b4969ff96 \
 DEPENDS = zlib
 PR = r0
 
+PN = libpng12
+S = ${WORKDIR}/libpng-${PV}
+
 SRC_URI = 
${SOURCEFORGE_MIRROR}/project/libpng/libpng12/${PV}/libpng-${PV}.tar.xz
 
 SRC_URI[md5sum] = a3e00fccbfe356174ab515b5c00641c7
 SRC_URI[sha256sum] = 
4724f81f8c92ac7f360ad1fbf173396ea7c535923424db9fbaff07bfd9d8e8e7
 
+BINCONFIG_GLOB = ${PN}-config
+
 inherit autotools binconfig pkgconfig
 
-PACKAGES =+ ${PN}12
+do_install_append() {
+   unlink ${D}/${includedir}/png.h
+   unlink ${D}/${includedir}/pngconf.h
+
+   unlink ${D}/${libdir}/libpng.la
+   unlink ${D}/${libdir}/libpng.so
+   unlink ${D}/${libdir}/libpng.a
+   unlink ${D}/${libdir}/pkgconfig/libpng.pc
 
-FILES_${PN}12 = ${libdir}/libpng12${SOLIBS}
-RPROVIDES_${PN}-dev += ${PN}12-dev
+   unlink ${D}/${bindir}/libpng-config
+}
-- 
1.7.5.4


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


[OE-core] [PATCH 2/2] libpng12: remove prefer version and add it to lsb packagegroup

2013-04-08 Thread Kang Kai
Because rename libpng_1.2.50 to libpng, remove the perfer verion from
default-versions.inc and add libpng12 to lsb packagegroup.

Signed-off-by: Kang Kai kai.k...@windriver.com
---
 meta/conf/distro/include/default-versions.inc  |3 ---
 .../packagegroups/packagegroup-core-lsb.bb |1 +
 2 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/meta/conf/distro/include/default-versions.inc 
b/meta/conf/distro/include/default-versions.inc
index 0a5b2f4..53ec2e7 100644
--- a/meta/conf/distro/include/default-versions.inc
+++ b/meta/conf/distro/include/default-versions.inc
@@ -9,6 +9,3 @@ PREFERRED_VERSION_python-native ?= 2.7.3
 
 # Force the older version of liberation-fonts until we fix the fontforge issue
 PREFERRED_VERSION_liberation-fonts ?= 1.04
-
-# Set libpng default version for linuxstdbase
-PREFERRED_VERSION_libpng_linuxstdbase ?= 1.2.50
diff --git a/meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb 
b/meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb
index d692a26..cf04f0f 100644
--- a/meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb
+++ b/meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb
@@ -161,6 +161,7 @@ RDEPENDS_packagegroup-core-lsb-core = \
 ncurses \
 zlib \
 nspr \
+libpng12 \
 
 
 SUMMARY_packagegroup-core-lsb-perl = LSB Runtime Languages (Perl)
-- 
1.7.5.4


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


Re: [OE-core] RFC: gstreamer 1.0 recipes

2013-04-08 Thread Burton, Ross
Op 8 apr. 2013, om 01:38 heeft Carlos Rafael Giani
d...@pseudoterminal.org het volgende geschreven:
 to start working with GStreamer 1.0 on several embedded platforms, I decided 
 to
 adapt the existing GStreamer recipes from danny for 1.0. Out of this I 
 created a layer,
 which is hosted at https://github.com/dv1/meta-gstreamer1.0 .

 I write this RFC, because I figure that GStreamer 1.0 support is already on 
 TODO lists
 of several people, and will eventually end up in OE-Core, just like GStreamer 
 0.10 did.

Excellent!  I'd love to see this merged into oe-core as soon as we've
branched for 1.4.

 I built the packages for several platforms (cubox, beagleboard, imx6), and 
 they worked
 well. Since 1.0 has been designed to be able to coexist with 0.10 in the same 
 rootfs, I
 changed the naming convention for the packages: they all start with 
 gstreamer1.0- .

Perfect.

 * gstreamer: check_fix.patch (I could not reproduce the documented problem), 
 and gstregistrybin.c/.h (why are these present?)

The binary registry stuff is upstream now I think.

 * gst-plugins-base: configure.ac-fix-subparse-plugin.patch (not sure what 
 this is trying to fix, and this line does not exist in 1.0), and 
 gst-plugins-base-tremor.patch (again, works well without it)

Fine, is there are problems we'll soon find them again. :)

 I am also considering some kind of autodetection for when yasm and liborc 
 are available (for example, because meta-oe was added to bblayers.conf), and 
 then switches on Orc and enables yasm in gst-libav. Or better yet, if libav 
 is available as a recipe, it lets gst-libav use it instead of its internal 
 copy. These autodetections would improve performance significantly.

Build-time autodetection has to also be deterministic, i.e. it
examines the MACHINE and layers instead of poking at the sysroot.

So...

On 8 April 2013 08:57, Koen Kooi k...@dominion.thruhere.net wrote:
 I've been meaning to ask the same questions, since I need both orc and 
 external libav to make gstreamer  (0.10, but still) useable on my platforms. 
 I came up with the following ideas:

 1) Make every external dep a PACKAGECONFIG option in OE-core gstreamer
 2) Add bbappends with extra PACKAGECONFIG options for gstreamer in the layer 
 that has the external dep
 3) add bbappens in the DISTRO layer that enables extra external dependencies.

 Angstrom is currently using 3) and I absolutely hate it. 2) is scales a lot 
 better, but apparently is verboten judging from the patches sent that remove 
 the qt bbappends that do the same.
 Which leaves 1), which will cause problems for most users, since they will 
 spot the PACKAGECONFIG options but not bother to read the comments saying 
 they need to enable extra layers.

 I favour 1) since that has all the knowledge in a single place, looked after 
 by the OE gst maintainer (which we don't have yet, but still).

(1) definitely needs to happen in my opinion  I'm not a massive fan of
(2) in general as it leads to stuff changing simply by adding a layer
- you may pull in meta-oe for some other packages but suddenly the
GStreamer package is pulling in more dependencies.  I guess for things
like GStreamer where the dependencies are generally isolated into
separate packages this is less of an issue though.

Ross

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


Re: [OE-core] RFC: gstreamer 1.0 recipes

2013-04-08 Thread Paul Eggleton
On Monday 08 April 2013 12:47:51 Koen Kooi wrote:
 Op 8 apr. 2013, om 11:35 heeft Burton, Ross ross.bur...@intel.com het 
volgende geschreven:
  On 8 April 2013 08:57, Koen Kooi k...@dominion.thruhere.net wrote:
  I've been meaning to ask the same questions, since I need both orc and
  external libav to make gstreamer  (0.10, but still) useable on my
  platforms. I came up with the following ideas:
  
  1) Make every external dep a PACKAGECONFIG option in OE-core gstreamer
  2) Add bbappends with extra PACKAGECONFIG options for gstreamer in the
  layer that has the external dep 3) add bbappens in the DISTRO layer that
  enables extra external dependencies.
  
  Angstrom is currently using 3) and I absolutely hate it. 2) is scales a
  lot better, but apparently is verboten judging from the patches sent
  that remove the qt bbappends that do the same. Which leaves 1), which
  will cause problems for most users, since they will spot the
  PACKAGECONFIG options but not bother to read the comments saying they
  need to enable extra layers.
  
  I favour 1) since that has all the knowledge in a single place, looked
  after by the OE gst maintainer (which we don't have yet, but still). 
  (1) definitely needs to happen in my opinion  I'm not a massive fan of
  (2) in general as it leads to stuff changing simply by adding a layer
  - you may pull in meta-oe for some other packages but suddenly the
  GStreamer package is pulling in more dependencies.  I guess for things
  like GStreamer where the dependencies are generally isolated into
  separate packages this is less of an issue though.
 
 Same thing is true to Qt, the *sql plugins are all subpackages. I realize 2)
 is unclear on wether the extra PACKAGECONFIG default to on or off.
 
 Both Qt and gstreamer have nice automated packaging of their plugins, so
 bbappends are generally safe to use, but both of them also have configure
 options that enable/disable complete libraries (e.g. QtOpenGL) that might
 trigger missing symbols in other libraries in Qt/gstreamer (again,
 QtOpenGL).
 
 So were do we as OE(-core) community draw the line for bbappends in layers?

Let's be clear, we're talking about bbappends in meta-oe and other software 
layers.

 For Qt it seems to be at No bbappends allowed, but should we consider
 drawing it at safe plugins only? For generic layers like the ones in
 meta-oe I sure as hell don't want the unsafe libraries bbappends.

Apart from not having its configuration changed, given how huge Qt is I want it 
to *not* be rebuilt just as a consequence of adding meta-oe to my 
bblayers.conf. (Of course I can't have that yet because the bbappends have to 
at least have PRINC set in them still, but as soon as PV gets bumped...).

Honestly I think solution 1 is fine; in fact we already have that for some 
recipes. We would have had PACKAGECONFIG for these Qt SQL plugins except that 
as I've mentioned numbers of times, PACKAGECONFIG can't work for these three-
state options. One alternative would be to auto-add DEPENDS / CFLAGS based on 
QT_SQL_DRIVER_FLAGS.

Returning to the original topic, a case could be made for just bringing orc 
and libav into OE-Core. That's not going to work for every situation but we 
ought to be looking at these kind of things on a case-by-case basis.
 
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: gstreamer 1.0 recipes

2013-04-08 Thread Carlos Rafael Giani

On 2013-04-08 13:09, Paul Eggleton wrote:


Returning to the original topic, a case could be made for just bringing orc
and libav into OE-Core. That's not going to work for every situation but we
ought to be looking at these kind of things on a case-by-case basis.
  
Cheers,

Paul



You should add yasm to that list. libav isn't happy without it ;)

Carlos

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


Re: [OE-core] [PATCHv2 1/2][for-danny] linux-dtb: Add simple DTB symlinks for devicetree

2013-04-08 Thread Maupin, Chase
Ping on this set


 -Original Message-
 From: Maupin, Chase
 Sent: Thursday, April 04, 2013 8:26 AM
 To: openembedded-core@lists.openembedded.org
 Cc: Maupin, Chase
 Subject: [PATCHv2 1/2][for-danny] linux-dtb: Add simple DTB
 symlinks for devicetree
 
 * This is similar to the symlinks provided for the kernel image
   in the /boot directory of a file system.  The goal is to have
   simply named symlinks in /boot that mirror the device tree
   name in the kernel sources.  This is so that programs like
   U-Boot can easily find the default device tree binary in the
   /boot directory and use that when booting the kernel.
 * Use update-alternatives to handle proper creation and removal
   of the symlinks.
 * This patch has already been accepted into the master branch
   http://cgit.openembedded.org/openembedded-
 core/commit/?id=750a9554e1b85d9bd23d18e0630723c3c193c604
 
 Signed-off-by: Chase Maupin chase.mau...@ti.com
 ---
 * Updated in version 2
 * Changed the variable names to use variables that match the
   do_install and do_deploy more closely for consistency.
 i.e.
   using DTB_SYMLINK_NAME instead of DTB_NAME.
 * The above changes were based on input from:
 * Darren Hart dvh...@linux.intel.com
 * Bruce Ashfield bruce.ashfi...@windriver.com
 ---
  meta/recipes-kernel/linux/linux-dtb.inc |   20
 
  1 files changed, 20 insertions(+), 0 deletions(-)
 
 diff --git a/meta/recipes-kernel/linux/linux-dtb.inc
 b/meta/recipes-kernel/linux/linux-dtb.inc
 index d39f49d..36852b5 100644
 --- a/meta/recipes-kernel/linux/linux-dtb.inc
 +++ b/meta/recipes-kernel/linux/linux-dtb.inc
 @@ -45,3 +45,23 @@ do_deploy_append() {
  done
  fi
  }
 +
 +pkg_postinst_kernel-devicetree () {
 +cd /${KERNEL_IMAGEDEST}
 +for DTS_FILE in ${KERNEL_DEVICETREE}
 +do
 +DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F . '{print
 $1}'`
 +DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} |
 sed s/${MACHINE}/${DTS_BASE_NAME}/g`
 +update-alternatives --install
 /${KERNEL_IMAGEDEST}/${DTS_BASE_NAME}.dtb ${DTS_BASE_NAME}.dtb
 devicetree-${DTB_SYMLINK_NAME}.dtb ${KERNEL_PRIORITY} || true
 +done
 +}
 +
 +pkg_postrm_kernel-devicetree () {
 +cd /${KERNEL_IMAGEDEST}
 +for DTS_FILE in ${KERNEL_DEVICETREE}
 +do
 +DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F . '{print
 $1}'`
 +DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} |
 sed s/${MACHINE}/${DTS_BASE_NAME}/g`
 +update-alternatives --remove ${DTS_BASE_NAME}.dtb
 devicetree-${DTB_SYMLINK_NAME}.dtb ${KERNEL_PRIORITY} || true
 +done
 +}
 --
 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] tcf-agent: Use kill instead of killproc to stop agent

2013-04-08 Thread Bogdan Marinescu
From: Ioana Grigoropol ioanax.grigoro...@intel.com

[Yocto #3928]

Signed-off-by: Ioana Grigoropol ioanax.grigoro...@intel.com
Signed-off-by: Bogdan Marinescu bogdan.a.marine...@intel.com
---
 .../tcf-agent/tcf-agent/fix_tcf-agent.init.patch   |6 --
 meta/recipes-devtools/tcf-agent/tcf-agent_git.bb   |2 +-
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch 
b/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
index fefaf04..8ea5b43 100644
--- a/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
+++ b/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
@@ -13,7 +13,7 @@ Upstream-Status: Inappropriate [poky-specific script]
install -c -t $(INSTALLROOT)$(INCLUDE)/tcf/services -m 644 services/*.h
 --- /dev/null
 +++ b/tcf-agent.init
-@@ -0,0 +1,78 @@
+@@ -0,0 +1,80 @@
 +#!/bin/sh
 +### BEGIN INIT INFO
 +# Provides:  tcf-agent
@@ -50,14 +50,16 @@ Upstream-Status: Inappropriate [poky-specific script]
 +stop)
 +echo -n Stopping $DAEMON_NAME: 
 +count=0
++pid=$(/bin/pidof $DAEMON_PATH)
 +while [ -n `/bin/pidof $DAEMON_PATH` -a $count -lt 10 ] ; do
-+killproc $DAEMON_PATH  /dev/null
++kill $pid  /dev/null 21
 +sleep 1
 +RETVAL=$?
 +if [ $RETVAL != 0 -o -n `/bin/pidof $DAEMON_PATH` ] ; then
 +sleep 3
 +fi
 +count=`expr $count + 1`
++pid=$(/bin/pidof $DAEMON_PATH)
 +done
 +rm -f /var/lock/subsys/$DAEMON_NAME
 +if [ -n `/bin/pidof $DAEMON_PATH` ] ; then
diff --git a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb 
b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
index 4d43c62..ced2b41 100644
--- a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
+++ b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
file://edl-v10.html;md5=522a390a83dc186513f0500543ad3679
 
 SRCREV = 4ef94ecb927a8912c3d79ce137182247786cff8f
 PV = 0.4.0+git${SRCPV}
-PR = r0
+PR = r1
 
 SRC_URI = 
git://git.eclipse.org/gitroot/tcf/org.eclipse.tcf.agent.git;protocol=git \
file://fix_ranlib.patch \
-- 
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] tcf-agent: Use kill instead of killproc to stop agent

2013-04-08 Thread Richard Purdie
On Mon, 2013-04-08 at 14:58 +0300, Bogdan Marinescu wrote:
 From: Ioana Grigoropol ioanax.grigoro...@intel.com

Why?

 [Yocto #3928]

I could read this but I shouldn't have to...

Cheers,

Richard

 Signed-off-by: Ioana Grigoropol ioanax.grigoro...@intel.com
 Signed-off-by: Bogdan Marinescu bogdan.a.marine...@intel.com
 ---
  .../tcf-agent/tcf-agent/fix_tcf-agent.init.patch   |6 --
  meta/recipes-devtools/tcf-agent/tcf-agent_git.bb   |2 +-
  2 files changed, 5 insertions(+), 3 deletions(-)
 
 diff --git 
 a/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch 
 b/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
 index fefaf04..8ea5b43 100644
 --- a/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
 +++ b/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
 @@ -13,7 +13,7 @@ Upstream-Status: Inappropriate [poky-specific script]
   install -c -t $(INSTALLROOT)$(INCLUDE)/tcf/services -m 644 services/*.h
  --- /dev/null
  +++ b/tcf-agent.init
 -@@ -0,0 +1,78 @@
 +@@ -0,0 +1,80 @@
  +#!/bin/sh
  +### BEGIN INIT INFO
  +# Provides:  tcf-agent
 @@ -50,14 +50,16 @@ Upstream-Status: Inappropriate [poky-specific script]
  +stop)
  +echo -n Stopping $DAEMON_NAME: 
  +count=0
 ++pid=$(/bin/pidof $DAEMON_PATH)
  +while [ -n `/bin/pidof $DAEMON_PATH` -a $count -lt 10 ] ; do
 -+killproc $DAEMON_PATH  /dev/null
 ++kill $pid  /dev/null 21
  +sleep 1
  +RETVAL=$?
  +if [ $RETVAL != 0 -o -n `/bin/pidof $DAEMON_PATH` ] ; then
  +sleep 3
  +fi
  +count=`expr $count + 1`
 ++pid=$(/bin/pidof $DAEMON_PATH)
  +done
  +rm -f /var/lock/subsys/$DAEMON_NAME
  +if [ -n `/bin/pidof $DAEMON_PATH` ] ; then
 diff --git a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb 
 b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
 index 4d43c62..ced2b41 100644
 --- a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
 +++ b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
 @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
 file://edl-v10.html;md5=522a390a83dc186513f0500543ad3679
  
  SRCREV = 4ef94ecb927a8912c3d79ce137182247786cff8f
  PV = 0.4.0+git${SRCPV}
 -PR = r0
 +PR = r1
  
  SRC_URI = 
 git://git.eclipse.org/gitroot/tcf/org.eclipse.tcf.agent.git;protocol=git \
 file://fix_ranlib.patch \



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


[OE-core] [PATCH] systemd: Set the default firmware path to enable firmware loading in udev

2013-04-08 Thread Holger Hans Peter Freyther
From: Holger Hans Peter Freyther ze...@selfish.org

After some breakage in udev the kernel gained direct firmware loading.
For older kernels (e.g. 3.2 in my case) udev still needs to load the
firmware. Firmware loading is enabled once a default firmware path is
set. Apply a compile fix from the upstream project.
---
 .../systemd/systemd/199-firmware.patch |   98 
 meta/recipes-core/systemd/systemd_199.bb   |3 +
 2 files changed, 101 insertions(+)
 create mode 100644 meta/recipes-core/systemd/systemd/199-firmware.patch

diff --git a/meta/recipes-core/systemd/systemd/199-firmware.patch 
b/meta/recipes-core/systemd/systemd/199-firmware.patch
new file mode 100644
index 000..aaab59b
--- /dev/null
+++ b/meta/recipes-core/systemd/systemd/199-firmware.patch
@@ -0,0 +1,98 @@
+Upstream-Status: Backport
+http://cgit.freedesktop.org/systemd/systemd/patch/?id=d8d4bee76cf3b40ea923bc57d44aa0815ca9b5ff
+
+From d8d4bee76cf3b40ea923bc57d44aa0815ca9b5ff Mon Sep 17 00:00:00 2001
+From: Kay Sievers k...@vrfy.org
+Date: Thu, 28 Mar 2013 14:28:10 +
+Subject: build-sys: fix HAVE/ENABLE_FIRMWARE
+
+https://bugs.freedesktop.org/show_bug.cgi?id=62864
+---
+diff --git a/configure.ac b/configure.ac
+index 5b88bcf..e73cd5c 100644
+--- a/configure.ac
 b/configure.ac
+@@ -728,6 +728,7 @@ for i in $with_firmware_path; do
+ done
+ IFS=$OLD_IFS
+ AC_SUBST(FIRMWARE_PATH)
++AS_IF([test x${FIRMWARE_PATH} != x], [ AC_DEFINE(HAVE_FIRMWARE, 1, 
[Define if FIRMWARE is available]) ])
+ AM_CONDITIONAL(ENABLE_FIRMWARE, [test x${FIRMWARE_PATH} != x])
+ 
+ # 
--
+@@ -736,7 +737,6 @@ AC_ARG_ENABLE([gudev],
+[], [enable_gudev=yes])
+ AS_IF([test x$enable_gudev = xyes], [ PKG_CHECK_MODULES([GLIB], [glib-2.0 
= 2.22.0 gobject-2.0 = 2.22.0 gio-2.0]) ])
+ AM_CONDITIONAL([ENABLE_GUDEV], [test x$enable_gudev = xyes])
+-
+ AS_IF([test x$enable_gudev = xyes], [ AC_DEFINE(HAVE_GLIB, 1, [Define if 
glib is available]) ])
+ 
+ # 
--
+diff --git a/src/udev/udev-builtin.c b/src/udev/udev-builtin.c
+index 13922d3..c7d4319 100644
+--- a/src/udev/udev-builtin.c
 b/src/udev/udev-builtin.c
+@@ -34,7 +34,7 @@ static const struct udev_builtin *builtins[] = {
+ [UDEV_BUILTIN_BLKID] = udev_builtin_blkid,
+ #endif
+ [UDEV_BUILTIN_BTRFS] = udev_builtin_btrfs,
+-#ifdef ENABLE_FIRMWARE
++#ifdef HAVE_FIRMWARE
+ [UDEV_BUILTIN_FIRMWARE] = udev_builtin_firmware,
+ #endif
+ [UDEV_BUILTIN_HWDB] = udev_builtin_hwdb,
+diff --git a/src/udev/udev.h b/src/udev/udev.h
+index aa2edbe..906dfba 100644
+--- a/src/udev/udev.h
 b/src/udev/udev.h
+@@ -140,7 +140,7 @@ enum udev_builtin_cmd {
+ UDEV_BUILTIN_BLKID,
+ #endif
+ UDEV_BUILTIN_BTRFS,
+-#ifdef ENABLE_FIRMWARE
++#ifdef HAVE_FIRMWARE
+ UDEV_BUILTIN_FIRMWARE,
+ #endif
+ UDEV_BUILTIN_HWDB,
+@@ -169,7 +169,7 @@ struct udev_builtin {
+ extern const struct udev_builtin udev_builtin_blkid;
+ #endif
+ extern const struct udev_builtin udev_builtin_btrfs;
+-#ifdef ENABLE_FIRMWARE
++#ifdef HAVE_FIRMWARE
+ extern const struct udev_builtin udev_builtin_firmware;
+ #endif
+ extern const struct udev_builtin udev_builtin_hwdb;
+diff --git a/src/udev/udevd.c b/src/udev/udevd.c
+index b30bedf..2ad7388 100644
+--- a/src/udev/udevd.c
 b/src/udev/udevd.c
+@@ -98,7 +98,7 @@ struct event {
+ dev_t devnum;
+ int ifindex;
+ bool is_block;
+-#ifdef ENABLE_FIRMWARE
++#ifdef HAVE_FIRMWARE
+ bool nodelay;
+ #endif
+ };
+@@ -444,7 +444,7 @@ static int event_queue_insert(struct udev_device *dev)
+ event-devnum = udev_device_get_devnum(dev);
+ event-is_block = streq(block, udev_device_get_subsystem(dev));
+ event-ifindex = udev_device_get_ifindex(dev);
+-#ifdef ENABLE_FIRMWARE
++#ifdef HAVE_FIRMWARE
+ if (streq(udev_device_get_subsystem(dev), firmware))
+ event-nodelay = true;
+ #endif
+@@ -527,7 +527,7 @@ static bool is_devpath_busy(struct event *event)
+ return true;
+ }
+ 
+-#ifdef ENABLE_FIRMWARE
++#ifdef HAVE_FIRMWARE
+ /* allow to bypass the dependency tracking */
+ if (event-nodelay)
+ continue;
+--
+cgit v0.9.0.2-2-gbebe
diff --git a/meta/recipes-core/systemd/systemd_199.bb 
b/meta/recipes-core/systemd/systemd_199.bb
index e574548..2464b83 100644
--- a/meta/recipes-core/systemd/systemd_199.bb
+++ b/meta/recipes-core/systemd/systemd_199.bb
@@ -9,6 +9,7 @@ LIC_FILES_CHKSUM = 
file://LICENSE.GPL2;md5=751419260aa954499f7abaabaa882bbe \
 PROVIDES = udev
 
 PE = 1
+PR = r2
 
 DEPENDS = kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl 
readline dbus libcap libcgroup tcp-wrappers glib-2.0
 DEPENDS += ${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}
@@ -26,6 +27,7 @@ SRC_URI = 

[OE-core] [PATCH v2] tcf-agent: Use kill instead of killproc to stop agent

2013-04-08 Thread Bogdan Marinescu
From: Ioana Grigoropol ioanax.grigoro...@intel.com

When shutting down a core-image-lsb-sdk image, there is a lot of time spend 
stopping tcf-agent,
which slows down the whole process. The reason for this slowdown is the fact 
that it tries in a
loop to kill tcf-agent service by using killproc with the path of the 
executable and killproc
does not seem to available in lsb images. This patch fixes the issue by using 
kill instead of
killproc.

[Yocto #3928]

Signed-off-by: Ioana Grigoropol ioanax.grigoro...@intel.com
Signed-off-by: Bogdan Marinescu bogdan.a.marine...@intel.com
---
 .../tcf-agent/tcf-agent/fix_tcf-agent.init.patch   |6 --
 meta/recipes-devtools/tcf-agent/tcf-agent_git.bb   |2 +-
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch 
b/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
index fefaf04..8ea5b43 100644
--- a/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
+++ b/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
@@ -13,7 +13,7 @@ Upstream-Status: Inappropriate [poky-specific script]
install -c -t $(INSTALLROOT)$(INCLUDE)/tcf/services -m 644 services/*.h
 --- /dev/null
 +++ b/tcf-agent.init
-@@ -0,0 +1,78 @@
+@@ -0,0 +1,80 @@
 +#!/bin/sh
 +### BEGIN INIT INFO
 +# Provides:  tcf-agent
@@ -50,14 +50,16 @@ Upstream-Status: Inappropriate [poky-specific script]
 +stop)
 +echo -n Stopping $DAEMON_NAME: 
 +count=0
++pid=$(/bin/pidof $DAEMON_PATH)
 +while [ -n `/bin/pidof $DAEMON_PATH` -a $count -lt 10 ] ; do
-+killproc $DAEMON_PATH  /dev/null
++kill $pid  /dev/null 21
 +sleep 1
 +RETVAL=$?
 +if [ $RETVAL != 0 -o -n `/bin/pidof $DAEMON_PATH` ] ; then
 +sleep 3
 +fi
 +count=`expr $count + 1`
++pid=$(/bin/pidof $DAEMON_PATH)
 +done
 +rm -f /var/lock/subsys/$DAEMON_NAME
 +if [ -n `/bin/pidof $DAEMON_PATH` ] ; then
diff --git a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb 
b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
index 4d43c62..ced2b41 100644
--- a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
+++ b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
file://edl-v10.html;md5=522a390a83dc186513f0500543ad3679
 
 SRCREV = 4ef94ecb927a8912c3d79ce137182247786cff8f
 PV = 0.4.0+git${SRCPV}
-PR = r0
+PR = r1
 
 SRC_URI = 
git://git.eclipse.org/gitroot/tcf/org.eclipse.tcf.agent.git;protocol=git \
file://fix_ranlib.patch \
-- 
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] tcf-agent: Use kill instead of killproc to stop agent

2013-04-08 Thread Marinescu, Bogdan A
On Mon, Apr 8, 2013 at 3:16 PM, Richard Purdie 
richard.pur...@linuxfoundation.org wrote:

 On Mon, 2013-04-08 at 14:58 +0300, Bogdan Marinescu wrote:
  From: Ioana Grigoropol ioanax.grigoro...@intel.com

 Why?


Because she is the author of the patch, I've just added a signed-off-by
line.


  [Yocto #3928]

 I could read this but I shouldn't have to...


I've sent the second version of the patch to address this issue.

Thanks,
Bogdan



 Cheers,

 Richard

  Signed-off-by: Ioana Grigoropol ioanax.grigoro...@intel.com
  Signed-off-by: Bogdan Marinescu bogdan.a.marine...@intel.com
  ---
   .../tcf-agent/tcf-agent/fix_tcf-agent.init.patch   |6 --
   meta/recipes-devtools/tcf-agent/tcf-agent_git.bb   |2 +-
   2 files changed, 5 insertions(+), 3 deletions(-)
 
  diff --git
 a/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
 b/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
  index fefaf04..8ea5b43 100644
  --- a/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
  +++ b/meta/recipes-devtools/tcf-agent/tcf-agent/fix_tcf-agent.init.patch
  @@ -13,7 +13,7 @@ Upstream-Status: Inappropriate [poky-specific script]
install -c -t $(INSTALLROOT)$(INCLUDE)/tcf/services -m 644
 services/*.h
   --- /dev/null
   +++ b/tcf-agent.init
  -@@ -0,0 +1,78 @@
  +@@ -0,0 +1,80 @@
   +#!/bin/sh
   +### BEGIN INIT INFO
   +# Provides:  tcf-agent
  @@ -50,14 +50,16 @@ Upstream-Status: Inappropriate [poky-specific script]
   +stop)
   +echo -n Stopping $DAEMON_NAME: 
   +count=0
  ++pid=$(/bin/pidof $DAEMON_PATH)
   +while [ -n `/bin/pidof $DAEMON_PATH` -a $count -lt 10 ] ; do
  -+killproc $DAEMON_PATH  /dev/null
  ++kill $pid  /dev/null 21
   +sleep 1
   +RETVAL=$?
   +if [ $RETVAL != 0 -o -n `/bin/pidof $DAEMON_PATH` ] ;
 then
   +sleep 3
   +fi
   +count=`expr $count + 1`
  ++pid=$(/bin/pidof $DAEMON_PATH)
   +done
   +rm -f /var/lock/subsys/$DAEMON_NAME
   +if [ -n `/bin/pidof $DAEMON_PATH` ] ; then
  diff --git 
  a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bbb/meta/recipes-devtools/tcf-agent/
 tcf-agent_git.bb
  index 4d43c62..ced2b41 100644
  --- a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
  +++ b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb
  @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM =
 file://edl-v10.html;md5=522a390a83dc186513f0500543ad3679
 
   SRCREV = 4ef94ecb927a8912c3d79ce137182247786cff8f
   PV = 0.4.0+git${SRCPV}
  -PR = r0
  +PR = r1
 
   SRC_URI = git://
 git.eclipse.org/gitroot/tcf/org.eclipse.tcf.agent.git;protocol=git \
  file://fix_ranlib.patch \



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


[OE-core] [PATCH] bluez4: add readline dependency

2013-04-08 Thread Alex DAMIAN
From: Alexandru DAMIAN alexandru.dam...@intel.com

bluez4 uses readline to be build, but the dependency is not listed
This is listed in the configuration log.
So we add it.

Signed-off-by: Alexandru DAMIAN alexandru.dam...@intel.com
---
 meta/recipes-connectivity/bluez/bluez4.inc |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/bluez/bluez4.inc 
b/meta/recipes-connectivity/bluez/bluez4.inc
index bff24d3..42d82b0 100644
--- a/meta/recipes-connectivity/bluez/bluez4.inc
+++ b/meta/recipes-connectivity/bluez/bluez4.inc
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
 file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \
 
file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e \
 
file://sbc/sbc.c;beginline=1;endline=25;md5=1a40781ed30d50d8639323a184aeb191
-DEPENDS = udev libusb dbus-glib glib-2.0 libcheck
+DEPENDS = udev libusb dbus-glib glib-2.0 libcheck readline
 RDEPENDS_${PN}-dev = bluez-hcidump
 
 PACKAGECONFIG ??= \
-- 
1.7.10.4


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


Re: [OE-core] [PATCH] base.bbclass: Fix matching of SOC_FAMILY in COMPATIBLE_MACHINE

2013-04-08 Thread Otavio Salvador
On Sun, Apr 7, 2013 at 2:37 AM, Eric Bénard e...@eukrea.com wrote:
 Le Sat, 6 Apr 2013 17:58:30 -0300,
 Otavio Salvador ota...@ossystems.com.br a écrit :

 On Sat, Apr 6, 2013 at 3:02 PM, Eric Bénard e...@eukrea.com wrote:
  Hi Otavio,
 
  Le Sat,  6 Apr 2013 14:17:48 -0300,
  Otavio Salvador ota...@ossystems.com.br a écrit :
 
  When a SOC_FAMILY has more than one value, split by ':' as usual
  OVERRIDES, this were not being properly checked in COMPATIBLE_MACHINE
  matching as we need to iterate over each SoC family and check if it is
  compatible or not.
 
  Signed-off-by: Otavio Salvador ota...@ossystems.com.br
  ---
   meta/classes/base.bbclass | 12 +++-
   1 file changed, 7 insertions(+), 5 deletions(-)
 
  diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
  index abd6a52..6f24064 100644
  --- a/meta/classes/base.bbclass
  +++ b/meta/classes/base.bbclass
  @@ -515,11 +515,13 @@ python () {
   need_machine = d.getVar('COMPATIBLE_MACHINE', True)
   if need_machine:
   import re
  -this_machine = d.getVar('MACHINE', True)
  -if this_machine and not re.match(need_machine, this_machine):
  -this_soc_family = d.getVar('SOC_FAMILY', True)
  -if (this_soc_family and not re.match(need_machine, 
  this_soc_family)) or not this_soc_family:
  -raise bb.parse.SkipPackage(incompatible with 
  machine %s (not in COMPATIBLE_MACHINE) % this_machine)
  +compat_machines = [d.getVar('MACHINE', True)]
  +compat_machines.extend((d.getVar('SOC_FAMILY', True) or 
  ).split(:))
  +for this_machine in compat_machines:
  +if re.match(need_machine, this_machine):
  +break
  +else:
  +raise bb.parse.SkipPackage(incompatible with machine %s 
  (not in COMPATIBLE_MACHINE) % this_machine)
 
  aren't you breaking this log here vs what is was supposed to
  print before ?

 The 'else' is used when no 'break' is done inside of for loop.

 but will this_machine contain the value of the MACHINE variable ?

Yes, check:

...
+compat_machines = [d.getVar('MACHINE', True)]
+compat_machines.extend((d.getVar('SOC_FAMILY', True) or
).split(:))
...

--
Otavio Salvador O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854  http://projetos.ossystems.com.br

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


Re: [OE-core] [PATCH 1/2] alsa-tools: fix build when x11 and gtk+ not available

2013-04-08 Thread Trevor Woerner
With a rather recent HEAD

Build Configuration:
BB_VERSION= 1.17.1
BUILD_SYS = x86_64-linux
NATIVELSBSTRING   = openSUSE-project-12.3
TARGET_SYS= x86_64-poky-linux
MACHINE   = qemux86-64
DISTRO= poky
DISTRO_VERSION= 1.3+snapshot-20130407
TUNE_FEATURES = m64
TARGET_FPU= 
meta
meta-yocto
meta-yocto-bsp= master:813127247a1100b1abe179dfba25795560eac864

I am able to successfully perform a bitbake world on a number of
different targets:

- qemux86
- qemuarm
- qemumips
- qemux86-64

Unfortunately qemuppc fails when building alsa-tools. Is anyone else seeing
the same thing?

| make[1]: Entering directory
`/home/trevor/build/yocto/fullbuild/qemuppc/work/ppc603e-poky-linux/alsa-tools/1.0.26.1-r1/alsa-tools-1.0.26.1/hda-verb'
| make[1]: warning: jobserver unavailable: using -j1.  Add `+' to parent
make rule.
| powerpc-poky-linux-gcc  -m32 -mhard-float  -mcpu=603e
--sysroot=/home/trevor/build/yocto/fullbuild/qemuppc/sysroots/qemuppc
-DPACKAGE_NAME=\\ -DPACKAGE_TARNAME=\\ -DPACKAGE_VERSION=\\
-DPACKAGE_STRING=\\ -DPACKAGE_BUGREPORT=\\ -DPACKAGE_URL=\\
-DPACKAGE=\hda-verb\ -DVERSION=\0.4\ -DSTDC_HEADERS=1 -I. -O2 -Wall
-pipe -g -MT hda-verb.o -MD -MP -MF .deps/hda-verb.Tpo -c -o hda-verb.o
hda-verb.c
| hda-verb.c:17:20: fatal error: sys/io.h: No such file or directory
| compilation terminated.
| make[1]: *** [hda-verb.o] Error 1
| make[1]: Leaving directory
`/home/trevor/build/yocto/fullbuild/qemuppc/work/ppc603e-poky-linux/alsa-tools/1.0.26.1-r1/alsa-tools-1.0.26.1/hda-verb'
| make: *** [all] Error 1
| ERROR: oe_runmake failed
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] base.bbclass: Fix matching of SOC_FAMILY in COMPATIBLE_MACHINE

2013-04-08 Thread Eric Bénard
Hi Otavio,

Le Mon, 8 Apr 2013 10:31:17 -0300,
Otavio Salvador ota...@ossystems.com.br a écrit :

 On Sun, Apr 7, 2013 at 2:37 AM, Eric Bénard e...@eukrea.com wrote:
  Le Sat, 6 Apr 2013 17:58:30 -0300,
  Otavio Salvador ota...@ossystems.com.br a écrit :
 
  On Sat, Apr 6, 2013 at 3:02 PM, Eric Bénard e...@eukrea.com wrote:
   Hi Otavio,
  
   Le Sat,  6 Apr 2013 14:17:48 -0300,
   Otavio Salvador ota...@ossystems.com.br a écrit :
   +compat_machines = [d.getVar('MACHINE', True)]
   +compat_machines.extend((d.getVar('SOC_FAMILY', True) or 
   ).split(:))
   +for this_machine in compat_machines:
   +if re.match(need_machine, this_machine):
   +break
   +else:
   +raise bb.parse.SkipPackage(incompatible with machine 
   %s (not in COMPATIBLE_MACHINE) % this_machine)
  
   aren't you breaking this log here vs what is was supposed to
   print before ?
 
  The 'else' is used when no 'break' is done inside of for loop.
 
  but will this_machine contain the value of the MACHINE variable ?
 
 Yes, check:
 
 ...
 +compat_machines = [d.getVar('MACHINE', True)]
 +compat_machines.extend((d.getVar('SOC_FAMILY', True) or
 ).split(:))
 ...
 
I'm not very experienced in python so sorry is that's a stupid
remark but after these lines you are using this_machine to go
through the content of compat_machines so in the end, when it reach the
log message it will contain the last value in compat_machine (and thus
the last one in SOC_FAMILY instead of the machine name) ?

Eric

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


Re: [OE-core] [PATCH] base.bbclass: Fix matching of SOC_FAMILY in COMPATIBLE_MACHINE

2013-04-08 Thread Otavio Salvador
On Mon, Apr 8, 2013 at 10:37 AM, Eric Bénard e...@eukrea.com wrote:
 Hi Otavio,

 Le Mon, 8 Apr 2013 10:31:17 -0300,
 Otavio Salvador ota...@ossystems.com.br a écrit :

 On Sun, Apr 7, 2013 at 2:37 AM, Eric Bénard e...@eukrea.com wrote:
  Le Sat, 6 Apr 2013 17:58:30 -0300,
  Otavio Salvador ota...@ossystems.com.br a écrit :
 
  On Sat, Apr 6, 2013 at 3:02 PM, Eric Bénard e...@eukrea.com wrote:
   Hi Otavio,
  
   Le Sat,  6 Apr 2013 14:17:48 -0300,
   Otavio Salvador ota...@ossystems.com.br a écrit :
   +compat_machines = [d.getVar('MACHINE', True)]
   +compat_machines.extend((d.getVar('SOC_FAMILY', True) or 
   ).split(:))
   +for this_machine in compat_machines:
   +if re.match(need_machine, this_machine):
   +break
   +else:
   +raise bb.parse.SkipPackage(incompatible with machine 
   %s (not in COMPATIBLE_MACHINE) % this_machine)
  
   aren't you breaking this log here vs what is was supposed to
   print before ?
 
  The 'else' is used when no 'break' is done inside of for loop.
 
  but will this_machine contain the value of the MACHINE variable ?

 Yes, check:

 ...
 +compat_machines = [d.getVar('MACHINE', True)]
 +compat_machines.extend((d.getVar('SOC_FAMILY', True) or
 ).split(:))
 ...

 I'm not very experienced in python so sorry is that's a stupid
 remark but after these lines you are using this_machine to go
 through the content of compat_machines so in the end, when it reach the
 log message it will contain the last value in compat_machine (and thus
 the last one in SOC_FAMILY instead of the machine name) ?

Indeed this is indeed a bug. I will follow-up with a proper fix for it.

--
Otavio Salvador O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854  http://projetos.ossystems.com.br

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


[OE-core] [PATCH v2] base.bbclass: Fix matching of SOC_FAMILY in COMPATIBLE_MACHINE

2013-04-08 Thread Otavio Salvador
When a SOC_FAMILY has more than one value, split by ':' as usual
OVERRIDES, this were not being properly checked in COMPATIBLE_MACHINE
matching as we need to iterate over each SoC family and check if it is
compatible or not.

Signed-off-by: Otavio Salvador ota...@ossystems.com.br
---
Changes for v2:
- Properly handle machine so the error message clearly cite the right
  machine name (Eric)

 meta/classes/base.bbclass | 11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index abd6a52..080caa8 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -516,10 +516,13 @@ python () {
 if need_machine:
 import re
 this_machine = d.getVar('MACHINE', True)
-if this_machine and not re.match(need_machine, this_machine):
-this_soc_family = d.getVar('SOC_FAMILY', True)
-if (this_soc_family and not re.match(need_machine, 
this_soc_family)) or not this_soc_family:
-raise bb.parse.SkipPackage(incompatible with machine %s 
(not in COMPATIBLE_MACHINE) % this_machine)
+compat_machines = [this_machine]
+compat_machines.extend((d.getVar('SOC_FAMILY', True) or 
).split(:))
+for m in compat_machines:
+if re.match(need_machine, m):
+break
+else:
+raise bb.parse.SkipPackage(incompatible with machine %s (not 
in COMPATIBLE_MACHINE) % this_machine)
 
 
 bad_licenses = (d.getVar('INCOMPATIBLE_LICENSE', True) or ).split()
-- 
1.8.1


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


[OE-core] [PATCH] udev: Move udevd back to /sbin

2013-04-08 Thread Radu Moisan
Fixes [Yocto #4046]

Signed-off-by: Radu Moisan radu.moi...@intel.com
---
 meta/recipes-core/udev/udev.inc|3 ++-
 meta/recipes-core/udev/udev_182.bb |2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index e358d2d..c4d2ce4 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -40,7 +40,7 @@ EXTRA_OECONF = --disable-introspection \
 ac_cv_file__usr_share_hwdata_pci_ids=no \
 ac_cv_file__usr_share_misc_pci_ids=yes \
 --sbindir=${base_sbindir} \
---libexecdir=${base_libdir} \
+--libexecdir=${base_sbindir} \
 --with-rootlibdir=${base_libdir} \
 --with-rootprefix= \

@@ -61,6 +61,7 @@ RRECOMMENDS_${PN} += udev-utils
 FILES_${PN}-dbg += ${libexecdir}/.debug
 FILES_${PN}-dbg += ${base_libdir}/udev/.debug/
 FILES_${PN}-dbg += ${base_libdir}/udev/.debug/*
+FILES_${PN}-dbg += ${base_sbindir}/udev/.debug/*
 FILES_${PN}-dev = ${datadir}/pkgconfig/udev.pc
 FILES_libudev = ${base_libdir}/libudev.so.*
 FILES_libudev-dbg = ${base_libdir}/.debug/libudev.so.*
diff --git a/meta/recipes-core/udev/udev_182.bb 
b/meta/recipes-core/udev/udev_182.bb
index 42b4d08..d66292e 100644
--- a/meta/recipes-core/udev/udev_182.bb
+++ b/meta/recipes-core/udev/udev_182.bb
@@ -1,6 +1,6 @@
 include udev.inc
 
-PR = r6
+PR = r7
 
 # module-init-tools from kmod_git will provide libkmod runtime
 DEPENDS += module-init-tools
-- 
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] smart: disable CHANNELSDIR

2013-04-08 Thread Bogdan Marinescu
Make CHANNELSDIR in smart empty, since this causes host contamination issues
on some RPM-based hosts on which smart is already installed.

[YOCTO #3881]

Signed-off-by: Bogdan Marinescu bogdan.a.marine...@intel.com
---
 .../python/python-smartpm/smart-channelsdir.patch  | 24 ++
 .../python/python-smartpm_1.4.1.bb |  3 ++-
 2 files changed, 26 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-devtools/python/python-smartpm/smart-channelsdir.patch

diff --git 
a/meta/recipes-devtools/python/python-smartpm/smart-channelsdir.patch 
b/meta/recipes-devtools/python/python-smartpm/smart-channelsdir.patch
new file mode 100644
index 000..e621b33
--- /dev/null
+++ b/meta/recipes-devtools/python/python-smartpm/smart-channelsdir.patch
@@ -0,0 +1,24 @@
+Make CHANNELSDIR in smart empty, since this causes host contamination issues
+on some RPM-based hosts on which smart is already installed.
+
+[YOCTO #3881]
+
+Upstream-Status: Inappropriate [embedded specific]
+
+diff --git a/smart/plugins/channelsync.py b/smart/plugins/channelsync.py
+index 3ba95ff..646d696 100644
+--- a/smart/plugins/channelsync.py
 b/smart/plugins/channelsync.py
+@@ -23,7 +23,11 @@ from smart.channel import *
+ from smart import *
+ import os
+ 
+-CHANNELSDIR = /etc/smart/channels/
++# For now, we leave the definition of CHANNELSDIR empty. This prevents smart
++# from erroneously consider the  build host's channels while setting up its
++# channels [YOCTO #3881]. If this feature will be used in the future, 
CHANNELSDIR
++# should be set to a proper value.
++CHANNELSDIR = 
+ 
+ def syncChannels(channelsdir, force=None):
+ 
diff --git a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb 
b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb
index d92933f..001d9e4 100644
--- a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb
+++ b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb
@@ -11,7 +11,7 @@ LICENSE = GPLv2
 LIC_FILES_CHKSUM = file://LICENSE;md5=393a5ca445f6965873eca0259a17f833
 
 DEPENDS = python rpm
-PR = r8
+PR = r9
 SRCNAME = smart
 
 SRC_URI = \
@@ -27,6 +27,7 @@ SRC_URI = \
   file://smart-improve-error-reporting.patch \
   file://smart-multilib-fixes.patch \
   file://smart-yaml-error.patch \
+  file://smart-channelsdir.patch \
   
 
 SRC_URI[md5sum] = 573ef32ba177a6b3c4bf7ef04873fcb6
-- 
1.8.1.4


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


Re: [OE-core] [PATCH 1/2] libpng12: rename libpng_1.2.50 to libpng12

2013-04-08 Thread Mark Hatle

On 4/8/13 4:12 AM, Kang Kai wrote:

As Mark's suggestion, rename libpng_1.2.50 to libpng12 that
multi-versions libpng could coexist. And drop files that conflict with
higher version.

We want to make sure we have both the old and new versions to meet LSB
compliance (for people who have that enabled) as well as the new version
for newer applications.

CC: Mark Hatle mark.ha...@windriver.com

Signed-off-by: Kang Kai kai.k...@windriver.com
---
  .../{libpng_1.2.50.bb = libpng12_1.2.50.bb}   |   18 +++---
  1 files changed, 15 insertions(+), 3 deletions(-)
  rename meta/recipes-lsb4/libpng/{libpng_1.2.50.bb = libpng12_1.2.50.bb} (62%)

diff --git a/meta/recipes-lsb4/libpng/libpng_1.2.50.bb 
b/meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
similarity index 62%
rename from meta/recipes-lsb4/libpng/libpng_1.2.50.bb
rename to meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
index 8fdc41b..43ff75a 100644
--- a/meta/recipes-lsb4/libpng/libpng_1.2.50.bb
+++ b/meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
@@ -8,14 +8,26 @@ LIC_FILES_CHKSUM = 
file://LICENSE;md5=c3d807a85c09ebdff087f18b4969ff96 \
  DEPENDS = zlib
  PR = r0

+PN = libpng12
+S = ${WORKDIR}/libpng-${PV}
+
  SRC_URI = 
${SOURCEFORGE_MIRROR}/project/libpng/libpng12/${PV}/libpng-${PV}.tar.xz

  SRC_URI[md5sum] = a3e00fccbfe356174ab515b5c00641c7
  SRC_URI[sha256sum] = 
4724f81f8c92ac7f360ad1fbf173396ea7c535923424db9fbaff07bfd9d8e8e7

+BINCONFIG_GLOB = ${PN}-config
+
  inherit autotools binconfig pkgconfig

-PACKAGES =+ ${PN}12
+do_install_append() {
+   unlink ${D}/${includedir}/png.h
+   unlink ${D}/${includedir}/pngconf.h


You should move those two into a subdirectory called 
${D}/${includedir}/libpng12/

That way they will still be available for anyone who needs them.


+
+   unlink ${D}/${libdir}/libpng.la
+   unlink ${D}/${libdir}/libpng.so
+   unlink ${D}/${libdir}/libpng.a


The .la, .a seen above should be similarly renamed to be libpng12...  The .so 
should be dropped, as anyone who needs the alternative version would need to 
directly specify it.



+   unlink ${D}/${libdir}/pkgconfig/libpng.pc


Should be renamed to be libpng12.pc.

(Note both the .la and .pc files will likely need to be modified to reference 
the correct header and library paths.)



-FILES_${PN}12 = ${libdir}/libpng12${SOLIBS}
-RPROVIDES_${PN}-dev += ${PN}12-dev
+   unlink ${D}/${bindir}/libpng-config
+}


The above should be renamed to be libpng12-config.

(Again, make sure that the results of it match the renamed .a/.so and include 
paths.)


--Mark

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


[OE-core] [PATCH 0/2] Fix postinstall fallback and create gtk-update-icon-cache wrapper script

2013-04-08 Thread Laurentiu Palcu
Laurentiu Palcu (2):
  image.bbclass: fix postinstall intercepts fallback
  gtk-update-icon-cache-native: create wrapper script

 meta/classes/image.bbclass |2 +-
 .../gtk+/gtk-update-icon-cache-native_3.4.4.bb |4 
 2 files changed, 5 insertions(+), 1 deletion(-)

-- 
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 2/2] gtk-update-icon-cache-native: create wrapper script

2013-04-08 Thread Laurentiu Palcu
When using the sstate from another build machine, the path to the pixbuf
loader's cache points to a path on the remote machine. Hence, the update
of the icon cache fails on host.

Signed-off-by: Laurentiu Palcu laurentiu.pa...@intel.com
---
 .../gtk+/gtk-update-icon-cache-native_3.4.4.bb |4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-gnome/gtk+/gtk-update-icon-cache-native_3.4.4.bb 
b/meta/recipes-gnome/gtk+/gtk-update-icon-cache-native_3.4.4.bb
index 93c30a7..73b7644 100644
--- a/meta/recipes-gnome/gtk+/gtk-update-icon-cache-native_3.4.4.bb
+++ b/meta/recipes-gnome/gtk+/gtk-update-icon-cache-native_3.4.4.bb
@@ -40,4 +40,8 @@ do_compile() {
 do_install() {
install -d ${D}${bindir}
 install -m 0755 ${B}/gtk-update-icon-cache ${D}${bindir}
+
+   create_wrapper ${D}/${bindir}/gtk-update-icon-cache \
+   
GDK_PIXBUF_MODULE_FILE=${STAGING_LIBDIR_NATIVE}/gdk-pixbuf-2.0/2.10.0/loaders.cache
+
 }
-- 
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 1/1] gst-ffmpeg: fix --disable-yasm

2013-04-08 Thread tom . zanussi
From: Tom Zanussi tom.zanu...@linux.intel.com

The gst-ffmpeg build shows the following warning:

configure: WARNING: unrecognized options: --disable-yasm

which means that the following test in configure always fails and
--disable-yasm never gets passed to the embedded ffmpeg build:

'if test x$disable_yasm = xyes; then'
  embffmpeg_configure_args=$embffmpeg_configure_args --disable-yasm

commit 4d309730 ['gst-ffmpeg: configure-fix patch used wrong test']
actually fixed the obviously backwards syntax by reversing the test -
prior to that, --disable-yasm would always unconditionally be passed
into the embedded ffmpeg config.

This fixes things so that the variable actually exists and makes the
test meaningful.

Signed-off-by: Tom Zanussi tom.zanu...@linux.intel.com
---
 .../gstreamer/gst-ffmpeg-0.10.13/configure-fix.patch| 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/meta/recipes-multimedia/gstreamer/gst-ffmpeg-0.10.13/configure-fix.patch 
b/meta/recipes-multimedia/gstreamer/gst-ffmpeg-0.10.13/configure-fix.patch
index 2bb124b..9ef6f7c 100644
--- a/meta/recipes-multimedia/gstreamer/gst-ffmpeg-0.10.13/configure-fix.patch
+++ b/meta/recipes-multimedia/gstreamer/gst-ffmpeg-0.10.13/configure-fix.patch
@@ -7,11 +7,13 @@ Signed-off-by: Shane Wang shane.w...@intel.com
 diff -r f2f8f74c6e30 configure.ac
 --- a/configure.ac Thu Dec 22 23:56:09 2011 +0800
 +++ b/configure.ac Thu Dec 22 23:57:37 2011 +0800
-@@ -325,6 +325,10 @@
+@@ -325,6 +325,12 @@
  --enable-gpl
fi
  
-+  if test x$disable_yasm = xyes; then
++ AC_ARG_ENABLE(yasm,
++  [AC_HELP_STRING([--disable-yasm], [disable use of yasm 
assembler])])
++  if test x$enable_yasm = xno; then
 +embffmpeg_configure_args=$embffmpeg_configure_args --disable-yasm
 +  fi
 +
-- 
1.7.11.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] gst-ffmpeg yasm check fix

2013-04-08 Thread tom . zanussi
From: Tom Zanussi tom.zanu...@linux.intel.com

Builds started failing here, tracked it down to commit:

4d309730 ['gst-ffmpeg: configure-fix patch used wrong test']

which essentially just has the effect of never disabling yasm instead
of always disabling it as before.  With this patch it can be conditionally
enabled or disabled as perhaps intended.

The following changes since commit 6d4d42d63db4c3fcffd831ce359599f3ee1d710e:

  qemuimage-tests/sanity/boot: Increase timeout (2013-04-06 17:22:21 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib.git tzanussi/gst-ffmpeg-yasm-check-fix
  
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=tzanussi/gst-ffmpeg-yasm-check-fix

Tom Zanussi (1):
  gst-ffmpeg: fix --disable-yasm

 .../gstreamer/gst-ffmpeg-0.10.13/configure-fix.patch| 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

-- 
1.7.11.4


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


Re: [OE-core] [PATCH] udev: Move udevd back to /sbin

2013-04-08 Thread Koen Kooi

Op 8 apr. 2013, om 16:39 heeft Radu Moisan radu.moi...@intel.com het volgende 
geschreven:

 Fixes [Yocto #4046]

Can you eloborate a bit in the commit message?
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Sanity Failures - Segfaults in qemu images

2013-04-08 Thread Richard Purdie
On Sun, 2013-04-07 at 15:32 -0700, Khem Raj wrote:
 On Sun, Apr 7, 2013 at 1:23 AM, Richard Purdie
 richard.pur...@linuxfoundation.org wrote:
 
 http://autobuilder.yoctoproject.org:8011/builders/nightly-x86-64-lsb/builds/87/steps/Running%20Sanity%20Tests/logs/stdio


 what does complete dmesg output looks like ? I have seen ld.so
 segfaults on real x86_64 hardware but havent narrowed it down since
 it does not seem to bother my testing.​

I don't have the full dmesg output however I have put a wealth of info
about this into:

https://bugzilla.yoctoproject.org/show_bug.cgi?id=4216

Basically these do look like random segfaults happening any point on
the system and having a variety of effects. The double fault is
particularly worrying as it suggests the problem is kernel or qemu. I've
put a script which tends to reproduce the issue into the bugzilla.

When you say you've seen ld.so faults on real hardware, have you any
more details? Are they happening often? Just ld.so?

I need to know if this is a qemu issue or on real hardware too since the
latter is a release blocker and extremely serious.

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] smart: disable CHANNELSDIR

2013-04-08 Thread Mark Hatle

On 4/8/13 10:02 AM, Bogdan Marinescu wrote:

Make CHANNELSDIR in smart empty, since this causes host contamination issues
on some RPM-based hosts on which smart is already installed.

[YOCTO #3881]

Signed-off-by: Bogdan Marinescu bogdan.a.marine...@intel.com
---
  .../python/python-smartpm/smart-channelsdir.patch  | 24 ++
  .../python/python-smartpm_1.4.1.bb |  3 ++-
  2 files changed, 26 insertions(+), 1 deletion(-)
  create mode 100644 
meta/recipes-devtools/python/python-smartpm/smart-channelsdir.patch

diff --git 
a/meta/recipes-devtools/python/python-smartpm/smart-channelsdir.patch 
b/meta/recipes-devtools/python/python-smartpm/smart-channelsdir.patch
new file mode 100644
index 000..e621b33
--- /dev/null
+++ b/meta/recipes-devtools/python/python-smartpm/smart-channelsdir.patch
@@ -0,0 +1,24 @@
+Make CHANNELSDIR in smart empty, since this causes host contamination issues
+on some RPM-based hosts on which smart is already installed.
+
+[YOCTO #3881]
+
+Upstream-Status: Inappropriate [embedded specific]
+
+diff --git a/smart/plugins/channelsync.py b/smart/plugins/channelsync.py
+index 3ba95ff..646d696 100644
+--- a/smart/plugins/channelsync.py
 b/smart/plugins/channelsync.py
+@@ -23,7 +23,11 @@ from smart.channel import *
+ from smart import *
+ import os
+
+-CHANNELSDIR = /etc/smart/channels/
++# For now, we leave the definition of CHANNELSDIR empty. This prevents smart
++# from erroneously consider the  build host's channels while setting up its
++# channels [YOCTO #3881]. If this feature will be used in the future, 
CHANNELSDIR
++# should be set to a proper value.
++CHANNELSDIR = 


I don't remember if the channelsdir is used by default on the target or if there 
is a different directory.


Did you check if (on the target) you can still add channels and do a remove 
install/update of a package?



+
+ def syncChannels(channelsdir, force=None):
+
diff --git a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb 
b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb
index d92933f..001d9e4 100644
--- a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb
+++ b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb
@@ -11,7 +11,7 @@ LICENSE = GPLv2
  LIC_FILES_CHKSUM = file://LICENSE;md5=393a5ca445f6965873eca0259a17f833

  DEPENDS = python rpm
-PR = r8
+PR = r9
  SRCNAME = smart

  SRC_URI = \
@@ -27,6 +27,7 @@ SRC_URI = \
file://smart-improve-error-reporting.patch \
file://smart-multilib-fixes.patch \
file://smart-yaml-error.patch \
+  file://smart-channelsdir.patch \


  SRC_URI[md5sum] = 573ef32ba177a6b3c4bf7ef04873fcb6




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


Re: [OE-core] [PATCH] udev: Move udevd back to /sbin

2013-04-08 Thread Radu Moisan


On 04/08/2013 06:42 PM, Koen Kooi wrote:

Op 8 apr. 2013, om 16:39 heeft Radu Moisan radu.moi...@intel.com het volgende 
geschreven:


Fixes [Yocto #4046]

Can you eloborate a bit in the commit message?


Yes, will follow up with a v2

Radu

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


[OE-core] [PATCH] scripts/oe-pkgdata-util: find complementary packages for split packages

2013-04-08 Thread Paul Eggleton
Check after getting the original package name (e.g. undoing Debian
renaming) if there is a complementary package for that name, e.g. if
the glob is *-dev, then libudev0 - libudev - libudev-dev.

Fixes [YOCTO #4136].

Signed-off-by: Paul Eggleton paul.eggle...@linux.intel.com
---
 scripts/oe-pkgdata-util |   13 ++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/scripts/oe-pkgdata-util b/scripts/oe-pkgdata-util
index 900c27a..629b2d5 100755
--- a/scripts/oe-pkgdata-util
+++ b/scripts/oe-pkgdata-util
@@ -115,14 +115,21 @@ def glob(args):
 if not os.path.exists(fwdfile + .packaged):
 mappedpkg = 
 else:
-# That didn't work, so now get the PN, substitute that, 
then map in the other direction
 revlink = revpkgdata(pkg)
 if os.path.exists(revlink):
-pn = readpn(revlink)
-newpkg = g.replace(*, pn)
+# Check if we can map after undoing the package 
renaming (by resolving the symlink)
+origpkg = os.path.basename(os.readlink(revlink))
+newpkg = g.replace(*, origpkg)
 fwdfile = fwdpkgdata(newpkg)
 if os.path.exists(fwdfile):
 mappedpkg = readrenamed(fwdfile)
+else:
+# That didn't work, so now get the PN, substitute 
that, then map in the other direction
+pn = readpn(revlink)
+newpkg = g.replace(*, pn)
+fwdfile = fwdpkgdata(newpkg)
+if os.path.exists(fwdfile):
+mappedpkg = readrenamed(fwdfile)
 if not os.path.exists(fwdfile + .packaged):
 mappedpkg = 
 else:
-- 
1.7.10.4


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


Re: [OE-core] [PATCH 2/2] gtk-update-icon-cache-native: create wrapper script

2013-04-08 Thread Martin Jansa
On Mon, Apr 08, 2013 at 06:06:41PM +0300, Laurentiu Palcu wrote:
 When using the sstate from another build machine, the path to the pixbuf
 loader's cache points to a path on the remote machine. Hence, the update
 of the icon cache fails on host.
 
 Signed-off-by: Laurentiu Palcu laurentiu.pa...@intel.com
 ---
  .../gtk+/gtk-update-icon-cache-native_3.4.4.bb |4 
  1 file changed, 4 insertions(+)
 
 diff --git a/meta/recipes-gnome/gtk+/gtk-update-icon-cache-native_3.4.4.bb 
 b/meta/recipes-gnome/gtk+/gtk-update-icon-cache-native_3.4.4.bb
 index 93c30a7..73b7644 100644
 --- a/meta/recipes-gnome/gtk+/gtk-update-icon-cache-native_3.4.4.bb
 +++ b/meta/recipes-gnome/gtk+/gtk-update-icon-cache-native_3.4.4.bb
 @@ -40,4 +40,8 @@ do_compile() {
  do_install() {
   install -d ${D}${bindir}
  install -m 0755 ${B}/gtk-update-icon-cache ${D}${bindir}

^ can you fix indentation in this line, when you're changing this file?

Thanks

JaMa (who proposed to use consistent white-space for shell/python tasks
long time ago)

 +
 + create_wrapper ${D}/${bindir}/gtk-update-icon-cache \
 + 
 GDK_PIXBUF_MODULE_FILE=${STAGING_LIBDIR_NATIVE}/gdk-pixbuf-2.0/2.10.0/loaders.cache
 +
  }
 -- 
 1.7.9.5
 
 
 ___
 Openembedded-core mailing list
 Openembedded-core@lists.openembedded.org
 http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

-- 
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 v2] udev: Move udevd back to /sbin

2013-04-08 Thread Radu Moisan
Along with v182 upgrade udevd was moved to ${base_libdir}
making scripts like init-live.sh to fail in finding udevd

Fixes [Yocto #4046]

Signed-off-by: Radu Moisan radu.moi...@intel.com
---
 meta/recipes-core/udev/udev.inc|3 ++-
 meta/recipes-core/udev/udev_182.bb |2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index e358d2d..c4d2ce4 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -40,7 +40,7 @@ EXTRA_OECONF = --disable-introspection \
 ac_cv_file__usr_share_hwdata_pci_ids=no \
 ac_cv_file__usr_share_misc_pci_ids=yes \
 --sbindir=${base_sbindir} \
---libexecdir=${base_libdir} \
+--libexecdir=${base_sbindir} \
 --with-rootlibdir=${base_libdir} \
 --with-rootprefix= \

@@ -61,6 +61,7 @@ RRECOMMENDS_${PN} += udev-utils
 FILES_${PN}-dbg += ${libexecdir}/.debug
 FILES_${PN}-dbg += ${base_libdir}/udev/.debug/
 FILES_${PN}-dbg += ${base_libdir}/udev/.debug/*
+FILES_${PN}-dbg += ${base_sbindir}/udev/.debug/*
 FILES_${PN}-dev = ${datadir}/pkgconfig/udev.pc
 FILES_libudev = ${base_libdir}/libudev.so.*
 FILES_libudev-dbg = ${base_libdir}/.debug/libudev.so.*
diff --git a/meta/recipes-core/udev/udev_182.bb 
b/meta/recipes-core/udev/udev_182.bb
index 42b4d08..d66292e 100644
--- a/meta/recipes-core/udev/udev_182.bb
+++ b/meta/recipes-core/udev/udev_182.bb
@@ -1,6 +1,6 @@
 include udev.inc
 
-PR = r6
+PR = r7
 
 # module-init-tools from kmod_git will provide libkmod runtime
 DEPENDS += module-init-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] local.conf.sample: Add info about -ptest package group

2013-04-08 Thread Trevor Woerner
On Fri, Apr 5, 2013 at 11:35 AM, maxin.j...@enea.com wrote:
 +#  ptest-pkgs - add -ptest packages for all ptest-enabled packages
 +# (useful if you want to run the package test suites)

Is there a simple way to discover which packages are ptest-enabled?

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


Re: [OE-core] [PATCH] openssl: Upgrade to v1.0.1e

2013-04-08 Thread Richard Purdie
On Mon, 2013-04-08 at 19:47 +0300, Radu Moisan wrote:
 Dropped obolete patches and pulled updates for debian patches

Isn't there some CVE this upgrade fixes which would be worth a mention
in here?

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] openssl: Upgrade to v1.0.1e

2013-04-08 Thread Radu Moisan


On 04/08/2013 07:54 PM, Richard Purdie wrote:

On Mon, 2013-04-08 at 19:47 +0300, Radu Moisan wrote:

Dropped obolete patches and pulled updates for debian patches

Isn't there some CVE this upgrade fixes which would be worth a mention
in here?


With respect to what we had (1.0.0j) Scott pointed out the following

http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2012-2686
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2013-0166
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2013-0169

Yocto #3965  https://bugzilla.yoctoproject.org/show_bug.cgi?id=3965


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


Re: [OE-core] [PATCH 2/3] oe-buildenv-internal: Only add to $PATH if needed

2013-04-08 Thread Trevor Woerner
On Fri, Apr 5, 2013 at 12:59 PM, Peter Kjellerstedt
peter.kjellerst...@axis.com wrote:
 +[ ${PATH#$NEWPATHS} != $PATH ] || PATH=$NEWPATHS$PATH

This is certainly a welcome addition in functionality, but it relies
on the pattern remaining at the start of the PATH (i.e. the user
hasn't played with PATH in any way). Could we not use the
${parameter/pattern/string} parameter expansion instead (e.g.
${PATH/$NEWPATHS/}) so it doesn't matter whether the user has
further modified the PATH?

Best regards,
Trevor

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


Re: [OE-core] [PATCH 2/3] oe-buildenv-internal: Only add to $PATH if needed

2013-04-08 Thread Paul Eggleton
On Monday 08 April 2013 13:31:50 Trevor Woerner wrote:
 On Fri, Apr 5, 2013 at 12:59 PM, Peter Kjellerstedt
 peter.kjellerst...@axis.com wrote:
  +[ ${PATH#$NEWPATHS} != $PATH ] || PATH=$NEWPATHS$PATH
 
 This is certainly a welcome addition in functionality, but it relies
 on the pattern remaining at the start of the PATH (i.e. the user
 hasn't played with PATH in any way). Could we not use the
 ${parameter/pattern/string} parameter expansion instead (e.g.
 ${PATH/$NEWPATHS/}) so it doesn't matter whether the user has
 further modified the PATH?

Unfortunately I think this is specific to bash, so it may not be portable. 
Maybe the equivalent can be achieved with sed however.

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] bluez4: add readline dependency

2013-04-08 Thread Saul Wold

On 04/08/2013 05:56 AM, Alex DAMIAN wrote:

From: Alexandru DAMIAN alexandru.dam...@intel.com

bluez4 uses readline to be build, but the dependency is not listed
This is listed in the configuration log.
So we add it.

As far as I can tell it's needed only for gatttool, is this a tool that 
we need / want to provide for bluez?


This seems to be a tools for viewing the Generic Attribute Profile (GATT).

It can be disabled by setting ac_cv_lib_readline_main=no


Sau!



Signed-off-by: Alexandru DAMIAN alexandru.dam...@intel.com
---
  meta/recipes-connectivity/bluez/bluez4.inc |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/bluez/bluez4.inc 
b/meta/recipes-connectivity/bluez/bluez4.inc
index bff24d3..42d82b0 100644
--- a/meta/recipes-connectivity/bluez/bluez4.inc
+++ b/meta/recipes-connectivity/bluez/bluez4.inc
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
  file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \
  
file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e \
  
file://sbc/sbc.c;beginline=1;endline=25;md5=1a40781ed30d50d8639323a184aeb191
-DEPENDS = udev libusb dbus-glib glib-2.0 libcheck
+DEPENDS = udev libusb dbus-glib glib-2.0 libcheck readline
  RDEPENDS_${PN}-dev = bluez-hcidump

  PACKAGECONFIG ??= \



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


Re: [OE-core] [PATCH] openssl: Upgrade to v1.0.1e

2013-04-08 Thread Saul Wold

On 04/08/2013 10:00 AM, Radu Moisan wrote:


On 04/08/2013 07:54 PM, Richard Purdie wrote:

On Mon, 2013-04-08 at 19:47 +0300, Radu Moisan wrote:

Dropped obolete patches and pulled updates for debian patches

Isn't there some CVE this upgrade fixes which would be worth a mention
in here?


With respect to what we had (1.0.0j) Scott pointed out the following

http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2012-2686
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2013-0166
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2013-0169

Yocto #3965  https://bugzilla.yoctoproject.org/show_bug.cgi?id=3965



Please put this in the commit message is what RP is getting after here.

Thanks
Sau!


Radu


___
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] bluez4: add readline dependency

2013-04-08 Thread Mark Hatle

On 4/8/13 12:57 PM, Saul Wold wrote:

On 04/08/2013 05:56 AM, Alex DAMIAN wrote:

From: Alexandru DAMIAN alexandru.dam...@intel.com

bluez4 uses readline to be build, but the dependency is not listed
This is listed in the configuration log.
So we add it.


As far as I can tell it's needed only for gatttool, is this a tool that
we need / want to provide for bluez?

This seems to be a tools for viewing the Generic Attribute Profile (GATT).

It can be disabled by setting ac_cv_lib_readline_main=no


We -really- want to avoid readline if possible, as it brings in GPLv3 
dependencies into the system.


If that tool is generically useful, I'd suggest a PACKAGECONFIG setting then.

--Mark



Sau!



Signed-off-by: Alexandru DAMIAN alexandru.dam...@intel.com
---
   meta/recipes-connectivity/bluez/bluez4.inc |2 +-
   1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/bluez/bluez4.inc 
b/meta/recipes-connectivity/bluez/bluez4.inc
index bff24d3..42d82b0 100644
--- a/meta/recipes-connectivity/bluez/bluez4.inc
+++ b/meta/recipes-connectivity/bluez/bluez4.inc
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
   file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \
   
file://src/main.c;beginline=1;endline=24;md5=9bc54b93cd7e17bf03f52513f39f926e \
   
file://sbc/sbc.c;beginline=1;endline=25;md5=1a40781ed30d50d8639323a184aeb191
-DEPENDS = udev libusb dbus-glib glib-2.0 libcheck
+DEPENDS = udev libusb dbus-glib glib-2.0 libcheck readline
   RDEPENDS_${PN}-dev = bluez-hcidump

   PACKAGECONFIG ??= \



___
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 1/2] alsa-tools: fix build when x11 and gtk+ not available

2013-04-08 Thread Saul Wold

On 04/08/2013 06:33 AM, Trevor Woerner wrote:

With a rather recent HEAD

Build Configuration:
BB_VERSION= 1.17.1
BUILD_SYS = x86_64-linux
NATIVELSBSTRING   = openSUSE-project-12.3
TARGET_SYS= x86_64-poky-linux
MACHINE   = qemux86-64
DISTRO= poky
DISTRO_VERSION= 1.3+snapshot-20130407
TUNE_FEATURES = m64
TARGET_FPU= 
meta
meta-yocto
meta-yocto-bsp= master:813127247a1100b1abe179dfba25795560eac864

I am able to successfully perform a bitbake world on a number of
different targets:

- qemux86
- qemuarm
- qemumips
- qemux86-64

Unfortunately qemuppcfails when building alsa-tools. Is anyone else
seeing the same thing?

| make[1]: Entering directory
`/home/trevor/build/yocto/fullbuild/qemuppc/work/ppc603e-poky-linux/alsa-tools/1.0.26.1-r1/alsa-tools-1.0.26.1/hda-verb'
| make[1]: warning: jobserver unavailable: using -j1.  Add `+' to parent
make rule.
| powerpc-poky-linux-gcc  -m32 -mhard-float  -mcpu=603e
--sysroot=/home/trevor/build/yocto/fullbuild/qemuppc/sysroots/qemuppc
-DPACKAGE_NAME=\\ -DPACKAGE_TARNAME=\\ -DPACKAGE_VERSION=\\
-DPACKAGE_STRING=\\ -DPACKAGE_BUGREPORT=\\ -DPACKAGE_URL=\\
-DPACKAGE=\hda-verb\ -DVERSION=\0.4\ -DSTDC_HEADERS=1 -I. -O2
-Wall -pipe -g -MT hda-verb.o -MD -MP -MF .deps/hda-verb.Tpo -c -o
hda-verb.o hda-verb.c
| hda-verb.c:17:20: fatal error: sys/io.h: No such file or directory
| compilation terminated.
| make[1]: *** [hda-verb.o] Error 1
| make[1]: Leaving directory
`/home/trevor/build/yocto/fullbuild/qemuppc/work/ppc603e-poky-linux/alsa-tools/1.0.26.1-r1/alsa-tools-1.0.26.1/hda-verb'
| make: *** [all] Error 1
| ERROR: oe_runmake failed


I recall doing a patch for this, I just reviewed it and it seems I did 
not do the right #if directive.


I was fixing the mips build which had the same problem and did not 
correctly retest.


Patch out shortly.

Sau!


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


Re: [OE-core] [PATCH 1/2] systemd: set INHIBIT_UPDATERCD_BBCLASS without sysvinit in features

2013-04-08 Thread Martin Jansa
On Thu, Apr 04, 2013 at 11:38:41PM +0100, Richard Purdie wrote:
 On Thu, 2013-04-04 at 18:55 +0200, Martin Jansa wrote:
  On Thu, Apr 04, 2013 at 05:46:48PM +0100, Richard Purdie wrote:
   On Thu, 2013-04-04 at 18:42 +0200, Martin Jansa wrote:
* fixes udev configure in run-postinsts failing with:
  update-rc.d: /etc/init.d/systemd-udev: file does not exist
  because systemd-udev is installed only with sysvinit in features
  but update-rc.d was always called from PN postinst

Signed-off-by: Martin Jansa martin.ja...@gmail.com
---
 meta/recipes-core/systemd/systemd_199.bb | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/meta/recipes-core/systemd/systemd_199.bb 
b/meta/recipes-core/systemd/systemd_199.bb
index ba1d133..e574548 100644
--- a/meta/recipes-core/systemd/systemd_199.bb
+++ b/meta/recipes-core/systemd/systemd_199.bb
@@ -239,6 +239,12 @@ INITSCRIPT_PACKAGES = udev
 INITSCRIPT_NAME_udev = systemd-udevd
 INITSCRIPT_PARAMS_udev = start 03 S .
 
+python __anonymous() {
+features = d.getVar(DISTRO_FEATURES, True).split()
+if sysvinit not in features:
+d.setVar(INHIBIT_UPDATERCD_BBCLASS, 1)
+}
+
 # TODO:
 # u-a for runlevel and telinit
   
   Would this make sense to be in systemd.bbclass?
  
  Similar logic is in systemd.bbclass already, but systemd is not inherited 
  from
  systemd and dbus recipes.
 
 Ok, fair enough. I hadn't realised that.
 
  Also the version from systemd.bbclass does check also for systemd in
  DISTRO_FEATURES, but that's not wanted here, because decision to install 
  init.d
  script is based only on sysvinit in DISTRO_FEATURES.
  
  Lot's of fun with all init systems sharing the same PN :/.

There is also error from prerm :/

//var/lib/opkg/info/dbus-1.prerm: line 3: /etc/init.d/dbus-1: No such file or 
directory

updatercd_prerm() {
if test x$D = x; then
${INIT_D_DIR}/${INITSCRIPT_NAME} stop
fi
}

not sure if testing update-rc.d existence like in postinst/postrm 
if type update-rc.d /dev/null 2/dev/null; then
is right way, checking ${INIT_D_DIR}/${INITSCRIPT_NAME}
existence will possibly hide some real issues...

sigh
-- 
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] [RFC] [PATCH 0/2] Routerstation Pro: kernel.bbclass: do_sizecheck, do_strip

2013-04-08 Thread michel . thebeau
Hi Richard,

Here is request for comment, v2 of fixes to kernel.bbclass for 3514 and
3514 bugs (and now bug 4220).  

For the first patch you had asked that KERNEL_OUTPUT be used
consistently for the kernel.

The second patch is new, and what I'm especially looking for comments
on.  It is to get back to what 3515 original said, to strip the elf
image, and what Bruce said: finding the right place to strip the image.

Commit 9cd3816e4db97c8fd093a120a75a2b5d193afcdd didn't quite work,
which made vmlinux.bin (binary format) the default kernel image.  See
also bug 4220.

FYI, the binary vmlinux.bin does boot when you know how.  0x806a5430
comes from copying the entry address of the elf.
RedBoot load -v -m tftp -h 128.224.149.6 -b 0x8006 -r mthebeau/vmlinux.bin
RedBoot exec -b 0x8006 -c console=ttyS0,115200 root=/dev/sda1 rw 
rootdelay=2 board=UBNT-RSPRO 0x806a5430

I have a version of this patch that strips the kernel in place
instead of carrying a copy.  One reason to carry a seperate copy could
be the following build warning, which is issued when stripping the
image in place:

WARNING: File '/usr/src/kernel/arch/mips/boot/vmlinux.stripped' from
linux-yocto was already stripped, this will prevent future debugging!

A third patch here for your reference only; will resend it to the poky
list if you like this approach.

Thanks,

M




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


[OE-core] [PATCH 2/2] kernel.bbclass: do_strip: allow recipes to strip the kernel

2013-04-08 Thread michel . thebeau
From: Michel Thebeau michel.theb...@windriver.com

Allow recipes to specify sections to be stripped from the kernel output
using KERNEL_IMAGE_STRIP_EXTRA_SECTIONS.  For example:

KERNEL_IMAGE_STRIP_EXTRA_SECTIONS = .comment .unwanted

The file to be stripped is a copy of ${KERNEL_OUTPUT} and will be given
the same name with an additional .stripped suffix.  The suffix can be
overridden using KERNEL_STRIP_SUFFIX.

Since the toolchain does not give indication when the specified sections
are absent, we read the sections first and make this report by issuing a
warning to the developer.

The toolchain by default strips the image with the -s option (even
when -s is not specified):
-s --strip-all   Remove all symbol and relocation information

For example, these sections are always removed:
.debug_aranges
.debug_info
.debug_abbrev
.debug_line
.debug_frame
.debug_str
.debug_loc
.debug_ranges
.symtab
.strtab

In addition to these, the sections listed in
KERNEL_IMAGE_STRIP_EXTRA_SECTIONS will also be removed.

Only stripping of vmlinux (elf) is supported at this time.  A warning
will be given if the image type is not vmlinux.

Stripping the image could also be done in the kernel, but that would
only work for linux-yocto based kernels, so it's not the route we
decided to go.

[YOCTO 3515]

Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com
Signed-off-by: Michel Thebeau michel.theb...@windriver.com
---
 meta/classes/kernel.bbclass |   65 +--
 1 file changed, 63 insertions(+), 2 deletions(-)

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index af58887..60da4e2 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -41,6 +41,10 @@ KERNEL_RELEASE ?= ${KERNEL_VERSION}
 KERNEL_OUTPUT ?= arch/${ARCH}/boot/${KERNEL_IMAGETYPE}
 KERNEL_IMAGEDEST = boot
 
+# When we strip the output, it is here
+KERNEL_STRIPPED_SUFFIX ?= .stripped
+KERNEL_OUTPUT_STRIPPED ?= ${KERNEL_OUTPUT}${KERNEL_STRIPPED_SUFFIX}
+
 #
 # configuration
 #
@@ -109,6 +113,12 @@ kernel_do_install() {
install -d ${D}/${KERNEL_IMAGEDEST}
install -d ${D}/boot
install -m 0644 ${KERNEL_OUTPUT} 
${D}/${KERNEL_IMAGEDEST}/${KERNEL_IMAGETYPE}-${KERNEL_VERSION}
+
+   if [ -n ${KERNEL_IMAGE_STRIP_EXTRA_SECTIONS} ]; then
+   install -m 0644 ${KERNEL_OUTPUT_STRIPPED} \
+   
${D}/${KERNEL_IMAGEDEST}/${KERNEL_IMAGETYPE}-${KERNEL_VERSION}${KERNEL_STRIPPED_SUFFIX}
+   fi;
+
install -m 0644 System.map ${D}/boot/System.map-${KERNEL_VERSION}
install -m 0644 .config ${D}/boot/config-${KERNEL_VERSION}
install -m 0644 vmlinux ${D}/boot/vmlinux-${KERNEL_VERSION}
@@ -153,6 +163,12 @@ kernel_do_install() {
cd $pwd
fi
install -m 0644 ${KERNEL_OUTPUT} $kerneldir/${KERNEL_IMAGETYPE}
+
+   if [ -n ${KERNEL_IMAGE_STRIP_EXTRA_SECTIONS} ]; then
+   install -m 0644 ${KERNEL_OUTPUT_STRIPPED} \
+   $kerneldir/${KERNEL_IMAGETYPE}${KERNEL_STRIPPED_SUFFIX}
+   fi
+
install -m 0644 System.map $kerneldir/System.map-${KERNEL_VERSION}
 
#
@@ -289,12 +305,46 @@ python split_kernel_packages () {
 do_split_packages(d, root='/lib/firmware', file_regex='^(.*)\.cis$', 
output_pattern='kernel-firmware-%s', description='Firmware for %s', 
recursive=True, extra_depends='')
 }
 
+do_strip() {
+   if [ -n ${KERNEL_IMAGE_STRIP_EXTRA_SECTIONS} ]; then
+   if [[ ${KERNEL_IMAGETYPE} != vmlinux ]]; then
+   bbwarn image type will not be stripped (not 
supported): ${KERNEL_IMAGETYPE}
+   return
+   fi
+
+   cd ${B}
+   cp ${KERNEL_OUTPUT} ${KERNEL_OUTPUT_STRIPPED}
+
+   headers=`$CROSS_COMPILEreadelf -S ${KERNEL_OUTPUT_STRIPPED} | 
\
+ grep ^ \{1,\}\[[0-9 ]\{1,\}\] [^ ] | \
+ sed s/^ \{1,\}\[[0-9 ]\{1,\}\] // | \
+ gawk '{print $1}'`
+
+   for str in ${KERNEL_IMAGE_STRIP_EXTRA_SECTIONS}; do {
+   if [[ $headers != *$str* ]]; then
+   bbwarn Section not found: $str;
+   fi
+
+   $CROSS_COMPILEstrip -s -R $str 
${KERNEL_OUTPUT_STRIPPED}
+   }; done
+   fi;
+}
+do_strip[dirs] = ${B}
+
+addtask do_strip before do_sizecheck after do_kernel_link_vmlinux
+
 # Support checking the kernel size since some kernels need to reside in 
partitions
 # with a fixed length or there is a limit in transferring the kernel to memory
 do_sizecheck() {
+   if [ -n ${KERNEL_IMAGE_STRIP_EXTRA_SECTIONS} ]; then
+   koutf=${KERNEL_OUTPUT_STRIPPED}
+   else
+   koutf=${KERNEL_OUTPUT}
+   fi
+
if [ ! -z ${KERNEL_IMAGE_MAXSIZE} ]; then
cd ${B}
-   size=`ls -lL ${KERNEL_OUTPUT} | awk '{ print $5}'`
+   

[OE-core] [PATCH 1/2] kernel.bbclass: do_sizecheck: update path to build image and do not delete

2013-04-08 Thread michel . thebeau
From: Michel Thebeau michel.theb...@windriver.com

do_sizecheck has a few issues especially with vmlinux image type.

It breaks because KERNEL_OUTPUT is a path relative to ${B}.  When
do_sizecheck runs it does not find the file (because the working
directory is elsewhere) and does not fail.

Also, the image file referenced by KERNEL_OUTPUT may be a link.

Finally, when do_sizecheck deletes the oversized kernel image it leaves
the previously run do_compile task with inaccurate status.

So, do the following:
 - specify that the working directory should be ${B}
 - use ls -L to reference to the real file, and ensure that the link
   file is created
 - keep the oversized image file so the status of do_compile is valid

[YOCTO #3514]

Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com
Signed-off-by: Michel Thebeau michel.theb...@windriver.com
---
 meta/classes/kernel.bbclass |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index d57d1f5..af58887 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -293,15 +293,16 @@ python split_kernel_packages () {
 # with a fixed length or there is a limit in transferring the kernel to memory
 do_sizecheck() {
if [ ! -z ${KERNEL_IMAGE_MAXSIZE} ]; then
-   size=`ls -l ${KERNEL_OUTPUT} | awk '{ print $5}'`
+   cd ${B}
+   size=`ls -lL ${KERNEL_OUTPUT} | awk '{ print $5}'`
if [ $size -ge ${KERNEL_IMAGE_MAXSIZE} ]; then
-   rm ${KERNEL_OUTPUT}
die This kernel (size=$size  ${KERNEL_IMAGE_MAXSIZE}) 
is too big for your device. Please reduce the size of the kernel by making more 
of it modular.
fi
fi
 }
+do_sizecheck[dirs] = ${B}
 
-addtask sizecheck before do_install after do_compile
+addtask sizecheck before do_install after do_kernel_link_vmlinux
 
 KERNEL_IMAGE_BASE_NAME ?= 
${KERNEL_IMAGETYPE}-${PE}-${PV}-${PR}-${MACHINE}-${DATETIME}
 # Don't include the DATETIME variable in the sstate package signatures
-- 
1.7.9.7


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


[OE-core] [poky] [PATCH 1/1] routerstationpro: strip the output kernel of .comment section

2013-04-08 Thread michel . thebeau
From: Michel Thebeau michel.theb...@windriver.com

The routerstationpro has a 16mb flash which the kernel image should
fit into.  The default build type for vmlinux then should be a
stripped vmlinux.

Use KERNEL_IMAGE_STRIP_EXTRA_SECTIONS  to do this.

Reverts commit 9cd3816e4db97c8fd093a120a75a2b5d193afcdd, which causes:
RedBoot load -v vlm-boards/19256/kernel
Using default protocol (TFTP)
Unrecognized image type: 0x0

[YOCTO 3515]
[YOCTO 4220]

Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com
Signed-off-by: Michel Thebeau michel.theb...@windriver.com
---
 meta-yocto-bsp/conf/machine/routerstationpro.conf |5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta-yocto-bsp/conf/machine/routerstationpro.conf 
b/meta-yocto-bsp/conf/machine/routerstationpro.conf
index a727e2a..723625b 100644
--- a/meta-yocto-bsp/conf/machine/routerstationpro.conf
+++ b/meta-yocto-bsp/conf/machine/routerstationpro.conf
@@ -6,8 +6,9 @@ require conf/machine/include/tune-mips32.inc
 
 MACHINE_FEATURES = screen keyboard pci usbhost ext2 ext3 serial
 
-KERNEL_ALT_IMAGETYPE = vmlinux
-KERNEL_IMAGETYPE = vmlinux.bin
+KERNEL_IMAGETYPE = vmlinux
+KERNEL_ALT_IMAGETYPE = vmlinux.bin
+KERNEL_IMAGE_STRIP_EXTRA_SECTIONS  = .comment
 
 PREFERRED_PROVIDER_virtual/kernel ?= linux-yocto
 PREFERRED_VERSION_linux-yocto ?= 3.4%
-- 
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] udev: Move udevd back to /sbin

2013-04-08 Thread Koen Kooi

Op 8 apr. 2013, om 18:29 heeft Radu Moisan radu.moi...@intel.com het volgende 
geschreven:

 Along with v182 upgrade udevd was moved to ${base_libdir}

By upstream, yes

 making scripts like init-live.sh to fail in finding udevd

So you should fix those scripts, not mess around with udev.

 Fixes [Yocto #4046]
 
 Signed-off-by: Radu Moisan radu.moi...@intel.com
 ---
 meta/recipes-core/udev/udev.inc|3 ++-
 meta/recipes-core/udev/udev_182.bb |2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)
 
 diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
 index e358d2d..c4d2ce4 100644
 --- a/meta/recipes-core/udev/udev.inc
 +++ b/meta/recipes-core/udev/udev.inc
 @@ -40,7 +40,7 @@ EXTRA_OECONF = --disable-introspection \
 ac_cv_file__usr_share_hwdata_pci_ids=no \
 ac_cv_file__usr_share_misc_pci_ids=yes \
 --sbindir=${base_sbindir} \
 ---libexecdir=${base_libdir} \
 +--libexecdir=${base_sbindir} \
 --with-rootlibdir=${base_libdir} \
 --with-rootprefix= \

 @@ -61,6 +61,7 @@ RRECOMMENDS_${PN} += udev-utils
 FILES_${PN}-dbg += ${libexecdir}/.debug
 FILES_${PN}-dbg += ${base_libdir}/udev/.debug/
 FILES_${PN}-dbg += ${base_libdir}/udev/.debug/*
 +FILES_${PN}-dbg += ${base_sbindir}/udev/.debug/*
 FILES_${PN}-dev = ${datadir}/pkgconfig/udev.pc
 FILES_libudev = ${base_libdir}/libudev.so.*
 FILES_libudev-dbg = ${base_libdir}/.debug/libudev.so.*
 diff --git a/meta/recipes-core/udev/udev_182.bb 
 b/meta/recipes-core/udev/udev_182.bb
 index 42b4d08..d66292e 100644
 --- a/meta/recipes-core/udev/udev_182.bb
 +++ b/meta/recipes-core/udev/udev_182.bb
 @@ -1,6 +1,6 @@
 include udev.inc
 
 -PR = r6
 +PR = r7
 
 # module-init-tools from kmod_git will provide libkmod runtime
 DEPENDS += module-init-tools
 -- 
 1.7.9.5
 
 
 ___
 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] alsa-tools: Fix sys/io.h patch

2013-04-08 Thread Saul Wold
I blew my #if expression!

Signed-off-by: Saul Wold s...@linux.intel.com
---
 meta/recipes-multimedia/alsa/alsa-tools/mips_has_no_io_h.patch | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-multimedia/alsa/alsa-tools/mips_has_no_io_h.patch 
b/meta/recipes-multimedia/alsa/alsa-tools/mips_has_no_io_h.patch
index 7083cb2..09b10f1 100644
--- a/meta/recipes-multimedia/alsa/alsa-tools/mips_has_no_io_h.patch
+++ b/meta/recipes-multimedia/alsa/alsa-tools/mips_has_no_io_h.patch
@@ -1,3 +1,6 @@
+Upstream-Status: Pending
+Signed-off-by: Saul Wold s...@linux.intel.com
+
 Index: alsa-tools-1.0.26.1/hda-verb/hda-verb.c
 ===
 --- alsa-tools-1.0.26.1.orig/hda-verb/hda-verb.c
@@ -7,7 +10,7 @@ Index: alsa-tools-1.0.26.1/hda-verb/hda-verb.c
  #include unistd.h
  #include sys/ioctl.h
 -#ifndef __PPC__
-+#if __PPC__ || __MIPS__
++#if !(__PPC__ || __mips__)
  #include sys/io.h
  #endif
  #include sys/types.h
-- 
1.8.0.2


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


Re: [OE-core] [PATCH v2] udev: Move udevd back to /sbin

2013-04-08 Thread Otavio Salvador
On Mon, Apr 8, 2013 at 5:32 PM, Koen Kooi k...@dominion.thruhere.net wrote:

 Op 8 apr. 2013, om 18:29 heeft Radu Moisan radu.moi...@intel.com het 
 volgende geschreven:

 Along with v182 upgrade udevd was moved to ${base_libdir}

 By upstream, yes

 making scripts like init-live.sh to fail in finding udevd

 So you should fix those scripts, not mess around with udev.

 Fixes [Yocto #4046]

 Signed-off-by: Radu Moisan radu.moi...@intel.com

This move has been done long time ago so I agree with Koen here: the
scripts needs to be fixed. Another thing, it should also be tested
with systemd feature enabled as udev will be called systemd-udevd in
this case, IIRC.

--
Otavio Salvador O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854  http://projetos.ossystems.com.br

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


Re: [OE-core] [PATCH 2/2] kernel.bbclass: do_strip: allow recipes to strip the kernel

2013-04-08 Thread Richard Purdie
On Mon, 2013-04-08 at 16:15 -0400, michel.theb...@windriver.com wrote:
 From: Michel Thebeau michel.theb...@windriver.com
 
 Allow recipes to specify sections to be stripped from the kernel output
 using KERNEL_IMAGE_STRIP_EXTRA_SECTIONS.  For example:
 
 KERNEL_IMAGE_STRIP_EXTRA_SECTIONS = .comment .unwanted
 
 The file to be stripped is a copy of ${KERNEL_OUTPUT} and will be given
 the same name with an additional .stripped suffix.  The suffix can be
 overridden using KERNEL_STRIP_SUFFIX.
 
 Since the toolchain does not give indication when the specified sections
 are absent, we read the sections first and make this report by issuing a
 warning to the developer.
 
 The toolchain by default strips the image with the -s option (even
 when -s is not specified):
 -s --strip-all   Remove all symbol and relocation information
 
 For example, these sections are always removed:
 .debug_aranges
 .debug_info
 .debug_abbrev
 .debug_line
 .debug_frame
 .debug_str
 .debug_loc
 .debug_ranges
 .symtab
 .strtab
 
 In addition to these, the sections listed in
 KERNEL_IMAGE_STRIP_EXTRA_SECTIONS will also be removed.
 
 Only stripping of vmlinux (elf) is supported at this time.  A warning
 will be given if the image type is not vmlinux.
 
 Stripping the image could also be done in the kernel, but that would
 only work for linux-yocto based kernels, so it's not the route we
 decided to go.
 
 [YOCTO 3515]
 
 Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com
 Signed-off-by: Michel Thebeau michel.theb...@windriver.com

Can we please just have one output kernel, not two. Is the unstripped
version useful anywhere?

Cheers,

Richard




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


[OE-core] [PATCH v3] base.bbclass: Fix matching of MACHINEOVERRIDES in COMPATIBLE_MACHINE

2013-04-08 Thread Otavio Salvador
When a MACHINEOVERRIDES has more than one value, split by ':' as usual
OVERRIDES, this were not being properly checked in COMPATIBLE_MACHINE
matching as we need to iterate over each SoC family and check if it is
compatible or not.

Signed-off-by: Otavio Salvador ota...@ossystems.com.br
---
Changes for v3:
- Stop checking for SOC_FAMILY as it is just for compatibility with
  old BSPs; we move to MACHINEOVERRIDES for this case (RP)

 meta/classes/base.bbclass | 11 ++-
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index abd6a52..313359c 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -515,11 +515,12 @@ python () {
 need_machine = d.getVar('COMPATIBLE_MACHINE', True)
 if need_machine:
 import re
-this_machine = d.getVar('MACHINE', True)
-if this_machine and not re.match(need_machine, this_machine):
-this_soc_family = d.getVar('SOC_FAMILY', True)
-if (this_soc_family and not re.match(need_machine, 
this_soc_family)) or not this_soc_family:
-raise bb.parse.SkipPackage(incompatible with machine %s 
(not in COMPATIBLE_MACHINE) % this_machine)
+compat_machines.extend((d.getVar('MACHINEOVERRIDES', True) or 
).split(:))
+for m in compat_machines:
+if re.match(need_machine, m):
+break
+else:
+raise bb.parse.SkipPackage(incompatible with machine %s (not 
in COMPATIBLE_MACHINE) % d.getVar('MACHINE', True))
 
 
 bad_licenses = (d.getVar('INCOMPATIBLE_LICENSE', True) or ).split()
-- 
1.8.1


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


Re: [OE-core] [PATCH v2] udev: Move udevd back to /sbin

2013-04-08 Thread Richard Purdie
On Mon, 2013-04-08 at 17:40 -0300, Otavio Salvador wrote:
 On Mon, Apr 8, 2013 at 5:32 PM, Koen Kooi k...@dominion.thruhere.net wrote:
 
  Op 8 apr. 2013, om 18:29 heeft Radu Moisan radu.moi...@intel.com het 
  volgende geschreven:
 
  Along with v182 upgrade udevd was moved to ${base_libdir}
 
  By upstream, yes
 
  making scripts like init-live.sh to fail in finding udevd
 
  So you should fix those scripts, not mess around with udev.
 
  Fixes [Yocto #4046]
 
  Signed-off-by: Radu Moisan radu.moi...@intel.com
 
 This move has been done long time ago so I agree with Koen here: the
 scripts needs to be fixed. Another thing, it should also be tested
 with systemd feature enabled as udev will be called systemd-udevd in
 this case, IIRC.

It is not the scripts, see the previous discussion that was sent out and
Koen at least replied to. The scripts are the least of the problems.

Yes, the commit message here could be more accurate but that doesn't
change the problem of the issue reported in #4046 and previously
discussed.

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 2/2] kernel.bbclass: do_strip: allow recipes to strip the kernel

2013-04-08 Thread Michel Thebeau


On 13-04-08 04:54 PM, Richard Purdie wrote:
 On Mon, 2013-04-08 at 16:15 -0400, michel.theb...@windriver.com wrote:
 From: Michel Thebeau michel.theb...@windriver.com

 Allow recipes to specify sections to be stripped from the kernel output
 using KERNEL_IMAGE_STRIP_EXTRA_SECTIONS.  For example:

 KERNEL_IMAGE_STRIP_EXTRA_SECTIONS = .comment .unwanted

 The file to be stripped is a copy of ${KERNEL_OUTPUT} and will be given
 the same name with an additional .stripped suffix.  The suffix can be
 overridden using KERNEL_STRIP_SUFFIX.

 Since the toolchain does not give indication when the specified sections
 are absent, we read the sections first and make this report by issuing a
 warning to the developer.

 The toolchain by default strips the image with the -s option (even
 when -s is not specified):
 -s --strip-all   Remove all symbol and relocation information

 For example, these sections are always removed:
 .debug_aranges
 .debug_info
 .debug_abbrev
 .debug_line
 .debug_frame
 .debug_str
 .debug_loc
 .debug_ranges
 .symtab
 .strtab

 In addition to these, the sections listed in
 KERNEL_IMAGE_STRIP_EXTRA_SECTIONS will also be removed.

 Only stripping of vmlinux (elf) is supported at this time.  A warning
 will be given if the image type is not vmlinux.

 Stripping the image could also be done in the kernel, but that would
 only work for linux-yocto based kernels, so it's not the route we
 decided to go.

 [YOCTO 3515]

 Signed-off-by: Bruce Ashfield bruce.ashfi...@windriver.com
 Signed-off-by: Michel Thebeau michel.theb...@windriver.com
 
 Can we please just have one output kernel, not two. Is the unstripped
 version useful anywhere?
 


The unstripped image is bootable, and it is conceivable that someone may
even want to do load -m tftp from the boot script.

But, if a single image is desirable then I'd go with the image stripped
in place.  Here is that other patch...   I'll make sure to add text to
the log so it is clear about what happened to the image.

M


 Cheers,
 
 Richard
 
 
 

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


[OE-core] [PATCH 2/2] kernel.bbclass: do_strip: allow recipes to strip the kernel

2013-04-08 Thread michel . thebeau
From: Michel Thebeau michel.theb...@windriver.com

Allow recipes to specify sections to be stripped from the kernel output
using KERNEL_IMAGE_STRIP.  For example:

KERNEL_IMAGE_STRIP = .comment .unwanted

The kernel output is stripped in place.

Since the toolchain does not give indication when the specified sections
are absent, we read the sections first and make this report by issuing a
warning to the developer.

The toolchain by default strips the image with the -s option (even
when -s is not specified):
-s --strip-all   Remove all symbol and relocation information

For example, these sections are always removed:
.debug_aranges
.debug_info
.debug_abbrev
.debug_line
.debug_frame
.debug_str
.debug_loc
.debug_ranges
.symtab
.strtab

In addition to these, the sections listed in KERNEL_IMAGE_STRIP will
also be removed.

Only stripping of vmlinux (elf) is supported at this time.  A warning
will be given if the image type is not vmlinux.

Stripping the image could also be done in the kernel, but that would
only work for linux-yocto based kernels, so it's not the route we
decided to go.

[YOCTO 3515]

Signed-off-by: Michel Thebeau michel.theb...@windriver.com
---
 meta/classes/kernel.bbclass |   30 --
 1 file changed, 28 insertions(+), 2 deletions(-)

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index af58887..4c2c9b9 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -88,7 +88,7 @@ do_compile_kernelmodules() {
bbnote no modules to compile
fi
 }
-addtask compile_kernelmodules after do_compile before do_install
+addtask compile_kernelmodules after do_compile before do_strip
 
 kernel_do_install() {
#
@@ -289,6 +289,32 @@ python split_kernel_packages () {
 do_split_packages(d, root='/lib/firmware', file_regex='^(.*)\.cis$', 
output_pattern='kernel-firmware-%s', description='Firmware for %s', 
recursive=True, extra_depends='')
 }
 
+do_strip() {
+   if [ -n ${KERNEL_IMAGE_STRIP} ]; then
+   if [[ ${KERNEL_IMAGETYPE} != vmlinux ]]; then
+   bbwarn image type will not be stripped (not 
supported): ${KERNEL_IMAGETYPE}
+   return
+   fi
+
+   cd ${B}
+   headers=`$CROSS_COMPILEreadelf -S ${KERNEL_OUTPUT} | \
+ grep ^ \{1,\}\[[0-9 ]\{1,\}\] [^ ] | \
+ sed s/^ \{1,\}\[[0-9 ]\{1,\}\] // | \
+ gawk '{print $1}'`
+
+   for str in ${KERNEL_IMAGE_STRIP}; do {
+   if [[ $headers != *$str* ]]; then
+   bbwarn Section not found: $str;
+   fi
+
+   $CROSS_COMPILEstrip -s -R $str ${KERNEL_OUTPUT}
+   }; done
+   fi;
+}
+do_strip[dirs] = ${B}
+
+addtask do_strip before do_sizecheck after do_kernel_link_vmlinux
+
 # Support checking the kernel size since some kernels need to reside in 
partitions
 # with a fixed length or there is a limit in transferring the kernel to memory
 do_sizecheck() {
@@ -302,7 +328,7 @@ do_sizecheck() {
 }
 do_sizecheck[dirs] = ${B}
 
-addtask sizecheck before do_install after do_kernel_link_vmlinux
+addtask sizecheck before do_install after do_strip
 
 KERNEL_IMAGE_BASE_NAME ?= 
${KERNEL_IMAGETYPE}-${PE}-${PV}-${PR}-${MACHINE}-${DATETIME}
 # Don't include the DATETIME variable in the sstate package signatures
-- 
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 2/2] kernel.bbclass: do_strip: allow recipes to strip the kernel

2013-04-08 Thread Michel Thebeau


On 13-04-08 05:24 PM, michel.theb...@windriver.com wrote:
 From: Michel Thebeau michel.theb...@windriver.com
 
 Allow recipes to specify sections to be stripped from the kernel output
 using KERNEL_IMAGE_STRIP.  For example:
 
 KERNEL_IMAGE_STRIP = .comment .unwanted

s/KERNEL_IMAGE_STRIP/KERNEL_IMAGE_STRIP_EXTRA_SECTIONS

Throughout

 
 The kernel output is stripped in place.

The subject was supposed to say v2


 
 Since the toolchain does not give indication when the specified sections
 are absent, we read the sections first and make this report by issuing a
 warning to the developer.
 
 The toolchain by default strips the image with the -s option (even
 when -s is not specified):
 -s --strip-all   Remove all symbol and relocation information
 
 For example, these sections are always removed:
 .debug_aranges
 .debug_info
 .debug_abbrev
 .debug_line
 .debug_frame
 .debug_str
 .debug_loc
 .debug_ranges
 .symtab
 .strtab
 
 In addition to these, the sections listed in KERNEL_IMAGE_STRIP will
 also be removed.
 
 Only stripping of vmlinux (elf) is supported at this time.  A warning
 will be given if the image type is not vmlinux.
 
 Stripping the image could also be done in the kernel, but that would
 only work for linux-yocto based kernels, so it's not the route we
 decided to go.
 
 [YOCTO 3515]
 
 Signed-off-by: Michel Thebeau michel.theb...@windriver.com
 ---
  meta/classes/kernel.bbclass |   30 --
  1 file changed, 28 insertions(+), 2 deletions(-)
 
 diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
 index af58887..4c2c9b9 100644
 --- a/meta/classes/kernel.bbclass
 +++ b/meta/classes/kernel.bbclass
 @@ -88,7 +88,7 @@ do_compile_kernelmodules() {
   bbnote no modules to compile
   fi
  }
 -addtask compile_kernelmodules after do_compile before do_install
 +addtask compile_kernelmodules after do_compile before do_strip
  
  kernel_do_install() {
   #
 @@ -289,6 +289,32 @@ python split_kernel_packages () {
  do_split_packages(d, root='/lib/firmware', file_regex='^(.*)\.cis$', 
 output_pattern='kernel-firmware-%s', description='Firmware for %s', 
 recursive=True, extra_depends='')
  }
  
 +do_strip() {
 + if [ -n ${KERNEL_IMAGE_STRIP} ]; then
 + if [[ ${KERNEL_IMAGETYPE} != vmlinux ]]; then
 + bbwarn image type will not be stripped (not 
 supported): ${KERNEL_IMAGETYPE}
 + return
 + fi
 +
 + cd ${B}
 + headers=`$CROSS_COMPILEreadelf -S ${KERNEL_OUTPUT} | \
 +   grep ^ \{1,\}\[[0-9 ]\{1,\}\] [^ ] | \
 +   sed s/^ \{1,\}\[[0-9 ]\{1,\}\] // | \
 +   gawk '{print $1}'`
 +
 + for str in ${KERNEL_IMAGE_STRIP}; do {
 + if [[ $headers != *$str* ]]; then
 + bbwarn Section not found: $str;
 + fi
 +
 + $CROSS_COMPILEstrip -s -R $str ${KERNEL_OUTPUT}
 + }; done

And I'll add a comment here

bbnote KERNEL_IMAGE_STRIP_EXTRA_SECTIONS is set, stripping sections: \
 ${KERNEL_IMAGE_STRIP_EXTRA_SECTIONS}

Michel

 + fi;
 +}
 +do_strip[dirs] = ${B}
 +
 +addtask do_strip before do_sizecheck after do_kernel_link_vmlinux
 +
  # Support checking the kernel size since some kernels need to reside in 
 partitions
  # with a fixed length or there is a limit in transferring the kernel to 
 memory
  do_sizecheck() {
 @@ -302,7 +328,7 @@ do_sizecheck() {
  }
  do_sizecheck[dirs] = ${B}
  
 -addtask sizecheck before do_install after do_kernel_link_vmlinux
 +addtask sizecheck before do_install after do_strip
  
  KERNEL_IMAGE_BASE_NAME ?= 
 ${KERNEL_IMAGETYPE}-${PE}-${PV}-${PR}-${MACHINE}-${DATETIME}
  # Don't include the DATETIME variable in the sstate package signatures
 

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


Re: [OE-core] [PATCH v3] base.bbclass: Fix matching of MACHINEOVERRIDES in COMPATIBLE_MACHINE

2013-04-08 Thread Otavio Salvador
On Mon, Apr 8, 2013 at 5:58 PM, Otavio Salvador ota...@ossystems.com.br wrote:
 When a MACHINEOVERRIDES has more than one value, split by ':' as usual
 OVERRIDES, this were not being properly checked in COMPATIBLE_MACHINE
 matching as we need to iterate over each SoC family and check if it is
 compatible or not.

 Signed-off-by: Otavio Salvador ota...@ossystems.com.br
...
 +compat_machines.extend((d.getVar('MACHINEOVERRIDES', True) or 
 ).split(:))
...

I did run it and it fails. I am fixing it and will send a new  version
well tested.

--
Otavio Salvador O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854  http://projetos.ossystems.com.br

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


Re: [OE-core] [PATCH v3] base.bbclass: Fix matching of MACHINEOVERRIDES in COMPATIBLE_MACHINE

2013-04-08 Thread Richard Purdie
On Mon, 2013-04-08 at 17:58 -0300, Otavio Salvador wrote:
 When a MACHINEOVERRIDES has more than one value, split by ':' as usual
 OVERRIDES, this were not being properly checked in COMPATIBLE_MACHINE
 matching as we need to iterate over each SoC family and check if it is
 compatible or not.
 
 Signed-off-by: Otavio Salvador ota...@ossystems.com.br
 ---
 Changes for v3:
 - Stop checking for SOC_FAMILY as it is just for compatibility with
   old BSPs; we move to MACHINEOVERRIDES for this case (RP)
 
  meta/classes/base.bbclass | 11 ++-
  1 file changed, 6 insertions(+), 5 deletions(-)
 
 diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
 index abd6a52..313359c 100644
 --- a/meta/classes/base.bbclass
 +++ b/meta/classes/base.bbclass
 @@ -515,11 +515,12 @@ python () {
  need_machine = d.getVar('COMPATIBLE_MACHINE', True)
  if need_machine:
  import re
 -this_machine = d.getVar('MACHINE', True)
 -if this_machine and not re.match(need_machine, this_machine):
 -this_soc_family = d.getVar('SOC_FAMILY', True)
 -if (this_soc_family and not re.match(need_machine, 
 this_soc_family)) or not this_soc_family:
 -raise bb.parse.SkipPackage(incompatible with machine %s 
 (not in COMPATIBLE_MACHINE) % this_machine)
 +compat_machines.extend((d.getVar('MACHINEOVERRIDES', True) or 
 ).split(:))

No variable compat_machines exists before this line so this patch
cannot possibly work. What did you test?

We're close to release and I'm getting pushed hard in places like IRC to
take patches like this which clearly don't work :(

I'm not happy.

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 v3] base.bbclass: Fix matching of MACHINEOVERRIDES in COMPATIBLE_MACHINE

2013-04-08 Thread Otavio Salvador
On Mon, Apr 8, 2013 at 6:31 PM, Richard Purdie
richard.pur...@linuxfoundation.org wrote:
 On Mon, 2013-04-08 at 17:58 -0300, Otavio Salvador wrote:
 When a MACHINEOVERRIDES has more than one value, split by ':' as usual
 OVERRIDES, this were not being properly checked in COMPATIBLE_MACHINE
 matching as we need to iterate over each SoC family and check if it is
 compatible or not.

 Signed-off-by: Otavio Salvador ota...@ossystems.com.br
 ---
 Changes for v3:
 - Stop checking for SOC_FAMILY as it is just for compatibility with
   old BSPs; we move to MACHINEOVERRIDES for this case (RP)

  meta/classes/base.bbclass | 11 ++-
  1 file changed, 6 insertions(+), 5 deletions(-)

 diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
 index abd6a52..313359c 100644
 --- a/meta/classes/base.bbclass
 +++ b/meta/classes/base.bbclass
 @@ -515,11 +515,12 @@ python () {
  need_machine = d.getVar('COMPATIBLE_MACHINE', True)
  if need_machine:
  import re
 -this_machine = d.getVar('MACHINE', True)
 -if this_machine and not re.match(need_machine, this_machine):
 -this_soc_family = d.getVar('SOC_FAMILY', True)
 -if (this_soc_family and not re.match(need_machine, 
 this_soc_family)) or not this_soc_family:
 -raise bb.parse.SkipPackage(incompatible with machine 
 %s (not in COMPATIBLE_MACHINE) % this_machine)
 +compat_machines.extend((d.getVar('MACHINEOVERRIDES', True) or 
 ).split(:))

 No variable compat_machines exists before this line so this patch
 cannot possibly work. What did you test?

I got it when testing, hence my previous e-mail saying I found a problem.

--
Otavio Salvador O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854  http://projetos.ossystems.com.br

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


[OE-core] [PATCH v4] base.bbclass: Fix matching of MACHINEOVERRIDES in COMPATIBLE_MACHINE

2013-04-08 Thread Otavio Salvador
When a MACHINEOVERRIDES has more than one value, split by ':' as usual
OVERRIDES, this were not being properly checked in COMPATIBLE_MACHINE
matching as we need to iterate over each SoC family and check if it is
compatible or not.

Signed-off-by: Otavio Salvador ota...@ossystems.com.br
---
Changes for v4:
- Fix wrong assignment of compat_machines

 meta/classes/base.bbclass | 11 ++-
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index abd6a52..641316d 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -515,11 +515,12 @@ python () {
 need_machine = d.getVar('COMPATIBLE_MACHINE', True)
 if need_machine:
 import re
-this_machine = d.getVar('MACHINE', True)
-if this_machine and not re.match(need_machine, this_machine):
-this_soc_family = d.getVar('SOC_FAMILY', True)
-if (this_soc_family and not re.match(need_machine, 
this_soc_family)) or not this_soc_family:
-raise bb.parse.SkipPackage(incompatible with machine %s 
(not in COMPATIBLE_MACHINE) % this_machine)
+compat_machines = (d.getVar('MACHINEOVERRIDES', True) or 
).split(:)
+for m in compat_machines:
+if re.match(need_machine, m):
+break
+else:
+raise bb.parse.SkipPackage(incompatible with machine %s (not 
in COMPATIBLE_MACHINE) % d.getVar('MACHINE', True))
 
 
 bad_licenses = (d.getVar('INCOMPATIBLE_LICENSE', True) or ).split()
-- 
1.8.1


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


Re: [OE-core] [PATCH 2/2] kernel.bbclass: do_strip: allow recipes to strip the kernel

2013-04-08 Thread Richard Purdie
On Mon, 2013-04-08 at 17:28 -0400, Michel Thebeau wrote:
 
 On 13-04-08 05:24 PM, michel.theb...@windriver.com wrote:
  From: Michel Thebeau michel.theb...@windriver.com
  
  Allow recipes to specify sections to be stripped from the kernel output
  using KERNEL_IMAGE_STRIP.  For example:
  
  KERNEL_IMAGE_STRIP = .comment .unwanted
 
 s/KERNEL_IMAGE_STRIP/KERNEL_IMAGE_STRIP_EXTRA_SECTIONS
 
 Throughout

So are you resending this with the appropriate changes, tested? It looks
much better this way to me than the first version anyway, thanks.

Richard




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


Re: [OE-core] [PATCH 2/3] oe-buildenv-internal: Only add to $PATH if needed

2013-04-08 Thread Chris Larson
On Mon, Apr 8, 2013 at 10:48 AM, Paul Eggleton 
paul.eggle...@linux.intel.com wrote:

 On Monday 08 April 2013 13:31:50 Trevor Woerner wrote:
  On Fri, Apr 5, 2013 at 12:59 PM, Peter Kjellerstedt
  peter.kjellerst...@axis.com wrote:
   +[ ${PATH#$NEWPATHS} != $PATH ] || PATH=$NEWPATHS$PATH
 
  This is certainly a welcome addition in functionality, but it relies
  on the pattern remaining at the start of the PATH (i.e. the user
  hasn't played with PATH in any way). Could we not use the
  ${parameter/pattern/string} parameter expansion instead (e.g.
  ${PATH/$NEWPATHS/}) so it doesn't matter whether the user has
  further modified the PATH?

 Unfortunately I think this is specific to bash, so it may not be portable.
 Maybe the equivalent can be achieved with sed however.


Also, neither version matches the : separator, which means we could in
theory get false positive matches.
-- 
Christopher Larson
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] RFC: gstreamer 1.0 recipes

2013-04-08 Thread Carlos Rafael Giani

Hello,

thinking about this whole issue I began to wonder: why is gstreamer in 
oe-core and not in meta-multimedia?

wouldnt it be easier then to move liborc into meta-multimedia as well?
Or is gstreamer considered an essential component for many 
installations, thus justifying its presence in oe-core?


regards,
carlos

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


Re: [OE-core] [PATCH 1/2] libpng12: rename libpng_1.2.50 to libpng12

2013-04-08 Thread Kang Kai

On 2013年04月08日 23:01, Mark Hatle wrote:

On 4/8/13 4:12 AM, Kang Kai wrote:

As Mark's suggestion, rename libpng_1.2.50 to libpng12 that
multi-versions libpng could coexist. And drop files that conflict with
higher version.

We want to make sure we have both the old and new versions to meet LSB
compliance (for people who have that enabled) as well as the new version
for newer applications.

CC: Mark Hatle mark.ha...@windriver.com

Signed-off-by: Kang Kai kai.k...@windriver.com
---
.../{libpng_1.2.50.bb = libpng12_1.2.50.bb} | 18 +++---
1 files changed, 15 insertions(+), 3 deletions(-)
rename meta/recipes-lsb4/libpng/{libpng_1.2.50.bb = 
libpng12_1.2.50.bb} (62%)


diff --git a/meta/recipes-lsb4/libpng/libpng_1.2.50.bb 
b/meta/recipes-lsb4/libpng/libpng12_1.2.50.bb

similarity index 62%
rename from meta/recipes-lsb4/libpng/libpng_1.2.50.bb
rename to meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
index 8fdc41b..43ff75a 100644
--- a/meta/recipes-lsb4/libpng/libpng_1.2.50.bb
+++ b/meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
@@ -8,14 +8,26 @@ LIC_FILES_CHKSUM = 
file://LICENSE;md5=c3d807a85c09ebdff087f18b4969ff96 \

DEPENDS = zlib
PR = r0

+PN = libpng12
+S = ${WORKDIR}/libpng-${PV}
+
SRC_URI = 
${SOURCEFORGE_MIRROR}/project/libpng/libpng12/${PV}/libpng-${PV}.tar.xz


SRC_URI[md5sum] = a3e00fccbfe356174ab515b5c00641c7
SRC_URI[sha256sum] = 
4724f81f8c92ac7f360ad1fbf173396ea7c535923424db9fbaff07bfd9d8e8e7


+BINCONFIG_GLOB = ${PN}-config
+
inherit autotools binconfig pkgconfig

-PACKAGES =+ ${PN}12
+do_install_append() {
+ unlink ${D}/${includedir}/png.h
+ unlink ${D}/${includedir}/pngconf.h




Hi Mark,

You should move those two into a subdirectory called 
${D}/${includedir}/libpng12/


That way they will still be available for anyone who needs them.

Files
/usr/include/libpng12/png.h
/usr/include/libpng12/pngconf.h

already exist. Those two files unlinked link to them and conflict with 
libpng16.





+
+ unlink ${D}/${libdir}/libpng.la
+ unlink ${D}/${libdir}/libpng.so
+ unlink ${D}/${libdir}/libpng.a


The .la, .a seen above should be similarly renamed to be libpng12... 
The .so should be dropped, as anyone who needs the alternative version 
would need to directly specify it.


Same reason. They are links to libpng12.*.




+ unlink ${D}/${libdir}/pkgconfig/libpng.pc


Should be renamed to be libpng12.pc.

(Note both the .la and .pc files will likely need to be modified to 
reference the correct header and library paths.)


libpng12.pc exists too, and the patch I checked that are right.




-FILES_${PN}12 = ${libdir}/libpng12${SOLIBS}
-RPROVIDES_${PN}-dev += ${PN}12-dev
+ unlink ${D}/${bindir}/libpng-config
+}


The above should be renamed to be libpng12-config.

Link file too.

(Again, make sure that the results of it match the renamed .a/.so and 
include paths.)


--Mark



Thanks for detailed review, I'll add some comment to make it clear.

--
Regards,
Neil | Kai Kang


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


[OE-core] [PATCH 0/2] V2: rename libpng_1.2.50 to libpng12

2013-04-08 Thread Kang Kai
V2: add comments to explain why drop some link files

The following changes since commit e57284abca76fe7e6c29484104ae4349459c63dc:

  kernel.bbclass: do_sizecheck: update path to build image and do not delete 
(2013-04-08 22:26:24 +0100)

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

Kang Kai (2):
  libpng12: rename libpng_1.2.50 to libpng12
  libpng12: remove prefer version and add it to lsb packagegroup

 meta/conf/distro/include/default-versions.inc  |3 ---
 .../packagegroups/packagegroup-core-lsb.bb |1 +
 .../{libpng_1.2.50.bb = libpng12_1.2.50.bb}   |   20 +---
 3 files changed, 18 insertions(+), 6 deletions(-)
 rename meta/recipes-lsb4/libpng/{libpng_1.2.50.bb = libpng12_1.2.50.bb} (55%)

-- 
1.7.5.4


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


[OE-core] [PATCH 2/2] libpng12: remove prefer version and add it to lsb packagegroup

2013-04-08 Thread Kang Kai
Because rename libpng_1.2.50 to libpng, remove the perfer verion from
default-versions.inc and add libpng12 to lsb packagegroup.

Signed-off-by: Kang Kai kai.k...@windriver.com
---
 meta/conf/distro/include/default-versions.inc  |3 ---
 .../packagegroups/packagegroup-core-lsb.bb |1 +
 2 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/meta/conf/distro/include/default-versions.inc 
b/meta/conf/distro/include/default-versions.inc
index 0a5b2f4..53ec2e7 100644
--- a/meta/conf/distro/include/default-versions.inc
+++ b/meta/conf/distro/include/default-versions.inc
@@ -9,6 +9,3 @@ PREFERRED_VERSION_python-native ?= 2.7.3
 
 # Force the older version of liberation-fonts until we fix the fontforge issue
 PREFERRED_VERSION_liberation-fonts ?= 1.04
-
-# Set libpng default version for linuxstdbase
-PREFERRED_VERSION_libpng_linuxstdbase ?= 1.2.50
diff --git a/meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb 
b/meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb
index d692a26..cf04f0f 100644
--- a/meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb
+++ b/meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb
@@ -161,6 +161,7 @@ RDEPENDS_packagegroup-core-lsb-core = \
 ncurses \
 zlib \
 nspr \
+libpng12 \
 
 
 SUMMARY_packagegroup-core-lsb-perl = LSB Runtime Languages (Perl)
-- 
1.7.5.4


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


[OE-core] [PATCH 1/2] libpng12: rename libpng_1.2.50 to libpng12

2013-04-08 Thread Kang Kai
As Mark's suggestion, rename libpng_1.2.50 to libpng12 that
multi-versions libpng could coexist.

We want to make sure we have both the old and new versions to meet LSB
compliance (for people who have that enabled) as well as the new version
for newer applications.

And drop link files that conflict with higher version.

[YOCTO #4221]

Signed-off-by: Kang Kai kai.k...@windriver.com
CC: Mark Hatle mark.ha...@windriver.com
---
 .../{libpng_1.2.50.bb = libpng12_1.2.50.bb}   |   20 +---
 1 files changed, 17 insertions(+), 3 deletions(-)
 rename meta/recipes-lsb4/libpng/{libpng_1.2.50.bb = libpng12_1.2.50.bb} (55%)

diff --git a/meta/recipes-lsb4/libpng/libpng_1.2.50.bb 
b/meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
similarity index 55%
rename from meta/recipes-lsb4/libpng/libpng_1.2.50.bb
rename to meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
index 8fdc41b..cfefd41 100644
--- a/meta/recipes-lsb4/libpng/libpng_1.2.50.bb
+++ b/meta/recipes-lsb4/libpng/libpng12_1.2.50.bb
@@ -8,14 +8,28 @@ LIC_FILES_CHKSUM = 
file://LICENSE;md5=c3d807a85c09ebdff087f18b4969ff96 \
 DEPENDS = zlib
 PR = r0
 
+PN = libpng12
+S = ${WORKDIR}/libpng-${PV}
+
 SRC_URI = 
${SOURCEFORGE_MIRROR}/project/libpng/libpng12/${PV}/libpng-${PV}.tar.xz
 
 SRC_URI[md5sum] = a3e00fccbfe356174ab515b5c00641c7
 SRC_URI[sha256sum] = 
4724f81f8c92ac7f360ad1fbf173396ea7c535923424db9fbaff07bfd9d8e8e7
 
+BINCONFIG_GLOB = ${PN}-config
+
 inherit autotools binconfig pkgconfig
 
-PACKAGES =+ ${PN}12
+do_install_append() {
+   # The follow link files link to corresponding png12*.h and libpng12* 
files
+   # They conflict with higher verison, so drop them
+   unlink ${D}/${includedir}/png.h
+   unlink ${D}/${includedir}/pngconf.h
+
+   unlink ${D}/${libdir}/libpng.la
+   unlink ${D}/${libdir}/libpng.so
+   unlink ${D}/${libdir}/libpng.a
+   unlink ${D}/${libdir}/pkgconfig/libpng.pc
 
-FILES_${PN}12 = ${libdir}/libpng12${SOLIBS}
-RPROVIDES_${PN}-dev += ${PN}12-dev
+   unlink ${D}/${bindir}/libpng-config
+}
-- 
1.7.5.4


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