[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: Fixes to make compile under OpenServer 5 (ITS#972)



At 12:48 AM 1/19/01 +0000, oazmon@telsoft-solutions.com wrote:
>Full_Name: Omer Azmon
>Version: OpenLDAP 2.0.7
>OS: OpenServer 5.0.5
>URL: 
>Submission from: (NULL) (63.193.150.91)
>
>
>1. The following change was done as when the SQL backend was disabled from
>compile the SLAPD_LIBS value became 'SLAPD_LIBS = \' messing up the compile.

IIRC, this change has already been made to HEAD and OPENLDAP_REL_ENG_2.


>2. The following two files were changed as in SCO OpenServer 5 the variable
>MAXPATHLEN was removed from stdlib.h

Please determine were MAXPATHLEN or equivalent is defined
so that <ac/param.h> can be updated.

>3. The following file was changed as in SCO OpenServer 5 complied with
>pthreads the fgets returns upon thread switch and so never reads the
>response from the backend.

That's very bad as any early return of fgets() leaves the
stream in undefined state.  back-shell does not support
pthreads anyways due to its use of fork().


>4. The following file was changed as in SCO OpenServer Skunkware version of
>pthreads the value of socklen_t is already defined the pthread.h header
>file.  (NOTE: This possibly is better corrected in ./configure)
>./include/ldap_int_thread.h
>22a23,26
>> #ifdef socklen_t
>> #undef socklen_t
>> #endif

That's a dangerous fix.  The more appropriate fix would be
adjust configure to detect socklen_t.  Please check your
headers to determine whether socklen_t is defined in more
appropriate (socket) headers.

>5. The following two files were changed to make the system call the correct
>function name for Kerberos5 version 1.2.1 (don't know when change occurred)
>./libraries/liblutil/passwd.c
>758c758
><   krb5_free_cred_contents( context, &creds );
>---
>>   krb5_free_creds( context, &creds );

Such changes should be conditional upon the flavor (Heimdal v MIT)
and version of Kerberos in use using autoconf.

>6. The following change was made as the library name is different in
>Kerberos5 release 1.2.1 (don't know when change occurred)

This should also be done via configure.