From 1bb4cfe960647acb61820608918be582ff0f7611 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Johan=20Bergstr=C3=B6m?= Date: Wed, 3 Dec 2014 18:28:40 +1100 Subject: [PATCH] configure: remove shared cares build option Bundled cares differs from upstream which will result in a compilation error when trying to used a shared cares. Fixes: https://github.com/joyent/node/pull/8786 PR-URL: https://github.com/iojs/io.js/pull/38 Reviewed-By: Fedor Indutny Reviewed-By: Ben Noordhuis --- configure | 35 ----------------------------------- node.gyp | 6 +----- tools/install.py | 3 +-- 3 files changed, 2 insertions(+), 42 deletions(-) diff --git a/configure b/configure index b9e5473816cadc..09741bc41c4b1a 100755 --- a/configure +++ b/configure @@ -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', @@ -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) @@ -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) diff --git a/node.gyp b/node.gyp index 21391e0e3bd359..8615bf72736f7e 100644 --- a/node.gyp +++ b/node.gyp @@ -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', @@ -78,6 +77,7 @@ 'dependencies': [ 'node_js2c#host', + 'deps/cares/cares.gyp:cares' ], 'include_dirs': [ @@ -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' ], }], diff --git a/tools/install.py b/tools/install.py index 52c11099411dfb..c99dc1579bfeb0 100755 --- a/tools/install.py +++ b/tools/install.py @@ -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)