-
Notifications
You must be signed in to change notification settings - Fork 511
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
Bump go version to 1.7.1, since 1.6.3 cannot build working binaries on OS X Sierra #984
Bump go version to 1.7.1, since 1.6.3 cannot build working binaries on OS X Sierra #984
Conversation
3f62c24
to
a22e76f
Compare
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.
thank you for fixing this, LGTM on green!
@@ -1 +1 @@ | |||
0.4.1 | |||
0.4.2 |
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 make a quick changelog note?
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.
Good point, thanks!
a22e76f
to
43850f7
Compare
@@ -1,5 +1,8 @@ | |||
# Changelog | |||
|
|||
## [v0.4.2](https://github.com/docker/notary/releases/tag/v0.4.0) 9/30/2016 | |||
+ Bump the cross compiler to golang 1.7.1, since [1.6.3 builds binaries that could have non-deterministic bugs in OS X Sierra](https://groups.google.com/forum/#!msg/golang-dev/Jho5sBHZgAg/cq6d97S1AwAJ) [#984](https://github.com/docker/notary/pull/984) |
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.
👍
@justincormack approved. ping @endophage for one more +1? |
…n Mac OS Sierra Signed-off-by: Ying Li <[email protected]>
43850f7
to
8e602b9
Compare
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.
LGTM
Signed-off-by: Ying Li <[email protected]>
:|
See https://groups.google.com/forum/#!msg/golang-dev/Jho5sBHZgAg/cq6d97S1AwAJ