Re: STOP this patch for now.
stardiviner writes: On Jul 8, 2021, at 4:59 PM, Christian Hopps wrote: It may eventually be incorporated into the very popular emacs-mac port (railwaycat tap in homebrew); however, it will probably not be incorporated into the nextstep/emacs main code. I started looking at doing a version for the mainline code, but it’s hard to get motivated b/c using that version of emacs on OS X is a pretty sub-par experience. Thanks for your work on this support. I found upstream is less active. Don’t know when will be merged. I only commented on this b/c I think you might are disabling notifications-notify which work great with my code changes, and using something else if you see Darwin OS, and that will break my native “Just Works” support for notifications, which again may end up on many peoples machines. I would ask that the patch be modified in a way that didn’t break native support if present before it was accepted. Also as you can see by the multiple patches you’ve submitted there’s really no good answer for an external notifier, so whatever you pick is probably going to be wrong for someone I guess. Yes, this troubled my too. Currently no good solution. I will stop this patch for now. Wait for upstream emacs-mac port support. Hope it will be arrived in at leas half of year. You could add a comment to the pull-request in support of merging. :) https://bitbucket.org/mituharu/emacs-mac/pull-requests/10 FWIW, if you are currently using the railwaycat tap in homebrew you can just switch to my tap and you'll get the support right now. It should be just as easy to install. Thanks, Chris. If this patch is going to be accepted I would ask that it 1) be conditional (disable-able with a variable) 2) do the check for the custom installed external notifier and if not present then fallback to using the emacs supplied notifications-notify 3) not restrict notifications-notify to gnu/linux only. That way people that have already developed solutions for this won’t have them broken. Thanks, Chris. On Jul 7, 2021, at 8:00 PM, stardiviner wrote: Hi Chris, thanks for your work. I have a question, will your patch of notification code be merged to upstream? If yes, I think my patch will be not necessary. If no, then I think add a my workaround for macOS is considerable. On Jul 7, 2021, at 2:23 AM, Christian Hopps < cho...@chopps.org> wrote: It supports imagemagick (specify —with-imagemagick), and it includes svg by default, I simply forked the railwaycat version and added the native notification code. Thanks, Chris. On Jul 6, 2021, at 11:30 AM, stardiviner < numbch...@gmail.com> wrote: Thanks for your suggestion. Does your Emacs build supports imagemagick image view and svg feature support? Because company-mode now have built-in icons support. This is the reason that I switch from https: //emacsformacosx.com/ to Homebrew cask Emacs version. On Jul 6, 2021, at 12:21 PM, Christian Hopps < cho...@chopps.org> wrote: Hi, Please consider: I added full native notification support to the popular OS X Emacs build available in homebrew. This supports rewrites notifications-notify defun to use the native code rather than dbus, and so everything "Just Works". Info can be found here: https://github.com/choppsv1/homebrew-emacsmacport Thanks, Chris. stardiviner writes: Here is the new patch which invokes notifications though Emacs built-in API `ns-do-applescript`. [2. text/x-patch; 0001-org-clock.el-Make-org-notify-support-macOS-notificat.patch]... On Jul 6, 2021, at 8:06 AM, Tim Cross < theophil...@gmail.com> wrote: stardiviner writes: On Jul 5, 2021, at 7:55 PM, Maxim Nikulin wrote: On 05/07/2021 10:50, stardiviner wrote: I updated the patch, I found the package `osx-lib` contains solution.
Re: [new patch] [PATCH] make org-notify support for macOS desktop notification
It may eventually be incorporated into the very popular emacs-mac port (railwaycat tap in homebrew); however, it will probably not be incorporated into the nextstep/emacs main code. I started looking at doing a version for the mainline code, but it’s hard to get motivated b/c using that version of emacs on OS X is a pretty sub-par experience. I only commented on this b/c I think you might are disabling notifications-notify which work great with my code changes, and using something else if you see Darwin OS, and that will break my native “Just Works” support for notifications, which again may end up on many peoples machines. I would ask that the patch be modified in a way that didn’t break native support if present before it was accepted. Also as you can see by the multiple patches you’ve submitted there’s really no good answer for an external notifier, so whatever you pick is probably going to be wrong for someone I guess. If this patch is going to be accepted I would ask that it 1) be conditional (disable-able with a variable) 2) do the check for the custom installed external notifier and if not present then fallback to using the emacs supplied notifications-notify 3) not restrict notifications-notify to gnu/linux only. That way people that have already developed solutions for this won’t have them broken. Thanks, Chris. > On Jul 7, 2021, at 8:00 PM, stardiviner wrote: > > Hi Chris, thanks for your work. I have a question, will your patch of > notification code be merged to upstream? > If yes, I think my patch will be not necessary. If no, then I think add a my > workaround for macOS is considerable. > >> On Jul 7, 2021, at 2:23 AM, Christian Hopps > <mailto:cho...@chopps.org>> wrote: >> >> It supports imagemagick (specify —with-imagemagick), and it includes svg by >> default, I simply forked the railwaycat version and added the native >> notification code. >> >> Thanks, >> Chris. >> >>> On Jul 6, 2021, at 11:30 AM, stardiviner >> <mailto:numbch...@gmail.com>> wrote: >>> >>> Thanks for your suggestion. Does your Emacs build supports imagemagick >>> image view and svg feature support? Because company-mode now have built-in >>> icons support. This is the reason that I switch from >>> https://emacsformacosx.com/ <https://emacsformacosx.com/> to Homebrew cask >>> Emacs version. >>> >>>> On Jul 6, 2021, at 12:21 PM, Christian Hopps >>> <mailto:cho...@chopps.org>> wrote: >>>> >>>> Hi, >>>> >>>> Please consider: I added full native notification support to the popular >>>> OS X Emacs build available in homebrew. This supports rewrites >>>> notifications-notify defun to use the native code rather than dbus, and so >>>> everything "Just Works". >>>> >>>> Info can be found here: >>>> >>>> https://github.com/choppsv1/homebrew-emacsmacport >>>> <https://github.com/choppsv1/homebrew-emacsmacport> >>>> >>>> Thanks, >>>> Chris. >>>> >>>> stardiviner mailto:numbch...@gmail.com>> writes: >>>> >>>>> Here is the new patch which invokes notifications though Emacs built-in >>>>> API `ns-do-applescript`. >>>>> >>>>> [2. text/x-patch; >>>>> 0001-org-clock.el-Make-org-notify-support-macOS-notificat.patch]... >>>>> >>>>> >>>>> >>>>>> On Jul 6, 2021, at 8:06 AM, Tim Cross >>>>> <mailto:theophil...@gmail.com>> wrote: >>>>>> >>>>>> >>>>>> stardiviner mailto:numbch...@gmail.com>> writes: >>>>>> >>>>>>>> On Jul 5, 2021, at 7:55 PM, Maxim Nikulin >>>>>>> <mailto:maniku...@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 expe
Re: [new patch] [PATCH] make org-notify support for macOS desktop notification
It supports imagemagick (specify —with-imagemagick), and it includes svg by default, I simply forked the railwaycat version and added the native notification code. Thanks, Chris. > On Jul 6, 2021, at 11:30 AM, stardiviner wrote: > > Thanks for your suggestion. Does your Emacs build supports imagemagick image > view and svg feature support? Because company-mode now have built-in icons > support. This is the reason that I switch from https://emacsformacosx.com/ > <https://emacsformacosx.com/> to Homebrew cask Emacs version. > >> On Jul 6, 2021, at 12:21 PM, Christian Hopps > <mailto:cho...@chopps.org>> wrote: >> >> Hi, >> >> Please consider: I added full native notification support to the popular OS >> X Emacs build available in homebrew. This supports rewrites >> notifications-notify defun to use the native code rather than dbus, and so >> everything "Just Works". >> >> Info can be found here: >> >> https://github.com/choppsv1/homebrew-emacsmacport >> <https://github.com/choppsv1/homebrew-emacsmacport> >> >> Thanks, >> Chris. >> >> stardiviner mailto:numbch...@gmail.com>> writes: >> >>> Here is the new patch which invokes notifications though Emacs built-in API >>> `ns-do-applescript`. >>> >>> [2. text/x-patch; >>> 0001-org-clock.el-Make-org-notify-support-macOS-notificat.patch]... >>> >>> >>> >>>> On Jul 6, 2021, at 8:06 AM, Tim Cross >>> <mailto:theophil...@gmail.com>> wrote: >>>> >>>> >>>> stardiviner mailto:numbch...@gmail.com>> writes: >>>> >>>>>> On Jul 5, 2021, at 7:55 PM, Maxim Nikulin >>>>> <mailto:maniku...@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? >>>> >>>> I think you can add dbus support to macOS using homebrew and that might >>>> resolve the issue. At the very least, this will need to be investigated >>>> because otherwise, adding this patch may break configurations for users >>>> who have added dbus support via homebrew and have notifications working, >>>> but have not installed the osx-lib package. >>>> >>>> My only small concern with your proposed changes is that it will add a >>>> dependency on a new package osx-lib, which I think is only available in >>>> melpa. At the very least, this will need to be documented somewhere. >>>> However, I'm not sure what the situation is wrt adding code which >>>> depends on an external package which is not available in either elpa or >>>> nongnuELPA? As org mode is a part of GNU Emacs, I suspect that any code >>>> which 'encourages' the use of melpa packages will not be acceptable. >>>> >>>> -- >>>> Tim Cross >>>> >> >
Re: [new patch] Re: [PATCH] make org-notify support for macOS desktop notification
Hi, Please consider: I added full native notification support to the popular OS X Emacs build available in homebrew. This supports rewrites notifications-notify defun to use the native code rather than dbus, and so everything "Just Works". Info can be found here: https://github.com/choppsv1/homebrew-emacsmacport Thanks, Chris. stardiviner writes: Here is the new patch which invokes notifications though Emacs built-in API `ns-do-applescript`. [2. text/x-patch; 0001-org-clock.el-Make-org-notify-support-macOS-notificat.patch]... On Jul 6, 2021, at 8:06 AM, Tim Cross wrote: stardiviner writes: On Jul 5, 2021, at 7:55 PM, Maxim Nikulin 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? I think you can add dbus support to macOS using homebrew and that might resolve the issue. At the very least, this will need to be investigated because otherwise, adding this patch may break configurations for users who have added dbus support via homebrew and have notifications working, but have not installed the osx-lib package. My only small concern with your proposed changes is that it will add a dependency on a new package osx-lib, which I think is only available in melpa. At the very least, this will need to be documented somewhere. However, I'm not sure what the situation is wrt adding code which depends on an external package which is not available in either elpa or nongnuELPA? As org mode is a part of GNU Emacs, I suspect that any code which 'encourages' the use of melpa packages will not be acceptable. -- Tim Cross
Re: bi-monthly steps.
Thanks, and thanks for adding the test and doc updates! Chris. > On Mar 26, 2020, at 9:12 PM, Kyle Meyer wrote: > > Christian Hopps writes: > >> I've attached an updated patch using "semimonth". > > Applied in 8a99404c8, along with a follow-up commit that adds a test, > updates the manual, and add a NEWS entry. > > Thanks. >
Re: bi-monthly steps.
I've attached an updated patch using "semimonth". Thanks, Chris. 0001-org-clock.el-add-semimonth-step-for-clocktables.patch Description: Binary data
[PATCH] Add semimonth step for clocktables
From 25818bb40620311d7bb497320acafc175c9f4276 Mon Sep 17 00:00:00 2001 From: Christian Hopps Date: Fri, 20 Mar 2020 06:58:29 -0400 --- lisp/org-clock.el | 4 1 file changed, 4 insertions(+) diff --git a/lisp/org-clock.el b/lisp/org-clock.el index 345fd9b5a..c902635f9 100644 --- a/lisp/org-clock.el +++ b/lisp/org-clock.el @@ -2784,6 +2784,7 @@ a number of clock tables." (pcase step (`day "Daily report: ") (`week "Weekly report starting on: ") +(`semimonth "Semimonthly report starting on: ") (`month "Monthly report starting on: ") (`year "Annual report starting on: ") (_ (user-error "Unknown `:step' specification: %S" step @@ -2833,6 +2834,9 @@ a number of clock tables." (let ((offset (if (= dow week-start) 7 (mod (- week-start dow) 7 (list 0 0 org-extend-today-until (+ d offset) m y))) + (`semimonth (list 0 0 0 + (if (< d 16) 16 1) + (if (< d 16) m (1+ m)) y)) (`month (list 0 0 0 month-start (1+ m) y)) (`year (list 0 0 org-extend-today-until 1 1 (1+ y))) (table-begin (line-beginning-position 0)) -- 2.25.0 signature.asc Description: PGP signature
semimonthly steps [Re: bi-monthly steps.]
FWIW the code was actually unambiguous. :) I meant "semimonthly" or twice a month (i.e., like some people get paid: 1-15th, 16th-endofmonth). Thanks, Chris. Loris Bennett writes: Eric S Fraga writes: On Friday, 13 Mar 2020 at 00:47, Peter Neilson wrote: I think the original question from Christian Hopps presumes the meaning, "Every two months." Oh, I took it to mean bi-weekly. ;-) Exactly. So "bi-monthly" just means "twice a week". I'm sure we can all agree on that :-) signature.asc Description: PGP signature
Re: bi-monthly steps.
could use semimonth then :) > On Mar 12, 2020, at 2:15 PM, Nick Dokos wrote: > > "Bi-monthly" is ambiguous: it can mean twice a month or it can mean > once every two months and there is no convincing anybody that their > use of it is wrong :-) > > -- > Nick > > "There are only two hard problems in computer science: cache > invalidation, naming things, and off-by-one errors." -Martin Fowler > >
bi-monthly steps.
I've the need to display bimonthly clock tables, I was able to add this pretty easy in case the project would find it useful --- org-clock.el2020-03-12 06:52:14.0 -0400 +++ /Users/chopps/org-clock.el 2020-03-12 06:51:43.0 -0400 @@ -2719,6 +2719,7 @@ (pcase step (`day "Daily report: ") (`week "Weekly report starting on: ") +(`bimonth "Bi-Monthly report starting on: ") (`month "Monthly report starting on: ") (`year "Annual report starting on: ") (_ (user-error "Unknown `:step' specification: %S" step @@ -2768,6 +2769,7 @@ (let ((offset (if (= dow week-start) 7 (mod (- week-start dow) 7 (list 0 0 org-extend-today-until (+ d offset) m y))) + (`bimonth (list 0 0 0 (if (< d 16) 16 1) (if (< d 16) m (1+ m)) y)) (`month (list 0 0 0 month-start (1+ m) y)) (`year (list 0 0 org-extend-today-until 1 1 (1+ y))) (table-begin (line-beginning-position 0)) Thanks, Chris. signature.asc Description: PGP signature