-
Notifications
You must be signed in to change notification settings - Fork 1.8k
[Bug]: Doesn't compile for Teensy 4.1 #1110
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
Comments
Sorry Max, |
Is it acceptable to use PlatformIO for the CI with Teensy boards? That's like a 2 minute job. I don't feel like massaging a non-conforming Arduino core to work with arduino-cli when PlatformIO works fine. Of course the build will fail though without code modifications. |
Actually, hold up. Looking at https://www.pjrc.com/teensy/td_download.html it looks like it installs regularly now without the custom installer just with the board package URL. So.. it should already work with arduino-cli? |
How do you do the CI with Platform IO? It might be acceptable, but I doubt the 2 minutes 😀 |
You do it with the template given in the official documentation: https://docs.platformio.org/en/latest/integration/ci/github-actions.html This is literally just installing Python, PlatformIO and executing the dedicated |
Are you allowed to see the actions of this repo? |
Yes, the |
The type of board you are using.
Teensy board
The boards name or FQBN (e.g. esp8266:esp8266:d1_mini:eesz=4M3M,xtal=80) for unspecified board.
Teensy 4.1
The board manager URL used for unspecified board.
See ArduinoTeensy
What IDE are you using?
PlatformIO IDE
What IR protocol are you using?
Unknown
Pin(s) used for IR-receive, if not default.
Arduino pin number: default
Example(s) you have checked while hunting the bug.
Example(s) to reproduce the issue.
The library version you are working with.
What are the steps to reproduce this issue?
src/
What happens?
Fails to compile.
The serial output which indicates the error happened.
What were you expecting to happen?
Compilation os OK.
Additional context.
Although #928 was merged, this fails to compile.
Final checklist for the bug report.
The text was updated successfully, but these errors were encountered: