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

Issue 808703005: Run gn check for an android build on presubmit (Closed)

Created:
6 years ago by qsr
Modified:
6 years ago
Reviewers:
sky
CC:
mojo-reviews_chromium.org, qsr+mojo_chromium.org, viettrungluu+watch_chromium.org, yzshen+watch_chromium.org, abarth-chromium, Aaron Boodman, darin (slow to review), ben+mojo_chromium.org
Base URL:
https://github.com/domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Run gn check for an android build on presubmit Also, fix gn check errors. R=sky@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/8f4371ea5a1bfc490e39c4c25af8f70b8f74e450

Patch Set 1 #

Patch Set 2 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -33 lines) Patch
M PRESUBMIT.py View 1 chunk +29 lines, -24 lines 0 comments Download
M examples/device_name/BUILD.gn View 1 chunk +1 line, -0 lines 0 comments Download
M mojo/edk/embedder/BUILD.gn View 3 chunks +7 lines, -9 lines 0 comments Download
M mojo/services/network/BUILD.gn View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
qsr
6 years ago (2014-12-17 10:25:24 UTC) #1
sky
LGTM
6 years ago (2014-12-17 16:27:16 UTC) #2
qsr
6 years ago (2014-12-17 16:28:30 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
8f4371ea5a1bfc490e39c4c25af8f70b8f74e450 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698