[GitHub] [mynewt-core] ccollins476ad opened a new pull request #2277: sys/config: read/write access config for mgmt cmd

[GitBox <gi...@apache.org>]

ccollins476ad opened a new pull request #2277:
URL: https://github.com/apache/mynewt-core/pull/2277


   Before PR: the `CONFIG_MGMT` syscfg setting disabled or enabled the config newtmgr command in its entirety.  There was no way to enable only the read half of the command (for example).
   
   After PR: there is a second syscfg setting: `CONFIG_MGMT_RW`.  The semantics mirror the `CONFIG_CLI_RW` setting.  That is:
   
   * If `CONFIG_MGMT` is 0, the entire command is disabled.
   
   Otherwise:
   
   * If `CONFIG_MGMT_RW` is 1, only read commands are allowed.
   * If `CONFIG_MGMT_RW` is 2, only write commands are allowed.
   * If `CONFIG_MGMT_RW` is 3, both reads and writes are allowed.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [mynewt-core] apache-mynewt-bot commented on issue #2277: sys/config: read/write access config for mgmt cmd

[GitBox <gi...@apache.org>]

apache-mynewt-bot commented on issue #2277:
URL: https://github.com/apache/mynewt-core/pull/2277#issuecomment-617467042


   
   <!-- style-bot -->
   
   ## Style check summary
   
   #### No suggestions at this time!
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org