-
Notifications
You must be signed in to change notification settings - Fork 520
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
riyazdf
left a comment
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
| # 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
endophage
left a comment
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