Openembedded Core Discussions
 help / color / mirror / Atom feed
* Re: [PATCH 1/1] bash: make job control really work
From: Cui, Dexuan @ 2011-11-02  7:10 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <1320142868.13918.0.camel@ted>

Richard Purdie wrote on 2011-11-01:
> On Tue, 2011-11-01 at 16:05 +0800, Dexuan Cui wrote:
>> It turns out 9393ff833f44570fd5f500bc9de6c72db94b0296 didn't really
>> fix the bug.
>> 
>> This patch is made and tested after I read the link below
>> http://www.mail-archive.com/bug-bash@gnu.org/msg03107.html
>> 
>> [YOCTO #487]
>> 
>> Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>
>> ---
>>  meta/recipes-extended/bash/bash.inc    |    1 +
>>  meta/recipes-extended/bash/bash_4.2.bb |    2 +-
>>  2 files changed, 2 insertions(+), 1 deletions(-)
>> diff --git a/meta/recipes-extended/bash/bash.inc
>> b/meta/recipes-extended/bash/bash.inc
>> index d55e517..d495538 100644
>> --- a/meta/recipes-extended/bash/bash.inc
>> +++ b/meta/recipes-extended/bash/bash.inc
>> @@ -23,6 +23,7 @@ ALTERNATIVE_LINK = "${base_bindir}/sh"
>>  ALTERNATIVE_PRIORITY = "100"
>>  
>>  do_configure () { +	export bash_cv_job_control_missing=present
>>  	gnu-configize 	oe_runconf }
> 
> This really should go into the common site files...
Hi Richard,
I found we do define the variable:
meta/site/common-linux:33:bash_cv_job_control_missing=${bash_cv_job_control_missing=present}
but looks autoconf doesn't realize the variable has been assigned the value 'present'?
I think this is because of the below do_configure in bash.inc -- looks autoreconf is skipped?
do_configure () {
    gnu-configize
    oe_runconf
}
Why do we need a customized do_configure to replace autotools_do_configure?

Later, after I added
do_configure_prepend () {
    autoreconf -f -i -s
}
The generated config.log does show bash_cv_job_control_missing is assigned with 'present'.
(BTW, common-linux also introduces many other variables -- would this be safe? Actually here I only need to introduce bash_cv_job_control_missing.)

However, finally, do_compile got a strange failure:
| shell.c: In function 'shell_reinitialize':
| shell.c:1742:20: error: 'PPROMPT' undeclared (first use in this function)
| shell.c:1742:20: note: each undeclared identifier is reported only once for each function it appears in
| shell.c:1743:22: error: 'SPROMPT' undeclared (first use in this function)

Could you please give some suggestions?

Thanks,
-- Dexuan





^ permalink raw reply

* [PATCH] libxslt: use Copyright in LIC_FILES_CHKSUM instead of COPYING
From: Martin Jansa @ 2011-11-02  7:42 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <4EB08D70.4050602@intel.com>

* COPYING is replaced by symlink to Copyright during do_configure
  (see configure.in), then we end with link to nonexistent file like this:

OE om-gta02@shr ~/shr-core $ ll tmp/deploy/licenses/libxslt/
total 40
drwxr-xr-x   2 bitbake bitbake  4096 Nov  2 00:27 ./
drwxr-xr-x 818 bitbake bitbake 32768 Nov  2 00:27 ../
lrwxrwxrwx   1 bitbake bitbake     9 Nov  2 00:27 COPYING ->  Copyright
lrwxrwxrwx   1 bitbake bitbake    52 Nov  2 00:27 generic_MIT ->  /OE/shr-core/tmp/deploy/licenses/common-licenses/MIT

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-support/libxslt/libxslt_1.1.26.bb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-support/libxslt/libxslt_1.1.26.bb b/meta/recipes-support/libxslt/libxslt_1.1.26.bb
index 96bffc6..addc863 100644
--- a/meta/recipes-support/libxslt/libxslt_1.1.26.bb
+++ b/meta/recipes-support/libxslt/libxslt_1.1.26.bb
@@ -3,11 +3,11 @@ HOMEPAGE = "http://xmlsoft.org/XSLT/"
 BUGTRACKER = "https://bugzilla.gnome.org/"
 
 LICENSE = "MIT"
-LIC_FILES_CHKSUM = "file://COPYING;md5=0cd9a07afbeb24026c9b03aecfeba458"
+LIC_FILES_CHKSUM = "file://Copyright;md5=0cd9a07afbeb24026c9b03aecfeba458"
 
 SECTION = "libs"
 DEPENDS = "libxml2"
-PR = "r3"
+PR = "r4"
 
 SRC_URI = "ftp://xmlsoft.org/libxslt//libxslt-${PV}.tar.gz \
            file://pkgconfig_fix.patch"
-- 
1.7.7.1




^ permalink raw reply related

* [PATCH] udev-164: Update init script to do an explicit add action
From: Kumar Gala @ 2011-11-02  7:23 UTC (permalink / raw)
  To: openembedded-core

With udev 152 or greater the default action for 'udevadm trigger' was
modified to be 'change' instead of 'add.

To ensure initial coldplug events at boot are seen be scripts the are
expecting them as 'add' events we invoke udevadm with an explicit
'--action=add'.

Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
---
 meta/recipes-core/udev/udev-164/init |    4 ++--
 meta/recipes-core/udev/udev_164.bb   |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-core/udev/udev-164/init b/meta/recipes-core/udev/udev-164/init
index 9ce95ee..073942f 100644
--- a/meta/recipes-core/udev/udev-164/init
+++ b/meta/recipes-core/udev/udev-164/init
@@ -48,10 +48,10 @@ kill_udevd > "/dev/null" 2>&1
 
         /sbin/udevadm control --env=STARTUP=1
 		if [ "$not_first_boot" != "" ];then
-			/sbin/udevadm trigger --subsystem-nomatch=tty --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole --subsystem-nomatch=misc --subsystem-nomatch=dcon --subsystem-nomatch=pci_bus  --subsystem-nomatch=graphics  --subsystem-nomatch=backlight --subsystem-nomatch=video4linux  --subsystem-nomatch=platform
+			/sbin/udevadm trigger --action=add --subsystem-nomatch=tty --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole --subsystem-nomatch=misc --subsystem-nomatch=dcon --subsystem-nomatch=pci_bus  --subsystem-nomatch=graphics  --subsystem-nomatch=backlight --subsystem-nomatch=video4linux  --subsystem-nomatch=platform
 			(/sbin/udevadm settle --timeout=3; /sbin/udevadm control --env=STARTUP=)&
 		else
-			/sbin/udevadm trigger
+			/sbin/udevadm trigger --action=add
 			/sbin/udevadm settle
 		fi
 
diff --git a/meta/recipes-core/udev/udev_164.bb b/meta/recipes-core/udev/udev_164.bb
index 7cbe4d8..d487add 100644
--- a/meta/recipes-core/udev/udev_164.bb
+++ b/meta/recipes-core/udev/udev_164.bb
@@ -1,6 +1,6 @@
 include udev-new.inc
 
-PR = "r6"
+PR = "r7"
 
 SRC_URI += "file://udev-166-v4l1-1.patch"
 
-- 
1.7.3.4




^ permalink raw reply related

* [PATCH] lame: add SRC_URI checksums
From: Martin Jansa @ 2011-11-02  8:11 UTC (permalink / raw)
  To: openembedded-core

---
 meta/recipes-multimedia/lame/lame_3.98.4.bb |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-multimedia/lame/lame_3.98.4.bb b/meta/recipes-multimedia/lame/lame_3.98.4.bb
index 6e098c6..680ea48 100644
--- a/meta/recipes-multimedia/lame/lame_3.98.4.bb
+++ b/meta/recipes-multimedia/lame/lame_3.98.4.bb
@@ -10,6 +10,9 @@ PR = "r0"
 SRC_URI = "${SOURCEFORGE_MIRROR}/lame/lame-${PV}.tar.gz \
            file://no-gtk1.patch"
 
+SRC_URI[md5sum] = "8e9866ad6b570c6c95c8cba48060473f"
+SRC_URI[sha256sum] = "ac3144c76617223a9be4aaa3e28a66b51bcab28141050c3af04cb06836f772c8"
+
 inherit autotools
 
 PACKAGES += "libmp3lame libmp3lame-dev"
-- 
1.7.7.1




^ permalink raw reply related

* Re: [Pull v3 0/4] Add User Fixes
From: Richard Purdie @ 2011-11-02  8:21 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <cover.1320191934.git.sgw@linux.intel.com>

On Tue, 2011-11-01 at 17:21 -0700, Saul Wold wrote:
> Richard,
> 
> Hope that third times the charm!
> 
> Fixes 1699, additional commit comments, fixed whitespace.
> 
> Sau!
> 
> 
> The following changes since commit d0a2c87d068f2d769d47c8455bb1434b90a7a844:
> 
>   Fix HOMEPAGE values in libzypp and sat-solver .bb files (2011-11-01 18:28:06 +0000)
> 
> are available in the git repository at:
>   git://git.openembedded.org/openembedded-core-contrib sgw/fix
>   http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgw/fix
> 
> Saul Wold (4):
>   avahi: use useradd to create avahi user for avahi-daemon
>   xserver-nodm-init: Use useradd to add the xuser for rootless X
>   connman: Use useradd to add the xuser for DBus
>   dbus: ensure that the useradd shell is set to /bin/false

Thanks for the tweaks, merged to master.

Cheers,

Richard




^ permalink raw reply

* Re: [PATCH] lame: add SRC_URI checksums
From: Richard Purdie @ 2011-11-02  8:22 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <1320221478-9707-1-git-send-email-Martin.Jansa@gmail.com>

On Wed, 2011-11-02 at 09:11 +0100, Martin Jansa wrote:
> ---
>  meta/recipes-multimedia/lame/lame_3.98.4.bb |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)

