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

Issue 16151006: Add scripts for using cygprofile to repository (Closed)

Created:
7 years, 6 months ago by aberent
Modified:
7 years, 6 months ago
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Add scripts for using cygprofile to repository These scripts, which are used to create a link order file for Linux based builds (Linux, Android, ChromeOS). This commit simply moves them from a Google private web page where they have lived for the last couple of years. The scripts are: mergetraces.py - merge multiple cygprofile traces into a single trace symbolise.py - convert traces into an order file with symbols patch_orderfile.py - complete the order file with symbols from Chrome. I will seperately add a page on creating an order file to the Chromium web site. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203143

Patch Set 1 #

Patch Set 2 : Cygprofile scripts - reformat to Chromium standard #

Unified diffs Side-by-side diffs Delta from patch set Stats (+554 lines, -0 lines) Patch
A tools/cygprofile/mergetraces.py View 1 chunk +186 lines, -0 lines 0 comments Download
A tools/cygprofile/patch_orderfile.py View 1 1 chunk +117 lines, -0 lines 0 comments Download
A tools/cygprofile/symbolize.py View 1 chunk +251 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
aberent
7 years, 6 months ago (2013-05-30 10:43:51 UTC) #1
digit
lgtm :)
7 years, 6 months ago (2013-05-30 11:25:40 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/aberent@chromium.org/16151006/1
7 years, 6 months ago (2013-05-30 11:27:07 UTC) #3
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=5862
7 years, 6 months ago (2013-05-30 11:37:16 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/aberent@chromium.org/16151006/8001
7 years, 6 months ago (2013-05-30 11:53:36 UTC) #5
commit-bot: I haz the power
7 years, 6 months ago (2013-05-30 14:51:57 UTC) #6
Message was sent while issue was closed.
Change committed as 203143

Powered by Google App Engine
This is Rietveld 408576698