[SSSD] [PATCH] MAN: Fix a conversion of seconds to hours

Jakub Hrozek jhrozek at redhat.com
Mon Sep 1 14:44:01 UTC 2014


On Mon, Sep 01, 2014 at 10:37:42AM +0200, Pavel Reichl wrote:
> 
> On 09/01/2014 07:40 AM, Lukas Slebodnik wrote:
> >ehlo.
> >
> >simple patch is attached.
> Patch applies fine and SSSD builds without problem.
> ACK
> >
> >BTW: Do we want to document that this interval in ipa is 3x shorter?
> >We could also change default value in ipa provider.
> I don't know the reason for different value in case of IPA provider, but I
> would keep it as it is. But I think you are right to document the
> difference.
> >
> >sh$ git grep ldap_purge_cache_timeout -- src/providers/
> >src/providers/ad/ad_opts.h:    { "ldap_purge_cache_timeout", DP_OPT_NUMBER, { .number = 10800 }, NULL_NUMBER },
> >src/providers/ipa/ipa_opts.h:    { "ldap_purge_cache_timeout", DP_OPT_NUMBER, { .number = 3600 }, NULL_NUMBER },

I would prefer to fix the default to match the man page. But in a
separate patch, please 

> >src/providers/ldap/ldap_opts.h:    { "ldap_purge_cache_timeout", DP_OPT_NUMBER, { .number = 10800 }, NULL_NUMBER },
> >
> >LS
> >
> >
> >_______________________________________________
> >sssd-devel mailing list
> >sssd-devel at lists.fedorahosted.org
> >https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> 

> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel




More information about the sssd-devel mailing list