Closed Bug 389548 Opened 13 years ago Closed 13 years ago

fixup from cvs copy (airbag -> crashreporter)

Categories

(Toolkit :: Crash Reporting, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: ted, Assigned: ted)

References

Details

Attachments

(1 file)

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.
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)
Attachment #274652 - Flags: review?(benjamin) → review+
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?
Blocks: 391359
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?
Checked in.  I posted a warning note to m.d.{builds,platform} about the configure flag change.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.