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

Warning: dep ensure removes unmanaged files from vendor #206

Closed
nathany opened this issue Jan 31, 2017 · 4 comments
Closed

Warning: dep ensure removes unmanaged files from vendor #206

nathany opened this issue Jan 31, 2017 · 4 comments

Comments

@nathany
Copy link
Contributor

nathany commented Jan 31, 2017

Running dep ensure removes files from vendor/ that is doesn't manage.

For example, I had a third-party shell script contained under vendor, and of course the manifest file from the vendor tool I was previously using. Both were deleted.

ensure

Experienced with dep version 76df5ce.

@peterbourgon
Copy link
Contributor

AFAIU, dep assumes complete control of vendor/, and may indeed blow things away if it feels like it. @sdboyer is this correct? If yes, my intuition is to document that better, somehow.

@sdboyer
Copy link
Member

sdboyer commented Feb 3, 2017

Yep, @peterbourgon is correct - we treat vendor/ as entirely volatile. It's very hard to do otherwise and keep the tool consistent - especially before we have #121 figured out.

@sdboyer
Copy link
Member

sdboyer commented Feb 3, 2017

It is definitely worth documenting, though - I'll leave this open for docs purposes?

@sdboyer sdboyer added the docs label Feb 3, 2017
@carolynvs carolynvs mentioned this issue Apr 19, 2017
krisnova pushed a commit to krisnova/dep that referenced this issue Apr 21, 2017
Add Release() to the SourceManager interface
@carolynvs
Copy link
Collaborator

Closing now that this has been documented in the FAQ.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants