Skip to content

tools: add build path support on host_info tool#18919

Merged
linguini1 merged 1 commit into
apache:masterfrom
fdcavalcanti:feature/nxdiag-cmake-support
May 20, 2026
Merged

tools: add build path support on host_info tool#18919
linguini1 merged 1 commit into
apache:masterfrom
fdcavalcanti:feature/nxdiag-cmake-support

Conversation

@fdcavalcanti
Copy link
Copy Markdown
Contributor

@fdcavalcanti fdcavalcanti commented May 20, 2026

Summary

  • tools: add build path support on host_info tool

Adds a build_path argument for when CMake builds are used.
This solves a problem when building the nxdiag tool with CMake.

This is related to this PR on nuttx-apps.

Impact

Impact on user: No.

Impact on build: If using CMake, especially with Espressif devices, this will fix a problem related to the script path. Does not affect Make builds.

Impact on hardware: No.

Impact on documentation: No.

Impact on security: No.

Impact on compatibility: No.

Testing

Build with Make works fine, with CMake it would fail since build and source directory differs.

Building

Make example:

  • ./tools/configure.sh esp32h2-devkit:nsh
  • Enable CONFIG_SYSTEM_NXDIAG and CONFIG_SYSTEM_NXDIAG_ESPRESSIF
  • build

CMake example:

  • cmake -B build -DBOARD_CONFIG=esp32h2-devkit:nsh -GNinja
  • Enable CONFIG_SYSTEM_NXDIAG and CONFIG_SYSTEM_NXDIAG_ESPRESSIF
  • build

Running

Make:

  • python3 -B tools/host_info_dump.py /home/fdcavalcanti/nuttxspace5/nuttx --target_info

CMake:

  • Need to pass the build directory to '-b'
  • python3 -B tools/host_info_dump.py /home/fdcavalcanti/nuttxspace5/nuttx -b build --target_info

On both cases output works fine.

Adds a build_path argument for when CMake builds are used.
This solves a problem when building the nxdiag tool with CMake.

Signed-off-by: Filipe Cavalcanti <filipe.cavalcanti@espressif.com>
@linguini1 linguini1 merged commit ec9a849 into apache:master May 20, 2026
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Tooling Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants