Bug 225346
Summary: | Review Request: python-pgsql - Enhanced python interface to PostgreSQL | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Konstantin Ryabitsev <icon> |
Component: | Package Review | Assignee: | Kevin Fenzi <kevin> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | kevin |
Target Milestone: | --- | Flags: | kevin:
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: | 2007-06-10 13:52:39 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
Konstantin Ryabitsev
2007-01-30 00:05:26 UTC
Hey Konstantin. Do you still wish to submit this package? If so, could you update to the latest version (0.9.5 seems to be the latest upstream). Also, is the python2.3 patch needed? What versions are you targeting with this package? Absolutely. http://blues.mcgill.ca/~icon/fe/python-pgsql.spec http://blues.mcgill.ca/~icon/fe/python-pgsql-0.9.5-1.el4.wsg.src.rpm Version 0.9.5 works with python-2.3 without the patch. I'd like to provide this package for EL4, hence why I care about python-2.3. :) ok. I would be happy to review this package... look for a full review in a few. OK - Package meets naming and packaging guidelines OK - Spec file matches base package name. OK - Spec has consistant macro usage. OK - Meets Packaging Guidelines. OK - License (PSF) OK - License field in spec matches OK - License file included in package OK - Spec in American English OK - Spec is legible. OK - Sources match upstream md5sum: 8c56355c93c88efacaba94119283175e python-pgsql-0.9.5.tar.gz 8c56355c93c88efacaba94119283175e python-pgsql-0.9.5.tar.gz.1 OK - BuildRequires correct See below - Package has %defattr and permissions on files is good. OK - Package has a correct %clean section. OK - Package has correct buildroot OK - Package is code or permissible content. OK - Packages %doc files don't affect runtime. OK - Package has rm -rf RPM_BUILD_ROOT at top of %install OK - Package compiles and builds on at least one arch. OK - Package has no duplicate files in %files. OK - Package doesn't own any directories other packages own. OK - Package owns all the directories it creates. See below - No rpmlint output. OK - final provides and requires are sane. SHOULD Items: OK - Should build in mock. OK - Should build on all supported archs OK - Should have dist tag OK - Should package latest version Issues: 1. Our friend rpmlint says: E: python-pgsql non-executable-script /usr/lib/python2.5/site-packages/pgsql.py 0644 Should this be mode 755? or more likely not have the '#!/usr/bin/env python' at the top? W: python-pgsql invalid-license PSF W: python-pgsql invalid-license PSF W: python-pgsql-debuginfo invalid-license PSF rpmlint wants you to call this: "Python Software Foundation License" Both of those are pretty minor. Provided you fix them before importing I will go ahead and APPROVE this package. I've adjusted the license name to appease rpmlint, but removing the "#!/usr/bin/env python" is a bit too anal, in my opinion. :) It's completely inoffensive where it is, and just complicates the specfile needlessly. Will request CVS bits in a moment. New Package CVS Request ======================= Package Name: python-pgsql Short Description: Enhanced python interface to PostgreSQL Owners: mricon Branches: FC-6 F-7 EL-4 EL-5 InitialCC: icon Well, the "#!/usr/bin/env python" on that script might make someone think they can run it directly, but yes, thats a minor issue. cvs done. Thanks bunches! |