Skip to content
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

Closed
manodeep opened this issue Apr 6, 2016 · 0 comments
Closed

Python set via an alias #52

manodeep opened this issue Apr 6, 2016 · 0 comments

Comments

@manodeep
Copy link
Owner

manodeep commented Apr 6, 2016

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 contains python 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.

@manodeep manodeep added this to the v1.2 milestone Apr 8, 2016
@manodeep manodeep modified the milestones: v1.2, v2.0 Apr 9, 2016
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
Projects
None yet
Development

No branches or pull requests

1 participant