[Bug 567733] Review Request: marave - A Simple text editor

[bugzilla at redhat.com]

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 at fedoraproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
               Flag|fedora-review?              |fedora-review+

--- Comment #5 from Thomas Janssen <thomasj at fedoraproject.org> 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 at fedoraproject.org> 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.