Bug 494219

Summary: Review Request: trac-accountmanager-plugin - Account manager plugin for trac
Product: [Fedora] Fedora Reporter: Lorenzo Villani <lorenzo>
Component: Package ReviewAssignee: Paul Howarth <paul>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: fedora-package-review, notting, paul
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-07-22 09:06:08 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 201449    

Description Lorenzo Villani 2009-04-05 13:46:33 EDT
Spec URL: http://git.binaryhelix.net/cgit/rpms/plain/trac-accountmanager-plugin/trac-accountmanager-plugin.spec?id=d5ca2fc49091fec96331f599a58e5587787b28f1
SRPM URL:http://fedorapeople.org/~arbiter/trac-accountmanager-plugin-0.2.1-1.20090406svn5484.fc11.src.rpm
Description:
The AccountManagerPlugin offers several features for managing user accounts:
    * allow users to register new accounts
    * login via an HTML form instead of using HTTP authentication
    * allow existing users to change their passwords or delete their accounts
Comment 1 Paul Howarth 2009-06-11 09:51:42 EDT
Thanks for packaging this up. I've used this package to set up my own trac instance using mod_fcgid and it all seems to be working fine.

I've documented the process here:
http://www.city-fan.org/tips/TracWithFastCGIonFedora

Anyway, I'll now return the favour by reviewing this package for you.
Comment 2 Paul Howarth 2009-06-11 11:09:33 EDT
Review Guidelines
=================
- rpmlint clean
- package name is OK and is consistent with existing trac plugin packages in Fedora
- spec file name consistent with package name
- package meets guidelines
- Beerware license is OK for Fedora
- correct "Copyright only" License field used in spec
- no detached license file to include
- spec file written in English and is legible
- source pulling instructions are missing "-r" option to set svn revision
- sources from SRPM match sources from upstream VCS
- package builds OK in mock for Fedora 10 (x86_64)
- buildreqs fine
- no locale data, libs, or devel files to worry about
- not relocatable
- directory ownership is fine
- no duplicate entries in %files list
- %defattr(...) and %clean present and correct
- macro usage is consistent
- code, not content
- no large docs (in fact hardly any docs at all!)
- docs don't affect runtime
- not a GUI app, no .desktop file needed
- %install section properly cleans buildroot
- filenames all valid UTF-8
- no scriptlets needed
- package appears to work as advertised
- no subpackages used
- no file dependencies

Notes
=====

Perhaps add comment that python-setuptools runtime dependency is
needed for pkg_resources import?

Please include contrib/sessionstore_convert.py as %doc
(see http://www.mailinglistarchive.com/trac-users@googlegroups.com/msg03696.html)

Please add -r $svnrel to svn export instructions
Comment 3 Paul Howarth 2009-07-08 16:17:19 EDT
These could do with being fixed too, though that's probably more of an upstream issue:

/usr/lib/python2.6/site-packages/acct_mgr/pwhash.py:13: DeprecationWarning: the md5 module is deprecated; use hashlib instead
  import md5, sha
/usr/lib/python2.6/site-packages/acct_mgr/pwhash.py:13: DeprecationWarning: the sha module is deprecated; use the hashlib module instead
  import md5, sha
Comment 4 Lorenzo Villani 2009-07-22 09:06:08 EDT
I stopped using trac and switched to another PMS, I have no more interest updating this package. You can take this package and get it reviewed if you want.

I'm changing to CLOSED WONTFIX, feel free to re-open it if you are interested.
Comment 5 Paul Howarth 2009-08-18 11:16:57 EDT
I've now made a new package submission in Bug #518046