Task #2525
closed
Import ndnping and peek/poke
100%
Description
Import the following tools to ndn-tools repository:
- ndnping and ndnpingserver
- ndn-tlv-peek and ndn-tlv-poke
Changes should be minimized. Improvements of these tools will be separate issues.
Updated by Junxiao Shi almost 10 years ago
- Blocked by Task #2524: Initialize repository added
Updated by Junxiao Shi over 9 years ago
http://gerrit.named-data.net/1888 imports ndnping and ndnpingserver from ndn-tlv-ping:commit:1283684bacb2352de38c8f0ad21a069875583d91.
I am aware that the code won't compile in debug mode due to its usage of deprecated Name::set methods.
However, I plan to change that (and do other fixes) in the next commit, so that these changes are more visible.
Updated by Junxiao Shi over 9 years ago
- Status changed from In Progress to Code review
- % Done changed from 0 to 100
http://gerrit.named-data.net/1904 imports ndnpeek and ndnpoke from NFD.
I am copying version.hpp from NFD for now. Version number printing will be improved in #2671.
Updated by Junxiao Shi over 9 years ago
- Description updated (diff)
- Status changed from Code review to In Progress
- % Done changed from 100 to 70
Alex agreed that ndndump should be imported as well.
Updated by Junxiao Shi over 9 years ago
- Description updated (diff)
- Status changed from In Progress to Closed
- % Done changed from 70 to 100
It seems that importing ndndump is non-trivial because it not only has additional dependency (which ndn-tools build scripts can already handle), but also has additional configuration checks (which needs investigation). This work is split to #2692.
Updated by Junxiao Shi over 9 years ago
- Status changed from Closed to In Progress
- % Done changed from 100 to 80
I also need to update these tools to conform to naming conventions, as part of this issue.
Updated by Junxiao Shi over 9 years ago
- Status changed from In Progress to Code review
- % Done changed from 80 to 100
http://gerrit.named-data.net/1908 changes namespace and class names in ping and peek, and does minor cleanups.
Updated by Junxiao Shi over 9 years ago
- Status changed from Code review to Closed
Updated by Junxiao Shi over 8 years ago
- Status changed from Closed to Code review
I notice that ndn-cxx README and NFD docs still links to ndn-tlv-ping.
https://gerrit.named-data.net/2922 and https://gerrit.named-data.net/2923 correct this problem.
Updated by Junxiao Shi over 8 years ago
- Status changed from Code review to Closed