Closed
Bug 389548
Opened 17 years ago
Closed 17 years ago
fixup from cvs copy (airbag -> crashreporter)
Categories
(Toolkit :: Crash Reporting, defect)
Toolkit
Crash Reporting
Tracking
()
RESOLVED
FIXED
People
(Reporter: ted, Assigned: ted)
References
Details
Attachments
(1 file)
29.94 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
In bug 371069 we copied toolkit/airbag to toolkit/crashreporter (and some other renames in the process). We should now do some cleanup to switch completely to that dir, and cvs remove toolkit/airbag.
Assignee | ||
Comment 1•17 years ago
|
||
This switches everything to build from toolkit/crashreporter, and gets rid of all the remaining instances of airbag from the source. That includes changing the configure flag to --disable-crashreporter. I'll post about that on the newsgroups before checking this in. Not included in this patch is a cvs remove of toolkit/airbag, but I intend to do that as well.
Attachment #274652 -
Flags: review?(benjamin)
Updated•17 years ago
|
Attachment #274652 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 2•17 years ago
|
||
Comment on attachment 274652 [details] [diff] [review] switch to using toolkit/crashreporter, do some renaming Requesting approval1.9, this patch should be a no-op, just switches from using toolkit/airbag to toolkit/crashreporter, which was done in a CVS copy. I'd like to CVS remove toolkit/airbag.
Attachment #274652 -
Flags: approval1.9?
Assignee | ||
Comment 3•17 years ago
|
||
Comment on attachment 274652 [details] [diff] [review] switch to using toolkit/crashreporter, do some renaming bsmedberg tells me we don't need approval in toolkit right now.
Attachment #274652 -
Flags: approval1.9?
Assignee | ||
Comment 4•17 years ago
|
||
Checked in. I posted a warning note to m.d.{builds,platform} about the configure flag change.
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
•