[Bug 448874] Review Request: pyroom - PyRoom is a full screen text editor and a clone of Writeroom

[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 report.

Summary: Review Request: pyroom - PyRoom is a full screen text editor and a clone of Writeroom


https://bugzilla.redhat.com/show_bug.cgi?id=448874


tibbs at math.uh.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tibbs at math.uh.edu




------- Additional Comments From tibbs at math.uh.edu  2008-06-26 21:25 EST -------
So did anyone decide to pick this package up?

All that's left from rpmlint is:
  pyroom.noarch: E: script-without-shebang /usr/share/pyroom/pyroom_error.py
  pyroom.noarch: E: script-without-shebang /usr/share/pyroom/autosave.py
  pyroom.noarch: E: script-without-shebang /usr/share/pyroom/gui.py
  pyroom.noarch: E: script-without-shebang /usr/share/pyroom/preferences.py
  pyroom.noarch: E: script-without-shebang /usr/share/pyroom/basic_edit.py
I don't see why these are executable; a quick chmod should fix them.

I think something's wrong with the Source0: URL; spectool gets a 404 trying to
download it.

I agree that a .desktop file is needed.

Otherwise I think this looks good, and if nobody else wants to review it I'll be
happy to take a look at an updated package.



-- 
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, or are watching someone who is.