-
Notifications
You must be signed in to change notification settings - Fork 69
Python interpreter can now be overridden #87
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
|
I signed it! (alex@128technology.com should now be a member of opensource@128technology.com) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@apt-itude Unfortunately I no longer work for Google, and don't have commit access to this repo any longer. Change looks good to me. A test case would be nice, though.
@apt-itude this adds a basic test: tmc@4db9b96 |
tmc@ expressed an interest in forking this PR. Regarding the CLA situation: I believe the CLA bot hasn't updated this PR because no new commit has been pushed, but I can verify that alex@128technology.com is covered by the 128 Technology CLA. |
Hey guys, sorry I haven't been on top of this. I'll push a new commit with that test case. |
CLAs look good, thanks! |
Thanks for the PR! |
In case the Python runtime used to build a par is not exactly the same path as the target interpreter, this allows the interpreter to be explicitly set instead of inferred from the stub file.
Example: