nvidia-holoscan / holoscan-cli / 26881814874

77%

main: 77%

Build Type

Pull #180

github

Committed by wyli

Commit Message

Wire external module-sites resolution into build (#1587)

holoscan-cli shipped the external_resolver + cmake_manifest library code but
never invoked it from the build flow, so `holoscan build` wrote no
external_operators_manifest.cmake and external Holoscan Modules (e.g. the
DELTACAST.TV operators) couldn't be lazily fetched. Port HoloHub #1587:

- external_resolver: add parse_module_sites() (parse modules/module-sites.json
  into ModuleDep records: external / in-tree / local-override) and merge_deps()
  (sites own canonical git coords + provides_operators; project deps supply
  override_path and fallback ops). Add an `env` override param for testability.
- build: before cmake configure, merge module-sites + project deps and write
  external_operators_manifest.cmake into the build dir (the project root
  CMakeLists includes it OPTIONAL). Also os.path.expandvars() on configure_args
  so mode-supplied -D args can reference env vars.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #180: Sync post-consolidation HoloHub CLI changes (#1576, #1582, #1587, #1596)

Coverage Stats

63 of 68 new or added lines in 4 files covered. (92.65%)

1 existing line in 1 file now uncovered.

2937 of 3836 relevant lines covered (76.56%)

0.77 hits per line