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: luma - A graphical tool for managing LDAP servers https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200139 tibbs@xxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tibbs@xxxxxxxxxxx ------- Additional Comments From tibbs@xxxxxxxxxxx 2006-10-06 01:19 EST ------- This builds fine in mock; rpmlint says: E: luma hardcoded-library-path in ${RPM_BUILD_ROOT}/usr/lib which seems bogus as the spec is doing this to fix brokenness in the upstream source. W: luma mixed-use-of-spaces-and-tabs (spaces: line 63, tab: line 5) Fix if you like. The "Comment=" line in the .desktop file is ungrammatical; please consider changing it to read "Tool for managing LDAP servers". Please also consider s/server/servers/ in %description. When you call desktop-file-install, the vendor should be "fedora", not "Fedora". -- 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