-
Notifications
You must be signed in to change notification settings - Fork 181
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
scripts/checks: adjust to ensure go 1.4.2 is in use #144
Conversation
Thinking more this change in itself might not actually work as the ++++++ I have a slight reservation with this stricter check that it might come as restricting for users running a slightly old or new version of go1.4 on host. We had similar check before and had to relax it as part of #30. Also I am not sure if current Thinking about alternate approaches. I experimented with modifying the
|
It works for the host build, which IIRC was the problem. The restriction lives to keep people from building bad copies. The systemtests do not work on mac -- they need the netlink library. |
sgtm we still need to add |
Yeah, probably a good idea. Let me amend this, one minute. |
this should be ready after tests pass. |
Anything left to do here? |
changes look good, it needs a rebase though. |
73db278
to
d9706d9
Compare
once CI passes, let's merge this |
Signed-off-by: Erik Hollensbe <[email protected]>
Signed-off-by: Erik Hollensbe <[email protected]>
scripts/checks: adjust to ensure go 1.4.2 is in use
Signed-off-by: Erik Hollensbe [email protected]