Skip to content

Commit

Permalink
build: rename node_code_cache_path to node_code_cache
Browse files Browse the repository at this point in the history
This commit renames node_code_cache_path and removes the path from as
it does not really describe the configuration property as a path
anymore. Instead it indicates whether the code cache is used or not.
  • Loading branch information
danbev committed Jul 2, 2019
1 parent 9db4857 commit 5dfb434
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -1111,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'
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 5dfb434

Please sign in to comment.