Closed
Bug 380906
Opened 17 years ago
Closed 17 years ago
make suite nsXULAppInfo ready for breakpad
Categories
(SeaMonkey :: Build Config, defect)
SeaMonkey
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kairo, Assigned: kairo)
References
Details
Attachments
(1 file, 1 obsolete file)
626 bytes,
patch
|
kairo
:
review+
|
Details | Diff | Splinter Review |
This is the suite version of / followup to bug 380421 - we should add a crashReporterURL to nsSuiteApp.cpp to get us prepared for breakpad integration. It's not entirely clear what crashReporterURL to use as policies of who can use the main socorro server of MoCo or what community projects like ours should/can use. I'll attach a preliminary patch using the MoCo server for now, so that it's tracked what we need to do to get this fixed.
Assignee | ||
Updated•17 years ago
|
Attachment #265020 -
Flags: review?(ted.mielczarek)
Comment 1•17 years ago
|
||
Comment on attachment 265020 [details] [diff] [review] patch, using MoCo socorro server This is reasonable, but I'd still like to get a firm ok from IT on community projects using this Socorro server. Would you mind holding off on this until I get that settled?
Attachment #265020 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 2•17 years ago
|
||
Bug 383167 bitrotted this patch by moving us from having the info inside nsSuiteApp.cpp to using application.ini instead. <KaiRo> luser: I guess you need to re-review a patch for bug 380906 now that the breakpad URL is going into application.ini instead os nsSuiteApp.cpp <luser> KaiRo: yeah, i realized that :) <luser> r=me to put it in the right place <KaiRo> luser: well, we only need ServerURL in there for now, right? and omit Enabled completely <luser> Enabled=0 <KaiRo> good, will add that
Attachment #265020 -
Attachment is obsolete: true
Attachment #269695 -
Flags: review+
Assignee | ||
Comment 3•17 years ago
|
||
I got word today that IT is OK with supporting community projects on the MoCo socorro server. I'll check this patch in as soon as I can watch tinderboxen.
Assignee | ||
Comment 4•17 years ago
|
||
Checked in, next step will be bug 386726
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•