Re: A proposed solution for svn admin directory names

Andrew, thanks very much for bringing this up. I was getting ready to
send another email.

Ben, yes, it's #1, but the solution is simple, and you're right, the
con no longer exists. The code functions, passes the tests, I've been
using a build to do my work for the past week. The value of this
change is that it affects all clients built on these libraries, and
gives us quite a flexible solution. At some point, you could
introduce a more complicated process, but for now calling getenv()
from the macro works, is simple, and adds flexibility.

It's a tiny change:

#define SVN_WC_ADM_DIR_NAME ( ( getenv( "SVN_ADM_DIR") ) ? ( getenv(
"SVN_ADM_DIR") ) : (".svn") )

Thanks for taking the time to look at this Ben--I know it's been
hashed and rehashed, but I think this is a simple solution that works.

Jonathan Malek

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jun 29 17:52:42 2005