Skip to content

Commit

Permalink
configure: remove shared cares build option
Browse files Browse the repository at this point in the history
Bundled cares differs from upstream which will result
in a compilation error when trying to used a shared cares.

Fixes: nodejs/node-v0.x-archive#8786
PR-URL: #38
Reviewed-By: Fedor Indutny <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
  • Loading branch information
jbergstroem authored and rvagg committed Dec 4, 2014
1 parent 206389d commit 766d063
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 42 deletions.
35 changes: 0 additions & 35 deletions configure
Original file line number Diff line number Diff line change
Expand Up @@ -83,26 +83,6 @@ parser.add_option('--openssl-use-sys',
dest='shared_openssl',
help=optparse.SUPPRESS_HELP)

parser.add_option('--shared-cares',
action='store_true',
dest='shared_cares',
help='link to a shared cares DLL instead of static linking')

parser.add_option('--shared-cares-includes',
action='store',
dest='shared_cares_includes',
help='directory containing cares header files')

parser.add_option('--shared-cares-libname',
action='store',
dest='shared_cares_libname',
help='alternative lib name to link to (default: \'cares\')')

parser.add_option('--shared-cares-libpath',
action='store',
dest='shared_cares_libpath',
help='a directory to search for the shared cares DLL')

parser.add_option('--shared-http-parser',
action='store_true',
dest='shared_http_parser',
Expand Down Expand Up @@ -586,20 +566,6 @@ def configure_http_parser(o):
o['include_dirs'] += [options.shared_http_parser_includes]


def configure_cares(o):
o['variables']['node_shared_cares'] = b(options.shared_cares)

# assume shared cares if one of these is set?
if options.shared_cares_libpath:
o['libraries'] += ['-L%s' % options.shared_cares_libpath]
if options.shared_cares_libname:
o['libraries'] += ['-l%s' % options.shared_cares_libname]
elif options.shared_cares:
o['libraries'] += ['-lcares']
if options.shared_cares_includes:
o['include_dirs'] += [options.shared_cares_includes]


def configure_libuv(o):
o['variables']['node_shared_libuv'] = b(options.shared_libuv)

Expand Down Expand Up @@ -863,7 +829,6 @@ output = {
configure_node(output)
configure_libz(output)
configure_http_parser(output)
configure_cares(output)
configure_libuv(output)
configure_v8(output)
configure_openssl(output)
Expand Down
6 changes: 1 addition & 5 deletions node.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
'node_shared_v8%': 'false',
'node_shared_zlib%': 'false',
'node_shared_http_parser%': 'false',
'node_shared_cares%': 'false',
'node_shared_libuv%': 'false',
'node_use_openssl%': 'true',
'node_shared_openssl%': 'false',
Expand Down Expand Up @@ -78,6 +77,7 @@

'dependencies': [
'node_js2c#host',
'deps/cares/cares.gyp:cares'
],

'include_dirs': [
Expand Down Expand Up @@ -313,10 +313,6 @@
'dependencies': [ 'deps/http_parser/http_parser.gyp:http_parser' ],
}],

[ 'node_shared_cares=="false"', {
'dependencies': [ 'deps/cares/cares.gyp:cares' ],
}],

[ 'node_shared_libuv=="false"', {
'dependencies': [ 'deps/uv/uv.gyp:libuv' ],
}],
Expand Down
3 changes: 1 addition & 2 deletions tools/install.py
Original file line number Diff line number Diff line change
Expand Up @@ -154,8 +154,7 @@ def files(action):
'src/smalloc.h',
], 'include/node/')

if 'false' == variables.get('node_shared_cares'):
subdir_files('deps/cares/include', 'include/node/', action)
subdir_files('deps/cares/include', 'include/node/', action)

if 'false' == variables.get('node_shared_libuv'):
subdir_files('deps/uv/include', 'include/node/', action)
Expand Down

0 comments on commit 766d063

Please sign in to comment.