emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* patch for org-mac-link-grabber
@ 2010-06-24 18:53 Anthony Lander
  2010-06-29  4:53 ` Carsten Dominik
  0 siblings, 1 reply; 2+ messages in thread
From: Anthony Lander @ 2010-06-24 18:53 UTC (permalink / raw)
  To: Carsten Dominik; +Cc: emacs-orgmode List

Hi Carsten,

Could you please apply the following patch to fix a problem where  
Together links were being incorrectly opened with AddressBook?

Thanks

   -Anthony



----------------------------------------------
diff --git a/contrib/lisp/org-mac-link-grabber.el b/contrib/lisp/org- 
mac-link-grabber.el
index 46a9565..bb12204 100644
--- a/contrib/lisp/org-mac-link-grabber.el
+++ b/contrib/lisp/org-mac-link-grabber.el
@@ -318,7 +318,7 @@ applications and inserting them in org documents"

  (defun org-mac-together-item-open (uid)
    "Open the given uid, which is a reference to an item in Together"
-  (shell-command (concat "open \"x-together-item:" uid "\"")))
+  (shell-command (concat "open -a Together \"x-together-item:" uid  
"\"")))

  (defun as-get-selected-together-items ()
    (do-applescript
@@ -378,9 +378,9 @@ applications and inserting them in org documents"
  ;;
  ;;

-(org-add-link-type "addressbook" 'org-mac-together-item-open)
+(org-add-link-type "addressbook" 'org-mac-addressbook-item-open)

-(defun org-mac-together-item-open (uid)
+(defun org-mac-addressbook-item-open (uid)
    "Open the given uid, which is a reference to an item in Together"
    (shell-command (concat "open \"addressbook:" uid "\"")))

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

* Re: patch for org-mac-link-grabber
  2010-06-24 18:53 patch for org-mac-link-grabber Anthony Lander
@ 2010-06-29  4:53 ` Carsten Dominik
  0 siblings, 0 replies; 2+ messages in thread
From: Carsten Dominik @ 2010-06-29  4:53 UTC (permalink / raw)
  To: Anthony Lander; +Cc: emacs-orgmode List

Anthony,

this somehow got garbles in the patchworks catcher.  Can you please  
submit the patch again and attach it, so that you email program cannot  
garble it?

Thanks.

- Carsten

On Jun 24, 2010, at 8:53 PM, Anthony Lander wrote:

> Hi Carsten,
>
> Could you please apply the following patch to fix a problem where  
> Together links were being incorrectly opened with AddressBook?
>
> Thanks
>
>  -Anthony
>
>
>
> ----------------------------------------------
> diff --git a/contrib/lisp/org-mac-link-grabber.el b/contrib/lisp/org- 
> mac-link-grabber.el
> index 46a9565..bb12204 100644
> --- a/contrib/lisp/org-mac-link-grabber.el
> +++ b/contrib/lisp/org-mac-link-grabber.el
> @@ -318,7 +318,7 @@ applications and inserting them in org documents"
>
> (defun org-mac-together-item-open (uid)
>   "Open the given uid, which is a reference to an item in Together"
> -  (shell-command (concat "open \"x-together-item:" uid "\"")))
> +  (shell-command (concat "open -a Together \"x-together-item:" uid  
> "\"")))
>
> (defun as-get-selected-together-items ()
>   (do-applescript
> @@ -378,9 +378,9 @@ applications and inserting them in org documents"
> ;;
> ;;
>
> -(org-add-link-type "addressbook" 'org-mac-together-item-open)
> +(org-add-link-type "addressbook" 'org-mac-addressbook-item-open)
>
> -(defun org-mac-together-item-open (uid)
> +(defun org-mac-addressbook-item-open (uid)
>   "Open the given uid, which is a reference to an item in Together"
>   (shell-command (concat "open \"addressbook:" uid "\"")))
>

- Carsten

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

end of thread, other threads:[~2010-06-29  4:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-24 18:53 patch for org-mac-link-grabber Anthony Lander
2010-06-29  4:53 ` 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).