Page MenuHomeFreeBSD

devel/pear: Not PHP72 ready

Authored by joneum on Nov 24 2017, 7:07 PM.
Referenced Files
Unknown Object (File)
Tue, May 21, 6:33 PM
Unknown Object (File)
Mon, May 20, 12:47 PM
Unknown Object (File)
Mon, May 20, 12:42 PM
Unknown Object (File)
Mon, May 20, 12:42 PM
Unknown Object (File)
Mon, May 20, 12:21 PM
Unknown Object (File)
May 11 2024, 11:39 PM
Unknown Object (File)
May 8 2024, 8:07 PM
Unknown Object (File)
May 5 2024, 8:55 AM



add patch-php72, now devel/pear is PHP 7.2 ready

Approved by: xxx (mentor)
#Differential Revision:

Test Plan

Build fine with PHP70 and 71, but not with 72:

For info: The days, I will test all my pear and peacl ports with PHP70, 71 and 72

Diff Detail

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

MFH: 2017Q4 -- does that even have 72?

MFH: 2017Q4 -- does that even have 72?

ährm ..... nopp ... :D
I deleted it

I see that fedora has the pear package for php72:

So maybe take a peak at what they do, copy it, and don't marke it ignored :)

[probably there are other distros to steal/take inspriation from too]

I have already looked at this page today and found nothing that would help me further

Hi Miwi,

I added you here once. You built that pear back then. Do you have any idea here to fix this with PHP 72?

you need this one

#  cat patch-git_772f7d
From 85ebf1e6190681820634c891af624bacde49a017 Mon Sep 17 00:00:00 2001
From: Andy Thompson <>
Date: Sun, 11 Jun 2017 09:25:12 +0100
Subject: [PATCH] Remove unneeded by-reference on built-in fuction

PHP 7.2 is complaining about this. It also doesn't appear to be
needed, as the argument array items aren't being passed by-reference.
 Archive/Tar.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Archive/Tar.php b/Archive/Tar.php
index 202fffb..0bd1c6c 100644
--- go-pear-bundle/Tar.php
+++ go-pear-bundle/Tar.php
@@ -659,7 +659,7 @@ public function setAttribute()

         // ----- Get the arguments
-        $v_att_list = & func_get_args();
+        $v_att_list = func_get_args();

         // ----- Read the attributes
         $i = 0;
15 ↗(On Diff #35708)

^ add patch, get rid of this :D

This revision now requires changes to proceed.Nov 24 2017, 10:20 PM

You need to bump the PORTREVISION as this clearly changes the files in the installed package :)

After that, ship it.

This revision is now accepted and ready to land.Dec 1 2017, 10:02 PM
This revision was automatically updated to reflect the committed changes.