Spec URL: http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-View.spec SRPM URL: http://nb.fedorapeople.org/horde-reviews/php-horde-Horde-View-1.0.1-1.fc16.src.rpm Description: A simple View pattern implementation.
I am not an official package reviewer, but here are some comments I have for this package: *** It would help readability to group your Build* and Requires* statements together instead of mixing them with each other and your Provides and Conflicts statements *** Your BuildRequires should be BuildRequires: php-pear(PEAR) >= 1.7.0 instead of BuildRequires: php-pear >= 1:1.4.9-1.2 (this would match what you have in bug 785606 and satisfy this package's package.xml dependency) *** Per package.xml requirements, please add: Requires: php-common >= 5.2.0 *** phpci results: For completeness, you may wish to require the virtual packages (and to prevent any future packaging issues due to PHP package changes) "php-date", "php-json", "php-pcre", "php-reflection", and "php-spl". *** Up to you, but package.xml lists the following optional packages if you choose to require: - php-pear(pear.horde.org/Horde_Controller) < 2.0.0 - php-pear(pear.horde.org/Horde_Routes) < 2.0.0 (if you do choose to require these packages, please update this ticket's "Depends On") *** Please remove "rm -rf $RPM_BUILD_ROOT" from the %install section (see http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag) -- I'm assuming no EPEL 5 build because your other php-horde-* packages do not.
I have updated / submit this one *** This bug has been marked as a duplicate of bug 909662 ***