On Tue, 19 Dec 2006, David James wrote:
> 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.
Java does not support environment variables, so there is no way to
access $HOME.
The Subversion core libraries handle this default properly on trunk,
but IIRC, not gracefully enough in the 1.4.x and earlier series.
- application/pgp-signature attachment: stored
Received on Wed Dec 20 07:31:59 2006