-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add native ARMv4T, ARMv5TE and ARMv8-A support #3801
Conversation
I think you also will need to fix up the tests for these, too. e.g https://github.com/pi-hole/pi-hole/blob/development/test/test_automated_install.py#L190 |
f09654d
to
3e5235d
Compare
…and ARMv8-A binaries we provide for FTL. Signed-off-by: DL6ER <dl6er@dl6er.de>
e9b35d6
to
01c9bbd
Compare
Had to force-push once again to trigger the Travis build (it was always picking the second last commit, otherwise) |
@PromoFaux Tests are all ✔️ |
pi-hole/FTL#903 should be merged (already approved) immediately after merging this PR |
I'll let you do the honours! |
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/pihole-ftl-for-armv5l-precompiled/32645/2 |
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/pi-hole-core-web-v5-2-and-ftl-v5-3-released/40909/1 |
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/ftl-install-failure-on-change-of-ip-address/41592/16 |
By submitting this pull request, I confirm the following:
git rebase
)What does this PR aim to accomplish?:
Add support for pre-compiled ARMv4R, ARMv5TE and ARMv8-A FTL binaries
How does this PR accomplish the above?:
Improve detection and add new criteria
Dependencies of this PR
This PR depends on pi-hole/docker-base-images#31 (:heavy_check_mark:) and should be merged at the same time as pi-hole/FTL#903 (already approved)
What documentation changes (if any) are needed to support this PR?:
None