diff options
| author | Bill Hoffman <bill.hoffman@kitware.com> | 2017-11-30 22:36:43 (GMT) |
|---|---|---|
| committer | Bill Hoffman <bill.hoffman@kitware.com> | 2017-12-01 21:23:45 (GMT) |
| commit | bfcc20343c966ee031426e04c5eab504a0af8e27 (patch) | |
| tree | 681a5655d4c6714e2f7d00d894fdad57a157a6b1 /Source/cmcmd.cxx | |
| parent | 8b5ae1c1d310fc6eb13b2cde5712b4e319f01f3e (diff) | |
| download | CMake-bfcc20343c966ee031426e04c5eab504a0af8e27.zip CMake-bfcc20343c966ee031426e04c5eab504a0af8e27.tar.gz CMake-bfcc20343c966ee031426e04c5eab504a0af8e27.tar.bz2 | |
Update cpplint support to return 0 and mark warnings for CDash.
This commit makes cpplint act like the other compiler mirroring tools. It
will always return 0 even if it reports warnings and will only return non
zero if there is a problem running the command. In addition, it will now
add some extra text to allow CTest to recognize the warnings and report
them correctly to CDash.
Diffstat (limited to 'Source/cmcmd.cxx')
| -rw-r--r-- | Source/cmcmd.cxx | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/Source/cmcmd.cxx b/Source/cmcmd.cxx index 3d9f65a..b78fbe6 100644 --- a/Source/cmcmd.cxx +++ b/Source/cmcmd.cxx @@ -270,10 +270,12 @@ static int HandleCppLint(const std::string& runCmd, << "\n"; return 1; } - + std::cerr << "Warning: cpplint diagnostics:\n"; // Output the output from cpplint to stderr std::cerr << stdOut; - return ret; + // always return 0 so the build can continue as cpplint returns non-zero + // for any warning + return 0; } static int HandleCppCheck(const std::string& runCmd, |