[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] make org-notify support for macOS desktop notification
From: |
stardiviner |
Subject: |
Re: [PATCH] make org-notify support for macOS desktop notification |
Date: |
Tue, 6 Jul 2021 06:36:51 +0800 |
> On Jul 5, 2021, at 7:55 PM, Maxim Nikulin <manikulin@gmail.com> wrote:
>
> 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.
>
>
I indeed tried `notifications-notify`. And it does not work, reports error that
it needs dbus. PS. I used the Homebrew formulae version Emacs.
I considered the order of conditions. Because notifications and notify-send etc
requires dbus. So I guess only Linux supports that. So add system-type
detection will be better. WDYT?
- [PATCH] make org-notify support for macOS desktop notification, stardiviner, 2021/07/03
- Re: [PATCH] make org-notify support for macOS desktop notification, Maxim Nikulin, 2021/07/04
- Re: [PATCH] make org-notify support for macOS desktop notification, stardiviner, 2021/07/04
- [new patch] Re: [PATCH] make org-notify support for macOS desktop notification, stardiviner, 2021/07/05
- Re: [new patch] Re: [PATCH] make org-notify support for macOS desktop notification, Christian Hopps, 2021/07/06
- Re: [new patch] [PATCH] make org-notify support for macOS desktop notification, stardiviner, 2021/07/06
- Re: [new patch] [PATCH] make org-notify support for macOS desktop notification, Christian Hopps, 2021/07/06
- Re: [new patch] [PATCH] make org-notify support for macOS desktop notification, stardiviner, 2021/07/07
- Re: [new patch] [PATCH] make org-notify support for macOS desktop notification, Christian Hopps, 2021/07/08
- STOP this patch for now., stardiviner, 2021/07/08