twisted / twisted/twisted

Deprecate inet_pton, inet_ntop, and adict in twisted.python.compat

Open
#5,895 21 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

core new priority-normal task
Dominant language
Python
Stars
6k
Forks
1.2k
Avg merge
2d 10h
Merged PRs (30d)
10

Description

itamarst's avatar @itamarst reported
Trac ID trac#5895
Type task
Created 2012-08-21 12:33:38Z
Branch https://github.com/twisted/twisted/tree/inet-compat-5895

inet_pton, inet_ntop, and adict in twisted.python.compat are no longer necessary (2.6 has good implementations of first two, third is just junk). They should be deprecated.

Attachments:

  • ticket5895.patch (10273 bytes) - added by mateusbraga on 2013-04-15 03:28:14Z - Patch to solve this ticket.
  • ticket5895_2.patch (10754 bytes) - added by mateusbraga on 2013-04-16 01:45:03Z - twisted/topfiles/5895.removal added to the patch
  • ticket5895_3.patch (10907 bytes) - added by mateusbraga on 2013-04-27 04:23:27Z - Included some of thijs suggestions
  • ticket5895_4.patch (11665 bytes) - added by mateusbraga on 2013-04-28 00:06:55Z - Included thijs suggestions
  • ticket5895_5.patch (506 bytes) - added by mateusbraga on 2013-05-16 01:46:25Z - Patch to pass test
Searchable metadata
trac-id__5895 5895
type__task task
reporter__itamar itamar
priority__normal normal
milestone__De_Python_2ify De-Python-2ify
branch__branches_inet_compat_5895 branches/inet-compat-5895
branch_author__mateusbraga mateusbraga
status__new new
resolution__None None
component__core core
keywords__ 
time__1345552418000000 1345552418000000
changetime__1595670454676254 1595670454676254
version__None None
owner__ 
cc__thijs

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by inspecting twisted.python.compat and the attached ticket5895 patches, including the later versions and the added twisted/topfiles/5895.removal file. Review the existing test changes in those patches and confirm that inet_pton, inet_ntop, and adict are deprecated as intended without breaking compatibility. Done means the relevant tests pass and the removal metadata is consistent.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend, networking
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.