Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Summary: Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: wdierkes@rackspace.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
SPEC: http://5dollarwhitebox.org/fedora/python-cloudfiles.spec SRPMS: http://5dollarwhitebox.org/fedora/python-cloudfiles-1.7.7-1.fc14.src.rpm
Description: python-cloudfiles provides a simple interface to the Rackspace Cloud Files service. "Cloud Files is reliable, scalable and affordable web-based storage for backing up and archiving all your static content". Find out more at http://www.rackspacecloud.com/cloud_hosting_products/files.
$ rpmlint -i SPECS/python-cloudfiles.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings.
$ rpmlint -i RPMS/noarch/python-cloudfiles-1.7.7-1.fc14.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
BJ Dierkes wdierkes@rackspace.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |acturneruk@gmail.com
--- Comment #1 from BJ Dierkes wdierkes@rackspace.com 2011-01-31 19:04:03 EST --- *** Bug 542436 has been marked as a duplicate of this bug. ***
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Rahul Sundaram metherid@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |metherid@gmail.com Flag| |fedora-review+
--- Comment #2 from Rahul Sundaram metherid@gmail.com 2011-02-10 21:15:30 EST ---
http://download.github.com/rackspace-python-cloudfiles-1.7.7-0-ge8fd112.tar.... is not accessible. Fix that
APPROVED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #3 from Rahul Sundaram metherid@gmail.com 2011-02-15 09:57:54 EST --- Ping?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #4 from BJ Dierkes wdierkes@rackspace.com 2011-02-15 11:57:46 EST --- Sorry for the delay... been really busy.
So, I found that the URL for the download is a cache.... and removed after some time. I really am not sure how best to handle GitHub downloads as the upstream source. I spoke with the maintainers upstream and they feel its a non-issue.... and there has to be tons of other fedora packages that use GitHub as upstream.
Not sure how to move forward.... but let me revisit this all again today.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
BJ Dierkes wdierkes@rackspace.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?
--- Comment #5 from BJ Dierkes wdierkes@rackspace.com 2011-02-16 16:53:36 EST --- I've started a thread on the devel list... definitely not the first package to have issues with GitHub. For now i've updated Source0 to use the PyPi download location (also maintained by the upstream vendor).
SPEC: http://5dollarwhitebox.org/fedora/python-cloudfiles.spec SRPM: http://5dollarwhitebox.org/fedora/python-cloudfiles-1.7.8-1.fc14.src.rpm
$ rpmlint -i SPECS/python-cloudfiles.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings.
$ rpmlint -i RPMS/noarch/python-cloudfiles-1.7.8-1.fc14.noarch.rpm python-cloudfiles.noarch: W: no-documentation The package contains no documentation (README, doc, etc). You have to include documentation files.
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
Working with upstream to fix their source tarbal to include ChangeLog/COPYING etc.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Rahul Sundaram metherid@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo? |
--- Comment #6 from Rahul Sundaram metherid@gmail.com 2011-03-01 05:38:47 EST ---
Can you import this package into rawhide and Fedora 15? I need it for enabling rackspace cloud support in Deja-dup and we are nearing the Alpha release.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #7 from BJ Dierkes wdierkes@rackspace.com 2011-03-01 15:12:51 EST --- I can't do anything until a formal review is completed.
http://fedoraproject.org/wiki/Package_Review_Process#Reviewer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #8 from BJ Dierkes wdierkes@rackspace.com 2011-03-01 15:16:27 EST --- I see you added the flag... but you gotta post more than just 'approved':
Include the text of your review in a comment in the ticket. For easy readability, simply use a regular comment instead of an attachment.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #9 from Rahul Sundaram metherid@gmail.com 2011-03-01 23:00:13 EST ---
*shrug* there isn't much of a review text here and I haven't added any attachments. If you want a checklist of items, here you go:
* Follows the naming guidelines * rpmlint has no warnings and errors * License is acceptable and the license tag is correct * Redundant but harmless buildroot definition and %clean section * Verified build and run time requirements with a mock build * %doc should include license information. Not done. * Macro definitions follows guidelines
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #10 from BJ Dierkes wdierkes@rackspace.com 2011-03-02 12:57:49 EST --- Sorry, the above was a quote from Package Review Process. The text of the review is the 'pass' / 'fail' output for each item of the Must Have's, and Should Have's check list:
http://fedoraproject.org/wiki/Packaging:ReviewGuidelines
I use the following:
http://fedoraproject.org/wiki/User:Jlaska/Package_Review_Checklist
I'm not trying to be a hard ass.... just, following the standard process. Also, please assign the bug to you if you as you have/are doing the review.
Thanks.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Rahul Sundaram metherid@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |metherid@gmail.com
--- Comment #11 from Rahul Sundaram metherid@gmail.com 2011-03-03 00:37:24 EST ---
There isn't a standard check list. Just a whole bunch of different ones floating around. I have checked it against what applies in this case.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #12 from BJ Dierkes wdierkes@rackspace.com 2011-03-04 14:11:48 EST --- New Package SCM Request ======================= Package Name: python-cloudfiles Short Description: Python language bindings for Rackspace CloudFiles API Owners: derks jeffreyness Branches: EL5 EL6 FC14 FC15 devel InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
BJ Dierkes wdierkes@rackspace.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #13 from BJ Dierkes wdierkes@rackspace.com 2011-03-04 14:12:52 EST --- New Package SCM Request ======================= Package Name: python-cloudfiles Short Description: Python language bindings for Rackspace CloudFiles API Owners: derks jeffreyness Branches: el5 el6 f14 f15 InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #14 from Jason Tibbitts tibbs@math.uh.edu 2011-03-04 16:20:21 EST --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |MODIFIED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2011-03-04 20:14:57 EST --- python-cloudfiles-1.7.8-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/python-cloudfiles-1.7.8-1.fc14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #16 from Fedora Update System updates@fedoraproject.org 2011-03-04 20:15:00 EST --- python-cloudfiles-1.7.8-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/python-cloudfiles-1.7.8-1.el5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #17 from Fedora Update System updates@fedoraproject.org 2011-03-04 20:15:13 EST --- python-cloudfiles-1.7.8-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-cloudfiles-1.7.8-1.el6
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #18 from Fedora Update System updates@fedoraproject.org 2011-03-04 20:15:16 EST --- python-cloudfiles-1.7.8-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/python-cloudfiles-1.7.8-1.fc15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #19 from Fedora Update System updates@fedoraproject.org 2011-03-05 14:23:32 EST --- python-cloudfiles-1.7.8-1.fc15 has been pushed to the Fedora 15 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 python-cloudfiles'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/python-cloudfiles-1.7.8-1.fc15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #20 from Fedora Update System updates@fedoraproject.org 2011-03-08 22:08:16 EST --- python-cloudfiles-1.7.8-1.fc15 has been pushed to the Fedora 15 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |python-cloudfiles-1.7.8-1.f | |c15 Resolution| |ERRATA Last Closed| |2011-03-08 22:08:21
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #21 from Fedora Update System updates@fedoraproject.org 2011-03-13 17:21:28 EDT --- python-cloudfiles-1.7.8-1.fc14 has been pushed to the Fedora 14 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|python-cloudfiles-1.7.8-1.f |python-cloudfiles-1.7.8-1.f |c15 |c14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #22 from Fedora Update System updates@fedoraproject.org 2011-03-22 12:30:55 EDT --- python-cloudfiles-1.7.8-1.el5 has been pushed to the Fedora EPEL 5 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|python-cloudfiles-1.7.8-1.f |python-cloudfiles-1.7.8-1.e |c14 |l5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
--- Comment #23 from Fedora Update System updates@fedoraproject.org 2011-03-22 12:32:12 EDT --- python-cloudfiles-1.7.8-1.el6 has been pushed to the Fedora EPEL 6 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=674207
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|python-cloudfiles-1.7.8-1.e |python-cloudfiles-1.7.8-1.e |l5 |l6
package-review@lists.fedoraproject.org