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

Issue 64753002: Build mac and linux release builds with symbols and don't strip them by default (Closed)

Created:
7 years, 1 month ago by jochen (gone - plz use gerrit)
Modified:
7 years, 1 month ago
Reviewers:
Nico, Elliot Glaysher
CC:
chromium-reviews
Visibility:
Public.

Description

Build mac and linux release builds with symbols and don't strip them by default On Mac, we already build with symbols, but strip them. Building with symbols (a) matches what we do for official builds and (b) makes crashes on release builders more useful. BUG=315598 R=erg@chromium.org, thakis@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233887

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M build/common.gypi View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
jochen (gone - plz use gerrit)
plz review
7 years, 1 month ago (2013-11-07 18:55:36 UTC) #1
Nico
lgtm
7 years, 1 month ago (2013-11-07 21:49:47 UTC) #2
Elliot Glaysher
lgtm
7 years, 1 month ago (2013-11-07 21:55:24 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jochen@chromium.org/64753002/1
7 years, 1 month ago (2013-11-08 07:47:55 UTC) #4
commit-bot: I haz the power
Step "update" is always a major failure. Look at the try server FAQ for more ...
7 years, 1 month ago (2013-11-08 08:03:59 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jochen@chromium.org/64753002/1
7 years, 1 month ago (2013-11-08 08:56:14 UTC) #6
jochen (gone - plz use gerrit)
7 years, 1 month ago (2013-11-08 12:31:04 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 manually as r233887 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698