Skip to content
This repository was archived by the owner on Jun 6, 2022. It is now read-only.

Godep changes source code it vendors #31

Closed
twaugh opened this issue Sep 11, 2018 · 0 comments
Closed

Godep changes source code it vendors #31

twaugh opened this issue Sep 11, 2018 · 0 comments

Comments

@twaugh
Copy link
Collaborator

twaugh commented Sep 11, 2018

When godep vendors source code it removes "import comments" from package statements. We will need to accommodate this somehow.

Perhaps we need to

  • identify whether the top-level package using godep (look for Godeps/Godeps.json)
  • if so, for each upstream tag/commit in a vendored project, try first with 'git ls-tree ...' and then if that fails try building a FileHashes map from files in the working tree (that match the subset we are looking for) that we modify in the same way as godep
twaugh added a commit that referenced this issue Sep 14, 2018
twaugh added a commit that referenced this issue Sep 14, 2018
twaugh added a commit that referenced this issue Sep 14, 2018
twaugh added a commit that referenced this issue Sep 14, 2018
twaugh added a commit that referenced this issue Sep 14, 2018
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

1 participant