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

Issue 170273005: android envsetup: Stop setting GYP_DEFINES. (Closed)

Created:
6 years, 10 months ago by Nico
Modified:
6 years, 8 months ago
Reviewers:
Yaron
CC:
chromium-reviews, craigdh+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org, klundberg+watch_chromium.org, frankf+watch_chromium.org
Visibility:
Public.

Description

android envsetup: Stop setting GYP_DEFINES. After this, envsetup no longer sets OS=android and no longer translates GOMADIR to use_goma=1. You now need to set these yourself, but since envsetup no longer overwrites GYP_DEFINES, setting them is now much easier. (For example, just run `export GYP_DEFINES='OS=android use_goma=1'` before running `gclient sync`). BUG=330631 R=yfriedman@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260548

Patch Set 1 #

Patch Set 2 : . #

Patch Set 3 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -13 lines) Patch
M build/android/buildbot/bb_run_bot.py View 1 2 1 chunk +1 line, -1 line 0 comments Download
M build/android/envsetup.sh View 3 chunks +0 lines, -12 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Nico
I suppose I need an lgtm for this if I want to land this on ...
6 years, 8 months ago (2014-03-27 18:58:22 UTC) #1
Yaron
On 2014/03/27 18:58:22, Nico wrote: > I suppose I need an lgtm for this if ...
6 years, 8 months ago (2014-03-27 19:08:06 UTC) #2
Nico
Committed patchset #3 manually as r260548 (tree was closed).
6 years, 8 months ago (2014-03-31 15:12:50 UTC) #3
Miguel Garcia
On 2014/03/31 15:12:50, Nico wrote: > Committed patchset #3 manually as r260548 (tree was closed). ...
6 years, 8 months ago (2014-03-31 16:39:14 UTC) #4
Nico
6 years, 8 months ago (2014-03-31 16:50:16 UTC) #5
Message was sent while issue was closed.
On 2014/03/31 16:39:14, Miguel Garcia wrote:
> On 2014/03/31 15:12:50, Nico wrote:
> > Committed patchset #3 manually as r260548 (tree was closed).
> 
> While I agree that envsetup had grew into an enormous monster, this was
actually
> useful. How are we going to remember to set these two gyp defines every time
we
> open a new shell? This is the kind of thing we would script and share which is
> precisely what envesetup was doing...

See this thread:
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/_lULw0aZhkk

I updated all build instruction pages I could find with the new steps.

Powered by Google App Engine
This is Rietveld 408576698