[SSSD] [DING-LIBS] [INI] [PATCH] Fix comment in ini_augment_ut.c

Jakub Hrozek jhrozek at redhat.com
Tue Sep 1 06:41:19 UTC 2015


On Tue, Sep 01, 2015 at 06:25:43AM +0200, Lukas Slebodnik wrote:
> On (31/08/15 13:30), Robbie Harwood wrote:
> >In the unit test for ini_config_augment(), there is an incorrect comment
> >on the `patterns` field.  For more information, see
> >https://fedorahosted.org/sssd/ticket/2769
> 
> >From 8e5d5769095edc0ebd84036b68b0f567959bb0d8 Mon Sep 17 00:00:00 2001
> >From: "Robbie Harwood (frozencemetery)" <rharwood at redhat.com>
> >Date: Mon, 31 Aug 2015 13:13:32 -0400
> >Subject: [PATCH] Fix comment in ini_augment_ut.c
> >
> >Resolves: https://fedorahosted.org/sssd/ticket/2769
> >Signed-off-by: Robbie Harwood (frozencemetery) <rharwood at redhat.com>
> >---
> > ini/ini_augment_ut.c | 5 ++---
> > 1 file changed, 2 insertions(+), 3 deletions(-)
> >
> >diff --git a/ini/ini_augment_ut.c b/ini/ini_augment_ut.c
> >index 917773a5fb006908db430803c52a0a7051f0b3aa..1122818be721902aaa91d0b5be82e141aed98411 100644
> >--- a/ini/ini_augment_ut.c
> >+++ b/ini/ini_augment_ut.c
> >@@ -211,15 +211,14 @@ static int basic_test(void)
> >     * ending with '.conf' */
> >     const char *patterns[] = { "#",
> >                                "^[^r][a-z]*\\.conf$",
> >                                "^real\\.conf$",
> >                                NULL };
> > 
> >-    /* Match all that do not start with 'r'
> >-    * and end with '.conf' and then match all
> >-    * ending with '.conf' */
> >+    /* Match only the config, monitor, domains, services, and provider
> >+    * sections */
> >     const char *sections[] = { "config",
> >                                "monitor",
> >                                "domains",
> >                                "services",
> >                                "provider",
> >                                NULL };
> 
> Thank you very much for patch.
> 
> ACK
> 
> LS

Pushed to ding-libs master: 4b45e0a12ba5cb883891cfa0ab596c00bc9dac6a


More information about the sssd-devel mailing list