apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From r..@covalent.net
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:52:26 GMT

> > > > >   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.

This shouldn't require a vote.  What we have now is broken, and we are in
commit then review mode.

Ryan

_______________________________________________________________________________
Ryan Bloom                        	rbb@apache.org
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------


Mime
View raw message