CMake COMPONENT_BlueNRG_MS: fix mbed-ble

[jeromecoutant]

Summary of changes

Fix ARMmbed/mbed-os-example-ble#364

BLE component was not compiled for:

• DISCO_L562QE
• DISCO_L475VG_IOT01A
• B_L4S5I_IOT01A

@boraozgen

Impact of changes

Migration actions required

Documentation

---

Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

---

Test results

[] No Tests required for this change (E.g docs only update)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

---

Reviewers

---

[ciarmcom]

@jeromecoutant, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

[boraozgen]

Fixes the issue. I would also apply this to other controllers (BlueNRG_2 etc.) and remove the mbed-ble-blue_nrg target from mbed-os/connectivity CMake file.

BTW during testing with DISCO_L562QE and BLE_GattServer_CharacteristicWrite example, I noticed that sometimes it hangs after Ble process started.. It starts working after a couple resets. Seems totally random. Any ideas?

[jeromecoutant]

Fixes the issue. I would also apply this to other controllers (BlueNRG_2 etc.) and remove the mbed-ble-blue_nrg target from mbed-os/connectivity CMake file.

@hugueskamba @0xc0170
Maybe you could provide a global correction and close this one ?

[0xc0170]

Can you rewrite the commit - this is fixing an error - fixing the target (mbed-ble) rather than cli2 support (related).

[0xc0170]

Maybe you could provide a global correction and close this one ?

Yes, we can do. Let's close this one and create a new issue about this. @jeromecoutant Can you create new issue? i'll triage it.

[0xc0170]

mbed-ble-blue_nrg target from mbed-os/connectivity CMake file.

@jeromecoutant Can this fix this component and we can fix others separately?

[jeromecoutant]

You can start CI or close this one and make a better one!
Up to you !

[0xc0170]

I'll start CI, please also remove mbed-ble-blue_nrg from connectivity as stated above (add_library is there).

Pull request has been modified.

[0xc0170]

I see force push but I do not see add_library(mbed-ble-blue_nrg INTERFACE) removed from connectivity/CMakeLists.txt

[jeromecoutant]

I see force push but I do not see add_library(mbed-ble-blue_nrg INTERFACE) removed from connectivity/CMakeLists.txt

Done

[0xc0170]

cI restarted

[mbed-ci]

Jenkins CI Test : ✔️ SUCCESS

Build Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs	Status
jenkins-ci/mbed-os-ci_unittests	✔️
jenkins-ci/mbed-os-ci_cmake-example-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_cmake-example-ARM	✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM	✔️
jenkins-ci/mbed-os-ci_build-example-ARM	✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_cmake-cloud-example-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_build-cloud-example-ARM	✔️
jenkins-ci/mbed-os-ci_cmake-cloud-example-ARM	✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_cmake-example-test	✔️
jenkins-ci/mbed-os-ci_greentea-test	✔️