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

Issue 12545010: Revert 186522 (Closed)

Created:
7 years, 9 months ago by miu
Modified:
7 years, 9 months ago
CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, simonmorris+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, rmsousa+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Revert 186522 > Build src/remoting/remoting.gyp:remoting_host_installation on the main waterfall. > > remoting_host_installation uses lots of non-trivial GYP that tend to break because of differences between ninja and msbuild. Building it on the main waterfall gives us an early warning if something goes wrong. > > BUG=180600 > > Review URL: https://codereview.chromium.org/12543006 TBR=alexeypa@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=186526

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -33 lines) Patch
M build/all.gyp View 1 chunk +19 lines, -28 lines 0 comments Download
M remoting/remoting.gyp View 1 chunk +6 lines, -5 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
miu
7 years, 9 months ago (2013-03-06 23:08:08 UTC) #1
miu
7 years, 9 months ago (2013-03-06 23:08:16 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r186526.

Powered by Google App Engine
This is Rietveld 408576698