| Summary: | Review Request: libldb - A schema-less, ldap like, API and database | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Stephen Gallagher <sgallagh> |
| Component: | Package Review | Assignee: | Matthew Barnes <mbarnes> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, mbarnes, notting |
| Target Milestone: | --- | Flags: | mbarnes:
fedora-review+
kevin: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | libldb-0.9.22-8.fc15 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2011-02-07 15:49:44 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
|
Description
Stephen Gallagher
2011-01-28 11:47:49 UTC
Successfully scratch-built in Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2747385 rpmlint output: libldb.i686: W: spelling-error Summary(en_US) ldap -> lap, dap, leap libldb.i686: W: spelling-error %description -l en_US tdb -> db, tab, tub Obviously bogus. libldb.i686: W: shared-lib-calls-exit /usr/lib/libldb.so.0.9.22 exit I assume this is okay? Not sure what exit() call it's complaining about. pyldb.i686: W: private-shared-object-provides /usr/lib/python2.7/site-packages/ldb.so ldb.so Python C modules => bogus. libldb.i686: W: no-documentation pyldb.i686: W: no-documentation pyldb-devel.i686: W: no-documentation Didn't see any relevant documentation files in the tarball. Are there any license or README files from Samba that would be appropriate to include? * MUST: The License field in the package spec file must match the actual license.
Seems to be the case from the random sample of source files I looked at, but a LICENSE or COPYING file in the top-level directory would be more helpful.
* MUST: Packages must NOT bundle copies of system libraries.
The /lib directory contains several libraries which are packaged separately in Fedora, but they don't appear to be used during building nor are they installed. Just wanted to confirm that this is true.
Other:
* Package builds fine under mock in a Rawhide environment.
* The BuildRoot tag isn't needed in current Fedora releases, but I see
conditionals for older Fedora / RHEL releases. Not sure which releases
you're targeting. Might be needed for older releases, just wanted to
mention it.
* In the "tools" subpackage I see:
Requires: libldb >= %{version}-%{release}
Should that be '=' instead of '>='? It's '=' in all other subpackages.
Other than these nitpicky things, it looks good to me.
The exit() call it's complaining about isn't actually reachable by libldb. It's compiled in from part of the statically-linked libreplace from samba. So that's safe to ignore. Upstream didn't package any license or docs in the tarball, so I can't package them in the RPMs. I will ask upstream to include COPYING for the next release. The libraries in /lib are available to be built in-tree with libldb, but the configure flag --bundled-libraries=NONE forces the build to use system libraries instead. The BuildRoot I just copied from an older (RHEL 5) spec file for libldb. It's not harmful, but I can remove it if you prefer. Whoops, you are right about = vs >= Once you reply about the BuildRoot, I'll spin another version and resubmit. Thanks for the review! As I understand it the BuildRoot tag isn't forbidden, just unnecessary now. I don't really care either way; I'll leave that one up to you. I'm satisfied with everything else. Spec URL: http://sgallagh.fedorapeople.org/packagereview/libldb/libldb.spec SRPM URL: http://sgallagh.fedorapeople.org/packagereview/libldb/libldb-0.9.22-8.fc14.src.rpm Successfully scratch-built in Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2761817 Looks good, package approved. New Package SCM Request ======================= Package Name: libldb Short Description: A schema-less, ldap-like API and database Owners: sgallagh Branches: InitialCC: (Branches was left blank above because only rawhide is requested) undepreciated the existing orphaned devel branch and made you owner. Built in rawhide |