Michael Witten <mfwitten@xxxxxxxxx> writes: > Signed-off-by: Michael Witten <mfwitten@xxxxxxxxx> > --- > git-send-email.perl | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/git-send-email.perl b/git-send-email.perl > index 273c8c7..63d6063 100755 > --- a/git-send-email.perl > +++ b/git-send-email.perl > @@ -251,7 +251,7 @@ my $rc = GetOptions("sender|from=s" => \$sender, > "chain-reply-to!" => \$chain_reply_to, > "delay=i" => \$delay, > "smtp-server=s" => \$smtp_server, > - "smtp-server-port=s" => \$smtp_server_port, > + "smtp-server-port=i" => \$smtp_server_port, > "smtp-user=s" => \$smtp_authuser, > "smtp-pass:s" => \$smtp_authpass, > "smtp-ssl" => sub { $smtp_encryption = 'ssl' }, Hmm, I have to wonder if there somebody who is using symbolic names for ports, e.g. --smtp-server-port=ssmtp which this patch may start rejecting. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html