Bug 513779 - Review Request: python-simplesettings - Simple settings initialization for Python libraries
Review Request: python-simplesettings - Simple settings initialization for Py...
Status: CLOSED WONTFIX
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
:
Depends On:
Blocks: FE-DEADREVIEW
  Show dependency treegraph
 
Reported: 2009-07-25 14:16 EDT by Fabian Affolter
Modified: 2010-01-04 05:12 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2009-11-14 09:55:18 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Fabian Affolter 2009-07-25 14:16:42 EDT
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/python-simplesettings.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/python-simplesettings-0.5-1.fc11.src.rpm

Project URL: http://code.google.com/p/simplesettings/

Description:
Simple settings initialization for third party apps and libraries
in Python.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1514001

rpmlint output:
[fab@laptop09 SRPMS]$ rpmlint python-simplesettings-0.5-1.fc11.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

[fab@laptop09 noarch]$ rpmlint python-simplesettings-0.5-1.fc11.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
Comment 1 Mads Kiilerich 2009-07-25 20:58:58 EDT
Neither summary of description or upstream web site or any documentation or comments in the source could help me understand what the module in this package does.

Reading the code reveals that it is about returning the dict of one python module optionally overruled by another. It seems like the intention is that it can be used for simple configuration systems where the defaults can be overridden by local settings made in a py file.

The library is very small - only 26 lines of code but exposes a complex and very specific API. For most practically purposes I would say that the same functionality can be achieved in simpler and better ways.

I don't see why anybody should use this library. If they do then they would probably include a copy in their source and extend and customize it to their needs.

I don't think having this as a Fedora package is a good idea.
Comment 2 Jason Tibbitts 2009-09-23 14:04:00 EDT
Fabian, I don't necessarily agree with what Mads has written above, but I (and I expect many of the other reviewers) would like to see some response to it, if for nothing else than to know that you still wish to submit this.
Comment 3 Jason Tibbitts 2009-11-07 23:20:42 EST
I guess I'll close this soon if there's no response.
Comment 4 Fabian Affolter 2009-11-14 09:55:18 EST
At the moment I'm a bit short on time because of that I don't want to go on with this request for now.
Comment 5 Fabian Affolter 2010-01-04 05:12:53 EST
The source was deleted at Google Code.

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