Re: [PATCH] fixes svn's chdir before invoking $EDITOR for commit log message

Alexis Huxley <ahuxley@gmx.net> writes:
> I submitted this patch at the beginning of the week, but I
> think it got ignored - presumably 'cos I hadn't read 'HACKING'
> all the way to the end ;-)

No, just in the pending patches list and no one's happened to grab it
yet (my personal excuse is: been busy with 0.14.2). But thanks for
resubmitting.

> Is '<envvar>=<value> <command>' portable?

Nope. :-)

> (Rather than looking at how to integrate a pseudo-editor into the
> test directories/PATH/etc, I thought I might invoke
>
> EDITOR="python -c 'import os; print os.getcwd()'" svn commit
>
> but perhaps that only works on Unix-like OSs.)
>
> Can anyone advise? Thanks! Anyway, here's the patch ... be gentle ;-)

Why not just make a single python program to be the editor? It gets
invoked on the temp msg file name, like any $EDITOR, and it just puts
some hardcoded content in there, and exits with success. That's
portable.

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Aug 23 19:40:23 2002