Skip to content
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 Task.h #40

Merged
merged 2 commits into from
Jan 9, 2023
Merged

Update Task.h #40

merged 2 commits into from
Jan 9, 2023

Conversation

Chris685
Copy link
Contributor

@Chris685 Chris685 commented Jan 8, 2023

Fixing Issue after Updating to ESP Arduino Version 3.1.0 #38

Fixing Issue after Updating to ESP Arduino Version 3.1.0 nrwiersma#38
@nrwiersma
Copy link
Owner

nrwiersma commented Jan 8, 2023

It looks like you need to update the build flags in the platform.ini to change the framework version.
Or use the platform to specify a version like here: https://docs.platformio.org/en/latest/projectconf/sections/env/options/platform/platform.html

@nrwiersma nrwiersma self-assigned this Jan 8, 2023
@nrwiersma
Copy link
Owner

Specifically you need to add this at the end of [env:nodemcuv2]:

; this should be replaced with the correct platform version once espressif8266 has caught up.
platform_packages =
    framework-arduinoespressif8266 @ https://github.com/esp8266/Arduino.git
    toolchain-xtensa@~2.100300.210717

Which fails based on cont_yield for the current version.

@Chris685
Copy link
Contributor Author

Chris685 commented Jan 9, 2023

Sorry, i dont understand this problem :(

@nrwiersma
Copy link
Owner

As a test, a Github action runs platformio ci to check it builds. By adding the above snippet to the end of the platformio.ini, it changes the version of ESP8266 Arduino used to the correct version, allowing the test to pass.

@Chris685
Copy link
Contributor Author

Chris685 commented Jan 9, 2023

Got it now, This was not as easy for me - thanks for your support - i try to learn to work with Github more and more :)

Copy link
Owner

@nrwiersma nrwiersma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@nrwiersma nrwiersma merged commit a2a1ff5 into nrwiersma:master Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants