Skip to content
This repository has been archived by the owner on May 9, 2019. It is now read-only.

Fix dependency management in the project #165

Closed
thegrumpylion opened this issue Dec 18, 2018 · 4 comments
Closed

Fix dependency management in the project #165

thegrumpylion opened this issue Dec 18, 2018 · 4 comments
Assignees
Milestone

Comments

@thegrumpylion
Copy link

Currently QMSTR uses dep for dependency management. We should either:

  • Use dep properly by commiting Gopkg.lock to the repo.
  • Migrate to go mod
@thegrumpylion thegrumpylion self-assigned this Jan 11, 2019
@thegrumpylion
Copy link
Author

After some back and forth with Markus, the choice is obvious. Go mod is the way to go (no pun intended) since is build into the whole go tooling, and many more.

@thegrumpylion
Copy link
Author

thegrumpylion commented Jan 13, 2019

The steps for this should be simple

go mod init
go mod tidy

But this fails with

...
go: github.com/Sirupsen/[email protected]: parsing go.mod: unexpected module path "github.com/sirupsen/logrus"
go: error loading module requirements

The reason is that it pulls an old version of docker that depends on a a path with upercase letter. After digging around it seems this will not be an issue with go 1.12

Even if you try manually pulling docker by the means of

go mod init
go get github.com/docker/[email protected]
go mod tidy

it still fails with the same issue. The reason being that v17.05.0-ce (which is the latest?) is relatively old (2017-05-04)

A solution to this is pulling a version i found in this golang/go#26208 which is from 2018-04-22

go mod init
go get github.com/docker/[email protected]
go mod tidy

So this got me thinking that versions from that date an on should also work. So i tried with pulling docker master and it worked as expected.

go mod init
go get github.com/docker/docker@master
go mod tidy

So the question becomes on which commit of the docker repository do we want to depend on? We can try building with current master and if everything works we can stick with that until we find a better version to update to.

@hemarkus any input?

@thegrumpylion
Copy link
Author

Let's wait for golang 1.12 to be out and look into this again

@mirkoboehm mirkoboehm added this to the Sprint 4.4 milestone Apr 4, 2019
@hemarkus
Copy link

hemarkus commented Apr 4, 2019

Find task for migration here QMSTR/qmstr#262

@hemarkus hemarkus closed this as completed Apr 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants