[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Specify external editor on command line for "svn propedit"?

From: Branko Čibej <brane_at_xbc.nu>
Date: 2003-03-02 22:46:16 CET

Gareth McCaughan wrote:

>Suppose you have a property on a file that's something other than
>plain text: a thumbnail image, perhaps. Then to edit it you have to
>use "svn propget" and "svn propset"; you can't just use "svn
>propedit".
>
>Imagine that "svn propedit" takes a command-line option called
>"--editor-cmd", which specifies what external editor is to be called
>to edit the property. Then you can say
>
> svn propedit thumbnail whatever-file --editor-cmd image-editor
>
>or some variant thereof that actually works for your image file.
>I claim this would be a good thing, so I've implemented it; the
>attached diffs do what's needed. (Attached because the last
>time I sent a patch my mailer mangled it.)
>
>Comments?
>
It feels like the command-line client is slowly transmogrifying into a
full-blown GUI...but maybe that's just me. Apart from that, I don't see
anything wrong with the idea.

>There was one decision that troubled me. Should the command-line
>option be called "--editor-cmd" (which might be useful for other
>Subversion subcommands in the future), or something more specific
>like "--property-editor-command"?
>
Our policy is to make the option names as generic as possible (and
sane). There's no reason not to use --editor-cmd for other purposes, not
just svn propedit. On that note...

>+ {"editor-cmd", svn_cl__editor_cmd_opt, 1,
>+ "Use \"ARG\" as external editor for all but log msgs"},
>
Why "for all but log msgs?" There's no reason why "svn commit", and
other commands that accept an -m or -F for log messages, shouldn't
accept --editor-cmd, too.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Mar 2 22:47:19 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.