Ticket #5948 enhancement closed fixed

Opened 20 months ago

Last modified 20 months ago

Remove support for string exceptions from twisted.python.failure

Reported by: itamar Owned by: itamar
Priority: normal Milestone: Python 3.3 Minimal
Component: core Keywords:
Cc: Branch: branches/no-string-failures-5948
(diff, github, buildbot, log)
Author: itamarst Launchpad Bug:

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

1

Changed 20 months ago by itamarst

  • branch set to branches/no-string-failures-5948
  • branch_author set to itamarst

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

2

Changed 20 months ago by itamar

  • keywords review added
  • owner set to exarkun

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

3

Changed 20 months ago by exarkun

  • owner changed from exarkun to itamar
  • keywords review removed
  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.

4

Changed 20 months ago by itamarst

  • status changed from new to closed
  • resolution set to fixed

(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.