Bug 495949 (python-kinterbasdb) - Review Request: python-kinterbasdb - A Python DB-API 2.0 compliant interface to Firebird
Summary: Review Request: python-kinterbasdb - A Python DB-API 2.0 compliant interface ...
Alias: python-kinterbasdb
Product: Fedora
Classification: Fedora
Component: Package Review
Version: 10
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Peter Lemenkov
QA Contact: Fedora Extras Quality Assurance
Depends On: 498790
TreeView+ depends on / blocked
Reported: 2009-04-15 17:05 UTC by Philippe Makowski
Modified: 2009-05-14 02:56 UTC (History)
3 users (show)

Fixed In Version: 3.3.0-0.fc11
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-05-13 11:22:08 UTC
Type: ---
lemenkov: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Philippe Makowski 2009-04-15 17:05:16 UTC
Spec URL: http://ibphoenix.fr/fedora/kinterbasdb.spec
SRPM URL: http://ibphoenix.fr/fedora/kinterbasdb-3.3.0-0.fc10.src.rpm
Description: KInterbasDB is a Python extension package that implements Python Database API 2.0-compliant support for the open source relational database Firebird and some versions of its proprietary cousin Borland Interbase.
In addition to the minimal feature set of the standard Python DB API,
KInterbasDB also exposes nearly the entire native client API of the database

Kinterbasdb is part of the Firebird project
I'm member of the Firebird Admin Team, Q/A tester
and I need a sponsor for that rpm

Comment 1 Peter Lemenkov 2009-04-22 05:55:50 UTC
I'll review it (after we'll finish with firebird).

Comment 2 Peter Lemenkov 2009-05-03 12:11:15 UTC
Unblocking FE-NEEDSPONSOR - I sponsored Philippe.

Comment 3 Philippe Makowski 2009-05-07 21:36:27 UTC
can you explain me what is wrong there ?


Comment 4 Peter Lemenkov 2009-05-08 08:45:34 UTC
Your application is arch-specific , that means that it will be installed in %{python_sitearch}, not in %{python_sitelib}.

Just change 




Comment 5 Peter Lemenkov 2009-05-08 08:48:54 UTC
Regarding spec-file, I see the following issues:

* use full path in Source0.
* apply patch with saving original file with some descriptive suffix, e.g. change



%patch0 -b .some_descriptive_suffix

Comment 7 Peter Lemenkov 2009-05-09 13:55:42 UTC
Ok, looks good.

There is one purely bureaucratic issue. In Fedora there is a strange policy that every lang-specific library should be named as lang-libraryname, e.g. according to this rule this package should be named as python-kinterbasdb.

I advice you to rename package to python-kinterbasdb and add another one provides:

Provides: kinterbasdb = %{version}-%{release}

See, for example, http://peter.fedorapeople.org/python-sippy.spec - I was forced to add "Provides:       sippy = %{version}-%{release}"

Comment 9 Peter Lemenkov 2009-05-11 08:33:31 UTC
(In reply to comment #8)
> ok
> so something like that :
> http://ibphoenix.fr/fedora/python-kinterbasdb.spec
> http://ibphoenix.fr/fedora/python-kinterbasdb-3.3.0-0.fc10.src.rpm  

Yes, exactly!
Please, open another review request with proper name (python-kinterbasdb), and mark this one as duplicate of that.

Comment 10 Mamoru TASAKA 2009-05-11 09:20:50 UTC
Just changing Summary

Comment 11 Peter Lemenkov 2009-05-11 09:27:25 UTC
Yeah! Thanks, Mamoru, for the tip.

Comment 12 Peter Lemenkov 2009-05-11 10:28:57 UTC

+/- rpmlint is not silent:

[petro@Sulaco Desktop]$ rpmlint python-kinterbasdb-*
python-kinterbasdb.ppc: W: wrong-file-end-of-line-encoding /usr/share/doc/python-kinterbasdb-3.3.0/README
python-kinterbasdb.src: W: mixed-use-of-spaces-and-tabs (spaces: line 15, tab: line 5)
3 packages and 0 specfiles checked; 0 errors, 2 warnings.
[petro@Sulaco Desktop]$ 

The former warning must be fixed in %prep section. Just add the following line:
sed -i 's|\r||g' README

The latter warning may be ignored, but you should consider suppressing it as well. Not a blocker.

+ The package is named according to the Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format %{name}.spec.

- The package meets the Packaging Guidelines, except leftover in install section. Please remove "%{__python} setup.py install --skip-build --root $RPM_BUILD_ROOT" string. It duplicates by the following string and doesn't explicitly contain -O1. 

Another one issue with this package is duplicated content - it contains doc-files as %doc and inside the %{python_sitearch}/%{origname}/ . I advice you to remove the doc-directory from %{python_sitearch}/%{origname}/ at the %install stage (quick fix), or to patch setup.py (more proper way).

+ The package is licensed with a Fedora approved license and meets the Licensing Guidelines .
+ The License field in the package spec file matches the actual license (BSD)
+ The file, containing the text of the license(s) for the package, is included in %doc (doc/license.html).
+ The spec file is written in American English.
+ The spec file for the package is legible (as almost all python packages - thanks to distutils!)
+ The sources used to build the package matches the upstream source

[petro@Sulaco SOURCES]$ md5sum kinterbasdb-3.3.0.tar.bz2*
0296d2dbc7b76dac140b5fec2ad59f0e  kinterbasdb-3.3.0.tar.bz2
0296d2dbc7b76dac140b5fec2ad59f0e  kinterbasdb-3.3.0.tar.bz2.1
[petro@Sulaco SOURCES]$

+ The package successfully compiles and builds into binary rpms on at least one primary architecture.


+ All build dependencies are listed in BuildRequires.
+ The package owns all directories that it creates.
+ The package does not list a file more than once in the spec file's %files listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissible content.
+ No extremely large documentation files.
+ Everything, the package includes as %doc, does not affect the runtime of the application.
+ The package does not own files or directories already owned by other packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.


* Suppress rpmlint warning(s).
* Get rid of duplicated install attemts in the %install section.
* Remove docs from %{python_sitearch}/%{origname}/ or explain why it's necessary to install them here.

Comment 14 Peter Lemenkov 2009-05-11 12:29:27 UTC
Good. The package is


Comment 15 Philippe Makowski 2009-05-11 12:46:35 UTC
New Package CVS Request
Package Name: python-kinterbasdb
Short Description: A Python DB-API 2.0 compliant interface to Firebird
Owners: makowski
Branches: F-9 F-10 F-11 EL-4 EL-5 

Comment 16 Kevin Fenzi 2009-05-13 05:20:45 UTC
cvs done.

Comment 17 Fedora Update System 2009-05-13 11:20:23 UTC
python-kinterbasdb-3.3.0-0.fc10 has been submitted as an update for Fedora 10.

Comment 18 Fedora Update System 2009-05-13 11:20:29 UTC
python-kinterbasdb-3.3.0-0.fc9 has been submitted as an update for Fedora 9.

Comment 19 Fedora Update System 2009-05-13 11:20:34 UTC
python-kinterbasdb-3.3.0-0.fc11 has been submitted as an update for Fedora 11.

Comment 20 Fedora Update System 2009-05-14 02:52:04 UTC
python-kinterbasdb-3.3.0-0.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 21 Fedora Update System 2009-05-14 02:55:59 UTC
python-kinterbasdb-3.3.0-0.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 22 Fedora Update System 2009-05-14 02:56:43 UTC
python-kinterbasdb-3.3.0-0.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.