emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* [PATCH] Detect Mac OS X in iCal import
@ 2010-03-30 21:03 Jonathan Creekmore
  2010-03-30 21:25 ` Christopher Suckling
  2010-03-30 22:16 ` Carsten Dominik
  0 siblings, 2 replies; 3+ messages in thread
From: Jonathan Creekmore @ 2010-03-30 21:03 UTC (permalink / raw)
  To: emacs-orgmode

This patch modifies the Mac OS X detection code to check for both
Leopard and Snow Leopard, since they both store iCal events in separate
files for Spotlight searching purposes.

Jonathan

--8<---------------cut here---------------start------------->8---
diff --git a/contrib/lisp/org-mac-iCal.el b/contrib/lisp/org-mac-iCal.el
index 0d0b4f8..2510aa7 100644
--- a/contrib/lisp/org-mac-iCal.el
+++ b/contrib/lisp/org-mac-iCal.el
@@ -98,8 +98,8 @@ the the Emacs diary"
 
   ;; for each calendar, concatenate individual events into a single ics file
   (with-temp-buffer
-    (shell-command "sw_vers" " *temp*")
-    (when (re-search-backward "10.5" nil t)
+    (shell-command "sw_vers" (current-buffer))
+    (when (re-search-backward "10\\.[56]" nil t)
       (omi-concat-leopard-ics all-calendars)))
   
   ;; move all caldav ics files to the same place as local ics files
--8<---------------cut here---------------start------------->8---

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

* Re: [PATCH] Detect Mac OS X in iCal import
  2010-03-30 21:03 [PATCH] Detect Mac OS X in iCal import Jonathan Creekmore
@ 2010-03-30 21:25 ` Christopher Suckling
  2010-03-30 22:16 ` Carsten Dominik
  1 sibling, 0 replies; 3+ messages in thread
From: Christopher Suckling @ 2010-03-30 21:25 UTC (permalink / raw)
  To: Jonathan Creekmore; +Cc: emacs-orgmode

Thanks. I did this in my local repository last September and completely
forgot to submit it. Though curiously the contrib version seems to work
fine on my Snow Leopard install.

Best, Christopher

On Tue, 30 Mar 2010 16:03:08 -0500, Jonathan Creekmore <jonathan@thecreekmores.org> wrote:

> This patch modifies the Mac OS X detection code to check for both
> Leopard and Snow Leopard, since they both store iCal events in separate
> files for Spotlight searching purposes.
>
> Jonathan
>
> diff --git a/contrib/lisp/org-mac-iCal.el b/contrib/lisp/org-mac-iCal.el
> index 0d0b4f8..2510aa7 100644
> --- a/contrib/lisp/org-mac-iCal.el
> +++ b/contrib/lisp/org-mac-iCal.el
> @@ -98,8 +98,8 @@ the the Emacs diary"
>  
>    ;; for each calendar, concatenate individual events into a single ics file
>    (with-temp-buffer
> -    (shell-command "sw_vers" " *temp*")
> -    (when (re-search-backward "10.5" nil t)
> +    (shell-command "sw_vers" (current-buffer))
> +    (when (re-search-backward "10\\.[56]" nil t)
>        (omi-concat-leopard-ics all-calendars)))
>    
>    ;; move all caldav ics files to the same place as local ics files
>
>
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

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

* Re: [PATCH] Detect Mac OS X in iCal import
  2010-03-30 21:03 [PATCH] Detect Mac OS X in iCal import Jonathan Creekmore
  2010-03-30 21:25 ` Christopher Suckling
@ 2010-03-30 22:16 ` Carsten Dominik
  1 sibling, 0 replies; 3+ messages in thread
From: Carsten Dominik @ 2010-03-30 22:16 UTC (permalink / raw)
  To: Jonathan Creekmore; +Cc: emacs-orgmode

Applied, thanks.

- Carsten

On Mar 30, 2010, at 11:03 PM, Jonathan Creekmore wrote:

> This patch modifies the Mac OS X detection code to check for both
> Leopard and Snow Leopard, since they both store iCal events in  
> separate
> files for Spotlight searching purposes.
>
> Jonathan
>
> --8<---------------cut here---------------start------------->8---
> diff --git a/contrib/lisp/org-mac-iCal.el b/contrib/lisp/org-mac- 
> iCal.el
> index 0d0b4f8..2510aa7 100644
> --- a/contrib/lisp/org-mac-iCal.el
> +++ b/contrib/lisp/org-mac-iCal.el
> @@ -98,8 +98,8 @@ the the Emacs diary"
>
>   ;; for each calendar, concatenate individual events into a single  
> ics file
>   (with-temp-buffer
> -    (shell-command "sw_vers" " *temp*")
> -    (when (re-search-backward "10.5" nil t)
> +    (shell-command "sw_vers" (current-buffer))
> +    (when (re-search-backward "10\\.[56]" nil t)
>       (omi-concat-leopard-ics all-calendars)))
>
>   ;; move all caldav ics files to the same place as local ics files
> --8<---------------cut here---------------start------------->8---
>
>
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

- Carsten

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

end of thread, other threads:[~2010-03-30 22:17 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-03-30 21:03 [PATCH] Detect Mac OS X in iCal import Jonathan Creekmore
2010-03-30 21:25 ` Christopher Suckling
2010-03-30 22:16 ` Carsten Dominik

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).