-
Notifications
You must be signed in to change notification settings - Fork 74
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
termux-docker images should be properly labeled with their native arch #35
Comments
Yeah, we could tag them using |
So, any idea to run the
|
See #34 (comment). |
Hi @licy183 ,
I'm running it through the multiarch/qemu-user-static , and this is the error when running it with the command Any idea to run the ARM Termux container in a x86_64 box? |
That's due to the uncorrect platform label as this issue proposed. For example, |
Hi @licy183 , Ok. Thank you for the explanation. I allways use the And... the use of the correct image Regards. |
Hi @licy183 , If the PR is merged and the new images are public in the docker repository, then why not close this issue? |
No 2 is not yet solved |
https://hub.docker.com/r/termux/termux-docker/tags
Currently all the images are labelled as linux/amd64. I think they should be properly fixed so that supposedly they can run on their own native architectures without any Docker warnings.
Though I am not quite settled for:
arm to use linux/arm or linux/arm/v7 or linux/arm32v7?
aarch64 to use linux/aarch64 or linux/arm/v8 or linux/arm64 or linux/arm64v8?
i686 (which is also "latest") to change to linux/i386?
We also need to look into adding busybox that are native to their arches
The one at https://github.com/termux/termux-docker/blob/4c08c24bceb13cecd1d18153dfddbbf5fe70bb71/system/arm/bin/busybox is currently 32bit arm static binary (checksum cd04052b8b6885f75f50b2a280bfcbf849d8710c8e61d369c533acf307eda064) which based on https://github.com/hazy/google-distroless/blob/master/WORKSPACE#L423 is coming from https://busybox.net/downloads/binaries/1.31.0-defconfig-multiarch-musl/
The text was updated successfully, but these errors were encountered: