Skip to content

Commit

Permalink
build: add "--partly-static" build options
Browse files Browse the repository at this point in the history
A lot of machines don't upgrade libstdc++.so library for a long time,
but the new version of node requires the latest GLIBCXX.

Using "--fully-static" configurable options may resolve this problem,
but the side effect is that the size of the executable file will be
increased.

Adding "--partly-static" configurable options it will only build
libgcc and libstdc++ libraries into executable file, resolve the
problem and control the size of file.

PR-URL: nodejs#4152
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
  • Loading branch information
medns authored and Michael Scovetta committed Apr 2, 2016
1 parent 06cba84 commit 51b4312
Showing 1 changed file with 19 additions and 6 deletions.
25 changes: 19 additions & 6 deletions configure
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,13 @@ parser.add_option("--fully-static",
action="store_true",
dest="fully_static",
help="Generate an executable without external dynamic libraries. This "
"will not work on OSX when using default compilation environment")
"will not work on OSX when using the default compilation environment")

parser.add_option("--partly-static",
action="store_true",
dest="partly_static",
help="Generate an executable with libgcc and libstdc++ libraries. This "
"will not work on OSX when using the default compilation environment")

parser.add_option("--enable-vtune-profiling",
action="store_true",
Expand Down Expand Up @@ -816,12 +822,19 @@ def configure_openssl(o):
configure_library('openssl', o)


def configure_fullystatic(o):
if options.fully_static:
o['libraries'] += ['-static']
def configure_static(o):
if options.fully_static or options.partly_static:
if flavor == 'mac':
print("Generation of static executable will not work on OSX "
"when using default compilation environment")
"when using the default compilation environment")
return

if options.fully_static:
o['libraries'] += ['-static']
elif options.partly_static:
o['libraries'] += ['-static-libgcc', '-static-libstdc++']
if options.enable_asan:
o['libraries'] += ['-static-libasan']


def configure_winsdk(o):
Expand Down Expand Up @@ -1119,7 +1132,7 @@ configure_v8(output)
configure_openssl(output)
configure_winsdk(output)
configure_intl(output)
configure_fullystatic(output)
configure_static(output)

# variables should be a root level element,
# move everything else to target_defaults
Expand Down

0 comments on commit 51b4312

Please sign in to comment.