commit: 0e95698ff9584aa8045275705c0247a77264d044
Author: Matija Skala <mskala <AT> gmx <DOT> com>
AuthorDate: Thu Nov 9 16:49:37 2017 +0000
Commit: Sergei Trofimovich <slyfox <AT> gentoo <DOT> org>
CommitDate: Sun Dec 31 21:26:29 2017 +0000
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0e95698f
sys-libs/glibc: avoid libm-2.26.so symlink clash on merged /usr, bug #627378
The problem in bug #627378 manifests as libm-2.26.so file corruption:
Before the change glibc package contained a 'libm-2.26.so'
symlink from '/usr/lib64' to '/lib64':
$ equery f sys-libs/glibc | sed 's@usr/lib@lib@g' | sort | uniq -d
/lib64/libm-2.26.so
When both are the same directory all depends on the merge order:
- symlink first, then real file. real file overwrites symlink, all is good
- real file first, then symlink. symlink overwrites the file and points to
itself. Binaries linked against libm fail to start.
The change is to get rid of symlink (symlink was a workaround to portage's
QA check) and move 'libm-2.26.a' from '/usr/lib64' to '/usr/lib64/glibc-<pv>'.
Reported-by: Duncan
Fixed-by: Matija Skala
Closes: https://bugs.gentoo.org/627378
Closes: https://github.com/gentoo/gentoo/pull/4268
sys-libs/glibc/{glibc-2.26-r4.ebuild => glibc-2.26-r5.ebuild} | 11 +++++++----
sys-libs/glibc/glibc-9999.ebuild | 11 +++++++----
2 files changed, 14 insertions(+), 8 deletions(-)
diff --git a/sys-libs/glibc/glibc-2.26-r4.ebuild
b/sys-libs/glibc/glibc-2.26-r5.ebuild
similarity index 97%
rename from sys-libs/glibc/glibc-2.26-r4.ebuild
rename to sys-libs/glibc/glibc-2.26-r5.ebuild
index 97cf7ef1cbc..abde9e62f95 100644
--- a/sys-libs/glibc/glibc-2.26-r4.ebuild
+++ b/sys-libs/glibc/glibc-2.26-r5.ebuild
@@ -607,15 +607,18 @@ glibc_do_src_install() {
find "${D}" -name "libnsl.a" -delete
find "${D}" -name "libnsl.so" -delete
- # Normally real_pv is ${PV}. Live ebuilds are exception, there we need
+ # Normally upstream_pv is ${PV}. Live ebuilds are exception, there we
need
# to infer upstream version:
# '#define VERSION "2.26.90"' -> '2.26.90'
local upstream_pv=$(sed -n -r 's/#define VERSION "(.*)"/\1/p'
"${S}"/version.h)
- # Newer versions get fancy with libm linkage to include vectorized
support.
- # While we don't really need a ldscript here, portage QA checks get
upset.
if [[ -e ${ED}$(alt_usrlibdir)/libm-${upstream_pv}.a ]] ; then
- dosym ../../$(get_libdir)/libm-${upstream_pv}.so
$(alt_usrlibdir)/libm-${upstream_pv}.so
+ # Move versioned .a file out of libdir to evade portage QA
checks
+ # instead of using gen_usr_ldscript(). We fix ldscript as:
+ # "GROUP ( /usr/lib64/libm-<pv>.a ..." -> "GROUP (
/usr/lib64/glibc-<pv>/libm-<pv>.a ..."
+ sed -i "s@\(libm-${upstream_pv}.a\)@${P}/\1@"
"${ED}"$(alt_usrlibdir)/libm.a || die
+ dodir $(alt_usrlibdir)/${P}
+ mv "${ED}"$(alt_usrlibdir)/libm-${upstream_pv}.a
"${ED}"$(alt_usrlibdir)/${P}/libm-${upstream_pv}.a || die
fi
# We'll take care of the cache ourselves
diff --git a/sys-libs/glibc/glibc-9999.ebuild b/sys-libs/glibc/glibc-9999.ebuild
index a5bf3b51ebc..2aed938db02 100644
--- a/sys-libs/glibc/glibc-9999.ebuild
+++ b/sys-libs/glibc/glibc-9999.ebuild
@@ -614,15 +614,18 @@ glibc_do_src_install() {
find "${D}" -name "libnsl.a" -delete
find "${D}" -name "libnsl.so" -delete
- # Normally real_pv is ${PV}. Live ebuilds are exception, there we need
+ # Normally upstream_pv is ${PV}. Live ebuilds are exception, there we
need
# to infer upstream version:
# '#define VERSION "2.26.90"' -> '2.26.90'
local upstream_pv=$(sed -n -r 's/#define VERSION "(.*)"/\1/p'
"${S}"/version.h)
- # Newer versions get fancy with libm linkage to include vectorized
support.
- # While we don't really need a ldscript here, portage QA checks get
upset.
if [[ -e ${ED}$(alt_usrlibdir)/libm-${upstream_pv}.a ]] ; then
- dosym ../../$(get_libdir)/libm-${upstream_pv}.so
$(alt_usrlibdir)/libm-${upstream_pv}.so
+ # Move versioned .a file out of libdir to evade portage QA
checks
+ # instead of using gen_usr_ldscript(). We fix ldscript as:
+ # "GROUP ( /usr/lib64/libm-<pv>.a ..." -> "GROUP (
/usr/lib64/glibc-<pv>/libm-<pv>.a ..."
+ sed -i "s@\(libm-${upstream_pv}.a\)@${P}/\1@"
"${ED}"$(alt_usrlibdir)/libm.a || die
+ dodir $(alt_usrlibdir)/${P}
+ mv "${ED}"$(alt_usrlibdir)/libm-${upstream_pv}.a
"${ED}"$(alt_usrlibdir)/${P}/libm-${upstream_pv}.a || die
fi
# We'll take care of the cache ourselves