[SSSD] [DING-LIBS] [INI] [PATCH] Document ini_config_augment alphabetical file processing

Jakub Hrozek jhrozek at redhat.com
Thu Sep 3 08:20:17 UTC 2015


On Wed, Sep 02, 2015 at 06:23:09AM +0200, Lukas Slebodnik wrote:
> On (01/09/15 18:19), Robbie Harwood wrote:
> >The sorted file processing is a useful guarantee to have, and is
> >provided by the current code.  https://fedorahosted.org/sssd/ticket/2771
> 
> >From d11f7814c968cb94d8321e723460391df556e2d6 Mon Sep 17 00:00:00 2001
> >From: Robbie Harwood <rharwood at redhat.com>
> >Date: Mon, 31 Aug 2015 15:14:12 -0400
> >Subject: [PATCH] Document ini_config_augment alphabetical file processing
> >
> >Ticket: https://fedorahosted.org/sssd/ticket/2771
> >Signed-off-by: Robbie Harwood <rharwood at redhat.com>
> >---
> > ini/ini_configobj.h | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> >diff --git a/ini/ini_configobj.h b/ini/ini_configobj.h
> >index 363d9f93021c16f58c6a6e1aefb200c85d0da43b..6f2d6920ea94e4ee14bf2a19db7603c8442d116d 100644
> >--- a/ini/ini_configobj.h
> >+++ b/ini/ini_configobj.h
> >@@ -1005,7 +1005,8 @@ int ini_config_merge(struct ini_cfgobj *first,
> >  *
> >  * Function merges the main configuration file
> >  * with the configuration file snippets
> >- * read from a specified directory.
> >+ * read from a specified directory
> >+ * ordered according to the locale setting.
> >  *
> >  * @param[in]  base_cfg         A configuration object
> >  *                              that will be augmented.
> >-- 
> >2.5.0
> >
> I'm not native speaker but LGTM.
> 
> LS

To me too (and Robbie is a native speaker :-)), so I pushed the patch to
ding-libs master: 34e06f47a1ff54212721dd4f63a660a329fce6d3


More information about the sssd-devel mailing list