Re: [PATCH V3] checkpatch: Change format of --color argument to --color[=WHEN]

2017-06-07 Thread John Brooks
On Tue, Jun 06, 2017 at 06:50:06PM -0700, Joe Perches wrote: > From: John Brooks <j...@fastquake.com> > > The boolean --color argument did not offer the ability to force colourized > output even if stdout is not a terminal. Change the format of the argument > to the f

Re: [PATCH V3] checkpatch: Change format of --color argument to --color[=WHEN]

2017-06-07 Thread John Brooks
On Tue, Jun 06, 2017 at 06:50:06PM -0700, Joe Perches wrote: > From: John Brooks > > The boolean --color argument did not offer the ability to force colourized > output even if stdout is not a terminal. Change the format of the argument > to the familiar --color[=WHEN] co

Re: [PATCH v2] checkpatch: Change format of --color argument to --color[=WHEN]

2017-06-06 Thread John Brooks
On Tue, Jun 06, 2017 at 12:21:22PM -0700, Joe Perches wrote: > On Tue, 2017-06-06 at 13:07 -0400, John Brooks wrote: > > The boolean --color argument did not offer the ability to force colourized > > output even if stdout is not a terminal. Change the format of the argument >

Re: [PATCH v2] checkpatch: Change format of --color argument to --color[=WHEN]

2017-06-06 Thread John Brooks
On Tue, Jun 06, 2017 at 12:21:22PM -0700, Joe Perches wrote: > On Tue, 2017-06-06 at 13:07 -0400, John Brooks wrote: > > The boolean --color argument did not offer the ability to force colourized > > output even if stdout is not a terminal. Change the format of the argument >

[PATCH v2] checkpatch: Change format of --color argument to --color[=WHEN]

2017-06-06 Thread John Brooks
other args in e.g. ./scripts/checkpatch.pl --color foo.patch Signed-off-by: John Brooks <j...@fastquake.com> --- scripts/checkpatch.pl | 31 +-- 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index

[PATCH v2] checkpatch: Change format of --color argument to --color[=WHEN]

2017-06-06 Thread John Brooks
other args in e.g. ./scripts/checkpatch.pl --color foo.patch Signed-off-by: John Brooks --- scripts/checkpatch.pl | 31 +-- 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 4b9569f..9d03

Re: [PATCH] checkpatch: Change format of --color argument to --color[=WHEN]

2017-06-05 Thread John Brooks
On Mon, Jun 05, 2017 at 04:10:30PM -0700, Joe Perches wrote: > On Mon, 2017-06-05 at 18:27 -0400, John Brooks wrote: > > The boolean --color argument did not offer the ability to force colourized > > output even if stdout is not a terminal. > > OK, but why is colorizing ou

Re: [PATCH] checkpatch: Change format of --color argument to --color[=WHEN]

2017-06-05 Thread John Brooks
On Mon, Jun 05, 2017 at 04:10:30PM -0700, Joe Perches wrote: > On Mon, 2017-06-05 at 18:27 -0400, John Brooks wrote: > > The boolean --color argument did not offer the ability to force colourized > > output even if stdout is not a terminal. > > OK, but why is colorizing ou