Skip to content

Conversation

@trzecieu
Copy link
Collaborator

@trzecieu trzecieu commented Jun 4, 2020

Without --network host It's impossible to build images (might be some recent addition for docker engine, as I don't recall it was a case before)

Copy link
Collaborator

@sbc100 sbc100 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rubberstamp lgtm

@kripken
Copy link
Member

kripken commented Jun 4, 2020

The docker docs say this flag only works on linux. Is that the only place this is used?

@sbc100
Copy link
Collaborator

sbc100 commented Jun 4, 2020

We shouldn't be running docker commands anywhere except on linux.

@kripken
Copy link
Member

kripken commented Jun 4, 2020

sgtm then.

@kripken kripken merged commit ede26f1 into emscripten-core:master Jun 4, 2020
@trzecieu
Copy link
Collaborator Author

trzecieu commented Jun 5, 2020

@kripken, it's supported on windows as well in my version:

PS C:\Users\piotr> docker build --help | grep net
      --network string          Set the networking mode for the RUN
PS C:\Users\piotr> docker --version
Docker version 19.03.8, build afacb8b

Thx for merging !

vargaz pushed a commit to vargaz/emsdk that referenced this pull request Nov 22, 2023
emscripten-core#514)

* Update dependencies from https://github.com/dotnet/source-build-reference-packages build 20230922.1

Microsoft.SourceBuild.Intermediate.source-build-reference-packages
 From Version 9.0.0-alpha.1.23468.3 -> To Version 9.0.0-alpha.1.23472.1

* Update dependencies from https://github.com/dotnet/source-build-reference-packages build 20230922.1

Microsoft.SourceBuild.Intermediate.source-build-reference-packages
 From Version 9.0.0-alpha.1.23468.3 -> To Version 9.0.0-alpha.1.23472.1

* Update dependencies from https://github.com/dotnet/source-build-reference-packages build 20230922.1

Microsoft.SourceBuild.Intermediate.source-build-reference-packages
 From Version 9.0.0-alpha.1.23468.3 -> To Version 9.0.0-alpha.1.23472.1

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants