Task #2692
closedImport ndndump
100%
Description
Import ndndump to ndn-tools repository.
Changes should be minimized. Improvements will be separate issues.
Updated by Joao Pereira over 9 years ago
Is this just as simple as:
1 - pick the source from github
2 - paste it in ndn-tools repo
3 - change waf files to compile it
Updated by Junxiao Shi over 9 years ago
- Related to Feature #2670: wscript: build a subset of tools added
Updated by Junxiao Shi over 9 years ago
note-1 is mostly correct. The directory for ndndump is tools/dump
.
The complication is: unlike all existing tools that depend on only ndn-cxx, ndndump has a dependency on libpcap.
As described in #2524 note-16 second option, libpcap dependency is required only if ndndump is enabled at configuration time.
Therefore, the dependency should be declared within ndndump, not globally. Top-level wscript
needs to be tweaked to collect dependencies from wscript
s in subdirectories.
Updated by Junxiao Shi over 9 years ago
- Status changed from New to In Progress
- Assignee set to Junxiao Shi
Updated by Junxiao Shi over 9 years ago
- % Done changed from 0 to 50
http://gerrit.named-data.net/2053
ndndump
contains a deprecated usage of Block::fromBuffer
.
I want the import commit to be an exact copy (except wscript
), so I'll fix the problem in ndndump repository before continuing in ndn-tools.
ndndump issue: https://github.com/named-data/ndndump/issues/2
Updated by Junxiao Shi over 9 years ago
- Status changed from In Progress to Code review
- % Done changed from 50 to 100
http://gerrit.named-data.net/2053
ndndump
depends on extra Boost libraries, but I didn't find a way to import boost.py
waf-tool at tools/dump/wscript
level, and had to do it at the top level.
Updated by Junxiao Shi over 9 years ago
- Blocks Feature #2855: README and manpages for ndndump added
Updated by Junxiao Shi over 9 years ago
- Status changed from Code review to Closed
Updated by Junxiao Shi over 9 years ago
- Status changed from Closed to In Progress
- % Done changed from 100 to 80
I still need to change namespaces to conform to README-dev.md requirements in another commit.
Updated by Junxiao Shi over 9 years ago
- Status changed from In Progress to Code review
- % Done changed from 80 to 100
Updated by Junxiao Shi over 9 years ago
- Status changed from Code review to Closed