Bug 1419291
Summary: | Review Request: python-backports-unittest_mock - backport of unittest mock for python 3.2 and before | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Kevin Fenzi <kevin> |
Component: | Package Review | Assignee: | Neal Gompa <ngompa13> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | ngompa13, package-review |
Target Milestone: | --- | Flags: | ngompa13:
fedora-review+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-02-06 18:00:13 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Kevin Fenzi
2017-02-04 22:56:43 UTC
BTW, this package is needed for the python2 tests of python-setuptools recent versions. Taking this review. Review notes: + Package is appropriately named + Subpackages are named correctly per current Python packaging guidelines - %python_provide macro usage is missing on Python module subpackages + License noted correctly - License file is not included in the module subpackages + Unit tests are run and pass in the %check section Spec URL: http://www.scrye.com/~kevin/fedora/review/python-backports-unittest_mock/python-backports-unittest_mock.spec SRPM URL: http://www.scrye.com/~kevin/fedora/review/python-backports-unittest_mock/python-backports-unittest_mock-1.2.1-2.fc26.src.rpm Added the python_provide macro. Added a copy of the MIT license, as upstream doesn't include it on purpose. I'll argue with them about it later, but in the mean time I am including a copy. Looks good to me. APPROVED. Thanks for the quick review! Package requested. Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/python-backports-unittest_mock Built in rawhide. Thanks. |