From: Jeff King Date: Wed, 24 Aug 2016 14:17:02 GMT Subject: Re: [PATCH v4] config: add conditional include Message-ID: <20160824141702.rszccucv62rqrcxs@sigill.intra.peff.net> In-Reply-To: On Wed, Aug 24, 2016 at 02:44:29PM +0200, Jakub Narębski wrote: > 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