Vendor check to succeed even if copyright information is not found

Chris Price · Chris Price · commit 9f94cfdc5748 · 2020-02-13T06:08:44.000-08:00
The documentation for validate_vendor_used in hack/validate/vendor states
that a warning will be emitted if license information cannot be found in
a vendored package. However, because the script is run with pipefail set
(owing to the inclusion of the common validation script .validate) and
`grep -c` is used, the entire script will fail whenever license information
cannot be found in a vendored package.

Signed-off-by: Chris Price &lt;cprice@mirantis.com&gt;
diff --git a/hack/validate/vendor b/hack/validate/vendor
@@ -40,8 +40,7 @@ validate_vendor_diff(){
 validate_vendor_used() {
 	for f in $(mawk '/^[a-zA-Z0-9]/ { print $1 }' vendor.conf); do
 	if [ -d "vendor/$f" ]; then
-		found=$(echo "vendor/$f/"* | grep -iEc '/(LICENSE|COPYING)')
-		if [ "$found" -eq 0 ]; then
+		if ! echo "vendor/$f"/* | grep -qiEc '/(LICENSE|COPYING)'; then
 		echo "WARNING: could not find copyright information for $f"
 		fi
 	else
