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

dep ensure --update to pick up a new gospinner and other things #173

Merged
merged 2 commits into from
Jun 22, 2018

Conversation

dnmurray
Copy link
Contributor

gospinner has a new release tag, to address #160. Did a dep ensure --update and picked up that and a few other changes. This should be part of the next RC so we can get some testing done.

@tekante tekante requested review from febbraro and tekante June 21, 2018 17:03
@febbraro
Copy link
Member

I see a lock file was updated, but what about the Gopkg.toml file?

@dnmurray
Copy link
Contributor Author

dep ensure --update didn't make any changes, and I didn't make any changes. I didn't delve real deep in how semantic version matches are happening, but it did pick up the update to gospinner and the other packages, without me changing the .toml file. On current master:

rig$ dep ensure -update
Warning: the following project(s) have [[constraint]] stanzas in Gopkg.toml:

  ✗  github.com/mattn/go-isatty

However, these projects are not direct dependencies of the current project:
they are not imported in any .go files, nor are they in the 'required' list in
Gopkg.toml. Dep only applies [[constraint]] rules to direct dependencies, so
these rules will have no effect.

Either import/require packages from these projects so that they become direct
dependencies, or convert each [[constraint]] to an [[override]] to enforce rules
on these projects, if they happen to be transitive dependencies,

rig$ git status
On branch master
Your branch is up to date with 'origin/master'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

        modified:   Gopkg.lock

no changes added to commit (use "git add" and/or "git commit -a")

@febbraro
Copy link
Member

What about if you delete the vendor directory and do a dep ensure does it capture the correct version? I'm guessing a clean install may not, so we would need to update the .toml file too

@dnmurray
Copy link
Contributor Author

Verified that we needed to update the .toml file (I created a fresh install to test, and dep ensure just brought down the pinned packages). yaml.v2 got updated in the original dep ensure but since the .toml file specifies a branch and not a version, I didn't change that.

@febbraro febbraro merged commit 20cf623 into phase2:develop Jun 22, 2018
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.

2 participants