Commit 6b7749f2 by Greg Price

Fix bug that omits commits in release email script

The command built to generate the list of commits committed but not
authored by a particular individual was incorrect, so some commits
were incorrectly omitted in the table as entries belonging to the
committer.
parent b2cd0d28
......@@ -36,15 +36,13 @@ echo
echo '||Author||Changes||Commit Link||Testing Notes||'
for EMAIL in $RESPONSIBLE; do
AUTHORED_BY="$LOG_CMD --author=<${EMAIL}>"
COMMITTED_BY="$LOG_CMD --committer=<${EMAIL}>"
COMMITTED_NOT_AUTHORED="$COMMITTED_BY $($AUTHORED_BY --format='tformat:^%h')"
$AUTHORED_BY --format="tformat:|$EMAIL|%s|[commit|https://github.com/edx/edx-platform/commit/%h]| |" | head -n 1
$AUTHORED_BY --format="tformat:| |%s|[commit|https://github.com/edx/edx-platform/commit/%h]| |" | tail -n +2
if [[ $($COMMITTED_NOT_AUTHORED) != "" ]]; then
$COMMITTED_NOT_AUTHORED --format="tformat:|$EMAIL|%s|[commit|https://github.com/edx/edx-platform/commit/%h]|Committed, didn't author|" | head -n 1
$COMMITTED_NOT_AUTHORED --format="tformat:| |%s|[commit|https://github.com/edx/edx-platform/commit/%h]| |" | tail -n +2
fi
AUTHORED_BY=$($LOG_CMD --author="<${EMAIL}>" --format='tformat:%h')
COMMITTED_BY=$($LOG_CMD --committer="<${EMAIL}>" --format='tformat:%h')
ALL_COMMITS=$(for HASH in $AUTHORED_BY $COMMITTED_BY; do echo $HASH; done | sort | uniq)
EMAIL_COL="$EMAIL"
for HASH in $ALL_COMMITS; do
git log --format="tformat:|$EMAIL_COL|%s|[commit|https://github.com/edx/edx-platform/commit/%h]| |" -n 1 $HASH
EMAIL_COL=" "
done
done
\ No newline at end of file
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment