-
Notifications
You must be signed in to change notification settings - Fork 584
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
Incorect Python version on MacOS latest #179
Comments
Also adding on to the issue here, all dependencies in our tests were installed in |
Hello,
I have done a few experiments: runs-on: macos-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v2
with:
python-version: 3.7
- run: |
echo $PATH
python --version
which python
- run: |
echo $PATH
python --version
which python
shell: bash {0}
- run: |
echo $PATH
python --version
which python
shell: bash -l {0} The first bash step output (specified Python is the first in PATH):
The second bash step output (specified Python is the first in PATH):
The third bash step output (bash is run with
Unfortunately, there is no way to fix it from task side since it is by design.
|
Thanks for the explanations @maxim-lobanov ! |
…t be used with actions/setup-python; refer to actions/setup-python#179 (comment) for details
For windows compilation Need to change not by ! or by || and by && fMapOfDigiInVolume need to contain pointer to GateDigiAdderInVolume instead of element The templated function with specialization (.icc) need to be in the cpp but the instanciation of the class only works with windows https://learn.microsoft.com/bs-latn-ba/cpp/cpp/source-code-organization-cpp-templates?view=msvc-160 For windows on github actions, move dist repository Add tests for windows Need to add PLATFORM variable because in windows it's just amd not x68 - intel Use cibuildwheel for Windows CI Repair wheel for windows with delvewheel Own delvewheel to take into account case sensitivity Do not use QT because delvewheel do not take into account very well Example to try ssh with github actions Be sure to have a correct seed for windows if windows, the long integer type are 4 bytes instead of 8 bytes for python and unix system https://fr.wikipedia.org/wiki/Entier_long Remove MT for windows And remove fork/spawn: fork is not usable on windows and spawn produces an error # to be able to run process, we will need to start the example in __main__ # https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Encode to be able to print the dump_tree_of_volumes https://stackoverflow.com/questions/27092833/unicodeencodeerror-charmap-codec-cant-encode-characters Remove -l option to bash actions/setup-python#179
For windows compilation Need to change not by ! or by || and by && fMapOfDigiInVolume need to contain pointer to GateDigiAdderInVolume instead of element The templated function with specialization (.icc) need to be in the cpp but the instanciation of the class only works with windows https://learn.microsoft.com/bs-latn-ba/cpp/cpp/source-code-organization-cpp-templates?view=msvc-160 For windows on github actions, move dist repository Add tests for windows Need to add PLATFORM variable because in windows it's just amd not x68 - intel Use cibuildwheel for Windows CI Repair wheel for windows with delvewheel Own delvewheel to take into account case sensitivity Do not use QT because delvewheel do not take into account very well Example to try ssh with github actions Be sure to have a correct seed for windows if windows, the long integer type are 4 bytes instead of 8 bytes for python and unix system https://fr.wikipedia.org/wiki/Entier_long Remove MT for windows And remove fork/spawn: fork is not usable on windows and spawn produces an error # to be able to run process, we will need to start the example in __main__ # https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Encode to be able to print the dump_tree_of_volumes https://stackoverflow.com/questions/27092833/unicodeencodeerror-charmap-codec-cant-encode-characters Remove -l option to bash actions/setup-python#179
For windows compilation Need to change not by ! or by || and by && fMapOfDigiInVolume need to contain pointer to GateDigiAdderInVolume instead of element The templated function with specialization (.icc) need to be in the cpp but the instanciation of the class only works with windows https://learn.microsoft.com/bs-latn-ba/cpp/cpp/source-code-organization-cpp-templates?view=msvc-160 For windows on github actions, move dist repository Add tests for windows Need to add PLATFORM variable because in windows it's just amd not x68 - intel Use cibuildwheel for Windows CI Repair wheel for windows with delvewheel Own delvewheel to take into account case sensitivity Do not use QT because delvewheel do not take into account very well Example to try ssh with github actions Be sure to have a correct seed for windows if windows, the long integer type are 4 bytes instead of 8 bytes for python and unix system https://fr.wikipedia.org/wiki/Entier_long Remove MT for windows And remove fork/spawn: fork is not usable on windows and spawn produces an error # to be able to run process, we will need to start the example in __main__ # https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Encode to be able to print the dump_tree_of_volumes https://stackoverflow.com/questions/27092833/unicodeencodeerror-charmap-codec-cant-encode-characters Remove -l option to bash actions/setup-python#179
For windows compilation Need to change not by ! or by || and by && fMapOfDigiInVolume need to contain pointer to GateDigiAdderInVolume instead of element The templated function with specialization (.icc) need to be in the cpp but the instanciation of the class only works with windows https://learn.microsoft.com/bs-latn-ba/cpp/cpp/source-code-organization-cpp-templates?view=msvc-160 For windows on github actions, move dist repository Add tests for windows Need to add PLATFORM variable because in windows it's just amd not x68 - intel Use cibuildwheel for Windows CI Repair wheel for windows with delvewheel Own delvewheel to take into account case sensitivity Do not use QT because delvewheel do not take into account very well Example to try ssh with github actions Be sure to have a correct seed for windows if windows, the long integer type are 4 bytes instead of 8 bytes for python and unix system https://fr.wikipedia.org/wiki/Entier_long Remove MT for windows And remove fork/spawn: fork is not usable on windows and spawn produces an error # to be able to run process, we will need to start the example in __main__ # https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Encode to be able to print the dump_tree_of_volumes https://stackoverflow.com/questions/27092833/unicodeencodeerror-charmap-codec-cant-encode-characters Remove -l option to bash actions/setup-python#179
For windows compilation Need to change not by ! or by || and by && fMapOfDigiInVolume need to contain pointer to GateDigiAdderInVolume instead of element The templated function with specialization (.icc) need to be in the cpp but the instanciation of the class only works with windows https://learn.microsoft.com/bs-latn-ba/cpp/cpp/source-code-organization-cpp-templates?view=msvc-160 For windows on github actions, move dist repository Add tests for windows Need to add PLATFORM variable because in windows it's just amd not x68 - intel Use cibuildwheel for Windows CI Repair wheel for windows with delvewheel Own delvewheel to take into account case sensitivity Do not use QT because delvewheel do not take into account very well Example to try ssh with github actions Be sure to have a correct seed for windows if windows, the long integer type are 4 bytes instead of 8 bytes for python and unix system https://fr.wikipedia.org/wiki/Entier_long Remove MT for windows And remove fork/spawn: fork is not usable on windows and spawn produces an error # to be able to run process, we will need to start the example in __main__ # https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Encode to be able to print the dump_tree_of_volumes https://stackoverflow.com/questions/27092833/unicodeencodeerror-charmap-codec-cant-encode-characters Remove -l option to bash actions/setup-python#179
For windows compilation Need to change not by ! or by || and by && fMapOfDigiInVolume need to contain pointer to GateDigiAdderInVolume instead of element The templated function with specialization (.icc) need to be in the cpp but the instanciation of the class only works with windows https://learn.microsoft.com/bs-latn-ba/cpp/cpp/source-code-organization-cpp-templates?view=msvc-160 For windows on github actions, move dist repository Add tests for windows Need to add PLATFORM variable because in windows it's just amd not x68 - intel Use cibuildwheel for Windows CI Repair wheel for windows with delvewheel Own delvewheel to take into account case sensitivity Do not use QT because delvewheel do not take into account very well Example to try ssh with github actions Be sure to have a correct seed for windows if windows, the long integer type are 4 bytes instead of 8 bytes for python and unix system https://fr.wikipedia.org/wiki/Entier_long Remove MT for windows And remove fork/spawn: fork is not usable on windows and spawn produces an error # to be able to run process, we will need to start the example in __main__ # https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Encode to be able to print the dump_tree_of_volumes https://stackoverflow.com/questions/27092833/unicodeencodeerror-charmap-codec-cant-encode-characters Remove -l option to bash actions/setup-python#179
For windows compilation Need to change not by ! or by || and by && fMapOfDigiInVolume need to contain pointer to GateDigiAdderInVolume instead of element The templated function with specialization (.icc) need to be in the cpp but the instanciation of the class only works with windows https://learn.microsoft.com/bs-latn-ba/cpp/cpp/source-code-organization-cpp-templates?view=msvc-160 For windows on github actions, move dist repository Add tests for windows Need to add PLATFORM variable because in windows it's just amd not x68 - intel Use cibuildwheel for Windows CI Repair wheel for windows with delvewheel Own delvewheel to take into account case sensitivity Do not use QT because delvewheel do not take into account very well Example to try ssh with github actions Be sure to have a correct seed for windows if windows, the long integer type are 4 bytes instead of 8 bytes for python and unix system https://fr.wikipedia.org/wiki/Entier_long Remove MT for windows And remove fork/spawn: fork is not usable on windows and spawn produces an error # to be able to run process, we will need to start the example in __main__ # https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Encode to be able to print the dump_tree_of_volumes https://stackoverflow.com/questions/27092833/unicodeencodeerror-charmap-codec-cant-encode-characters Remove -l option to bash actions/setup-python#179
For windows compilation Need to change not by ! or by || and by && fMapOfDigiInVolume need to contain pointer to GateDigiAdderInVolume instead of element The templated function with specialization (.icc) need to be in the cpp but the instanciation of the class only works with windows https://learn.microsoft.com/bs-latn-ba/cpp/cpp/source-code-organization-cpp-templates?view=msvc-160 For windows on github actions, move dist repository Add tests for windows Need to add PLATFORM variable because in windows it's just amd not x68 - intel Use cibuildwheel for Windows CI Repair wheel for windows with delvewheel Own delvewheel to take into account case sensitivity Do not use QT because delvewheel do not take into account very well Example to try ssh with github actions Be sure to have a correct seed for windows if windows, the long integer type are 4 bytes instead of 8 bytes for python and unix system https://fr.wikipedia.org/wiki/Entier_long Remove MT for windows And remove fork/spawn: fork is not usable on windows and spawn produces an error # to be able to run process, we will need to start the example in __main__ # https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Encode to be able to print the dump_tree_of_volumes https://stackoverflow.com/questions/27092833/unicodeencodeerror-charmap-codec-cant-encode-characters Remove -l option to bash actions/setup-python#179
Seems there is a need to remove the -l argument otherwise the macos runner will always use Python 2.7.18. See this thread for further details: actions/setup-python#179 (comment)
Describe the bug
On MacOS-latest, the version of Python is not the expected one.
The following input:
yields the following output:
Ubuntu
Windows
MacOS
Which version of the action are you using?
v1
v2
v2.0.1
ormaster
)Environment
Am I missing something here?
Thanks!
The text was updated successfully, but these errors were encountered: