Skip to content

Commit f87c8b2

Browse files
committed
sea: add --build-sea to generate SEA directly with Node.js binary
Instead of relying on a WASM build of postject to perform the injection, add LIEF as dependency and generate the SEA directly from core via a new CLI option --build-sea which takes the SEA config. This simplifies SEA generation for users and makes it easier to debug/maintain the SEA building process.
1 parent 4769e62 commit f87c8b2

37 files changed

+2007
-138
lines changed

.github/CODEOWNERS

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -153,13 +153,15 @@
153153
/test/parallel/test-runner-* @nodejs/test_runner
154154

155155
# Single Executable Applications
156+
/deps/LIEF @nodejs/single-executable
156157
/deps/postject @nodejs/single-executable
157158
/doc/api/single-executable-applications.md @nodejs/single-executable
158159
/doc/contributing/maintaining/maintaining-single-executable-application-support.md @nodejs/single-executable
159160
/src/node_sea* @nodejs/single-executable
160161
/test/fixtures/postject-copy @nodejs/single-executable
161162
/test/sea @nodejs/single-executable
162163
/tools/dep_updaters/update-postject.sh @nodejs/single-executable
164+
/tools/dep_updaters/update-lief.sh @nodejs/single-executable
163165

164166
# Permission Model
165167
/doc/api/permissions.md @nodejs/security-wg

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,8 @@ tools/*/*.i.tmp
145145
# Ignore dependencies fetched by tools/v8/fetch_deps.py
146146
/deps/.cipd
147147
!deps/LIEF/**
148+
deps/LIEF/*.vcxproj*
149+
deps/LIEF/*.sln
148150

149151
# === Rules for Windows vcbuild.bat ===
150152
/temp-vcbuild

LICENSE

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1228,7 +1228,8 @@ The externally maintained libraries used by Node.js are:
12281228
SOFTWARE.
12291229
"""
12301230

1231-
- postject, located at test/fixtures/postject-copy, is licensed as follows:
1231+
- postject, located at test/fixtures/postject-copy and used as a basis for
1232+
src/node_sea_bin.cc, is licensed as follows:
12321233
"""
12331234
Postject is licensed for use as follows:
12341235

configure.py

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -841,6 +841,12 @@
841841
default=None,
842842
help='do not install the bundled Amaro (TypeScript utils)')
843843

844+
parser.add_argument('--without-lief',
845+
action='store_true',
846+
dest='without_lief',
847+
default=None,
848+
help='build without LIEF (Library for instrumenting executable formats)')
849+
844850
parser.add_argument('--without-npm',
845851
action='store_true',
846852
dest='without_npm',
@@ -1710,6 +1716,14 @@ def configure_node(o):
17101716
o['variables']['single_executable_application'] = b(not options.disable_single_executable_application)
17111717
if options.disable_single_executable_application:
17121718
o['defines'] += ['DISABLE_SINGLE_EXECUTABLE_APPLICATION']
1719+
o['variables']['node_use_lief'] = 'false'
1720+
else:
1721+
if (options.without_lief is not None):
1722+
o['variables']['node_use_lief'] = b(not options.without_lief)
1723+
elif flavor in ('mac', 'linux', 'win'):
1724+
o['variables']['node_use_lief'] = 'true'
1725+
else:
1726+
o['variables']['node_use_lief'] = 'false'
17131727

17141728
o['variables']['node_with_ltcg'] = b(options.with_ltcg)
17151729
if flavor != 'win' and options.with_ltcg:

0 commit comments

Comments
 (0)