From: Ramsay Jones <ramsay@ramsayjones.plus.com>
Date: Thu, 11 May 2017 16:47:28 GMT
Subject: Re: [PATCH 1/4] docs/config: clarify include/includeIf relationship
Message-ID: <0013fb73-12f2-a365-963e-ed517ea53dc7@ramsayjones.plus.com>
In-Reply-To: <20170511091047.2dw52pnriulhjky3@sigill.intra.peff.net>



On 11/05/17 10:10, Jeff King wrote:
> The "includeIf" directives behave exactly like include ones,
> except they only kick in when the conditional is true. That
> was mentioned in the "conditional" section, but let's make
> it more clear for the whole "includes" section, since people
> don't necessarily read the documentation top to bottom.
> 
> Signed-off-by: Jeff King <peff@peff.net>
> ---
>  Documentation/config.txt | 17 +++++++++++------
>  1 file changed, 11 insertions(+), 6 deletions(-)
> 
> diff --git a/Documentation/config.txt b/Documentation/config.txt
> index 475e874d5..d5a453ed3 100644
> --- a/Documentation/config.txt
> +++ b/Documentation/config.txt
> @@ -79,14 +79,20 @@ escape sequences) are invalid.
>  Includes
>  ~~~~~~~~
>  
> +The `include` and `includeIf` sections allow you include config

s/you include/you to include/

ATB,
Ramsay Jones