Re: qt5-qtbase

[Mark Davies <mark%ecs.vuw.ac.nz@localhost>]

On 30/03/19 8:50 am, Jason Bacon wrote:
>> qtlocation: these are new entries with 4.12.2.  I struck the same issue
>> when I built it yesterday.  I could just add a PLIST.Linux for it, but
>> from my initial investigation looks like they should be being built for
>> the BSD's as well so want to work out why they are not.
> Fair enough.

Figured out why its not building this on the BSD's so added the entries
to PLIST.Linux.

> What do you think it the right approach for fixing qtbase?  My hack is
> pretty innocuous, but seems like it shouldn't be necessary and it would
> be more satisfying to know the root cause.


I agree.  The hack isn't needed on my ArchLinux boxes.  I can see from
the Joyent Ubuntu bulk builds that it isn't necessary there but the
Joyent CentOS bulk builds show the same issue you were seeing.

I've tried the attached variant of your patch on ArchLinux and confirmed
that it doesn't alter the results there, so if we can't figure out the
root issue shortly, I'll commit that.  Actually as I was writing that
last sentence realised that we also only want to do it if on a 64bit
platform.

cheers
mark


Index: Makefile
===================================================================
RCS file: /src/cvs/netbsd/pkgsrc/x11/qt5-qtbase/Makefile,v
retrieving revision 1.71
diff -u -r1.71 Makefile
--- Makefile	27 Mar 2019 10:16:34 -0000	1.71
+++ Makefile	29 Mar 2019 21:03:32 -0000
@@ -95,6 +95,8 @@
 LDFLAGS+=		-lresolv
 .  endif
 .elif ${OPSYS} == "Linux"
+# CentOS seems to need this
+CONFIGURE_ARGS+=	-L "/usr/lib64"
 LDFLAGS+=		-ldl
 .elif ${OPSYS} == "NetBSD" || ${OPSYS} == "FreeBSD"
 .  if exists(/usr/lib/libexecinfo.so)