Merged to master, thanks.

Richard




^ permalink raw reply

* Re: [PATCH] libxslt: use Copyright in LIC_FILES_CHKSUM instead of COPYING
From: Richard Purdie @ 2011-11-02  8:23 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <1320219727-9233-1-git-send-email-Martin.Jansa@gmail.com>

On Wed, 2011-11-02 at 08:42 +0100, Martin Jansa wrote:
> * COPYING is replaced by symlink to Copyright during do_configure
>   (see configure.in), then we end with link to nonexistent file like this:
> 
> OE om-gta02@shr ~/shr-core $ ll tmp/deploy/licenses/libxslt/
> total 40
> drwxr-xr-x   2 bitbake bitbake  4096 Nov  2 00:27 ./
> drwxr-xr-x 818 bitbake bitbake 32768 Nov  2 00:27 ../
> lrwxrwxrwx   1 bitbake bitbake     9 Nov  2 00:27 COPYING ->  Copyright
> lrwxrwxrwx   1 bitbake bitbake    52 Nov  2 00:27 generic_MIT ->  /OE/shr-core/tmp/deploy/licenses/common-licenses/MIT
> 
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>

Merged to master, thanks.

Richard




^ permalink raw reply

* Re: [Pull v2 2/4] xserver-nodm-init: Add xuser (hardcoded)
From: Richard Purdie @ 2011-11-02  8:23 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <4EB0DF26.1000200@bluegiga.com>

On Wed, 2011-11-02 at 08:11 +0200, Lauri Hintsala wrote:
> Hi Saul,
> 
> On 11/01/2011 11:44 PM, Saul Wold wrote:
> > Signed-off-by: Saul Wold<sgw@linux.intel.com>
> > ---
> >   .../x11-common/xserver-nodm-init.bb                |   30 +++++++------------
> >   1 files changed, 11 insertions(+), 19 deletions(-)
> >
> > diff --git a/meta/recipes-graphics/x11-common/xserver-nodm-init.bb b/meta/recipes-graphics/x11-common/xserver-nodm-init.bb
> > index ea4222d..5b06bc6 100644
> > --- a/meta/recipes-graphics/x11-common/xserver-nodm-init.bb
> > +++ b/meta/recipes-graphics/x11-common/xserver-nodm-init.bb
> > @@ -2,7 +2,7 @@ DESCRIPTION = "Simple Xserver Init Script (no dm)"
> >   LICENSE = "GPLv2"
> >   LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
> >   SECTION = "x11"
> > -PR = "r26"
> > +PR = "r28"
> >   RDEPENDS_${PN} = "sudo"
> >
> >   SRC_URI = "file://xserver-nodm \
> > @@ -19,27 +19,19 @@ do_install() {
> >       install xserver-nodm ${D}/etc/init.d
> >       if [ "${ROOTLESS_X}" = "1" ] ; then
> >           install -d ${D}/etc/X11
> > -        install Xusername ${D}/etc/X11
> > +	install Xusername ${D}/etc/X11
> 
> Is this indentation change a typo?

Saul fixed this in the v3 which I've taken.

Cheers,

Richard




^ permalink raw reply

* Re: [PATCH] meta: glib-2.0: don't apply qsort_r test removable patch for native version
From: Martin Jansa @ 2011-11-02  8:26 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <1320185290-18117-1-git-send-email-morphis@gravedo.de>

[-- Attachment #1: Type: text/plain, Size: 2146 bytes --]

On Tue, Nov 01, 2011 at 11:08:10PM +0100, Simon Busch wrote:
> On some buildhosts with an older version of native glib-2.0 installed (in this case
> 2.16.6) the qsort_r test removable patch leads to a compilation error:
> 
> | ./.libs/libglib-2.0.so: undefined reference to `qsort_r'
> | collect2: ld returned 1 exit status
> 
> This patch fixes this so the patch gets only applied for the native version of this
> recipe.
> 
> Signed-off-by: Simon Busch <morphis@gravedo.de>

Acked-by: Martin Jansa <Martin.Jansa@gmail.com>

> ---
>  meta/recipes-core/glib-2.0/glib-2.0_2.30.0.bb |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-core/glib-2.0/glib-2.0_2.30.0.bb b/meta/recipes-core/glib-2.0/glib-2.0_2.30.0.bb
> index 566355d..0efce40 100644
> --- a/meta/recipes-core/glib-2.0/glib-2.0_2.30.0.bb
> +++ b/meta/recipes-core/glib-2.0/glib-2.0_2.30.0.bb
> @@ -1,6 +1,6 @@
>  require glib.inc
>  
> -PR = "r2"
> +PR = "r3"
>  PE = "1"
>  
>  DEPENDS += "libffi python-argparse-native"
> @@ -9,11 +9,14 @@ DEPENDS_virtclass-nativesdk += "libffi-nativesdk python-argparse-native zlib-nat
>  
>  SHRT_VER = "${@bb.data.getVar('PV',d,1).split('.')[0]}.${@bb.data.getVar('PV',d,1).split('.')[1]}"
>  
> +QSORT_PATCH = "file://remove.test.for.qsort_r.patch"
> +QSORT_PATCH_virtclass-native = ""
> +
>  SRC_URI = "${GNOME_MIRROR}/glib/${SHRT_VER}/glib-${PV}.tar.bz2 \
>             file://configure-libtool.patch \
>             file://60_wait-longer-for-threads-to-die.patch \
>             file://g_once_init_enter.patch \
> -           file://remove.test.for.qsort_r.patch \
> +           ${QSORT_PATCH} \
>            "
>  SRC_URI[md5sum] = "fee101d9d7daa8ddfbae00325f307f3b"
>  SRC_URI[sha256sum] = "ca9c731017ab370859e847f8b70079bc6dcf389dc0ccb0d0419925aff81b9687"
> -- 
> 1.7.5.4
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply

* [PATCH 1/3] task-core-boot, keymaps: add another VIRTUAL-RUNTIME to allow distributions to use different set of initscripts or no initscripts at all
From: Martin Jansa @ 2011-11-02  8:58 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-bsp/keymaps/keymaps_1.0.bb   |    6 +++++-
 meta/recipes-core/tasks/task-core-boot.bb |    6 ++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-bsp/keymaps/keymaps_1.0.bb b/meta/recipes-bsp/keymaps/keymaps_1.0.bb
index 23a3051..4fe7987 100644
--- a/meta/recipes-bsp/keymaps/keymaps_1.0.bb
+++ b/meta/recipes-bsp/keymaps/keymaps_1.0.bb
@@ -1,7 +1,11 @@
 SUMMARY = "Keyboard maps"
 DESCRIPTION = "Keymaps and initscript to set the keymap on bootup."
 SECTION = "base"
-RDEPENDS_${PN} = "initscripts console-tools"
+
+# Distro can override initscripts provider
+VIRTUAL-RUNTIME_initscripts ?= "initscripts"
+
+RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_initscripts} console-tools"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 PACKAGE_ARCH = "${MACHINE_ARCH}"
diff --git a/meta/recipes-core/tasks/task-core-boot.bb b/meta/recipes-core/tasks/task-core-boot.bb
index 9e63ebb..05c280d 100644
--- a/meta/recipes-core/tasks/task-core-boot.bb
+++ b/meta/recipes-core/tasks/task-core-boot.bb
@@ -9,7 +9,7 @@ LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 DEPENDS = "virtual/kernel"
 ALLOW_EMPTY = "1"
-PR = "r8"
+PR = "r9"
 
 #
 # Set by the machine configuration with packages essential for device bootup
@@ -23,6 +23,8 @@ VIRTUAL-RUNTIME_dev_manager ?= "udev"
 VIRTUAL-RUNTIME_login_manager ?= "tinylogin"
 # Distro can override init_manager provider
 VIRTUAL-RUNTIME_init_manager ?= "sysvinit"
+# Distro can override initscripts provider
+VIRTUAL-RUNTIME_initscripts ?= "initscripts"
 
 PACKAGES = "\
     task-core-boot \
@@ -34,7 +36,7 @@ RDEPENDS_task-core-boot = "\
     base-files \
     base-passwd \
     busybox \
-    initscripts \
+    ${VIRTUAL-RUNTIME_initscripts} \
     ${@base_contains("MACHINE_FEATURES", "keyboard", "keymaps", "", d)} \
     modutils-initscripts \
     netbase \
-- 
1.7.7.1




^ permalink raw reply related

* [PATCH 2/3] keymaps: use VIRTUAL-RUNTIME_kbd instead of console-tools directly
From: Martin Jansa @ 2011-11-02  8:58 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <1320224305-6747-1-git-send-email-Martin.Jansa@gmail.com>

* nowadays kbd seems more active and it's available ie in meta-openembedded
* other option is to move kbd from meta-openembedded to
  openembedded-core and changed DEPENDS

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-bsp/keymaps/keymaps_1.0.bb |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-bsp/keymaps/keymaps_1.0.bb b/meta/recipes-bsp/keymaps/keymaps_1.0.bb
index 4fe7987..754bec6 100644
--- a/meta/recipes-bsp/keymaps/keymaps_1.0.bb
+++ b/meta/recipes-bsp/keymaps/keymaps_1.0.bb
@@ -5,11 +5,14 @@ SECTION = "base"
 # Distro can override initscripts provider
 VIRTUAL-RUNTIME_initscripts ?= "initscripts"
 
-RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_initscripts} console-tools"
+# Distro can override kbd provider
+VIRTUAL-RUNTIME_kbd ?= "console-tools"
+
+RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_initscripts} ${VIRTUAL-RUNTIME_kbd}"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 PACKAGE_ARCH = "${MACHINE_ARCH}"
-PR = "r19"
+PR = "r20"
 
 INHIBIT_DEFAULT_DEPS = "1"
 
-- 
1.7.7.1




^ permalink raw reply related

* [PATCH 3/3] task-core-x11: use VIRTUAL-RUNTIME variables for xserver_common and graphical_init_manager
From: Martin Jansa @ 2011-11-02  8:58 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <1320224305-6747-1-git-send-email-Martin.Jansa@gmail.com>

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-sato/tasks/task-core-x11.bb |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-sato/tasks/task-core-x11.bb b/meta/recipes-sato/tasks/task-core-x11.bb
index 26d550a..106bc0f 100644
--- a/meta/recipes-sato/tasks/task-core-x11.bb
+++ b/meta/recipes-sato/tasks/task-core-x11.bb
@@ -30,6 +30,12 @@ ALLOW_EMPTY = "1"
 FILEMANAGER ?= "pcmanfm"
 FILEMANAGER_mips ?= ""
 
+# xserver-common, x11-common
+VIRTUAL-RUNTIME_xserver_common ?= "x11-common"
+
+# elsa, xserver-nodm-init
+VIRTUAL-RUNTIME_graphical_init_manager ?= "xserver-nodm-init"
+
 
 RDEPENDS_task-core-x11-base = "\
     dbus \
@@ -42,8 +48,8 @@ RDEPENDS_task-core-x11-base = "\
     matchbox-desktop \
     matchbox-session \
     ${XSERVER} \
-    x11-common \
-    xserver-nodm-init \
+    ${VIRTUAL-RUNTIME_xserver_common} \
+    ${VIRTUAL-RUNTIME_graphical_init_manager} \
     liberation-fonts \
     xauth \
     xhost \
-- 
1.7.7.1




^ permalink raw reply related

* Re: [PATCH] meta: glib-2.0: don't apply qsort_r test removable patch for native version
From: Richard Purdie @ 2011-11-02  9:08 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <1320185290-18117-1-git-send-email-morphis@gravedo.de>

On Tue, 2011-11-01 at 23:08 +0100, Simon Busch wrote:
> On some buildhosts with an older version of native glib-2.0 installed (in this case
> 2.16.6) the qsort_r test removable patch leads to a compilation error:
> 
> | ./.libs/libglib-2.0.so: undefined reference to `qsort_r'
> | collect2: ld returned 1 exit status
> 
> This patch fixes this so the patch gets only applied for the native version of this
> recipe.
> 
> Signed-off-by: Simon Busch <morphis@gravedo.de>
> ---
>  meta/recipes-core/glib-2.0/glib-2.0_2.30.0.bb |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)

