smtpd uses library outside of the standard libraries

[tleeuwenburg]

BPO	12816
Nosy	@warsaw, @terryjreedy, @ncoghlan, @bitdancer, @akheron, @berkerpeksag
Files	MailmanProxy.diff

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2018-01-10.00:53:46.122>
created_at = <Date 2011-08-22.08:45:32.628>
labels = ['library']
title = 'smtpd uses library outside of the standard libraries'
updated_at = <Date 2018-01-10.00:53:46.076>
user = 'https://bugs.python.org/tleeuwenburg'

bugs.python.org fields:

activity = <Date 2018-01-10.00:53:46.076>
actor = 'barry'
assignee = 'none'
closed = True
closed_date = <Date 2018-01-10.00:53:46.122>
closer = 'barry'
components = ['Library (Lib)']
creation = <Date 2011-08-22.08:45:32.628>
creator = 'tleeuwenburg'
dependencies = []
files = ['22995']
hgrepos = []
issue_num = 12816
keywords = ['patch']
message_count = 4.0
messages = ['142701', '142721', '142729', '309749']
nosy_count = 8.0
nosy_names = ['barry', 'terry.reedy', 'ncoghlan', 'r.david.murray', 'petri.lehtinen', 'berker.peksag', 'tleeuwenburg', 'lpolzer']
pr_nums = []
priority = 'normal'
resolution = 'wont fix'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue12816'
versions = []

[tleeuwenburg]

Hi,

I was writing some coverage tests, and started on a class called MailmanProxy inside of smtpd.py. It imports Mailman, which isn't in the standard library. I've attached a failing test to demonstrate the problem, but I didn't want to delete the section from smtpd.py as a beginner.

Thanks,
-Tennessee

[bitdancer]

That was a correct decision :)

It certainly shouldn't be simply deleted, since that would break backward compatibility. Barry, is this something we (ie: you, in this case :) wish to continue to maintain?

If not, we have the options of deprecation-and-removal, or deprecation-in-place (in the latter case we just stick a comment in the file...)

[warsaw]

I think it should be deprecated and eventually removed. I don't remember why I put it in this file, and besides Mailman 3 won't use it.

[warsaw]

I'm closing this as won't fix since smtpd.py is deprecated and will likely not get any future development. Please see aiosmtpd as a much better third party replacement.