Add a plumb: url scheme to plumb.app

[robkroeger, 2011-12-15 13:00:07]

This patch adds a plumb: url helper scheme to plumb.app so that links of the
form of plumb://open?url=/tmp/foo.txt in a html page would request the plumber
to open foo.txt.

Please have a look.

[rsc, 2011-12-15 15:35:12]

It seems like there is a lot of unnecessary syntax here
(open, url, file://).  Is there a reason for the generality
as compared with, say, plumb:/tmp/foo.txt?action=edit

[robkroeger, 2011-12-16 13:26:40]

On 2011/12/15 15:35:12, rsc wrote:
> It seems like there is a lot of unnecessary syntax here
> (open, url, file://).  Is there a reason for the generality
> as compared with, say, plumb:/tmp/foo.txt?action=edit

My goal was to re-use tools that I had found from a time when I was exploring
TextMate so I took the URL scheme from there
(http://manual.macromates.com/en/using_textmate_from_terminal#url_scheme_html).
It seemed easier to write one slightly more complicated plumber rule and replace
txtmt: with plumb: in script output instead of needing to understand someone
else's code for html generation.

The objective C code delivers the entire url so I'm perfectly happy to write a
simpler example scheme for plumb/base and keep my (theoretically
effort-reducing) more complicated rule in a personal plumbing rule if that would
be more lgtm-able. Please advise.

Rob.

[rsc, 2011-12-16 15:22:28]

For now let's just do
    plumb:<thing>
means run 'plumb -d edit thing'.

The script in Plumb.app should handle the case
instead of having to do it in the plumb rules.

Russ

[robkroeger, 2012-02-01 12:34:51]

PTAL. Simpler patch usually means better patch. :-)

[robkroeger, 2012-02-08 11:44:34]

Please take another look at the revised change.

[robkroeger, 2012-10-20 12:19:44]

On 2012/02/08 11:44:34, robkroeger wrote:
> Please take another look at the revised change.

Hi Russ,

I think this CL got lost in the shuffle. Could you maybe take a look?

[rsc, 2012-10-21 16:53:42]

*** Submitted as http://code.google.com/p/plan9port/source/detail?r=64a908db98f0
***

plumb.app: accept plumb:foo as alias for foo

R=rsc
CC=plan9port.codebot
http://codereview.appspot.com/5495046

Committer: Russ Cox <rsc@swtch.com>