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

Use sa_sigaction instead of sa_union.__su_sigaction for AIX #2614

Merged
merged 1 commit into from
Mar 24, 2025

Conversation

xingxue-ibm
Copy link
Contributor

What does this PR do

Change to use sa_sigaction instead of sa_union.__su_sigaction, now that the libc crate implementation for AIX defines sa_sigaction as a direct member of struct sigaction, aligning it with implementations on other similar platforms. (rust-lang/libc#4250).

Checklist:

  • I have read CONTRIBUTING.md
  • I have written necessary tests and rustdoc comments
  • A change log has been added if this PR modifies nix's API

Copy link
Member

@SteveLauC SteveLauC left a comment

Choose a reason for hiding this comment

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

Thanks!

@SteveLauC
Copy link
Member

I took the liberty to rebase your branch to pull #2618 in:)

@SteveLauC SteveLauC enabled auto-merge March 24, 2025 02:20
@SteveLauC SteveLauC added this pull request to the merge queue Mar 24, 2025
Merged via the queue into nix-rust:master with commit cc6b161 Mar 24, 2025
40 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants