Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 20 Aug 2012 03:45:34 +0200 (CEST)
From:      Rodrigo OSORIO <rodrigo@bebik.net>
To:        FreeBSD-gnats-submit@FreeBSD.org
Subject:   ports/170776: textproc/libmrss fix for php
Message-ID:  <20120820014534.1D9D92934C5@oldfaithful.bebik.local>
Resent-Message-ID: <201208200150.q7K1oDWC041599@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>Number:         170776
>Category:       ports
>Synopsis:       textproc/libmrss fix for php
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Mon Aug 20 01:50:12 UTC 2012
>Closed-Date:
>Last-Modified:
>Originator:     Rodrigo OSORIO
>Release:        FreeBSD 9.0-STABLE amd64
>Organization:
>Environment:
System: FreeBSD oldfaithful.bebik.local 9.0-STABLE FreeBSD 9.0-STABLE #2: Thu Jul 5 22:22:11 CEST 2012 root@hodja.bebik.local:/usr/src/obj/usr/src/sys/GENERIC amd64


	
>Description:
	Attached to this PR a patch to fix the broken state of the port. 
	This PR only fix the PHP issue and don't resolve the other warnings reported by portlint.

	This issue seems to be caused by the use of function_entry who seems to be deprecate for
	php 5.4 and later and must be replaced by zend_function_entry. Some forums point this
	URL as an explanation : https://bugs.php.net/bug.php?id=60016
	I'll propagate the fix upstream for the future versions but, imho, the following patch must be 
	commited to unbroke the port.

	
	
>How-To-Repeat:
	
>Fix:

	

--- libmrss.patch begins here ---
diff -ruN libmrss/Makefile libmrss/Makefile
--- libmrss/Makefile	2012-07-26 07:40:22.000000000 +0200
+++ libmrss/Makefile	2012-08-20 03:28:01.000000000 +0200
@@ -7,7 +7,7 @@
 
 PORTNAME=	libmrss
 PORTVERSION=	0.19.2
-PORTREVISION=	3
+PORTREVISION=	4
 CATEGORIES=	textproc
 MASTER_SITES=	http://www.autistici.org/bakunin/libmrss/
 
@@ -33,7 +33,6 @@
 .include <bsd.port.pre.mk>
 
 .if defined(WITH_PHPMRSS)
-BROKEN=		does not compile
 USE_PHP=	yes
 USE_PHP_BUILD=	yes
 USE_AUTOTOOLS+=	autoconf:env
diff -ruN libmrss/files/patch-phpmrss_php.c libmrss/files/patch-phpmrss_php.c
--- libmrss/files/patch-phpmrss_php.c	1970-01-01 01:00:00.000000000 +0100
+++ libmrss/files/patch-phpmrss_php.c	2012-08-20 03:26:04.000000000 +0200
@@ -0,0 +1,11 @@
+--- phpmrss/php.c	2008-01-25 19:32:24.000000000 +0100
++++ phpmrss/php.c 	2012-08-20 03:10:45.000000000 +0200
+@@ -58,7 +58,7 @@
+ return SUCCESS;
+ }
+ 
+-static function_entry mrss_functions[] = {
++static zend_function_entry mrss_functions[] = {
+   PHP_FE (mrss_parse_url, NULL) 
+   PHP_FE (mrss_parse_url_with_options, NULL) 
+   PHP_FE (mrss_parse_file, NULL) 
--- libmrss.patch ends here ---


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



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