-
Notifications
You must be signed in to change notification settings - Fork 1k
Add initial support for PlatformIO ecosystem #924
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
Hi @valeros and thanks About the CI,did you plan or is it already done to move on GitHub actions ? I'm currently working to move our build and astyle checks on it. Linked to platformio/platform-ststm32#345 |
Hi @fpistm !
Sure thing, ping me if there is something wrong with our build.
It should't be hard to port PlatformIO env to Github Actions, we probably can create a standalone workflow for it. Let me know when you're ready to switch to Github Actions. Thanks! |
@valeros |
Hi @fpistm ! What would you like to parse from |
I talk about those kind of hardcoded stuff: and several other hardcoded definition |
Hi @fpistm , We have a feature request for this platformio/platformio-core#969. PlatformIO does not work with CC or C++ to build commands. We need a strict declaration of each flag in its special environment scope. Our users control the whole build process, they can add or remove any flag dynamically from a specific scope. The same, we have Library Dependency Finder which is built on top of virtual C Preprocessor based on Python. This is why PlatformIO is super fast and requires minimal memory and CPU resources. You can check:
We can actually use intermediate interface (JSON file or etc.) where you will provide all flags related to GCC Preprocessor scopes. Then you can use some script which will compress all of them into the 1 line for Arduino. P.S: We don't have the aim to reinvent a wheel. We resolve the problems around the project building and maintaining. |
@ivankravets |
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.
Thanks @valeros and @ivankravets
I will apply black and flake8 later.
Thanks !!! Please do not hesitate to contact us if something builds with Arduino IDE and does not build with PlatformIO. We will help ASAP. |
This PR adds build script for @platformio and a new Travis CI environment for testing internal libraries using
platformio ci
command.