[Bug 513779] Review Request: python-simplesettings - Simple settings initialization for Python libraries

bugzilla at redhat.com bugzilla at redhat.com
Sun Jul 26 00:58:59 UTC 2009


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=513779


Mads Kiilerich <mads at kiilerich.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mads at kiilerich.com




--- Comment #1 from Mads Kiilerich <mads at kiilerich.com>  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.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list