All of lore.kernel.org
 help / color / mirror / Atom feed
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


  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.