From: Rongqing Li <rongqing.li@windriver.com>
To: "Burton, Ross" <ross.burton@intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/1] systemd: ignore qemu output, and check its return code
Date: Thu, 8 Aug 2013 16:55:42 +0800 [thread overview]
Message-ID: <52035D0E.4090502@windriver.com> (raw)
In-Reply-To: <CAJTo0LZw9kOSmQbeQM71JGZh36nYy58o=ZEB_oA8nYOFBtqZTw@mail.gmail.com>
On 08/07/2013 07:55 PM, Burton, Ross wrote:
> On 7 August 2013 08:30, Rongqing Li <rongqing.li@windriver.com> wrote:
>>> ${@qemu_run_binary(d, '$D', '${base_bindir}/udevadm')}
>>> hwdb --update \
>>> - --root $D
>>> + --root $D 2>/dev/null
>>> +
>>> + [ $? -ne 0 ] && exit 1
>
> The $? check can be removed replaced by set -e, as many postinst
> scripts do already.
>
> Why/where is the output from runqemu considered an error? I wasn't
> aware that logs were monitored for that error message. I don't think
> we should be hiding the output of runqemu as any real errors will also
> be hidden.
>
> Ross
I always get the below on error on my machine(Ubuntu 12.10) when enable
systemd:
| 102:Installing libnfsidmap0
######################################## [ 26%]
| 103:Installing run-postinsts
######################################## [ 26%]
| Output from run-postinsts-1.0-r9@i586:
| --
| 257:Installing liberation-fo..
######################################## [ 65%]
| 258:Installing glib-networking
######################################## [ 65%]
| 259:Installing wpa-supplicant
######################################## [ 66%]
| 260:Installing udev-hwdb
######################################## [ 66%]
| Output from udev-hwdb-1:204-r0@i586:
| ERROR: ld.so: object 'libpseudo.so' from LD_PRELOAD cannot be
preloaded: ignored.
| qemu: Unsupported syscall: 240
| 261:Installing udev
######################################## [ 66%]
| 262:Installing util-linux-fsck
######################################## [ 67%]
| Output from util-linux-fsck-2.23.2-r0@i586:
| update-alternatives: Linking
/buildarea/build/poky/build/tmp/work/qemux86-poky-linux/core-image-sato/1.0-r0/rootfs//sbin/fsck.minix
to /sbin/fsck.minix.util-linux
|
| ERROR: Function failed: do_rootfs (log file is located at
/buildarea/build/poky/build/tmp/work/qemux86-poky-linux/core-image-sato/1.0-r0/temp/log.do_rootfs.26461)
ERROR: Task 7
(/buildarea/build/poky/meta/recipes-sato/images/core-image-sato.bb,
do_rootfs) failed with exit code '1'
NOTE: Tasks Summary: Attempted 5548 tasks of which 924 didn't need to be
rerun and 1 failed.
No currently running tasks (5547 of 5549)
Summary: 1 task failed:
/buildarea/build/poky/meta/recipes-sato/images/core-image-sato.bb,
do_rootfs
Summary: There was 1 ERROR message shown, returning a non-zero exit code.
After using the patch, it works well.
Other place used the same method to handle the qemu_run_binary result:
lirq@lirq-OptiPlex-780:/buildarea/build/poky/meta$ grep qemu_run_binary
./ -r -A2
./classes/qemu.bbclass:# ${@qemu_run_binary(d, '$D',
'/usr/bin/test_app')} [test_app arguments]
./classes/qemu.bbclass-#
./classes/qemu.bbclass:def qemu_run_binary(data, rootfs_path, binary):
./classes/qemu.bbclass- qemu_binary = qemu_target_binary(data)
./classes/qemu.bbclass- if qemu_binary == "qemu-allarch":
--
./classes/gtk-immodules-cache.bbclass: ${@qemu_run_binary(d,
'$D', '${bindir}/gtk-query-immodules-$maj_ver.0')} \
./classes/gtk-immodules-cache.bbclass- $IMFILES >
$D/etc/gtk-$maj_ver.0/gtk.immodules 2>/dev/null &&
./classes/gtk-immodules-cache.bbclass- sed -i -e "s:$D::"
$D/etc/gtk-$maj_ver.0/gtk.immodules
--
./classes/gtk-immodules-cache.bbclass: ${@qemu_run_binary(d,
'$D', '${bindir}/gtk-query-immodules-$maj_ver.0')} \
./classes/gtk-immodules-cache.bbclass- $IMFILES >
$D/etc/gtk-$maj_ver.0/gtk.immodules 2>/dev/null &&
./classes/gtk-immodules-cache.bbclass- sed -i -e "s:$D::"
$D/etc/gtk-$maj_ver.0/gtk.immodules
--
./recipes-graphics/pango/pango.inc: ${@qemu_run_binary(d,
'$D','${bindir}/${MLPREFIX}pango-querymodules')} \
./recipes-graphics/pango/pango.inc-
$D${libdir}/pango/${LIBV}/modules/*.so \
./recipes-graphics/pango/pango.inc- >
$D${sysconfdir}/pango/${MLPREFIX}pango.modules 2>/dev/null
-Roy
>
>
--
Best Reagrds,
Roy | RongQing Li
next prev parent reply other threads:[~2013-08-08 8:55 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-24 5:24 [PATCH 0/1] systemd: ignore qemu output, and check its return code rongqing.li
2013-07-24 5:24 ` [PATCH 1/1] " rongqing.li
2013-07-24 5:34 ` Khem Raj
2013-07-24 6:15 ` Rongqing Li
2013-07-26 11:53 ` Burton, Ross
2013-08-05 5:21 ` Rongqing Li
2013-08-07 7:30 ` Rongqing Li
2013-08-07 11:55 ` Burton, Ross
2013-08-08 8:55 ` Rongqing Li [this message]
2013-08-08 10:57 ` Burton, Ross
2013-08-07 18:26 ` Khem Raj
2013-08-08 8:53 ` Rongqing Li
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52035D0E.4090502@windriver.com \
--to=rongqing.li@windriver.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=ross.burton@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.