Re: [PATCH v3] config: add conditional include
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Jul 13, 2016, 15:57 UTC
- Message-ID
- <CACsJy8DrPPfvp3C=tBL0qBmAwdEAZtXrxPLNW3HgcgJGO1u3QA@mail.gmail.com>
- In-Reply-To
- <vpqa8hmroji.fsf@anie.imag.fr>
On Wed, Jul 13, 2016 at 9:21 AM, Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> wrote:
Show 22 quoted lines
> Nguyễn Thái Ngọc Duy <pclouds@gmail.com> writes: > >> +`gitdir`:: >> + The environment variable `GIT_DIR` must match the following >> + pattern for files to be included. The pattern can contain >> + standard globbing wildcards and two additional ones, `**/` and >> + `/**`, that can match multiple path components. Please refer >> + to linkgit:gitignore[5] for details. For convenience: > > It's unclear to me whether the whole content of GIT_DIR must match, or > whether the pattern should be included (or a be prefix) of $GIT_DIR. > From this text, I read it as "the whole content", but ... > >> + ; include for all repositories inside /path/to/group >> + [include "gitdir:/path/to/group/"] >> + path = /path/to/foo.inc >> + >> + ; include for all repositories inside $HOME/to/group >> + [include "gitdir:~/to/group/"] >> + path = /path/to/foo.inc > > ... here it seems it only has to be a prefix.
I should have written "with two additional ones... and a few exceptions"., One of the bullet point below would say the trailing slash is rewritten to "/**" so it becomes prefix match. If it proves confusing, I will probably just get rid of that.
-- Duy