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=650643 Steven Garcia <webwhammy@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |webwhammy@xxxxxxxxx --- Comment #1 from Steven Garcia <webwhammy@xxxxxxxxx> 2010-11-07 18:07:43 EST --- Hello I'm not a sponsor, but I have a few suggestions reviewing you package. rpmlint SPECS/cegui06.spec SRPMS/cegui06-0.6.2-1.fc14.src.rpm SPECS/cegui06.spec:9: W: macro-in-comment %{version} SPECS/cegui06.spec: W: invalid-url Source0: CEGUI-0.6.2.tar.gz cegui06.src: W: spelling-error %description -l en_US codecs -> codes, coders, code's cegui06.src: W: spelling-error %description -l en_US xml -> XML, cml, ml cegui06.src:9: W: macro-in-comment %{version} cegui06.src: W: invalid-url Source0: CEGUI-0.6.2.tar.gz 1 packages and 1 specfiles checked; 0 errors, 6 warnings. Change the comments above the Source0 field on how to generate the tarball used in the srpm Use a link such as: http://downloads.sourceforge.net/crayzedsgui/CEGUI-0.6.2b.tar.gz I would remove the macro %{version}, from your comment Change the spelling of xml to XML in your %description -- 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