-
Notifications
You must be signed in to change notification settings - Fork 19
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
get_deref broken on Mac #57
Closed
Comments
Stebalien
added a commit
to Stebalien/rustix
that referenced
this issue
Apr 15, 2024
fixes the remainder of bytecodealliance#957 fixes Stebalien/xattr#57
@tcmulcahy upstream fix is here: bytecodealliance/rustix#1045. Could you make a PR (here) with a failing regression test? |
Merged
sunfishcode
pushed a commit
to bytecodealliance/rustix
that referenced
this issue
Apr 15, 2024
…1045) fixes the remainder of #957 fixes Stebalien/xattr#57
Merged
Stebalien
added a commit
that referenced
this issue
May 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@notgull the fix for get - bytecodealliance/rustix@10361e7 - also needs to be applied to get_deref.
The easiest way to repro is to change the
xattr::get
toxattr::get_deref
in https://github.com/Stebalien/xattr/blob/master/tests/main.rs#L60Test output:
The text was updated successfully, but these errors were encountered: