You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@yetus.apache.org by aa...@apache.org on 2020/02/20 05:06:59 UTC

[yetus] branch master updated: YETUS-941. Wrong URL in the checkstyle report. (#90)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 46cb589  YETUS-941. Wrong URL in the checkstyle report. (#90)
46cb589 is described below

commit 46cb58913f05b0fa257d923681268c03ff504f84
Author: Akira Ajisaka <aa...@apache.org>
AuthorDate: Thu Feb 20 14:06:49 2020 +0900

    YETUS-941. Wrong URL in the checkstyle report. (#90)
    
    Signed-off-by: Allen Wittenauer <aw...@apache.org>
---
 precommit/src/main/shell/test-patch.d/checkstyle.sh | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/precommit/src/main/shell/test-patch.d/checkstyle.sh b/precommit/src/main/shell/test-patch.d/checkstyle.sh
index 270f265..3260482 100755
--- a/precommit/src/main/shell/test-patch.d/checkstyle.sh
+++ b/precommit/src/main/shell/test-patch.d/checkstyle.sh
@@ -155,6 +155,7 @@ function checkstyle_runner
   declare savestop
   declare output
   declare logfile
+  declare logfilename
   declare modulesuffix
   declare cmd
   declare line
@@ -267,10 +268,11 @@ function checkstyle_runner
       modulesuffix=root
     fi
 
+    logfilename=$(basename "${logfile}")
     if [[ ${cmdresult} == 0 ]]; then
-      module_status ${i} +1 "${logfile}" "${BUILDMODEMSG} passed checkstyle in ${modulesuffix}"
+      module_status ${i} +1 "${logfilename}" "${BUILDMODEMSG} passed checkstyle in ${modulesuffix}"
     else
-      module_status ${i} -1 "${logfile}" "${BUILDMODEMSG} fails to run checkstyle in ${modulesuffix}"
+      module_status ${i} -1 "${logfilename}" "${BUILDMODEMSG} fails to run checkstyle in ${modulesuffix}"
       ((result = result + 1))
     fi