fix(native): drop ctt-lcd pin back to 0.4.0 (0.4.1 was never released) - #49
Open
meelyn-pandit wants to merge 1 commit into
Open
fix(native): drop ctt-lcd pin back to 0.4.0 (0.4.1 was never released)#49meelyn-pandit wants to merge 1 commit into
meelyn-pandit wants to merge 1 commit into
Conversation
f9b6743 bumped ctt-lcd 0.4.0 -> 0.4.1, but no ctt-lcd-v0.4.1 release was ever cut. install-native.sh fetches prebuilt armhf binaries by release tag and never compiles, so every OTA on this branch 404s on releases/download/ctt-lcd-v0.4.1/ctt-lcd-0.4.1-armhf and warns "keeping existing binary, will retry next OTA" — forever. Observed on both a v3.0 and a v3.3 board. The bump carried no code: diffing the 0.4.0 release commit (63cccc5) against f9b6743 over native/src/ctt-lcd/ and the PCF8574 LCD chip driver shows only the VERSION file changing. Every earlier ctt-lcd bump (0.1.0 -> 0.4.0) shipped real source changes; this one didn't. f9b6743's message credits the bump to a boot splash "Booting up...", but the splash string is " CTT Sensor Station" at both versions (added back at 0.3.0), so no such change is present. Rather than cut an identical binary under a new number, point the pin back at the release that matches the source. Verified: this tree's ctt-lcd source is identical to the published ctt-lcd-v0.4.0 tag, so the 0.4.0 binary already installed on the fleet is the correct build for it. If a boot-splash change was actually intended, it should land as a real source commit and then be released as 0.4.1 — this revert makes that gap visible instead of masking it behind a permanent OTA warning. Note: the pin only ever existed on web-wifi-manual-entry, not lts_26_07.iso, so no station outside that branch was affected. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
install-native.shfetches prebuilt armhf binaries by release tag and never compiles on the station.f9b6743bumped thectt-lcdpin to 0.4.1, but noctt-lcd-v0.4.1release was ever cut — the latest published isctt-lcd-v0.4.0(2026-06-23). So every OTA on this branch 404s:Non-fatal by design, but it recurs on every update forever. Observed on both a v3.0 and a v3.3 board.
The bump carried no code
Diffing the 0.4.0 release commit (
63cccc5) againstf9b6743across thectt-lcdsource and its PCF8574 LCD chip driver:Only the VERSION file. Every earlier
ctt-lcdbump (0.1.0 → 0.4.0) shipped real source changes; this one didn't.f9b6743's message credits the bump to a boot splash"Booting up...", but the splash string is" CTT Sensor Station"at both versions (added back at 0.3.0,6404902) — so no such change is present in the tree.Fix
Point the pin back at the release that matches the source, rather than cutting an identical binary under a new number.
Verified this tree's
ctt-lcdsource is identical to the publishedctt-lcd-v0.4.0tag:So the 0.4.0 binary already installed on the fleet is the correct build for this source — no station needs a new download, and
install-nativewill reportctt-lcd already at 0.4.0and stop warning.Notes for review
web-wifi-manual-entry.f9b6743exists only on this branch (git branch -r --contains f9b6743→origin/web-wifi-manual-entryalone), so the bad pin never reachedlts_26_07.isoand no station outside this branch was affected.f9b6743's author. This revert makes that gap visible rather than masking it behind a permanent OTA warning; the alternative (cutting actt-lcd-v0.4.1release from identical source) would have silenced the warning while leaving the missing change unnoticed.CHANGELOG.mdneeds no update — it never referenced 0.4.1.🤖 Generated with Claude Code