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

doc: clarify Python configuration, etc #1908

Closed
wants to merge 1 commit into from
Closed

doc: clarify Python configuration, etc #1908

wants to merge 1 commit into from

Conversation

sam-github
Copy link
Contributor

Clarify that:

  • node-gyp is not used to build Node.js
  • PYTHON can be used to add a Python search path
  • NODE_GYP_FORCE_PYTHON can be used to override all Python search paths
  • That a compatible Python is searched for
Checklist
  • npm install && npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

@sam-github
Copy link
Contributor Author

Note that this is not semver-major, what it documents is equally true for 5.x and 6.x, since they are both compatible with the same Python versions, and the docs don't say in which order it searches for Pythons.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Clarify that:
- node-gyp is not used to build Node.js
- PYTHON can be used to add a Python search path
- NODE_GYP_FORCE_PYTHON can be used to override all Python search paths
- That a compatible Python is searched for
rvagg pushed a commit that referenced this pull request Oct 4, 2019
Clarify that:
- node-gyp is not used to build Node.js
- PYTHON can be used to add a Python search path
- NODE_GYP_FORCE_PYTHON can be used to override all Python search paths
- That a compatible Python is searched for

PR-URL: #1908
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: João Reis <reis@janeasystems.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@rvagg
Copy link
Member

rvagg commented Oct 4, 2019

landed in b1bf32e, and will backport

@rvagg rvagg closed this Oct 4, 2019
rvagg pushed a commit that referenced this pull request Oct 4, 2019
Clarify that:
- node-gyp is not used to build Node.js
- PYTHON can be used to add a Python search path
- NODE_GYP_FORCE_PYTHON can be used to override all Python search paths
- That a compatible Python is searched for

PR-URL: #1908
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: João Reis <reis@janeasystems.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@cclauss cclauss added the Python label Oct 4, 2019
rvagg pushed a commit that referenced this pull request Oct 4, 2019
Clarify that:
- node-gyp is not used to build Node.js
- PYTHON can be used to add a Python search path
- NODE_GYP_FORCE_PYTHON can be used to override all Python search paths
- That a compatible Python is searched for

PR-URL: #1908
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: João Reis <reis@janeasystems.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants