My request to better mark sections of main.c responsible for different concerns still stands, but is not a show stopper. Same for my request to make it a part of dpdk-testpmd instead of a secondary process. Anyone interested can probably do either or both locally in half an hour in a code agent anyway.
Summarizing the exchange to v2, our own copy of rpcap-protocol.h is
unavoidable here since one included in libpcap is an internal header
that is not installed by design (irrespective of package maintainers). I
am ready to ack the patch after this is clarified in the code to prevent
other people from being similarly confused since situation with the
dependencies here is quite complex.
- [PATCH] examples/rpcapd: demo version of packet capture ... Stephen Hemminger
- [PATCH v2] examples/rpcapd: demo version of packet ... Stephen Hemminger
- Re: [PATCH v2] examples/rpcapd: demo version of... Marat Khalili
- Re: [PATCH v2] examples/rpcapd: demo versio... Stephen Hemminger
- Re: [PATCH v2] examples/rpcapd: demo versio... Stephen Hemminger
- Re: [PATCH v2] examples/rpcapd: demo ve... Marat Khalili
- Re: [PATCH v2] examples/rpcapd: de... Stephen Hemminger
- Re: [PATCH v2] examples/rpcapd: demo versio... Stephen Hemminger
- Re: [PATCH v2] examples/rpcapd: demo version of... Stephen Hemminger
- [PATCH v3] examples/rpcapd: demo version of packet ... Stephen Hemminger
- Re: [PATCH v3] examples/rpcapd: demo version of... Marat Khalili
- Re: [PATCH v3] examples/rpcapd: demo versio... Stephen Hemminger
- [PATCH v4 0/4] add rpcap remote capture daemon Stephen Hemminger
- [PATCH v4 1/4] pcapng: add API to read back cap... Stephen Hemminger
- [PATCH v4 2/4] app/rpcapd: remote pcap daemon Stephen Hemminger
- [PATCH v4 4/4] app/rpcapd: add host list option Stephen Hemminger
- [PATCH v4 3/4] app/rpcapd: add TLS support Stephen Hemminger

