* Re: [PATCH] Add new IMAGE_CLASSES variable for classes for image generation
From: McClintock Matthew-B29882 @ 2011-11-02 0:54 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
In-Reply-To: <CAEsOVNdaEPihB8ecoMuxzSw-TR-CsK1ZKsCgcj5nktjtHmA32Q@mail.gmail.com>
On Tue, Nov 1, 2011 at 7:50 PM, McClintock Matthew-B29882
<B29882@freescale.com> wrote:
> IMAGE_CLASSES += "image_types_uboot" and leave the other bit as is...
Except it seems I have to do this:
IMAGE_CLASSES_append = " image_types_uboot"
if I want the value to actually be appended.
-M
^ permalink raw reply
* Re: [PATCH] Add new IMAGE_CLASSES variable for classes for image generation
From: Saul Wold @ 2011-11-02 0:56 UTC (permalink / raw)
To: McClintock Matthew-B29882,
Patches and discussions about the oe-core layer
In-Reply-To: <CAEsOVNdaEPihB8ecoMuxzSw-TR-CsK1ZKsCgcj5nktjtHmA32Q@mail.gmail.com>
On 11/01/2011 05:50 PM, McClintock Matthew-B29882 wrote:
> On Tue, Nov 1, 2011 at 7:31 PM, Saul Wold<saul.wold@intel.com> wrote:
>>> +# Additional image generation features
>>> +#
>>> +# The following is a list of classes to import to use in the generation
>>> of images
>>> +# currently an example class is image_types_uboot
>>> +# IMAGE_CLASSES ?= "image_types_uboot"
>>> +
>>> #
>>> # Runtime testing of images
>>> #
>>> diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
>>> index 05f4331..e932879 100644
>>> --- a/meta/classes/image.bbclass
>>> +++ b/meta/classes/image.bbclass
>>> @@ -111,7 +111,8 @@ def get_devtable_list(d):
>>> str += " %s" % bb.which(bb.data.getVar('BBPATH', d, 1), devtable)
>>> return str
>>>
>>> -inherit image_types
>>> +IMAGE_CLASSES = "image_types"
>>> +inherit ${IMAGE_CLASSES}
>>>
>> Does this really work with =, should it not be ?= here?
>
> Ugh. No. This was an attempt to fix this:
>
> +IMAGE_CLASSES ??= ""
> +inherit image_types ${IMAGE_CLASSES}
>
> which gives the following bitbake error:
>
> ERROR: classes/.bbclass is not a BitBake file
> ERROR: Command execution failed: Exited with 1
>
> when IMAGE_CLASSES is left as "". It's trying to inherit a ".bbclass"
> file. There is no good alternative because I have to enforce
> IMAGE_CLASSES is only modified by appending to it. So the only
> solution is to modifed the local.conf.sample to say
>
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.
Sau!
> -M
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
^ permalink raw reply
* [PATCH 0/1] distro_tracking_fields: Update the Month to 'MMM'
From: wenzong.fan @ 2011-11-02 5:52 UTC (permalink / raw)
To: openembedded-core
From: Wenzong Fan <wenzong.fan@windriver.com>
Update the format of Month to 'MMM' instead of spell it out.
The following changes since commit 4a951e0433a99cd985515843f0a48fadc7050aca:
Robert P. J. Day (1):
Fix HOMEPAGE values in libzypp and sat-solver .bb files
are available in the git repository at:
git://git.pokylinux.org/poky-contrib wenzong/distro_tracking_fields_update
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=wenzong/distro_tracking_fields_update
Wenzong Fan (1):
distro_tracking_fields: Update the Month to 'MMM'
.../conf/distro/include/distro_tracking_fields.inc | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
^ permalink raw reply
* [PATCH 1/1] distro_tracking_fields: Update the Month to 'MMM'
From: wenzong.fan @ 2011-11-02 5:52 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <cover.1320212931.git.wenzong.fan@windriver.com>
From: Wenzong Fan <wenzong.fan@windriver.com>
Use format 'MMM' instead of spell out a Month.
Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
---
.../conf/distro/include/distro_tracking_fields.inc | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/meta/conf/distro/include/distro_tracking_fields.inc b/meta/conf/distro/include/distro_tracking_fields.inc
index 10835dd..998eabf 100644
--- a/meta/conf/distro/include/distro_tracking_fields.inc
+++ b/meta/conf/distro/include/distro_tracking_fields.inc
@@ -5979,14 +5979,14 @@ DISTRO_PN_ALIAS_pn-libsdl = "Fedora=SDL Opensuse=SDL"
RECIPE_STATUS_pn-ccache = "green"
RECIPE_MAINTAINER_pn-ccache = "Wenzong Fan <wenzong.fan@windriver.com>"
RECIPE_LATEST_VERSION_pn-ccache = "3.1.6"
-RECIPE_LAST_UPDATE_pn-ccache = "October 19, 2011"
-RECIPE_MANUAL_CHECK_DATE_pn-ccache = "October 25, 2011"
+RECIPE_LAST_UPDATE_pn-ccache = "Oct 19, 2011"
+RECIPE_MANUAL_CHECK_DATE_pn-ccache = "Oct 25, 2011"
RECIPE_STATUS_pn-chkconfig = "green"
RECIPE_MAINTAINER_pn-chkconfig = "Wenzong Fan <wenzong.fan@windriver.com>"
RECIPE_LATEST_VERSION_pn-chkconfig = "1.3.55"
-RECIPE_LAST_UPDATE_pn-chkconfig = "October 19, 2011"
-RECIPE_MANUAL_CHECK_DATE_pn-chkconfig = "October 25, 2011"
+RECIPE_LAST_UPDATE_pn-chkconfig = "Oct 19, 2011"
+RECIPE_MANUAL_CHECK_DATE_pn-chkconfig = "Oct 25, 2011"
DISTRO_PN_ALIAS_pn-lame = "Debian=lame Ubuntu=lame"
DISTRO_PN_ALIAS_pn-libiconv = "Fedora=mingw-libiconv Opensuse=cross-mingw-libiconv"
--
1.7.0.4
^ permalink raw reply related
* Re: [PATCH 0/1] distro_tracking_fields: update packages tracking fields
From: wenzong fan @ 2011-11-02 5:48 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
In-Reply-To: <4EB0909C.5020202@intel.com>
On 11/02/2011 08:36 AM, Saul Wold wrote:
> On 10/24/2011 07:49 PM, wenzong.fan@windriver.com wrote:
>> From: Wenzong Fan<wenzong.fan@windriver.com>
>>
>> Update distro tracking fields for ccache, chkconfig.
>>
>> The following changes since commit
>> 53ee27a1c5b589cba874f674c428e1a36557cc3b:
>> Joshua Lock (1):
>> doc: update manual for checksum changes
>>
>> are available in the git repository at:
>>
>> git://git.pokylinux.org/poky-contrib
>> wenzong/distro_tracking_fields_update
>>
>> http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=wenzong/distro_tracking_fields_update
>>
>>
>> Wenzong Fan (1):
>> distro_tracking_fields: update packages tracking fields
>>
>> .../conf/distro/include/distro_tracking_fields.inc | 12 ++++++++++++
>> 1 files changed, 12 insertions(+), 0 deletions(-)
>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
> This got pulled, but I still need a fix for Month format to MMM please
Fixed, please see another PR whcih with subject "distro_tracking_fields:
Update the Month to 'MMM'".
Thanks
Wenzong
>
> Thanks
> Sau!
>
>
^ permalink raw reply
* Re: [Pull v2 2/4] xserver-nodm-init: Add xuser (hardcoded)
From: Lauri Hintsala @ 2011-11-02 6:11 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
In-Reply-To: <453f61fd0043629181379d5d3766c118bdd46dd2.1320183757.git.sgw@linux.intel.com>
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?
Lauri
> fi
> }
>
> -pkg_postinst_${PN} () {
> - if [ "x$D" != "x" ] ; then
> - exit 1
> - fi
> -
> - if [ -f /etc/X11/Xusername ]; then
> - # create the rootless X user, and add user to group tty, video, audio
> - username=`cat /etc/X11/Xusername`
> - adduser --disabled-password $username
> - # FIXME: use addgroup if busybox addgroup is ready
> - sed -i -e "s/^video:.*/&${username}/g" /etc/group
> - sed -i -e "s/^tty:.*/&${username}/g" /etc/group
> - sed -i -e "s/^audio:.*/&${username}/g" /etc/group
> - fi
> -}
> -
> -inherit update-rc.d
> +inherit update-rc.d useradd
>
> INITSCRIPT_NAME = "xserver-nodm"
> INITSCRIPT_PARAMS = "start 9 5 2 . stop 20 0 1 6 ."
> +
> +# Use fixed Xusername of xuser for now, this will need to be
> +# fixed if the Xusername changes from xuser
> +USERADD_PACKAGES = "${PN}"
> +USERADD_PARAM_${PN} = "--system --no-create-home \
> + --shell /bin/false --groups video,tty,audio \
> + --user-group xuser"
> +
^ permalink raw reply
* [PATCH 0/1] [Yocto Bug 1700] Fix for buildstats diskio on non physical disks
From: Beth Flanagan @ 2011-11-02 6:41 UTC (permalink / raw)
To: openembedded-core
From: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
tmpfs/encryptfs/ramfs have no entry in /proc/diskstats. This modifies
buildstats to not collect diskio statistics when we encounter a case where
the os.major/os.minor is not represented with an entry in /proc/diskstats.
Longer term solution to this is to:
- Identify all fs types that don't have representation in /proc/diskstats
- Gather meaningful iostats of these fs types if possible.
The following changes since commit 4a951e0433a99cd985515843f0a48fadc7050aca:
Fix HOMEPAGE values in libzypp and sat-solver .bb files (2011-11-01 18:28:19 +0000)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib eflanagan/diskio_bug1700
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=eflanagan/diskio_bug1700
Elizabeth Flanagan (1):
[Yocto Bug 1700] Fix for buildstats on tmpfs
meta/classes/buildstats.bbclass | 37 ++++++++++++++++++++++++++-----------
1 files changed, 26 insertions(+), 11 deletions(-)
^ permalink raw reply
* [PATCH 1/1] [Yocto Bug 1700] Fix for buildstats on tmpfs
From: Beth Flanagan @ 2011-11-02 6:41 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <cover.1320190748.git.elizabeth.flanagan@intel.com>
From: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
tmpfs/encryptfs/(and most likely, but not confirmed)ramfs TMPDIRs
cause diskstats to choke. No device entry ends up in /proc/diskstats
for these fs types, which ends up causing the failure.
The short term solution is to exclude these fs types from diskstat
collection. Longer term we will want to see if we can collect
meaningful diskio for each of these, and other, use cases, but for
this cleans up Bug 1700.
Signed-off-by: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
---
meta/classes/buildstats.bbclass | 37 ++++++++++++++++++++++++++-----------
1 files changed, 26 insertions(+), 11 deletions(-)
diff --git a/meta/classes/buildstats.bbclass b/meta/classes/buildstats.bbclass
index 55cbb3c..96c98d4 100644
--- a/meta/classes/buildstats.bbclass
+++ b/meta/classes/buildstats.bbclass
@@ -48,13 +48,24 @@ def set_device(e):
# something like stick DL_DIR on a different partition and this would
# throw stats gathering off. The same goes with SSTATE_DIR. However, let's
# get the basics in here and work on the cornercases later.
+ # A note. /proc/diskstats does not contain info on encryptfs, tmpfs, etc.
+ # If we end up hitting one of these fs, we'll just skip diskstats collection.
############################################################################
device=os.stat(tmpdir)
majordev=os.major(device.st_dev)
minordev=os.minor(device.st_dev)
+ ############################################################################
+ # Bug 1700:
+ # Because tmpfs/encryptfs/ramfs etc inserts no entry in /proc/diskstats
+ # we set rdev to NoLogicalDevice and search for it later. If we find NLD
+ # we do not collect diskstats as the method to collect meaningful statistics
+ # for these fs types requires a bit more research.
+ ############################################################################
for line in open("/proc/diskstats", "r"):
if majordev == int(line.split()[0]) and minordev == int(line.split()[1]):
rdev=line.split()[2]
+ else:
+ rdev="NoLogicalDevice"
file = open(bb.data.getVar('DEVFILE', e.data, True), "w")
file.write(rdev)
file.close()
@@ -133,10 +144,11 @@ def write_task_data(status, logfile, dev, e):
# For the best information, running things with BB_TOTAL_THREADS = "1"
# would return accurate per task results.
############################################################################
- diskdata = get_diskdata("__diskdata_task", dev, e.data)
- if diskdata:
- for key in sorted(diskdata.iterkeys()):
- file.write(key + ": " + diskdata[key] + "\n")
+ if dev != "NoLogicalDevice":
+ diskdata = get_diskdata("__diskdata_task", dev, e.data)
+ if diskdata:
+ for key in sorted(diskdata.iterkeys()):
+ file.write(key + ": " + diskdata[key] + "\n")
if status is "passed":
file.write("Status: PASSED \n")
else:
@@ -169,7 +181,8 @@ python run_buildstats () {
bb.mkdirhier(bsdir)
except:
pass
- set_diskdata("__diskdata_build", device, e.data)
+ if device != "NoLogicalDevice":
+ set_diskdata("__diskdata_build", device, e.data)
set_timedata("__timedata_build", e.data)
build_time = os.path.join(bsdir, "build_stats")
# write start of build into build_time
@@ -185,7 +198,7 @@ python run_buildstats () {
elif isinstance(e, bb.event.BuildCompleted):
bn = get_bn(e)
- dev = get_device(e)
+ device = get_device(e)
bsdir = os.path.join(bb.data.getVar('BUILDSTATS_BASE', e.data, True), bn)
taskdir = os.path.join(bsdir, bb.data.expand("${PF}", e.data))
build_time = os.path.join(bsdir, "build_stats")
@@ -201,10 +214,11 @@ python run_buildstats () {
file.write("Elapsed time: %0.2f seconds \n" % (time))
if cpu:
file.write("CPU usage: %0.1f%% \n" % cpu)
- diskio = get_diskdata("__diskdata_build", dev, e.data)
- if diskio:
- for key in sorted(diskio.iterkeys()):
- file.write(key + ": " + diskio[key] + "\n")
+ if device != "NoLogicalDevice":
+ diskio = get_diskdata("__diskdata_build", device, e.data)
+ if diskio:
+ for key in sorted(diskio.iterkeys()):
+ file.write(key + ": " + diskio[key] + "\n")
file.close()
if isinstance(e, bb.build.TaskStarted):
@@ -212,7 +226,8 @@ python run_buildstats () {
device = get_device(e)
bsdir = os.path.join(bb.data.getVar('BUILDSTATS_BASE', e.data, True), bn)
taskdir = os.path.join(bsdir, bb.data.expand("${PF}", e.data))
- set_diskdata("__diskdata_task", device, e.data)
+ if device != "NoLogicalDevice":
+ set_diskdata("__diskdata_task", device, e.data)
set_timedata("__timedata_task", e.data)
try:
bb.mkdirhier(taskdir)
--
1.7.1
^ permalink raw reply related
* 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
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox