Re: [PATCH v4 2/3] config: add 'type' to config_source struct that identifies config type

From: Jeff King <peff@peff.net>
Date: Feb 15, 2016, 17:42 UTC
Message-ID: <20160215174218.GB26443@sigill.intra.peff.net>
In-Reply-To: <1455531466-16617-3-git-send-email-larsxschneider@gmail.com>

On Mon, Feb 15, 2016 at 11:17:45AM +0100, larsxschneider@gmail.com wrote:

Show 13 quoted lines

> From: Lars Schneider <larsxschneider@gmail.com>
> 
> Use the config type to print more detailed error messages that inform
> the user about the origin of a config error (file, stdin, blob).
> 
> Signed-off-by: Lars Schneider <larsxschneider@gmail.com>
> ---
>  cache.h                |  6 ++++--
>  config.c               | 36 +++++++++++++++++++++++++-----------
>  submodule-config.c     |  4 ++--
>  t/t1300-repo-config.sh |  8 +++++++-
>  t/t1308-config-set.sh  |  4 ++--
>  5 files changed, 40 insertions(+), 18 deletions(-)

Looks good to me.

-Peff