On Thu, Jun 13, 2013 at 12:20:31PM -0400, Pavel Brezina wrote:
>
>
> ----- Original Message -----
>> From: "Jakub Hrozek" <jhrozek(a)redhat.com>
>> To: sssd-devel(a)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(a)/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?
Just the hardcoded path.