emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* [BUG] org-timer requires org-notify from org-clock
@ 2010-09-06 20:21 Łukasz Stelmach
  2010-09-07  0:28 ` Bastien
  0 siblings, 1 reply; 2+ messages in thread
From: Łukasz Stelmach @ 2010-09-06 20:21 UTC (permalink / raw)
  To: emacs-orgmode

Hello.

It seems like there has to be (require 'org-clock) on top of
org-timer.el because otherwise the notification lambda
in-org-timer-set-timer fails because org-notify is not defined and the
timer starts going forward after it passes 0:00:00. It also looks like

(declare-function org-notify "org-clock" (notification &optional
                  play-sound))

is not enough. (BTW. now its d-f org-show-notification which is not used
in org-timer). Simple

(require 'org-clock)

helps, as anything else that loads org-clock like clocking-in.

-- 
Miłego dnia,
Łukasz Stelmach

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [BUG] org-timer requires org-notify from org-clock
  2010-09-06 20:21 [BUG] org-timer requires org-notify from org-clock Łukasz Stelmach
@ 2010-09-07  0:28 ` Bastien
  0 siblings, 0 replies; 2+ messages in thread
From: Bastien @ 2010-09-07  0:28 UTC (permalink / raw)
  To: Łukasz Stelmach; +Cc: emacs-orgmode

Łukasz Stelmach <lukasz.stelmach@iem.pw.edu.pl> writes:

> It seems like there has to be (require 'org-clock) on top of
> org-timer.el because otherwise the notification lambda
> in-org-timer-set-timer fails because org-notify is not defined and the
> timer starts going forward after it passes 0:00:00. It also looks like
>
> (declare-function org-notify "org-clock" (notification &optional
>                   play-sound))
>
> is not enough. (BTW. now its d-f org-show-notification which is not used
> in org-timer). Simple
>
> (require 'org-clock)
>
> helps, as anything else that loads org-clock like clocking-in.

Fixed, thanks.

-- 
 Bastien

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-09-07  0:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-06 20:21 [BUG] org-timer requires org-notify from org-clock Łukasz Stelmach
2010-09-07  0:28 ` Bastien

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).