Document how to use custom mbed-os version in platformio#214
Document how to use custom mbed-os version in platformio#214douardda wants to merge 1 commit intoplatformio:developfrom
Conversation
frameworks/mbed_extra.rst
Outdated
| Custom version of Mbed | ||
| ^^^^^^^^^^^^^^^^^^^^^^ | ||
|
|
||
| PlatformIO comes with only a few versions of Mbed supported out of the box. If you want |
There was a problem hiding this comment.
I'd change this wording to something like "PlatformIO only has certain versions of mbed-os in the official package package registry, as queryable through the API", just to give people a pointer as to where to find natively supported versions.
|
The general problem with telling people how to use custom versions is that they then think
This is definitely useful information for many people, but having it in official docs would give a false sense of "this always works, just do that". |
frameworks/mbed_extra.rst
Outdated
| } | ||
| EOF | ||
|
|
||
| Then one ca use the following `platformio.ini` file: |
There was a problem hiding this comment.
slight typo in ca -> can here
| platform = ststm32 | ||
| framework = mbed | ||
| platform_packages = | ||
| framework-mbed @ file://framework-mbed |
There was a problem hiding this comment.
- configuration is missing
board = ...line to make it complete, does not mention to add it - Maybe general link to https://docs.platformio.org/en/latest/projectconf/section_env_platform.html#platform-packages would be good
|
Thanks a lot @maxgerhardt I'll try to address your comments ASAP. |
I agree and propose to include a big disclamer in this section. |
4fb2179 to
0376a40
Compare
|
I've updated a bit the PR, hopefully addressing all your comments. Thanks again. |
|
Bump...merge this? |
No description provided.