Project

General

Profile

Actions

Task #2817

closed

Deprecate ndn-tlv-peek and ndn-tlv-poke

Added by Junxiao Shi over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Tools
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
1.50 h

Description

Modify ndn-tlv-peek and ndn-tlv-poke programs to print a notice to stderr:

ndn-tlv-peek is deprecated. Use ndnpeek program from ndn-tools repository.
ndn-tlv-poke is deprecated. Use ndnpoke program from ndn-tools repository.

Apart from this notice, all features of these programs should be preserved.

On the manpage of these tools, add the same notice, and provide a link to ndn-tools repository.


Related issues 2 (0 open2 closed)

Blocked by ndn-tools - Task #2816: README and manpages for ndnpeek and ndnpokeClosedJunxiao Shi

Actions
Blocks NFD - Task #2819: Delete ndn-tlv-peek, ndn-tlv-poke, tlvdumpClosedDavide Pesavento

Actions
Actions #1

Updated by Junxiao Shi over 9 years ago

  • Blocked by Task #2816: README and manpages for ndnpeek and ndnpoke added
Actions #2

Updated by Junxiao Shi over 9 years ago

  • Blocks Task #2819: Delete ndn-tlv-peek, ndn-tlv-poke, tlvdump added
Actions #3

Updated by Eric Newberry over 9 years ago

Looks good to me. Will ndn-tlv-ping be deprecated in favor of ndnping as well?

Actions #4

Updated by Junxiao Shi over 9 years ago

Will ndn-tlv-ping be deprecated in favor of ndnping as well?

Yes, see https://github.com/named-data/ndn-tlv-ping/issues/9

Actions #5

Updated by Eric Newberry over 9 years ago

Junxiao Shi wrote:

Will ndn-tlv-ping be deprecated in favor of ndnping as well?

Yes, see https://github.com/named-data/ndn-tlv-ping/issues/9

Who's responsible for completing the deprecation task for ndn-tlv-ping?

Actions #6

Updated by Junxiao Shi over 9 years ago

Question in note-5 is irrelevant to this issue. Please discuss on ndn-tlv-ping issue 9.

Actions #7

Updated by Alex Afanasyev over 9 years ago

I'm fine with the task, the only suggestion I have is that we have a more verbose message with a link to some documentation, not just mentioning ndn-tools repo.

Actions #8

Updated by Junxiao Shi over 9 years ago

  • Status changed from New to In Progress
  • Assignee set to Junxiao Shi
Actions #9

Updated by Junxiao Shi over 9 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 0 to 100
Actions #10

Updated by Junxiao Shi over 9 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF