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

ABI fixes for Rust #981

Closed
wants to merge 4 commits into from
Closed

ABI fixes for Rust #981

wants to merge 4 commits into from

Conversation

no92
Copy link
Member

@no92 no92 commented Feb 7, 2024

No description provided.

no92 added 4 commits February 7, 2024 22:27
The `request` argument in `ioctl` is defined to be `unsigned long`,
which leads to warnings when generating rust bindings as we can't figure
out the intended type from the lexed macro tokens. To avoid this
confusion, this commit manually specifies the type to be extra clear.
@no92 no92 added bug This issue reports a bug affects ABI labels Feb 7, 2024
@no92 no92 requested review from 64 and Dennisbonke February 7, 2024 21:28
@qookei
Copy link
Member

qookei commented Feb 7, 2024

RTLD constands are not ABI we care about when it comes to being compatible (musl vs glibc is also incompatible in this regard)

@no92 no92 marked this pull request as draft February 8, 2024 21:22
@no92
Copy link
Member Author

no92 commented Feb 15, 2024

Closing in favor of #989.

@no92 no92 closed this Feb 15, 2024
@no92 no92 deleted the rust-abi-fixes-2 branch February 15, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects ABI bug This issue reports a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants