From: Jeff King Date: Mon, 15 Feb 2016 17:42:18 GMT Subject: Re: [PATCH v4 2/3] config: add 'type' to config_source struct that identifies config type 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: > From: Lars Schneider > > 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 > --- > 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