-
Notifications
You must be signed in to change notification settings - Fork 435
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
Update PlatformIO builder script + docs #612
Comments
Argh, sorry for breaking your build. I need to do a Platform.IO job as part of CI. :( |
Pretty sure it did not break the build (e.g., |
Hi @maxgerhardt and @earlephilhower, We have received 35 packages for review in the PlatformIO registry. What is the further fate of these packages? platformio/platform-raspberrypi#13 - Could you provide a PR with support for this core? Thanks! |
Ideally we'd want to have the pico-toolchain, Raspberrypi's tool-openocd version and |
All packages are approved => https://registry.platformio.org/search?q=owner%3Aearlephilhower&t=tool Please tell us which packages are not required, we will remove them to keep the registry clean. |
Do we need a custom ARM toolchain and OpenOCD? |
Hmm I think I had problems with the tool-openocd-raspberrypi version tha PlatformIO had (1 year old) when trying PicoProbe etc. support, so a newer build is needed, as provided by https://github.com/earlephilhower/pico-quick-toolchain/releases. And as far as I understand it, this core builds on custom configuration + features of the ARM toolchain, e.g. with exception code etc. (linking against
Great, I'll see if I can work with that. As detailed in this issue, we probably need a small builder script fixup and release 2.0.4 to get full / equal build options support with the Arduino IDE. |
Thanks for the provided information. We would be thankful for the PR. I've just done a quick look at your fork and it looks good. The only note is to use |
@ivankravets PR created for platform-raspberrypi per above, PR for PlatformIO docs and here builder script + doc update here will follow. |
Commits
require an update of docs in regards to
require an update of the builder script in regards to
-lstdc++
or-lstdc++-exc
depending on whether exceptions are enabled (-fexceptions
found in build flags)(reminder for me later)
The text was updated successfully, but these errors were encountered: