Skip to content

Commit

Permalink
build: set disable_glibcxx_debug flag
Browse files Browse the repository at this point in the history
This breaks a few tests when Node is built in debug mode, because
V8 is built with `-D_GLIBCXX_DEBUG=1` and Node is built without,
which makes e.g. `std::vector` ABI-incompatible between the two.

PR-URL: #16159
Reviewed-By: James Snell <jasnell@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
  • Loading branch information
addaleax authored and targos committed Oct 18, 2017
1 parent ea0fec2 commit e32b10f
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions common.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@
# Don't bake anything extra into the snapshot.
'v8_use_external_startup_data%': 0,

# Some STL containers (e.g. std::vector) do not preserve ABI compatibility
# between debug and non-debug mode.
'disable_glibcxx_debug': 1,

# Don't use ICU data file (icudtl.dat) from V8, we use our own.
'icu_use_data_file_flag%': 0,

Expand Down

0 comments on commit e32b10f

Please sign in to comment.