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

Add variable for python interpreter. #31

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zmward99
Copy link

I was trying to use this package with a new system that doesn't have python2.7, and figured it would be easier to set the interpreter with this variable.

@benma
Copy link
Owner

benma commented Jan 29, 2023

This package is supposed to work with all Python versions, including python2 and python3. What was the issue that you ran into exactly?

@zmward99
Copy link
Author

The python interpreter on my machine is called python3, and rather than changing the long string variable (vr--command-python-default) to call the correct python version, I created a variable that can be set to either python or python3. It's prepended onto the vr--command-python-default variable.

@benma
Copy link
Owner

benma commented Jan 31, 2023

Thanks. In this case you should add another defcustom var which can be customized using customize-variable - the vr-- vars are considered internal.

Not sure what happens when you use one defcustom var inside another (vr/command-python in this case) - ideally the default for it would then be using the variable as customized by the user. I think this is what would happen but I am not quite sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants