-
Notifications
You must be signed in to change notification settings - Fork 3k
Make the presence of cmsis-pack-manager package optional #10089
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
@arekzaluski Do you need this change for Studio? |
@theotherjimmy thanks for bringing @arekzaluski into the loop. Yeah if at all possible, this should be in the final release candidate for 5.12.0 if the current version of Mbed Studion (0.4.0) is to have any way of using 5.12.0. |
That's unfortunate, but a pretty good reason to get this into RC3. |
CI started |
Test run: SUCCESSSummary: 13 of 13 test jobs passed |
Does that mean the next studio version will contain a fix for this ? |
Please let us know. This can go to master at least now |
@0xc0170 @bridadan @theotherjimmy Yes we need it, we will test it today in Mbed Studio. Thanks @bridadan |
Description
Currently, the 5.12.0 release candidates do not build in 0.4.0 Mbed Studio. This patch will allow you to use the "active program tools" in 5.12.0 projects to build correctly. The issue is that Mbed Studio has limited Python environment that does not contain the cmsis-pack-manager Python package. Since we can't install this at this time for Mbed Studio, this make the package optional so you can still compile and use the local CMSIS Pack index.
FYI @SenRamakri @thegecko
Pull request type
Reviewers
Release Notes