Skip to content

Commit d1b7193

Browse files
refackaddaleax
authored andcommitted
deps,v8: silence V8 self-deprecation warnings
PR-URL: #25394 Reviewed-By: Ujjwal Sharma <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Yang Guo <[email protected]>
1 parent eb5aab2 commit d1b7193

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

common.gypi

+5-4
Original file line numberDiff line numberDiff line change
@@ -45,12 +45,13 @@
4545
# Old time default, now explicitly stated.
4646
'v8_use_snapshot': 'true',
4747

48+
# These are more relevant for V8 internal development.
4849
# Refs: https://github.com/nodejs/node/issues/23122
4950
# Refs: https://github.com/nodejs/node/issues/23167
50-
# Enable compiler warnings when using V8_DEPRECATED apis.
51-
'v8_deprecation_warnings': 1,
52-
# Enable compiler warnings when using V8_DEPRECATE_SOON apis.
53-
'v8_imminent_deprecation_warnings': 1,
51+
# Enable compiler warnings when using V8_DEPRECATED apis from V8 code.
52+
'v8_deprecation_warnings': 0,
53+
# Enable compiler warnings when using V8_DEPRECATE_SOON apis from V8 code.
54+
'v8_imminent_deprecation_warnings': 0,
5455

5556
# Enable disassembler for `--print-code` v8 options
5657
'v8_enable_disassembler': 1,

deps/v8/gypfiles/features.gypi

+4
Original file line numberDiff line numberDiff line change
@@ -164,9 +164,13 @@
164164
}],
165165
['v8_deprecation_warnings==1', {
166166
'defines': ['V8_DEPRECATION_WARNINGS',],
167+
},{
168+
'defines!': ['V8_DEPRECATION_WARNINGS',],
167169
}],
168170
['v8_imminent_deprecation_warnings==1', {
169171
'defines': ['V8_IMMINENT_DEPRECATION_WARNINGS',],
172+
},{
173+
'defines!': ['V8_IMMINENT_DEPRECATION_WARNINGS',],
170174
}],
171175
['v8_enable_i18n_support==1', {
172176
'defines': ['V8_INTL_SUPPORT',],

0 commit comments

Comments
 (0)