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

Issue 137993014: Fix Windows GN build. (Closed)

Created:
6 years, 11 months ago by brettw
Modified:
6 years, 11 months ago
Reviewers:
sky
CC:
chromium-reviews
Visibility:
Public.

Description

Fix Windows GN build. This sets use_aura on Windows in all cases (fixes base build). Passes new argument to gyp link wrapper (GYP revision 1835). R=sky@chromium.org TBR=sky Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=244840

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -1 line) Patch
M build/config/BUILD.gn View 1 chunk +4 lines, -0 lines 0 comments Download
M build/config/BUILDCONFIG.gn View 1 chunk +3 lines, -0 lines 1 comment Download
M build/toolchain/win/BUILD.gn View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
brettw
6 years, 11 months ago (2014-01-14 23:34:53 UTC) #1
sky
LGTM https://codereview.chromium.org/137993014/diff/1/build/config/BUILDCONFIG.gn File build/config/BUILDCONFIG.gn (right): https://codereview.chromium.org/137993014/diff/1/build/config/BUILDCONFIG.gn#newcode109 build/config/BUILDCONFIG.gn:109: # Windows currelty implies Aura. currently
6 years, 11 months ago (2014-01-14 23:35:47 UTC) #2
brettw
Committed patchset #1 manually as r244840 (presubmit successful).
6 years, 11 months ago (2014-01-15 06:11:04 UTC) #3
Matt Giuca
Hey, this CL broke my Windows build when I have: GYP_DEFINES="use_aura=1" ERROR at //build/config/BUILDCONFIG.gn:110:3: Overwriting ...
6 years, 11 months ago (2014-01-17 03:28:07 UTC) #4
brettw
6 years, 11 months ago (2014-01-17 03:30:12 UTC) #5
Message was sent while issue was closed.
Yeah, I have a fix for this but haven't gotten it checked in yet. You can remove
the aura setting from GYP since it's redundant now.

Powered by Google App Engine
This is Rietveld 408576698