Bug 1010302 - Please introduce python3-iniparse package
Summary: Please introduce python3-iniparse package
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: python-iniparse
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Tim Lauridsen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-20 13:19 UTC by Bohuslav "Slavek" Kabrda
Modified: 2013-10-14 20:58 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-10-02 17:53:34 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Introduce python3 compat patch and python3- subpackage (20.12 KB, patch)
2013-09-20 13:19 UTC, Bohuslav "Slavek" Kabrda
no flags Details | Diff
Dist-git python3-compat commit (20.60 KB, patch)
2013-09-23 06:52 UTC, Bohuslav "Slavek" Kabrda
no flags Details | Diff
Fixed dist-git python3-compat commit (21.08 KB, patch)
2013-09-24 08:38 UTC, Bohuslav "Slavek" Kabrda
no flags Details | Diff

Description Bohuslav "Slavek" Kabrda 2013-09-20 13:19:59 UTC
Created attachment 800506 [details]
Introduce python3 compat patch and python3- subpackage

Hi,
I'm attaching a patch that includes introducing python3-iniparse subpackage, as well as a python 3 compatibility patch for the library itself. Would you please consider accepting it?
Thanks!

Comment 1 Tim Lauridsen 2013-09-21 17:16:58 UTC
Looks fine, but I have some problems adding the patch

fedpkg clone python-iniparse
cd python-iniparse
git apply /path/to/patch.txt

error: patch failed: python-iniparse.spec:1
error: python-iniparse.spec: patch does not apply

I am no git patch jedi master, so what do i do wrong

Comment 2 Tim Lauridsen 2013-09-22 08:19:19 UTC
commit your changes to a local copy of master and do a git format-patch -1 to get a patch to be applied with git apply.

Comment 3 Bohuslav "Slavek" Kabrda 2013-09-23 06:52:43 UTC
Created attachment 801497 [details]
Dist-git python3-compat commit

Ok, here's the commit, hope it works now.

Comment 4 Tim Lauridsen 2013-09-24 08:01:12 UTC
some things is broken in the patch

checkout upstream:
$ git svn clone https://iniparse.googlecode.com/svn/trunk/ iniparse
$ cd iniparse
download the patch from here:
$ https://code.google.com/p/iniparse/issues/detail?id=22#c13

Apply the patch
$ patch -p0 < iniparse-python-3-with-six.patch

Run Tests

$./runtests.py

$ ./runtests.py 
Traceback (most recent call last):
  File "./runtests.py", line 4, in <module>
    import tests
  File "/home/tim/udv/work/iniparse/tests/__init__.py", line 3, in <module>
    from . import test_ini
  File "/home/tim/udv/work/iniparse/tests/test_ini.py", line 4, in <module>
    from iniparse import ini
  File "/home/tim/udv/work/iniparse/iniparse/__init__.py", line 6, in <module>
    from .ini import INIConfig, change_comment_syntax
  File "/home/tim/udv/work/iniparse/iniparse/ini.py", line 45, in <module>
    from .configparser import DEFAULTSECT, ParsingError, MissingSectionHeaderError
ImportError: No module named configparser

Comment 5 Bohuslav "Slavek" Kabrda 2013-09-24 08:38:58 UTC
Created attachment 802097 [details]
Fixed dist-git python3-compat commit

<sigh> Sorry about that. I created that configparser as a new file and of course forgot to "svn add" it before running "svn diff", so it wasn't part of the patch. I fixed that, hope everything finally works.

Comment 6 Tim Lauridsen 2013-10-02 17:53:34 UTC
Look good, test patch against upstream checkout and now all tests are ok in both python2 & 3.

I have pushed it and build it for rawhide

http://koji.fedoraproject.org/koji/taskinfo?taskID=6016981

Comment 7 Honza Silhan 2013-10-11 12:30:22 UTC
Hi, we need this package in F20 for dnf. Is it possible?

Comment 8 Tim Lauridsen 2013-10-11 12:55:33 UTC
Yes, I will make a build a little later today :)

Comment 9 Tim Lauridsen 2013-10-11 16:15:09 UTC
Build for F20 and bodhi request created

https://admin.fedoraproject.org/updates/python-iniparse-0.4-9.fc20

Comment 10 Honza Silhan 2013-10-14 20:58:34 UTC
Thanks, Tim.


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