Conversation
|
This check was created to support traditional python installer from https://www.python.org/downloads/ (assuming installation path is added to the path). The script is doing the right thing. (Python from the traditional installer does not have a suffix, it is just python.exe) |
|
That is false. searching for I just now downloaded python3.9.5 from https://www.python.org/downloads/ and installed it to c:\Python39 python3 is there, and waiting to be used in the script's search. |
|
Found more info. I think we are both errant, but can likely fix that. Python and python3 app exec aliases were added to Windows 10 1903 "...every install of Windows will include python and python3 commands that take you directly to the Python store page." https://devblogs.microsoft.com/python/python-in-the-windows-10-may-2019-update/ Above means that these Win 1903 app exec aliases can easily have path priority over any other install of python. And therefore encounter the bug my fix addresses. The buggy method will find the 0 byte exe alias and fail. https://docs.python.org/3/using/windows.html provides many install methods. So many, that its likely not worth investigating them all. Instead, to take an approach like cmake --> look on path and frequently used install locations, for known exe's.
python.org download added python3.exe sometime between 3.9.0 and 3.9.5 I recommend the script then
How about that approach? |

scripts/verify-windows.ps1fails to find installed python3 #1583Fixes #1583
Description of the changes:
Before submitting a Pull Request:
I tested changes on:
via projects automated test scripts. And used in my my project.