Skip to content

Commit e03a7b2

Browse files
committed
build: turn on thin static archives
Thin archives were disabled in 2012 as a workaround (IIRC) for obsolete tooling on one of Joyent's platforms. The last binutils versions that didn't support them was released in 2007 so I think it's safe to assume we can drop support for that now - except on SmartOS, where the tooling still has a distinctive vintage feel to it. Thin archives save space - it shrinks the size of PRODUCT_DIR by 30% - and speed up the final linking step because it doesn't have to assemble 50 MB of static archives (twice! - first to create the archive, then to copy it to PRODUCT_DIR). The archives are just 3.5 MB now and no longer copied around. PR-URL: #7957 Reviewed-By: Johan Bergström <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 55852e1 commit e03a7b2

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

common.gypi

+3-2
Original file line numberDiff line numberDiff line change
@@ -271,8 +271,9 @@
271271
'cflags_cc': [ '-fno-rtti', '-fno-exceptions', '-std=gnu++0x' ],
272272
'ldflags': [ '-rdynamic' ],
273273
'target_conditions': [
274-
['_type=="static_library"', {
275-
'standalone_static_library': 1, # disable thin archive which needs binutils >= 2.19
274+
# The 1990s toolchain on SmartOS can't handle thin archives.
275+
['_type=="static_library" and OS=="solaris"', {
276+
'standalone_static_library': 1,
276277
}],
277278
],
278279
'conditions': [

node.gyp

+3-1
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,9 @@
366366
'conditions': [
367367
['OS in "linux freebsd" and node_shared=="false"', {
368368
'ldflags': [
369-
'-Wl,--whole-archive <(PRODUCT_DIR)/<(OPENSSL_PRODUCT)',
369+
'-Wl,--whole-archive,'
370+
'<(PRODUCT_DIR)/obj.target/deps/openssl/'
371+
'<(OPENSSL_PRODUCT)',
370372
'-Wl,--no-whole-archive',
371373
],
372374
}],

0 commit comments

Comments
 (0)