Todd Lipcon has posted comments on this change.

Change subject: Fix up dist_test for C++11, add dist_test support for Jenkins 
builds
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/1791/4/build-support/jenkins/build-and-test.sh
File build-support/jenkins/build-and-test.sh:

Line 163:   BUILD_TYPE=fastdebug
> What prompted this change?  I'm curious because I think we do this elsewher
I got errors in the pass-through case where it was trying to run 
CC=/blah/blah/blah as a command rather than using it to set environment 
variables. I think we've only tested this on systems that have devtoolset, 
whereas my new upstream slave is ubuntu 14


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I816f87ca5d1c6147746cfa3230bfe449c849e561
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to