Opened 9 years ago

Closed 9 years ago

#5783 release blocker: regression closed fixed (fixed)

SSL support needs to be optional in twisted mail, again

Reported by: Jean-Paul Calderone Owned by: Jean-Paul Calderone
Priority: highest Milestone:
Component: mail Keywords:
Cc: Branch: branches/mail-without-ssl-5783
branch-diff, diff-cov, branch-cov, buildbot
Author: exarkun


r34769 / #4963 introduced a hard dependency on OpenSSL in twisted.mail.tap, whereas previously it was an optional dependency. It should be optional again.

Change History (4)

comment:1 Changed 9 years ago by Jean-Paul Calderone

Author: exarkun
Branch: branches/mail-without-ssl-5783

(In [34825]) Branching to 'mail-without-ssl-5783'

comment:2 Changed 9 years ago by Jean-Paul Calderone

Keywords: review added
Priority: normalhighest

comment:3 Changed 9 years ago by Glyph

Keywords: review removed
Owner: set to Jean-Paul Calderone

LGTM, as soon as the rest of the tests pass. Thanks for the review softball :-).

comment:4 Changed 9 years ago by Jean-Paul Calderone

Resolution: fixed
Status: newclosed

(In [34833]) Merge mail-without-ssl-5783

Author: exarkun Reviewer: glyph Fixes: #5783

Make twistd mail's SSL dependency optional again.

Note: See TracTickets for help on using tickets.