-
Notifications
You must be signed in to change notification settings - Fork 3k
CMake: Rename CMake targets #13803
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: Rename CMake targets #13803
Conversation
@hugueskamba, thank you for your changes. |
@rajkan01 Please review, once approved, I'll merge examples and this one. |
* mbed-os renamed mbed-core * mbed-os-<COMPONENT> renamed mbed-<COMPONENT>
This ensures that applications which link with the mbed-os CMake target still work. We want applications that require the OS to use the mbed-os CMake target as we will be creating a new one that will include mbed-core and mbed-rtos.
a178bb8
to
b1512ce
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
Travis is pending for already more than a hour, queue is empty 😭 Trying to find out what is going on. |
Tested with 2 examples, as Travis is not progressing and this is blocking other PRs. I'll merge this to the feature branch. |
This is part of the work to create a baremetal configuration.
mbed-os
renamedmbed-core
mbed-os-<COMPONENT>
renamedmbed-<COMPONENT>
Also, create a
mbed-os
alias library.This ensures that applications which link with the
mbed-os
CMake target still work. We want applications that require the OS
to use the
mbed-os
CMake target as we will be creating a new onethat will include
mbed-core
andmbed-rtos
.Summary of changes
Impact of changes
All applications supported will need to be amended to include the component libraries with the
mbed-
prtefix instead ofmbed-os-
.Migration actions required
The following PRs need to be merged for the CI to pass:
Documentation
Pull request type
Test results
Reviewers