Closed
Bug 630655
Opened 13 years ago
Closed 13 years ago
Remove support for building without MOZ_CRASHREPORTER
Categories
(Toolkit :: Crash Reporting, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: jrmuizel, Unassigned)
Details
Are there any compelling reasons for keeping around the ability to build without crashreporter? It would be nice to rip out a bunch #ifdef MOZ_CRASHREPORTER.
Comment 1•13 years ago
|
||
Most Linux distros disable the crash reporter for their builds.
Reporter | ||
Comment 2•13 years ago
|
||
(In reply to comment #1) > Most Linux distros disable the crash reporter for their builds. The code or just the feature? Do you know why they disable crash reporter?
Comment 3•13 years ago
|
||
Part of it is that we don't have symbols for their builds. I suspect there are also distros that see the crash reporter as a privacy issue.
Reporter | ||
Comment 4•13 years ago
|
||
(In reply to comment #3) > Part of it is that we don't have symbols for their builds. I suspect there are > also distros that see the crash reporter as a privacy issue. Even without MOZ_CRASHREPORTER, crash reporter can still be disable at runtime.
Comment 5•13 years ago
|
||
(In reply to comment #1) > Most Linux distros disable the crash reporter for their builds. This is no longer true, Fedora, Ubuntu and OpenSuSE are all uploading symbols to our symbol server. That being said, the most compelling reason for MOZ_CRASHREPORTER is that it only supports a very specific set of platforms, and for most people doing ports, it's not a high priority, so you'd be raising the bar for a port with very little gain for the person doing the work.
Reporter | ||
Comment 6•13 years ago
|
||
(In reply to comment #5) > That being said, the most compelling reason for MOZ_CRASHREPORTER is that it > only supports a very specific set of platforms, and for most people doing > ports, it's not a high priority, so you'd be raising the bar for a port with > very little gain for the person doing the work. That makes sense to me.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Comment 7•13 years ago
|
||
I doubt it's worth the work, but another option would be to always build the platform-agnostic parts and/or perform the various runtime annotations.
Reporter | ||
Comment 8•13 years ago
|
||
(In reply to comment #7) > I doubt it's worth the work, but another option would be to always build the > platform-agnostic parts and/or perform the various runtime annotations. This is my plan if I decide that it matters enough.
Comment 9•13 years ago
|
||
I would support that, it wouldn't be much work to change MOZ_CRASHREPORTER to mean "has a working Breakpad implementation", and always build the code that supplies the crashreporter APIs (like AnnotateCrashReport).
You need to log in
before you can comment on or make changes to this bug.
Description
•