-
Notifications
You must be signed in to change notification settings - Fork 19
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
build: test against upcoming arduino-esp32 3.0 #113
Conversation
Not sure why the build fails with:
Looks like it's not supported by platformio yet: platformio/platform-espressif32#1225 |
I'm looking forward to ESP-IDF 5.x since I hope it will support native USB on ESP32-S3. But I fear that we may run into compatibility issues with running Edrumulus on the old ESP32 chip. But let's see...
Would be interesting to see if changes are necessary. At least we should avoid using the new API since then we would not be compatible to the old IDF anymore.
Ok, I guess in this case you should close this pull request and create a new one as soon as it is supported? |
I'll make it a draft PR if that's ok so I can use this to test. |
I have a pull request up for platform-espressif32 that fixes the error: KeyError: 'framework-arduinoespressif32-libs': platformio/platform-espressif32#1281 You can use the branch in that PR with arduino-esp32#master, but also need to add the location of the library to platform packages overrides as well. (The changes are quite small, to just add the library to the list of dependencies)
You can see if your test branch gets past the libs error with the above. Note that alpha v3.0.0 has breaking changes, so old code might not work or even compile, the above branch will at least get past the missing libs to the actual compile stage. |
@thijstriemstra do you have time to check the new code provided by sgryphon? |
@thijstriemstra, I have created a new Pull Request #144 based on your code and the patch from sgryphon which replaces your Pull Request here. @sgryphon, thank you for your patch. It works fine :-) I'll close this Pull Request now. |
Test against upcoming arduino-esp32 3.0, see https://blog.espressif.com/announcing-the-arduino-esp32-core-version-3-0-0-3bf9f24e20d4
There's also a migration guide for 2.x to 3.x: https://docs.espressif.com/projects/arduino-esp32/en/latest/migration_guides/2.x_to_3.0.html