Closed
Bug 554024
Opened 14 years ago
Closed 14 years ago
switch mozilla-central Linux builds to use -gdwarf-2 instead of -gstabs
Categories
(Release Engineering :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ted, Assigned: ted)
References
Details
Attachments
(1 file)
18.42 KB,
patch
|
catlee
:
review+
ted
:
checked-in+
|
Details | Diff | Splinter Review |
After the Breakpad update in bug 548113, we should now be able to dump DWARF symbols on Linux. We should switch from stabs because we'll get better symbols in some cases, and it will also allow us to use -fomit-frame-pointer.
Updated•14 years ago
|
Assignee: nobody → ccooper
Priority: -- → P3
Assignee | ||
Comment 1•14 years ago
|
||
Stealing this bug, another sed-generated patch, via: find . -name "mozconfig" | xargs grep -l MOZ_DEBUG_SYMBOLS | grep -v "1.9" | grep -v lorentz | grep -v "seamonkey/" | grep -v thunderbird | grep linux | xargs sed -i -e 's/stabs[+]/dwarf-2/'
Assignee: ccooper → ted.mielczarek
Status: NEW → ASSIGNED
Attachment #434599 -
Flags: review?(catlee)
Updated•14 years ago
|
Attachment #434599 -
Flags: review?(catlee) → review+
Comment 2•14 years ago
|
||
By the way, there is also -gstabs, in js/src/configure.in.
Assignee | ||
Comment 3•14 years ago
|
||
Comment on attachment 434599 [details] [diff] [review] s/stabs[+]/dwarf-2/ Landed: http://hg.mozilla.org/build/buildbot-configs/rev/78147c4334ba
Attachment #434599 -
Flags: checked-in+
Assignee | ||
Comment 4•14 years ago
|
||
(In reply to comment #2) > By the way, there is also -gstabs, in js/src/configure.in. That's in some ancient QNX block, not relevant for this discussion.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 5•14 years ago
|
||
linux64 symlinks more branches back to mozilla-central than linux does, so there were changes to mozilla-1.9.1 and 1.9.2 which may not have been intended. But we don't build symbols on linux64 for those branches ?
Assignee | ||
Comment 6•14 years ago
|
||
I didn't know that, and it was unintentional, but yeah, it's harmless since we don't have Breakpad support for Linux64 on anything but trunk currently. (Thanks for the heads up!)
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•