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

Use executable from sys module instead of hardcoded python #189

Merged
merged 1 commit into from
Aug 26, 2017

Conversation

frenzymadness
Copy link
Contributor

Hello.

I met this issue when trying to run tests of doit module on a Python3-only system. Problem is that on a lot of systems python still means Python 2 and when you have only Python 3 installed, there is no python command at all.

I think that the best will be to execute commands inside test suite under the same Python as test suite itself.

This fix will also enable to run the test suite with various Python versions.

What do you think?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.0002%) to 99.73% when pulling 35efd05 on frenzymadness:python-executable-in-tests into 40c9043 on pydoit:master.

@schettino72
Copy link
Member

nice. I didnt know this sys.executable existed 😁

@schettino72 schettino72 merged commit 0c518d6 into pydoit:master Aug 26, 2017
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.

None yet

3 participants