ThePassionate opened a new pull request, #3349: URL: https://github.com/apache/nuttx-apps/pull/3349
## Summary This PR fixes the compilation error in system/cu and adds improvements: 1. **Fix signal handler to use sa_sigaction** - The previous commit `782de2748f` introduced a bug where `sigint()` function uses `siginfo->si_user` but was declared with only one parameter `(int sig)`. This causes compilation error because `siginfo` is undeclared. Fixed by using `sa_sigaction` with proper 3-parameter signature. 2. **Merge cu.h to cu_main.c** - Simplify code structure by merging the header file contents directly into cu_main.c. 3. **Support SIGNAL disabled configuration** - Use `CONFIG_DISABLE_SIGNALS` to conditionally compile signal handling code, allowing cu to work on systems with signals disabled. ## Impact Fixes CI build error and improves cu tool portability. ## Testing Build tested with and without CONFIG_DISABLE_SIGNALS. -- 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]
