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=784359 Volker Fröhlich <volker27@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: |Review Request: |qpid-guitools - GUI |qpid-gui-tools - GUI |utilities for Red Hat MRG |utilities for Red Hat MRG |qpid |qpid --- Comment #11 from Volker Fröhlich <volker27@xxxxxx> 2012-02-21 17:52:56 EST --- Please always upload a new release when you're making changes and keep a detailed changelog inside the spec file. It's a lot easier for the reviewer that way. By the way, the spec file is not accessible. Please use %global instead of %define, see guidelines. (You can install the desktop file with desktop-file-install as well, by the way.) rpmlint: qpid-gui-tools.src: W: strange-permission qpid-gui-tools.spec 0600L -- 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