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

Issue 2734423002: Revert of Add microdump_stackwalk target to chromium.android builders. (Closed)

Created:
3 years, 9 months ago by mikecase (-- gone --)
Modified:
3 years, 9 months ago
Reviewers:
bpastene, martiniss
CC:
chromium-reviews, agrieve
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Add microdump_stackwalk target to chromium.android builders. (patchset #1 id:1 of https://codereview.chromium.org/2724273003/ ) Reason for revert: So, needed to build dump_sym binary as well. Or the step fails. Building microdump_stalkwalker caused the step to run, but not having dump_sym causes it to fail Original issue's description: > Add microdump_stackwalk target to chromium.android builders. > > This will allow the testers to symbolize microdump crash logs. > > BUG=593106 > > Review-Url: https://codereview.chromium.org/2724273003 > Cr-Commit-Position: refs/heads/master@{#455516} > Committed: https://chromium.googlesource.com/chromium/src/+/37fc2c0e956c32dda663bc7c3e93bdf16779b35d TBR=bpastene@chromium.org,martiniss@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=593106 Review-Url: https://codereview.chromium.org/2734423002 Cr-Commit-Position: refs/heads/master@{#455545} Committed: https://chromium.googlesource.com/chromium/src/+/20fc04fae6337887e6b82d753cfac07325ed9820

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -10 lines) Patch
M testing/buildbot/chromium.android.json View 1 chunk +0 lines, -10 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
mikecase (-- gone --)
Created Revert of Add microdump_stackwalk target to chromium.android builders.
3 years, 9 months ago (2017-03-08 21:25:15 UTC) #2
mikecase (-- gone --)
3 years, 9 months ago (2017-03-08 21:25:29 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/2734423002/1
3 years, 9 months ago (2017-03-08 21:25:40 UTC) #4
commit-bot: I haz the power
3 years, 9 months ago (2017-03-08 21:29:24 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/20fc04fae6337887e6b82d753cfa...

Powered by Google App Engine
This is Rietveld 408576698