Closed Bug 852461 Opened 7 years ago Closed 7 years ago

Remove growl support from comm-central

Categories

(Thunderbird :: OS Integration, defect, blocker)

x86
macOS
defect
Not set
blocker

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 22.0

People

(Reporter: standard8, Assigned: florian)

References

Details

(Whiteboard: [relnote = from TB 22])

Bug 782211 has now landed, so we need to re-land this that was originally landed and then backed out.

+++ This bug was initially created as a clone of Bug #792793 +++

Bug 777409 removed growl support, and the comm-central tree is currently busted:
mailnews/base/src/nsMessengerOSXIntegration.mm:40:10: fatal error: 'nsINotificationsList.h' file not found
make[8]: *** [nsMessengerOSXIntegration.o] Error 1

The attached patch removes Growl specific code from mailnews, but keeps the call to nsIAlertsService, as that will use the new notification center on 10.8.
Depends on: 777409
Depends on: 792793
No longer depends on: 777409
Obviously forgot to close this bug.

I'd re-landed the patch with a couple of extra changes for packaging files:

http://hg.mozilla.org/comm-central/rev/03cd0b12d3ef
http://hg.mozilla.org/comm-central/rev/6770dd37290f
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 22.0
Wow. What a monumental step backwards for those of us who are using Growl 1.2.2 on Snow Leopard. Yesterday, SeaMonkey 2.17.1 offered up notifications in line with every other application. As of 2.19, notifications are in a "Hey, LOOK AT ME" balloon".

Lovely. Not. Surely, there could have been another way.
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.