Opened 5 years ago

Closed 5 years ago

#5948 enhancement closed fixed (fixed)

Remove support for string exceptions from twisted.python.failure

Reported by: Itamar Turner-Trauring Owned by: Itamar Turner-Trauring
Priority: normal Milestone: Python 3.3 Minimal
Component: core Keywords:
Cc: Branch: branches/no-string-failures-5948
branch-diff, diff-cov, branch-cov, buildbot
Author: itamarst

Description

Python 2.6 is now minimal version, we don't need support for string exceptions and it's making Python 3 port harder.

Change History (4)

comment:1 Changed 5 years ago by itamarst

Author: itamarst
Branch: branches/no-string-failures-5948

(In [35530]) Branching to 'no-string-failures-5948'

comment:2 Changed 5 years ago by Itamar Turner-Trauring

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

Ready for review: buildbot.twistedmatrix.com/boxes-supported?branch=/branches/no-string-failures-5948

comment:3 Changed 5 years ago by Jean-Paul Calderone

Keywords: review removed
Owner: changed from Jean-Paul Calderone to Itamar Turner-Trauring
  1. I'm mildly concerned about twisted.spread.pb.CopiedFailure, but as far as I can tell, it overrides the only Failure method which is losing support for a string type attribute (printTraceback), so I guess I shouldn't worry (at least not any more than I already did about the mere existence of a Failure subclass in twisted.spread).
  2. There's a new pyflakes warning about test_failure.py (util' imported but unused).

Otherwise looks good, please merge.

comment:4 Changed 5 years ago by itamarst

Resolution: fixed
Status: newclosed

(In [35542]) Merge no-string-failures-5948: no more Failure support for string exceptions.

Author: itamar Review: exarkun Fixes: #5948

Failure will no longer even pretend to work with string exceptions.

Note: See TracTickets for help on using tickets.