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

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