Open danhunsaker opened 2 years ago
Thanks @danhunsaker, sorry for the late response but I'm a bit low on time lately. @elijahr looks good to me and it's a step toward the removal of the Dockerfiles.
No rush. The folks over at the archarm repo still haven't responded, so you're doing better than they are. 😁
@uraimo just a heads up, I'm attending to some personal matters and likely won't have time to review PRs here for at least a month.
@elijahr sure don't worry, everything else comes first.
hi there! any chance we could get this to work?
This PR does a couple of things. Primarily, it passes the target architecture to the various Docker commands so that it will stop complaining about not being told in advance. (resolves #21). Secondarily, it groups output lines for easier debugging and suchforth (resolves #62).
As an added bonus, it switches to a fork of archarm with more recent images, which solves the issue seen in testing and downstream use cases where a full system upgrade refuses to complete due to insufficient storage space (resolves #48). This part is expected to be reverted once the upstream merges agners/archlinuxarm-docker#8 and their own builds resume, but who knows how long that might take. The expected reversion is why it's in its own commit, here.