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

Solarish: add the arc4random* API family #3944

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

devnexen
Copy link
Contributor

@devnexen devnexen commented Sep 26, 2024

@rustbot
Copy link
Collaborator

rustbot commented Sep 26, 2024

r? @tgross35

rustbot has assigned @tgross35.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot
Copy link
Collaborator

rustbot commented Sep 26, 2024

Some changes occurred in solarish module

cc @jclulow, @pfmooney

@devnexen
Copy link
Contributor Author

@rustbot label stable-nominated

@rustbot rustbot added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Sep 26, 2024
@RalfJung
Copy link
Member

RalfJung commented Sep 28, 2024

@tgross35
Copy link
Contributor

Thanks for linking the docs. LGTM but I'll wait for one of the solarish people to double check this, or merge it in another day or two if we don't hear back.

@RalfJung
Copy link
Member

RalfJung commented Sep 28, 2024

@devnexen maybe you can add the docs link to the PR description, so they become part of the commit message.

(As I already told you in multiple past PRs, please write descriptions for your PRs! It makes them much easier to review, and therefore much more likely to be landed. It is not a great practice to throw a piece of code over the fence without saying a single sentence about the what and why and how.)

@devnexen devnexen force-pushed the illumos_arc4random_buf branch from 70a5bb8 to 84cb528 Compare September 28, 2024 15:11
@tgross35
Copy link
Contributor

@jclulow, @pfmooney would one of you be able to ack this?

@tgross35 tgross35 changed the title adding arc4random* api family for solarish. Solarish: add the arc4random* api family Nov 6, 2024
@tgross35 tgross35 changed the title Solarish: add the arc4random* api family Solarish: add the arc4random* API family Nov 6, 2024
@tgross35
Copy link
Contributor

This looks fine to me so I will merge it, but wait for a solarish set of eyes before backport (@jclulow)

@tgross35 tgross35 enabled auto-merge November 14, 2024 00:36
@tgross35 tgross35 removed the O-wasm label Nov 14, 2024
@tgross35 tgross35 added this pull request to the merge queue Nov 14, 2024
Merged via the queue into rust-lang:main with commit ba828b1 Nov 14, 2024
43 checks passed
tgross35 pushed a commit to tgross35/rust-libc that referenced this pull request Nov 16, 2024
@tgross35 tgross35 mentioned this pull request Nov 16, 2024
tgross35 pushed a commit to tgross35/rust-libc that referenced this pull request Nov 16, 2024
tgross35 pushed a commit to tgross35/rust-libc that referenced this pull request Nov 16, 2024
@tgross35 tgross35 removed the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Nov 16, 2024
@tgross35 tgross35 added the stable-applied This PR has been cherry-picked to libc's stable release branch label Nov 16, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
O-solarish O-unix S-waiting-on-review stable-applied This PR has been cherry-picked to libc's stable release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants