From a350d8b7805765744a10415551bd4a741b6519ff Mon Sep 17 00:00:00 2001 From: Michaƫl Zasso Date: Wed, 9 Oct 2019 11:16:40 +0200 Subject: deps: V8: cherry-pick 53e62af Original commit message: [build] Include string in v8.h Explicitly #include in v8.h, since std::string is referenced in it. In the C++ STL shipped with Visual Studio 2019, none of the headers included in v8.h ends up including the C++ string header, which caused a compile error. Bug: v8:9793 Change-Id: I84a133dd10dd6dcc7b70287af393e82cf0dc97df Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1834321 Reviewed-by: Adam Klein Commit-Queue: Adam Klein Cr-Commit-Position: refs/heads/master@{#64074} Refs: https://github.com/v8/v8/commit/53e62affd33ad036a169eff7d6c1d50b0adcd21a PR-URL: https://github.com/nodejs/node/pull/29898 Reviewed-By: David Carlier Reviewed-By: Richard Lau Reviewed-By: Colin Ihrig Reviewed-By: Ruben Bridgewater Reviewed-By: Gus Caplan --- common.gypi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'common.gypi') diff --git a/common.gypi b/common.gypi index 10b4ecfc52..78a57d9291 100644 --- a/common.gypi +++ b/common.gypi @@ -39,7 +39,7 @@ # Reset this number to 0 on major V8 upgrades. # Increment by one for each non-official patch applied to deps/v8. - 'v8_embedder_string': '-node.13', + 'v8_embedder_string': '-node.14', ##### V8 defaults for Node.js ##### -- cgit v1.2.3