[Bug 578981] Review Request: libdrizzle - Drizzle Client & Protocol Library
bugzilla at redhat.com
bugzilla at redhat.com
Fri Apr 2 13:31:44 UTC 2010
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=578981
Martin Gieseking <martin.gieseking at uos.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |martin.gieseking at uos.de
--- Comment #1 from Martin Gieseking <martin.gieseking at uos.de> 2010-04-02 09:31:43 EDT ---
A couple of initial comments:
- drop BR: gcc gcc-c++ (they are picked up automatically)
- drop INSTALL from %doc
- change the defattrs to %defattr(-,root,root,-)
- remove %attr(0755,root,root) as the permissions are set automatically
- .la files must not be packaged. Remove them in the %build section
(see https://fedoraproject.org/wiki/Packaging/Guidelines#StaticLibraries)
- I suggest to use the %{name} macro instead of explicitly stating libdrizzle
(in Source0, %files)
- append a slash to directories, e.g. %{_includedir}/%{name}/
--
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.
More information about the package-review
mailing list