Re: longfin missing gssapi_ext.h

[Jonathan S. Katz]

On 4/12/23 10:47 AM, Stephen Frost wrote:

Greetings,

* Jonathan S. Katz (jk...@postgresql.org) wrote:

On 4/12/23 10:33 AM, Stephen Frost wrote:

* Tom Lane (t...@sss.pgh.pa.us) wrote:

Stephen Frost <sfr...@snowman.net> writes:

Understood.  Please find attached the updated patch with changes to the
commit message to indicate that we now require MIT Kerberos, an
additional explicit check for gssapi_ext.h in configure.ac/configure,
along with updated documentation explicitly saying we require MIT
Kerberos for GSSAPI support.

Um ... could you package this as a straight un-revert of the
previous commit, then a delta patch?  Would be easier to review.

Sure, reworked that way and attached.

Docs read well. A few questions/commenets:

* On [1] -- do we want to add a note that it's not just Kerberos, but MIT
Kerberos?

Yes, makes sense, updated.

* On [2] -- we mention "kadmin tool of MIT-compatible Kerberos 5" which is
AIUI is still technically correct, but do we want to drop the "-compatible?"
(precedent in [3])

Yup, cleaned that up also.

Updated patch set attached.

Thanks! I'll sign off on the docs portion.



Thanks,

Jonathan

The meson build code looks good to me (I just compared it to what already exists). Similar comment to the autoconf code.

OpenPGP_signature
Description: OpenPGP digital signature