public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/8] add fontcache.bbclass
Date: Thu, 07 Feb 2013 22:43:06 +0000	[thread overview]
Message-ID: <1360276986.10722.58.camel@ted> (raw)
In-Reply-To: <5113CA88.9020508@linaro.org>

On Thu, 2013-02-07 at 16:38 +0100, Marcin Juszkiewicz wrote:
> W dniu 05.02.2013 10:34, Laurentiu Palcu pisze:
> > All font packages should inherit this class in order to generate the
> > proper postinst/postrm scriptlets.
> > 
> > The scriptlets will actually create a host intercept hook that will be
> > executed at the end, at do_rootfs time, after all packages have been
> > installed. This is good when there are many font packages.
> > 
> > [YOCTO #2923]
> > 
> > Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
> > ---
> >  meta/classes/fontcache.bbclass |   47 ++++++++++++++++++++++++++++++++++++++++
> 
> > +${@qemu_run_binary(d, '$D', '/usr/bin/fc-cache')} --sysroot=$D >/dev/null 2>&1
> 
> What about platform which do not have qemu support? Like AArch64...
> 
> | Configuring libpcre-staticdev.
> | Building from feeds activated!
> | Running intercept scripts:
> | > Executing update_font_cache
> | ERROR: Function failed: do_rootfs (see
> /home/hrw/HDD/devel/canonical/aarch64/openembedded/build/tmp-eglibc/work/genericarmv8-oe-linux/linaro-image-sdk/1.0-r2/temp/log.do_rootfs.23211
> for further information)

Ideally the intercept code should fall back to running the postinst on
the target if qemu isn't available. Patches welcome to make that happen.

The hard part is this is an "allarch" recipe so we can't disable this
conditionally on aarch64. We can disable all intercept scripts I guess.

Or we make these packages arch specific on aarch64 I guess. 

Cheers,

Richard




  reply	other threads:[~2013-02-07 23:00 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-05  9:34 [PATCH 0/8] Generate fonts cache on host and other fixes Laurentiu Palcu
2013-02-05  9:34 ` [PATCH 1/8] add fontcache.bbclass Laurentiu Palcu
2013-02-07 15:38   ` Marcin Juszkiewicz
2013-02-07 22:43     ` Richard Purdie [this message]
2013-02-07 23:18     ` Phil Blundell
2013-02-05  9:34 ` [PATCH 2/8] add qemuwrapper-cross recipe Laurentiu Palcu
2013-02-05  9:34 ` [PATCH 3/8] qemu.bbclass: return qemuwrapper instead of qemu-allarch Laurentiu Palcu
2013-02-05  9:34 ` [PATCH 4/8] image.bbclass: add a proper error message if hook script fails Laurentiu Palcu
2013-02-05  9:34 ` [PATCH 5/8] fontconfig: add sysroot option to fc-cache and fc-cat Laurentiu Palcu
2013-02-05  9:35 ` [PATCH 6/8] liberation-fonts: use the new fontcache.bbclass Laurentiu Palcu
2013-02-05  9:35 ` [PATCH 7/8] package_ipk, rootfs_ipk: remove the "set -x" Laurentiu Palcu
2013-02-05  9:35 ` [PATCH 8/8] pulseaudio: do not postpone postinstall Laurentiu Palcu

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=1360276986.10722.58.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=marcin.juszkiewicz@linaro.org \
    --cc=openembedded-core@lists.openembedded.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox