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 76422da  Make the RAT license check pipeline more precise (#103)
76422da is described below

commit 76422da02d242746ef0b922fcc2c045ec403ade2
Author: Brian Neradt <[email protected]>
AuthorDate: Sat Jun 17 16:28:46 2023 -0500

    Make the RAT license check pipeline more precise (#103)
    
    It turns out that we've been missing the libswoc license failures from
    RAT because there are 30 failures, making the report contain this:
    
    30 Unknown Licenses
    
    Our grep currently sees the '0 Unknown Licenses' substring on this and
    passes. This makes sure that the 0 is a leading zero at the start of the
    line.
    
    We must not merge this until some fix is made for the libswoc licenses
    (either ignore them or update the libswoc licenses to be the more
    verbose versions).
---
 jenkins/github/rat.pipeline | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/jenkins/github/rat.pipeline b/jenkins/github/rat.pipeline
index c28272c..7d8e02a 100644
--- a/jenkins/github/rat.pipeline
+++ b/jenkins/github/rat.pipeline
@@ -51,7 +51,7 @@ pipeline {
                         make rat | tee RAT.txt
 
                         # Mark as failed if there are any unknown licenses
-                        grep '0 Unknown Licenses' RAT.txt > /dev/null || exit 
-1
+                        grep '^0 Unknown Licenses' RAT.txt > /dev/null || exit 
-1
                         exit 0
                     '''
                 }

Reply via email to