Closed
Bug 554021
Opened 15 years ago
Closed 15 years ago
enable compiling Breakpad on Linux/x86-64 by default
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(status1.9.2 .5-fixed)
RESOLVED
FIXED
mozilla1.9.3a5
Tracking | Status | |
---|---|---|
status1.9.2 | --- | .5-fixed |
People
(Reporter: ted, Assigned: ted)
References
Details
(Whiteboard: [first patch was fixed in mozilla1.9.3a4])
Attachments
(1 file, 1 obsolete file)
1.08 KB,
patch
|
jimb
:
review+
pavlov
:
approval1.9.2.5+
|
Details | Diff | Splinter Review |
bug 548113 landed some symbol dumper changes as well as client changes to support Linux/x86-64 and Linux/ARM. We should start compiling the client code there by default. We'll need to take a minidump_stackwalk update on the server first, since the symbols produced will include new STACK CFI records that the current processor won't understand.
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → ted.mielczarek
Assignee | ||
Updated•15 years ago
|
Blocks: support-L64
Assignee | ||
Comment 1•15 years ago
|
||
This broke on the try server maemo builds because scratchbox doesn't have libcurl-dev. :-/
Assignee | ||
Comment 2•15 years ago
|
||
I split this patch into two parts, since Linux/ARM needs a build slave update. This one just cleans up the block where we select which targets get Breakpad enabled by default, and adds Linux/x86-64 to that list.
Attachment #435133 -
Flags: review?(jim)
Assignee | ||
Updated•15 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•15 years ago
|
||
And this enables it for Linux/ARM.
Attachment #435134 -
Flags: review?(jim)
Assignee | ||
Comment 4•15 years ago
|
||
After fixing a few minor things, I'm still hitting an internal compiler error compiling in Scratchbox:
/home/cltbld/mozilla-central/toolkit/crashreporter/google-breakpad/src/common/linux/language.cc:76: internal compiler error: in reference_to_unused, at dwarf2out.c:10094
(irony: gcc erroring in its dwarf-producing code compiling our dwarf-consuming code.)
Comment 5•15 years ago
|
||
Comment on attachment 435133 [details] [diff] [review]
enable Breakpad on Linux/x86-64
[Checkin: Comment 8]
>+i?86-apple-darwin*|powerpc-apple-darwin*)
>+ MOZ_CRASHREPORTER=1
Can we generate PowerPC crashes? I didn't find the code to do so...
Otherwise, looks good to me.
Attachment #435133 -
Flags: review?(jim) → review+
Comment 6•15 years ago
|
||
If we were to delta-reduce that source file to find out what's causing the ICE, I wonder if that would suggest a way to work around it?
Comment 7•15 years ago
|
||
Comment on attachment 435134 [details] [diff] [review]
enable Breakpad on Linux/ARM
We obviously can't land this, but the patch itself looks good to me.
Attachment #435134 -
Flags: review?(jim) → review+
Assignee | ||
Comment 8•15 years ago
|
||
Comment on attachment 435133 [details] [diff] [review]
enable Breakpad on Linux/x86-64
[Checkin: Comment 8]
Pushed to m-c:
http://hg.mozilla.org/mozilla-central/rev/74ffccda23b8
Maybe I should split ARM off into a separate bug.
Comment 9•15 years ago
|
||
This caused build failures on debug linux x64 with
ac_add_options --enable-debug="-gstabs+"
export CFLAGS="-gstabs+"
export CXXFLAGS="-gstabs+"
firefox-debug/dist/lib/libxpcomglue_s.a(nsGenericFactory.o): In function `nsGenericFactory::GetClassID(nsID**)':
mozilla/xpcom/glue/nsGenericFactory.cpp:143: undefined reference to `nsMemory::Clone(void const*, unsigned long)'
/usr/bin/ld: mozilla/firefox-debug/dist/lib/libxpcomglue_s.a(nsGenericFactory.o): relocation R_X86_64_PC32 against `nsMemory::Clone(void const*, unsigned long)' can not be used when making a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: Bad value
opt builds fine with
export CFLAGS="-gstabs+"
export CXXFLAGS="-gstabs+"
What are the appropriate mozconfig settings for debug 64bit linux now?
Assignee | ||
Comment 10•15 years ago
|
||
Er, I dunno, the tinderbox Linux64 debug builds are just fine using this mozconfig:
http://hg.mozilla.org/build/buildbot-configs/file/3ed8c28d1dcc/mozilla2/linux64/mozilla-central/debug/mozconfig
Are you sure it was this changeset? That error doesn't look like anything related to this.
FWIW, you can switch to -gdwarf-2 now if you want.
Assignee | ||
Comment 11•15 years ago
|
||
I split Linux/ARM off to bug 555674. I'll reattach the ARM patch there.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
No longer depends on: 554999
Resolution: --- → FIXED
Summary: enable compiling Breakpad on Linux/x86-64 and Linux/ARM by default → enable compiling Breakpad on Linux/x86-64 by default
Assignee | ||
Updated•15 years ago
|
Target Milestone: --- → mozilla1.9.3a4
Assignee | ||
Comment 12•15 years ago
|
||
Comment on attachment 435134 [details] [diff] [review]
enable Breakpad on Linux/ARM
This patch moved to bug 435575.
Attachment #435134 -
Attachment is obsolete: true
Assignee | ||
Comment 13•15 years ago
|
||
Comment 14•15 years ago
|
||
(In reply to comment #10)
> Er, I dunno, the tinderbox Linux64 debug builds are just fine using this
> mozconfig:
> http://hg.mozilla.org/build/buildbot-configs/file/3ed8c28d1dcc/mozilla2/linux64/mozilla-central/debug/mozconfig
>
I'll check it out.
> Are you sure it was this changeset? That error doesn't look like anything
> related to this.
Yeah, bisection fingered this patch.
>
> FWIW, you can switch to -gdwarf-2 now if you want.
cool.
Comment 15•15 years ago
|
||
syncing the mozconfigs with buildbot's works. I think my problem was a use of ac_cv_visibility_pragma=no instead of ac_cv_have_visibility_class_bug=yes. Thanks ted.
Comment 16•15 years ago
|
||
Comment on attachment 435133 [details] [diff] [review]
enable Breakpad on Linux/x86-64
[Checkin: Comment 8]
could we have that on the branch please as all the other crashreporter changes landed there already?
Attachment #435133 -
Flags: approval1.9.2.4?
Assignee | ||
Comment 17•15 years ago
|
||
FWIW, this is a pretty trivial change and only impacts Linux/x86-64, which I don't think we're officially shipping on 1.9.2 (but Linux distros are).
Updated•15 years ago
|
Attachment #435133 -
Attachment description: enable Breakpad on Linux/x86-64 → enable Breakpad on Linux/x86-64
[Checkin: Comment 8]
Comment 18•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/9707858ee63b
enable compiling Breakpad on Linux/ARM by default
Whiteboard: [first patch was fixed in mozilla1.9.3a4]
Target Milestone: mozilla1.9.3a4 → mozilla1.9.3a5
Assignee | ||
Comment 19•15 years ago
|
||
Uh, I split that out into a new bug, I don't think you need to note that here.
Assignee | ||
Comment 20•15 years ago
|
||
Comment on attachment 435133 [details] [diff] [review]
enable Breakpad on Linux/x86-64
[Checkin: Comment 8]
This isn't going to make 1.9.2.4, but we can try for 1.9.2.5.
Attachment #435133 -
Flags: approval1.9.2.4? → approval1.9.2.5?
Assignee | ||
Comment 21•15 years ago
|
||
Drivers: this patch has essentially zero impact on Firefox builds, since we don't ship Linux x86-64 releases from 1.9.2, but would make Linux distros happy. (Also the patch in bug 555674 is written on top of this patch.)
Updated•15 years ago
|
Attachment #435133 -
Flags: approval1.9.2.5? → approval1.9.2.5+
Assignee | ||
Comment 22•15 years ago
|
||
Pushed to 1.9.2:
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/3b3d0d34e9c5
status1.9.2:
--- → .5-fixed
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•