acassis commented on code in PR #18872:
URL: https://github.com/apache/nuttx/pull/18872#discussion_r3234521374


##########
fs/Kconfig:
##########
@@ -94,6 +94,28 @@ config PSEUDOFS_ATTRIBUTES
                Enable support for attributes(e.g. mode, uid, gid and time)
                in the pseudo file system.
 
+config FS_PERMISSION
+       bool "Enable filesystem permission checks"
+       default n
+       depends on !DISABLE_PSEUDOFS_OPERATIONS
+       select SCHED_USER_IDENTITY

Review Comment:
   @Abhishekmishra2808 you can add a comment just after your filesystem 
permission to instruct users to enable the dependencies, some Kconfig already 
uses it, something like:
   
   ```
   comment "SCHED_USER_IDENTIFY=y and PSEUDOFS_ATTRIBUTES=y and 
DISABLE_PSEUDOFS_OPERATIONS=n are needed to support Unix Filesystem Support"
       depends on !SCHED_USER_IDENTIFY || !PSEUDOFS_ATTRIBUTES || 
DISABLE_PSEUDOFS_OPERATIONS
   ``



-- 
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]

Reply via email to