Re: [PATCH v6 1/1] config: add conditional include
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Feb 26, 2017, 03:02 UTC
- Message-ID
- <CACsJy8Bq-P8Aw+f0omndCmQU54n_p7ZwDVDK9APuH_3vXDac2g@mail.gmail.com>
- In-Reply-To
- <29A09E4EDB1F4F4D9E77E67A7A8A33FF@PhilipOakley>
On Sat, Feb 25, 2017 at 5:08 AM, Philip Oakley <philipoakley@iee.org> wrote:
Show 10 quoted lines
>> +Conditional includes >> +~~~~~~~~~~~~~~~~~~~~ >> + >> +You can include one config file from another conditionally by setting > > > On first reading I thought this implied you can only have one `includeIf` > within the config file. > I think it is meant to mean that each `includeIf`could include one other > file, and that users can have multiple `includeIf` lines.
Yes. Not sure how to put it better though (I basically copied the first paragraph from the unconditional include section above, which shares the same confusion). Perhaps just write "the variable can be specified multiple times"? Or "multiple variables include multiple times, the last variable does not override the previous ones"?
-- Duy