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


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[EMAIL PROTECTED]




------- Additional Comments From [EMAIL PROTECTED]  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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to