Opened 6 years ago

Closed 6 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
(diff, github, buildbot, log)
Author: teratorn Launchpad Bug:

Description

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

Change History (9)

comment:1 Changed 6 years ago by teratorn

  • author set to teratorn
  • Branch set to branches/delete-t-im-3507

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

comment:2 Changed 6 years ago by teratorn

  • Keywords review added

comment:3 Changed 6 years ago by exarkun

  • Keywords review removed
  • Owner changed from glyph to teratorn

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

comment:4 Changed 6 years ago by teratorn

  • Keywords review added

thanks, done now.

comment:5 Changed 6 years ago by teratorn

  • Owner teratorn deleted

comment:6 Changed 6 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 6 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 6 years ago by exarkun

  • Keywords review removed
  • Owner set to teratorn

Looks good, please merge. Thanks.

comment:9 Changed 6 years ago by teratorn

  • Resolution set to fixed
  • Status changed from new to closed

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

Note: See TracTickets for help on using tickets.