[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Steve Dower added the comment: Thanks for the contribution! -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Steve Dower added the comment: New changeset 5fc784e28ab9cd02c30395d1973ff825b787efa2 by Miss Islington (bot) in branch '3.9': bpo-44572: On Windows, disconnect STDIN in platform._syscmd_ver() to prevent erroneous STDIN consumption (GH-27092)

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Pablo Galindo Salgado added the comment: New changeset 425756abdc03263ab3a52b068befd1ddb16c2dd2 by Miss Islington (bot) in branch '3.10': bpo-44572: On Windows, disconnect STDIN in platform._syscmd_ver() to prevent erroneous STDIN consumption (GH-27092) (GH-27124)

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Change by miss-islington : -- pull_requests: +25668 pull_request: https://github.com/python/cpython/pull/27125 ___ Python tracker ___

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Change by miss-islington : -- nosy: +miss-islington nosy_count: 5.0 -> 6.0 pull_requests: +25667 pull_request: https://github.com/python/cpython/pull/27124 ___ Python tracker

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Steve Dower added the comment: New changeset 0ee0a740e12ec8568aafa033aa6bb08b265afe26 by Konstantin-Glukhov in branch 'main': bpo-44572: On Windows, disconnect STDIN in platform._syscmd_ver() to prevent erroneous STDIN consumption (GH-27092)

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Change by Константин Глухов : -- keywords: +patch pull_requests: +25640 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/27092 ___ Python tracker

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Zachary Ware added the comment: You can make a PR entirely through the Github web interface, though we do generally encourage running the tests locally, especially since it would be good to add a new test for this case. -- ___ Python tracker

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Константин Глухов added the comment: I do not understand why a one line change in the code requires full blown git clone. Shouldn't you guys have a system that allows to change code over the web interface and your bots will do the rest: checks, tests, regressions, etc.? --

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Zachary Ware added the comment: The pull request you opened was invalid, because it was just trying to merge the `main` branch of the python/cpython repository into the `3.9` branch. What we need instead is a pull request between a branch from `main` in your own fork of python/cpython with

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Константин Глухов added the comment: Well, I opened pull request, but it was closed. You guys have a very good team work going on out there :) Good luck! -- ___ Python tracker

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Change by Константин Глухов : -- keywords: +patch pull_requests: +25606 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/27050 ___ Python tracker

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Константин Глухов added the comment: I will try -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Zachary Ware added the comment: Would you like to create a pull request to that effect? -- components: -IO stage: -> needs patch versions: +Python 3.10, Python 3.11 ___ Python tracker

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

Константин Глухов added the comment: The obvious fix is to add "stdin=subprocess.DEVNULL" argument to check_output() call. -- ___ Python tracker ___

[issue44572] Calls to platform._syscmd_ver() dependent functions consume STDIN

New submission from Константин Глухов : Starting with version 3.9.5 platform.win32* functions have been re-written and consume STDIN. The bug comes down to running 'ver', 'command /c ver', 'cmd /c ver' in platform._syscmd_ver() via subprocess.check_output(). The following code demonstrate the