Remove growl support from comm-central

RESOLVED FIXED in Thunderbird 22.0

Status

Thunderbird
OS Integration
--
blocker
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: standard8, Assigned: florian)

Tracking

Trunk
Thunderbird 22.0
x86
Mac OS X
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [relnote = from TB 22])

(Reporter)

Description

4 years ago
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.
(Reporter)

Updated

4 years ago
Depends on: 777409
(Reporter)

Updated

4 years ago
Depends on: 792793
No longer depends on: 777409
(Reporter)

Comment 1

4 years ago
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
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 22.0

Comment 2

4 years ago
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.
(Reporter)

Updated

3 years ago
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.