Re: Patch to add Heimdal kerberos support - Mailing list pgsql-patches

From Bill Studenmund
Subject Re: Patch to add Heimdal kerberos support
Date
Msg-id Pine.NEB.4.33.0111141403550.29259-100000@vespasia.home-net.internetconnect.net
Whole thread Raw
In response to Re: Patch to add Heimdal kerberos support  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: Patch to add Heimdal kerberos support  (Peter Eisentraut <peter_e@gmx.net>)
List pgsql-patches
On Wed, 14 Nov 2001, Peter Eisentraut wrote:

> Tom Lane writes:
>
> > Couldn't we do this in a way that doesn't require a user configure switch?
>
> We've seen a Heimdal patch before that had the same source changes but
> solved the configury in some other ugly way.  It's not trivial to write a

There is another test we can do. There is a variable which contains the
Heimdal version number, '__heimdal_version'. If that symbol is present,
then we have Heimdal, and if it's not, we have MIT.

So something like 'nm /usr/lib/libkrb5.a | grep __heimdal_version' is a
good test.

> macro that checks for struct members.  However, that other patch didn't
> need to link in all these extra libraries as this patch tries; it only
> used -lkrb5.  There is only one Heimdal I trust?

Yes, there is only one Heimdal. However some linkers are better with
inter-library dependencies than others. NetBSD 1.5, the OS I've used for
developing this, doesn't handle them automagically. So you have to
explicitly include them. I think -current and 1.5.2 have this fixed, I'm
just stuck in the stone age. ;-)

Something like --with-extralibs would suffice to cover this. But the extra
libs would need to be used for compiling the kerberos test cases. :-)

I would appreciate help with this. I think for now the difference in API
needs to be supported, but everything else about the config setup can be
changed as desired. :-)

Take care,

Bill


pgsql-patches by date:

Previous
From: "Zeugswetter Andreas SB SD"
Date:
Subject: Re: [HACKERS] Open Items (was: RE: [HACKERS] Beta going well)
Next
From: Bill Studenmund
Date:
Subject: Re: Patch to add Heimdal kerberos support