Re: commit: ldap/servers/slapd/back-bdb init.c

[Pierangelo Masarati <ando@sys-net.it>]

Hallvard B Furuseth wrote:

> ando@OpenLDAP.org writes:
>  
>
> > Modified Files:
> > 	init.c  1.223 -> 1.224
> > Log Message:
> > silence warning(s)
> >
> > +#ifdef HAVE_UTIME_H
> > +#include <utime.h>
> > +#endif /* HAVE_UTIME_H */
> >    
>
> The Linux doc says utimes.h requires sys/types.h.
> So I think this should be
>
> #ifdef HAVE_SYS_TYPES_H
> #include <sys/types.h>
> #endif /* HAVE_SYS_TYPES_H */
> #ifdef HAVE_UTIME_H
> #include <utime.h>
> #endif /* HAVE_UTIME_H */
>
> or <ac/utime.h>.
>  
Right.  I vote for the former by now, since utime.h is needed only 
there.  In case, we can add "ac/utime.h" later.

p.


   SysNet - via Dossi,8 27100 Pavia Tel: +390382573859 Fax: +390382476497