[Bug 648466] Review Request: php-phpunit-PHP-CodeBrowser - PHP_CodeBrowser for integration in Hudson and CruiseControl

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=648466

--- Comment #5 from Christof Damian <christof at damian.net> 2010-11-07 13:45:27 EST ---
(In reply to comment #3)
> > I hope the doc location is to your liking :-)
> NO !
> I will prefer a better solution. But this is the standard one used in fedora.

Very well, I will change it once Fedora provides a better standard.

> This package have optional deps :
>  "pear/Text_Highlighter" (version >= 0.7.1)
>  "channel://pear.phpunit.de/PHPUnit" (version >= 3.4.0)
>  "channel://pear.phpundercontrol.org/phpUnderControl" (version >= 0.5.0)
>  "pear/PHP_CodeSniffer" (version >= 1.2.0)
>  "pear/PhpDocumentor" (version >= 1.4.3)
> 
> With pear package, I think optional dep can be make "mandatory" (ie Requires in
> spec, when available in the repo.) as packages are small, and don't eat memory
> if not used.

OK, I think we got them except of phpUnderControl.

thanks for the review and your help.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.