You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dubbo.apache.org by hy...@apache.org on 2019/08/04 12:22:04 UTC

[dubbo] branch master updated: Polish license check script (#4736)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 87703ad  Polish license check script (#4736)
87703ad is described below

commit 87703adb16a354456e89724b2ec6dd9e3d02de9c
Author: Huxing Zhang <hu...@gmail.com>
AuthorDate: Sun Aug 4 20:21:50 2019 +0800

    Polish license check script (#4736)
    
    * Polish license check script
    * Remove temp files
---
 licenseCheck.sh | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/licenseCheck.sh b/licenseCheck.sh
index a2d3565..da0aa17 100755
--- a/licenseCheck.sh
+++ b/licenseCheck.sh
@@ -18,9 +18,9 @@ else
     echo "checking whole project"
 fi
 
-echo "Running command: ./mvnw clean package -DskipTests=true -PlicenseCheck $APPEND_ARG"
+echo "Running command: mvn clean package -DskipTests=true -PlicenseCheck $APPEND_ARG"
 
-./mvnw clean package -DskipTests=true -PlicenseCheck $APPEND_ARG
+mvn clean package -DskipTests=true -PlicenseCheck $APPEND_ARG
 
 status=$?
 if [ $status -eq 0 ]; then
@@ -91,7 +91,7 @@ allowLicense=(
 )
 
 #filter allow license
-license_need_check=`cat $TARGET_FILE | grep -v "generated-sources/license/THIRD-PARTY.txt" | grep -v "third-party dependencies" | grep -v $LINE_FLAG`
+license_need_check=`cat $TARGET_FILE | grep -v "generated-sources/license/THIRD-PARTY.txt" | grep -v "third-party dependencies" | grep -v "The project has no dependencies." | grep -v $LINE_FLAG`
 
 for i in "${allowLicense[@]}"; do
     license_need_check=`echo "$license_need_check"|grep -vi "$i"`
@@ -106,3 +106,5 @@ else
     echo "${red}Please check below license${reset}"
     cat license-need-check
 fi
+
+rm -f license-list license-need-check