On Sat, Dec 12, 2015 at 5:57 PM, Edmundo Carmona Antoranz <eantoranz@xxxxxxxxx> wrote: > + if (incremental || (output_option & OUTPUT_PORCELAIN)) { > + if (show_progress > 0) > + die("--progress can't be used with --incremental or porcelain formats"); > + show_progress = 0; > + } else if (show_progress < 0) > + show_progress = isatty(2); > + I think I took care of all the comments from Eric gracefully. Hope that block is ok for detection of mutually exclusivity related to --progress. Cheers! -- 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