https://bugzilla.redhat.com/show_bug.cgi?id=1243499
Zbigniew Jędrzejewski-Szmek <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |[email protected] Assignee|[email protected] |[email protected] Flags| |fedora-review? --- Comment #1 from Zbigniew Jędrzejewski-Szmek <[email protected]> --- 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 [email protected] https://admin.fedoraproject.org/mailman/listinfo/package-review
