Skip to content

Commit e12a9c5

Browse files
committed
build: re-enable snapshots in v8.x
When 5d5e31c was backported to v8.x as e2b306c it was missing the configure changes from 8dce05f. This patch actually re-enables snapshots! Fixes: #14870 PR-URL: #14875 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]>
1 parent 05a7fc5 commit e12a9c5

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

configure

+4-4
Original file line numberDiff line numberDiff line change
@@ -437,12 +437,12 @@ parser.add_option('--without-perfctr',
437437
# Dummy option for backwards compatibility
438438
parser.add_option('--with-snapshot',
439439
action='store_true',
440-
dest='with_snapshot',
440+
dest='unused_with_snapshot',
441441
help=optparse.SUPPRESS_HELP)
442442

443443
parser.add_option('--without-snapshot',
444444
action='store_true',
445-
dest='unused_without_snapshot',
445+
dest='without_snapshot',
446446
help=optparse.SUPPRESS_HELP)
447447

448448
parser.add_option('--without-ssl',
@@ -827,7 +827,7 @@ def configure_node(o):
827827
cross_compiling = (options.cross_compiling
828828
if options.cross_compiling is not None
829829
else target_arch != host_arch)
830-
want_snapshots = 1 if options.with_snapshot else 0
830+
want_snapshots = not options.without_snapshot
831831
o['variables']['want_separate_host_toolset'] = int(
832832
cross_compiling and want_snapshots)
833833
o['variables']['want_separate_host_toolset_mkpeephole'] = int(
@@ -984,7 +984,7 @@ def configure_v8(o):
984984
o['variables']['v8_optimized_debug'] = 0 # Compile with -O0 in debug builds.
985985
o['variables']['v8_random_seed'] = 0 # Use a random seed for hash tables.
986986
o['variables']['v8_promise_internal_field_count'] = 1 # Add internal field to promises for async hooks.
987-
o['variables']['v8_use_snapshot'] = b(options.with_snapshot)
987+
o['variables']['v8_use_snapshot'] = 'false' if options.without_snapshot else 'true'
988988
o['variables']['v8_trace_maps'] = 1 if options.trace_maps else 0
989989
o['variables']['node_use_v8_platform'] = b(not options.without_v8_platform)
990990
o['variables']['node_use_bundled_v8'] = b(not options.without_bundled_v8)

0 commit comments

Comments
 (0)