apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@lyra.org>
Subject Re: cvs commit: httpd-2.0/srclib/sdbm .cvsignore Makefile.in sdbm.c sdbm.h sdbm_hash.c sdbm_lock.c sdbm_pair.c sdbm_pair.h sdbm_private.h sdbm_tune.h sdbmlib.dsp
Date Fri, 08 Dec 2000 23:13:31 GMT
On Fri, Dec 08, 2000 at 02:28:27PM -0800, rbb@covalent.net wrote:
> 
> > > >   Removed:     srclib/sdbm .cvsignore Makefile.in sdbm.c 
> > > >                sdbm.h sdbm_hash.c
> > > >                sdbm_lock.c sdbm_pair.c sdbm_pair.h 
> > > >                sdbm_private.h
> > > >                sdbm_tune.h sdbmlib.dsp
> > > >   Log:
> > > >   no longer needed. this is picked up from apr-util instead.
> > > 
> > > Warning; anything hiding in a distro folder of a source branch in
> > > apr (such as mm, sdbm) should remain invisible to the apr and
> > > apr-util lib consumers.  That means mod_auth_dbm.c must be hacked
> > > for the apr_dbm mechanism, before we roll.
> > 
> > Huh? sdbm.h is exported by APRUTIL. It is a publically available export. Or,
> > it should be at least.
> 
> Then the header name at the very least needs to change.  As it is now, if
> we install Apache on a machine that already has sdbm, we will break things
> horribly.

Well, that's always been the case :-)

Hmm. I don't think that I'd have a problem renaming SDBM entry points and
the header to apr_sdbm. I'll go put a line into the STATUS file for voting
input.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

Mime
View raw message