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 <[email protected]> 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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to