Skip to content
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

manifest: update nrf_modem build system files #12648

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

lemrey
Copy link
Contributor

@lemrey lemrey commented Oct 11, 2023

Bring in build system updates.

@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest labels Oct 11, 2023
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 11, 2023

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@f8ceac1 nrfconnect/sdk-nrfxlib@6db0962 (main) nrfconnect/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

west.yml Outdated
@@ -141,7 +141,7 @@ manifest:
- name: nrfxlib
repo-path: sdk-nrfxlib
path: nrfxlib
revision: 15307eb68e1d465550c80f8dd81e0b12ae44a78c
revision: pull/1089/HEAD
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to use lowercase head

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 11, 2023

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf-iot_libmodem-nrf X
test-fw-nrfconnect-nrf-iot_lwm2m X
test-fw-nrfconnect-nrf-iot_mosh X
test-fw-nrfconnect-nrf-iot_positioning X
test-fw-nrfconnect-nrf-iot_samples X
test-fw-nrfconnect-nrf-iot_serial_lte_modem X
test-fw-nrfconnect-nrf-iot_zephyr_lwm2m X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

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.

@lemrey lemrey force-pushed the update-manifest branch 2 times, most recently from 26442ed to 04fff96 Compare November 14, 2023 14:55
@lemrey lemrey removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Nov 14, 2023
@lemrey
Copy link
Contributor Author

lemrey commented Nov 14, 2023

I do not think this change is visible to the user so I removed the changelog-entry-required label.

west.yml Outdated
@@ -141,7 +141,7 @@ manifest:
- name: nrfxlib
repo-path: sdk-nrfxlib
path: nrfxlib
revision: f8ceac162de19cbcee9652cb428b217b1cf85ab3
revision: pull/1089/head
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6db09621d01fd246d21010adc6ac8bdcb2a0e16a

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Nov 15, 2023
Bring in build system updates.
Update nrf_modem_lib Kconfig:
- REQUIRES_FULL_LIBC is selected by NRF_MODEM
- NRFX_IPC is select by NRF_MODEM
- depend on TRUSTED_EXECUTION_NONSECURE

Signed-off-by: Emanuele Di Santo <[email protected]>
@NordicBuilder NordicBuilder removed the DNM label Nov 15, 2023
@lemrey lemrey removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Nov 15, 2023
@anangl anangl merged commit 275b4fb into nrfconnect:main Nov 16, 2023
21 checks passed
@lemrey lemrey deleted the update-manifest branch November 16, 2023 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants