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.

Sign up for GitHub

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

Jump to bottom

doc: release notes recommend wrong version of NaCl #12062

Closed
davecheney opened this issue Aug 6, 2015 · 4 comments
Closed

doc: release notes recommend wrong version of NaCl #12062

davecheney opened this issue Aug 6, 2015 · 4 comments
Labels
FrozenDueToAge
Milestone
Go1.5

Comments

@davecheney
Copy link
Contributor

On NaCl, Go 1.5 requires SDK version pepper-39 or above because it now uses the get_random_bytes system call.

I believe this is incorrect. Go does not use the new get_random_bytes syscall, see #11961. As best I can tell NaCl requires pepper < 43

/cc @adg @rsc
@adg
Copy link
Contributor

adg commented Aug 6, 2015

I actually don't know. Happy to amend the docs.

@davecheney
Copy link
Contributor Author

OK, I'll send a PR. The range of usable pepper versions is quite narrow.

On Fri, 7 Aug 2015 09:09 Andrew Gerrand notifications@github.com wrote:

I actually don't know. Happy to amend the docs.

—
Reply to this email directly or view it on GitHub
#12062 (comment).

@randall77
Copy link
Contributor

Yet more evidence that compiling to static binaries is awesome.

@gopherbot
Copy link

CL https://golang.org/cl/13341 mentions this issue.

@mikioh mikioh added this to the Go1.5 milestone Aug 10, 2015
@golang golang locked and limited conversation to collaborators Aug 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Go1.5
Development

No branches or pull requests
5 participants
@davecheney @mikioh @randall77 @adg @gopherbot