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=567733 Thomas Janssen <thomasj@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Flag|fedora-review? |fedora-review+ --- Comment #5 from Thomas Janssen <thomasj@xxxxxxxxxxxxxxxxx> 2010-02-25 13:06:13 EST --- (In reply to comment #4) > /usr/lib/python2.6/site-packages/marave/editor/spelltextedit.py > Looks like a standalone widget, so should be kosher as-is. > (It has been argued packages should be patched to use > #!/usr/bin/python > instead, but it's a contentious issue, and is why it's not in the python > guidelines yet Hm.. Ok. > Otherwise, fixed up the other cosmetic items, > > Spec URL: http://rdieter.fedorapeople.org/rpms/marave/marave.spec > SRPM URL: http://rdieter.fedorapeople.org/rpms/marave/marave-0.7-2.fc12.src.rpm > > %changelog > * Thu Feb 25 2010 Rex Dieter <rdieter@xxxxxxxxxxxxxxxxx> 0.7-2 > - fix whitespace > - Group: Applications/Editors APPROVED -- 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