Re: [PATCH] $EDITOR with spaces

cmpilato@collab.net writes:

> You know what, I reread my algorithm, and I *do* think it works in the
> case you cited:
>
> First, Subversion would try:
>
> cmd="/path/to/foo dir/program arg1 arg2"
> args=( "msg.00000.12345.tmp" )
>
> This would fail. We'd try:
>
> cmd="/path/to/foo dir/program arg1"
> args=( "args2", "msg.00000.12345.tmp" )
>
> This would fail.

No it wouldn't. As well as as the executable file

    /path/to/foo dir/program

which is the one I want to run, I also have one called

    /path/to/foo dir/program arg1

i.e. one with a space in the program name as well as in one of the
directories. Unfortunately this one deletes my working copy :-(

> We'd try:
>
> cmd="/path/to/foo dir/program"
> args=( "arg1", "arg2", "msg.00000.12345.tmp" )

I'd never get this far.

>
> This *should* succeed, unless APR or the OS simply doesn't allow it.
> And sure, there are probably ways to break this algorithm, but you can

Yup.

> only do so much (I'm thinking about the whole "how to remove a file
> whose name is `-rf' on Unix" fun...)

-- 
Philip
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org