Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: pymongo - Python driver for MongoDB
https://bugzilla.redhat.com/show_bug.cgi?id=605413
Summary: Review Request: pymongo - Python driver for MongoDB Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: ian@ianweller.org QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://ianweller.fedorapeople.org/SRPMS/pymongo/1.6-1/pymongo.spec SRPM URL: http://ianweller.fedorapeople.org/SRPMS/pymongo/1.6-1/pymongo-1.6-1.fc13.src... Description: The PyMongo distribution contains tools for interacting with MongoDB database from Python. The pymongo package is a native Python driver for MongoDB. The gridfs package is a gridfs implementation on top of pymongo.
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=605413
Thomas Spura tomspur@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |tomspur@fedoraproject.org
--- Comment #1 from Thomas Spura tomspur@fedoraproject.org 2010-06-19 11:57:14 EDT --- Just a few comments for now:
- There is a new version, released just seconds, after your review request ;)
- There is an anoying private-shared-object-provides warning from rpmlint. This can be easily grepped out of the provides list, but that's only a workaround.
I just mailed the python list about this and hopefully this will be fixed in the find-requires script and not workaround this in any python package, that's not noarch... http://lists.fedoraproject.org/pipermail/python-devel/2010-June/000260.html
- Didn't have a close look yet, but on upstreams page is a testsuite mentioned, could you please add a %check section?
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=605413
--- Comment #2 from Ian Weller ian@ianweller.org 2010-06-19 15:01:38 EDT --- (In reply to comment #1)
- There is a new version, released just seconds, after your review request ;)
Well, that's irritating. ;)
- Didn't have a close look yet, but on upstreams page is a testsuite mentioned,
could you please add a %check section?
The test suite requires a running mongoDB server so I don't think that's something we want to hack together in %check.
New spec in a bit, just wanted to respond to that last thing before I forgot.
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=605413
Marian Csontos mcsontos@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mcsontos@redhat.com
--- Comment #3 from Marian Csontos mcsontos@redhat.com 2010-09-03 06:49:58 EDT --- Though I do not like uppercase, may be using PyMongo as name would be more appropriate: the project refers to itself PyMongo everywhere.
This distinguishes between PyMongo project and pymongo python package.
IIUC PyMongo == pymongo + gridfs
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=605413
Matthias Runge mrunge@matthias-runge.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mrunge@matthias-runge.de
--- Comment #4 from Matthias Runge mrunge@matthias-runge.de 2010-09-06 05:45:38 EDT --- As Marian suggested, you should probably divide gridfs and pymongo or make at least subpackages. e.g. PyMongo-gridfs and PyMongo-pymongo.
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=605413
--- Comment #5 from Matthias Runge mrunge@matthias-runge.de 2010-09-23 03:26:15 EDT --- Ping Ian.
Any progress? mongodb is now a package included in fedora etc.
https://admin.fedoraproject.org/pkgdb/acls/name/mongodb/
How can I support you?
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=605413
Silas Sewell silas@sewell.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |silas@sewell.ch
--- Comment #6 from Silas Sewell silas@sewell.ch 2010-09-28 11:10:07 EDT --- Here is my personal spec that addresses some things mentioned above (as you can see I prefer lowercase names, something Debian gets right).
http://github.com/silas/rpms/raw/master/pymongo/pymongo.spec
Ian Weller if you update your spec I'd be happy to review it, otherwise I'd also be happy to take the request over if you're not interested anymore.
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=605413
Nathaniel McCallum nathaniel@natemccallum.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |nathaniel@natemccallum.com
--- Comment #7 from Nathaniel McCallum nathaniel@natemccallum.com 2010-10-10 23:50:19 EDT --- Silas, if you want to run with it, I'll do the review. Or vice-versa. I just want it packaged. ;)
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=605413
--- Comment #8 from Ian Weller ian@ianweller.org 2010-10-10 23:51:46 EDT --- Either of you can go ahead and take it, I've got no time. Sorry for the delays guys :(
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=605413
--- Comment #9 from Nathaniel McCallum nathaniel@natemccallum.com 2010-10-11 00:52:54 EDT --- Created attachment 452639 --> https://bugzilla.redhat.com/attachment.cgi?id=452639 pymongo spec with el5 fixes
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=605413
--- Comment #10 from Matthias Runge mrunge@matthias-runge.de 2010-10-11 03:23:48 EDT --- Oh great, guys. If you need any help, I'd jump in.
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=605413
Thomas Spura tomspur@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution| |NOTABUG Last Closed| |2010-10-11 03:55:04
--- Comment #11 from Thomas Spura tomspur@fedoraproject.org 2010-10-11 03:55:04 EDT --- (In reply to comment #8)
Either of you can go ahead and take it, I've got no time. Sorry for the delays guys :(
-> Closing as NOTABUG
Anyone, who wants to continue the review of pymongo needs to open a own review request and close this one as a dub of the new one, see: http://fedoraproject.org/wiki/Extras/Policy/StalledReviews#Submitter_not_res...
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=605413
Silas Sewell silas@sewell.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|NOTABUG |DUPLICATE
--- Comment #12 from Silas Sewell silas@sewell.ch 2010-10-19 22:14:59 EDT ---
*** This bug has been marked as a duplicate of bug 644708 ***
package-review@lists.fedoraproject.org