Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(33)

Issue 189093017: Less attempted magic in pulling toolchain (Closed)

Created:
6 years, 9 months ago by scottmg
Modified:
6 years, 9 months ago
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, M-A Ruel, iannucci+depot_tools_chromium.org, ilevy-cc_chromium.org
Visibility:
Public.

Description

Less attempted magic in pulling toolchain - Don't try to wrap download_from_google_storage --config, instead just request that the user runs it manually. (This is annoying either way but making it magically run makes it less clear what's going on, and so harder to debug when something goes wrong, per linked bug). - Check that SHA1s are passed as expected on the command line in case the script is run directly, rather than from gyp_chromium. R=scherkus@chromium.org BUG=349596 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=256049

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -28 lines) Patch
M win_toolchain/get_toolchain_if_necessary.py View 3 chunks +26 lines, -28 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
scottmg
6 years, 9 months ago (2014-03-10 20:38:29 UTC) #1
scottmg
6 years, 9 months ago (2014-03-10 20:38:47 UTC) #2
scherkus (not reviewing)
lgtm from me -- thanks for the help sorting out my windows build woes!
6 years, 9 months ago (2014-03-10 22:01:08 UTC) #3
iannucci
rslgtm
6 years, 9 months ago (2014-03-10 22:03:07 UTC) #4
scottmg
The CQ bit was checked by scottmg@chromium.org
6 years, 9 months ago (2014-03-10 22:14:25 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/scottmg@chromium.org/189093017/1
6 years, 9 months ago (2014-03-10 22:14:46 UTC) #6
commit-bot: I haz the power
6 years, 9 months ago (2014-03-10 22:19:15 UTC) #7
Message was sent while issue was closed.
Change committed as 256049

Powered by Google App Engine
This is Rietveld 408576698