Skip to content

Commit

Permalink
build: remove --code-cache-path help option
Browse files Browse the repository at this point in the history
This commit removes the now obsolete option.

PR-URL: #28446
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
danbev committed Jul 4, 2019
1 parent af3992b commit 39a9358
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
7 changes: 1 addition & 6 deletions configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -515,11 +515,6 @@
dest='without_siphash',
help=optparse.SUPPRESS_HELP)

parser.add_option('--code-cache-path',
action='store',
dest='code_cache_path',
help='optparse.SUPPRESS_HELP')

# End dummy list.

parser.add_option('--without-ssl',
Expand Down Expand Up @@ -1116,7 +1111,7 @@ def configure_node(o):
o['variables']['node_no_browser_globals'] = b(options.no_browser_globals)
# TODO(refack): fix this when implementing embedded code-cache when cross-compiling.
if o['variables']['want_separate_host_toolset'] == 0:
o['variables']['node_code_cache_path'] = 'yes'
o['variables']['node_code_cache'] = 'yes' # For testing
o['variables']['node_shared'] = b(options.shared)
node_module_version = getmoduleversion.get_version()

Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-code-cache.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ if (!process.features.cached_builtins) {
}
} else { // Native compiled
assert.strictEqual(
process.config.variables.node_code_cache_path,
process.config.variables.node_code_cache,
'yes'
);

Expand Down

0 comments on commit 39a9358

Please sign in to comment.