tidy up email creating a bit

diff --git a/bad_fixes b/bad_fixes
index 7b3e2637495b90fb68427eed65a80064fcc8adc6..ad5413f894f8c3011d927f97d8e1b0e93bdeec8a 100755 (executable)
--- a/bad_fixes
+++ b/bad_fixes
@@ -3,9 +3,6 @@
 subject="Fixes tag needs some work in the BRANCH tree"
 
 tools_dir=$(dirname "$0")
-"$tools_dir"/message_helper "$@" "$subject" <<EOF
-Commit
-
-EOF
+"$tools_dir"/message_helper "$@" "$subject" </dev/null
 
 exit 0

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