Popen.wait does not account for negative return codes

[MrTroble]

BPO	41386
Nosy	@pfmoore, @tjguk, @zware, @eryksun, @zooba, @MrTroble
PRs	bpo-41386 Fixed Popen.wait does not account for negative return codes #21607

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2020-07-25.08:58:20.743>
created_at = <Date 2020-07-24.15:54:41.937>
labels = ['invalid', 'type-bug', 'library']
title = 'Popen.wait does not account for negative return codes'
updated_at = <Date 2020-07-27.14:40:12.026>
user = 'https://github.com/MrTroble'

bugs.python.org fields:

activity = <Date 2020-07-27.14:40:12.026>
actor = 'eryksun'
assignee = 'none'
closed = True
closed_date = <Date 2020-07-25.08:58:20.743>
closer = 'MrTroble'
components = ['Library (Lib)']
creation = <Date 2020-07-24.15:54:41.937>
creator = 'MrTroble'
dependencies = []
files = []
hgrepos = []
issue_num = 41386
keywords = ['patch']
message_count = 7.0
messages = ['374194', '374195', '374209', '374212', '374219', '374248', '374383']
nosy_count = 6.0
nosy_names = ['paul.moore', 'tim.golden', 'zach.ware', 'eryksun', 'steve.dower', 'MrTroble']
pr_nums = ['21607']
priority = 'normal'
resolution = 'not a bug'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue41386'
versions = []

[MrTroble]

Following problem occurred.

A C style program can have negative return codes. However this is not correctly implemented in the Windows API. Therefore it is being returned as unsigned long by the API hence it leads to ambiguity while comparing return codes.

For reference regarding this topic see:
https://docs.microsoft.com/en-us/cpp/cpp/main-function-command-line-args?redirectedfrom=MSDN&view=vs-2019
https://docs.microsoft.com/en-us/windows/win32/api/processthreadsapi/nf-processthreadsapi-getexitcodeprocess

I suggest a

[MrTroble]

I suggest implementing a singed conversation into _wait

[MrTroble]

I don't know if it should be a good idea to add a flag to turn off the negative conversation?