diff options
author | Ioan-Adrian Ratiu <adrian.ratiu@ni.com> | 2018-07-20 14:18:16 +0300 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-26 13:16:40 +0100 |
commit | 62f5f01b5bf4287a44f9f89db5c5d65bfd9873ad (patch) | |
tree | 7250ad062a98196b806d9446b4bd417799e2566e | |
parent | 594c3d9a424b047e9002ae1dfcf2c049042e8640 (diff) | |
download | poky-62f5f01b5bf4287a44f9f89db5c5d65bfd9873ad.tar.gz |
lsb: fix usrmerge install paths
${base_prefix} is set in bitbake.conf to empty. This makes lsb_release
always install under /bin which is a problem if usrmerge is in
DISTRO_FEATURES, because it needs to be installed under /usr/bin.
By using ${root_prefix} instead, we fix the usrmerge install path and
the following QA warning goes away while keeping the non-usrmerge path
identical.
WARNING: lsb-5.0-r0 do_package: QA Issue: lsb: Files/directories were
installed but not shipped in any package:
/bin
/bin/lsb_release
Please set FILES such that these items are packaged. Alternatively
if they are unneeded, avoid installing them or delete them within do_install.
lsb: 2 installed and not shipped files. [installed-vs-shipped]
Also if usrmerge is defined don't try to create the lib64 symlink
because base-files handles it to avoid the do_rootfs error
Collected errors:
* check_data_file_clashes: Package lsb wants to install file
tmp-glibc/work/x64-nilrt-linux/test-image/1.0-r0/rootfs/lib64
But that file is already provided by package * base-files
(From OE-Core rev: 852c9498ae365bc8c149b5e6df0c8667bd885c79)
Signed-off-by: Ioan-Adrian Ratiu <adrian.ratiu@ni.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/recipes-extended/lsb/lsb_5.0.bb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/meta/recipes-extended/lsb/lsb_5.0.bb b/meta/recipes-extended/lsb/lsb_5.0.bb index df4812e4bc..d1de334008 100644 --- a/meta/recipes-extended/lsb/lsb_5.0.bb +++ b/meta/recipes-extended/lsb/lsb_5.0.bb | |||
@@ -34,7 +34,7 @@ S = "${WORKDIR}/lsb-release-1.4" | |||
34 | CLEANBROKEN = "1" | 34 | CLEANBROKEN = "1" |
35 | 35 | ||
36 | do_install() { | 36 | do_install() { |
37 | oe_runmake install prefix=${D}${base_prefix} mandir=${D}${datadir}/man/ DESTDIR=${D} | 37 | oe_runmake install prefix=${D}${root_prefix} mandir=${D}${datadir}/man/ DESTDIR=${D} |
38 | 38 | ||
39 | # these two dirs are needed by package lsb-dist-checker | 39 | # these two dirs are needed by package lsb-dist-checker |
40 | mkdir -p ${D}${sysconfdir}/opt | 40 | mkdir -p ${D}${sysconfdir}/opt |
@@ -99,7 +99,9 @@ do_install_append() { | |||
99 | fi | 99 | fi |
100 | 100 | ||
101 | if [ "${TARGET_ARCH}" = "x86_64" ]; then | 101 | if [ "${TARGET_ARCH}" = "x86_64" ]; then |
102 | if [ "${base_libdir}" != "${base_prefix}/lib64" ]; then | 102 | # don't symlink if usrmerge is in DISTRO_FEATURES as it manages the symlink |
103 | if ${@bb.utils.contains('DISTRO_FEATURES','usrmerge','false','true',d)} && \ | ||
104 | [ "${base_libdir}" != "${base_prefix}/lib64" ]; then | ||
103 | lnr ${D}${base_libdir} ${D}${base_prefix}/lib64 | 105 | lnr ${D}${base_libdir} ${D}${base_prefix}/lib64 |
104 | fi | 106 | fi |
105 | cd ${D}${base_libdir} | 107 | cd ${D}${base_libdir} |