fdcavalcanti opened a new pull request, #18919:
URL: https://github.com/apache/nuttx/pull/18919

   ## Summary
   
   <!-- This field should contain a summary of the changes. It will be 
pre-filled with the commit's message and descriptions. Adjust it accordingly -->
   
   * 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 PR ####.
   
   ## Impact
   <!-- Please fill the following sections with YES/NO and provide a brief 
explanation -->
   
   Impact on user: No.
   <!-- Does it impact user's applications? How? -->
   
   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.
   <!-- Does it impact on building NuttX? How? (please describe the required 
changes on the build system) -->
   
   Impact on hardware: No.
   <!-- Does it impact a specific hardware supported by NuttX? -->
   
   Impact on documentation: No.
   <!-- Does it impact the existing documentation? Please provide additional 
documentation to reflect that -->
   
   Impact on security: No.
   <!-- Does it impact NuttX's security? -->
   
   Impact on compatibility: No.
   <!-- Does it impact compatibility between previous and current versions? Is 
this a breaking change? -->
   
   ## Testing
   <!-- Please provide all the testing procedure. Consider that upstream 
reviewers should be able to reproduce the same testing performed internally -->
   Build with Make works fine, with CMake it would fail since build and source 
directory differs.
   
   
   ### Building
   <!-- Provide how to build the test for each SoC being tested -->
   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
   <!-- Provide how to run the test for each SoC being tested -->
   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.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to