Re: Two patches for better heimdal support

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" <huaraz_at_moeller.plus.com> wrote in message
news:ibpome$psh$1_at_dough.gmane.org...
> Here is an update using only #if / #elif and changed the order a bit.
>
> Markus
>
> "Amos Jeffries" <squid3_at_treenet.co.nz> wrote in message
> news:104be24899d2c3a232288ea0fa5a7fc4_at_mail.treenet.co.nz...
>> On Sun, 14 Nov 2010 18:37:39 -0000, "Markus Moeller"
>> <huaraz_at_moeller.plus.com> 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
>>
>>
>