Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11457 )

Change subject: clang_tidy_gerrit.py: fix output when no changes found in first 
path
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11457/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11457/1//COMMIT_MSG@12
PS1, Line 12:   Clang output
            :   No relevant changes found.
            :   No relevant changes found.
            :   ... <actual clang-tidy changes>
            :   Traceback (most recent call last):
            :     File "build-support/clang_tidy_gerrit.py", line 209, in 
<module>
            :       parsed = parse_clang_output(clang_output)
            :     File "build-support/clang_tidy_gerrit.py", line 103, in 
parse_clang_output
            :       raise Exception("bad warning: " + w)
            :   Exception: bad warning: No relevant changes found.
            :   No relevant changes found.
Without digging into the clang tidy script, I'm not sure why certain patches 
would yield "No relevant..." errors. What patches would the job fail on before?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I111dff7508f841489ba1625a4ca4b7af92f3d8d0
Gerrit-Change-Number: 11457
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Tue, 18 Sep 2018 04:57:07 +0000
Gerrit-HasComments: Yes

Reply via email to