[issue17283] Lib/test/__main__.py should share code with regrtest.py

2013-02-23 Thread Chris Jerdonek
New submission from Chris Jerdonek: As discussed here: http://bugs.python.org/issue15305#msg182853 this issue is for Lib/test/__main__.py to share code with Lib/test/regrtest.py to minimize duplication of code: http://hg.python.org/cpython/file/96f08a22f562/Lib/test/regrtest.py#l1594 http://h

[issue17283] Lib/test/__main__.py should share code with regrtest.py

2013-02-24 Thread Chris Jerdonek
Chris Jerdonek added the comment: Attaching patch. The use of "global TEMPDIR" isn't ideal. Alternatively, TEMPDIR's value when sysconfig.is_python_build() is true can be set when initially setting TEMPDIR: http://hg.python.org/cpython/file/96f08a22f562/Lib/test/regrtest.py#l203 -- k

[issue17283] Lib/test/__main__.py should share code with regrtest.py

2013-02-24 Thread Chris Jerdonek
Changes by Chris Jerdonek : -- stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue17283] Lib/test/__main__.py should share code with regrtest.py

2013-02-24 Thread Chris Jerdonek
Chris Jerdonek added the comment: Here is a new patch which does not use the global keyword. -- Added file: http://bugs.python.org/file29228/issue-17283-2.patch ___ Python tracker __

[issue17283] Lib/test/__main__.py should share code with regrtest.py

2013-02-26 Thread Petri Lehtinen
Petri Lehtinen added the comment: LGTM. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue17283] Lib/test/__main__.py should share code with regrtest.py

2013-02-27 Thread Roundup Robot
Roundup Robot added the comment: New changeset e0f3dcd30af8 by Chris Jerdonek in branch 'default': Issue #17283: Share code between __main__.py and regrtest.py in Lib/test. http://hg.python.org/cpython/rev/e0f3dcd30af8 -- nosy: +python-dev ___ Python

[issue17283] Lib/test/__main__.py should share code with regrtest.py

2013-02-27 Thread Chris Jerdonek
Chris Jerdonek added the comment: Thanks a lot for the review, Petri. -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker ___