Commit 536bd76a authored by Myles Borins's avatar Myles Borins Committed by Commit Bot

[tools] update node fetch_deps.py for gn

Various fixes necessary to get the script working with V8 6.6

Upstreamed from https://github.com/nodejs/node/pull/19201

Change-Id: Ic7819eb17cf4be8380b8c1811e569236244e7400
Reviewed-on: https://chromium-review.googlesource.com/996858
Commit-Queue: Myles Borins <mborins@google.com>
Reviewed-by: 's avatarYang Guo <yangguo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#52421}
parent 503e07c3
......@@ -3,7 +3,6 @@
# all paths in here must match this assumption.
vars = {
'build_for_node': False,
'checkout_instrumented_libraries': False,
'chromium_url': 'https://chromium.googlesource.com',
'download_gcmole': False,
......@@ -100,7 +99,7 @@ hooks = [
{
'name': 'clang_format_win',
'pattern': '.',
'condition': 'host_os == "win" and build_for_node != True',
'condition': 'host_os == "win"',
'action': [ 'download_from_google_storage',
'--no_resume',
'--platform=win32',
......@@ -112,7 +111,7 @@ hooks = [
{
'name': 'clang_format_mac',
'pattern': '.',
'condition': 'host_os == "mac" and build_for_node != True',
'condition': 'host_os == "mac"',
'action': [ 'download_from_google_storage',
'--no_resume',
'--platform=darwin',
......@@ -124,7 +123,7 @@ hooks = [
{
'name': 'clang_format_linux',
'pattern': '.',
'condition': 'host_os == "linux" and build_for_node != True',
'condition': 'host_os == "linux"',
'action': [ 'download_from_google_storage',
'--no_resume',
'--platform=linux*',
......@@ -159,7 +158,7 @@ hooks = [
{
'name': 'luci-go_win',
'pattern': '.',
'condition': 'host_os == "win" and build_for_node != True',
'condition': 'host_os == "win"',
'action': [ 'download_from_google_storage',
'--no_resume',
'--platform=win32',
......@@ -171,7 +170,7 @@ hooks = [
{
'name': 'luci-go_mac',
'pattern': '.',
'condition': 'host_os == "mac" and build_for_node != True',
'condition': 'host_os == "mac"',
'action': [ 'download_from_google_storage',
'--no_resume',
'--platform=darwin',
......@@ -183,7 +182,7 @@ hooks = [
{
'name': 'luci-go_linux',
'pattern': '.',
'condition': 'host_os == "linux" and build_for_node != True',
'condition': 'host_os == "linux"',
'action': [ 'download_from_google_storage',
'--no_resume',
'--platform=linux*',
......@@ -232,7 +231,6 @@ hooks = [
{
'name': 'wasm_spec_tests',
'pattern': '.',
'condition': 'build_for_node != True',
'action': [ 'download_from_google_storage',
'--no_resume',
'--no_auth',
......@@ -244,7 +242,6 @@ hooks = [
{
'name': 'closure_compiler',
'pattern': '.',
'condition': 'build_for_node != True',
'action': [ 'download_from_google_storage',
'--no_resume',
'--no_auth',
......@@ -323,7 +320,7 @@ hooks = [
{
'name': 'binutils',
'pattern': 'v8/third_party/binutils',
'condition': 'host_os == "linux" and build_for_node != True',
'condition': 'host_os == "linux"',
'action': [
'python',
'v8/third_party/binutils/download.py',
......
......@@ -32,7 +32,6 @@ GCLIENT_SOLUTION = [
"v8/test/mozilla/data" : None,
"v8/test/test262/data" : None,
"v8/test/test262/harness" : None,
"v8/test/wasm-js" : None,
"v8/third_party/android_tools" : None,
"v8/third_party/catapult" : None,
"v8/third_party/colorama/src" : None,
......@@ -40,9 +39,6 @@ GCLIENT_SOLUTION = [
"v8/tools/luci-go" : None,
"v8/tools/swarming_client" : None,
},
"custom_vars": {
"build_for_node" : True,
},
},
]
......@@ -55,6 +51,8 @@ def EnsureGit(v8_path):
return False
print "Initializing temporary git repository in v8."
subprocess.check_call(["git", "init"], cwd=v8_path)
subprocess.check_call(["git", "config", "user.name", "\"Ada Lovelace\""], cwd=v8_path)
subprocess.check_call(["git", "config", "user.email", "\"ada@lovela.ce\""], cwd=v8_path)
subprocess.check_call(["git", "commit", "--allow-empty", "-m", "init"],
cwd=v8_path)
return True
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment