Re: [PATCH v4] config: add conditional include
- From
Jeff King <peff@peff.net>
- Date
- Aug 24, 2016, 14:17 UTC
- Message-ID
- <20160824141702.rszccucv62rqrcxs@sigill.intra.peff.net>
- In-Reply-To
- <febf6605-70e7-4653-1e3e-cddc1882e037@gmail.com>
On Wed, Aug 24, 2016 at 02:44:29PM +0200, Jakub Narębski wrote:
Show 13 quoted lines
> W dniu 24.08.2016 o 11:37, Duy Nguyen pisze: > > > It works for me either way. So I'm going to assume we want > > "[include-if "gitdir-is:..."]", unless people think it needs more > > discussion (then just write something, anything, so I can put it back > > in my backlog) > > A final note: [include "if-gitdir:..."] is inclusive by default > (I think), that is old Git would include it unconditionally, > while [include-if "gitdir-is:..."] is exclusive by default, that > is old Git would ignore it. > > But I might be mistaken.
I don't think this is the case. Conditionals were considered when the include feature was added, and anything except "include.path" is explicitly ignored. So either syntax will behave the same.
-Peff