[PATCH] email-test-patchset: fix pcfg_reply_to_all

Jacob Keller jacob.e.keller at intel.com
Thu Feb 13 14:28:19 PST 2014


Through the course of several refactors, one of the patches
accidentallly broke reply_to_all functionality. In the older code, we
did a negative check, and if reply_to_all was not set, we would then
unset $to in order to prevent the compose_email function/code from
adding any Cc addresses.

However, one of the refactors missed this negative check, assumed it was
positive, and then changed the semantics so that reply_to_all = 0
caused reply behavior rather than otherwise.

This patch corrects the given commit by making it check for 1, instead
of not 1.

Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
Fixes: 092f2ca246 "(email-test-patchset: further improve readability)"
---
 email/aiaiai-email-test-patchset |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/email/aiaiai-email-test-patchset b/email/aiaiai-email-test-patchset
index beb3e96..ed7c08c 100755
--- a/email/aiaiai-email-test-patchset
+++ b/email/aiaiai-email-test-patchset
@@ -320,7 +320,7 @@ if [ -z "$pcfg_name" ]; then
 fi
 
 # Create the Cc list for replies that we'll be sending
-if [ "$pcfg_reply_to_all" != "1" ]; then
+if [ "$pcfg_reply_to_all" = "1" ]; then
 	# Replies all the patch recipients will be CCed
 	reply_cc="$(merge_addresses "$to" "$cc")"
 	reply_cc="$(merge_addresses "$reply_cc" "$pcfg_always_cc")"




More information about the aiaiai mailing list