[PATCH] org-protocol: Make org protocol defaults work on windows

[PATCH] org-protocol: Make org protocol defaults work on windows

[Robert]

[-- Attachment #1: Type: text/plain, Size: 161 bytes --]

Hei,

Attached a little patch to make the default org-protocol handlers work on windows.

Tested with Windows 10 and emacs 27.1

Best Regards

Robert


[-- Attachment #2: 0001-org-protocol-Make-org-protocol-defaults-work-on-wind.patch --]
[-- Type: application/octet-stream, Size: 1903 bytes --]

From 40257be885a73bc055195c52d2acd354ce2ba0f9 Mon Sep 17 00:00:00 2001
From: Robert Ewald <roew@iris.no>
Date: Fri, 20 Nov 2020 15:52:00 +0100
Subject: [PATCH] org-protocol: Make org protocol defaults work on windows

* org-protocol (org-protocol-protocol-alist-default): Make default org-protocol
handlers work on windows

Windows inserts a slash in the protocol handler. By defining these
protocols with a trailing slash we are able to handle this case.

TINYCHANGE
---
 lisp/org-protocol.el | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/lisp/org-protocol.el b/lisp/org-protocol.el
index 4bc7cee31..5b6862b78 100644
--- a/lisp/org-protocol.el
+++ b/lisp/org-protocol.el
@@ -138,10 +138,15 @@ for `org-protocol-the-protocol' and sub-protocols defined in
 
 ;;; Variables:
 
+;; Windows inserts a slash in the protocol handler. Handle this case with the constants
+
 (defconst org-protocol-protocol-alist-default
-  '(("org-capture"     :protocol "capture"     :function org-protocol-capture  :kill-client t)
-    ("org-store-link"  :protocol "store-link"  :function org-protocol-store-link)
-    ("org-open-source" :protocol "open-source" :function org-protocol-open-source))
+  '(("org-capture"     :protocol "capture/"     :function org-protocol-capture  :kill-client t)
+    ("org-capture"     :protocol "capture"      :function org-protocol-capture  :kill-client t)
+    ("org-store-link"  :protocol "store-link/"  :function org-protocol-store-link)
+    ("org-store-link"  :protocol "store-link"   :function org-protocol-store-link)
+    ("org-open-source" :protocol "open-source/" :function org-protocol-open-source)
+    ("org-open-source" :protocol "open-source"  :function org-protocol-open-source))
   "Default protocols to use.
 See `org-protocol-protocol-alist' for a description of this variable.")
 
-- 
2.29.2

Re: [PATCH] org-protocol: Make org protocol defaults work on windows

[Kyle Meyer]

Robert writes:

> Hei,
>
> Attached a little patch to make the default org-protocol handlers work on windows.
>
> Tested with Windows 10 and emacs 27.1

Thank you for the patch.

Any org-protocol and/or Windows users out there that can provide
feedback?

Re: [PATCH] org-protocol: Make org protocol defaults work on windows

[Maxim Nikulin]

On 20/11/2020 23:26, Robert wrote:
> Attached a little patch to make the default org-protocol
> handlers work on windows.

Unsure, but the issue might be addressed by the following change:
https://code.orgmode.org/bzg/org-mode/commit/928e67df
org-protocol: Allow slashes after sub-protocol

I am curious if alternative approach could help. Have you tried
one or three slashes after "org-protocol:"? E.g.
org-protocol:/capture?template=... or
org-protocol:///capture?template=... instead of
org-protocol://capture?template=...

I suspect that with double slash subprotocol is considered as a name
of some host so "/" as path before query part is added. With other
number of slashes subprotocol does not look like a host name
similar to file:/// scheme. See
https://orgmode.org/list/s3flvl$a48$1@ciao.gmane.io/

Re: [PATCH] org-protocol: Make org protocol defaults work on windows

[Bastien]

Hi,

Maxim Nikulin <manikulin@gmail.com> writes:

> On 20/11/2020 23:26, Robert wrote:
>> Attached a little patch to make the default org-protocol
>> handlers work on windows.
>
> Unsure, but the issue might be addressed by the following change:
> https://code.orgmode.org/bzg/org-mode/commit/928e67df
> org-protocol: Allow slashes after sub-protocol

I'm optimistically closing this patch.  If the OP confirmes it is
still needed and Maxim confirms it is correct, we can apply a new 
one.

Thanks,

-- 
 Bastien