Opened 9 years ago

Closed 9 years ago

#3507 defect closed fixed (fixed)

bin/t-im should be deleted

Reported by: teratorn Owned by: teratorn
Priority: normal Milestone:
Component: core Keywords:
Cc: ralphm Branch: branches/delete-t-im-3507
branch-diff, diff-cov, branch-cov, buildbot
Author: teratorn

Description

t-im references twisted.scripts.im which doesn't exist anymore.

Change History (9)

comment:1 Changed 9 years ago by teratorn

author: teratorn
Branch: branches/delete-t-im-3507

(In [25142]) Branching to 'delete-t-im-3507'

comment:2 Changed 9 years ago by teratorn

Keywords: review added

comment:3 Changed 9 years ago by Jean-Paul Calderone

Keywords: review removed
Owner: changed from Glyph to teratorn

You missed some parts. Grep the repository for t-im.

comment:4 Changed 9 years ago by teratorn

Keywords: review added

thanks, done now.

comment:5 Changed 9 years ago by teratorn

Owner: teratorn deleted

comment:6 Changed 9 years ago by ralphm

Cc: ralphm added
Keywords: review removed
Owner: set to teratorn

I noticed that twisted.words.scripts.im does exist. I'm not sure about the usefulness of rewiring t-im back to it, though.

comment:7 Changed 9 years ago by teratorn

Keywords: review added
Owner: teratorn deleted

bin/words/im is the binary that references twisted.words.scripts.im, so I don't think we need to worry about bin/t-im... presumably it has been broken for years, but I don't know. trac failed to tell me when twisted.scripts.im was last seen.

comment:8 Changed 9 years ago by Jean-Paul Calderone

Keywords: review removed
Owner: set to teratorn

Looks good, please merge. Thanks.

comment:9 Changed 9 years ago by teratorn

Resolution: fixed
Status: newclosed

(In [25178]) merge delete-t-im-3507 Author: teratorn Reviewer: exarkun Fixes: #3507

Note: See TracTickets for help on using tickets.