Task #2817
closed
Deprecate ndn-tlv-peek and ndn-tlv-poke
Added by Junxiao Shi over 9 years ago.
Updated over 9 years ago.
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.
- Blocked by Task #2816: README and manpages for ndnpeek and ndnpoke added
- Blocks Task #2819: Delete ndn-tlv-peek, ndn-tlv-poke, tlvdump added
Looks good to me. Will ndn-tlv-ping be deprecated in favor of ndnping as well?
Question in note-5 is irrelevant to this issue. Please discuss on ndn-tlv-ping issue 9.
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.
- Status changed from New to In Progress
- Assignee set to Junxiao Shi
- Status changed from In Progress to Code review
- % Done changed from 0 to 100
- Status changed from Code review to Closed
Also available in: Atom
PDF