Re: Two patches for better heimdal support

Sorry I have another addition for config.test

=== modified file 'helpers/external_acl/kerberos_ldap_group/config.test'
--- helpers/external_acl/kerberos_ldap_group/config.test 2010-12-05
00:25:25 +0000
+++ helpers/external_acl/kerberos_ldap_group/config.test 2010-12-06
21:09:15 +0000
@@ -9,6 +9,9 @@
                if [ -f /usr/lib/libsasl.la -o -f /usr/lib/libsasl2.la ];
then
                        exit 0
                fi
+ if [ -f /usr/lib/libsasl.so -o -f /usr/lib/libsasl2.so ];
then
+ exit 0
+ fi
                if [ -f /usr/local/lib/libsasl.so -o -f
/usr/local/lib/libsasl2.so ]; then
                        exit 0
                fi

Markus

"Markus Moeller" <huaraz_at_moeller.plus.com> wrote in message
news:idde7p$8a5$1_at_dough.gmane.org...
> 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
>>>
>>>
>>
>
Received on Mon Dec 06 2010 - 21:12:52 MST