On Fri, Nov 08, 2019 at 05:58:17PM +0100, Charlene Wendling wrote:
> On Fri, 8 Nov 2019 17:25:17 +0100
> Antoine Jacoutot wrote:
>
> > On Fri, Nov 08, 2019 at 05:01:03PM +0100, Charlene Wendling wrote:
> > > On Fri, 8 Nov 2019 16:30:58 +0100
> > > Antoine Jacoutot wrote:
> > >
> > > > On Fri, Nov 08, 2019 at 04:09:50PM +0100, Charlene Wendling wrote:
> > > > > Hi,
> > > > >
> > > > > > http://build-failures.rhaalovely.net//sparc64/last/x11/xkbcommon.log
> > > > > > https://bin.charlenew.xyz/xkbcommon.fail.log
> > > > > (no bulk report yet on macppc)
> > > > >
> > > > > ld.bfd is unable to find libraries again. The issue here is that
> > > > > a few hundreds ports are depending on it.
> > > > >
> > > > > Because meson is used, i tried at first to disable
> > > > > "-Wl,--as-needed", as it saved the day on some other ports,
> > > > > without any success.
> > > > >
> > > > > I'm proposing to manually add proper LDFLAGS for libs that
> > > > > cannot be found. Because it hides something, i did it in a way
> > > > > the issue is still known for further work on it. I don't like
> > > > > it, but given the importance of this port, i think this change
> > > > > should go in, this way or another.
> > > > >
> > > > > With the following diff, i was able to fix the build [0], build
> > > > > qt5 and use qtfm without issues.
> > > > >
> > > > > Comments/feedback are welcome,
> > > > >
> > > > > Charlène.
> > > > >
> > > > >
> > > > > [0] https://bin.charlenew.xyz/xkbcommon.log
> > > >
> > > >
> > > > Would this work? (it's simpler imho):
> > > > CONFIGURE_ENV= LDFLAGS="-L${X11BASE}/lib"
> > >
> > > It sure works [1]!
> > >
> > > Charlène.
> > >
> > >
> > > [1] https://bin.charlenew.xyz/xkbcommon.simpler.log
> >
> > Looks fine but do we need to bump since it didn't built?
>
> It failed on mips64 as well, so you're right indeed.
>
> Charlène.
OK
>
>
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/x11/xkbcommon/Makefile,v
> retrieving revision 1.17
> diff -u -p -u -p -r1.17 Makefile
> --- Makefile 23 Oct 2019 08:18:36 -0000 1.17
> +++ Makefile 8 Nov 2019 16:55:11 -0000
> @@ -31,6 +31,12 @@ TEST_DEPENDS = devel/bison \
> shells/bash
> TEST_IS_INTERACTIVE= x11
>
> +# Fix undefined references errors
> +.include <bsd.port.arch.mk>
> +.if !${PROPERTIES:Mlld}
> +CONFIGURE_ENV += LDFLAGS="-L${X11BASE}/lib"
> +.endif
> +
> # XXX: This test still fails
> pre-test:
> sed -i 's,bin/bash,${LOCALBASE}/bin/bash,'
> ${WRKSRC}/test/symbols-leak-test.bash
--
Antoine
