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. |