send-email: add and use a local copy of Mail::Address
commitbd869f67b93b36e27405db5e288ee82f3a49051a
authorMatthieu Moy <git@matthieu-moy.fr>
Fri, 5 Jan 2018 18:36:51 +0000 (5 19:36 +0100)
committerJunio C Hamano <gitster@pobox.com>
Fri, 5 Jan 2018 20:21:31 +0000 (5 12:21 -0800)
tree769f097a317dd38285aadb7c9607f73f2440053f
parent1eaabe34fc6f486367a176207420378f587d3b48
send-email: add and use a local copy of Mail::Address

We used to have two versions of the email parsing code. Our
parse_mailboxes (in Git.pm), and Mail::Address which we used if
installed. Unfortunately, both versions have different sets of bugs, and
changing the behavior of git depending on whether Mail::Address is
installed was a bad idea.

A first attempt to solve this was cc90750 (send-email: don't use
Mail::Address, even if available, 2017-08-23), but it turns out our
parse_mailboxes is too buggy for some uses. For example the lack of
nested comments support breaks get_maintainer.pl in the Linux kernel
tree:

  https://public-inbox.org/git/20171116154814.23785-1-alex.bennee@linaro.org/

This patch goes the other way: use Mail::Address anyway, but have a
local copy from CPAN as a fallback, when the system one is not
available.

The duplicated script is small (276 lines of code) and stable in time.
Maintaining the local copy should not be an issue, and will certainly be
less burden than maintaining our own parse_mailboxes.

Another option would be to consider Mail::Address as a hard dependency,
but it's easy enough to save the trouble of extra-dependency to the end
user or packager.

Signed-off-by: Matthieu Moy <git@matthieu-moy.fr>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-send-email.perl
perl/Git/FromCPAN/Mail/Address.pm [new file with mode: 0644]
perl/Git/Mail/Address.pm [new file with mode: 0755]