[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Fix compile when DSOs not available

From: Daniel Rall <dlr_at_collab.net>
Date: 2006-09-13 00:31:31 CEST

On Tue, 12 Sep 2006, Alex Waugh wrote:

> "Garrett Rooney" <rooneg@electricjellyfish.net> wrote:
> > On 9/7/06, Alex Waugh <alex@alexwaugh.com> wrote:
> > > Hi
> > >
> > > On platforms where APR_HAS_DSO is not defined, Subversion no longer
> > > compiles. This patch fixes it.
> >
> > Interesting. What platform is this, just out of curiosity?
> > I'm not sure if the patch is entirely correct... Should we really
> > just not have svn_dso_load, or should we just make it return an
> > APR_ENOTIMPL error?
> The problem is that the apr_dso_handle_t type is not defined unless
> APR_HAS_DSO is set, so the compiler falls over on the function
> prototype as well as the actual code.

Our other uses of svn_dso_load() and apr_dso_handle_t in fs-loader.c
and ra_loader.c already gate it in "#if APR_HAS_DSO" checks. I've
committed Alex's patch in r21473 (with some very minor stylistic
tweaks), and nominated it for backport to the 1.4.x line.

An alternate approach which returns something like APR_ENOTIMPL would
need to further insulate svn_dso.h from APR's #if/#ifdef's.

  • application/pgp-signature attachment: stored
Received on Wed Sep 13 00:33:29 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.