nvidia-holoscan / holoscan-cli / 30896481402

79%

main: 79%

Build Type

Pull #217

github

Committed by wyli

Commit Message

feat: add --prefix to install for install-time destination selection

`holoscan install` emitted a bare `cmake --install <build-dir>`, so the only
way to pick a destination was `--configure-args="-DCMAKE_INSTALL_PREFIX=..."`.
That bakes the path into the build tree's CMake cache: a configured tree can
only ever install to one location, and changing it forces a reconfigure.

Add `--prefix`, forwarded to `cmake --install --prefix`. It is resolved at
install time and leaves the cache untouched, so one built tree can be
installed to several destinations. The flag is spelled to match CMake's own
option; internally it is `install_prefix` so it cannot be confused with the
unrelated `cli.prefix` metadata placeholder prefix used in the same module.

Both paths are covered: the local branch appends the option to the cmake
invocation, and the container branch forwards it (shell-quoted) into the
recursive in-container `holoscan install ... --local` command.

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #217: feat: add --prefix to install for install-time destination selection

Coverage Stats

7 of 7 new or added lines in 1 file covered. (100.0%)

23 existing lines in 1 file now uncovered.

3243 of 4131 relevant lines covered (78.5%)

0.79 hits per line