Ah somehow reviewboard and precommit hook didn't catch those, thanks Vinod!
Tim On Tue, May 12, 2015 at 11:59 AM, Vinod Kone <[email protected]> wrote: > nm. i committed the fix. > > On Tue, May 12, 2015 at 11:44 AM, Vinod Kone <[email protected]> wrote: >> >> tim, looks like this is related to your recent commits? can you take a >> look? >> >> On Tue, May 12, 2015 at 11:12 AM, Apache Jenkins Server >> <[email protected]> wrote: >>> >>> See <https://builds.apache.org/job/mesos-reviewbot/5699/> >>> >>> ------------------------------------------ >>> [URLTrigger] A change within the response URL invocation (log) >>> [URLTrigger] A change within the response URL invocation (log) >>> Building remotely on H11 (Ubuntu ubuntu) in workspace >>> <https://builds.apache.org/job/mesos-reviewbot/ws/> >>> > git rev-parse --is-inside-work-tree # timeout=10 >>> Fetching changes from the remote Git repository >>> > git config remote.origin.url >>> https://git-wip-us.apache.org/repos/asf/mesos.git # timeout=10 >>> Fetching upstream changes from >>> https://git-wip-us.apache.org/repos/asf/mesos.git >>> > git --version # timeout=10 >>> > git fetch --tags --progress >>> https://git-wip-us.apache.org/repos/asf/mesos.git >>> +refs/heads/*:refs/remotes/origin/* >>> > git rev-parse origin/master^{commit} # timeout=10 >>> Checking out Revision 946f2ec1c7a119baa417126ea5421ed4041184e7 >>> (origin/master) >>> > git config core.sparsecheckout # timeout=10 >>> > git checkout -f 946f2ec1c7a119baa417126ea5421ed4041184e7 >>> > git rev-list 946f2ec1c7a119baa417126ea5421ed4041184e7 # timeout=10 >>> > git tag -a -f -m Jenkins Build #5699 jenkins-mesos-reviewbot-5699 # >>> timeout=10 >>> [mesos-reviewbot] $ /bin/bash -xe /tmp/hudson128198071270900228.sh >>> + export JAVA_HOME=/home/jenkins/tools/java/jdk1.6.0_20-64 >>> + JAVA_HOME=/home/jenkins/tools/java/jdk1.6.0_20-64 >>> + export >>> PATH=/home/jenkins/tools/java/jdk1.6.0_20-64/bin:/home/jenkins/tools/java/latest1.6/bin:/home/jenkins/tools/java/latest1.6/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games >>> + >>> PATH=/home/jenkins/tools/java/jdk1.6.0_20-64/bin:/home/jenkins/tools/java/latest1.6/bin:/home/jenkins/tools/java/latest1.6/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games >>> + export M2_HOME=/home/jenkins/tools/maven/latest >>> + M2_HOME=/home/jenkins/tools/maven/latest >>> + export >>> PATH=/home/jenkins/tools/maven/latest/bin:/home/jenkins/tools/java/jdk1.6.0_20-64/bin:/home/jenkins/tools/java/latest1.6/bin:/home/jenkins/tools/java/latest1.6/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games >>> + >>> PATH=/home/jenkins/tools/maven/latest/bin:/home/jenkins/tools/java/jdk1.6.0_20-64/bin:/home/jenkins/tools/java/latest1.6/bin:/home/jenkins/tools/java/latest1.6/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games >>> + date >>> Tue May 12 18:11:43 UTC 2015 >>> + chmod -R +w 3rdparty bin bootstrap CHANGELOG configure.ac Dockerfile >>> docs Doxyfile ec2 frameworks include LICENSE m4 Makefile.am mesos.pc.in mpi >>> NOTICE README.md src support >>> + git clean -fdx >>> + git reset --hard HEAD >>> HEAD is now at 946f2ec Modifiy gdb scripts error message to check if gdb >>> is installed. >>> + ./support/mesos-style.py >>> Checking 612 files using filter >>> --filter=-,+build/class,+build/deprecated,+build/endif_comment,+readability/todo,+readability/namespace,+runtime/vlog,+whitespace/blank_line,+whitespace/comma,+whitespace/end_of_line,+whitespace/ending_newline,+whitespace/forcolon,+whitespace/indent,+whitespace/line_length,+whitespace/operators,+whitespace/semicolon,+whitespace/tab,+whitespace/todo >>> src/tests/docker_containerizer_tests.cpp:2894: Missing username in TODO; >>> it should look like "// TODO(my_username): Stuff." [readability/todo] [2] >>> src/tests/docker_containerizer_tests.cpp:2998: Missing username in TODO; >>> it should look like "// TODO(my_username): Stuff." [readability/todo] [2] >>> src/tests/docker_containerizer_tests.cpp:3102: Missing username in TODO; >>> it should look like "// TODO(my_username): Stuff." [readability/todo] [2] >>> Total errors found: 3 >>> Build step 'Execute shell' marked build as failure >> >> >
