Re: [RFC/PATCH 1/2] config: Add safe-include directive
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Oct 3, 2014, 05:34 UTC
- Message-ID
- <CAPc5daVYwLMiZzk5Dmm4v4etUhxjw3-ZjWvKEmc=RJ_mh9LFDA@mail.gmail.com>
- In-Reply-To
- <CAPc5daV_txE9NrwvH5VWhXK+UmE7Avy8R2QaZaX0SsTC_+TU-A@mail.gmail.com>
On Thu, Oct 2, 2014 at 10:27 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 6 quoted lines
> On Thu, Oct 2, 2014 at 6:37 PM, Rasmus Villemoes <rv@rasmusvillemoes.dk> wrote: >> This adds a variant of the include directive, where only certain >> config variables in the included files are honoured. The set of >> honoured variables consists of those the user has mentioned in a >> safe-include.whitelist directive, along with a small set of git.git >> blessed ones.
Another design decision we would need to make is if it should be allowed for a safe-included file to use safe-include directive to include other files. Offhand I do not think of a reason we absolutely need to support it, but there may be an interesting workflow enabled if we did so. I dunno.