From mboxrd@z Thu Jan 1 00:00:00 1970 From: Carsten Dominik Subject: Re: [patch] org-id: New org-id-method 'uuid' w/ elisp function that creates random uuid Date: Sun, 16 May 2010 11:39:20 +0200 Message-ID: References: <874oijaeni.wl%dmaus@ictsoc.de> <87EC457F-92D2-404E-AD87-B8AA1C73723F@gmail.com> <87aasa7a6e.wl%dmaus@ictsoc.de> <8B270AAC-10B3-40A8-B7C9-682308DEBD6F@gmail.com> <87sk5sb43b.wl%dmaus@ictsoc.de> Mime-Version: 1.0 (Apple Message framework v936) Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes Content-Transfer-Encoding: 7bit Return-path: Received: from [140.186.70.92] (port=60238 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1ODaJi-0000NA-45 for emacs-orgmode@gnu.org; Sun, 16 May 2010 05:39:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1ODaJg-0000pV-OT for emacs-orgmode@gnu.org; Sun, 16 May 2010 05:39:25 -0400 Received: from mail-ew0-f216.google.com ([209.85.219.216]:57225) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1ODaJg-0000pQ-JR for emacs-orgmode@gnu.org; Sun, 16 May 2010 05:39:24 -0400 Received: by ewy8 with SMTP id 8so1184107ewy.8 for ; Sun, 16 May 2010 02:39:23 -0700 (PDT) In-Reply-To: <87sk5sb43b.wl%dmaus@ictsoc.de> List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: David Maus Cc: org-mode On May 16, 2010, at 11:14 AM, David Maus wrote: > Carsten Dominik wrote: >>> >>> - `org-id-new' uses uuidgen if present and falls back to the elisp >>> function otherwise > >> I like that. WOuld you like to modify your patch accordingly? > > Yes, I'll do that. Just for the records: using the elisp function as > fall back for the external program changes the meaning of the > org-id-method uuidgen: It makes Org use UUIDs for the ID property and > the external program is the preferred method for doing this. Hence > the symbol should better be 'uuid, not 'uuidgen. So if there is a > list for backward incompatible changes for a further release (maybe > 7.x) that are reasonable but not important this should go there. Yes. But what I would like to to ask is this: Make the change and tell people to use `uuid' as the setting. But if a user has set it to uuidgen, be gentle and accept that setting as well, treat it like uuid. Thanks! - Carsten