Skip site navigation (1) Skip section navigation (2)

Re: Patch to add Heimdal kerberos support

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Bill Studenmund <wrstuden(at)netbsd(dot)org>,Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-patches(at)postgresql(dot)org
Subject: Re: Patch to add Heimdal kerberos support
Date: 2002-02-22 20:14:29
Message-ID: 200202222014.g1MKET015483@candle.pha.pa.us (view raw or flat)
Thread:
Lists: pgsql-patches
Peter, I assume this is to be applied to 7.3, right?

---------------------------------------------------------------------------

Peter Eisentraut wrote:
> Bill Studenmund writes:
> 
> > The problem I had with AC_SEARCH_LIBS was that if I don't add all of the
> > libraries (-lasn1 -lroken etc.) the test program won't compile, even
> > though the .o compiles fine. Can we make the test only depend on
> > generating a .o?
> 
> That wouldn't tell you a whole lot about the existence of a function.  You
> do need to link to be able to verify that.
> 
> > I'd like to make it a bug fix for 7.2 if we can, so I'd appreciate the
> > help. I can test any new versions of the patch. :-)
> 
> The attached patch should handle the different struct members (at least it
> doesn't break them for me) and it doesn't gratuitously fail on libraries
> that are not really needed.  I haven't done anything about -lasn1 -lroken
> -lcrypto because I'd need to know what functions you need from there.  You
> should be able to get it to work when you configure thus:
> 
> $ LIBS='-lasn1 -lroken -lcrypto' ./configure --with-krb5 ...other options...
> 
> If you can confirm that this patch works I can check it in so it appears
> in whatever beta3+1 is going to be.  However, be aware that we're
> approaching the first release candidate, and since this issue doesn't
> represent a regression from 7.1 it's not going to hold up the release.
> 
> -- 
> Peter Eisentraut   peter_e(at)gmx(dot)net

Content-Description: Patch

[ Attachment, skipping... ]

> 
> ---------------------------(end of broadcast)---------------------------
> TIP 2: you can get off all lists at once with the unregister command
>     (send "unregister YourEmailAddressHere" to majordomo(at)postgresql(dot)org)

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman(at)candle(dot)pha(dot)pa(dot)us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

In response to

Responses

pgsql-patches by date

Next:From: Larry RosenmanDate: 2002-02-22 20:17:01
Subject: Re: IPv6 Support for INET/CIDR types.
Previous:From: Bruce MomjianDate: 2002-02-22 20:06:46
Subject: Re: stupid patch of pg_dumplo

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group