Update instructions for cross compilation #27
Conversation
doc/docker.md
Outdated
| You might need to install `qemu-user-static` depending on your distribution. | ||
|
|
||
| ```console | ||
| docker build -f Dockerfile --progress=plain --build-arg --platform=linux/arm64 -t can-provider:latest . |
There was a problem hiding this comment.
I am not so sure if, this does "all the necessary things", I would prefer we just give the buildx version
There was a problem hiding this comment.
As I understand from https://www.docker.com/blog/faster-multi-platform-builds-dockerfile-cross-compilation-guide/ and docker/cli#3314 the two commands are today equal.
So then it becomes more a question of which syntax we want to promote :-)
I tested building with both, I however only verified that it reported the right architecture.
There was a problem hiding this comment.
If it is equivalent, what is --build-arg doing? In any case "buildx" might then be more compatible, and also working on docker(-cli?) < 23.00.
There was a problem hiding this comment.
That is a typo (good find), but I can remove the first alternative
Also removing unused file
Also removing unused file
Fixes #26