Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 198284 - Review Request: python-configobj - Config file reading, writing, and validation
Summary: Review Request: python-configobj - Config file reading, writing, and validation
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Aurelien Bompard
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 189338
TreeView+ depends on / blocked
 
Reported: 2006-07-10 20:41 UTC by Luke Macken
Modified: 2016-09-20 02:37 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-19 15:08:44 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Luke Macken 2006-07-10 20:41:09 UTC
Spec URL: http://people.redhat.com/lmacken/python-configobj.spec
SRPM URL: http://people.redhat.com/lmacken/python-configobj-4.3.2-1.src.rpm
Description:
ConfigObj is a simple but powerful config file reader and writer: an ini file
round tripper. Its main feature is that it is very easy to use, with a
straightforward programmer's interface and a simple syntax for config files. 
It has lots of other features though:
    * Nested sections (subsections), to any level
    * List values
    * Multiple line values
    * String interpolation (substitution)
    * Integrated with a powerful validation system
          o including automatic type checking/conversion
          o repeated sections
          o and allowing default values
    * All comments in the file are preserved
    * The order of keys/sections is preserved
    * No external dependencies
    * Full Unicode support
    * A powerful unrepr mode for storing basic datatypes

Comment 1 Aurelien Bompard 2006-07-17 19:10:54 UTC
* typo in the url (it's dl.sf.net, not ds.sf.net)
* add --single-version-externally-managed to the %install step, see
http://fedoraproject.org/wiki/Packaging/Python for more info


Comment 2 Luke Macken 2006-07-18 03:58:57 UTC
(In reply to comment #1)
> * typo in the url (it's dl.sf.net, not ds.sf.net)

fixed.

> * add --single-version-externally-managed to the %install step, see
> http://fedoraproject.org/wiki/Packaging/Python for more info

Looks like the setup.py doesn't support the flag:
     error: option --single-version-externally-managed not recognized
     error: Bad exit status from /var/tmp/rpm-tmp.90142 (%install)

http://people.redhat.com/lmacken/python-configobj-4.3.2-2.src.rpm
http://people.redhat.com/lmacken/python-configobj.spec

Comment 3 Aurelien Bompard 2006-07-19 07:36:11 UTC
Review for release 2:
* RPM name is OK
* Source configobj-4.3.2.zip is the same as upstream
* Builds fine in mock
* rpmlint looks OK
* File list looks OK
* Works fine
* According to
http://peak.telecommunity.com/DevCenter/setuptools#install-command the
--single-version-externally-managed option is automatically switched on when the
--root option is used, so the guidelines are probably wrong here (or maybe it's
a new feature ?)

Anyway, APPROVED

Comment 4 Luke Macken 2006-07-19 15:08:44 UTC
Imported into CVS, added to owners.list, requested FC5 branch, tagged and built
for FC6.  Thanks!


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