Re: [PATCH v4] config: add conditional include
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Jul 16, 2016, 16:36 UTC
- Message-ID
- <alpine.DEB.2.20.1607161834420.28832@virtualbox>
- In-Reply-To
- <20160716150835.GA24374@sigill.intra.peff.net>
Hi Peff,
On Sat, 16 Jul 2016, Jeff King wrote:
Show 12 quoted lines
> On Sat, Jul 16, 2016 at 03:30:45PM +0200, Johannes Schindelin wrote: > > > As an alternative solution to your problem, you could of course avoid all > > conditional includes. Simply by adding the include.path settings > > explicitly to the configs that require them. Now, that would make reasoning > > and trouble-shooting simple, wouldn't it? > > > > And the most beautiful aspect of it: no patch needed. > > And you can just "cat" the included files directly into your > .git/config. We don't even need include.path. Or ~/.gitconfig, for that > matter. But sometimes dynamic things are convenient.
Well, apparently you are not convinced by my argument. I thought it was pretty sound, but if you disagree, it cannot have been...
So I'll shut up ;-)
Ciao, Dscho