[issue24033] Update _test_multiprocessing.py to use script helpers

[STINNER Victor]

STINNER Victor  added the comment:

Sorry but these patches have been lost in the huge database of open bugs and we 
forgot it. They have been written 3 years ago. Since that, Python moved to 
GitHub. If you are still interested to convert tests to script_helper, please 
open GitHub pull requests. I would suggest to start with a small one, and then 
see if other tests should be converted. I close the issue. Reopen it if you 
want to work again on it.

--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Ezio Melotti]

Ezio Melotti added the comment:

Thanks for the patch.  I left a review on Rietveld.

--
nosy: +ezio.melotti, haypo, serhiy.storchaka
versions: +Python 3.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Christie]

Christie added the comment:

If #18576 gets merged before this does, I have some updated docs for the new 
helper functions.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Christie]

Christie added the comment:

Now using existing function spawn_python instead of creating a new function.

--
Added file: http://bugs.python.org/file39710/issue24033_multi.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Christie]

Christie added the comment:

run_python and spawn_python are basically the same, I need to reconcile those.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Davin Potts]

Changes by Davin Potts :


--
stage:  -> patch review
type:  -> enhancement

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Christie]

Christie added the comment:

Updated both tests in _test_mulitprocessing.py which were using sys.exectuable 
and Popen to use script_helpers instead, refactored script_helpers a bit to 
make this work.

--
keywords: +patch
Added file: 
http://bugs.python.org/file39347/issue24033_script_helper_multi_both.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Davin Potts]

Changes by Davin Potts :


--
nosy: +davin

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Christie]

Christie added the comment:

I am working on this!

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24033] Update _test_multiprocessing.py to use script helpers

[Christie]

New submission from Christie:

As described in Issue9517, many test modules do not make use of the helpers in 
script_helpers.py to invoke the python interpreter in a subprocess. Issue9517 
will be broken down into several smaller issues so we can address smaller 
change sets.

This issue is to update _test_multiprocessing.py to use script_helpers.py.

--
components: Tests
messages: 241837
nosy: bobcatfish
priority: normal
severity: normal
status: open
title: Update _test_multiprocessing.py to use script helpers
versions: Python 3.5

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com