Skip to content

Commit

Permalink
lib: drop "-2" flag for "py.exe" launcher
Browse files Browse the repository at this point in the history
Now that node-gyp supports both Python 2 and Python 3,
We don't need to explicitly try to find only Python 2.

Fixes: #2130
Refs: https://docs.python.org/3/using/windows.html#launcher
PR-URL: #2131
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
  • Loading branch information
DeeDeeG authored and rvagg committed May 18, 2020
1 parent 741ab09 commit c255ffb
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 10 deletions.
11 changes: 3 additions & 8 deletions lib/find-python.js
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ PythonFinder.prototype = {
checks.push({
before: () => {
this.addLog(
'checking if the py launcher can be used to find Python 2')
'checking if the py launcher can be used to find Python')
},
check: this.checkPyLauncher
})
Expand Down Expand Up @@ -188,15 +188,10 @@ PythonFinder.prototype = {
// Distributions of Python on Windows by default install with the "py.exe"
// Python launcher which is more likely to exist than the Python executable
// being in the $PATH.
// Because the Python launcher supports all versions of Python, we have to
// explicitly request a Python 2 version. This is done by supplying "-2" as
// the first command line argument. Since "py.exe -2" would be an invalid
// executable for "execFile", we have to use the launcher to figure out
// where the actual "python.exe" executable is located.
checkPyLauncher: function checkPyLauncher (errorCallback) {
this.log.verbose(
`- executing "${this.pyLauncher}" to get Python 2 executable path`)
this.run(this.pyLauncher, ['-2', ...this.argsExecutable], false,
`- executing "${this.pyLauncher}" to get Python executable path`)
this.run(this.pyLauncher, this.argsExecutable, false,
function (err, execPath) {
// Possible outcomes: same as checkCommand
if (err) {
Expand Down
3 changes: 1 addition & 2 deletions test/test-find-python.js
Original file line number Diff line number Diff line change
Expand Up @@ -146,14 +146,13 @@ test('find python - no python2, no python, unix', function (t) {
})

test('find python - no python, use python launcher', function (t) {
t.plan(4)
t.plan(3)

var f = new TestPythonFinder(null, done)
f.win = true

f.execFile = function (program, args, opts, cb) {
if (program === 'py.exe') {
t.notEqual(args.indexOf('-2'), -1)
t.notEqual(args.indexOf('-c'), -1)
return cb(null, 'Z:\\snake.exe')
}
Expand Down

0 comments on commit c255ffb

Please sign in to comment.