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

Fix build to not use incorrect OneButton version #4374

Merged
merged 5 commits into from
Aug 2, 2024

Conversation

geeksville
Copy link
Member

OneButton pushed out a new update today that has a different API rather than just use whichever new version they push, stay on 2.5.x until someone sees a need to update. Fixes build for wm1100 tracker.

OneButton pushed out a new update today that has a different API
rather than just use whichever new version they push, stay on 2.5.x until
someone sees a need to update.  Fixes build for wm1100 tracker.
@geeksville geeksville marked this pull request as ready for review August 2, 2024 16:46
@geeksville
Copy link
Member Author

btw @thebentern - I think the reason recent CI builds are now failing is because of this also.

@thebentern
Copy link
Contributor

@geeksville yep it definitely is 😅
I actually submitted a fix PR and it got merged and released upstream so I think we can peg this one to 2.6.1
https://github.com/mathertel/OneButton/releases/tag/2.6.1

@geeksville
Copy link
Member Author

ooh I see !
2.6.1 will have your fix bravo!

@geeksville
Copy link
Member Author

hmm @thebentern it seems like the platformio version fetching isn't working with their release name for 2.6.1. It only finds 2.6.0 if the platformio.ini file is updated. I think they might have strict rules and https://github.com/mathertel/OneButton/releases botched the name of the release (not calling it literally 2.6.1 but instead 'fixing blah blah). This apparently breaks the github auto version resolution for 'releases' (notice how interestingly https://github.com/mathertel/OneButton says the latest release is 2.6.0 still)

@thebentern
Copy link
Contributor

Weird. Perhaps we can just use the github tag?

@thebentern thebentern merged commit 703da1d into meshtastic:master Aug 2, 2024
100 of 107 checks passed
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