Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added getentropy to vita target #3230

Merged
merged 1 commit into from
May 1, 2023
Merged

Added getentropy to vita target #3230

merged 1 commit into from
May 1, 2023

Conversation

nikarh
Copy link
Contributor

@nikarh nikarh commented Apr 28, 2023

This is a late addition to #3209. This definition is required in order to implement random in std correctly.

As the previous PR, getentropy is a standard C function, should be implemented by newlib provider, and is not Sony's intellectual property.

@rustbot
Copy link
Collaborator

rustbot commented Apr 28, 2023

r? @JohnTitor

(rustbot has picked a reviewer for you, use r? to override)

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented May 1, 2023

📌 Commit 0b2ed16 has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented May 1, 2023

⌛ Testing commit 0b2ed16 with merge 66e988b...

@bors
Copy link
Contributor

bors commented May 1, 2023

☀️ Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 66e988b to master...

1 similar comment
@bors
Copy link
Contributor

bors commented May 1, 2023

☀️ Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 66e988b to master...

@bors bors merged commit 66e988b into rust-lang:master May 1, 2023
@bors
Copy link
Contributor

bors commented May 1, 2023

👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants