[issue43776] Popen with shell=True yield mangled repr output

Change by Gregory P. Smith : -- resolution: -> fixed stage: patch review -> commit review status: open -> closed ___ Python tracker ___

[issue43776] Popen with shell=True yield mangled repr output

Gregory P. Smith added the comment: New changeset 5a8ddcc4524dca3880d7fc2818814ffae1cfb8a2 by Gregory P. Smith in branch '3.9': [3.9] bpo-43776: Remove list call from args in Popen repr (GH-25338) (GH-26510) https://github.com/python/cpython/commit/5a8ddcc4524dca3880d7fc2818814ffae1cfb8a2

[issue43776] Popen with shell=True yield mangled repr output

Change by Gregory P. Smith : -- pull_requests: +25106 pull_request: https://github.com/python/cpython/pull/26510 ___ Python tracker ___

[issue43776] Popen with shell=True yield mangled repr output

miss-islington added the comment: New changeset db0c5b786df961785ae8c803f5572ae0c8dadcc7 by M. Kocher in branch 'master': bpo-43776: Remove list call from args in Popen repr (GH-25338) https://github.com/python/cpython/commit/db0c5b786df961785ae8c803f5572ae0c8dadcc7 -- nosy:

[issue43776] Popen with shell=True yield mangled repr output

Change by Gregory P. Smith : -- keywords: +3.9regression ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue43776] Popen with shell=True yield mangled repr output

mkocher added the comment: There also appears to be an issue when args are provided as a `pathlib.Path` instance. Python 3.9.2 | packaged by conda-forge | (default, Feb 21 2021, 05:02:20) Type 'copyright', 'credits' or 'license' for more information IPython 7.22.0 -- An enhanced

[issue43776] Popen with shell=True yield mangled repr output

mkocher added the comment: There's a PR up with removed `list` call as well as improved test coverage for the case where args is passed into Popen as a string. Hopefully this can get merged before things starting getting crazy with the 3.10 release crunch. --

[issue43776] Popen with shell=True yield mangled repr output

Change by Raymond Hettinger : -- nosy: +rhettinger ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue43776] Popen with shell=True yield mangled repr output

Change by Gregory P. Smith : -- assignee: -> gregory.p.smith nosy: +gregory.p.smith versions: +Python 3.10 ___ Python tracker ___

[issue43776] Popen with shell=True yield mangled repr output

Change by mkocher : -- keywords: +patch pull_requests: +24073 stage: -> patch review pull_request: https://github.com/python/cpython/pull/25338 ___ Python tracker ___

[issue43776] Popen with shell=True yield mangled repr output

William Pickard added the comment: Actually, the problem is independent of the value of "shell", the __repr__ function from the initial PR that introduced it expects "args" to be a sequence and converts it to a list. -- nosy: +WildCard65 ___

[issue43776] Popen with shell=True yield mangled repr output

New submission from mkocher : When using Popen with shell=True, the output of the repr is not particularly user friendly. When using the form `p = Popen('python --version'.split())`, the output is reasonably output in a user friendly form of ``. However, when running with `shell=True`,