-
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
lib: nrf_modem_lib: Turn off CONFIG_COAP_CLIENT_TRUNCATE_MSGS #17587
base: main
Are you sure you want to change the base?
lib: nrf_modem_lib: Turn off CONFIG_COAP_CLIENT_TRUNCATE_MSGS #17587
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 72c1ec50740bb8762fe603155bd955561fe6ab45 more detailssdk-nrf:
Github labels
List of changed files detected by CI (1)
Outputs:ToolchainVersion: 9583beca34 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commit and PR title should say lib: nrf_modem_lib
though.
0a20363
to
205ff15
Compare
205ff15
to
3000bf9
Compare
This upstream Kconfig defaults to y, but is not compatible with the modem's socket implementation. Signed-off-by: Pete Skeggs <[email protected]>
3000bf9
to
72c1ec5
Compare
This is to prepare for eventual upmerge of zephyrproject-rtos/zephyr#79155. An earlier version of that is included in the pre-NCS 2.8.0 upmerge already, but that earlier version had the new Kconfig defaulting to n.
This upstream Zephyr PR will default it to y, but is not compatible with the modem's socket implementation, so turn it off here.