[PATCH 0/1] fontcache: fix fonts postinst failed

[PATCH 0/1] fontcache: fix fonts postinst failed

[Hongxu Jia]

1. Test steps:

1) edit local.conf:
---
IMAGE_INSTALL_append = " liberation-fonts"
---

2) Build image.
bitbake core-image-minimal

2. Expected:
No build warning of "The postinstall intercept hook 'update_font_cache' failed"

//Hongxu

The following changes since commit ef627ab364d52fe19994c94c1a78fbe21620a32c:

  local.conf.sample.extended: remove RM_OLD_IMAGE (2016-10-19 17:07:41 +0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib hongxu/fix-font
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=hongxu/fix-font

Hongxu Jia (1):
  fontcache: fix fonts postinst failed

 meta/classes/fontcache.bbclass | 4 ++++
 1 file changed, 4 insertions(+)

-- 
2.8.1

[PATCH 1/1] fontcache: fix fonts postinst failed

[Hongxu Jia]

While instal liberation-fonts to image, the fonts postinst
failed at do_rootfs time:
-----------------------------------------------
|WARNING: core-image-minimal-1.0-r0 do_rootfs: The postinstall intercept hook
'update_font_cache' failed (exit code: 139)! See log for details!
|WARNING: core-image-minimal-1.0-r0 do_rootfs: The postinstalls for the
following packages will be postponed for first boot: liberation-fonts
-----------------------------------------------

Since the following commit applied in oe-core
-----------------------------------------------
commit ce1e7fcc60276040477c1d5e3129e029bb9f204b
Author: Richard Purdie <richard.purdie@linuxfoundation.org>
Date:   Fri Sep 30 17:43:23 2016 +0100

    allarch: Fixes to stop rebuilds when change multilibs
-----------------------------------------------
It assignes base_libdir="/none" which caused fonts postinst failed at
do_rootfs.

The fonts postinst invoke qemu to update font cache which requires
base_libdir arch relative(such as base_libdir="/lib64").

We refer what meta/conf/multilib.conf does, assign it with arch relative.

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 meta/classes/fontcache.bbclass | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/meta/classes/fontcache.bbclass b/meta/classes/fontcache.bbclass
index 8ebdfc4..ce22170 100644
--- a/meta/classes/fontcache.bbclass
+++ b/meta/classes/fontcache.bbclass
@@ -38,6 +38,10 @@ python () {
 }
 
 python add_fontcache_postinsts() {
+    baselib = d.getVar('BASE_LIB_tune-' + (d.getVar('DEFAULTTUNE', True) or 'INVALID'), True) or \
+              d.getVar('BASELIB', True)
+    d.setVar("baselib", baselib)
+
     for pkg in d.getVar('FONT_PACKAGES', True).split():
         bb.note("adding fonts postinst and postrm scripts to %s" % pkg)
         postinst = d.getVar('pkg_postinst_%s' % pkg, True) or d.getVar('pkg_postinst', True)
-- 
2.8.1

Re: [PATCH 1/1] fontcache: fix fonts postinst failed

[Christopher Larson]

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

On Tue, Oct 25, 2016 at 7:31 PM, Hongxu Jia <hongxu.jia@windriver.com>
wrote:

> While instal liberation-fonts to image, the fonts postinst
> failed at do_rootfs time:
> -----------------------------------------------
> |WARNING: core-image-minimal-1.0-r0 do_rootfs: The postinstall intercept
> hook
> 'update_font_cache' failed (exit code: 139)! See log for details!
> |WARNING: core-image-minimal-1.0-r0 do_rootfs: The postinstalls for the
> following packages will be postponed for first boot: liberation-fonts
> -----------------------------------------------
>
> Since the following commit applied in oe-core
> -----------------------------------------------
> commit ce1e7fcc60276040477c1d5e3129e029bb9f204b
> Author: Richard Purdie <richard.purdie@linuxfoundation.org>
> Date:   Fri Sep 30 17:43:23 2016 +0100
>
>     allarch: Fixes to stop rebuilds when change multilibs
> -----------------------------------------------
> It assignes base_libdir="/none" which caused fonts postinst failed at
> do_rootfs.
>
> The fonts postinst invoke qemu to update font cache which requires
> base_libdir arch relative(such as base_libdir="/lib64").
>
> We refer what meta/conf/multilib.conf does, assign it with arch relative.
>
> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>


Without this, we get the mentioned warnings with every qemux86-64 or
intel-corei7-64 build using our external toolchain. Were there any
objections to merging this? It doesn’t seem as though it ever went in, but
this was a year ago now.
-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics

[-- Attachment #2: Type: text/html, Size: 2354 bytes --]

Re: [PATCH 1/1] fontcache: fix fonts postinst failed

[Burton, Ross]

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

Ah, that would be why I have this in a local branch:

http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=ross/allarch

I rediscovered this on Friday...  personally I think my solution is better
as it's global instead of needing to be worked around in each place where
this can happen, but am open to debate on this.

Ross

On 27 November 2017 at 15:20, Christopher Larson <kergoth@gmail.com> wrote:

> On Tue, Oct 25, 2016 at 7:31 PM, Hongxu Jia <hongxu.jia@windriver.com>
> wrote:
>
>> While instal liberation-fonts to image, the fonts postinst
>> failed at do_rootfs time:
>> -----------------------------------------------
>> |WARNING: core-image-minimal-1.0-r0 do_rootfs: The postinstall intercept
>> hook
>> 'update_font_cache' failed (exit code: 139)! See log for details!
>> |WARNING: core-image-minimal-1.0-r0 do_rootfs: The postinstalls for the
>> following packages will be postponed for first boot: liberation-fonts
>> -----------------------------------------------
>>
>> Since the following commit applied in oe-core
>> -----------------------------------------------
>> commit ce1e7fcc60276040477c1d5e3129e029bb9f204b
>> Author: Richard Purdie <richard.purdie@linuxfoundation.org>
>> Date:   Fri Sep 30 17:43:23 2016 +0100
>>
>>     allarch: Fixes to stop rebuilds when change multilibs
>> -----------------------------------------------
>> It assignes base_libdir="/none" which caused fonts postinst failed at
>> do_rootfs.
>>
>> The fonts postinst invoke qemu to update font cache which requires
>> base_libdir arch relative(such as base_libdir="/lib64").
>>
>> We refer what meta/conf/multilib.conf does, assign it with arch relative.
>>
>> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
>
>
> Without this, we get the mentioned warnings with every qemux86-64 or
> intel-corei7-64 build using our external toolchain. Were there any
> objections to merging this? It doesn’t seem as though it ever went in, but
> this was a year ago now.
> --
> Christopher Larson
> kergoth at gmail dot com
> Founder - BitBake, OpenEmbedded, OpenZaurus
> Senior Software Engineer, Mentor Graphics
>

[-- Attachment #2: Type: text/html, Size: 3388 bytes --]

Re: [PATCH 1/1] fontcache: fix fonts postinst failed

[Christopher Larson]

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

On Mon, Nov 27, 2017 at 9:03 AM, Burton, Ross <ross.burton@intel.com> wrote:

> Ah, that would be why I have this in a local branch:
>
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/
> commit/?h=ross/allarch
>
> I rediscovered this on Friday...  personally I think my solution is better
> as it's global instead of needing to be worked around in each place where
> this can happen, but am open to debate on this.
>

Agreed, yours does seem cleaner, as long as there aren’t unintended
consequences. I can’t say I’m clear on the issues relating to allarch
sharing amongst multilib configurations.
-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics

[-- Attachment #2: Type: text/html, Size: 1379 bytes --]