Update WRKSRC=

.

Update Makefile

In D13235#278220, @tcberner wrote:

My issue with this is, that you will change the package name twice when doing it like that.

If you could keep it pear-SebastianBergmann-PHPCPD until you rename them all consistently you get one rename.
On the other hand if you do this now and then the other, you essentially do pear-SebastianBergmann-PHPCPD -> pear-SebastianBergmann-phpcpd -> pear-phpcpd which will lead to two ugly upgrades :)

As you can just make the port work with the PORTNAME (and thereby package name) it has now by setting
WRKDIR=${WRKSRC}/phpcpd
I think this is the better route to take.

That being said, I am in favour of better portnames :) you just should try to reduce the annoyance when doing them.

Buildlogs from the last change:

update Makefile

Updating D13058: math/dynare: Update from 4.5.1 to 4.5.3

yet another attempt, this crashed the i386 jail.

This is the current status

.

.

In D13235#277649, @tz wrote:

In D13235#276797, @joneum wrote:

what do the 2 other mentors @rene and @tz say about that?

About what concretely? After reading the log i'm not sure :D

bump portrevision

.

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

Please work on this setting.

I will gladly accept this advice, but you too should take your own advice. Work on your attitude.

Please do not just look for the error with me, but ask yourself if you have acted correctly. The answer should be "no".
Next I will not go into this discussion here

The maintainer is working on a master / slave setup. So you should be satisfied.

They complain about missing answers, but do not answer themselves to open questions. What exactly is in the make.conf?
If you criticize something, you should do it better and not as bad.
You came, complain, without writing exactly how it is better, and are gone again. They have often done that. Not good.
Here I am waiting since Monday for an answer from you: D13235
Please only complain about something that makes you better yourself!
You can find the PR number here at the top ;-)

Not correct. You did not say exactly what you would like to have!

I summarize:

• 2 weeks ago it was not ok to have ftp/php56-fastdfs, ftp/php70-fastdfs and ftp/php71-fastdfs external patches. I changed it.

If "svn cp php71-fastdfs/ php72-fastdfs/" was correct, and there is nothing to complain about the code, I ask for release. thx :-)

what do the 2 other mentors @rene and @tz say about that?

In D13258#276452, @rene wrote:

Again, the PR # is wrong, it should be 223377 and the reporter should be daniel@blodan.se

Otherwise approved.

to forget "make makesum"

remove PHP_DIR

After update php*/fastdfs, here is an copy for the new php72 port.

Hi Miwi,

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

switch back to PORTVERSION, remove PHP ignore

remove PHP Ignore

In D13235#275741, @tcberner wrote:

yes, it's not really a problem it's just not "beautiful" :D

remove PHP Ignore

In D13234#275734, @tcberner wrote:

Then I would be optimistic and assume that it works, and not pessimistic and commit 33 [possilbly wrong] IGNORE_WITH=72 lines :)
assume it's working until reported otherwise

remove testlines from the Makefile

In D13234#275696, @tcberner wrote:

Why mark them all as broken if you don't know? :) seems like uneeded work. I would try to only mark the ones that are broken

My plan is:
-All pear and pecl ports up to date
-All "as broken" repair
-Then test with PHP 70,71 and 72

see my comment in https://reviews.freebsd.org/D13233:

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

In D13233#275620, @tcberner wrote:

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