Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1259333 - Review Request: python-mox3 - Mock object framework for Python
Summary: Review Request: python-mox3 - Mock object framework for Python
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Lukas Bezdicka
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1259286
Blocks: 1154650 RDO-LIBERTY-REVIEWS 1259400
TreeView+ depends on / blocked
 
Reported: 2015-09-02 12:14 UTC by Lukas Bezdicka
Modified: 2015-11-22 02:25 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-11-20 15:25:13 UTC
Type: ---
Embargoed:
lrintel: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Lukas Bezdicka 2015-09-02 12:14:38 UTC 
Spec URL: https://social.fedorapeople.org/python-mox3.spec
SRPM URL: https://social.fedorapeople.org/python-mox3-0.9.0-3.fc24.src.rpm
Description: Mock object framework for Python
Fedora Account System Username: social
Comment 1 Lubomir Rintel 2015-09-03 08:42:10 UTC 
* Named correctly
* Version tag correct, packaging latest version
* The source file is the same as distributed by upstream
* License tag correct
* License fine for Fedora
* SPEC file clean and legible

- Does not build in mock
The fedora-23-x86_64 build fails:
http://paste.fedoraproject.org/262924/12693631

- The package seems to depend on some particular versions of other packages (hence the build failure)
Please use versioned Requires/BuildRequires whenever possible

- The URL tag is incorrect
You may want to use https://pypi.python.org/pypi/mox3 instead
Comment 2 Haïkel Guémar 2015-09-03 08:52:00 UTC 
Just for the record, package fails to build without the newer python-subunit build as tests fail.
http://koji.fedoraproject.org/koji/buildinfo?buildID=682335
I have to investigate with subunit fails to build in F23, but it'll be fixed.


Package complies with new python guidelines.

somes fixes
* drop the following comment or expand it:
# https://bugs.launchpad.net/heat-cfntools/+bug/1403214/
* missing requires to six, fixtures and testtools
* about versioning, I would only require that we set min version on python-six 1.9.0, as it could break updates on EL7 (this package will be needed for OpenStack)
Comment 3 Lubomir Rintel 2015-09-03 08:54:42 UTC 
* Filelists sane
* RPMlint happy
* Provides sane
* Requires sane
Comment 4 Alan Pevec 2015-09-03 09:15:14 UTC 
Do not:
# Remove bundled egg-info
rm -rf %{pypi_name}.egg-info

It was misguided packaging guideline, corrected in the meantime:
https://fedorahosted.org/fpc/ticket/488

Egg _binaries_ need to be removed, not Egg _metadata_
For packages using PBR metadata cannot be completely generated from sdist tarball.
Comment 5 Lukas Bezdicka 2015-09-03 10:27:18 UTC 
Spec URL: https://social.fedorapeople.org/python-mox3.spec
SRPM URL: https://social.fedorapeople.org/python-mox3-0.9.0-4.fc22.src.rpm
Comment 6 Lubomir Rintel 2015-09-03 13:18:54 UTC 
Fine now.

APPROVED
Comment 7 Lukas Bezdicka 2015-09-03 13:31:58 UTC 
New Package SCM Request
=======================
Package Name: python-mox3
Short Description: Mock object framework for Python
Upstream URL: http://git.openstack.org/cgit/openstack/mox3
Owners: social apevec chandankumar
Branches:
InitialCC:
Comment 8 Lukas Bezdicka 2015-09-03 16:44:27 UTC 
New Package SCM Request
=======================
Package Name: python-mox3
Short Description: Mock object framework for Python
Upstream URL: http://git.openstack.org/cgit/openstack/mox3
Owners: social apevec chandankumar
Branches:
InitialCC:
Comment 9 Gwyn Ciesla 2015-09-03 18:25:10 UTC 
Git done (by process-git-requests).
Comment 10 Lukas Bezdicka 2015-09-07 07:54:13 UTC 
Package Change Request
======================
Package Name: python-mox3
New Branches: f22 f23
Owners: social apevec chandankumar
InitialCC:
Comment 11 Gwyn Ciesla 2015-09-28 12:37:40 UTC 
Git done (by process-git-requests).
Comment 12 Fedora Update System 2015-09-29 12:57:50 UTC 
python-mox3-0.10.0-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-2798ddf72b
Comment 13 Fedora Update System 2015-09-29 13:02:28 UTC 
python-mox3-0.10.0-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-1359a57f2c
Comment 14 Fedora Update System 2015-10-01 23:54:15 UTC 
python-mox3-0.10.0-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update python-mox3'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-1359a57f2c
Comment 15 Fedora Update System 2015-10-02 01:50:03 UTC 
python-mox3-0.10.0-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update python-mox3'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-1359a57f2c
Comment 16 Fedora Update System 2015-10-04 01:37:56 UTC 
python-mox3-0.10.0-1.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update python-mox3'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-2798ddf72b
Comment 17 Fedora Update System 2015-11-20 15:25:10 UTC 
python-mox3-0.10.0-1.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.
Comment 18 Fedora Update System 2015-11-22 02:25:14 UTC 
python-mox3-0.10.0-1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.
Note You need to log in before you can comment on or make changes to this bug.