Hi Amos,

Please find attached more patches for better heimdal support as new heimdal version have gssapi_krb5 header files which were in the past only in older MIT releases available.

  1) kerberos_ldap_group_header.diff fixes the gssapi_krb5 header issue

Secondly to use kerberos_ldap_group on freebsd config.test has to be changed as freebsd installs additional packages in /usr/local. The following patch addresses this

  2) kerberos_ldap_group_config.diff

Thirdly on freebsd 7 the krb5.h file does not work with C++. This patch checks for it

3) kerberos_ldap_group_freebsd.diff (it includes the kerberos_ldap_group_header.diff patch)

Regards
Markus

"Markus Moeller" <[email protected]> wrote in message news:[email protected]...
Here is an update using only #if  / #elif  and changed the order a bit.

Markus

"Amos Jeffries" <[email protected]> wrote in message
news:[email protected]...
On Sun, 14 Nov 2010 18:37:39 -0000, "Markus Moeller"
<[email protected]> wrote:
Hi

 I noticed that the trunk does not compile on FreeBSD with Heimdal.
Here
are two patches against the trunk.

Markus

These appear to be reversions of the file-based inclusions. Would it not
be better just to add:

+#if HAVE_GSSAPI_GSSAPI_EXT_H
+#include <gssapi/gssapi_ext.h>
+#endif

to the end of the include lists?

Also, it is adding quite a messy mix of ifdef and if defined(). Please
just use #if / #elif either way.

Amos



Attachment: kerberos_ldap_group_config.diff
Description: Binary data

Attachment: kerberos_ldap_group_freebsd.diff
Description: Binary data

Attachment: kerberos_ldap_group_header.diff
Description: Binary data

Reply via email to