-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does it works with MacOS X? #2
Comments
It should be easy to make it work on MacOS X. Unfortunately I do not have access to a MacOS X machine, so I might not be able to provide a 'ready-to-use' snippet.
Looking at alert's documentation it seems (mu4e-alert-set-default-style 'notifier) Alternately if you have growl installed you can instruct (mu4e-alert-set-default-style 'growl) Please try the above and report back if works (or even if it does not work) Hope the above helps! |
@zilongshanren Hi, just wanted to know if any of the above methods worked for you |
Oops! I think I closed this by mistake |
That method works for moi |
Great! I guess I will update the README with instructions for OSX and close this issue. Thanks @justmytwospence |
@iqbalansari |
Closing this, I have added some instructions for enabling desktop notifications. to the README. Feel free to reopen if you face any issues or feel the documentation is not clear. Thanks |
Hi. I have installed terminal-notifier on OS X and I still can't get the alerts. My .emacs file has the following lines:
And it doesn't seem to call terminal-notifier (same issue with growl). I have verified that the notifier works and there should be no issue. I do see the mode line updated but that is all. Thanks in advance. |
Most probably this is because Emacs cannot find |
Things that I need to check. I verified that the binary was in my path but it is not found by Emacs. And there is no value for I'll go ahead and make sure that the binary is found and try again. The rest of this works pretty well, as the mode line shows me when I have mail. |
That explains the lack of notifications. For testing quickly you can manually set |
I will have to check the exec path to make sure it is finding it. I'll let you know. Jason K Bellew On May 3, 2016, 14:55 -0400, Iqbal Ansarinotifications@github.com, wrote:
|
And it worked thanks. Once I set the command manually everything is coming up roses. |
When switching contexts, wrong list of arguments passed to `advice-add`'s inner `apply` function. This commit fixes it. Closes iqbalansari#2.
I noticed that the screenshot is made under Linux, would it be possible to support MacOS X?
Thanks.
The text was updated successfully, but these errors were encountered: