[mesos] branch master updated: Fixed mesos-tidy to actually log clang-tidy errors on failure. (#429)

[cfnatali]

This is an automated email from the ASF dual-hosted git repository.

cfnatali pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/master by this push:
     new 4ecb61059 Fixed mesos-tidy to actually log clang-tidy errors on 
failure. (#429)
4ecb61059 is described below

commit 4ecb6105925546cae3013c689edb54f74b03fa4b
Author: cf-natali <cf.nat...@gmail.com>
AuthorDate: Tue Apr 26 23:21:28 2022 +0100

    Fixed mesos-tidy to actually log clang-tidy errors on failure. (#429)
    
    Because it uses `set -e`, the `entrypoint.sh` script would exit -
    silently - if `clang-tidy` exited with a non-zero exit status, without
    showing the `clang-tidy` output.
---
 support/mesos-tidy/entrypoint.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/support/mesos-tidy/entrypoint.sh b/support/mesos-tidy/entrypoint.sh
index 269b94d76..f9db6c43d 100755
--- a/support/mesos-tidy/entrypoint.sh
+++ b/support/mesos-tidy/entrypoint.sh
@@ -82,7 +82,7 @@ cat compile_commands.json \
       -extra-arg=-Wno-unknown-warning-option \
       -extra-arg=-Wno-unused-command-line-argument \
       -header-filter="^${SRCDIR}/.*\.hpp$" -checks="${CHECKS}" \
-  1> clang-tidy.log 2> /dev/null
+  1> clang-tidy.log 2> /dev/null || true
 
 # Propagate any errors.
 if test -s clang-tidy.log; then