Skip to content

Commit 8868378

Browse files
jbergstroemcjihrig
authored andcommitted
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: nodejs/node-v0.x-archive#8786 PR-URL: #38 Reviewed-By: Fedor Indutny <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
1 parent 550a444 commit 8868378

File tree

3 files changed

+2
-42
lines changed

3 files changed

+2
-42
lines changed

configure

Lines changed: 0 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -83,26 +83,6 @@ parser.add_option('--openssl-use-sys',
8383
dest='shared_openssl',
8484
help=optparse.SUPPRESS_HELP)
8585

86-
parser.add_option('--shared-cares',
87-
action='store_true',
88-
dest='shared_cares',
89-
help='link to a shared cares DLL instead of static linking')
90-
91-
parser.add_option('--shared-cares-includes',
92-
action='store',
93-
dest='shared_cares_includes',
94-
help='directory containing cares header files')
95-
96-
parser.add_option('--shared-cares-libname',
97-
action='store',
98-
dest='shared_cares_libname',
99-
help='alternative lib name to link to (default: \'cares\')')
100-
101-
parser.add_option('--shared-cares-libpath',
102-
action='store',
103-
dest='shared_cares_libpath',
104-
help='a directory to search for the shared cares DLL')
105-
10686
parser.add_option('--shared-http-parser',
10787
action='store_true',
10888
dest='shared_http_parser',
@@ -586,20 +566,6 @@ def configure_http_parser(o):
586566
o['include_dirs'] += [options.shared_http_parser_includes]
587567

588568

589-
def configure_cares(o):
590-
o['variables']['node_shared_cares'] = b(options.shared_cares)
591-
592-
# assume shared cares if one of these is set?
593-
if options.shared_cares_libpath:
594-
o['libraries'] += ['-L%s' % options.shared_cares_libpath]
595-
if options.shared_cares_libname:
596-
o['libraries'] += ['-l%s' % options.shared_cares_libname]
597-
elif options.shared_cares:
598-
o['libraries'] += ['-lcares']
599-
if options.shared_cares_includes:
600-
o['include_dirs'] += [options.shared_cares_includes]
601-
602-
603569
def configure_libuv(o):
604570
o['variables']['node_shared_libuv'] = b(options.shared_libuv)
605571

@@ -863,7 +829,6 @@ output = {
863829
configure_node(output)
864830
configure_libz(output)
865831
configure_http_parser(output)
866-
configure_cares(output)
867832
configure_libuv(output)
868833
configure_v8(output)
869834
configure_openssl(output)

node.gyp

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
'node_shared_v8%': 'false',
99
'node_shared_zlib%': 'false',
1010
'node_shared_http_parser%': 'false',
11-
'node_shared_cares%': 'false',
1211
'node_shared_libuv%': 'false',
1312
'node_use_openssl%': 'true',
1413
'node_shared_openssl%': 'false',
@@ -78,6 +77,7 @@
7877

7978
'dependencies': [
8079
'node_js2c#host',
80+
'deps/cares/cares.gyp:cares'
8181
],
8282

8383
'include_dirs': [
@@ -313,10 +313,6 @@
313313
'dependencies': [ 'deps/http_parser/http_parser.gyp:http_parser' ],
314314
}],
315315

316-
[ 'node_shared_cares=="false"', {
317-
'dependencies': [ 'deps/cares/cares.gyp:cares' ],
318-
}],
319-
320316
[ 'node_shared_libuv=="false"', {
321317
'dependencies': [ 'deps/uv/uv.gyp:libuv' ],
322318
}],

tools/install.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,8 +154,7 @@ def files(action):
154154
'src/smalloc.h',
155155
], 'include/node/')
156156

157-
if 'false' == variables.get('node_shared_cares'):
158-
subdir_files('deps/cares/include', 'include/node/', action)
157+
subdir_files('deps/cares/include', 'include/node/', action)
159158

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

0 commit comments

Comments
 (0)