Bug 1435876

Summary: Review Request: python-django-picklefield - Pickled object field for Django
Product: [Fedora] Fedora Reporter: Aurelien Bompard <aurelien>
Component: Package ReviewAssignee: Neal Gompa <ngompa13>
Status: CLOSED DUPLICATE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: jskarvad, ngompa13, package-review
Target Milestone: ---Flags: ngompa13: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-07-22 10:42:13 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:
Bug Depends On: 1458493    
Bug Blocks:    

Description Aurelien Bompard 2017-03-25 09:13:16 UTC
Spec URL: https://abompard.fedorapeople.org/reviews/python-django-picklefield/python-django-picklefield.spec
SRPM URL: https://abompard.fedorapeople.org/reviews/python-django-picklefield/python-django-picklefield-0.3.2-1.fc27.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: abompard

There's an rpmlint error: "E: zero-length /usr/lib/python2.7/site-packages/picklefield/models.py". We can't avoid it because some older versions of Django requires this file to be present.

Comment 1 Neal Gompa 2017-06-03 12:01:38 UTC
Taking this review.

Comment 2 Neal Gompa 2017-06-03 12:10:41 UTC
This package failed to run through fedora-review. The %check section failed:

Executing(%check): /bin/sh -e /var/tmp/rpm-tmp.swppdB
+ umask 022
+ cd /builddir/build/BUILD
+ cd django-picklefield-0.3.2
+ export PYTHONPATH=src:.
+ PYTHONPATH=src:.
+ /usr/bin/python2 /usr/bin/django-admin test --settings test_settings picklefield
Traceback (most recent call last):
  File "/usr/bin/django-admin", line 2, in <module>
    from django.core import management
  File "/usr/lib/python2.7/site-packages/django/core/management/__init__.py", line 13, in <module>
    from django.core.management.base import (
  File "/usr/lib/python2.7/site-packages/django/core/management/base.py", line 17, in <module>
    from django.db.migrations.exceptions import MigrationSchemaMissing
  File "/usr/lib/python2.7/site-packages/django/db/migrations/__init__.py", line 2, in <module>
    from .operations import *  # NOQA
  File "/usr/lib/python2.7/site-packages/django/db/migrations/operations/__init__.py", line 1, in <module>
    from .fields import AddField, AlterField, RemoveField, RenameField
  File "/usr/lib/python2.7/site-packages/django/db/migrations/operations/fields.py", line 3, in <module>
    from django.db.models.fields import NOT_PROVIDED
  File "/usr/lib/python2.7/site-packages/django/db/models/__init__.py", line 3, in <module>
    from django.db.models.aggregates import *  # NOQA
  File "/usr/lib/python2.7/site-packages/django/db/models/aggregates.py", line 5, in <module>
    from django.db.models.expressions import Func, Star
  File "/usr/lib/python2.7/site-packages/django/db/models/expressions.py", line 5, in <module>
    from django.db.backends import utils as backend_utils
  File "/usr/lib/python2.7/site-packages/django/db/backends/utils.py", line 12, in <module>
    from django.utils.timezone import utc
  File "/usr/lib/python2.7/site-packages/django/utils/timezone.py", line 8, in <module>
    import pytz
ImportError: No module named pytz
error: Bad exit status from /var/tmp/rpm-tmp.swppdB (%check)
    Bad exit status from /var/tmp/rpm-tmp.swppdB (%check)


It looks like the failure is caused by python2-django not pulling in python2-pytz. The same issue likely exists for python3-django (with python3-pytz).

Comment 3 Aurelien Bompard 2017-06-16 13:28:15 UTC
The python-django package has been fixed in Rawhide only, so I'll just add pytz in the dependencies for now.

Spec URL: https://abompard.fedorapeople.org/reviews/python-django-picklefield/python-django-picklefield.spec
SRPM URL: https://abompard.fedorapeople.org/reviews/python-django-picklefield/python-django-picklefield-0.3.2-1.fc27.src.rpm

Comment 5 Neal Gompa 2017-06-23 02:41:13 UTC
So, apparently, this package existed before and was retired. I'm going to treat this an unretirement request, then, and still proceed with the review.

Comment 6 Neal Gompa 2017-06-23 02:42:01 UTC
Review notes:

- Follows Python packaging guidelines
- Follows extended guidelines for EPEL
- Builds and installs
- No rpmlint issues
- No fedora-review issues

PACKAGE APPROVED.

Comment 7 Neal Gompa 2017-06-23 10:52:51 UTC
Oh dear, there were two parallel reviews going on for python-django-picklefield...

Comment 8 Jaroslav Škarvada 2017-06-23 11:37:00 UTC
(In reply to Neal Gompa from comment #7)
> Oh dear, there were two parallel reviews going on for
> python-django-picklefield...

Thanks for info, this is unlucky. Could Igor and Aurelien agree on further steps? I.e. who will maintain it?

Comment 9 Jaroslav Škarvada 2017-06-23 11:38:36 UTC
It's worth to point out that Igor's request was filled first.

Comment 10 Neal Gompa 2017-06-23 11:48:53 UTC
It's really up to Aurelien and Igor. This is a dependency for the Mailman 3 suite, which is why Aurelian was packaging it.

Comment 11 Igor Gnatenko 2017-07-22 10:42:13 UTC

*** This bug has been marked as a duplicate of bug 1370644 ***