Skip to content

Avoid creation of irrelevant temporary files on Windows#1462

Merged
tonistiigi merged 1 commit intomoby:masterfrom
simonferquel:dont-create-files-on-windows
Apr 27, 2020
Merged

Avoid creation of irrelevant temporary files on Windows#1462
tonistiigi merged 1 commit intomoby:masterfrom
simonferquel:dont-create-files-on-windows

Conversation

@simonferquel
Copy link
Copy Markdown
Contributor

binfmt_misc package creates temporary files containing alternate arch binaries to check if qemu binfmt support is enabled for them.

On Windows, it creates those file as well, even if there is no way those files can get executed on Windows as they are Linux binaries.

This PR avoids these unnecessary file creations on Windows, to avoid downstream projects such as dockerd to create those files on Windows.

Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
@tonistiigi tonistiigi merged commit e0b4ba5 into moby:master Apr 27, 2020
thaJeztah added a commit to thaJeztah/docker that referenced this pull request Apr 30, 2020
full diff: moby/buildkit@b26cff2...59e305a

- moby/buildkit#1469 Avoid creation of irrelevant temporary files on Windows
    - backport of moby/buildkit#1462 for the docker-19.03/v0.6 branch

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
docker-jenkins pushed a commit to docker-archive/docker-ce that referenced this pull request May 5, 2020
full diff: moby/buildkit@b26cff2...59e305a

- moby/buildkit#1469 Avoid creation of irrelevant temporary files on Windows
    - backport of moby/buildkit#1462 for the docker-19.03/v0.6 branch

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Upstream-commit: 63841af153595a18b66ba125d4b590e1e9e90a8e
Component: engine
@g3rzi
Copy link
Copy Markdown

g3rzi commented Jun 17, 2020

Was it pushed also to Docker Desktop?
If yes, what version, otherwise, when it is planned to be pushed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants