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

Re: svn commit: rev 5754 - in trunk: . subversion/include subversion/libsvn_ra subversion/libsvn_ra_svn subversion/libsvn_subr subversion/svnserve

From: Branko ╚ibej <brane_at_xbc.nu>
Date: 2003-04-29 07:56:11 CEST

Greg Hudson wrote:

>On Tue, 2003-04-29 at 01:27, Branko ╚ibej wrote:
>>Wouldn't it be better if we just used the config instead of an
>>environment variable? So, the "ssh" scheme would run ssh by default, or
>>whatever is defined in the config file. That would work out of the box
>>99% of the time.
>epg specifically wanted support for an environment variable, and it
>seemed like reasonably functionality to me. There are situations where
>it's more convenient to use one--for instance, a script, you could
>conceivably want to run "SVN_SSH=something svn import blah". And
>there's also precedent in CVS.
There are lots of precedents in CVS, including the one about directories
not being versioned. :-)

>Why do you find environment variables evil?
They're not evil; but I prefer to avoid controlling program behaviour
with environment variables, unless there's a very good reason. It makes
debugging more complicated, and the probability of collisions with other
programs is greater with environment variables than with config files.

In this case it would IMHO be better to introduce a mechanism to
override config file options on the command line.

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 Tue Apr 29 07:56:52 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.