Re: [PATCH 1/4] docs/config: clarify include/includeIf relationship

2017-05-12 Thread Jeff King
On Thu, May 11, 2017 at 05:47:28PM +0100, Ramsay Jones wrote: > > 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. > >

Re: [PATCH 1/4] docs/config: clarify include/includeIf relationship

2017-05-11 Thread Ramsay Jones
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 nece

[PATCH 1/4] docs/config: clarify include/includeIf relationship

2017-05-11 Thread Jeff King
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. S