[PATCH] liberation-fonts/cantarell-fonts: fix postinst failed warning

[PATCH] liberation-fonts/cantarell-fonts: fix postinst failed warning

[changqing.li]

From: Changqing Li <changqing.li@windriver.com>

font is not arch related, but these receipes inherit fontcache,
fontcache called update_font_cache, update_font_cache use qemuwrapper,
qemuwrapper is arch related, so these receipes cannot inherit allarch,
or update_font_cache will core dumped as below:

WARNING: The postinstall intercept hook 'update_font_cache' failed,
NOTE: Exit code 132. Output:
qemu: uncaught target signal 4 (Illegal instruction) - core dumped
qemuwrapper: line 5: 267705 Illegal instruction     (core dumped) qemu-x86_64 -r 3.2.0 -cpu core2duo "$@"

in update_font_cache, it runs:
qemuwrapper -L $D -E LD_LIBRARY_PATH=$D/${libdir}:$D/${base_libdir} -E ${fontconfigcacheenv} $D${bindir}/fc-cache --sysroot=$D --system-only

qemuwrapper sets LD_LIBRARY_PATH=$D/${libdir}:$D/${base_libdir} for fc-cache,
if inherit allarch, ${base_libdir} is "/lib", ${libdir} is "/usr/lib",
but in this case, fc-cache need /lib64 and /usr/lib64 to run,
So fc-cache doesn't work, and caused the warning.
so remove allarch from these two recipes who both inherit allarch and fontcahe to fix this problem.

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb | 2 +-
 meta/recipes-graphics/ttf-fonts/liberation-fonts_2.00.1.bb   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb b/meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb
index c71ab11..b5bc203 100644
--- a/meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb
+++ b/meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb
@@ -18,7 +18,7 @@ UPSTREAM_CHECK_GITTAGREGEX = "(?P<pver>(?!0\.13)(?!0\.10\.1)\d+\.\d+(\.\d+)+)"
 
 S = "${WORKDIR}/git"
 
-inherit autotools allarch fontcache pkgconfig
+inherit autotools fontcache pkgconfig
 
 PACKAGECONFIG ??= ""
 PACKAGECONFIG[fontforge] = "--enable-source-rebuild=yes,--enable-source-rebuild=no,fontforge-native"
diff --git a/meta/recipes-graphics/ttf-fonts/liberation-fonts_2.00.1.bb b/meta/recipes-graphics/ttf-fonts/liberation-fonts_2.00.1.bb
index 412da48..14919c8 100644
--- a/meta/recipes-graphics/ttf-fonts/liberation-fonts_2.00.1.bb
+++ b/meta/recipes-graphics/ttf-fonts/liberation-fonts_2.00.1.bb
@@ -10,7 +10,7 @@ LICENSE = "OFL-1.1"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=f96db970a9a46c5369142b99f530366b"
 PE = "1"
 
-inherit allarch fontcache
+inherit fontcache
 
 FONT_PACKAGES = "${PN}"
 
-- 
2.7.4

Re: [PATCH] liberation-fonts/cantarell-fonts: fix postinst failed warning

[Alexander Kanavin]

On 05/24/2018 05:18 AM, changqing.li@windriver.com wrote:
> in update_font_cache, it runs:
> qemuwrapper -L $D -E LD_LIBRARY_PATH=$D/${libdir}:$D/${base_libdir} -E ${fontconfigcacheenv} $D${bindir}/fc-cache --sysroot=$D --system-only
> 
> qemuwrapper sets LD_LIBRARY_PATH=$D/${libdir}:$D/${base_libdir} for fc-cache,
> if inherit allarch, ${base_libdir} is "/lib", ${libdir} is "/usr/lib",
> but in this case, fc-cache need /lib64 and /usr/lib64 to run,
> So fc-cache doesn't work, and caused the warning.
> so remove allarch from these two recipes who both inherit allarch and fontcahe to fix this problem.

We have discussed and attempted to fix this several times before, 
including the way you do above. Removing allarch is not right, as the 
content of the recipe is arch-independent and so should be built only once.

The right way to deal with this is to fix qemuwrapper-cross, so that it 
sets the library paths itself, rather than gets them from 
postinst_intercept.

Alex