[PATCH] buitin_config: return postitive status in get_value

From: Nikolai Vladimirov <nikolay@vladimiroff.com>
Date: Jul 28, 2012, 11:42 UTC
Message-ID: <1343475730-18743-1-git-send-email-nikolay@vladimiroff.com>

Returning -1 instead of 1 results in wrong exit status(255) since the output of get_value is passed to exit().

'git config missing_section' should now return proper exit status = 1, as specified by the git config documentation.

Signed-off-by: Nikolai Vladimirov <nikolay@vladimiroff.com>
---
 builtin/config.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/builtin/config.c b/builtin/config.c
index 8cd08da..c262cbb 100644
--- a/builtin/config.c
+++ b/builtin/config.c
@@ -160,7 +160,7 @@ static int show_config(const char *key_, const char *value_, void *cb)
 
 static int get_value(const char *key_, const char *regex_)
 {
-	int ret = -1;
+	int ret = 1;
 	char *global = NULL, *xdg = NULL, *repo_config = NULL;
 	const char *system_wide = NULL, *local;
 	struct config_include_data inc = CONFIG_INCLUDE_INIT;

-- 
1.7.11.2