Alex Bennée <[email protected]> writes: > Matthieu Moy <[email protected]> writes: > >> We now use Mail::Address unconditionaly, hence parse_mailboxes is now >> dead code. Remove it and its tests. >> >> Signed-off-by: Matthieu Moy <[email protected]> >> --- >> perl/Git.pm | 71 >> ---------------------------------------------------- >> t/t9000-addresses.sh | 27 -------------------- >> t/t9000/test.pl | 67 ------------------------------------------------- >> 3 files changed, 165 deletions(-) >> delete mode 100755 t/t9000-addresses.sh >> delete mode 100755 t/t9000/test.pl > > Should we add the tests for t9001-send-email.sh to guard against regressions?
Tests in t9001 were only useful with our parse_mailboxes (they were just comparing parse_mailboxes and Mail::Address), so there's no point keeping them after we delete parse_mailboxes. Your added tests from https://public-inbox.org/git/[email protected] would make sense OTOH. Not breaking Linux's flow is a nice thing to do ... Patch doing this follows (I'll resend the whole series with Eric's nit later). -- Matthieu Moy https://matthieu-moy.fr/
