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: Revisor - Revisor GUI https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642 ------- Additional Comments From tibbs@xxxxxxxxxxx 2007-05-24 23:02 EST ------- I received an updates SRPM via IRC. Here's a quick recap of the IRC discussion for the record: The desktop file gets installed properly, although desktop-file-install does emit a warning: /var/tmp/revisor-2.0.2-3.fc7-root/usr/share/applications/fedora-revisor.desktop: warning: The 'Application' category is not defined by the desktop entry specification. Please use one of "AudioVideo", "Audio", "Video", "Development", "Education", "Game", "Graphics", "Network", "Office", "Settings", "System", "Utility" instead I don't think this is a big deal, although it would be good to fix at some point. Either "Development" or "System" seem OK to me. The COPYING file still doesn't get included in the package. All you need to do is change: %doc README NEWS AUTHORS to %doc README NEWS AUTHORS COPYING in your %files section. That should be it. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review