On Tue, 19 Dec 2006, David James wrote:
> On 12/19/06, Mark Phippard <email@example.com> wrote:
> >On 12/19/06, Daniel Rall <firstname.lastname@example.org> 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
> > m_configDir = (configDir == NULL ? "" : configDir);
> >I now see that the issue is that this code is AFTER the svn library call
> >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
The Subversion core libraries handle this default properly on trunk,
but IIRC, not gracefully enough in the 1.4.x and earlier series.
Received on Wed Dec 20 07:31:59 2006
- application/pgp-signature attachment: stored