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

Lukas Slebodnik lslebodn at redhat.com
Tue Sep 1 04:25:43 UTC 2015


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


More information about the sssd-devel mailing list