Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675726 Pavel Zhukov <pavel@xxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pavel@xxxxxxxxxxx --- Comment #1 from Pavel Zhukov <pavel@xxxxxxxxxxx> 2011-02-07 09:57:08 EST --- few comments: -- you should NOT point bash to Requires: http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2 -- if you don't plan package for EPEL you should not point BuildRoot and clean sections; -- you SHOULD point full URL to Source (vendor doen't prohibit it) http://cdm.ghost1227.com/repo/cdm-latest.tar.gz or http://cdm.ghost1227.com/repo/cdm-0.5.3.tar.gz -- you SHOULD include README (create it) and COPYING (license text http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text) to your package. You can include CHANGELOG too. -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review