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: knemo Network monitor applet. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=189088 ------- Additional Comments From gauret@xxxxxxx 2006-05-25 06:02 EST ------- Needs work: * Desktop files installed in %{_datadir}/applications/kde don't need the "--vendor fedora" namespace, they already have kde (they are in the kde subdir). * As a consequence, you don't need to rename them afterwards. Notes: * Why drop the keywords from the desktop file ? (really) Minor: * The Patch0 line and the "--add-category X-KDE-settings-network" are not properly lined-up (tab instead of spaces) -- 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