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

Issue 10695024: Adding gdb-add-index script from Redhat's gdb rpm. (Closed)

Created:
8 years, 5 months ago by awong
Modified:
8 years, 5 months ago
CC:
chromium-reviews, Sami
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Adding gdb-add-index script from Redhat's gdb rpm. For linux users who are going to debug chromium, running gdb-add-index on the binary first will significantly speed up debugger start time. The originating RPM is gdb-7.4.50.20120603-1.1.x86_64.rpm. The script is just a bash script version of the instructions in the gdb man page. BUG=none TEST=ran it on a statically linked browser_tests locally. verfied .gdb_index section exists and that gdb does startup faster (from ~20 secs to ~3 seconds) Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=144644

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -0 lines) Patch
A build/gdb-add-index View 1 chunk +30 lines, -0 lines 1 comment Download

Messages

Total messages: 6 (0 generated)
awong
Moved the file location.
8 years, 5 months ago (2012-06-28 01:18:31 UTC) #1
nsylvain
lgtm
8 years, 5 months ago (2012-06-28 01:24:19 UTC) #2
cmp
lgtm with nit https://chromiumcodereview.appspot.com/10695024/diff/1/build/gdb-add-index File build/gdb-add-index (right): https://chromiumcodereview.appspot.com/10695024/diff/1/build/gdb-add-index#newcode26 build/gdb-add-index:26: objcopy --add-section .gdb_index="${file}.gdb-index" --set-section-flags .gdb_index=readonly "$file" ...
8 years, 5 months ago (2012-06-28 01:25:07 UTC) #3
awong
I snagged this file verbatim from the redhat sources so I'd prefer not to touch ...
8 years, 5 months ago (2012-06-28 01:26:00 UTC) #4
cmp
lgtm then
8 years, 5 months ago (2012-06-28 01:26:52 UTC) #5
Steve Block
8 years, 5 months ago (2012-07-06 09:58:17 UTC) #6
Hi Albert,

This is third-party code, so should be in a 'third_party' directory. See
http://www.chromium.org/developers/adding-3rd-party-libraries. Perhaps
build/third_party/redhat/gdb-add-index?

This is particularly important in this case, as the file has a GPL v3 license,
which can be problematic.

Powered by Google App Engine
This is Rietveld 408576698