Commit 648766b
build: do not build mksnapshot and mkcodecache for --shared
To build mkcodecache and mksnapshot (they are executables),
we currently build libnode with unresolved symbols, then build
the two exectuables with src/node_snapshot_stub.cc and
src/node_code_cache_stub.cc. Each of them write a C++ file to
disk when being run. We then use the generated C++ files & libnode
(with unresolved symbols) to build the final Node executable.
However, if libnode itself is the final product, then we should
not build it with unresolved symbols.
#28897 added the two stubs
for the libnode target when the --shared configure option is used,
but it did not get rid of the actions to build and run mksnapshot
and mkcodecache for --shared, so to get it working we also
need a patch to make sure --shared imply --without-node-code-cache
and --without-node-snapshot, until we actually fix the TODO so that
mksnapshot and mkcodecache do not use the libnode that way.
PR-URL: #30647
Refs: #28845
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 6545314 commit 648766b
1 file changed
Lines changed: 4 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -981,13 +981,15 @@ def configure_node(o): | |||
| 981 | 981 | o['variables']['want_separate_host_toolset'] = int(cross_compiling) | |
| 982 | 982 | ||
| 983 | 983 | if not options.without_node_snapshot: | |
| 984 | - o['variables']['node_use_node_snapshot'] = b(not cross_compiling) | ||
| 984 | + o['variables']['node_use_node_snapshot'] = b( | ||
| 985 | + not cross_compiling and not options.shared) | ||
| 985 | 986 | else: | |
| 986 | 987 | o['variables']['node_use_node_snapshot'] = 'false' | |
| 987 | 988 | ||
| 988 | 989 | if not options.without_node_code_cache: | |
| 989 | 990 | # TODO(refack): fix this when implementing embedded code-cache when cross-compiling. | |
| 990 | - o['variables']['node_use_node_code_cache'] = b(not cross_compiling) | ||
| 991 | + o['variables']['node_use_node_code_cache'] = b( | ||
| 992 | + not cross_compiling and not options.shared) | ||
| 991 | 993 | else: | |
| 992 | 994 | o['variables']['node_use_node_code_cache'] = 'false' | |
| 993 | 995 | ||
0 commit comments