Bug 832178 - Review Request: python-django-picklefield - Implementation of a pickled object field
Summary: Review Request: python-django-picklefield - Implementation of a pickled objec...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Bohuslav "Slavek" Kabrda
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 736776
TreeView+ depends on / blocked
 
Reported: 2012-06-14 18:33 UTC by Matthias Runge
Modified: 2012-06-29 07:33 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-06-25 18:46:55 UTC
Type: ---
Embargoed:
bkabrda: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Matthias Runge 2012-06-14 18:33:59 UTC
Spec URL: http://www.matthias-runge.de/fedora/python-django-picklefield.spec
SRPM URL: http://www.matthias-runge.de/fedora/python-django-picklefield-0.2.1-1.fc17.src.rpm
Description: django-picklefield provides an implementation of a pickled object field.
Such fields can contain any picklable objects.

The implementation is taken and adopted from Django snippet #1694 by
Taavi Taijala, which is in turn based on Django snippet #513 by
Oliver Beattie.
Fedora Account System Username: mrunge

This is a review required for package rename.

[mrunge@sofja SPECS]$ rpmlint ./python-django-picklefield.spec ../SRPMS/python-django-picklefield-0.2.1-1.fc17.src.rpm ../RPMS/noarch/python-django-picklefield-0.2.1-1.fc17.noarch.rpm 
python-django-picklefield.src: W: spelling-error %description -l en_US picklable -> pickle
python-django-picklefield.noarch: W: spelling-error %description -l en_US picklable -> pickle
python-django-picklefield.noarch: E: zero-length /usr/lib/python2.7/site-packages/picklefield/models.py
2 packages and 1 specfiles checked; 1 errors, 2 warnings.

That zero-length file won't harm in any way.

Comment 1 Bohuslav "Slavek" Kabrda 2012-06-18 06:09:21 UTC
I'll take this one for a review.

Comment 2 Bohuslav "Slavek" Kabrda 2012-06-18 07:39:19 UTC
The only thing I would advise here is adding runtime dependency on Django (Requires: python-django), as the package needs Django to do something.

Otherwise I see no problems => The package is APPROVED.

Comment 3 Matthias Runge 2012-06-18 09:07:48 UTC
thank you for the review!

I'll fix the requirement before submitting to scm.

New Package SCM Request
=======================
Package Name: python-django-picklefield
Short Description: Implementation of a pickled object field
Owners: mrunge
Branches: devel

Comment 4 Gwyn Ciesla 2012-06-18 13:04:46 UTC
Git done (by process-git-requests).

Comment 5 Matthias Runge 2012-06-25 18:46:55 UTC
imported and built. Thanks

Comment 6 Matthias Runge 2012-06-29 07:33:26 UTC
forgot to add: https://fedorahosted.org/rel-eng/ticket/5231


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