Fix rpi-lgpio installation on non-ARM systems#2558
Open
astnerdev wants to merge 4 commits intoMiczFlor:future3/developfrom
Open
Fix rpi-lgpio installation on non-ARM systems#2558astnerdev wants to merge 4 commits intoMiczFlor:future3/developfrom
astnerdev wants to merge 4 commits intoMiczFlor:future3/developfrom
Conversation
- Added platform_machine condition to rpi-lgpio in requirements.txt - Prevents installation on non-Raspberry Pi systems
s-martin
approved these changes
Nov 11, 2025
Collaborator
|
Hi, the phoniebox software still supports the Pi1 Model. For this the architecture is "armv6l". This should also be added. |
Collaborator
|
Is this PR still necessary with #2568? |
# Conflicts: # requirements.txt
Pull Request Test Coverage Report for Build 19716136597Details
💛 - Coveralls |
Author
|
@s-martin From my point of view it is still required, otherwise the Dockerfile When building the Dockerfile, I get the following error:
To make both use cases possible, I extended the pull request. Docker is only needed for development (e.g. for web development), not for the Pi. |
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.
This update restricts the installation of
rpi-lgpioto Raspberry Pi architectures only (armv7loraarch64) using environment markers inrequirements.txt.Previously, the dependency caused installation errors on x86 or other non-ARM hosts, which resulted in failed Docker builds. With this change applied, the following command should no longer fail: