-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mgmt: fmfu: Hide MGMT_FMFU logging options when not enabled #19951
base: main
Are you sure you want to change the base?
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: fec7cdbf71d461f0487832da0ccbbe6005fb2b4f more detailssdk-nrf:
Github labels
List of changed files detected by CI (1)
Outputs:ToolchainVersion: 342151af73 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. Note: This comment is automatically posted by the Documentation Publish GitHub Action. |
When building an application we don't want CONFIG_MGMT_FMFU_LOG_LEVEL_DEFAULT to turn up in the generated .config file unless CONFIG_MGMT_FMFU is enabled. Signed-off-by: Rubin Gerritsen <[email protected]>
1bb9b0a
to
fec7cdb
Compare
@nrfconnect/ncs-pluto , would you mind having a look at this one? |
already did |
Need one more approver :) |
@nrfconnect/ncs-pluto would you mind having a look? |
When building an application we don't want
CONFIG_MGMT_FMFU_LOG_LEVEL_DEFAULT to turn up in the generated .config file unless CONFIG_MGMT_FMFU is enabled.