Alan-Jowett / bpf_conformance / 22083048693

95%

push

github

Refactor into lib/ and tools/ (#461)

* Refactor into lib/ (pure C++20) and tools/ (with deps)

Split bpf_conformance into:
- lib/: Pure C++20 core library (bpf_conformance_core)
  - Assembler, disassembler, test parser, conformance logic
  - Optional ELFIO support for bpf_writer (detected at configure time)
  - No Boost dependency - uses std::function executor interface
- tools/: Boost/ELFIO-dependent tools
  - Conformance runner, bpf_asm, bpf_disasm CLIs
  - Process executor using Boost.Process

Key changes:
- New bpf_executor_fn callback interface for pluggable execution
- bpf_conformance_run() accepts any executor implementation
- make_process_executor() provides Boost.Process-based executor
- Backwards-compatible bpf_conformance_options() API preserved
- BPF_CONFORMANCE_BUILD_TOOLS CMake option (ON by default)
- BPF_CONFORMANCE_HAS_ELFIO defined when ELFIO available

Consumers can now:
- Include just lib/ for dependency-free core functionality
- Build from root for full functionality (no change)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Refactor: Add executor callback interface to core library

- lib/ now contains pure C++20 bpf_conformance_run() with executor callback
- tools/ contains Boost.Process executor via make_boost_process_executor()
- Public header include/bpf_conformance.h now forwards to core header for types
- Backwards-compatible: bpf_conformance_options() API unchanged

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Fix libbpf_plugin: Link to bpf_conformance_core for include path

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Address PR review comments

- Fix duplicate space in _base_16_encode (lib/src/bpf_conformance.cc)
- Fix Boost_INCLUDE_DIR -> Boost_INCLUDE_DIRS typo (tools/CMakeLists... (continued)

64 of 65 new or added lines in 3 files covered. (98.46%)

1389 of 1462 relevant lines covered (95.01%)

12232.34 hits per line