Author alkuzad
Recipients alkuzad
Date 2020-05-11.08:50:15
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1589187015.71.0.593402074503.issue40592@roundup.psfhosted.org>
In-reply-to
Content
Shutil's which implementation does not work correctly when someone set's empty item in `PATHEXT` environment variable. Example:

set PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC;.RB;.RBW;

I'm not 100% sure how I got this in my PATHEXT config, I wasn't changing that so maybe some bugged uninstaller not removed it's extension correctly.

This makes things confusing as Windows will find correctly binary, but Python will return nothing, due to this part:

```
if any(cmd.lower().endswith(ext.lower()) for ext in pathext):
            files = [cmd]
```

pathext is initialized as `pathext = os.environ.get("PATHEXT", "").split(os.pathsep)`, which ends producing '' as last element

Because any string ends with empty string (''), files list will have plain version added like `git`, which will then fail executable check.

Workaround is to use full name `git.exe`

Filtering out empty strings would fix that.
History
Date User Action Args
2020-05-11 08:50:15alkuzadsetrecipients: + alkuzad
2020-05-11 08:50:15alkuzadsetmessageid: <1589187015.71.0.593402074503.issue40592@roundup.psfhosted.org>
2020-05-11 08:50:15alkuzadlinkissue40592 messages
2020-05-11 08:50:15alkuzadcreate