-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22273/
-----------------------------------------------------------

(Updated June 11, 2014, 2:20 p.m.)


Review request for qpid, Andrew Stitcher and Rafael Schloming.


Changes
-------

Add AStitcher as reviewer


Bugs: PROTON-595
    https://issues.apache.org/jira/browse/PROTON-595


Repository: qpid


Description
-------

Popen(file.py, ...) needs to run with shell=True on windows. On Linux the tests 
hang when shell=True. This patch compromises by turning on the shell arg when 
running on windows.

Change the arg list to run Popen(python, file.py, ...) touches wider ranges of 
source code with some risk and with no obvious benefit.


Diffs
-----

  proton/trunk/tests/python/proton_tests/common.py 1601492 

Diff: https://reviews.apache.org/r/22273/diff/


Testing
-------

Passes tests on linux and windows.


Thanks,

Chug Rolke

Reply via email to