Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=914988 Remi Collet <fedora@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review? --- Comment #6 from Remi Collet <fedora@xxxxxxxxxxxxxxxxx> --- [!]: Rpmlint is run on all rpms the build produces. php-SymfonyCmfRouting.noarch: E: summary-too-long C Extends the Symfony2 routing component for dynamic routes and chaining several routers [!]: Package is named according to the Package Naming Guidelines. same comment about github snapshot. https://github.com/symfony-cmf/Routing/archive/1.0.0-alpha4.tar.gz Release: 0.2.alpha4%{dist} is enough (date + rev don't make sense here) [!]: %check is present and all tests pass. Local build (with symfony installed) Fatal error: Interface 'Psr\Log\LoggerInterface' not found Mock build 3 skipped tests [!]: All build dependencies are listed in BuildRequires, except for any that BuildRequires: php-PsrLog BuildRequires: php-pear(pear.symfony.com/Routing) BuildRequires: php-pear(pear.symfony.com/HttpKernel) [!]: Requires correct, justified where necessary. According to recent discussion on -devel 2 requires seems ok. Probably need (else please justify) Requires: php-PsrLog -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=x6tsCIbPff&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review