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

Issue 137443024: Hook up gomadir in the GN build. (Closed)

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

Description

Hook up gomadir in the GN build. Without this, GYP will fail if you manually specify gomadir in your GYP_DEFINES. R=jamesr@chromium.org, jamesr Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245595

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M build/gyp_chromium View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
brettw
6 years, 11 months ago (2014-01-17 20:08:39 UTC) #1
jamesr
lgtm
6 years, 11 months ago (2014-01-17 20:14:19 UTC) #2
brettw
Committed patchset #1 manually as r245595 (presubmit successful).
6 years, 11 months ago (2014-01-17 20:20:32 UTC) #3
sharpaquos1974
On 2014/01/17 20:20:32, brettw wrote: > Committed patchset #1 manually as r245595 (presubmit successful).
6 years, 11 months ago (2014-01-18 09:08:58 UTC) #4
sharpaquos1974
On 2014/01/17 20:14:19, jamesr wrote: > lgtm
6 years, 11 months ago (2014-01-18 09:09:08 UTC) #5
sharpaquos1974
On 2014/01/17 20:08:39, brettw wrote:
6 years, 11 months ago (2014-01-18 09:09:22 UTC) #6
sharpaquos1974
6 years, 11 months ago (2014-01-18 09:10:03 UTC) #7
Message was sent while issue was closed.
A revert of this CL has been created in
https://codereview.chromium.org/136473017/ by sharpaquos1974@gmail.com.

The reason for reverting is: login.

Powered by Google App Engine
This is Rietveld 408576698