Re: [PATCH v4 06/10] format-patch: explicitly switch off color when writing to files

2016-06-25 Thread Johannes Schindelin
Hi Junio, On Fri, 24 Jun 2016, Junio C Hamano wrote: > Johannes Schindelin writes: > > > We rely on the auto-detection ("is stdout a terminal?") to determine > > whether to use color in the output of format-patch or not. That > > happens to work because we freopen() stdout when redirecting the

Re: [PATCH v4 06/10] format-patch: explicitly switch off color when writing to files

2016-06-24 Thread Junio C Hamano
Johannes Schindelin writes: > We rely on the auto-detection ("is stdout a terminal?") to determine > whether to use color in the output of format-patch or not. That happens > to work because we freopen() stdout when redirecting the output to files. > > However, we are about to fix that work-aroun

[PATCH v4 06/10] format-patch: explicitly switch off color when writing to files

2016-06-22 Thread Johannes Schindelin
We rely on the auto-detection ("is stdout a terminal?") to determine whether to use color in the output of format-patch or not. That happens to work because we freopen() stdout when redirecting the output to files. However, we are about to fix that work-around, in which case the auto-detection has