Skip to content

cmake: detect GNU GSS - #15176

Closed
vszakats wants to merge 2 commits into
curl:masterfrom
vszakats:cm-gnu-gss
Closed

cmake: detect GNU GSS#15176
vszakats wants to merge 2 commits into
curl:masterfrom
vszakats:cm-gnu-gss

Conversation

@vszakats

@vszakats vszakats commented Oct 7, 2024

Copy link
Copy Markdown
Member

Fix to set HAVE_GSSGNU when GNU GSS is detected.

Also set the appropriate pkg-config dependency and do version
detection for the GNU GSS flavour.

Tested with pkg-config and partly tested without. The latter case
picks up everything else but, in my env. This is likely not the last
word to implement this detection correctly for all build-cases.

GNU GSS doesn't seem to have a Homebrew formula and building
it locally needs manual tweaks to make finish successfully.

Also move a MIT-specific header detection into to MIT-specific if
branch.


w/o whitespace: https://github.com/curl/curl/pull/15176/files?w=1

@vszakats vszakats closed this in 9e19a57 Oct 8, 2024
@vszakats
vszakats deleted the cm-gnu-gss branch October 8, 2024 09:50
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Fix to set `HAVE_GSSGNU` when GNU GSS is detected.

Also set the appropriate `pkg-config` dependency and do version
detection for the GNU GSS flavour.

Tested with `pkg-config` and partly tested without. The latter case
picks up everything else but, in my env. This is likely not the last
word to implement this detection correctly for all build-cases.

GNU GSS doesn't seem to have a Homebrew formula and building
it locally needs manual tweaks to make finish successfully.

Also move a MIT-specific header detection into to MIT-specific `if`
branch.

Closes curl#15176
vszakats added a commit that referenced this pull request Jun 16, 2026
To match the non-pkg-config path, and also suspected user expectation.

This comes with a small incompatibility in return for more consistency.

Bug: #22052 (comment)
Follow-up to 9e19a57 #15176

Closes #22053
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant