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.

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

Remove AIX specific definition of sigval #3189

Merged
merged 1 commit into from Apr 10, 2023

Conversation

ecnelises
Copy link
Contributor

AIX definition of sigval is actually the same as other unix. Remove the union definition as other platforms treat it like a pointer after #3185.

AIX definition of sigval is actually the same as other unix. Remove the
union definition as other platforms treat it like a pointer.
@rustbot
Copy link
Collaborator

rustbot commented Apr 6, 2023

r? @JohnTitor

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

@JohnTitor
Copy link
Member

Thanks! @bors r+

@bors
Copy link
Contributor

bors commented Apr 10, 2023

📌 Commit 7a1d2c6 has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Apr 10, 2023

⌛ Testing commit 7a1d2c6 with merge 514adf0...

bors added a commit that referenced this pull request Apr 10, 2023
Remove AIX specific definition of sigval

AIX definition of sigval is actually the same as other unix. Remove the union definition as other platforms treat it like a pointer after #3185.
@bors
Copy link
Contributor

bors commented Apr 10, 2023

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Apr 10, 2023

⌛ Testing commit 7a1d2c6 with merge c1a63d5...

@bors
Copy link
Contributor

bors commented Apr 10, 2023

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

1 similar comment
@bors
Copy link
Contributor

bors commented Apr 10, 2023

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

@bors
Copy link
Contributor

bors commented Apr 10, 2023

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

@bors bors merged commit c1a63d5 into rust-lang:master Apr 10, 2023
9 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants