Spec URL: http://siwinski.fedorapeople.org/rpmbuild/SPECS/php-symfony2-Stopwatch.spec SRPM URL: http://siwinski.fedorapeople.org/rpmbuild/SRPMS/php-symfony2-Stopwatch-2.3.1-1.fc18.src.rpm Description: Stopwatch component provides a way to profile code. Fedora Account System Username: siwinski
Created attachment 813931 [details] phpci.log phpcompatinfo version 2.24.0.
Created attachment 813932 [details] review.txt Generated by fedora-review 0.5.0 (920221d) last change: 2013-08-30 Command line :/usr/bin/fedora-review -b 979794 Buildroot used: fedora-19-x86_64 Active plugins: Generic, PHP, Shell-api Disabled plugins: Java, C/C++, Python, SugarActivity, Perl, R, Ruby Disabled flags: EPEL5, EXARCH, DISTTAG
Issues: ======= [!]: Sources used to build the package match the upstream source, as provided in the spec URL. Note: Upstream MD5sum check error, diff is in /home/extras/SPECS/review/979794-php-symfony2-Stopwatch/diff.txt See: http://fedoraproject.org/wiki/Packaging/SourceURL => minor diff in the package.xml which have obviously be regenerated, not an issue for the review [~]: Package is named according to the Package Naming Guidelines. New Guildelines recommend the use of lowercase names. As all the symfony2 stack use CamelCase, I thinks this could be ignored. [!]: Requires correct, justified where necessary. php-common => php(language) [!]: Latest version is packaged. 2.3.6 available [!]: %check is present and all tests pass. As the test are ok, you can remove || : Temporarily ignore failed tests
I just submitted the single-spec php-symfony2 review request (bug 1021749) that I would prefer to move to.
*** This bug has been marked as a duplicate of bug 1021749 ***