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