Bug 667155
Summary: | Review Request: python26-httplib2 - A comprehensive HTTP client library | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Jeffrey Ness <jeffrey.ness> |
Component: | Package Review | Assignee: | Toshio Ernie Kuratomi <a.badger> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | a.badger, fedora-package-review, jochen, notting |
Target Milestone: | --- | Flags: | a.badger:
fedora-review+
j: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | python26-httplib2-0.6.0-2.el5 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2011-02-13 00:21:28 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
Jeffrey Ness
2011-01-04 16:50:09 UTC
Simple question: Can we get this package for python-2.7 or python-3? Jochen, This package is geared toward EPEL and currently the only version of Python maintained in EPEL is Python26, in the future if additional versions are added it shouldn't be to hard to port this package over (This however relies on my getting sponsored): http://download.fedora.redhat.com/pub/epel/5/x86_64/repoview/letter_p.group.html NEEDSWORK Good: * rpmlint clean * Package follows naming guidelines * Spec file name matches package name * License is MIT in source and spec file * MIT is an open source license * Spec file is legible American English * License is included in the README which is included in the package * Source matches upstream: sha256sum httplib2-0.6.0.tar.gz 58de068e5af5661c566abb871a75133cc8297e6473ff3f15e98079d57da3978c httplib2-0.6.0.tar.gz * Builds in koji * All build deps satisfied but see below; there's some extra ones. * No locale files that need to be marked with %find_lang * No shared libraries * No bundled libraries * Package is not relocatable * No files listed more than once * All files and directories created by the package owned by the package and no others. * Package contains code, not content. * No large documentation that needs to be in a separate subpackage * Nothing in %doc used at runtime * No GUI application included so no .desktop requirement * All filenames are valid utf-8 * No scriptlets * No file dependencies * No programs so no need for man pages Needswork: * Don't need python26-setuptools that I can see * In the %build section, need to either use %{__python} or python2.6 rather than "python" * The library comes with a test suite. You should run it in a %check section. If it needs to be able to contact the public internet you may need to disable those particular tests. If the test suite is just worthless without access to the public internet, please add a comment to the spec file in the %check section that says why the testsuite isn't being run. Cosmetic: * Would be better to name python_sitelib python26_sitelib to avoid ambiguity. * We normaly don't have A, The, or other articles in the summary: Summary: Comprehensive HTTP client library Fix at least the Needswork entries and I can approve the package. You also need to be sponsored it looks like and I am a sponsor. I see that you have some reviews in progress:: https://bugzilla.redhat.com/show_bug.cgi?id=639874 https://bugzilla.redhat.com/show_bug.cgi?id=668591 https://bugzilla.redhat.com/show_bug.cgi?id=670457 Especially since you are seeking sponsorship, your reviews should look more like mine - with a list of all the things you checked and found were good as well as the things that were in need of changing. This helps to show that you've read the packaging guidelines and remember at least the Review Guidelines: https://fedoraproject.org/wiki/Packaging:ReviewGuidelines I'll go ahead and CC myself on those package reviews and when you complete them, I'll sponsor you and you can finish approving them. If you have questions you can reach me via email or (better) on irc.freenode.net -- I'm abadger1999. Hello Toshio, Thank you very much for your time. I have looked over your review and corrected the 'Needswork' as well as the 'Cosmetic' issues you listed above, the new SPEC and SRPM can be found here: SPEC: http://flip-edesign.com/source/python26-httplib2/python26-httplib2.spec SRPM: http://flip-edesign.com/source/python26-httplib2/python26-httplib2-0.6.0-2.el5.src.rpm This package has successfully built on mock using configuration 'epel-5-i386', and checks with rpmlint without errors. I also wanted to thank you for the great pointers you gave on package reviews, I will be sure to use this approach in my future reviews as well as the 3 review you mentioned above. If I have any troubles I will hit you up on IRC, my handle will be "jness" Thanks Jeffrey- All mentioned problems taken care of. This package is APPROVED. Reading through the reviews you're doing in order to sponsor you now. And sponsored. Welcome aboard! You can now officially approve the packages you've been reviewing if the packager doesn't need a sponsor. (So it looks like you can do that for python-rocket and python26-markupsafe -- I'm querying the toilet packager to see if I can sponsor him.) The next steps for this package are on: https://fedoraproject.org/wiki/Package_Review_Process#Contributor You're on step 4, requesting that the package be added to the git repo via an SCM admin request. Since I sponsored you, feel free to ask me any questions you may have. New Package SCM Request ======================= Package Name: python26-httplib2 Short Description: A comprehensive HTTP client library that supports many features left out of other HTTP libraries. Owners: jeffreyness derks Branches: el5 Git done (by process-git-requests). python26-httplib2-0.6.0-2.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/python26-httplib2-0.6.0-2.el5 python26-httplib2-0.6.0-2.el5 has been pushed to the Fedora EPEL 5 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update python26-httplib2'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/python26-httplib2-0.6.0-2.el5 python26-httplib2-0.6.0-2.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report. |