From e7b082a4118d15e901f4f44b067394b314a0ae6a Mon Sep 17 00:00:00 2001 From: Nazri Ramliy Date: Fri, 2 Jul 2010 21:55:06 -0500 Subject: grep -O: Do not pass color sequences as filenames to pager With a .gitconfig like this: [color] ui = auto [color "grep"] filename = magenta if stdout is a terminal, the grep machinery will output the color sequence \e[36m before each filename in its output. In the case of "git grep -O foo", output is argv for the pager. Disable color when calling the grep machinery in this case. Signed-off-by: Nazri Ramliy Signed-off-by: Jonathan Nieder Signed-off-by: Junio C Hamano --- builtin/grep.c | 1 + 1 file changed, 1 insertion(+) (limited to 'builtin') diff --git a/builtin/grep.c b/builtin/grep.c index 4b8ddbe0d..7a9427d35 100644 --- a/builtin/grep.c +++ b/builtin/grep.c @@ -997,6 +997,7 @@ int cmd_grep(int argc, const char **argv, const char *prefix) if (show_in_pager == default_pager) show_in_pager = git_pager(1); if (show_in_pager) { + opt.color = 0; opt.name_only = 1; opt.null_following_name = 1; opt.output_priv = &path_list; -- cgit v1.2.1