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
Module updates/addition of a vendor directory #82
Conversation
@@ -1,3 +1,2 @@ | |||
*.sw* | |||
crc-driver-libvirt-* | |||
vendor/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we also have a make target to sync the vendor and have a check similar to what we have in crc to make sure it never become out of sync?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added make vendorcheck
, I'm unconvinced about adding make vendor
, as it's just calling go mod vendor
Can we have this crc-org/machine#54 merged before this one? |
Ah sure, it's merged now, so I'll update the branch in a bit. |
This removes most of the indirect go.mod dependencies.
The only untouched ones are the libvirt ones as this could cause machine-driver-libvirt to inadvertantly start linking with newer libvirt version depending on what is installed on the machine it's built on.
CI is unhappy, |
This aligns machine-driver-libvirt with what crc is doing.
the |
This PR takes advantage of the recent machine changes to remove most machine-driver-libvirt
dependencies, and adds a vendor/ directory.