Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 23 Nov 2015 02:36:41 -0700
From:      "John Hein" <jh-fbml@snkmail.com>
To:        <freebsd-gnome@freebsd.org>, <jbeich@vfemail.net>, "gecko@freebsd.org>, <mi+thun@aldan.algebra.com>"@microsemi.com
Subject:   Re: Non-prefix safe ports
Message-ID:  <21450-1448271401-856348@sneakemail.com>
In-Reply-To: <4mge-yjhj-wny@vfemail.net>
References:  <563A5861.6020706__9360.56868308715$1446667952$gmane$org@aldan.algebra.com>	<4mge-yjhj-wny@vfemail.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Jan Beich wrote at 01:18 +0100 on Nov 22, 2015:
 > "Mikhail T." <mi+thun@aldan.algebra.com> writes:
 > 
 > > Building out a new machine from scratch using /opt (rather than the
 > > usual /usr/local) as LOCALBASE, I found the following two ports, which
 > > break:
 > 
 > poudriere testport -P would have been easier. That said it would be nice
 > to have them as periodic exp-runs. I'm sure there're way more ports
 > with broken PREFIX != LOCALBASE e.g., anything lua-*.
 > 
 > >
 > >   * multimedia/openh264 - the files built by the port get installed into
 > >     stage/usr/local/... instead of stage/opt -- I had to manually move
 > >     the files around before "make install" succeeded
 > 
 > This affects all ports with USES=webplugin. Fixing isn't trivial, so
 > better file a bug or it will be forgotten.
 > 
 > >   * graphics/gtk-update-icon-cache - at configure-stage, the port
 > >     wrongly complaints, that the cairo installed on the system does not
 > >     support X11. After I created a symlink: /usr/local -> ../opt, the
 > >     problem went away.
 > 
 > make(1) flow control is evaluated at parsing time. Try using sh(1)
 > conditional to postpone it until the target is run.

I've had this patch in my ports tree forever - one of those patches
for which I just never got around to the last step: actually
submitting a bug.  Instead of hard-coding a path, it uses the
pkg-config tool to detect.  But it postpones the detection until
the target just as Jan did.

Index: Makefile
===================================================================
--- graphics/gtk-update-icon-cache/Makefile	(revision 402233)
+++ graphics/gtk-update-icon-cache/Makefile	(working copy)
@@ -43,10 +43,10 @@
 		${WRKSRC}/configure
 
 pre-configure:
-.if !exists(${LOCALBASE}/libdata/pkgconfig/cairo-xlib.pc)
-	@${ECHO_CMD} "${PKGNAME}: Needs cairo with X11 support enabled."
-	@${FALSE}
-.endif
+	@if ! pkg-config --exists cairo-xlib; then \
+		${ECHO_CMD} "${PKGNAME}: Needs cairo with X11 support enabled."; \
+		${FALSE}; \
+	fi
 
 do-install:
 	@${INSTALL_PROGRAM} ${WRKSRC}/gtk/gtk-update-icon-cache \





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?21450-1448271401-856348>