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

Default to v4.4 #58

Closed
wants to merge 2 commits into from
Closed

Default to v4.4 #58

wants to merge 2 commits into from

Conversation

MabezDev
Copy link
Member

Sister PR: esp-rs/esp-idf-template#9

AFAIK we can't change the esp-idf version in esp-idf? Therefore I removed the patching for pio.

Marking as draft until pio v4.4 release.

@MabezDev MabezDev requested a review from ivmarkov January 27, 2022 22:46
@MabezDev MabezDev force-pushed the feature/default-v4.4 branch from 9a72d48 to 2d88870 Compare January 27, 2022 22:47
@ivmarkov
Copy link
Collaborator

Thanks! I've accommodated bits and pieces of your PR here: 9d09145

That is, most of it except for not really switching to 4.4 as default (yet), as PlatformIO is still on 4.3.

By the way, last night we were hit by PlatformIO releasing their "espressif32" "platform" with 4.3.2 support (previously it was 4.3.1) and the esp-idf-sys stopped compiling under PIO as one of the v4.3 patches does no longer apply as it was merged upstream in 4.3.2). Fixed that here too: abebdf6

@ivmarkov ivmarkov closed this Jan 29, 2022
@ivmarkov ivmarkov deleted the feature/default-v4.4 branch January 29, 2022 07:22
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.

2 participants