Bug 248991 - Review Request: python-iniparse - A better INI parser for Python
Review Request: python-iniparse - A better INI parser for Python
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Ignacio Vazquez-Abrams
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-07-20 03:03 EDT by Tim Lauridsen
Modified: 2007-11-30 17:12 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-09-28 08:16:00 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
tla: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)
Patch for spec file (1.08 KB, patch)
2007-07-25 10:05 EDT, Ignacio Vazquez-Abrams
no flags Details | Diff

  None (edit)
Description Tim Lauridsen 2007-07-20 03:03:13 EDT
Spec URL: http://timlau.fedorapeople.org/iniparse.spec
SRPM URL: http://timlau.fedorapeople.org/iniparse-0.2-1.src.rpm
Description: 
iniparse is an INI parser for Python which is API compatible
with the standard library's ConfigParser, preserves structure of INI
files (order of sections & options, indentation, comments, and blank
lines are preserved when data is updated), and is more convenient to
use.
Comment 1 Ignacio Vazquez-Abrams 2007-07-25 08:21:45 EDT
iniparse-0.2-1.fc7.noarch.rpm:
W: iniparse spurious-executable-perm /usr/share/doc/iniparse-0.2/index.html

iniparse-0.2-1.fc7.src.rpm:
W: iniparse strange-permission iniparse.spec 0600

Also, the package should be named "python-iniparse".
Comment 2 Tim Lauridsen 2007-07-25 09:39:32 EDT
(In reply to comment #1)
> iniparse-0.2-1.fc7.noarch.rpm:
> W: iniparse spurious-executable-perm /usr/share/doc/iniparse-0.2/index.html
> 
> iniparse-0.2-1.fc7.src.rpm:
> W: iniparse strange-permission iniparse.spec 0600
> 
I will take a look at these warnings

> Also, the package should be named "python-iniparse".

Do you know any smart way to do this, without renaming the upstream source ???

Comment 3 Ignacio Vazquez-Abrams 2007-07-25 10:05:12 EDT
Created attachment 159925 [details]
Patch for spec file
Comment 4 Tim Lauridsen 2007-07-26 05:23:55 EDT
Thanks for the patch.

Here is a new spec & srpm.
http://timlau.fedorapeople.org/python-iniparse.spec
http://timlau.fedorapeople.org/python-iniparse-0.2-2.fc7.src.rpm

Changes:
Changed name to python-iniparse.
silent rpmlint on rpm & srpm.
Comment 5 Ignacio Vazquez-Abrams 2007-07-27 08:14:00 EDT
With the package name change the docs should be under
%{_docdir}/python-iniparse-%{version}, not %{_docdir}/iniparse-%{version}. Fix
that small problem and it's approved.
Comment 6 Tim Lauridsen 2007-07-27 08:59:22 EDT
Here is a new spec & srpm.
http://timlau.fedorapeople.org/python-iniparse.spec
http://timlau.fedorapeople.org/python-iniparse-0.2-3.fc7.src.rpm

doc relocated
Comment 7 Tim Lauridsen 2007-07-30 07:06:26 EDT
New Package CVS Request
=======================
Package Name: python-iniparse
Short Description:  A better INI parser for Python
Owners: tla@rasmil.dk
Branches: EL-5 FC-6 F-7
Comment 8 Tim Lauridsen 2007-07-30 07:07:26 EDT
Ups, wrong flag
Comment 9 Tim Lauridsen 2007-07-31 12:59:27 EDT
New Package CVS Request
=======================
Package Name: python-iniparse
Short Description:  A better INI parser for Python
Owners: tla@rasmil.dk
Branches: EL-5 FC-6 F-7

Comment 10 Kevin Fenzi 2007-07-31 14:21:41 EDT
cvs done.

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