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

Dont rm -rf all the repos #27

Closed
wants to merge 4 commits into from
Closed

Conversation

giuseppe
Copy link

No description provided.

a different rev can have different submodules

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe changed the title Dont rm rf Dont rm -rf all the repos Feb 28, 2017
@giuseppe
Copy link
Author

@LK4D4 this should fix the issue with refetching the entire repositories each time.

There is a new flag -clean-vcs to keep the previous behavior

@giuseppe giuseppe force-pushed the dont-rm-rf branch 4 times, most recently from f28e861 to abe5766 Compare February 28, 2017 13:41
@LK4D4
Copy link
Owner

LK4D4 commented Feb 28, 2017

@giuseppe @runcom I really wonder if you should try https://github.com/rancher/trash instead. It keeps vcs cache and stuff and it's better fit for your vendoring model. It can read same config format, so you just need to replace vndr binary with trash.
I'm okay with merging this, but just maybe we can keep vndr simpler and you happier while we wait for official vendoring tool. Let me know your thoughts.
Thanks!

@runcom
Copy link

runcom commented Feb 28, 2017

@LK4D4 yeah we can try that tool while waiting golang's without messing this repository :) thanks!

@runcom
Copy link

runcom commented Feb 28, 2017

@giuseppe we can use glide and glide-vc to keep vendoring in c/image (I think)

@giuseppe
Copy link
Author

thanks for your reply. I've added a patch to replace vndr with trash.

@LK4D4 Still you might be interested in this patch series though, it makes vndr as fast as trash when the vendor repos are already available.

giuseppe added 2 commits March 2, 2017 11:18
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
If the VCS doesn't support it (at the moment only git has the
implementation), fallback to the previous rm -rf && clone.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe force-pushed the dont-rm-rf branch 2 times, most recently from 6c7f096 to 2835fde Compare March 5, 2017 11:34
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe
Copy link
Author

giuseppe commented Mar 5, 2017

@LK4D4 in case it might be useful to you, I've modified the patch to not change the current behaviour but an user needs to specify -keep-vcs-files to keep the vcs files.

@LK4D4
Copy link
Owner

LK4D4 commented Mar 7, 2017

@giuseppe I'm reluctant to merge it because trash much better handles that use-case and I don't want to create illusion that vndr can be replacement in that case.
Thanks for your efforts, anyway! I appreciate it.

@LK4D4 LK4D4 closed this Mar 7, 2017
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.

3 participants