Skip to content

Commit 186b36e

Browse files
committed
deps: V8: cherry-pick b5b5d6c31bb0
Original commit message: Fix for no member named 'init_jmpbuf' on AIX In AIX, the system header file usr/include/sys/context.h file has jmpbuf redefined as __jmpbuf which is creating the problem here. Change-Id: I4393e260092016315ac7559465684e3fdbba4900 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4868434 Reviewed-by: Jakob Linke <jgruber@chromium.org> Commit-Queue: Milad Farazmand <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#90087} Refs: v8/v8@b5b5d6c PR-URL: #49639 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
1 parent 867586c commit 186b36e

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

common.gypi

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636

3737
# Reset this number to 0 on major V8 upgrades.
3838
# Increment by one for each non-official patch applied to deps/v8.
39-
'v8_embedder_string': '-node.8',
39+
'v8_embedder_string': '-node.9',
4040

4141
##### V8 defaults for Node.js #####
4242

deps/v8/src/common/globals.h

+6
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,12 @@
2121

2222
#define V8_INFINITY std::numeric_limits<double>::infinity()
2323

24+
// AIX has jmpbuf redefined as __jmpbuf in /usr/include/sys/context.h
25+
// which replaces v8's jmpbuf , resulting in undefined symbol errors
26+
#if defined(V8_OS_AIX) && defined(jmpbuf)
27+
#undef jmpbuf
28+
#endif
29+
2430
namespace v8 {
2531

2632
namespace base {

0 commit comments

Comments
 (0)