Skip to content

Commit

Permalink
[build] Port: Use CIPD packages for GN instead of GCS
Browse files Browse the repository at this point in the history
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 <[email protected]>
Commit-Queue: Oleh Prypin <[email protected]>
Cr-Commit-Position: refs/heads/master@{#27156}
  • Loading branch information
oprypin authored and Commit Bot committed Mar 18, 2019
1 parent df644be commit 02ba0ac
Showing 1 changed file with 36 additions and 38 deletions.
74 changes: 36 additions & 38 deletions DEPS
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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' + '@' +
Expand All @@ -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':
Expand Down Expand Up @@ -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',
Expand Down

0 comments on commit 02ba0ac

Please sign in to comment.