Commit e1bd53c
build: define
V8 and Node.js had defined `NOMINMAX` on Windows for a long time. In
recent changes, V8 added `std::numeric_limits::min` usages in its
header files which caused addons without `NOMINMAX` defines failed
to compile.
Define `NOMINMAX` in common.gypi so that addons can be compiled with
the latest V8 header files.
PR-URL: #52794
Fixes: nodejs/nan#968
Refs: nodejs/gyp-next#244
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>NOMINMAX in common.gypi1 parent 4dc8a38 commit e1bd53cCopy full SHA for e1bd53c
File tree
Expand file treeCollapse file tree
5 files changed
+33
-4
lines changedOpen diff view settings
Filter options
- test/addons/hello-world
Expand file treeCollapse file tree
5 files changed
+33
-4
lines changedOpen diff view settings
Collapse file
+4Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
458 | 458 | |
459 | 459 | |
460 | 460 | |
| 461 | + |
| 462 | + |
| 463 | + |
| 464 | + |
461 | 465 | |
462 | 466 | |
463 | 467 | |
|
Collapse file
-4Lines changed: 0 additions & 4 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
63 | 63 | |
64 | 64 | |
65 | 65 | |
66 | | - |
67 | | - |
68 | | - |
69 | | - |
70 | 66 | |
71 | 67 | |
72 | 68 | |
|
Collapse file
test/addons/hello-world/binding.gyp
Copy file name to clipboardExpand all lines: test/addons/hello-world/binding.gyp+5Lines changed: 5 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
4 | 4 | |
5 | 5 | |
6 | 6 | |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
7 | 12 | |
8 | 13 | |
9 | 14 | |
Collapse file
test/addons/hello-world/binding2.cc
Copy file name to clipboard+21Lines changed: 21 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
Collapse file
test/addons/hello-world/test.js
Copy file name to clipboardExpand all lines: test/addons/hello-world/test.js+3Lines changed: 3 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
6 | 6 | |
7 | 7 | |
8 | 8 | |
| 9 | + |
| 10 | + |
| 11 | + |
9 | 12 | |
10 | 13 | |
11 | 14 | |
|
0 commit comments