Closed
Bug 791869
Opened 12 years ago
Closed 12 years ago
Remove services/notifications
Categories
(Cloud Services Graveyard :: Server: WebPush, defect)
Cloud Services Graveyard
Server: WebPush
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla20
People
(Reporter: gps, Assigned: gps)
References
Details
Attachments
(1 file)
|
10.70 KB,
patch
|
rnewman
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #754062 +++
I'm pretty sure that services/notifications isn't going anywhere and can safely be removed from the tree. If we ever need it, it's there in the commit history. It's just a skeleton for the XPCOM foo anyway.
|
Assignee | |
Comment 1•12 years ago
|
||
Rubber stamp, please.
|
||
Updated•12 years ago
|
Attachment #698451 -
Flags: review?(rnewman) → review+
|
|
Assignee | |
Comment 2•12 years ago
|
||
Whiteboard: [fixed in services]
|
|
Assignee | |
Comment 3•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in services]
Target Milestone: --- → mozilla20
|
||
Comment 4•12 years ago
|
||
Should this one http://mxr.mozilla.org/comm-central/source/mozilla/browser/installer/package-manifest.in#590
590 #ifdef MOZ_SERVICES_NOTIFICATIONS
591 @BINPATH@/defaults/pref/services-notifications.js
592 #endif
also get removed?
|
|
Assignee | |
Comment 5•12 years ago
|
||
Yup. There is also a reference in configure.in that should be killed.
|
||
Updated•2 years ago
|
Product: Cloud Services → Cloud Services Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•