https://bugzilla.redhat.com/show_bug.cgi?id=1336735 Remi Collet <fedora@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora_requires_release_not | |e? --- Comment #4 from Remi Collet <fedora@xxxxxxxxxxxxxxxxx> --- [!]: Package is named according to the Package Naming Guidelines. "As for other packages, name should only use lowercase, underscore and slash replaced by dash. " So php-slim3 should be prefered [!]: If the package is a rename of another package, proper Obsoletes and Provides are present. Obsoletes: %{name}-tests < 3 Uneeded, this package doesn't exists [~] EL-5 stuff unneeded (not a blocker) [x]: %check is present and all tests pass. PHP 5: OK PHP 7: OK I encouter failure on local build, but ok in mock (so not a blocker) 1) Slim\Tests\ContainerTest::testGetWithDiConfigErrorThrownAsInvalidArgumentException Failed asserting that exception of type "InvalidArgumentException" matches expected exception "\Interop\Container\Exception\ContainerException". Message was: "Identifier "doesnt-exist" is not defined." at /dev/shm/extras/BUILD/Slim-794073a27b2dd9ef492124263038291b7f8daeb3/tests/ContainerTest.php:74 /dev/shm/extras/BUILDROOT/php-Slim3-3.4.1-1.fc21.remi.x86_64/usr/share/php/Slim3/Container.php:125 /dev/shm/extras/BUILD/Slim-794073a27b2dd9ef492124263038291b7f8daeb3/tests/ContainerTest.php:77 Minors - also fix package name in spec header - as you keep changelog from php-Slim, add a changelog entry "renamed to ..." -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx http://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx