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=733603 --- Comment #7 from Kalpa Welivitigoda <callkalpa@xxxxxxxxx> 2011-08-28 11:07:52 EDT --- (In reply to comment #6) > Two quick comments: > > - It's better to use macros. For instance: > %{__python} setup.py build > > - You should start updating the %changelog section every time you make changes > and post the new spec & SRPM here so that reviewers and commenters can keep > track of the changes you've made. (You also need to bump the Release number > every time you do so.) Thanks for reminding about the %changelog. Here are the new files, Spec URL: http://callkalpa.fedorapeople.org/sugar-ruler/sugar-ruler.spec SRPM URL: http://callkalpa.fedorapeople.org/sugar-ruler/sugar-ruler-11-4.fc15.src.rpm -- 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