Commit 7f347fc
committed
build: fix getting OpenSSL version on Windows
Node.js on Windows is built with `clang`, not `gcc`.
PR-URL: #59609
Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 8dbd0f1 commit 7f347fc
1 file changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -20,9 +20,9 @@ | |||
| 20 | 20 | original_argv = sys.argv[1:] | |
| 21 | 21 | ||
| 22 | 22 | # gcc and g++ as defaults matches what GYP's Makefile generator does, | |
| 23 | - # except on OS X. | ||
| 24 | - CC = os.environ.get('CC', 'cc' if sys.platform == 'darwin' else 'gcc') | ||
| 25 | - CXX = os.environ.get('CXX', 'c++' if sys.platform == 'darwin' else 'g++') | ||
| 23 | + # except on macOS and Windows. | ||
| 24 | + CC = os.environ.get('CC', 'cc' if sys.platform == 'darwin' else 'clang' if sys.platform == 'win32' else 'gcc') | ||
| 25 | + CXX = os.environ.get('CXX', 'c++' if sys.platform == 'darwin' else 'clang' if sys.platform == 'win32' else 'g++') | ||
| 26 | 26 | ||
| 27 | 27 | tools_path = Path('tools') | |
| 28 | 28 | ||
0 commit comments