Mike Percy has posted comments on this change.

Change subject: Allow to force-override color diagnostics
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3509/1/CMakeLists.txt
File CMakeLists.txt:

Line 230:   # We also provide a manual override as 
-DKUDU_FORCE_COLOR_DIAGNOSTICS=1.
> Curious: what's the motivation for the override? How is the existing heuris
I looked into it a little bit. I'm not exactly sure why it's failing for me. I 
wonder if it has to do with bash aliases and functions. But AFAICT I am not 
redirecting stderr.


-- 
To view, visit http://gerrit.cloudera.org:8080/3509
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c19651df6d74753b0d4e8ac85ff98326dee7b10
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-HasComments: Yes

Reply via email to