Gentoo Archives: gentoo-dev

From: "Marty E. Plummer" <hanetzer@×××××××××.com>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] sys-libs/ncurses: erronious deletion of *.dll.a files; possibly other packages affected
Date: Fri, 29 Sep 2017 01:39:20
Message-Id: 20170929013637.gz7ntmg4fb62mzel@proprietary-killer.fossland
In Reply to: Re: [gentoo-dev] sys-libs/ncurses: erronious deletion of *.dll.a files; possibly other packages affected by Mike Gilbert
1 On Thu, Sep 28, 2017 at 07:35:20PM +0000, Mike Gilbert wrote:
2 > On Wed, Sep 20, 2017 at 10:01 PM, Marty E. Plummer
3 > <hanetzer@×××××××××.com> wrote:
4 > > arfrever suggests I send a mail here, as there are other packages which
5 > > may be affected by this issue and perhaps a more generalized fix is
6 > > required instead of an explicit fix in sys-libs/ncurses and other ebuilds
7 > > that may require it.
8 >
9 > I think the solution here is to remove those overly broad "find
10 > -delete" statements and replace them with something safer.
11 >
12 > Ideally the build system(s) would be patched to not compile static
13 > libs in the first place.
14 >
15 > If that's not possible, perhaps an eclass function could be created to
16 > safely remove static libs.
17 >
18 Honestly I already have a pr up that fixes this particular package's
19 issue, fairly simple fix https://github.com/gentoo/gentoo/pull/5734
20
21 --- a/sys-libs/ncurses/ncurses-6.0-r1.ebuild
22 +++ b/sys-libs/ncurses/ncurses-6.0-r1.ebuild
23 @@ -241,7 +241,8 @@ multilib_src_install() {
24 # Provide a link for -lcurses.
25 ln -sf libncurses$(get_libname) "${ED}"/usr/$(get_libdir)/libcurses$(get_libname) || die
26 fi
27 - use static-libs || find "${ED}"/usr/ -name '*.a' -delete
28 + # don't delete '*.dll.a', needed for linking #631468
29 + use static-libs || find "${ED}"/usr/ -name '*.a' ! -name '*.dll.a' -delete
30
31 # Build fails to create this ...
32 dosym ../share/terminfo /usr/$(get_libdir)/terminfo
33
34 and done as far as that goes.
35
36 Only other issue is the bad symlinking in the original bug report.

Replies