You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by bn...@apache.org on 2023/11/21 15:21:52 UTC

(trafficserver-ci) branch main updated: Fix the RAT CI to check for failures (#296)

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

bneradt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/trafficserver-ci.git


The following commit(s) were added to refs/heads/main by this push:
     new 73e2bc7  Fix the RAT CI to check for failures (#296)
73e2bc7 is described below

commit 73e2bc7d750f171fb1a516ae4fa7bf4b39f44dd5
Author: Brian Neradt <br...@gmail.com>
AuthorDate: Tue Nov 21 09:21:47 2023 -0600

    Fix the RAT CI to check for failures (#296)
---
 jenkins/github/rat.pipeline | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/jenkins/github/rat.pipeline b/jenkins/github/rat.pipeline
index 741ae0c..80650f3 100644
--- a/jenkins/github/rat.pipeline
+++ b/jenkins/github/rat.pipeline
@@ -48,7 +48,7 @@ pipeline {
                         if [ -d cmake ]
                         then
                             cmake -B build
-                            cmake --build build --target rat
+                            cmake --build build --target rat | tee RAT.txt
                         else
                             # Pre 10.x branches only support autotools.
                             autoreconf -if && ./configure
@@ -56,10 +56,11 @@ pipeline {
 
                             # Run RAT
                             make rat | tee RAT.txt
-
-                            # Mark as failed if there are any unknown licenses
-                            grep '^0 Unknown Licenses' RAT.txt > /dev/null || exit -1
                         fi
+
+                        # Mark as failed if there are any unknown licenses
+                        grep '^0 Unknown Licenses' RAT.txt > /dev/null || exit -1
+
                         exit 0
                     '''
                 }