Merged to master, thanks.

Richard




^ permalink raw reply

* Re: [PATCH v3 01/16] libnl2: Fix a race on route/pktloc_syntax.h
From: Martin Jansa @ 2011-11-02  9:16 UTC (permalink / raw)
  To: McClintock Matthew-B29882,
	Patches and discussions about the oe-core layer
In-Reply-To: <CAEsOVNfOgHFMzMcvXt+DhBY0xtwNWhi--vNNDeKXC=aRsLKtdw@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1789 bytes --]

On Tue, Oct 11, 2011 at 08:57:47PM +0000, McClintock Matthew-B29882 wrote:
> On Tue, Oct 11, 2011 at 3:47 PM, Saul Wold <saul.wold@intel.com> wrote:
> > On 09/30/2011 10:21 AM, Matthew McClintock wrote:
> >>
> >> From: Tom Rini<tom_rini@mentor.com>
> >>
> >> At issue is that route/pktloc.c (not generated) depends on
> >> route/pktloc_syntax.h (generated).
> >>
> >> Signed-off-by: Tom Rini<tom_rini@mentor.com>
> >> Signed-off-by: Matthew McClintock<msm@freescale.com>
> >> ---
> >>  .../libnl/fix-pktloc_syntax_h-race.patch           |   26
> >> ++++++++++++++++++++
> >
> > I just noticed that this patch get put in the libnl directory, not the
> > libnl-2.0 directory.
> >
> > Can you pull all the libnl changes together and resbumit this, also is there
> > a reason you can not just update this to the latest libnl 3.2.1?
> 
> Are you referring to this: http://patches.openembedded.org/patch/12749/

This pach is moving fix-makefile.patch from libnl-2.0 to libnl dir as Saul mentioned 
and patch wasn't even formated with -M to make it easier to review such move, please
move it back to libnl-2.0 and send it again.

.../libnl/libnl-2.0/fix-makefile.patch             |   32 --------------------
 .../recipes-support/libnl/libnl/fix-makefile.patch |   32 ++++++++++++++++++++

Cheers,

> 
> > Cab you test that against the wpa-supplicant which seems to be libnl's sole
> > dependee.
> 
> I can't - don't have a board with wireless - dont't have a wireless network...
> 
> -M
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply

* Re: [PATCH 2/3] keymaps: use VIRTUAL-RUNTIME_kbd instead of console-tools directly
From: Koen Kooi @ 2011-11-02  9:27 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <1320224305-6747-2-git-send-email-Martin.Jansa@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 348 bytes --]


Op 2 nov. 2011, om 09:58 heeft Martin Jansa het volgende geschreven:

> * nowadays kbd seems more active and it's available ie in meta-openembedded
> * other option is to move kbd from meta-openembedded to
>  openembedded-core and changed DEPENDS

Since I really dislike VIRTUAL_RUNTIME_*, let's move kbd into OE-core.

regards,

Koen

[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 169 bytes --]

