diff options
| author | Chris Evich <cevich@redhat.com> | 2019-09-20 11:49:07 -0400 |
|---|---|---|
| committer | Chris Evich <cevich@redhat.com> | 2019-09-20 12:31:56 -0400 |
| commit | babe201dae730032ee0a9435e55e09245782000a (patch) | |
| tree | a0de41fb9673dc19997bce92e590ff46e08416f8 | |
| parent | 93869eb5005370ad8b24c3195d8c1b9a3282e27a (diff) | |
| download | podman-babe201dae730032ee0a9435e55e09245782000a.tar.gz podman-babe201dae730032ee0a9435e55e09245782000a.tar.bz2 podman-babe201dae730032ee0a9435e55e09245782000a.zip | |
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 <cevich@redhat.com>
| -rwxr-xr-x | contrib/cirrus/success.sh | 5 |
1 files changed, 4 insertions, 1 deletions
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:]]') |