ports/119039: [PATCH] Incorrect dependency between php5-spl and php5-simplexml

[Eugene M. Kim]

>Number:         119039
>Category:       ports
>Synopsis:       [PATCH] Incorrect dependency between php5-spl and php5-simplexml
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Wed Dec 26 12:20:00 UTC 2007
>Closed-Date:
>Last-Modified:
>Originator:     Eugene M. Kim
>Release:        FreeBSD 6.2-STABLE i386
>Organization:
>Environment:
System: FreeBSD purple.the-7.net 6.2-STABLE FreeBSD 6.2-STABLE #9: Sat Oct 13 14:45:27 PDT 2007 ab at purple.the-7.net:/home/FreeBSD/build/RELENG_6/obj/home/FreeBSD/build/RELENG_6/src/sys/PURPLE i386
>Description:
The simplexml module depends on the spl module in the PHP 5.2.5
distribution (according to ext/{spl,simplexml}/config.m4); however, the
dependency is set the other way around in ports/lang/php5/Makefile.ext,
such that spl depends on simplexml instead.

This breaks the build of simplexml in absence of spl (e.g. when
portupgrade decides to install simplexml before spl because it thinks
spl depends on simplexml), because the required spl header files are
not present on the system yet.
>How-To-Repeat:
On a clean system without any php5 ports installed, run "portinstall -R
php5-simplexml php5-spl".
>Fix:
The following patch modifies ports/lang/php5/Makefile.ext so that
simplexml correctly depends on spl, and not spl on simplexml.
--- patch-ports::lang::php5::Makefile.ext begins here ---
--- -	2007-12-26 03:59:37.419630345 -0800
+++ ports/lang/php5/Makefile.ext	2007-12-26 03:56:24.000000000 -0800
@@ -356,6 +356,8 @@
 		--with-libxml-dir=${LOCALBASE}
 
 USE_GNOME=	libxml2
+
+USE_PHP=	spl
 .endif
 
 .if ${PHP_MODNAME} == "snmp"
@@ -386,7 +388,7 @@
 
 USE_GNOME=	libxml2
 
-USE_PHP=	pcre simplexml
+USE_PHP=	pcre
 USE_PHP_BUILD=	yes
 .endif
 
--- patch-ports::lang::php5::Makefile.ext ends here ---
>Release-Note:
>Audit-Trail:
>Unformatted: