tidy up email creating a bit

[next-scripts] / message_helper

diff --git a/message_helper b/message_helper
index 3c450dd6ed6043d0e134ecb783cb9efc1492f563..48e2428565d98e5d7273d5194adcf6d9ebe9c2b0 100755 (executable)
--- a/message_helper
+++ b/message_helper
@@ -1,8 +1,9 @@
 #!/bin/bash
 
-. "$(dirname $0)/common.sh" ""
+# shellcheck source=./common.sh
+. "$(dirname "$0")/common.sh" ""
 
-log=$LOG_FILE
+log="$LOG_FILE"
 [ -f merge.log ] && log=merge.log
 
 branch=""
@@ -17,7 +18,7 @@ kconfig=""
         shift 2
 }
 
-cc="Linux Next Mailing List <linux-next@vger.kernel.org>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>"
+cc='Linux Next Mailing List <linux-next@vger.kernel.org>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>'
 
 [ "$branch" ] || {
        branch=$(grep '^Merging ' $log | tail -1 | sed 's/^Merging //' | sed 's/ .*$//')
@@ -25,20 +26,13 @@ cc="Linux Next Mailing List <linux-next@vger.kernel.org>, Linux Kernel Mailing L
 }
 
 contact=$(get_contacts "$branch")
-greeting=${contact%% *}
-greeting=${greeting#\"}
-[ "${contact%,*}" = "${contact}" ] || greeting="all"
-
 subject="${1//BRANCH/$branch}"
-filter="sed -e \"s/BRANCH/$branch/g\""
+filter="sed -e 's/BRANCH/$branch/g'"
 [ "$kconfig" ] &&
-       filter="$filter -e \"s/KCONFIG_NAME/$kconfig/g\""
+       filter="$filter -e 's/KCONFIG_NAME/$kconfig/g'"
 filter="$filter | fmt -s"
 
-$(dirname $0)/send_email "$contact" "$cc" "linux-next: $subject" <<EOF
-Hi $greeting,
-
-$(eval $filter)
-EOF
+{ printf 'Hi all,\n\n'; eval "$filter"; } |
+       "$bin_dir"/send_email "$contact" "$cc" "linux-next: $subject"
 
 exit 0