-
Notifications
You must be signed in to change notification settings - Fork 244
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add python3.exe symlink on windows builds. #917
Conversation
Hmm, our Nuget Python should(?) be first in the path, and it looks like it does include python3. Is the bug you are seeing true on all Python versions? Also, maybe you could print out the path? |
I can reproduce the error very well with different cpython versions on different windows images on Github Actions. Here are some infos for PATH
where python
where python3
which python
which python3
As far I can tell, As far as Azure builds go (here 2021-11-13T13:57:57.1464359Z ##[group]Installing Python cp36...
2021-11-13T13:57:57.1465470Z + C:\cibw\nuget.exe install pythonx86 -Version 3.6.8 -FallbackSource https://api.nuget.org/v3/index.json -OutputDirectory C:\cibw\python
2021-11-13T13:57:57.1469502Z Package "pythonx86.3.6.8" is already installed.
Debug output:
2021-11-13T13:57:57.1470172Z C:\cibw\python\pythonx86.3.6.8\tools\python3.exe already exists:
2021-11-13T13:57:57.1470983Z ['python.exe', 'python3.dll', 'python3.exe', 'python36.dll', 'pythonw.exe', 'vcruntime140.dll'] The pythonx86.3.6.8 package is already installed, I currently don't know where the package is installed from and how python3.exe is created, maybe the Use Python Version task also uses nuget and creates a symlink? Edit: What is also interesting: The Test cibuildwheel on windows-latest here https://github.com/pypa/cibuildwheel/runs/4198728005?check_suite_focus=true is completed successfully. |
Oops, sorry, one of us has to push a button to get the GHA builds to go! Sorry! I know GitHub Actions makes a python3 symlink (I requested it, actually), but why is that one overriding the local symlink? The path should be setup correctly so that we override any external environment. Oh, wait a minute, I might know what's happening. If Azure is using nuget to set up Python instead of the Action, I bet they added the python3 symlink manually there too when I asked for it in Actions. Therefore NuGet doesn't come with it by default (though IMO it should), so that's why we need to add it. Okay, conditional adding is fine with me, as long as we know why it's needed! |
Great to see this resolved! On a side note: GHA builds will be triggered without your consent by |
Co-authored-by: Henry Schreiner <HenrySchreinerIII@gmail.com>
This PR adds a
python3.exe
link pointing topython.exe
on windows, closes #915.