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

Issue 387693002: Port r282246 to gn. (Closed)

Created:
6 years, 5 months ago by Nico
Modified:
6 years, 5 months ago
Reviewers:
brettw, awong
CC:
chromium-reviews
Project:
chromium
Visibility:
Public.

Description

Port r282246 to gn. Temporarily disable -Werror on linux, since the gn build isn't yet warning-free with clang. Since gn doesn't use -Werror for all targets yet anyways (http://crbug.com/393046) that seems acceptable. BUG=360311, 393046 R=brettw@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284372

Patch Set 1 #

Patch Set 2 : . #

Patch Set 3 : no werror #

Patch Set 4 : actually #

Patch Set 5 : nope #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -2 lines) Patch
M build/config/BUILDCONFIG.gn View 1 2 1 chunk +1 line, -1 line 0 comments Download
M build/config/compiler/BUILD.gn View 1 2 4 3 chunks +7 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Nico
6 years, 5 months ago (2014-07-10 20:45:34 UTC) #1
brettw
lgtm
6 years, 5 months ago (2014-07-10 20:58:51 UTC) #2
Nico
6 years, 5 months ago (2014-07-20 19:09:02 UTC) #3
Message was sent while issue was closed.
Committed patchset #5 manually as r284372 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698