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

Issue 541983002: athena: Move the USE_ATHENA define into //chrome. (Closed)

Created:
6 years, 3 months ago by sadrul
Modified:
6 years, 3 months ago
Reviewers:
oshima
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

athena: Move the USE_ATHENA define into //chrome. Switching the use_athena gyp-flag only affects the code inside //chrome. So move the USE_ATHENA inside //chrome so that toggling the flag doesn't cause all targets to be rebuilt, and only code inside //chrome is rebuilt instead. This should also prevent accidental dependence on USE_ATHENA in code outside of //chrome. BUG=none R=oshima@chromium.org Committed: https://chromium.googlesource.com/chromium/src/+/ec96ebfa63de8e17aaf9ec496a5c70c0e085edb7

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -10 lines) Patch
M build/common.gypi View 7 chunks +3 lines, -10 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/chrome_browser_chromeos.gypi View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/chrome_browser_ui.gypi View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
sadrul
6 years, 3 months ago (2014-09-05 03:21:02 UTC) #2
oshima
On 2014/09/05 03:21:02, sadrul wrote: Is it possible to define in chrome.gyp so that we ...
6 years, 3 months ago (2014-09-05 14:27:09 UTC) #3
sadrul
On 2014/09/05 14:27:09, oshima wrote: > On 2014/09/05 03:21:02, sadrul wrote: > > Is it ...
6 years, 3 months ago (2014-09-05 15:44:47 UTC) #4
sadrul
Committed patchset #1 (id:1) manually as ec96ebf (presubmit successful).
6 years, 3 months ago (2014-09-05 19:18:49 UTC) #5
commit-bot: I haz the power
6 years, 3 months ago (2014-09-10 03:41:06 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/ec96ebfa63de8e17aaf9ec496a5c70c0e085edb7
Cr-Commit-Position: refs/heads/master@{#293566}

Powered by Google App Engine
This is Rietveld 408576698