From 79ee555bac7a5ff846c8a381792bbe93cdffbf97 Mon Sep 17 00:00:00 2001 From: "Eric W. Biederman" Date: Wed, 21 Jun 2006 07:17:31 -0600 Subject: Check and document the options to prevent mistakes. When multiple recipients are given to git-send-email on the same --cc line the code does not properly handle it. Full and proper parsing of the email addresses so I can detect which commas mean a new email address is more than I care to implement. In particular this email address: "bibo,mao" must not be treated as two email addresses. So this patch simply treats all commas in recipient lists as an error and fails if one is given. At the same time it documents that git-send-email wants multiple instances of --cc specified on the command line if you want to cc multiple recipients. Signed-off-by: Eric W. Biederman Signed-off-by: Junio C Hamano --- git-send-email.perl | 14 ++++++++++++++ 1 file changed, 14 insertions(+) (limited to 'git-send-email.perl') diff --git a/git-send-email.perl b/git-send-email.perl index 7b1cca70ab..c5d9e73351 100755 --- a/git-send-email.perl +++ b/git-send-email.perl @@ -65,6 +65,20 @@ my $rc = GetOptions("from=s" => \$from, "no-signed-off-cc|no-signed-off-by-cc" => \$no_signed_off_cc, ); +# Verify the user input + +foreach my $entry (@to) { + die "Comma in --to entry: $entry'\n" unless $entry !~ m/,/; +} + +foreach my $entry (@initial_cc) { + die "Comma in --cc entry: $entry'\n" unless $entry !~ m/,/; +} + +foreach my $entry (@bcclist) { + die "Comma in --bcclist entry: $entry'\n" unless $entry !~ m/,/; +} + # Now, let's fill any that aren't set in with defaults: sub gitvar { -- cgit v1.2.3