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

Issue 2301213002: Don't allow official component builds (Closed)

Created:
4 years, 3 months ago by brucedawson
Modified:
4 years, 3 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Don't allow official component builds Official component builds generally don't work, and they are logically contradictory, so it is better to prevent them with an assert in order to save developers from this mistake. Committed: https://crrev.com/c8d6d6517d2b5745814a3bd6c5fdf29e3ffd1671 Cr-Commit-Position: refs/heads/master@{#416321}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M BUILD.gn View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
brucedawson
An external developer hit this and reported it on chromium-dev. Any reason not to prohibit ...
4 years, 3 months ago (2016-09-02 01:25:07 UTC) #2
Dirk Pranke
I think this is probably fine, given that official builds are usually using WPO and ...
4 years, 3 months ago (2016-09-02 04:11:50 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2301213002/1
4 years, 3 months ago (2016-09-02 18:06:29 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-02 19:43:43 UTC) #6
commit-bot: I haz the power
4 years, 3 months ago (2016-09-02 19:45:41 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/c8d6d6517d2b5745814a3bd6c5fdf29e3ffd1671
Cr-Commit-Position: refs/heads/master@{#416321}

Powered by Google App Engine
This is Rietveld 408576698