ports/145684: [MAINTAINER][UNBREAK] audio/libmusicbrainz3: Fails to build with neon28

[Jason E. Hale]

>Number:         145684
>Category:       ports
>Synopsis:       [MAINTAINER][UNBREAK] audio/libmusicbrainz3: Fails to build with neon28
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          maintainer-update
>Submitter-Id:   current-users
>Arrival-Date:   Wed Apr 14 00:20:01 UTC 2010
>Closed-Date:
>Last-Modified:
>Originator:     Jason E. Hale
>Release:        FreeBSD 8.0-RELEASE i386
>Organization:
none 
>Environment:
System: FreeBSD mocha.verizon.net 8.0-RELEASE FreeBSD 8.0-RELEASE #0: Fri Nov 27 18:56:06 EST 2009 root at mocha.verizon.net:/usr/obj/usr/src/sys/MOCHA8 i386


	
>Description:
After an attempt to make audio/libmusicbrainz3 depend on older versions of neon
if installed, it appears the older versions are not being detected and the port
tries to install neon29 anyways.  This detection needs to happen after
bsd.port.pre.mk to work properly.
	
>How-To-Repeat:
Install www/neon28, then try to install audio/libmusicbrainz3.
	
>Fix:
Same code, just shifted below bsd.port.pre.mk.

	

--- 2010-04-13-libmusicbrainz3.diff begins here ---
diff -ruN libmusicbrainz3.orig/Makefile libmusicbrainz3/Makefile
--- libmusicbrainz3.orig/Makefile	2010-04-13 19:56:03.000000000 -0400
+++ libmusicbrainz3/Makefile	2010-04-13 19:58:55.000000000 -0400
@@ -19,14 +19,6 @@
 
 LIB_DEPENDS=	discid:${PORTSDIR}/audio/libdiscid
 
-.if exists(${LOCALBASE}/bin/neon-config)
-NEON_VER!=	${LOCALBASE}/bin/neon-config --version | \
-		${SED} -n -E  's/^neon [0-9]+\.([0-9]+)\..*$$/\1/p'
-.else
-NEON_VER=	29
-.endif
-LIB_DEPENDS+=	neon.${NEON_VER}:${PORTSDIR}/www/neon${NEON_VER}
-
 USE_CMAKE=	yes
 USE_GNOME=	pkgconfig
 USE_LDCONFIG=	yes
@@ -35,6 +27,14 @@
 
 .include <bsd.port.pre.mk>
 
+.if exists(${LOCALBASE}/bin/neon-config)
+NEON_VER!=	${LOCALBASE}/bin/neon-config --version | \
+		${SED} -n -E 's/^neon [0-9]+\.([0-9]+)\..*$$/\1/p'
+.else
+NEON_VER=	29
+.endif
+LIB_DEPENDS+=	neon.${NEON_VER}:${PORTSDIR}/www/neon${NEON_VER}
+
 pre-configure:
 	${REINPLACE_CMD} -e '/pkgconfig/s|$${LIB_INSTALL_DIR}|libdata|' \
 		-e '/CppUnit/d' ${WRKSRC}/CMakeLists.txt
--- 2010-04-13-libmusicbrainz3.diff ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted: