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

[Robbie Harwood]

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

[Lukas Slebodnik]

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(a)redhat.com&gt;
> 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(a)redhat.com&gt;
> ---
>  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

[Jakub Hrozek]

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(a)redhat.com&gt;
>  >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(a)redhat.com&gt;
>  >---
>  > 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