-
Notifications
You must be signed in to change notification settings - Fork 3k
CMake: Add support for all Silicon Labs Mbed targets #13959
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
Conversation
Started CI |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
617a904
to
6d289d1
Compare
Pull request has been modified.
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
6d289d1
to
d0a05a7
Compare
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.
LGTM just one removal not clear to me if it is part of this PP
tools/cmake/README.md
Outdated
- K64F | ||
- K66F | ||
- NRF52840_DK | ||
- WIO_3G |
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.
is this removal intentional?
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.
Yes it was because it is a STM target.
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
d0a05a7
to
f92e028
Compare
Jenkins CI Test : ❌ FAILEDBuild Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
I'll rebase and restart CI once ready |
f92e028
to
1005b7c
Compare
CI restarted |
Jenkins CI Test : ❌ FAILEDBuild Number: 3 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Jenkins CI Test : ✔️ SUCCESSBuild Number: 4 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
Add
CMakeLists.txt
so all Freescale Mbed targets can be built with CMake.Impact of changes
Migration actions required
Documentation
Pull request type
Test results
Reviewers