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@xxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tibbs@xxxxxxxxxxx ------- Additional Comments From tibbs@xxxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review