From 02ba0ac2e46862fb743c1f3473f241d7b35e18eb Mon Sep 17 00:00:00 2001 From: Oleh Prypin Date: Mon, 18 Mar 2019 09:24:17 +0100 Subject: [PATCH] [build] Port: Use CIPD packages for GN instead of GCS This ports: https://chromium-review.googlesource.com/c/chromium/src/+/1519726 > This CL does *not* change the current version of GN we're using; > it still pulls r1496 (0790d304). Copied from V8's fix CL: https://chromium-review.googlesource.com/c/v8/v8/+/1526014 This will unblock Chromium DEPS roll, which is failing: https://webrtc-review.googlesource.com/c/src/+/128204/ Bug: chromium:855791 Change-Id: I21a33ee047510ca424d1afc1585b3f42ff0755dc Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/127887 Reviewed-by: Oleksandr Iakovenko Commit-Queue: Oleh Prypin Cr-Commit-Position: refs/heads/master@{#27156} --- DEPS | 74 +++++++++++++++++++++++++++++------------------------------- 1 file changed, 36 insertions(+), 38 deletions(-) diff --git a/DEPS b/DEPS index e5d8269a5c..f037aca5be 100644 --- a/DEPS +++ b/DEPS @@ -41,6 +41,10 @@ vars = { # the commit queue can handle CLs rolling luci-go CIPD package version # and whatever else without interference from each other. 'luci_go': 'git_revision:25958d48e89e980e2a97daeddc977fb5e2e1fb8c', + # Three lines of non-changing comments so that + # the commit queue can handle CLs rolling GN CIPD package version + # and whatever else without interference from each other. + 'gn_version': 'git_revision:0790d3043387c762a6bacb1ae0a9ebe883188ab2', } deps = { # TODO(kjellander): Move this to be Android-only once the libevent dependency @@ -50,7 +54,7 @@ deps = { 'src/build': Var('chromium_git') + '/chromium/src/build' + '@' + 'f89a8d1d0f9a5f6105cf2deebe6a977943440ad2', 'src/buildtools': - Var('chromium_git') + '/chromium/src/buildtools' + '@' + '62f9eb0d64d6bf48f620b8233d9f7a1dc07f8414', + Var('chromium_git') + '/chromium/src/buildtools' + '@' + 'c79f3482c8152172a31e5c17823a27835a511dac', # Gradle 4.3-rc4. Used for testing Android Studio project generation for WebRTC. 'src/examples/androidtests/third_party/gradle': { 'url': Var('chromium_git') + '/external/github.com/gradle/gradle.git' + '@' + @@ -66,6 +70,37 @@ deps = { 'src/third_party': Var('chromium_git') + '/chromium/src/third_party' + '@' + '345968dc5db2cd479a86e5a93af14c141d31f3a0', + 'src/buildtools/linux64': { + 'packages': [ + { + 'package': 'gn/gn/linux-amd64', + 'version': Var('gn_version'), + } + ], + 'dep_type': 'cipd', + 'condition': 'checkout_linux', + }, + 'src/buildtools/mac': { + 'packages': [ + { + 'package': 'gn/gn/mac-amd64', + 'version': Var('gn_version'), + } + ], + 'dep_type': 'cipd', + 'condition': 'checkout_mac', + }, + 'src/buildtools/win': { + 'packages': [ + { + 'package': 'gn/gn/windows-amd64', + 'version': Var('gn_version'), + } + ], + 'dep_type': 'cipd', + 'condition': 'checkout_win', + }, + 'src/buildtools/clang_format/script': Var('chromium_git') + '/chromium/llvm-project/cfe/tools/clang-format.git' + '@' + '96636aa0e9f047f17447f2d45a094d0b59ed7917', 'src/buildtools/third_party/libc++/trunk': @@ -1336,43 +1371,6 @@ hooks = [ 'action': ['python', 'src/build/util/lastchange.py', '-o', 'src/build/util/LASTCHANGE'], }, - # Pull GN binaries. - { - 'name': 'gn_win', - 'pattern': '.', - 'condition': 'host_os == "win"', - 'action': [ 'download_from_google_storage', - '--no_resume', - '--platform=win32', - '--no_auth', - '--bucket', 'chromium-gn', - '-s', 'src/buildtools/win/gn.exe.sha1', - ], - }, - { - 'name': 'gn_mac', - 'pattern': '.', - 'condition': 'host_os == "mac"', - 'action': [ 'download_from_google_storage', - '--no_resume', - '--platform=darwin', - '--no_auth', - '--bucket', 'chromium-gn', - '-s', 'src/buildtools/mac/gn.sha1', - ], - }, - { - 'name': 'gn_linux64', - 'pattern': '.', - 'condition': 'host_os == "linux"', - 'action': [ 'download_from_google_storage', - '--no_resume', - '--platform=linux*', - '--no_auth', - '--bucket', 'chromium-gn', - '-s', 'src/buildtools/linux64/gn.sha1', - ], - }, # Pull clang-format binaries using checked-in hashes. { 'name': 'clang_format_win',