^ permalink raw reply

* Re: [PATCH 1/1] bash: make job control really work
From: Richard Purdie @ 2011-11-02 10:35 UTC (permalink / raw)
  To: Cui, Dexuan; +Cc: Patches and discussions about the oe-core layer
In-Reply-To: <1865303E0DED764181A9D882DEF65FB6D836780D9E@shsmsx502.ccr.corp.intel.com>

On Wed, 2011-11-02 at 15:10 +0800, Cui, Dexuan wrote:
> Richard Purdie wrote on 2011-11-01:
> > On Tue, 2011-11-01 at 16:05 +0800, Dexuan Cui wrote:
> >> It turns out 9393ff833f44570fd5f500bc9de6c72db94b0296 didn't really
> >> fix the bug.
> >> 
> >> This patch is made and tested after I read the link below
> >> http://www.mail-archive.com/bug-bash@gnu.org/msg03107.html
> >> 
> >> [YOCTO #487]
> >> 
> >> Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>
> >> ---
> >>  meta/recipes-extended/bash/bash.inc    |    1 +
> >>  meta/recipes-extended/bash/bash_4.2.bb |    2 +-
> >>  2 files changed, 2 insertions(+), 1 deletions(-)
> >> diff --git a/meta/recipes-extended/bash/bash.inc
> >> b/meta/recipes-extended/bash/bash.inc
> >> index d55e517..d495538 100644
> >> --- a/meta/recipes-extended/bash/bash.inc
> >> +++ b/meta/recipes-extended/bash/bash.inc
> >> @@ -23,6 +23,7 @@ ALTERNATIVE_LINK = "${base_bindir}/sh"
> >>  ALTERNATIVE_PRIORITY = "100"
> >>  
> >>  do_configure () { +	export bash_cv_job_control_missing=present
> >>  	gnu-configize 	oe_runconf }
> > 
> > This really should go into the common site files...
> Hi Richard,
> I found we do define the variable:
> meta/site/common-linux:33:bash_cv_job_control_missing=${bash_cv_job_control_missing=present}
> but looks autoconf doesn't realize the variable has been assigned the value 'present'?
> I think this is because of the below do_configure in bash.inc -- looks autoreconf is skipped?
> do_configure () {
>     gnu-configize
>     oe_runconf
> }
> Why do we need a customized do_configure to replace autotools_do_configure?
> 
> Later, after I added
> do_configure_prepend () {
>     autoreconf -f -i -s
> }
> The generated config.log does show bash_cv_job_control_missing is assigned with 'present'.
> (BTW, common-linux also introduces many other variables -- would this be safe? Actually here I only need to introduce bash_cv_job_control_missing.)
> 
> However, finally, do_compile got a strange failure:
> | shell.c: In function 'shell_reinitialize':
> | shell.c:1742:20: error: 'PPROMPT' undeclared (first use in this function)
> | shell.c:1742:20: note: each undeclared identifier is reported only once for each function it appears in
> | shell.c:1743:22: error: 'SPROMPT' undeclared (first use in this function)
> 
> Could you please give some suggestions?

This is why its a really bad idea for recipes to have their own
configure rather than using our core one :/.

I had a go at this problem myself and it took a bit of figuring out. The
problem is that bash ships "config.h.in" but autoheader overwrites it.
This removes the start/end includes from config.h or config-bot.h and
config-top.h. We can do something like this:

export AUTOHEADER = "true"

do_configure_prepend () {
       if [ ! -e acinclude.m4 ]; then
               cat aclocal.m4 > acinclude.m4
       fi
}
 
instead of the current do_configure override. The _prepend ensures the
bash specific macros are preserved and the export AUTOHEADER stops
autoheader from running at all.

Could you test those changes against bash 4.x and bash 3.x (replacing
the current do_configure in bash.inc) and then if that works send a
patch please?

Cheers,

Richard






^ permalink raw reply

* dbus broken again
From: Martin Jansa @ 2011-11-02 10:55 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 2381 bytes --]

After last attempt to fix dbus there is new issue

I have talked with Richard on IRC already, but will repeat it here just
to warn other dbus users.

Updated list of available packages in /var/lib/opkg/lists/jama-om_gta02.
Upgrading dbus-1 on root from 1.4.12-r2 to 1.4.12-r7...
Downloading
http://jama.dyndns-home.com/org.openembedded.shr-core//armv4t/dbus-1_1.4.12-r7_armv4t.ipk.
Running groupadd commands...
Note: group netdev already exists, not re-creating it
Running useradd commands...
Note: username messagebus already exists, not re-creating it
Upgrading libdbus-1-3 on root from 1.4.12-r2 to 1.4.12-r7...
Downloading
http://jama.dyndns-home.com/org.openembedded.shr-core//armv4t/libdbus-1-3_1.4.12-r7_armv4t.ipk.
Upgrading libglib-2.0-0 on root from 1:2.30.0-r2 to 1:2.30.0-r3...
Downloading
http://jama.dyndns-home.com/org.openembedded.shr-core//armv4t/libglib-2.0-0_2.30.0-r3_armv4t.ipk.
Configuring dbus-1.
 System startup links for /etc/init.d/dbus-1 already exist.
Configuring libdbus-1-3.
Configuring libglib-2.0-0.

SHR root@gjama / $ ll /usr/libexec/dbus-daemon-launch-helper
-rwsr-xr-- 1 root 998 142748 Nov  2 10:12
/usr/libexec/dbus-daemon-launch-helper

SHR root@gjama / $ ll -d /var/lib/dbus
drwxr-xr-x 2 999 998 4096 Nov  2 10:12 /var/lib/dbus

SHR root@gjama / $ grep message /etc/group
messagebus:x:101:
SHR root@gjama / $ grep message /etc/passwd
messagebus:x:42:101:Linux User,,,:/var/run/dbus:/bin/sh


and that's because useradd.bbclass is using UID/GID from sysroots
OE om-gta02@shr ~/shr-core $ grep message tmp/sysroots/om-gta02/etc/group
messagebus:x:998:
OE om-gta02@shr ~/shr-core $ grep message tmp/sysroots/om-gta02/etc/passwd
messagebus:!:999:998::/var/lib/dbus:

and nothing is updating /etc/passwd /etc/group IDs on target.

