devel/pear-PHP_CodeBrowser: unbreak

[joneum]

Summary

Unbreak this port

While here switch to DISTVERSION

Add LICENSE

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

Test Plan

poudriere testport:

10.3-amd64
10.3-i386
10.4-amd64
10.4-i386
11.1-amd64
11.1-i386
12.0-CURRENT r325045 amd64
12.0-CURRENT r325045 i386

[tcberner]

You might want to take a look at the warnings

warning: phpunit/PHP_CodeBrowser requires package "phpunit/File_Iterator" (version >= 1.3.0)
phpunit/PHP_CodeBrowser can optionally use package "pear/Text_Highlighter" (version >= 0.7.1)
phpunit/PHP_CodeBrowser can optionally use package "phpunit/PHPUnit" (version >= 3.4.0)
phpunit/PHP_CodeBrowser can optionally use package "pear/PHP_CodeSniffer" (version >= 1.2.0)
phpunit/PHP_CodeBrowser can optionally use package "pear/PhpDocumentor" (version >= 1.4.3)

and decide whether you ant to depend on them.

[joneum]

hmmmm.........

[mat]

devel/pear-PHP_CodeBrowser/Makefile

23 ↗

(On Diff #34550)

This change is bogus.

[joneum]

update

[mat]

devel/pear-PHP_CodeBrowser/Makefile

5 ↗

(On Diff #34746)

Also, please try not to add too much churn to the repo, don't change this for the sake of changing it, only do it with a version change.

[joneum]

ready to land?

[tcberner]

In D12877#271212, @joneum wrote:

ready to land?

@mat asked you to revert the PORTREVERSION line :), and told you that the USES line looks bogus :) -- you might want to investigate that first.

[joneum]

The USES line was cleaned up days ago.

[joneum]

update

[tcberner]

Sorry, thats my bad, I did not catch the changes :)

• you can mark inline comments as done, that way, it's easier to see what you already addressed
• also it is sometimes helpful to have a more descriptive update message than update :D