Skip to content

system/nxdiag: add CMake support for NXDiag tool#3491

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

system/nxdiag: add CMake support for NXDiag tool#3491
xiaoxiang781216 merged 1 commit into
apache:masterfrom
fdcavalcanti:feature/nxdiag-cmake-support

Conversation

@fdcavalcanti
Copy link
Copy Markdown
Contributor

Summary

  • system/nxdiag: add CMake support for NXDiag tool

Supports building NXDiag with CMake.

Relates to this PR on NuttX.

Impact

Impact on user: No.

Impact on build: Supports CMake builds.

Impact on hardware: No.

Impact on documentation: No.

Impact on security: No.

Impact on compatibility: No.

Testing

Tested on esp32h2-devkit with Make and CMake.

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

Results

  • nxdiag is available on Nuttshell:
nsh> ?
help usage:  help [-v] [<cmd>]

    .           cp          expr        mount       set         truncate    
    [           cmp         false       mv          kill        uname       
    ?           dirname     fdinfo      pidof       pkill       umount      
    alias       df          free        printf      sleep       unset       
    unalias     dmesg       help        ps          usleep      uptime      
    basename    echo        hexdump     pwd         source      watch       
    break       env         ls          reboot      test        xd          
    cat         exec        mkdir       rm          time        wait        
    cd          exit        mkrd        rmdir       true        

Builtin Apps:
    dd           nsh          nxdiag       getprime     
    dumpstack    sh           ostest       
nsh> 

Supports building NXDiag with CMake.

Signed-off-by: Filipe Cavalcanti <filipe.cavalcanti@espressif.com>
Copy link
Copy Markdown
Contributor

@cederom cederom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @fdcavalcanti :-)

@xiaoxiang781216 xiaoxiang781216 merged commit 43bc7e9 into apache:master May 21, 2026
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants