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

Make python version overridable by PYTHON variable #5

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

Conversation

pemensik
Copy link

Current version builds both python versions if available. Failures are
not fatal. Make it able to choose which version to compile, install only
single version. Or no python code installed, if PYTHON= variable is
passed.

Current version builds both python versions if available. Failures are
not fatal. Make it able to choose which version to compile, install only
single version. Or no python code installed, if PYTHON= variable is
passed.
@pemensik pemensik force-pushed the python branch 2 times, most recently from 7fca463 to e601d9e Compare February 28, 2021 20:39
Change logic to provide both python2 and python3 in PYTHON variable.
If set to empty, nothing is built. If set to specific, only that is
built. Simplify logic. Makes a bit uglier error reports when something
fails.
@joan2937
Copy link
Owner

joan2937 commented Mar 5, 2021

Just a note to say I'm busy doing other things at the moment but will get back to this within a week or so.

@ramok
Copy link

ramok commented Jul 6, 2022

I think is good idea to add variable PYTHON3 also

@pemensik
Copy link
Author

pemensik commented Jul 7, 2022

The idea of PYTHON variable is to use it like PYTHON=python3 or PYTHON=python2. I am not sure what should PYTHON3 variable do.

@ramok
Copy link

ramok commented Jul 7, 2022

If PYTHON3 unset - do not build for python3. if setted - build it, by $(PYTHON3) -q setup.py ....

Can be usefull for yocto build. For example for native build python execulable call nativepython and nativepythin3.

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.

3 participants