heimdal locator patch
locator.patch

Fields changed

description: When configuring sssd with heimdal support this error message is displayed:

...
checking for krb5_principal_get_realm... yes
checking krb5/locate_plugin.h usability... no
checking krb5/locate_plugin.h presence... yes
configure: WARNING: krb5/locate_plugin.h: present but cannot be compiled
configure: WARNING: krb5/locate_plugin.h: check for missing prerequisite headers?
configure: WARNING: krb5/locate_plugin.h: see the Autoconf documentation
configure: WARNING: krb5/locate_plugin.h: section "Present But Cannot Be Compiled"
configure: WARNING: krb5/locate_plugin.h: proceeding with the compiler's result
configure: WARNING: ## ------------------------------------------------ ##
configure: WARNING: ## Report this to sssd-devel@lists.fedorahosted.org ##
configure: WARNING: ## ------------------------------------------------ ##
checking for krb5/locate_plugin.h... no
configure: Kerberos locator plugin cannot be build
checking ares.h usability... yes
...

The attached patch (tries to) fix this.

=> When configuring sssd with heimdal support this error message is displayed:

{{{
checking for krb5_principal_get_realm... yes
checking krb5/locate_plugin.h usability... no
checking krb5/locate_plugin.h presence... yes
configure: WARNING: krb5/locate_plugin.h: present but cannot be compiled
configure: WARNING: krb5/locate_plugin.h: check for missing prerequisite headers?
configure: WARNING: krb5/locate_plugin.h: see the Autoconf documentation
configure: WARNING: krb5/locate_plugin.h: section "Present But Cannot Be Compiled"
configure: WARNING: krb5/locate_plugin.h: proceeding with the compiler's result
configure: WARNING: ## ------------------------------------------------ ##
configure: WARNING: ## Report this to sssd-devel@lists.fedorahosted.org ##
configure: WARNING: ## ------------------------------------------------ ##
checking for krb5/locate_plugin.h... no
configure: Kerberos locator plugin cannot be build
checking ares.h usability... yes
}}}

The attached patch (tries to) fix this.

Thank you for the bug report and the patch. I have made two minor corrections related to the fact that at least MIT Kerberos moved its header files into the krb5 subdirectory in include dir. So on my system, {{{/usr/include/krb5.h}}} just includes {{{/usr/include/krb5/krb5.h}}} and the comment seems to suggest that {{{/usr/include/krb5.h}}} might be going away at some point in the future.

I've included an {{{#ifdef-#else-#endif}}} to include the header that was detected on the system and changed the {{{krb5.h}}} include to {{{sss_krb5.h}}} which also contains the #ifdef.

Can you please test the attached patch?

Also, out of curiosity, what OS are you running? On Fedora/RHEL, there is no standalone Heimdal Kerberos, and on my Ubuntu test VM, I was able to configure and compile the SSSD even with Heimdal libraries.

A proposed patch
0001-Fixes-for-better-Heimdal-support.patch

I am using Gentoo as OS.
Without this patch, only the locator plugin failed to configure and was skipped. This did not prevent SSSD from compiling and working with the Heimdal libraries.

With your patch, the krb5-locator-plugin configures and compiles. So ACK and thanks.

Fields changed

owner: somebody => jhrozek
patch: 0 => 1
status: new => assigned

Fixed in:

• master: 7efbb82
• sssd-1.8: e37e4dc

milestone: NEEDS_TRIAGE => SSSD 1.8.4 (LTM)
resolution: => fixed
status: assigned => closed

Fields changed

rhbz: => 0

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/2367

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.