Bug 541355
Summary: | Review Request: python-couchdb - A Python library for working with CouchDB | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Sebastian Dziallas <sebastian> |
Component: | Package Review | Assignee: | Terje Røsten <terje.rosten> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | byron, fedora-package-review, jspaleta, notting, terje.rosten |
Target Milestone: | --- | Flags: | terje.rosten:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2009-12-17 01:04:09 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Sebastian Dziallas
2009-11-25 17:03:32 UTC
Tiny comments, you might want to more explicit in the %files section. This is a couchdb client? Is then couchdb really needed on the client host? Agreed on the %files section, will do so with the next release. It's, from what I understand, a library for accessing couchdb through python. So couchdb would be required (the author's page says so in the requirements: http://code.google.com/p/couchdb-python/). I was actually packaging it because it's a dependency for desktopcouch, which has so goal to integrate couchdb into desktop applications. More comments: - the srpm linked don't work, however the koji srpm builds fine - please include the doc/ dir - do chmod more elegante e.g. find $RPM_BUILD_ROOT/%{python_sitelib}/couchdb -name \*.py -print0 | xargs --null chmod 0644 - switch to %{__python} in build and install Fix these and I will do a formal review. Hey Terje, If Sebastian is cool with it... I'll make some of the requested edits and push you a new spec file and put myself up as co-maintainer. Opinion as to whether doc/ should be in a docs subpackage or in the main package? Terje, Here's the link to an updated specfile and packages. http://fedorapeople.org/~jspaleta/python-couchdb/ Koji isn't cooperating right now due to the intrastructure work going on i think but I built this locally on my F10 system without issue. I've made a devel subpackage for the api documentation as per the suggestion in the packaging guidelines. I've patched the python files under site-packages directory to remove the shebang as they are not meant to be executable and are not in an executable directory. This has already been filed upstream and should be fixed in later releases. https://bugs.launchpad.net/desktopcouch/+bug/486797 rpmlint is throwing one bogus error message python-couchdb.noarch: E: explicit-lib-dependency python-httplib2 This is a fault alarm.. python-httplib2 needs to be explicitly set as its the python bindins for the httplib2 library. -jef A should a documentation-only package be called -devel? Stranger things has happen. I am still unsure about Requires: couchdb Any new input here? Well, well, this seems good. Will do a proper review soon. - Terje The packaging guidelines on the wiki state that if the documentation is development related... like API documentation it should go in the -devel subpackage instead of -doc. So its a judgement call but in this case the situation exactly fits the example api documentation case. -jef OK, this seems mostly fine ok: rpmlint ok: naming ok: license BSD, every file seems to include header, good ok: language ok: md5sum: 446b8377cf2ddda94d5f2df29c4f705f CouchDB-0.6.tar.gz 446b8377cf2ddda94d5f2df29c4f705f CouchDB-0.6.tar.gz.1 ok: source url ok: dirs ok: perms ok: utf-8 koji is very slow, not possible to do a test build, local build seems fine. package the latest release: 0.6.1 and be more explicit in %files listing (makes things a lot easier for a new maintainer) and I will approve the package. 0.6.1 doesn't need the shebang patch! Is this good enough for the detail level in the files sections? %files %defattr(-,root,root,-) %doc ChangeLog.txt COPYING README.txt %{_bindir}/couchdb-dump %{_bindir}/couchdb-load %{_bindir}/couchdb-replicate %{_bindir}/couchpy %{python_sitelib}/CouchDB-%{version}-py2.5.egg-info %{python_sitelib}/couchdb %files devel %defattr(-,root,root,-) %doc doc/api doc/index.html Nice! Package is APPROVED. New Package CVS Request ======================= Package Name: python-couchdb Short Description: A Python library for working with CouchDB Owners: jspaleta sdz Branches: F-12 F-13 InitialCC: Cool, thanks everybody for taking on this one and sorry for being so calm lately, I've just been a bit swamped with work lately. Nice work! we don't have F-13 branches yet. Otherwise cvs done. rawhide build complete... just needed to make one small buildrequires change to add python-setuptools. |