Skip to content

config: drop support for Heimdal - #18928

Closed
bagder wants to merge 3 commits into
masterfrom
bagder/drop-heimdal
Closed

config: drop support for Heimdal#18928
bagder wants to merge 3 commits into
masterfrom
bagder/drop-heimdal

Conversation

@bagder

@bagder bagder commented Oct 8, 2025

Copy link
Copy Markdown
Member

The kerberos5 library Heimdal is one of three GSS libraries curl support. It has a memory leak triggered by the new test in #18917 and the project seems mostly abandoned.

Drop support and steer users to the MIT krb5 or GNU GSS libraries.

The kerberos5 library Heimdal is one of three GSS libraries curl
support. It has a memory leak triggered by the new test in #18917 and
the project seems mostly abandoned.

Drop support and steer users to the MIT krb5 or GNU GSS libraries.
@github-actions github-actions Bot added the build label Oct 8, 2025
@icing

icing commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Can't have memory leaks...😏

@bagder

bagder commented Oct 8, 2025

Copy link
Copy Markdown
Member Author

@vszakats can I ask you to have a look and maybe fix, as I clearely botched cmake here trying to remove heimdal detection?

@vszakats

vszakats commented Oct 8, 2025

Copy link
Copy Markdown
Member

I'll have a look.

@vszakats

vszakats commented Oct 8, 2025

Copy link
Copy Markdown
Member

Having a try at it at: #18932

@vszakats

vszakats commented Oct 8, 2025

Copy link
Copy Markdown
Member

CMake support is ready at the other PR.

Looking at ./configure, it first says ignore, then detects it anyway, and the build completes with Heimdal:

configure: WARNING: incompatible GSS library found but ignored (heimdal?)
checking for pkg-config... (cached) /usr/bin/pkg-config
checking for mit-krb5-gssapi options with pkg-config... no
checking if we can link against GSS-API library... yes

https://github.com/curl/curl/actions/runs/18353983224/job/52281120021?pr=18932#step:27:234

Made it an error via 32869e8

@vszakats vszakats closed this in 8be9a26 Oct 9, 2025
@bagder
bagder deleted the bagder/drop-heimdal branch December 1, 2025 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants