Bug 517412 - Review Request: django-profile - Django pluggable user profile zone
Summary: Review Request: django-profile - Django pluggable user profile zone
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Allisson Azevedo
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-08-13 19:23 UTC by Diego Búrigo Zacarão
Modified: 2009-09-03 23:18 UTC (History)
3 users (show)

Fixed In Version: 0.6-0.2.20090813svnr420.el5
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-08-18 21:16:22 UTC
Type: ---
Embargoed:
allisson: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Diego Búrigo Zacarão 2009-08-13 19:23:13 UTC
Spec URL: http://diegobz.fedorapeople.org/packages/django-profile/django-profile.spec
SRPM URL: http://diegobz.fedorapeople.org/packages/django-profile/django-profile-0.6-0.1.20090813svnr420.fc11.src.rpm

Description: 

This is a user private zone/profile management application, allowing
the user to take control of his account and insert information about
him in his profile.

Inside this package you will find a demo application which will show
you what can be accomplished with the rest of the utilities included
in the package.

This package is needed to suppress a dependency for the new Transifex 0.7-RC1.

Comment 1 Allisson Azevedo 2009-08-13 22:29:53 UTC
I'll make review soon as posible :)

Comment 2 Allisson Azevedo 2009-08-14 16:03:54 UTC
Well,

BAD:

* rpmlint issues:

django-profile.noarch: W: incoherent-version-in-changelog 0.6-0.1.20090813svn420 ['0.6-0.1.20090813svnr420.fc11', '0.6-0.1.20090813svnr420']
django-profile.noarch: W: no-documentation
django-profile.noarch: E: non-executable-script /usr/lib/python2.6/site-packages/demo/manage.py 0644 /usr/bin/env
1 packages and 0 specfiles checked; 1 errors, 2 warnings.

Please fix this issues.

* no %doc files:

Please add CHANGELOG.txt, LICENSE.txt, README.txt and TODO.txt in %doc section.

* License mismatch:

setup.py informs a MIT license, but, LICENSE.txt is a BSD license template, see those templates:

http://www.opensource.org/licenses/mit-license.php
http://www.opensource.org/licenses/bsd-license.php

Comment 3 Diego Búrigo Zacarão 2009-08-14 19:04:32 UTC
Spec URL:
http://diegobz.fedorapeople.org/packages/django-profile/django-profile.spec
SRPM URL:
http://diegobz.fedorapeople.org/packages/django-profile/django-profile-0.6-0.2.20090813svnr420.fc11.src.rpm

* Fri Aug 14 2009 Diego Búrigo Zacarão <diegobz> 0.6-0.2.20090813svnr420
- Added docs
- Fixed version in the previous changelog
- Fixed License to BSD based on the LICENSE.txt file


Is it enough to change the license to BSD?
I've opened a ticket upstream though.
http://code.google.com/p/django-profile/issues/detail?id=64

Comment 4 Allisson Azevedo 2009-08-14 19:57:31 UTC
BAD:

* rpmlint

django-profile.noarch: E: non-executable-script /usr/lib/python2.6/site-packages/demo/manage.py 0644 /usr/bin/env
1 packages and 0 specfiles checked; 1 errors, 0 warnings.

I think this demo program at (/usr/lib/python2.6/site-packages/demo/) should be moved to docs.

Comment 5 Diego Búrigo Zacarão 2009-08-14 20:33:27 UTC
Updated!

Spec URL: http://diegobz.fedorapeople.org/packages/django-profile/django-profile.spec
SRPM URL: http://diegobz.fedorapeople.org/packages/django-profile/django-profile-0.6-0.2.20090813svnr420.fc11.src.rpm

* Fri Aug 14 2009 Diego Búrigo Zacarão <diegobz> 0.6-0.2.20090813svnr420
(...)
- Moved demo app to docs

Comment 6 Allisson Azevedo 2009-08-14 21:11:42 UTC
Well,

You added demo/ in %doc but still here in /usr/lib/python2.6/site-packages/demo/ :)

Modify %files section and add a %exclude like this:

%files
%defattr(-,root,root,-)
%doc INSTALL.txt CHANGELOG.txt LICENSE.txt README.txt TODO.txt demo/
%{python_sitelib}/*
%exclude %{python_sitelib}/demo

Comment 7 Diego Búrigo Zacarão 2009-08-14 23:56:13 UTC
Ops... didn't know about it. :)

Spec URL: http://diegobz.fedorapeople.org/packages/django-profile/django-profile.spec
SRPM URL: http://diegobz.fedorapeople.org/packages/django-profile/django-profile-0.6-0.2.20090813svnr420.fc11.src.rpm

%changelog
* Fri Aug 14 2009 Diego Búrigo Zacarão <diegobz> 0.6-0.2.20090813svnr420
(...)
- Excluded demo app from python path

Comment 8 Allisson Azevedo 2009-08-15 02:12:46 UTC
It's ok now :)

APPROVED

Comment 9 Diego Búrigo Zacarão 2009-08-15 13:53:11 UTC
Thank you, Allisson.

New Package CVS Request
=======================
Package Name: django-profile
Short Description: Django pluggable user profile zone
Owners: diegobz
Branches: F-10 F-11 EL-5
InitialCC: diegobz

Comment 10 Kevin Fenzi 2009-08-17 18:27:52 UTC
cvs done.

Comment 11 Fedora Update System 2009-08-17 20:25:59 UTC
django-profile-0.6-0.2.20090813svnr420.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/django-profile-0.6-0.2.20090813svnr420.fc11

Comment 12 Fedora Update System 2009-08-17 20:26:36 UTC
django-profile-0.6-0.2.20090813svnr420.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/django-profile-0.6-0.2.20090813svnr420.el5

Comment 13 Fedora Update System 2009-08-18 00:24:54 UTC
django-profile-0.6-0.2.20090813svnr420.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 django-profile'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/EL-5/FEDORA-EPEL-2009-0293

Comment 14 Fedora Update System 2009-08-18 21:16:16 UTC
django-profile-0.6-0.2.20090813svnr420.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2009-09-03 23:18:25 UTC
django-profile-0.6-0.2.20090813svnr420.el5 has been pushed to the Fedora EPEL 5 stable repository.  If problems still persist, please make note of it in this bug report.


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