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

Issue 217833006: compile.py: Don't set OFFICIAL_BUILD for --mode=official builds. (Closed)

Created:
6 years, 8 months ago by Nico
Modified:
6 years, 8 months ago
Reviewers:
Lei Zhang
CC:
chromium-reviews, cmp-cc_chromium.org, kjellander+cc_chromium.org, kerz_chromium, Michael Moss, Nicolas Sylvain, M-A Ruel
Visibility:
Public.

Description

compile.py: Don't set OFFICIAL_BUILD for --mode=official builds. As far as I can tell, no internal or external script looks at this env var (there are several that look at CHROME_BUILD_TYPE though, which OFFICIAL_BUILD is redundant with). This env var was added long ago (it was in initial.commit and has only been moved around since), so it's probably no longer needed. BUG=330631 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=261561

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M build/scripts/slave/compile.py View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 9 (0 generated)
Nico
6 years, 8 months ago (2014-04-03 18:50:16 UTC) #1
Nico
(+ a few more people who might know things about initial.commit.)
6 years, 8 months ago (2014-04-03 18:50:53 UTC) #2
mmoss
On 2014/04/03 18:50:53, Nico wrote: > (+ a few more people who might know things ...
6 years, 8 months ago (2014-04-03 19:02:27 UTC) #3
Lei Zhang
So OFFICIAL_BUILD will still remain defined, and it's just the environment variable that will no ...
6 years, 8 months ago (2014-04-03 22:21:36 UTC) #4
Nico
On 2014/04/03 22:21:36, Lei Zhang wrote: > So OFFICIAL_BUILD will still remain defined The preprocessor ...
6 years, 8 months ago (2014-04-03 22:25:10 UTC) #5
Lei Zhang
Unless someone finds a need for this and hollers, I'd say lgtm.
6 years, 8 months ago (2014-04-03 22:45:44 UTC) #6
Nico
The CQ bit was checked by thakis@chromium.org
6 years, 8 months ago (2014-04-03 22:53:33 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/217833006/1
6 years, 8 months ago (2014-04-03 22:54:46 UTC) #8
commit-bot: I haz the power
6 years, 8 months ago (2014-04-03 22:55:44 UTC) #9
Message was sent while issue was closed.
Change committed as 261561

Powered by Google App Engine
This is Rietveld 408576698