Skip to content
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

Avoid creation of irrelevant temporary files on Windows #1462

Merged
merged 1 commit into from
Apr 27, 2020

Conversation

simonferquel
Copy link
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

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