|
From: | Maxim Nikulin |
Subject: | Re: [PATCH] make org-notify support for macOS desktop notification |
Date: | Mon, 5 Jul 2021 18:55:24 +0700 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
On 05/07/2021 10:50, stardiviner wrote:
I updated the patch, I found the package `osx-lib` contains solution. So I removed the directly osascript process invocation.
I have no objections any more. On the other hand I have no access to macOS, so I have not tested this patch. Feel free to ignore comments from this message, they are mostly matter of taste.
I expect that a simple script "notify-send" may allow to avoid modification of code. Something like (untested, unsure concerning "quoted form of ...")
#!/usr/bin/env osascript display notification (item 1 of argv)However if osx-lib in is installed automatically, it may be more convenient. Unsure if some of currently supported linux distributions have notify-send that can not handle title as the first argument.
- ((fboundp 'notifications-notify) + ((and (eq system-type 'gnu/linux) (fboundp 'notifications-notify))
Does it mean that `notifications-notify' is bound but it does not work on macOS? If so, maybe it is better to put new clause for 'darwin above and to drop 'gnu/linux here. From my point of view, it is preferable to avoid additional requirement for `notifications-notify'. If someone will create a feature request for `notifications-notify' for macOS, it will just work without installing of additional packages as soon as such feature is implemented.
[Prev in Thread] | Current Thread | [Next in Thread] |