Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Workaround firecracker-go-sdk go.mod issue using new commit #582

Merged

Conversation

stealthybox
Copy link
Contributor

Using v0.21.0 with go mod tidy using go versions 1.14.2, 1.12.17, and 1.12
produce a vendored dependency from firecracker that contains CHANGELOG.md
and version.go from a commit with v0.20.0 contents.

We have not yet determined which commit the incorrect source is coming from.
This should be opened up as an issue with github.com/golang/go with a
dedicated reproduction.

Problematic:

go mod edit -require github.com/firecracker-microvm/firecracker-go-sdk@v0.21.0
go mod tidy

Commands run to produce this workaround:

go mod edit -require github.com/firecracker-microvm/firecracker-go-sdk@e6eaff8
make tidy-in-docker
  # go mod tidy
  # go mod vendor
  # ... etc

cc @darkowlzz

Using `v0.21.0` with `go mod tidy` using go versions 1.14.2, 1.12.17, and 1.12
produce a vendored dependency from firecracker that contains `CHANGELOG.md`
and `version.go` from a commit with `v0.20.0` contents.

We have not yet determined which commit the incorrect source is coming from.
This should be opened up as an issue with github.com/golang/go with a
dedicated reproduction.

Problematic:
```shell
go mod edit -require github.com/firecracker-microvm/firecracker-go-sdk@v0.21.0
go mod tidy
```

Commands run to produce this workaround:
```shell
go mod edit -require github.com/firecracker-microvm/firecracker-go-sdk@e6eaff8
make tidy-in-docker
  # go mod tidy
  # go mod vendor
  # ... etc
```

cc @darkowlzz
@stealthybox stealthybox merged commit f5ba013 into weaveworks:master Apr 13, 2020
@stealthybox stealthybox deleted the workaround-firecracker-gomod branch April 13, 2020 21:31
@luxas luxas added this to the v0.7.0 milestone Jun 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants