all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Tom Breton (Tehom)" <tehom@panix.com>
To: Carsten Dominik <dominik@science.uva.nl>
Cc: emacs-orgmode@gnu.org
Subject: Re: Org-choose
Date: Fri, 13 Feb 2009 14:34:09 -0500 (EST)	[thread overview]
Message-ID: <1036.24.63.20.74.1234553649.squirrel@mail.panix.com> (raw)
In-Reply-To: <5E0C3EFF-06FA-4446-B869-F514B6748C5E@uva.nl>

Hi, Carsten.


> One more thought:  I did read in another message that you
> are using an older version of Emacs? Emacs 21?
>
> I have a vague memory that split-string might work different
> there, with empty fields at the beginning and end....
>
> I am not at all sure, though.  It it has to to, maybe
> you want to use org-split-string....

That just might be it!  I have changed it to org-split-string.  A
patch is attached.

I can't detect any difference - all tests pass either way - so I hope
you will tell me whether it works now.

> Hi Tom,
>
> first, I did apply your patch - even though I do not
> think the require org-agenda should be needed.

I agree.  But since debugging at a distance is painful, and
org-map-entries calls agenda functions, I am trying to play it safe.

One other play-it-safe change in the patch: (require 'cl) is now on
its own, not inside `eval-when-compile'.  I realized last night that I
always have `cl' loaded, so maybe that was the difference between my
setup and others'.

> I also did make the change (not pushed yet) 'org -> "org", also because
> this is compatible with XEmacs, the symbol version not.

Thank you.  And thank you for your patience with this.

Tom Breton (Tehom)

       reply	other threads:[~2009-02-13 19:34 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <273688C0-9A36-47D5-B747-A1371C2BD05A@uva.nl>
     [not found] ` <1782.65.96.63.84.1234471662.squirrel@mail.panix.com>
     [not found]   ` <5E0C3EFF-06FA-4446-B869-F514B6748C5E@uva.nl>
2009-02-13 19:34     ` Tom Breton (Tehom) [this message]
2009-02-13 21:34       ` Org-choose Carsten Dominik
2009-02-13 22:29         ` Org-choose Tom Breton (Tehom)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1036.24.63.20.74.1234553649.squirrel@mail.panix.com \
    --to=tehom@panix.com \
    --cc=dominik@science.uva.nl \
    --cc=emacs-orgmode@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.