Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

--with-gssapi with Heimdal is ignored on macOS unless you also pass --with-gssapi-libs #3841

Closed
JCMais opened this issue May 5, 2019 · 4 comments

Comments

@JCMais
Copy link
Contributor

@JCMais JCMais commented May 5, 2019

This happens because of this check:

curl/configure.ac

Lines 1411 to 1416 in c86d635

elif test -z "$GSSAPI_LIB_DIR"; then
case $host in
*-*-darwin*)
LIBS="-lgssapi_krb5 -lresolv $LIBS"
;;
*)

It ignores completely the cases where GSSAPI_ROOT is different than the default (/usr, set here https://github.com/curl/curl/blob/master/configure.ac#L1316)

I discovered that by building a static linked version of libcurl with Heimdal.

The curl binary was being linked with -lgssapi_krb5 instead of -lgssapi which is the library reported by $path_to_heimdal/bin/krb5-config --libs gssapi

For Kerberos (MIT) it works correctly, in reality it fails if we pass --with-gssapi-libs, because it then tries to link against -lgssapi but on Kerberos it's -lgssapi_krb5.

@JCMais JCMais changed the title --with-gssapi with MIT/Heimdal is ignored on macOS unless you also pass --with-gssapi-libs --with-gssapi with Heimdal is ignored on macOS unless you also pass --with-gssapi-libs May 5, 2019
@stale

This comment has been minimized.

Copy link

@stale stale bot commented Nov 1, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 1, 2019
@JCMais

This comment has been minimized.

Copy link
Contributor Author

@JCMais JCMais commented Nov 7, 2019

ty stale bot, but the problem still exists

@stale stale bot removed the stale label Nov 7, 2019
@bagder

This comment has been minimized.

Copy link
Member

@bagder bagder commented Dec 16, 2019

@JCMais any chance you can propose a fix? Otherwise I believe this issue will get mentioned in KNOWN_BUGS and this issue here get closed.

@JCMais

This comment has been minimized.

Copy link
Contributor Author

@JCMais JCMais commented Jan 8, 2020

@bagder tt's unlikely I will be able to propose a fix to this issue.

Feel free to mention it on KNOWN_BUGS and close the issue. 👍

@bagder bagder closed this in beea6f8 Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.