On Sat, Jun 18, 2016 at 6:04 AM, Johannes Schindelin <johannes.schindelin@xxxxxx> wrote: > We just taught the relevant functions to respect the diffopt.file field, > to allow writing somewhere else than stdout. Let's make use of it. > [...] > Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> > --- > diff --git a/builtin/log.c b/builtin/log.c > @@ -1569,6 +1570,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix) > setup_pager(); > > if (output_directory) { > + rev.diffopt.use_color = 0; What is this change about? It doesn't seem to be related to anything else in the patch. > if (use_stdout) > die(_("standard output, or directory, which one?")); > if (mkdir(output_directory, 0777) < 0 && errno != EEXIST) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html