patacongo commented on code in PR #8924:
URL: https://github.com/apache/nuttx/pull/8924#discussion_r1152168433
##########
sched/Kconfig:
##########
@@ -622,6 +622,12 @@ config SCHED_THREAD_LOCAL
This option enables architecture-specific TLS support
(__thread/thread_local keyword)
Note: Toolchain must be compiled with '--enable-tls' enabled
+config UID_DEFAULT
Review Comment:
> used by builtin apps side if not configured.
If it effects apps/ (which are optional), then it should not be in an OS
Kconfig file. It should be an an apps/ Kconfig file.
This is a very strong coupling between those particular apps/ and the OS.
We much maintain modular interfaces; The apps/ directory is optional. The OS
must stand alone. There can be no dependency of the OS on anything in apps/
--
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]