pussuw commented on PR #6197: URL: https://github.com/apache/incubator-nuttx/pull/6197#issuecomment-1118412518
Maybe some dep error again, some targets fail due to multiple definitions of exit() ``` ==================================================================================== Configuration/Tool: qemu-intel64/nsh ------------------------------------------------------------------------------------ Cleaning... Configuring... Building NuttX... ld: /github/workspace/sources/nuttx/staging/libc.a(lib_exit.o): in function `exit': /github/workspace/sources/nuttx/libs/libc/stdlib/lib_exit.c:39: multiple definition of `exit'; /github/workspace/sources/nuttx/staging/libsched.a(exit.o):/github/workspace/sources/nuttx/sched/task/exit.c:72: first defined here make[1]: *** [Makefile:114: nuttx.elf] Error 1 make: *** [tools/Unix.mk:509: nuttx.elf] Error 2 make: Target 'all' not remade because of errors. /github/workspace/sources/nuttx/tools/testbuild.sh: line 257: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory Normalize qemu-intel64/nsh ``` I think at least the kernel side should never reference the libc version of exit(), because that is userland code. But the kernel side calls exit() in many places, expecting to use the kernel side implementation. -- 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: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org