Re: (ITS#3978) Duplicate back-bdb/alock.c symbols

[hyc@symas.com]

h.b.furuseth@usit.uio.no wrote:
> Full_Name: Hallvard B Furuseth
> Version: HEAD
> OS: Tru64
> URL: 
> Submission from: (NULL) (129.240.186.42)
> Submitted by: hallvard
>
>
> ar complains that the symbols in back-bdb/alock.c are defined twice.
> Either alock needs to be moved into slapd,
> or the alock symbols need bdb_ and hdb_ prefixes,
> or alock.c after #include alock.h can be wrapped in something like:
>
> /* Equal in back-bdb and back-hdb, only compile once if both are present */
> #define _BACK_BDB_H_
> #include "back-bdb.h"		/* Get BDB_HIER for back-hdb/back-bdb.h */
> #if !(defined(BDB_HIER) && defined(SLAPD_BDB) && SLAPD_BDB == SLAPD_MOD_STATIC)
> ...
> #endif
>
> (Not sure ld can complain on some systems in last case if at least bdb is
> a module and both variants are loaded.)
>   
Perhaps we should move it into slapd. We could even call it from 
back-ldbm; even though we cannot recover from errors in ldbm, we can at 
least detect unclean shutdowns and alert the administrator.

-- 
  -- Howard Chu
  Chief Architect, Symas Corp.  http://www.symas.com
  Director, Highland Sun        http://highlandsun.com/hyc
  OpenLDAP Core Team            http://www.openldap.org/project/