-
Notifications
You must be signed in to change notification settings - Fork 57
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
Python set via an alias #52
Labels
Milestone
Comments
manodeep
added a commit
that referenced
this issue
Aug 3, 2016
…cks if the python specified in the common.mk file is the identical to currently running python and changes the common.mk file if necessary. Fixes #52. Also outputs the compile time options once each for the theory and mocks directories
manodeep
added a commit
that referenced
this issue
May 13, 2017
…cks if the python specified in the common.mk file is the identical to currently running python and changes the common.mk file if necessary. Fixes #52. Also outputs the compile time options once each for the theory and mocks directories
manodeep
added a commit
that referenced
this issue
May 13, 2017
…cks if the python specified in the common.mk file is the identical to currently running python and changes the common.mk file if necessary. Fixes #52. Also outputs the compile time options once each for the theory and mocks directories
manodeep
added a commit
that referenced
this issue
May 13, 2017
…cks if the python specified in the common.mk file is the identical to currently running python and changes the common.mk file if necessary. Fixes #52. Also outputs the compile time options once each for the theory and mocks directories
manodeep
added a commit
that referenced
this issue
May 13, 2017
…cks if the python specified in the common.mk file is the identical to currently running python and changes the common.mk file if necessary. Fixes #52. Also outputs the compile time options once each for the theory and mocks directories
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have encountered two cases where python is set via an alias. There is essentially no foolproof way of solving this alias issue -- since aliases are not available in non-interactive shells (as are the ones run by
make
with$(shell)
.PYTHON_VERSION_FULL := $(wordlist 2,4,$(subst ., ,$(shell python --version 2>&1)))
However, the user needs to be at least warned about this scenario, and a potential fix suggested. One way to solve it is to
alias | grep python
, and then checking if the RHS containspython
as the last word. In that case, the user has to replace all instances of python with this RHS.Or, I could just see if there is an alias and define a variable PYTHONCOMMAND appropriately and then use PYTHONCOMMAND exclusively in place of
python
.The text was updated successfully, but these errors were encountered: