-
Notifications
You must be signed in to change notification settings - Fork 38
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
Coreaudio compilation failure - bindings generation fail due to old rustfmt bug #7
Comments
Hey thanks for the issue! This seems to be an issue with older versions of rustfmt, could you possibly try updating to the latest rustfmt that ships with rustup? I actually came across this too, and I can't see us being the only ones. Perhaps we should try catching this error and then re-generating the bindings without running rustfmt for users relying on older versions of the rust tools. |
Indeed I don't think it's a good idea to run rustfmt on generated bindings which nobody is going to read anyway! |
Does cargo automatically run rustfmt on dependencies? |
A simple workaround if someone has no opportunity to update the rustfmt
|
Bindgen by default automatically run rust fmt on the generated bindings. This pr disables that automatic fmt |
Fixed by #13 . |
Hey, I'm using coreaudio-sys as a dependency for CPAL, and it seems that the pinned v 0.2.0 is failing to compile with the following error:
I can't find other sources of people having this issue, so I'm not sure if it is on my end or not.
Thanks!
The text was updated successfully, but these errors were encountered: