Ticket #6097 enhancement closed fixed

Opened 7 months ago

Last modified 7 months ago

Port twisted.protocols.policies to Python 3

Reported by: itamar Owned by: itamar
Priority: normal Milestone: Python 3.3 Minimal
Component: core Keywords:
Cc: Branch: branches/policies-py3-6097
Author: itamarst Launchpad Bug:

Description

twisted.protocols.policies should run on Python 3 (it's required by twisted.protocols.tls).

Change History

1

Changed 7 months ago by itamarst

  • branch set to branches/policies-py3-6097
  • branch_author set to itamarst

(In [36089]) Branching to 'policies-py3-6097'

2

Changed 7 months ago by itamar

  • owner changed from itamar to exarkun
  • keywords review added

Ready for review, I think:  http://buildbot.twistedmatrix.com/boxes-supported?branch=/branches/policies-py3-6097 has started.

  1. I improved coverage on ProtocolWrapper and WrappedFactory, which are what we really care about. Coverage is still imperfect elsewhere.
  2. For the logging factory, I decided to keep it as native strings written to the file; since it writes a repr() of the traffic, that doesn't seem like much of a problem.

3

Changed 7 months ago by exarkun

  • owner changed from exarkun to itamar
  • keywords review removed
  1. Typo in _getWrapper docstring - conncected
  2. ... stopConsuming? Uh, ok whatever...

Looks good otherwise, please merge after addressing the above.

4

Changed 7 months ago by itamarst

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

(In [36096]) Merge policies-py3-6097.

Author: itamar Review: exarkun Fixes: #6097

Port twisted.protocols.policies to Python 3.

Note: See TracTickets for help on using tickets.