[Bug 1243499] Review Request: python-configparser - Backport of python 3 configparser module to python 2.7 (and 2.6)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |zbyszek@xxxxxxxxx
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |zbyszek@xxxxxxxxx
              Flags|                            |fedora-review?



--- Comment #1 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
I was worried about issues with backwards compatiblity. But this package does
not override the module in Python 2.7 stdlib because the name is different
(configparser vs. ConfigParser). I think a note about this should be added to
%description, to avoid confusion.

You make the package only for Python 2.7, so any mention of other versions
should be removed from %description.

There is no license file.
Also, I think licensing might be wrong. CPython is licensed under PYTHON
SOFTWARE FOUNDATION LICENSE VERSION 2, and configparser is directly derived
from that, so should also be licensed the same. I think that licensing it as
MIT might be a mistake, unless configparser is indpendently derived from a
different source. Upstream maintainer of configparser in cpython prepared the
stand-alone configparser module, so it's possible that he is simply has
copyright to the code and decided to provide it under a different license.
Either way, please confirm the license, and ask upstream to include a license
file.

No issues with packaging otherwise.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]