https://bugzilla.redhat.com/show_bug.cgi?id=823042
Christof Damian <chris...@damian.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |chris...@damian.net Assignee|nob...@fedoraproject.org |chris...@damian.net Flags| |fedora-review? --- Comment #1 from Christof Damian <chris...@damian.net> --- Issues: [!]: MUST Buildroot is not present Note: Buildroot is not needed unless packager plans to package for EPEL5 See: http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag [!]: MUST Each %files section contains %defattr if rpm < 4.4 Note: defattr(....) present in %files section. This is OK if packaging for EPEL5. Otherwise not needed See: http://fedoraproject.org/wiki/Packaging/Guidelines#FilePermissions [!]: MUST Requires correct, justified where necessary. Use php-common instead of php, maybe also include php-json -- You are receiving this mail because: You are on the CC list for the bug.
_______________________________________________ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review