Closed
Bug 590492
Opened 16 years ago
Closed 15 years ago
hg remove reporter!
Categories
(Other Applications Graveyard :: Reporter, defect)
Other Applications Graveyard
Reporter
Tracking
(blocking2.0 -)
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| blocking2.0 | --- | - |
People
(Reporter: Pike, Assigned: Pike)
References
Details
(Whiteboard: [strings][removal only])
Attachments
(1 file)
|
61.47 KB,
patch
|
raccettura
:
review+
benjamin
:
review+
beltzner
:
approval2.0+
|
Details | Diff | Splinter Review |
Firefox isn't shipping reporter anymore (bug 572695), and SeaMonkey may stop doing so, too (bug 590487).
If there are no more apps shipping reporter, should we hg remove it?
| Assignee | ||
Comment 1•16 years ago
|
||
Robert, what's your take on this?
Comment 2•16 years ago
|
||
Can remove, can let it sit for "archival purposes" since it's not hooked up to a build (though in a VCS that's redundant)... I'm ambivalent.
It outlived it's usefulness.
Comment 3•16 years ago
|
||
OK, it has no users any more, both Firefox and SeaMonkey removed their usages, this can go ahead.
| Assignee | ||
Comment 4•16 years ago
|
||
OK, let's do this.
I'd love to get this in before string freeze and branch so that we don't carry this further along into the future than necessary.
blocking2.0: --- → ?
Summary: hg remove reporter? → hg remove reporter!
| Assignee | ||
Comment 5•16 years ago
|
||
I checked configure.in and the various confvars.sh, there seems to be no further mentioning of reporter, so I think a plain hg remove extensions/reporter will do the trick.
Attachment #470806 -
Flags: review?(robert)
Comment 6•16 years ago
|
||
Comment on attachment 470806 [details] [diff] [review]
hg rm extensions/reporter
looks/sounds right to me. r=raccettura
Attachment #470806 -
Flags: review?(robert) → review+
| Assignee | ||
Updated•16 years ago
|
Whiteboard: [strings][removal only]
| Assignee | ||
Comment 7•16 years ago
|
||
Comment on attachment 470806 [details] [diff] [review]
hg rm extensions/reporter
Benjamin, can you cross check this?
Attachment #470806 -
Flags: review?(benjamin)
Updated•16 years ago
|
Attachment #470806 -
Flags: review?(benjamin) → review+
| Assignee | ||
Updated•15 years ago
|
Assignee: robert → l10n
Comment 8•15 years ago
|
||
Comment on attachment 470806 [details] [diff] [review]
hg rm extensions/reporter
a=beltzner
Attachment #470806 -
Flags: approval2.0+
Updated•15 years ago
|
blocking2.0: ? → -
Keywords: checkin-needed
| Assignee | ||
Comment 9•15 years ago
|
||
Updated•7 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•