Remove growl support from comm-central

RESOLVED FIXED in Thunderbird 19.0

Status

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

People

(Reporter: florian, Assigned: florian)

Tracking

Trunk
Thunderbird 19.0
x86
Mac OS X
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 662957 [details] [diff] [review]
Patch

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.
Keywords: relnote
(Assignee)

Comment 1

5 years ago
Comment on attachment 662957 [details] [diff] [review]
Patch

Requesting review from a few people who are currently online. Whoever gets to it first can do it :).

The try server build is green: https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=ddd216b95ff2
Attachment #662957 - Flags: review?(mconley)
Attachment #662957 - Flags: review?(irving)
Comment on attachment 662957 [details] [diff] [review]
Patch

Removing r? because Irving is awesome and said he'd handle this.
Attachment #662957 - Flags: review?(mconley)
Comment on attachment 662957 [details] [diff] [review]
Patch

Review of attachment 662957 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me.
Attachment #662957 - Flags: review?(irving) → review+
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/comm-central/rev/30465284129e
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
Whiteboard: [relnote = from TB 18]

Updated

5 years ago
Blocks: 519053

Updated

5 years ago
Blocks: 519792

Updated

5 years ago
Blocks: 531244

Updated

5 years ago
Blocks: 538601

Updated

5 years ago
Blocks: 540956

Updated

5 years ago
Blocks: 515071

Updated

5 years ago
Blocks: 479034

Updated

5 years ago
Blocks: 510687

Updated

5 years ago
Blocks: 675583

Updated

5 years ago
Blocks: 476215

Updated

5 years ago
Blocks: 461144

Updated

5 years ago
Blocks: 741087

Updated

5 years ago
Blocks: 508270

Updated

5 years ago
Blocks: 570398
Depends on: 806254
Although this landed in TB 18, it was undone by bug 806254, updating flags.
Whiteboard: [relnote = from TB 18] → [relnote = from TB 19]
Target Milestone: Thunderbird 18.0 → Thunderbird 19.0
Blocks: 852461
Keywords: relnote
Whiteboard: [relnote = from TB 19]
You need to log in before you can comment on or make changes to this bug.