electron/node-gyp
1
0
Fork 0
mirror of https://github.com/electron/node-gyp.git synced 2025-08-15 12:58:19 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix reading msvs version on Windows (#2644)

Browse source 
* fix: fix reading msvs version on windows
This commit is contained in:
James Cook 2023-06-25 08:35:41 -07:00 • committed by GitHub
parent b030555cdb
commit 53c99ae573
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 11 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
lib/configure.js
View file

@ -85,7 +85,7 @@ function configure (gyp, argv, callback) {
'build dir', '"build" dir needed to be created?', isNew ? 'Yes' : 'No'
)
if (win) {
findVisualStudio(release.semver, gyp.opts.msvs_version,
findVisualStudio(release.semver, gyp.opts['msvs-version'],
createConfigFile)
} else {
createPythonSymlink()

2
lib/node-gyp.js
View file

@ -60,7 +60,7 @@ proto.configDefs = {
debug: Boolean, // 'build'
directory: String, // bin
make: String, // 'build'
msvs_version: String, // 'configure'
'msvs-version': String, // 'configure'
ensure: Boolean, // 'install'
solution: String, // 'build' (windows only)
proxy: String, // 'install'

9
test/test-options.js
View file

@ -38,4 +38,13 @@ describe('options', function () {
assert.strictEqual(g.opts['force-process-config'], 'true')
})
it('options with msvs_version', () => {
process.env.npm_config_msvs_version = '2017'
const g = gyp()
g.parseArgv(['rebuild']) // Also sets opts.argv.
assert.strictEqual(g.opts['msvs-version'], '2017')
})
})