classification
Title: Coverage support for CPython 2
Type: enhancement Stage: resolved
Components: Build Versions: Python 2.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: abarry, alecsandru.patrascu, python-dev, zach.ware
Priority: normal Keywords: patch

Created on 2015-12-22 14:07 by alecsandru.patrascu, last changed 2016-01-20 06:13 by zach.ware. This issue is now closed.

Files
File name Uploaded Description Edit
coverage-2.7-v01.patch alecsandru.patrascu, 2015-12-22 14:07 review
coverage-2.7-v02.patch alecsandru.patrascu, 2015-12-22 15:15 review
Messages (5)
msg256839 - (view) Author: Alecsandru Patrascu (alecsandru.patrascu) * Date: 2015-12-22 14:07
Hi All,

This is Alecsandru from Server Scripting Languages Optimization team at Intel Corporation. I would like to submit a patch that adds extended support for coverage, in the same way that it is used in CPython3. Used in conjunction with the regrtests, it provides a broader view of the hot and cold zones in the interpreter.

Thank you,
Alecsandru
msg256840 - (view) Author: Zachary Ware (zach.ware) * (Python committer) Date: 2015-12-22 15:02
Left a couple of review comments on Rietveld.
msg256841 - (view) Author: Alecsandru Patrascu (alecsandru.patrascu) * Date: 2015-12-22 15:15
Thank you for the reviews. I've added a new patch (v02) to correct the issues.
msg258644 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2016-01-20 06:12
New changeset ed7b602a7400 by Zachary Ware in branch '2.7':
Issue #25925: Backport C coverage reporting Makefile targets
https://hg.python.org/cpython/rev/ed7b602a7400
msg258645 - (view) Author: Zachary Ware (zach.ware) * (Python committer) Date: 2016-01-20 06:13
Committed (with a couple minor tweaks), thanks for the patch!
History
Date User Action Args
2016-01-20 06:13:25zach.waresetstatus: open -> closed
resolution: fixed
messages: + msg258645

stage: patch review -> resolved
2016-01-20 06:12:35python-devsetnosy: + python-dev
messages: + msg258644
2015-12-22 15:15:04alecsandru.patrascusetfiles: + coverage-2.7-v02.patch

messages: + msg256841
2015-12-22 15:02:47zach.waresetnosy: + zach.ware

messages: + msg256840
stage: patch review
2015-12-22 14:10:03abarrysetnosy: + abarry
2015-12-22 14:08:00alecsandru.patrascucreate