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

Re: Bug report -- space in env. var. VISUAL causes commits to fail (needs confirmation)

From: Peter Samuelson <peter_at_p12n.org>
Date: Tue, 7 Dec 2010 09:12:57 -0600

[David Dyer-Bennet]
> >> I suppose setting VISUAL="\"/path with spaces/to/editor/binary\"" is the
> >> easiest solution --- it requires no code changes so it will work with
> >> any svn binary out there.
> >
> > Yes, I think that's the best solution.
>
> And, in any case, VISUAL is a public interface, and I wonder how many
> other applications would break if I put that kind of thing into
> VISUAL.

A couple of random tests:

- vipw, vigr work
- visudo does not

I seem to recall the same question coming up about 'man' and 'PAGER'
some years ago. At least the Debian version uses system() and
interprets PAGER='"/with spaces/pager" -with -args' correctly.

-- 
Peter Samuelson | org-tld!p12n!peter | http://p12n.org/
Received on 2010-12-07 16:13:48 CET

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.