[SSSD] [PATCH] subdomains: touch krb5.conf when creating new domain-realm mappings
Pavel Brezina
pbrezina at redhat.com
Thu Jun 13 16:20:31 UTC 2013
----- Original Message -----
> From: "Jakub Hrozek" <jhrozek at redhat.com>
> To: sssd-devel at lists.fedorahosted.org
> Sent: Thursday, June 13, 2013 5:14:13 PM
> Subject: Re: [SSSD] [PATCH] subdomains: touch krb5.conf when creating new domain-realm mappings
>
> On Thu, Jun 13, 2013 at 03:48:37PM +0200, Pavel Březina wrote:
> > https://fedorahosted.org/sssd/ticket/1815
>
> > +errno_t sss_krb5_touch_config()
> > +{
> > + const char *config = NULL;
> > + errno_t ret;
> > +
> > + config = getenv("KRB5_CONFIG");
> > + if (config == NULL) {
> > + config = "/etc/krb5.conf";
> > + }
>
> I had a discussion with ab on IRC and the consensus was that we really
> should make the location configurable. So this code works fine, we just
> also need a configure-time macro to set the krb5.conf location. The
> macro should default to @sysconfdir@/krb5.conf
OK. Just to be clear, I should only replace "/etc/krb5.conf" with value from configure or I should remove getenv("KRB5_CONFIG") also?
More information about the sssd-devel
mailing list