Re: [PATCH 12/13] Build in merge
- From
Miklos Vajna <vmiklos@frugalware.org>
- Date
- Jun 27, 2008, 12:54 UTC
- Message-ID
- <20080627125443.GK29404@genesis.frugalware.org>
- In-Reply-To
- <4864C8F8.9020501@free.fr>
On Fri, Jun 27, 2008 at 01:03:20PM +0200, Olivier Marin <dkr+ml.git@free.fr> wrote:
Show 23 quoted lines
> Absolutely. And also color.ui. The following patch should do the trick: > > diff --git a/builtin-merge.c b/builtin-merge.c > index 98adca5..cf557f7 100644 > --- a/builtin-merge.c > +++ b/builtin-merge.c > @@ -7,6 +7,7 @@ > */ > > #include "cache.h" > +#include "color.h" > #include "parse-options.h" > #include "builtin.h" > #include "run-command.h" > @@ -712,6 +713,9 @@ int cmd_merge(int argc, const char **argv, const char *prefix) > git_config(git_merge_config, NULL); > git_config(git_diff_ui_config, NULL); > > + if (diff_use_color_default == -1) > + diff_use_color_default = git_use_color_default; > + > argc = parse_options(argc, argv, builtin_merge_options, > builtin_merge_usage, 0);
Have you tried this?
First, you need to call git_get_colorbool_config(), second this won't respect diff.color, while my patch does.
Thanks.