Skip to content

Pin LovyanGFX to 1.2.21 to fix build failure on fresh installs - #49

Open
selmapi wants to merge 1 commit into
MatixYo:mainfrom
selmapi:fix/pin-lovyangfx-1.2.21
Open

Pin LovyanGFX to 1.2.21 to fix build failure on fresh installs#49
selmapi wants to merge 1 commit into
MatixYo:mainfrom
selmapi:fix/pin-lovyangfx-1.2.21

Conversation

@selmapi

@selmapi selmapi commented Jul 5, 2026

Copy link
Copy Markdown

The lovyan03/LovyanGFX@^1.2.7 constraint in platformio.ini currently resolves to 1.2.24 (published 2026-06-24), which declares its own fonts namespace alias that collides with this project's namespace fonts = lgfx::v1::fonts; alias in radar_display.cpp, runway_overlay.cpp, and status_screens.cpp — three compile errors on any fresh clone or CI cache miss. The exact 1.2.7 floor also fails to build, since it predates the two-argument loadFont() used in display_font.cpp, so the safe range is narrower than the current constraint implies.

Pinning to 1.2.21 — the newest version confirmed to build cleanly against this code — stops the build from depending on a stale library cache to stay green. (Note: pio pkg install re-resolves floating ranges against the registry even with a matching cached version, which is why the cache workaround is fragile.)

This pin has been running in a fork of this project in daily use with no issues. Thanks for a great project!

🤖 Generated with Claude Code

^1.2.7 resolves to 1.2.24, whose new 'fonts' alias collides with this
project's; exact 1.2.7 predates the two-arg loadFont(). 1.2.21 is the
newest version that builds cleanly.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@selmapi
selmapi force-pushed the fix/pin-lovyangfx-1.2.21 branch from 48e1ae3 to f74ecf0 Compare July 5, 2026 14:10
@aaugusty

aaugusty commented Jul 6, 2026

Copy link
Copy Markdown

I was able to successfully build by commenting out namespace fonts = lgfx::v1::fonts; alias in radar_display.cpp, runway_overlay.cpp, and status_screens.cpp as a separate workaround.

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