Page MenuHomeFreeBSD

devel/pear: Not PHP72 ready
ClosedPublic

Authored by joneum on Nov 24 2017, 7:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 5:47 AM
Unknown Object (File)
Fri, Nov 8, 1:19 AM
Unknown Object (File)
Wed, Nov 6, 3:21 PM
Unknown Object (File)
Tue, Oct 22, 1:45 PM
Unknown Object (File)
Thu, Oct 17, 11:53 AM
Unknown Object (File)
Thu, Oct 17, 2:06 AM
Unknown Object (File)
Thu, Oct 17, 1:12 AM
Unknown Object (File)
Oct 15 2024, 1:12 PM
Subscribers

Details

Summary

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

Approved by: xxx (mentor)
#Differential Revision: https://reviews.freebsd.org/Dxxxxx

Test Plan

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

http://joneumbox.org/data/11-1_amd64-PHP72-ports/2017-11-24_20h03m17s/logs/errors/pear-1.10.5.log
http://joneumbox.org/data/11-1_i386-PHP72-ports/2017-11-24_20h03m17s/logs/errors/pear-1.10.5.log

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

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 13009
Build 13263: arc lint + arc unit

Event Timeline

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

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

ährm ..... nopp ... :D https://svnweb.freebsd.org/changeset/ports/454302
I deleted it

I see that fedora has the pear package for php72:
https://www.rpmfind.net/linux/RPM/remi/fedora/27/x86_64/php72-php-pear-1.10.5-2.fc27.remi.noarch.html

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?

https://github.com/pear/Archive_Tar/commit/772f7d0ffaf225d575ac1feca44eefcd4a9b144e

you need this one

#  cat patch-git_772f7d
From 85ebf1e6190681820634c891af624bacde49a017 Mon Sep 17 00:00:00 2001
From: Andy Thompson <me@andytson.com>
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;
devel/pear/Makefile
15

^ 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.