commit:     e14229b10b513a164f8379ff14cc8c644c071f27
Author:     Sergei Trofimovich <slyfox <AT> gentoo <DOT> org>
AuthorDate: Wed Sep 25 07:27:26 2019 +0000
Commit:     Sergei Trofimovich <slyfox <AT> gentoo <DOT> org>
CommitDate: Wed Sep 25 08:04:44 2019 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=e14229b1

sys-libs/glibc: drop prepallstrip, bug #587296

Stop using internal portage's 'prepallstrip' helper.
This effectively reverts https://bugs.gentoo.org/46186.
Modern gdb can inspect threads without debugging symbols
in libpthread/libthread_db.

Reported-by: Michał Górny
Closes: https://bugs.gentoo.org/587296
Bug: https://bugs.gentoo.org/46186
Package-Manager: Portage-2.3.76, Repoman-2.3.17
Signed-off-by: Sergei Trofimovich <slyfox <AT> gentoo.org>

 sys-libs/glibc/glibc-2.19-r2.ebuild  | 19 -------------------
 sys-libs/glibc/glibc-2.24-r4.ebuild  |  3 +--
 sys-libs/glibc/glibc-2.25-r11.ebuild |  3 +--
 sys-libs/glibc/glibc-2.26-r7.ebuild  |  4 +---
 sys-libs/glibc/glibc-2.27-r6.ebuild  | 19 -------------------
 sys-libs/glibc/glibc-2.28-r6.ebuild  | 19 -------------------
 sys-libs/glibc/glibc-2.29-r2.ebuild  | 19 -------------------
 sys-libs/glibc/glibc-2.29-r4.ebuild  | 19 -------------------
 sys-libs/glibc/glibc-2.29-r5.ebuild  | 19 -------------------
 sys-libs/glibc/glibc-2.30-r1.ebuild  | 20 --------------------
 sys-libs/glibc/glibc-2.30.ebuild     | 20 --------------------
 sys-libs/glibc/glibc-9999.ebuild     |  5 -----
 12 files changed, 3 insertions(+), 166 deletions(-)

diff --git a/sys-libs/glibc/glibc-2.19-r2.ebuild 
b/sys-libs/glibc/glibc-2.19-r2.ebuild
index 42ed9d4e04d..75f849f69dc 100644
--- a/sys-libs/glibc/glibc-2.19-r2.ebuild
+++ b/sys-libs/glibc/glibc-2.19-r2.ebuild
@@ -9,7 +9,6 @@ inherit prefix eutils toolchain-funcs flag-o-matic gnuconfig 
usr-ldscript \
 DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 SLOT="2.2"
 
 EMULTILIB_PKG="true"
@@ -1197,23 +1196,6 @@ glibc_headers_install() {
        dosym usr/include $(alt_prefix)/sys-include
 }
 
-src_strip() {
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread, so we need to strip it by hand.  libthread_db makes no
-       # sense stripped as it is only used when debugging.
-       local pthread=$(has splitdebug ${FEATURES} && echo "libthread_db" || 
echo "lib{pthread,thread_db}")
-       env \
-               -uRESTRICT \
-               CHOST=${CTARGET} \
-               STRIP_MASK="/*/{,tls/}${pthread}*" \
-               prepallstrip
-       # if user has stripping enabled and does not have split debug turned on,
-       # then leave the debugging sections in libpthread.
-       if ! has nostrip ${FEATURES} && ! has splitdebug ${FEATURES} ; then
-               ${STRIP:-${CTARGET}-strip} --strip-debug 
"${ED}"/*/libpthread-*.so
-       fi
-}
-
 src_install() {
        if just_headers ; then
                export ABI=default
@@ -1222,7 +1204,6 @@ src_install() {
        fi
 
        foreach_abi glibc_do_src_install
-       src_strip
 }
 
 # Simple test to make sure our new glibc isn't completely broken.

diff --git a/sys-libs/glibc/glibc-2.24-r4.ebuild 
b/sys-libs/glibc/glibc-2.24-r4.ebuild
index 40f30a0b8eb..b808611bd31 100644
--- a/sys-libs/glibc/glibc-2.24-r4.ebuild
+++ b/sys-libs/glibc/glibc-2.24-r4.ebuild
@@ -1,4 +1,4 @@
-# Copyright 1999-2018 Gentoo Foundation
+# Copyright 1999-2019 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI="5"
@@ -10,7 +10,6 @@ HOMEPAGE="https://www.gnu.org/software/libc/libc.html";
 
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
 KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~m68k ~mips ~ppc ~ppc64 ~s390 
~sh ~sparc ~x86"
-RESTRICT="strip" # strip ourself #46186
 EMULTILIB_PKG="true"
 
 # Configuration variables

diff --git a/sys-libs/glibc/glibc-2.25-r11.ebuild 
b/sys-libs/glibc/glibc-2.25-r11.ebuild
index 67c8f98ebb7..e2a0f9131bb 100644
--- a/sys-libs/glibc/glibc-2.25-r11.ebuild
+++ b/sys-libs/glibc/glibc-2.25-r11.ebuild
@@ -1,4 +1,4 @@
-# Copyright 1999-2018 Gentoo Foundation
+# Copyright 1999-2019 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI="5"
@@ -10,7 +10,6 @@ HOMEPAGE="https://www.gnu.org/software/libc/libc.html";
 
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
 KEYWORDS="alpha amd64 arm arm64 hppa ia64 m68k ~mips ppc ppc64 s390 sh sparc 
x86"
-RESTRICT="strip" # strip ourself #46186
 EMULTILIB_PKG="true"
 
 # Configuration variables

diff --git a/sys-libs/glibc/glibc-2.26-r7.ebuild 
b/sys-libs/glibc/glibc-2.26-r7.ebuild
index 7caf7ba188c..bfbe9b01b03 100644
--- a/sys-libs/glibc/glibc-2.26-r7.ebuild
+++ b/sys-libs/glibc/glibc-2.26-r7.ebuild
@@ -1,4 +1,4 @@
-# Copyright 1999-2018 Gentoo Authors
+# Copyright 1999-2019 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI=6
@@ -9,7 +9,6 @@ DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 EMULTILIB_PKG="true"
 
 # Configuration variables
@@ -784,7 +783,6 @@ src_install() {
        fi
 
        foreach_abi glibc_do_src_install
-       src_strip
 }
 
 pkg_preinst() {

diff --git a/sys-libs/glibc/glibc-2.27-r6.ebuild 
b/sys-libs/glibc/glibc-2.27-r6.ebuild
index fc6931ffb7c..42be8144de4 100644
--- a/sys-libs/glibc/glibc-2.27-r6.ebuild
+++ b/sys-libs/glibc/glibc-2.27-r6.ebuild
@@ -9,7 +9,6 @@ inherit prefix eutils versionator toolchain-funcs flag-o-matic 
gnuconfig usr-lds
 DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 SLOT="2.2"
 
 EMULTILIB_PKG="true"
@@ -1303,23 +1302,6 @@ glibc_headers_install() {
        dosym usr/include $(alt_prefix)/sys-include
 }
 
-src_strip() {
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread, so we need to strip it by hand.  libthread_db makes no
-       # sense stripped as it is only used when debugging.
-       local pthread=$(has splitdebug ${FEATURES} && echo "libthread_db" || 
echo "lib{pthread,thread_db}")
-       env \
-               -uRESTRICT \
-               CHOST=${CTARGET} \
-               STRIP_MASK="/*/{,tls/}${pthread}*" \
-               prepallstrip
-       # if user has stripping enabled and does not have split debug turned on,
-       # then leave the debugging sections in libpthread.
-       if ! has nostrip ${FEATURES} && ! has splitdebug ${FEATURES} ; then
-               ${STRIP:-${CTARGET}-strip} --strip-debug 
"${ED}"$(alt_prefix)/*/libpthread-*.so
-       fi
-}
-
 src_install() {
        if just_headers ; then
                export ABI=default
@@ -1328,7 +1310,6 @@ src_install() {
        fi
 
        foreach_abi glibc_do_src_install
-       src_strip
 }
 
 # Simple test to make sure our new glibc isn't completely broken.

diff --git a/sys-libs/glibc/glibc-2.28-r6.ebuild 
b/sys-libs/glibc/glibc-2.28-r6.ebuild
index 761c007900b..d40991f16ab 100644
--- a/sys-libs/glibc/glibc-2.28-r6.ebuild
+++ b/sys-libs/glibc/glibc-2.28-r6.ebuild
@@ -9,7 +9,6 @@ inherit prefix eutils eapi7-ver toolchain-funcs flag-o-matic 
gnuconfig usr-ldscr
 DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 SLOT="2.2"
 
 EMULTILIB_PKG="true"
@@ -1301,23 +1300,6 @@ glibc_headers_install() {
        dosym usr/include $(alt_prefix)/sys-include
 }
 
-src_strip() {
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread, so we need to strip it by hand.  libthread_db makes no
-       # sense stripped as it is only used when debugging.
-       local pthread=$(has splitdebug ${FEATURES} && echo "libthread_db" || 
echo "lib{pthread,thread_db}")
-       env \
-               -uRESTRICT \
-               CHOST=${CTARGET} \
-               STRIP_MASK="/*/{,tls/}${pthread}*" \
-               prepallstrip
-       # if user has stripping enabled and does not have split debug turned on,
-       # then leave the debugging sections in libpthread.
-       if ! has nostrip ${FEATURES} && ! has splitdebug ${FEATURES} ; then
-               ${STRIP:-${CTARGET}-strip} --strip-debug 
"${ED}"$(alt_prefix)/*/libpthread-*.so
-       fi
-}
-
 src_install() {
        if just_headers ; then
                export ABI=default
@@ -1326,7 +1308,6 @@ src_install() {
        fi
 
        foreach_abi glibc_do_src_install
-       src_strip
 }
 
 # Simple test to make sure our new glibc isn't completely broken.

diff --git a/sys-libs/glibc/glibc-2.29-r2.ebuild 
b/sys-libs/glibc/glibc-2.29-r2.ebuild
index 5352ae1f92b..b80ddd00814 100644
--- a/sys-libs/glibc/glibc-2.29-r2.ebuild
+++ b/sys-libs/glibc/glibc-2.29-r2.ebuild
@@ -11,7 +11,6 @@ inherit python-any-r1 prefix eutils eapi7-ver toolchain-funcs 
flag-o-matic gnuco
 DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 SLOT="2.2"
 
 EMULTILIB_PKG="true"
@@ -1343,23 +1342,6 @@ glibc_headers_install() {
        dosym usr/include $(alt_prefix)/sys-include
 }
 
-src_strip() {
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread, so we need to strip it by hand.  libthread_db makes no
-       # sense stripped as it is only used when debugging.
-       local pthread=$(has splitdebug ${FEATURES} && echo "libthread_db" || 
echo "lib{pthread,thread_db}")
-       env \
-               -uRESTRICT \
-               CHOST=${CTARGET} \
-               STRIP_MASK="/*/{,tls/}${pthread}*" \
-               prepallstrip
-       # if user has stripping enabled and does not have split debug turned on,
-       # then leave the debugging sections in libpthread.
-       if ! has nostrip ${FEATURES} && ! has splitdebug ${FEATURES} ; then
-               ${STRIP:-${CTARGET}-strip} --strip-debug 
"${ED}"$(alt_prefix)/*/libpthread-*.so
-       fi
-}
-
 src_install() {
        if just_headers ; then
                export ABI=default
@@ -1368,7 +1350,6 @@ src_install() {
        fi
 
        foreach_abi glibc_do_src_install
-       src_strip
 }
 
 # Simple test to make sure our new glibc isn't completely broken.

diff --git a/sys-libs/glibc/glibc-2.29-r4.ebuild 
b/sys-libs/glibc/glibc-2.29-r4.ebuild
index c49d8263726..7282d89d8ee 100644
--- a/sys-libs/glibc/glibc-2.29-r4.ebuild
+++ b/sys-libs/glibc/glibc-2.29-r4.ebuild
@@ -11,7 +11,6 @@ inherit python-any-r1 prefix eutils eapi7-ver toolchain-funcs 
flag-o-matic gnuco
 DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 SLOT="2.2"
 
 EMULTILIB_PKG="true"
@@ -1356,23 +1355,6 @@ glibc_headers_install() {
        dosym usr/include $(alt_prefix)/sys-include
 }
 
-src_strip() {
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread, so we need to strip it by hand.  libthread_db makes no
-       # sense stripped as it is only used when debugging.
-       local pthread=$(has splitdebug ${FEATURES} && echo "libthread_db" || 
echo "lib{pthread,thread_db}")
-       env \
-               -uRESTRICT \
-               CHOST=${CTARGET} \
-               STRIP_MASK="/*/{,tls/}${pthread}*" \
-               prepallstrip
-       # if user has stripping enabled and does not have split debug turned on,
-       # then leave the debugging sections in libpthread.
-       if ! has nostrip ${FEATURES} && ! has splitdebug ${FEATURES} ; then
-               ${STRIP:-${CTARGET}-strip} --strip-debug 
"${ED}"$(alt_prefix)/*/libpthread-*.so
-       fi
-}
-
 src_install() {
        if just_headers ; then
                export ABI=default
@@ -1381,7 +1363,6 @@ src_install() {
        fi
 
        foreach_abi glibc_do_src_install
-       src_strip
 }
 
 # Simple test to make sure our new glibc isn't completely broken.

diff --git a/sys-libs/glibc/glibc-2.29-r5.ebuild 
b/sys-libs/glibc/glibc-2.29-r5.ebuild
index d02d0fcb97d..1c3f4a15dde 100644
--- a/sys-libs/glibc/glibc-2.29-r5.ebuild
+++ b/sys-libs/glibc/glibc-2.29-r5.ebuild
@@ -11,7 +11,6 @@ inherit python-any-r1 prefix eutils eapi7-ver toolchain-funcs 
flag-o-matic gnuco
 DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 SLOT="2.2"
 
 EMULTILIB_PKG="true"
@@ -1356,23 +1355,6 @@ glibc_headers_install() {
        dosym usr/include $(alt_prefix)/sys-include
 }
 
-src_strip() {
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread, so we need to strip it by hand.  libthread_db makes no
-       # sense stripped as it is only used when debugging.
-       local pthread=$(has splitdebug ${FEATURES} && echo "libthread_db" || 
echo "lib{pthread,thread_db}")
-       env \
-               -uRESTRICT \
-               CHOST=${CTARGET} \
-               STRIP_MASK="/*/{,tls/}${pthread}*" \
-               prepallstrip
-       # if user has stripping enabled and does not have split debug turned on,
-       # then leave the debugging sections in libpthread.
-       if ! has nostrip ${FEATURES} && ! has splitdebug ${FEATURES} ; then
-               ${STRIP:-${CTARGET}-strip} --strip-debug 
"${ED}"$(alt_prefix)/*/libpthread-*.so
-       fi
-}
-
 src_install() {
        if just_headers ; then
                export ABI=default
@@ -1381,7 +1363,6 @@ src_install() {
        fi
 
        foreach_abi glibc_do_src_install
-       src_strip
 }
 
 # Simple test to make sure our new glibc isn't completely broken.

diff --git a/sys-libs/glibc/glibc-2.30-r1.ebuild 
b/sys-libs/glibc/glibc-2.30-r1.ebuild
index 2846ee6a61c..b3da5c0345e 100644
--- a/sys-libs/glibc/glibc-2.30-r1.ebuild
+++ b/sys-libs/glibc/glibc-2.30-r1.ebuild
@@ -11,7 +11,6 @@ inherit python-any-r1 prefix eutils eapi7-ver toolchain-funcs 
flag-o-matic gnuco
 DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 SLOT="2.2"
 
 EMULTILIB_PKG="true"
@@ -1363,23 +1362,6 @@ glibc_headers_install() {
        dosym usr/include $(alt_prefix)/sys-include
 }
 
-src_strip() {
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread, so we need to strip it by hand.  libthread_db makes no
-       # sense stripped as it is only used when debugging.
-       local pthread=$(has splitdebug ${FEATURES} && echo "libthread_db" || 
echo "lib{pthread,thread_db}")
-       env \
-               -uRESTRICT \
-               CHOST=${CTARGET} \
-               STRIP_MASK="/*/{,tls/}${pthread}*" \
-               prepallstrip
-       # if user has stripping enabled and does not have split debug turned on,
-       # then leave the debugging sections in libpthread.
-       if ! has nostrip ${FEATURES} && ! has splitdebug ${FEATURES} ; then
-               ${STRIP:-${CTARGET}-strip} --strip-debug 
"${ED}"$(alt_prefix)/*/libpthread-*.so
-       fi
-}
-
 src_install() {
        if just_headers ; then
                export ABI=default
@@ -1393,8 +1375,6 @@ src_install() {
                elog "Not installing static glibc libraries"
                find "${ED}" -name "*.a" -and -not -name "*_nonshared.a" -delete
        fi
-
-       src_strip
 }
 
 # Simple test to make sure our new glibc isn't completely broken.

diff --git a/sys-libs/glibc/glibc-2.30.ebuild b/sys-libs/glibc/glibc-2.30.ebuild
index 743f14c253d..5a8933d5d96 100644
--- a/sys-libs/glibc/glibc-2.30.ebuild
+++ b/sys-libs/glibc/glibc-2.30.ebuild
@@ -11,7 +11,6 @@ inherit python-any-r1 prefix eutils eapi7-ver toolchain-funcs 
flag-o-matic gnuco
 DESCRIPTION="GNU libc C library"
 HOMEPAGE="https://www.gnu.org/software/libc/";
 LICENSE="LGPL-2.1+ BSD HPND ISC inner-net rc PCRE"
-RESTRICT="strip" # Strip ourself #46186
 SLOT="2.2"
 
 EMULTILIB_PKG="true"
@@ -1363,23 +1362,6 @@ glibc_headers_install() {
        dosym usr/include $(alt_prefix)/sys-include
 }
 
-src_strip() {
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread, so we need to strip it by hand.  libthread_db makes no
-       # sense stripped as it is only used when debugging.
-       local pthread=$(has splitdebug ${FEATURES} && echo "libthread_db" || 
echo "lib{pthread,thread_db}")
-       env \
-               -uRESTRICT \
-               CHOST=${CTARGET} \
-               STRIP_MASK="/*/{,tls/}${pthread}*" \
-               prepallstrip
-       # if user has stripping enabled and does not have split debug turned on,
-       # then leave the debugging sections in libpthread.
-       if ! has nostrip ${FEATURES} && ! has splitdebug ${FEATURES} ; then
-               ${STRIP:-${CTARGET}-strip} --strip-debug 
"${ED}"$(alt_prefix)/*/libpthread-*.so
-       fi
-}
-
 src_install() {
        if just_headers ; then
                export ABI=default
@@ -1393,8 +1375,6 @@ src_install() {
                elog "Not installing static glibc libraries"
                find "${ED}" -name "*.a" -and -not -name "*_nonshared.a" -delete
        fi
-
-       src_strip
 }
 
 # Simple test to make sure our new glibc isn't completely broken.

diff --git a/sys-libs/glibc/glibc-9999.ebuild b/sys-libs/glibc/glibc-9999.ebuild
index cd66539a922..d529e346178 100644
--- a/sys-libs/glibc/glibc-9999.ebuild
+++ b/sys-libs/glibc/glibc-9999.ebuild
@@ -1181,11 +1181,6 @@ glibc_do_src_install() {
        # '#define VERSION "2.26.90"' -> '2.26.90'
        local upstream_pv=$(sed -n -r 's/#define VERSION "(.*)"/\1/p' 
"${S}"/version.h)
 
-       # gdb is lame and requires some debugging information to remain in
-       # libpthread.  libthread_db makes no sense stripped as it is only used 
when debugging.
-       dostrip -x $(alt_libdir)/libpthread-${upstream_pv}.so
-       dostrip -x $(alt_libdir)/libthread_db-1.0.so
-
        if [[ -e ${ED}/$(alt_usrlibdir)/libm-${upstream_pv}.a ]] ; then
                # Move versioned .a file out of libdir to evade portage QA 
checks
                # instead of using gen_usr_ldscript(). We fix ldscript as:

Reply via email to