Beginning on October 25th, 2016, Persona will no longer be an option for authentication on BMO. For more details see Persona Deprecated.
Last Comment Bug 792793 - Remove growl support from comm-central
: Remove growl support from comm-central
Product: Thunderbird
Classification: Client Software
Component: OS Integration (show other bugs)
: Trunk
: x86 Mac OS X
: -- blocker (vote)
: Thunderbird 19.0
Assigned To: Florian Quèze [:florian] [:flo]
Depends on: 806254
Blocks: 461144 476215 479034 508270 510687 515071 519053 519792 531244 538601 540956 570398 675583 741087 777409 852461
  Show dependency treegraph
Reported: 2012-09-20 04:09 PDT by Florian Quèze [:florian] [:flo]
Modified: 2013-04-10 00:32 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

Patch (7.50 KB, patch)
2012-09-20 04:09 PDT, Florian Quèze [:florian] [:flo]
irving: review+
Details | Diff | Splinter Review

Description Florian Quèze [:florian] [:flo] 2012-09-20 04:09:53 PDT
Created attachment 662957 [details] [diff] [review]

Bug 777409 removed growl support, and the comm-central tree is currently busted:
mailnews/base/src/ 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.
Comment 1 Florian Quèze [:florian] [:flo] 2012-09-20 07:20:22 PDT
Comment on attachment 662957 [details] [diff] [review]

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

The try server build is green:
Comment 2 Mike Conley (:mconley) - (high latency on reviews and needinfos) 2012-09-20 07:20:52 PDT
Comment on attachment 662957 [details] [diff] [review]

Removing r? because Irving is awesome and said he'd handle this.
Comment 3 :Irving Reid (No longer working on Firefox) 2012-09-20 08:00:38 PDT
Comment on attachment 662957 [details] [diff] [review]

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

Looks good to me.
Comment 4 Florian Quèze [:florian] [:flo] 2012-09-20 08:24:59 PDT
Comment 5 Mark Banner (:standard8) 2012-11-19 04:09:00 PST
Although this landed in TB 18, it was undone by bug 806254, updating flags.

Note You need to log in before you can comment on or make changes to this bug.