Ticket #6235 task closed fixed

Opened 5 months ago

Last modified 5 months ago

Merge `twisted/python/_utilpy3.py` into `twisted/python/util.py`

Reported by: exarkun Owned by: exarkun
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/util-merge-6235
Author: exarkun Launchpad Bug:

Description

Also merge the associated test module. See #6183.

Change History

1

Changed 5 months ago by exarkun

  • branch set to branches/util-merge-6235
  • branch_author set to exarkun

(In [36657]) Branching to 'util-merge-6235'

2

Changed 5 months ago by exarkun

  • owner exarkun deleted
  • keywords review added

It's done. I tried to provide good descriptions of the steps and the logic behind them in each commit message. The result is not as nice as I thought it would be, with respect to change history. I'm now convinced it's not feasible to restore any more of the history than this branch does (which is to say, any of the history of the moved code); perhaps svnadmin could do it, I don't know, but I don't think I'll try to go down that road.

The code moved into util.py and test_util.py is all copied verbatim from _utilpy3.py and test_utilpy3.py except that I had to change TestCase to unittest.TestCase for the class statements being moved into test_util.py.

It may be instructive to consider the diff 36657:36669 separately from r36670 and r36671. The former is "move stuff around without interestingly changing it" (r36668 not withstanding). The remainder is "adjust syntax and obliterate unported names" necessary to make the merged modules actually work on Python 3 (only to the level of functionality previously present in _utilpy3).

 Build results

3

Changed 5 months ago by tom.prince

  • owner set to tom.prince

4

Changed 5 months ago by tom.prince

  • keywords review removed
  • owner changed from tom.prince to exarkun

1. This is missing imports from __future__. 1. GetPasswordTest but isn't deleted and is skipped due to lack of process support on the reactor. It should be deleted. 1. InitGroupsTests) wasn't ported but is passing. It should be deleted or the appropriate functionality marked as supported. Looking closer, it appears that this isn't intended to be public anyway. (This can be done in a separate ticket).

Please commit after fixing 1+2 (and possibly 3).

(Note for archaeologists: there is no interesting history on trunk in util.py and test_util.py that is lost. There is one non-py3 related changeset in _utilpy3.py: r35964 closing #6063).

5

Changed 5 months ago by exarkun

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

(In [36719]) Merge util-merge-6235

Author: exarkun Reviewer: tom.prince Fixes: #6235

Re-integrate the Python 3 ported miscellaneous util implementations into twisted/python/util.py.

Note: See TracTickets for help on using tickets.