Skip to content

nxinit/Kconfig: Change --help-- to ---help---#3507

Closed
lupyuen wants to merge 1 commit into
apache:masterfrom
lupyuen260528:master
Closed

nxinit/Kconfig: Change --help-- to ---help---#3507
lupyuen wants to merge 1 commit into
apache:masterfrom
lupyuen260528:master

Conversation

@lupyuen
Copy link
Copy Markdown
Member

@lupyuen lupyuen commented May 28, 2026

Summary

PR #3504 introduced a typo in nxinit/Kconfig: --help--. This PR changes it to ---help--- to fix this build error:

$ tools/configure.sh rv-virt:nsh
apps/system/nxinit/Kconfig:81: syntax error
apps/system/nxinit/Kconfig:80: unknown option "--help--"

https://github.com/lupyuen/nuttx-riscv64/actions/runs/26546602390/job/78199683651#step:5:162

Impact

configure.sh now works correctly, see below.

Testing

## Tested on Ubuntu 24.04.2 LTS x86_64
## gcc version 13.2.0 (xPack GNU RISC-V Embedded GCC x86_64) 
$ tools/configure.sh rv-virt:nsh
  Copy files
  Select CONFIG_HOST_LINUX=y
  Refreshing...
  configuration written to .config
$ make -j 8
CPP:  /tmp/fix-nxinit/nuttx/boards/risc-v/qemu-rv/rv-virt/scripts/ld.script-> /tmp/fix-LD: nuttx

PR apache#3504 introduced a typo in nxinit/Kconfig: `--help--`. This PR changes it to `---help---` to fix this build error:
```
$ tools/configure.sh rv-virt:nsh
apps/system/nxinit/Kconfig:81: syntax error
apps/system/nxinit/Kconfig:80: unknown option "--help--"
```
https://github.com/lupyuen/nuttx-riscv64/actions/runs/26546602390/job/78199683651#step:5:162

Signed-off-by: Lup Yuen Lee <luppy@appkaki.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 @lupyuen :-)

@linguini1
Copy link
Copy Markdown
Contributor

Haha @lupyuen , I noticed the same issue and the patch for this is also part of #3506

@lupyuen
Copy link
Copy Markdown
Member Author

lupyuen commented May 28, 2026

@linguini1 Let's merge your PR then thanks :-)

@lupyuen lupyuen closed this May 28, 2026
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