* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
@ 2013-07-28 13:49 Oleh
2013-07-30 10:46 ` Bastien
2013-07-30 10:46 ` Bastien
0 siblings, 2 replies; 15+ messages in thread
From: Oleh @ 2013-07-28 13:49 UTC (permalink / raw)
To: 14975
[-- Attachment #1: Type: text/plain, Size: 137 bytes --]
Currently, this function behaves unconditionally
as if `org-clock-sound' was set to t.
I attach a patch of how I fixed it for my needs:
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-diff, Size: 847 bytes --]
From db2cc63bbdd4e51bd7a150186b131a39d7d10a6e Mon Sep 17 00:00:00 2001
From: oleh <wave@mail.ua>
Date: Sun, 28 Jul 2013 15:39:57 +0200
Subject: [PATCH] org-clock.el: `org-clock-notify-once-if-expired' respects
`org-clock-sound'.
---
lisp/org-clock.el | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lisp/org-clock.el b/lisp/org-clock.el
index 8ac215e..99479b2 100644
--- a/lisp/org-clock.el
+++ b/lisp/org-clock.el
@@ -718,7 +718,7 @@ Notification is shown only once."
(setq org-clock-notification-was-shown t)
(org-notify
(format "Task '%s' should be finished by now. (%s)"
- org-clock-heading org-clock-effort) t))
+ org-clock-heading org-clock-effort) org-clock-sound))
(setq org-clock-notification-was-shown nil)))))
(defun org-notify (notification &optional play-sound)
--
1.8.1.2
[-- Attachment #3: Type: text/plain, Size: 18 bytes --]
regards,
Oleh
^ permalink raw reply related [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-28 13:49 bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound' Oleh
2013-07-30 10:46 ` Bastien
@ 2013-07-30 10:46 ` Bastien
1 sibling, 0 replies; 15+ messages in thread
From: Bastien @ 2013-07-30 10:46 UTC (permalink / raw)
To: Oleh; +Cc: 14975-done
Hi Oleh,
Oleh <o.krehel@tue.nl> writes:
> Currently, this function behaves unconditionally
> as if `org-clock-sound' was set to t.
>
> I attach a patch of how I fixed it for my needs:
Please send those patches to the org-mode list first.
I applied it but added a ChangeLog entry in the commit message
and the TINYCHANGE change cookie that helps us track small changes
by contributors who did not assign their copyright to the FSF (yet).
Thanks for catching this,
--
Bastien
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-28 13:49 bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound' Oleh
@ 2013-07-30 10:46 ` Bastien
2013-07-30 10:50 ` Bastien
2013-07-30 10:50 ` Bastien
2013-07-30 10:46 ` Bastien
1 sibling, 2 replies; 15+ messages in thread
From: Bastien @ 2013-07-30 10:46 UTC (permalink / raw)
To: Oleh; +Cc: 14975-done
Hi Oleh,
Oleh <o.krehel@tue.nl> writes:
> Currently, this function behaves unconditionally
> as if `org-clock-sound' was set to t.
>
> I attach a patch of how I fixed it for my needs:
Please send those patches to the org-mode list first.
I applied it but added a ChangeLog entry in the commit message
and the TINYCHANGE change cookie that helps us track small changes
by contributors who did not assign their copyright to the FSF (yet).
Thanks for catching this,
--
Bastien
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 10:46 ` Bastien
@ 2013-07-30 10:50 ` Bastien
2013-07-30 10:50 ` Bastien
1 sibling, 0 replies; 15+ messages in thread
From: Bastien @ 2013-07-30 10:50 UTC (permalink / raw)
To: Oleh; +Cc: 14975-done
Bastien <bzg@altern.org> writes:
> Please send those patches to the org-mode list first.
PS: Maybe it's just me and I did not see the initial bug
report on this list -- ignore the heads up if that's so!
--
Bastien
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 10:46 ` Bastien
2013-07-30 10:50 ` Bastien
@ 2013-07-30 10:50 ` Bastien
2013-07-30 15:53 ` Glenn Morris
2013-07-30 15:53 ` Glenn Morris
1 sibling, 2 replies; 15+ messages in thread
From: Bastien @ 2013-07-30 10:50 UTC (permalink / raw)
To: Oleh; +Cc: 14975-done
Bastien <bzg@altern.org> writes:
> Please send those patches to the org-mode list first.
PS: Maybe it's just me and I did not see the initial bug
report on this list -- ignore the heads up if that's so!
--
Bastien
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 10:50 ` Bastien
@ 2013-07-30 15:53 ` Glenn Morris
2013-07-30 17:22 ` Bastien
2013-07-30 17:22 ` Bastien
2013-07-30 15:53 ` Glenn Morris
1 sibling, 2 replies; 15+ messages in thread
From: Glenn Morris @ 2013-07-30 15:53 UTC (permalink / raw)
To: Bastien; +Cc: 14975
Bastien wrote:
>> Please send those patches to the org-mode list first.
>
> PS: Maybe it's just me and I did not see the initial bug
> report on this list -- ignore the heads up if that's so!
If by "this list" you mean the org-mode list:
It was initially reported to bug-gnu-emacs. Then I assigned it to the
"emacs,org-mode" package, so that subsequent emails went to the org-mode
list as well as the bug-gnu-emacs list. This would have happened with
the initial report if it had specified a "Package: emacs,org-mode" line.
(Though there seems no point in having "emacs" in there really.)
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 10:50 ` Bastien
2013-07-30 15:53 ` Glenn Morris
@ 2013-07-30 15:53 ` Glenn Morris
1 sibling, 0 replies; 15+ messages in thread
From: Glenn Morris @ 2013-07-30 15:53 UTC (permalink / raw)
To: Bastien; +Cc: 14975
Bastien wrote:
>> Please send those patches to the org-mode list first.
>
> PS: Maybe it's just me and I did not see the initial bug
> report on this list -- ignore the heads up if that's so!
If by "this list" you mean the org-mode list:
It was initially reported to bug-gnu-emacs. Then I assigned it to the
"emacs,org-mode" package, so that subsequent emails went to the org-mode
list as well as the bug-gnu-emacs list. This would have happened with
the initial report if it had specified a "Package: emacs,org-mode" line.
(Though there seems no point in having "emacs" in there really.)
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 15:53 ` Glenn Morris
@ 2013-07-30 17:22 ` Bastien
2013-07-30 17:22 ` Bastien
1 sibling, 0 replies; 15+ messages in thread
From: Bastien @ 2013-07-30 17:22 UTC (permalink / raw)
To: Glenn Morris; +Cc: 14975
Hi Glenn,
Glenn Morris <rgm@gnu.org> writes:
>>> Please send those patches to the org-mode list first.
>>
>> PS: Maybe it's just me and I did not see the initial bug
>> report on this list -- ignore the heads up if that's so!
>
> If by "this list" you mean the org-mode list:
> It was initially reported to bug-gnu-emacs. Then I assigned it to the
> "emacs,org-mode" package, so that subsequent emails went to the org-mode
> list as well as the bug-gnu-emacs list. This would have happened with
> the initial report if it had specified a "Package: emacs,org-mode" line.
> (Though there seems no point in having "emacs" in there really.)
Thanks for the explanations.
This is what I expected, but I don't find the bug in the org-mode
mailing list. The only reference I can find is my reply:
http://lists.gnu.org/archive/html/emacs-orgmode/2013-07/msg01094.html
Does anyone receive this bug on the first place? Or someone can
browse the archives better than me?
--
Bastien
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 15:53 ` Glenn Morris
2013-07-30 17:22 ` Bastien
@ 2013-07-30 17:22 ` Bastien
2013-07-30 17:27 ` Glenn Morris
2013-07-30 17:27 ` Glenn Morris
1 sibling, 2 replies; 15+ messages in thread
From: Bastien @ 2013-07-30 17:22 UTC (permalink / raw)
To: Glenn Morris; +Cc: 14975
Hi Glenn,
Glenn Morris <rgm@gnu.org> writes:
>>> Please send those patches to the org-mode list first.
>>
>> PS: Maybe it's just me and I did not see the initial bug
>> report on this list -- ignore the heads up if that's so!
>
> If by "this list" you mean the org-mode list:
> It was initially reported to bug-gnu-emacs. Then I assigned it to the
> "emacs,org-mode" package, so that subsequent emails went to the org-mode
> list as well as the bug-gnu-emacs list. This would have happened with
> the initial report if it had specified a "Package: emacs,org-mode" line.
> (Though there seems no point in having "emacs" in there really.)
Thanks for the explanations.
This is what I expected, but I don't find the bug in the org-mode
mailing list. The only reference I can find is my reply:
http://lists.gnu.org/archive/html/emacs-orgmode/2013-07/msg01094.html
Does anyone receive this bug on the first place? Or someone can
browse the archives better than me?
--
Bastien
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 17:22 ` Bastien
@ 2013-07-30 17:27 ` Glenn Morris
2013-07-30 17:27 ` Glenn Morris
1 sibling, 0 replies; 15+ messages in thread
From: Glenn Morris @ 2013-07-30 17:27 UTC (permalink / raw)
To: Bastien; +Cc: 14975
Bastien wrote:
>> It was initially reported to bug-gnu-emacs. Then I assigned it to the
>> "emacs,org-mode" package, so that subsequent emails went to the org-mode
>> list as well as the bug-gnu-emacs list. This would have happened with
>> the initial report if it had specified a "Package: emacs,org-mode" line.
>> (Though there seems no point in having "emacs" in there really.)
>
> Thanks for the explanations.
>
> This is what I expected, but I don't find the bug in the org-mode
> mailing list.
That's what I'm saying. It was sent to bug-gnu-emacs ONLY. Because that
list is coupled to debbugs.gnu.org, and because I reassigned the bug
after it was received, subsequent replies went to both lists. But the
initial report is only on bug-gnu-emacs.
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 17:22 ` Bastien
2013-07-30 17:27 ` Glenn Morris
@ 2013-07-30 17:27 ` Glenn Morris
2013-07-30 18:24 ` Stefan Monnier
` (3 more replies)
1 sibling, 4 replies; 15+ messages in thread
From: Glenn Morris @ 2013-07-30 17:27 UTC (permalink / raw)
To: Bastien; +Cc: 14975
Bastien wrote:
>> It was initially reported to bug-gnu-emacs. Then I assigned it to the
>> "emacs,org-mode" package, so that subsequent emails went to the org-mode
>> list as well as the bug-gnu-emacs list. This would have happened with
>> the initial report if it had specified a "Package: emacs,org-mode" line.
>> (Though there seems no point in having "emacs" in there really.)
>
> Thanks for the explanations.
>
> This is what I expected, but I don't find the bug in the org-mode
> mailing list.
That's what I'm saying. It was sent to bug-gnu-emacs ONLY. Because that
list is coupled to debbugs.gnu.org, and because I reassigned the bug
after it was received, subsequent replies went to both lists. But the
initial report is only on bug-gnu-emacs.
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 17:27 ` Glenn Morris
2013-07-30 18:24 ` Stefan Monnier
@ 2013-07-30 18:24 ` Stefan Monnier
2013-07-30 20:54 ` Bastien
2013-07-30 20:54 ` Bastien
3 siblings, 0 replies; 15+ messages in thread
From: Stefan Monnier @ 2013-07-30 18:24 UTC (permalink / raw)
To: Glenn Morris; +Cc: Bastien, 14975
> That's what I'm saying. It was sent to bug-gnu-emacs ONLY. Because that
> list is coupled to debbugs.gnu.org, and because I reassigned the bug
> after it was received, subsequent replies went to both lists. But the
> initial report is only on bug-gnu-emacs.
It would be nice if some message were sent to org-mode's list when
`org-mode' is added to the "Package:" list.
Stefan "applicable to other packages than org-mode, obviously"
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 17:27 ` Glenn Morris
@ 2013-07-30 18:24 ` Stefan Monnier
2013-07-30 18:24 ` Stefan Monnier
` (2 subsequent siblings)
3 siblings, 0 replies; 15+ messages in thread
From: Stefan Monnier @ 2013-07-30 18:24 UTC (permalink / raw)
To: Glenn Morris; +Cc: Bastien, 14975
> That's what I'm saying. It was sent to bug-gnu-emacs ONLY. Because that
> list is coupled to debbugs.gnu.org, and because I reassigned the bug
> after it was received, subsequent replies went to both lists. But the
> initial report is only on bug-gnu-emacs.
It would be nice if some message were sent to org-mode's list when
`org-mode' is added to the "Package:" list.
Stefan "applicable to other packages than org-mode, obviously"
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 17:27 ` Glenn Morris
` (2 preceding siblings ...)
2013-07-30 20:54 ` Bastien
@ 2013-07-30 20:54 ` Bastien
3 siblings, 0 replies; 15+ messages in thread
From: Bastien @ 2013-07-30 20:54 UTC (permalink / raw)
To: Glenn Morris; +Cc: 14975
Glenn Morris <rgm@gnu.org> writes:
> That's what I'm saying. It was sent to bug-gnu-emacs ONLY. Because that
> list is coupled to debbugs.gnu.org, and because I reassigned the bug
> after it was received, subsequent replies went to both lists. But the
> initial report is only on bug-gnu-emacs.
Got it now, thanks!
--
Bastien, who agrees with Stefan's suggestion on this
^ permalink raw reply [flat|nested] 15+ messages in thread
* bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound'
2013-07-30 17:27 ` Glenn Morris
2013-07-30 18:24 ` Stefan Monnier
2013-07-30 18:24 ` Stefan Monnier
@ 2013-07-30 20:54 ` Bastien
2013-07-30 20:54 ` Bastien
3 siblings, 0 replies; 15+ messages in thread
From: Bastien @ 2013-07-30 20:54 UTC (permalink / raw)
To: Glenn Morris; +Cc: 14975
Glenn Morris <rgm@gnu.org> writes:
> That's what I'm saying. It was sent to bug-gnu-emacs ONLY. Because that
> list is coupled to debbugs.gnu.org, and because I reassigned the bug
> after it was received, subsequent replies went to both lists. But the
> initial report is only on bug-gnu-emacs.
Got it now, thanks!
--
Bastien, who agrees with Stefan's suggestion on this
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2013-07-30 21:39 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-28 13:49 bug#14975: 24.3; org-mode's `org-clock-notify-once-if-expired' doesn't respect `org-clock-sound' Oleh
2013-07-30 10:46 ` Bastien
2013-07-30 10:50 ` Bastien
2013-07-30 10:50 ` Bastien
2013-07-30 15:53 ` Glenn Morris
2013-07-30 17:22 ` Bastien
2013-07-30 17:22 ` Bastien
2013-07-30 17:27 ` Glenn Morris
2013-07-30 17:27 ` Glenn Morris
2013-07-30 18:24 ` Stefan Monnier
2013-07-30 18:24 ` Stefan Monnier
2013-07-30 20:54 ` Bastien
2013-07-30 20:54 ` Bastien
2013-07-30 15:53 ` Glenn Morris
2013-07-30 10:46 ` Bastien
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.