Skip to content

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Oct 10, 2025

Previous minimum was: 1.2.4 (2002-02-28)

  • assume gssapi/gssapi.h header for MIT Kerberos.

    Drop logic detecting this header, and drop alternate logic including
    a bare "gssapi.h". Bare gssapi.h is Heimdal-specific. MIT Kerberos
    added support for it for Heimdal compatibility on 2006-11-09,
    redirecting to gssapi/gssapi.h. MIT Kerberos supported the latter
    header in the 1990s already.

    Ref: 40e1a01 (2008-03-06)
    Ref: krb5/krb5@d119352 (2006-11-09)

  • configure.ac: stop using HAVE_GSSAPI_GSSAPI_H.

    Added in 2010 to support "ancient distros such as RHEL-3" where
    gssapi/gssapi_krb5.h did not include gssapi/gssapi.h.

    MIT Kerberos includes it since commit:
    krb5/krb5@d9e959e (2003-03-06)
    Released in 1.3 (2003-07-08).

    Bump minimum required version to avoid this issue.

    Reverts cca192e (2010-04-16)

Ref: https://web.mit.edu/kerberos/dist/historic.html
Ref: https://sources.debian.org/src/krb5/


@github-actions github-actions bot added the CI Continuous Integration label Oct 10, 2025
@vszakats vszakats changed the title kerberos: bump minimum to 1.3 (2003-07-08), drop legacy cruft kerberos: bump minimum to 1.3 (2003-07-08), drop legacy logic Oct 10, 2025
Drop logic detecting this header, and drop alternate logic including
a bare "gssapi.h". Bare `gssapi.h` is Heimdal-specific. MIT Kerberos
added support for it for Heimdal compatibility on 2006-11-09,
redirecting to `gssapi/gssapi.h`. MIT Kerberos supported the latter
header in the 1990s already.

Ref: 40e1a01 (2008-03-06)
Ref: krb5/krb5@d119352 (2006-11-09)
Added in 2010 to support "ancient distros such as RHEL-3".
Where `gssapi/gssapi_krb5.h` did not include `gssapi/gssapi.h`.
Probably not an issue with supported distros.

MIT Kerberos did the same since commit:
krb5/krb5@d9e959e (2003-03-06)
Released in 1.3.

Heimdal included the main header from its initial commit in:
heimdal/heimdal@bef2219 (2006-10-08)

Reverting cca192e (2010-04-16)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration

Development

Successfully merging this pull request may close these issues.

1 participant