docker-library / docker

Docker Official Image packaging for Docker
Apache License 2.0
1.14k stars 582 forks source link

re-enable arm32v6 and arm32v7 builds #447

Closed thaJeztah closed 1 year ago

thaJeztah commented 1 year ago

Builds for these architectures were broken for some time, but this should be resolved now, so we can re-enable these (fingers crossed).

This reverts commit 461a3ec6ed346995c5949cc2d70c08b9d049caa0.

thaJeztah commented 1 year ago

@tianon @yosifkit PTAL (not exactly sure how to verify this locally, so just push the PR)

thaJeztah commented 1 year ago

CI looks happy, and I see some armhf / armv6 strings in the logs (but not sure if it actually did those builds and tested them; it's possible other changes are needed to re-enable 🙈)

thaJeztah commented 1 year ago

Thanks for checking!

tianon commented 1 year ago

I tested on an arm32v7 box and they seem OK; my v6 box is less accessible, so it seems pretty reasonable IMO to roll the dice and assume it's fixed. :smile: