Gentoo Archives: gentoo-portage-dev

From: Zac Medico <zmedico@g.o>
To: gentoo-portage-dev@l.g.o, "Michał Górny" <mgorny@g.o>, William Hubbs <williamh@g.o>
Cc: qa@g.o
Subject: Re: [gentoo-portage-dev] [PATCH] install-qa-check.d: remove check that bans libtool files and static libs from /
Date: Sun, 03 Nov 2019 20:18:46
Message-Id: 7f17e7b6-c1f9-ab7b-6096-7f748228a90f@gentoo.org
In Reply to: Re: [gentoo-portage-dev] [PATCH] install-qa-check.d: remove check that bans libtool files and static libs from / by "Michał Górny"
1 On 11/3/19 11:53 AM, Michał Górny wrote:
2 > On Sun, 2019-11-03 at 11:49 -0800, Zac Medico wrote:
3 >> On 10/27/19 10:40 AM, William Hubbs wrote:
4 >>> Most upstreams and build systems do not make this distinction, so this
5 >>> causes unnecessary hacks in ebuilds.
6 >>>
7 >>> Signed-off-by: William Hubbs <williamh@g.o>
8 >>> ---
9 >>> bin/install-qa-check.d/80libraries | 10 ----------
10 >>> 1 file changed, 10 deletions(-)
11 >>>
12 >>> diff --git a/bin/install-qa-check.d/80libraries b/bin/install-qa-check.d/80libraries
13 >>> index d1d2c4fdd..e59369bf6 100644
14 >>> --- a/bin/install-qa-check.d/80libraries
15 >>> +++ b/bin/install-qa-check.d/80libraries
16 >>> @@ -152,16 +152,6 @@ lib_check() {
17 >>> done
18 >>> [[ ${abort} == "yes" ]] && die "add those ldscripts"
19 >>>
20 >>> - # Make sure people don't store libtool files or static libs in /lib
21 >>> - f=$(ls "${ED%/}"/lib*/*.{a,la} 2>/dev/null)
22 >>> - if [[ -n ${f} ]] ; then
23 >>> - __vecho -ne '\n'
24 >>> - eqawarn "QA Notice: Excessive files found in the / partition"
25 >>> - eqawarn "${f}"
26 >>> - __vecho -ne '\n'
27 >>> - die "static archives (*.a) and libtool library files (*.la) belong in /usr/lib*, not /lib*"
28 >>> - fi
29 >>> -
30 >>> # Verify that the libtool files don't contain bogus $D entries.
31 >>> local abort=no gentoo_bug=no always_overflow=no
32 >>> for a in "${ED%/}"/usr/lib*/*.la ; do
33 >>>
34 >>
35 >> Merged. Thanks!
36 >>
37 >> https://gitweb.gentoo.org/proj/portage.git/commit/?id=498900e5e51460502d8271f409a4c614a021613b
38 >>
39 >
40 > Please revert this. I should point out that this has been vetoed
41 > by a QA member, and is currently subject to QA vote. Therefore, I
42 > believe you shouldn't be making rash decisions based on patches
43 > submitted by a single developer. Especially given that so far nobody
44 > else has voiced his opinion either way, so it's 1:1.
45
46 Thanks for the info. Reverted:
47
48 https://gitweb.gentoo.org/proj/portage.git/commit/?id=ae2a76261fd7e519bfbb3b8c4b41aa511d039245
49 --
50 Thanks,
51 Zac

Attachments

File name MIME type
signature.asc application/pgp-signature

Replies