This is an automated email from the ASF dual-hosted git repository.
yihua pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hudi.git
The following commit(s) were added to refs/heads/master by this push:
new aebe3cc53e5 [MINOR] Fix validate_staged_release.sh (#7780)
aebe3cc53e5 is described below
commit aebe3cc53e564cbbb8bcb0f4f4a7bcfd33270e1d
Author: Y Ethan Guo <[email protected]>
AuthorDate: Sat Jan 28 02:32:41 2023 -0800
[MINOR] Fix validate_staged_release.sh (#7780)
This commit fixes `scripts/release/validate_staged_release.sh` to skip
checking `release/release_guide*` for "Binary Files Check" and "Licensing
Check".
---
scripts/release/validate_staged_release.sh | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/scripts/release/validate_staged_release.sh
b/scripts/release/validate_staged_release.sh
index 879583a45d9..641047ee966 100755
--- a/scripts/release/validate_staged_release.sh
+++ b/scripts/release/validate_staged_release.sh
@@ -126,11 +126,11 @@ cd hudi-${ARTIFACT_SUFFIX}
### BEGIN: Binary Files Check
echo "Checking for binary files in source release"
-numBinaryFiles=`find . -iname '*' | xargs -I {} file -I {} | grep -va
directory | grep -v "/src/test/" | grep -va 'application/json' | grep -va
'text/' | grep -va 'application/xml' | grep -va 'application/json' | wc -l |
sed -e s'/ //g'`
+numBinaryFiles=`find . -iname '*' | xargs -I {} file -I {} | grep -va
directory | grep -v "release/" | grep -v "/src/test/" | grep -va
'application/json' | grep -va 'text/' | grep -va 'application/xml' | grep -va
'application/json' | wc -l | sed -e s'/ //g'`
if [ "$numBinaryFiles" -gt "0" ]; then
echo -e "There were non-text files in source release. [ERROR]\n Please check
below\n"
- find . -iname '*' | xargs -I {} file -I {} | grep -va directory | grep -v
"/src/test/" | grep -va 'application/json' | grep -va 'text/' | grep -va
'application/xml'
+ find . -iname '*' | xargs -I {} file -I {} | grep -va directory | grep -v
"release/release_guide" | grep -v "/src/test/" | grep -va 'application/json' |
grep -va 'text/' | grep -va 'application/xml'
exit 1
fi
echo -e "\t\tNo Binary Files in Source Release? - [OK]\n"
@@ -163,10 +163,10 @@ echo -e "\t\tNotice file exists ? [OK]\n"
### Licensing Check
echo "Performing custom Licensing Check "
-numfilesWithNoLicense=`find . -iname '*' -type f | grep -v NOTICE | grep -v
LICENSE | grep -v '.json' | grep -v '.hfile' | grep -v '.data' | grep -v
'.commit' | grep -v DISCLAIMER | grep -v KEYS | grep -v '.mailmap' | grep -v
'.sqltemplate' | grep -v 'banner.txt' | grep -v "fixtures" | xargs grep -L
"Licensed to the Apache Software Foundation (ASF)" | wc -l`
+numfilesWithNoLicense=`find . -iname '*' -type f | grep -v NOTICE | grep -v
LICENSE | grep -v '.jpg' | grep -v '.json' | grep -v '.hfile' | grep -v '.data'
| grep -v '.commit' | grep -v DISCLAIMER | grep -v KEYS | grep -v '.mailmap' |
grep -v '.sqltemplate' | grep -v 'banner.txt' | grep -v "fixtures" | xargs grep
-L "Licensed to the Apache Software Foundation (ASF)" | wc -l`
if [ "$numfilesWithNoLicense" -gt "0" ]; then
echo "There were some source files that did not have Apache License [ERROR]"
- find . -iname '*' -type f | grep -v NOTICE | grep -v LICENSE | grep -v
'.json' | grep -v '.hfile' | grep -v '.data' | grep -v '.commit' | grep -v
DISCLAIMER | grep -v '.sqltemplate' | grep -v KEYS | grep -v '.mailmap' | grep
-v 'banner.txt' | grep -v "fixtures" | xargs grep -L "Licensed to the Apache
Software Foundation (ASF)"
+ find . -iname '*' -type f | grep -v NOTICE | grep -v LICENSE | grep -v
'.jpg' | grep -v '.json' | grep -v '.hfile' | grep -v '.data' | grep -v
'.commit' | grep -v DISCLAIMER | grep -v '.sqltemplate' | grep -v KEYS | grep
-v '.mailmap' | grep -v 'banner.txt' | grep -v "fixtures" | xargs grep -L
"Licensed to the Apache Software Foundation (ASF)"
exit 1
fi
echo -e "\t\tLicensing Check Passed [OK]\n"
