On 12/19/06, Mark Phippard <markphip@gmail.com> wrote:
> On 12/19/06, Daniel Rall <dlr@collab.net> wrote:
>
> > > 2) If the Subversion config area does not exist, then it is created
> > > automatically when the JavaHL library is loaded. However, if the
> > > folder does exist, or has incomplete contents, then no attempt is
> > > made to fill the folder with the correct contents. The way this
> > > manifests is that when the user is prompted to accept SSL
> > > credentials, the Accept Permanent does not work because the
> > > auth/svn.ssl.server folder does not exist. I did not test if
> > > password cache has same problem, but I'd expect that it would.
> > >
> > > I tested scenario #2 with the command line, and it created the
> > > necessary folders to store the cache info. JavaHL does not.
> >
> > Mark, is this caused by the bug injected by r21316? Or is this a
> > different problem?
>
> Hopefully it is the same problem. I have no way of knowing.
>
> Wouldn't it be possible to have the code treat "" the same as NULL? I think
> when we committed it, we thought that it did. Probably because line 2872 in
> SVNClient.cpp:
>
> m_configDir = (configDir == NULL ? "" : configDir);
>
> I now see that the issue is that this code is AFTER the svn library call has
> been made. I guess we can live with this if the change is reverted, but I
> am not sure a JVM crash is the lesser of two evils.
If we need to be compatible with Subversion 1.3.x DLLs, why don't we
simply fix JavaHL to set the config directory to "$HOME/.subversion"?
There's nothing wrong with being explicit, and this is the only
reliable way to initialize the config dir which is compatible with
both Subversion 1.3.x and 1.4.x DLLs.
Cheers,
David
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Dec 20 04:27:43 2006