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

cmake: sysbuild: b0/nrf700x: Fix not configuring string #16894

Merged
merged 2 commits into from
Aug 23, 2024

Conversation

nordicjm
Copy link
Contributor

Fixes issues with strings not being expanded

Fixes the key variable not being expanded for b0n image signing

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Fixes the key variable not being expanded for nrf700x image
signing

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
@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 Aug 16, 2024
@nordicjm nordicjm requested a review from 57300 August 16, 2024 07:13
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 16, 2024

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 1

Inputs:

Sources:

sdk-nrf: PR head: a7d878139c9a9f0688be52e039ed7a5915916716

more details

sdk-nrf:

PR head: a7d878139c9a9f0688be52e039ed7a5915916716
merge base: 5d3c406c30e831ea14eb3ec27cb30db29674d08d
target head (main): 5d3c406c30e831ea14eb3ec27cb30db29674d08d> Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (2)
cmake
│  ├── sysbuild
│  │  ├── b0_mcuboot_signing.cmake
│  │  │ image_signing_nrf700x.cmake

Outputs:

Toolchain

Version: cb46c8ba33
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:cb46c8ba33_81ed5a52d6

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 1242
  • ✅ Integration tests
    • ◻️ test-fw-nrfconnect-boot
    • ◻️ test-sdk-find-my
    • ◻️ test-sdk-mcuboot
    • ◻️ test-sdk-dfu
Disable integration tests
    • desktop52_verification
    • doc-internal
    • latest/ble/nrf/test_ble_commit
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi

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

@nordicjm nordicjm removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 23, 2024
@nordicjm nordicjm requested a review from gmarull August 23, 2024 09:27
@nordicjm nordicjm merged commit d49ccff into nrfconnect:main Aug 23, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants