Ticket #5917 enhancement closed fixed

Opened 10 months ago

Last modified 10 months ago

Port twisted.python.versions to Python 3

Reported by: exarkun Owned by: exarkun
Priority: normal Milestone: Python 3.3 Minimal
Component: core Keywords:
Cc: Branch: branches/versions-python3-5917-3
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

twisted/__init__.py would like this. Also, `twisted.python.deprecate` depends on twisted.python.versions.

Change History

1

Changed 10 months ago by exarkun

  • branch set to branches/versions-python3-5917
  • branch_author set to exarkun

(In [35386]) Branching to 'versions-python3-5917'

2

Changed 10 months ago by exarkun

  • branch changed from branches/versions-python3-5917 to branches/versions-python3-5917-2

(In [35406]) Branching to 'versions-python3-5917-2'

3

Changed 10 months ago by exarkun

(In [35409]) Switch to byte literals in a bunch of places. Do a little messing about with encodings. Close files instead of relying on the gc to do it. Add twisted.python.versions and twisted.python.test.test_versions to the ported list.

refs #5917

4

Changed 10 months ago by exarkun

  • branch changed from branches/versions-python3-5917-2 to branches/versions-python3-5917-3

(In [35417]) Branching to 'versions-python3-5917-3'

5

Changed 10 months ago by exarkun

  • keywords review added
  • owner changed from exarkun to itamar

Some stuff done,  build results (may they be favorable).

6

Changed 10 months ago by itamar

  • keywords review removed
  • owner changed from itamar to exarkun

Sorry for the late review.

  1. Close #5785 as a duplicate.
  2. Given #5886 is pretty trivial, maybe add it to this branch.
  3. Needs news file.
  4. Having the SVN revision as bytes means you get funny looking str(), e.g. 
>>> str(v)
"[twisted, version 1.0.0+rb'35423']"

So you probably want to nativeString it at some point in the code.

Fix those and merge.

7

Changed 10 months ago by exarkun

(In [35426]) re-enable twisted.version

refs #5917 refs #5886

8

Changed 10 months ago by exarkun

(In [35427]) Add a test for getVersionString result including an svn revision number, and make it pass on Python 3.x

refs #5917

9

Changed 10 months ago by exarkun

(In [35428]) Fix pyflakes warning

refs #5917

10

Changed 10 months ago by exarkun

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

(In [35431]) Merge versions-python3-5917-3

Author: exarkun Reviewer: itamar Fixes: #5917 Fixes: #5785 Fixes: #5886

Port twisted.python.versions to Python 3. Also add a bit of missing test coverage and restore the definition of twisted.__version__ on Python 3.

Note: See TracTickets for help on using tickets.