The possible solution would be to force specified UID/GID in
useradd.bbclass to make it consistent during rebuild from scratch or
teach useradd postinst to update UID/GIDs and chown all runtime created
files to new values instead of skiping useradd/groupadd commnads when
user already exists, like it did now:
Note: username messagebus already exists, not re-creating it
Note: group netdev already exists, not re-creating it

Also reported here:
http://bugzilla.pokylinux.org/show_bug.cgi?id=1711

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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply

* [PATCH] bash: Ensure we fully reautoconf the recipes so site data is used
From: Richard Purdie @ 2011-11-02 11:41 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <1320230106.20107.23.camel@ted>

This ensures bug 487 (missing job control functionality) really gets fixed.

[YOCTO #487]

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/recipes-extended/bash/bash.inc b/meta/recipes-extended/bash/bash.inc
index d55e517..876be1e 100644
--- a/meta/recipes-extended/bash/bash.inc
+++ b/meta/recipes-extended/bash/bash.inc
@@ -22,9 +22,12 @@ ALTERNATIVE_PATH = "${base_bindir}/bash"
 ALTERNATIVE_LINK = "${base_bindir}/sh"
 ALTERNATIVE_PRIORITY = "100"
 
-do_configure () {
-	gnu-configize
-	oe_runconf
+export AUTOHEADER = "true"
+
+do_configure_prepend () {
+	if [ ! -e acinclude.m4 ]; then
+		cat aclocal.m4 > acinclude.m4
+	fi
 }
 
 pkg_postinst_${PN} () {
diff --git a/meta/recipes-extended/bash/bash_3.2.48.bb b/meta/recipes-extended/bash/bash_3.2.48.bb
index 1520c4e..f2ba572 100644
--- a/meta/recipes-extended/bash/bash_3.2.48.bb
+++ b/meta/recipes-extended/bash/bash_3.2.48.bb
@@ -6,7 +6,7 @@ LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=fd5d9bcabd8ed5a54a01ce8d183d592a"
 DEPENDS = "ncurses"
 
-PR = "r7"
+PR = "r8"
 
 SRC_URI = "${GNU_MIRROR}/bash/bash-${PV}.tar.gz \
            ${GNU_MIRROR}/bash/bash-3.2-patches/bash32-049;apply=yes;striplevel=0 \
@@ -23,9 +23,12 @@ sbindir = "/sbin"
 EXTRA_OECONF = "--with-ncurses"
 export CC_FOR_BUILD = "${BUILD_CC}"
 
-do_configure () {
-	gnu-configize
-	oe_runconf
+export AUTOHEADER = "true"
+
+do_configure_prepend () {
+	if [ ! -e acinclude.m4 ]; then
+		cat aclocal.m4 > acinclude.m4
+	fi
 }
 
 pkg_postinst_${PN} () {
diff --git a/meta/recipes-extended/bash/bash_4.2.bb b/meta/recipes-extended/bash/bash_4.2.bb
index a0f5e4e..8070918 100644
--- a/meta/recipes-extended/bash/bash_4.2.bb
+++ b/meta/recipes-extended/bash/bash_4.2.bb
@@ -1,6 +1,6 @@
 require bash.inc
 
-PR = "r0"
+PR = "r1"
 
 SRC_URI = "${GNU_MIRROR}/bash/${BPN}-${PV}.tar.gz;name=tarball \
            ${GNU_MIRROR}/bash/bash-4.2-patches/bash42-001;apply=yes;striplevel=0;name=patch001 \





^ permalink raw reply related

* [PATCH] tslib: fix the bug with loading libts-1.0.so
From: Dmitry Cherukhin @ 2011-11-02 12:03 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <6f165b27054e1895460b20a11d360ac52171ab7b>

Touchpad did not work in the qtdemoE if the library libts-1.0.so was not loaded
manually using the LD_PRELOAD variable. This problem was fixed in the tslib mainline
https://github.com/kergoth/tslib after the 1.0 release. We just import the patch.

Signed-off-by: Dmitry Cherukhin <dima_ch@emcraft.com>
---
 .../tslib/0001-Link-plugins-against-libts.patch    |   57 ++++++++++++++++++++
 meta/recipes-graphics/tslib/tslib_1.0.bb           |    3 +-
 2 files changed, 59 insertions(+), 1 deletions(-)
 create mode 100644 meta/recipes-graphics/tslib/tslib/0001-Link-plugins-against-libts.patch

diff --git a/meta/recipes-graphics/tslib/tslib/0001-Link-plugins-against-libts.patch b/meta/recipes-graphics/tslib/tslib/0001-Link-plugins-against-libts.patch
new file mode 100644
index 0000000..c6b9f59
--- /dev/null
+++ b/meta/recipes-graphics/tslib/tslib/0001-Link-plugins-against-libts.patch
@@ -0,0 +1,57 @@
+From 9623bbedf4ff409e5036edfcfe52b2595932a6d7 Mon Sep 17 00:00:00 2001
+From: Chris Larson <clarson@kergoth.com>
+Date: Sat, 1 Nov 2008 20:46:07 +0000
+Subject: [PATCH] Link plugins against libts
+
+Some plugins use tslib functions. Link those plugins against libts.
+The problem is easy to see with LDFLAGS="-Wl,-no-undefined".
+Without this change DirectFB in unable to use tslib because symbols
+in the tslib plugins can't be resolved.
+
+Signed-off-by: Ville Syrjala <syrjala@sci.fi>
+Signed-off-by: Chris Larson <clarson@kergoth.com>
+
+The patch was imported from git server git://github.com/kergoth/tslib.git
+as of commit id 9623bbedf4ff409e5036edfcfe52b2595932a6d7.
+
+Upstream-Status: Accepted
+Signed-off-by: Dmitry Cherukhin <dima_ch@emcraft.com>
+---
+ plugins/Makefile.am |    5 +++++
+ 1 files changed, 5 insertions(+), 0 deletions(-)
+
+diff --git a/plugins/Makefile.am b/plugins/Makefile.am
+index 3b902c2..4c4ef8b 100644
+--- a/plugins/Makefile.am
++++ b/plugins/Makefile.am
+@@ -114,15 +114,19 @@ pluginexec_LTLIBRARIES = \
+   
+ variance_la_SOURCES	= variance.c
+ variance_la_LDFLAGS	= -module $(LTVSN)
++variance_la_LIBADD	= $(top_builddir)/src/libts.la
+   
+ dejitter_la_SOURCES	= dejitter.c
+ dejitter_la_LDFLAGS	= -module $(LTVSN)
++dejitter_la_LIBADD	= $(top_builddir)/src/libts.la
+   
+ linear_la_SOURCES	= linear.c
+ linear_la_LDFLAGS	= -module $(LTVSN)
++linear_la_LIBADD	= $(top_builddir)/src/libts.la
+ 
+ pthres_la_SOURCES	= pthres.c
+ pthres_la_LDFLAGS	= -module $(LTVSN)
++pthres_la_LIBADD	= $(top_builddir)/src/libts.la
+ 
+ # hw access
+ corgi_la_SOURCES	= corgi-raw.c
+@@ -148,6 +152,7 @@ tatung_la_LDFLAGS	= -module $(LTVSN)
+ 
+ input_la_SOURCES	= input-raw.c
+ input_la_LDFLAGS	= -module $(LTVSN)
++input_la_LIBADD		= $(top_builddir)/src/libts.la
+ 
+ linear_h2200_la_SOURCES	= linear-h2200.c
+ linear_h2200_la_LDFLAGS	= -module $(LTVSN)
+-- 
+1.7.6.4
+
diff --git a/meta/recipes-graphics/tslib/tslib_1.0.bb b/meta/recipes-graphics/tslib/tslib_1.0.bb
index ad9bfa7..b2315ff 100644
--- a/meta/recipes-graphics/tslib/tslib_1.0.bb
+++ b/meta/recipes-graphics/tslib/tslib_1.0.bb
@@ -10,10 +10,11 @@ SECTION = "base"
 LICENSE = "LGPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=f30a9716ef3762e3467a2f62bf790f0a"
 
-PR = "r18"
+PR = "r19"
 
 SRC_URI = "http://download.berlios.de/tslib/tslib-${PV}.tar.bz2 \
            file://fix_version.patch \
+           file://0001-Link-plugins-against-libts.patch \
            file://ts.conf \
            file://tslib.sh"
 
-- 
1.7.6.4




^ permalink raw reply related

* [PATCH] libcense.bbclass: fix OpenSSL mapping [BUGID# 1712]
From: Martin Jansa @ 2011-11-02 12:26 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/classes/license.bbclass                       |    2 +-
 .../recipes-connectivity/openssl/openssl_0.9.8r.bb |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass
index 3f93bf5..baf35f0 100644
--- a/meta/classes/license.bbclass
+++ b/meta/classes/license.bbclass
@@ -44,7 +44,7 @@ SPDXLICENSEMAP[MPLv1.1] = "MPL-1"
 SPDXLICENSEMAP[MIT-X] = "MIT"
 
 #Openssl variations
-SPDXLICENSEMAP[openssl] = "Openssl"
+SPDXLICENSEMAP[openssl] = "OpenSSL"
 
 #Other variations
 SPDXLICENSEMAP[AFL2.1] = "AFL-2"
diff --git a/meta/recipes-connectivity/openssl/openssl_0.9.8r.bb b/meta/recipes-connectivity/openssl/openssl_0.9.8r.bb
index 5add70e..555bacf 100644
--- a/meta/recipes-connectivity/openssl/openssl_0.9.8r.bb
+++ b/meta/recipes-connectivity/openssl/openssl_0.9.8r.bb
@@ -1,6 +1,6 @@
 require openssl.inc
 
-PR = "r6"
+PR = "r7"
 SRC_URI += "file://debian/ca.patch \
             file://debian/config-hurd.patch;apply=no \
             file://debian/debian-targets.patch \
-- 
1.7.7.1




^ permalink raw reply related

* [PATCH] image_types bbclass: use 4k bytes per inode so we don't run out of space immediately
From: Koen Kooi @ 2011-11-02 14:16 UTC (permalink / raw)
  To: openembedded-core; +Cc: Koen Kooi

genext2fs only creates the minimum number of inodes, after this patch it will scale with the rootfs size

Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
 meta/classes/image_types.bbclass |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
index 0d64705..9549a9e 100644
--- a/meta/classes/image_types.bbclass
+++ b/meta/classes/image_types.bbclass
@@ -38,36 +38,36 @@ IMAGE_CMD_jffs2 = "mkfs.jffs2 --root=${IMAGE_ROOTFS} --faketime --output=${DEPLO
 
 IMAGE_CMD_cramfs = "mkcramfs ${IMAGE_ROOTFS} ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.cramfs ${EXTRA_IMAGECMD}"
 
-IMAGE_CMD_ext2 = "genext2fs -b $ROOTFS_SIZE -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext2"
+IMAGE_CMD_ext2 = "genext2fs -b $ROOTFS_SIZE -i 4096 -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext2"
 IMAGE_CMD_ext2.gz () {
 	rm -rf ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN} && mkdir ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}
-	genext2fs -b $ROOTFS_SIZE -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext2
+	genext2fs -b $ROOTFS_SIZE -i 4096 -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext2
 	gzip -f -9 ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext2
 	mv ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext2.gz ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext2.gz
 	rmdir ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}
 }
 IMAGE_CMD_ext2.bz2 () {
 	rm -rf ${DEPLOY_DIR_IMAGE}/tmp.gz && mkdir ${DEPLOY_DIR_IMAGE}/tmp.gz
-	genext2fs -b $ROOTFS_SIZE -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/tmp.gz/${IMAGE_NAME}.rootfs.ext2
+	genext2fs -b $ROOTFS_SIZE -i 4096 -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/tmp.gz/${IMAGE_NAME}.rootfs.ext2
 	bzip2 -f -9 ${DEPLOY_DIR_IMAGE}/tmp.gz/${IMAGE_NAME}.rootfs.ext2
 	mv ${DEPLOY_DIR_IMAGE}/tmp.gz/${IMAGE_NAME}.rootfs.ext2.bz2 ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext2.bz2
 	rmdir ${DEPLOY_DIR_IMAGE}/tmp.gz
 }
 IMAGE_CMD_ext2.lzma () {
 	rm -rf ${DEPLOY_DIR_IMAGE}/tmp.gz && mkdir ${DEPLOY_DIR_IMAGE}/tmp.gz
-	genext2fs -b $ROOTFS_SIZE -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/tmp.gz/${IMAGE_NAME}.rootfs.ext2
+	genext2fs -b $ROOTFS_SIZE -i 4096 -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/tmp.gz/${IMAGE_NAME}.rootfs.ext2
 	lzma -f -7 ${DEPLOY_DIR_IMAGE}/tmp.gz/${IMAGE_NAME}.rootfs.ext2
 	mv ${DEPLOY_DIR_IMAGE}/tmp.gz/${IMAGE_NAME}.rootfs.ext2.lzma ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext2.lzma
 	rmdir ${DEPLOY_DIR_IMAGE}/tmp.gz
 }
 
 IMAGE_CMD_ext3 () {
-	genext2fs -b $ROOTFS_SIZE -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext3
+	genext2fs -b $ROOTFS_SIZE -i 4096 -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext3
 	tune2fs -j ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext3
 }
 IMAGE_CMD_ext3.gz () {
 	rm -rf ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN} && mkdir ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}
-	genext2fs -b $ROOTFS_SIZE -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext3
+	genext2fs -b $ROOTFS_SIZE -i 4096 -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext3
 	tune2fs -j ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext3
 	gzip -f -9 ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext3
 	mv ${DEPLOY_DIR_IMAGE}/tmp.gz-${PN}/${IMAGE_NAME}.rootfs.ext3.gz ${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.ext3.gz
@@ -75,7 +75,7 @@ IMAGE_CMD_ext3.gz () {
 }
 
 oe_mkext4fs () {
-	genext2fs -b $ROOTFS_SIZE -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} $1
+	genext2fs -b $ROOTFS_SIZE -i 4096 -d ${IMAGE_ROOTFS} ${EXTRA_IMAGECMD} $1
 	tune2fs -O extents,uninit_bg,dir_index,has_journal $1
 	e2fsck -yfDC0 $1 || chk=$?
 	case $chk in
-- 
1.7.2.5




^ permalink raw reply related

* Re: [PATCH] image_types bbclass: use 4k bytes per inode so we don't run out of space immediately
From: Koen Kooi @ 2011-11-02 14:22 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <1320243402-2116-1-git-send-email-koen@dominion.thruhere.net>

[-- Attachment #1: Type: text/plain, Size: 685 bytes --]


Op 2 nov. 2011, om 15:16 heeft Koen Kooi het volgende geschreven:

> genext2fs only creates the minimum number of inodes, after this patch it will scale with the rootfs size
> 
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>

The matching OE classic commit is:

koen@dominion:/OE/org.openembedded.dev$ git show 12275a16
commit 12275a1604e2a5e9e66c7ef900c450b01e5cf622
Author: Tom Rini <tom_rini@mentor.com>
Date:   Fri Dec 17 10:45:32 2010 -0700

    bitbake.conf: Use normal bytes per inode param to genext2fs

    Without this we get an unusably small number of inodes in
    our filesystem images.

    Signed-off-by: Tom Rini <tom_rini@mentor.com>



[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 169 bytes --]

^ permalink raw reply

* Re: [PATCH] image_types bbclass: use 4k bytes per inode so we don't run out of space immediately
From: Tom Rini @ 2011-11-02 14:52 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer; +Cc: Koen Kooi
In-Reply-To: <1320243402-2116-1-git-send-email-koen@dominion.thruhere.net>

On Wed, Nov 2, 2011 at 7:16 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
> genext2fs only creates the minimum number of inodes, after this patch it will scale with the rootfs size
>
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>

Acked-by: Tom Rini <tom.rini@gmail.com>

-- 
Tom



^ permalink raw reply

* Re: [PATCH] Add new IMAGE_CLASSES variable for classes for image generation
From: McClintock Matthew-B29882 @ 2011-11-02 15:03 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer
In-Reply-To: <4EB09538.3010701@intel.com>

On Tue, Nov 1, 2011 at 7:56 PM, Saul Wold <saul.wold@intel.com> wrote:
> Right I understood that part from before I think.  But why can't you have
>
> IMAGE_CLASSES ?= "image_types"
>
> and then in the local.conf override that with
>
> IMAGE_CLASSES = "image_types_uboot"
>
> since image_types_uboot inherits image_types.
>
>> IMAGE_CLASSES += "image_types_uboot" and leave the other bit as is...
>
> I have to admit I like this a little better with the possible thought of
> breaking up image_types a little more, keep more used ones in image_types,
> but move lesser used ones to their on .bbclass
>
> Them IMAGE_CLASSES truly is a list of image_type classes.

I think this is best, let the user override IMAGE_CLASSES to
(eventually) be able to add individual image_types_XYZ. Will send new
patch.

-M



^ permalink raw reply

* Re: [PATCH 1/1] bash: make job control really work
From: Cui, Dexuan @ 2011-11-02 15:48 UTC (permalink / raw)
  To: Richard Purdie; +Cc: Patches and discussions about the oe-core layer
In-Reply-To: <1320230106.20107.23.camel@ted>

Richard Purdie wrote on 2011-11-02:
> On Wed, 2011-11-02 at 15:10 +0800, Cui, Dexuan wrote:
>> Richard Purdie wrote on 2011-11-01:
>>> On Tue, 2011-11-01 at 16:05 +0800, Dexuan Cui wrote:
> I had a go at this problem myself and it took a bit of figuring out.
> The problem is that bash ships "config.h.in" but autoheader overwrites it.
> This removes the start/end includes from config.h or config-bot.h and
> config-top.h. We can do something like this:
> 
> export AUTOHEADER = "true"
> 
> do_configure_prepend () {
>        if [ ! -e acinclude.m4 ]; then
>                cat aclocal.m4 > acinclude.m4
>        fi
> }
> 
> instead of the current do_configure override. The _prepend ensures the
> bash specific macros are preserved and the export AUTOHEADER stops
> autoheader from running at all.
> 
> Could you test those changes against bash 4.x and bash 3.x (replacing
> the current do_configure in bash.inc) and then if that works send a patch please?
Looks bug 487 only happens to bash 4.x and bash 3.x doesn't have such a bug (the
recipe of bash 3.x doesn't use bash.inc, either)

RP, thanks a lot for the help! Here is the new patch:
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=dcui/bug487&id=b524337ed5670de2c0d294c3f6970e24f23847eb

Thanks,
-- Dexuan


^ permalink raw reply


This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox