Skip to content

Dockerfile.buildx: bump to Go 1.13.10#40893

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:bump_buildx_go_1.13
May 6, 2020
Merged

Dockerfile.buildx: bump to Go 1.13.10#40893
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:bump_buildx_go_1.13

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Looks like this Dockerfile was not yet updated; possibly because go mod issues when building buildx 0.3.1.

Given that the upstream repository uses vendoring, I:

  • removed the go mod download step
  • added -mod=vendor to use vendoring mode

Looks like this Dockerfile was not yet updated; possibly because
`go mod` issues when building buildx 0.3.1.

Given that the upstream repository uses vendoring, I:

- removed the `go mod download` step
- added `-mod=vendor` to use vendoring mode

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Copy Markdown
Member Author

@AkihiroSuda @tonistiigi was just thinking; would it make sense to have (multi-arch?) images for buildx pushed to docker hub?

@thaJeztah
Copy link
Copy Markdown
Member Author

@cpuguy83 ptal

Copy link
Copy Markdown
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
Copy link
Copy Markdown
Member Author

bringing this one in

@thaJeztah thaJeztah merged commit 60a4f9a into moby:master May 6, 2020
@thaJeztah thaJeztah deleted the bump_buildx_go_1.13 branch May 6, 2020 19:48
@thaJeztah thaJeztah added this to the 20.03.0 milestone Jun 10, 2020
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.

3 participants