|
|
DescriptionEnable -Werror for links in chromecast builds.
BUG=669076
Committed: https://crrev.com/21b2e5f86e4ac1ecd5bcc3da195cf2f7fd427eff
Cr-Commit-Position: refs/heads/master@{#434755}
Patch Set 1 #Patch Set 2 : . #Patch Set 3 : rebase #
Messages
Total messages: 22 (11 generated)
The CQ bit was checked by thakis@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
thakis@chromium.org changed reviewers: + antz@chromium.org, bcf@chromium.org, slan@chromium.org
Not sure who's the best reviewer here (no OWNERS file in build/config/chromecast)
lgtm Nico, do you remember why we disabled this on Cast builds in the first place?
On 2016/11/28 18:00:00, slan wrote: > lgtm > > Nico, do you remember why we disabled this on Cast builds in the first place? -Werror for links? We just never had it enabled anywhere, mostly due to oversight. I turned it on last week but disabled it on cast and asan since the trybots didn't come green there and I figured I'd investigate and fix those in a follow-up. This is that follow-up. (Also see bug)
The CQ bit was unchecked by thakis@chromium.org
The CQ bit was checked by thakis@chromium.org
Oh, and thanks for the fast review :-)
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was checked by thakis@chromium.org
The patchset sent to the CQ was uploaded after l-g-t-m from slan@chromium.org Link to the patchset: https://codereview.chromium.org/2532193002/#ps40001 (title: "rebase")
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
lgtm
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
The CQ bit was checked by thakis@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch. Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480363333854920, "parent_rev": "4598a3eedcc8ce60daeaf782849d41bb8f728a85", "commit_rev": "bd983934046837009bef85d14342d97c09b116d2"}
Message was sent while issue was closed.
Committed patchset #3 (id:40001)
Message was sent while issue was closed.
Description was changed from ========== Enable -Werror for links in chromecast builds. BUG=669076 ========== to ========== Enable -Werror for links in chromecast builds. BUG=669076 Committed: https://crrev.com/21b2e5f86e4ac1ecd5bcc3da195cf2f7fd427eff Cr-Commit-Position: refs/heads/master@{#434755} ==========
Message was sent while issue was closed.
Patchset 3 (id:??) landed as https://crrev.com/21b2e5f86e4ac1ecd5bcc3da195cf2f7fd427eff Cr-Commit-Position: refs/heads/master@{#434755} |