From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 26 Sep 2011 21:12:08 GMT
Subject: Re: config-file includes
Message-ID: <7vmxdr100n.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20110926200553.GA492@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

> We could allow arbitrary shell code like:
>
>   [include-if "test `uname -s` -eq Darwin"]
>
> Very flexible, though it makes me think we are getting a little
> overboard.  And it's an extra shell invocation whenever we read the
> config, which is ugly.

Let's not go there. It is not even like we only ever call git_config()
once in the lifetime of a process.