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

Re: Reminder: $EDITOR with spaces?

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2002-04-29 00:12:40 CEST

Alan Shutko <ats@acm.org> writes:

> Philip Martin <philip@codematters.co.uk> writes:
>
> > Perhaps we need an APR wrapper for system()?
>
> Sure, if the wrapper knows how to quote filenames for all the shells
> out there, so that operating on files with spaces, returns, colons,
> etc works on all the systems and all the shells out there.

What? The wrapper just passes the raw string to system(). It just
works.

>
> Ok, I'm revising my old proposal. Forget $EDITOR. Don't use it at
> all. It's worthless because people insist on doing stupid stuff like
>
> > EDITOR='LD_LIBRARY_PATH=/some/lib /some/program'
>
> in it.

Works fine with other programs.

> Use a new $SVNEDITOR which is defined as the name of a single
> program. No switches, no environment setting, no tricks. If a user
> wants tricks, they can put it in a script.
>
> Choosing between system() and svn_io_run_cmd() is basically a choice
> between allowing stupid $EDITOR tricks and making sure $EDITOR will
> actually work on all files. Me, I'd take correct behaviour on every
> file over worrying about edge cases whipped up by twisted minds.

A gratuitous incompatibility with other Unix programs is better?

-- 
Philip
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Apr 29 00:13:44 2002

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.