From babe201dae730032ee0a9435e55e09245782000a Mon Sep 17 00:00:00 2001 From: Chris Evich Date: Fri, 20 Sep 2019 11:49:07 -0400 Subject: Cirrus: Fix success script Fixed a typo. Also script was grabbing quotes and other non-email-address junk while looping. Filter before and after to make sure we get 'em all. Signed-off-by: Chris Evich --- contrib/cirrus/success.sh | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'contrib/cirrus') diff --git a/contrib/cirrus/success.sh b/contrib/cirrus/success.sh index 30d375d95..8a9618b91 100755 --- a/contrib/cirrus/success.sh +++ b/contrib/cirrus/success.sh @@ -18,18 +18,21 @@ then then SHARANGE="${CIRRUS_BASE_SHA}..${CIRRUS_CHANGE_IN_REPO}" EXCLUDE_RE='merge-robot' + EMAILCSET='[:alnum:]-+_@.' AUTHOR_NICKS=$(egrep -v '(^[[:space:]]*$)|(^[[:space:]]*#)' "$AUTHOR_NICKS_FILEPATH" | sort -u) # Depending on branch-state, it's possible SHARANGE could be _WAY_ too big MAX_NICKS=10 # newline separated GITLOG="git log --format='%ae'" - COMMIT_AUTHORS=$($GITLOGt $SHARANGE || $GITLOG -1 HEAD | \ + COMMIT_AUTHORS=$($GITLOG $SHARANGE || $GITLOG -1 HEAD | \ sort -u | \ egrep -v "$EXCLUDE_RE" | \ + tr --delete --complement "$EMAILCSET[:space:]" | \ tail -$MAX_NICKS) for c_email in $COMMIT_AUTHORS do + c_email=$(echo "$c_email" | tr --delete --complement "$EMAILCSET") echo -e "\tExamining $c_email" NICK=$(echo "$AUTHOR_NICKS" | grep -m 1 "$c_email" | \ awk --field-separator ',' '{print $2}' | tr -d '[[:blank:]]') -- cgit v1.2.3-54-g00ecf