send-email: don't use Mail::Address, even if available
commitcc9075067776ebd34cc08f31bf78bb05f12fd879
authorMatthieu Moy <git@matthieu-moy.fr>
Wed, 23 Aug 2017 10:21:02 +0000 (23 12:21 +0200)
committerJunio C Hamano <gitster@pobox.com>
Thu, 24 Aug 2017 21:40:17 +0000 (24 14:40 -0700)
tree9e7bd9e13aeb9a2d79cebbf0af76b03fe8b1a1c9
parentcb2922fe4b2a82e2174d419781836e768651ebff
send-email: don't use Mail::Address, even if available

Using Mail::Address made sense when we didn't have a proper parser. We
now have a reasonable address parser, and using Mail::Address
_if available_ causes much more trouble than it gives benefits:

* Developers typically test one version, not both.

* Users may not be aware that installing Mail::Address will change the
  behavior. They may complain about the behavior in one case without
  knowing that Mail::Address is involved.

* Having this optional Mail::Address makes it tempting to anwser "please
  install Mail::Address" to users instead of fixing our own code. We've
  reached the stage where bugs in our parser should be fixed, not worked
  around.

Signed-off-by: Matthieu Moy <git@matthieu-moy.fr>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-send-email.perl