Thank you for the report. The docs indeed say “If shell=True, the executable argument specifies which shell to use”.

Could you provide your change as a diff file following the guidelines on http://www.python.org/dev/patches/ ? Diffs are easier to review than whole files. A unit test is needed too, to make sure the bug is fixed, would you like to write a patch for test_subprocess.py?

Thanks again!