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

Is 0.2.7 backwards compatible with 0.2.6? #87

Closed
aclysma opened this issue Nov 18, 2019 · 2 comments
Closed

Is 0.2.7 backwards compatible with 0.2.6? #87

aclysma opened this issue Nov 18, 2019 · 2 comments

Comments

@aclysma
Copy link

aclysma commented Nov 18, 2019

Hello, I had trouble compiling cocoa when using 0.2.7, but was able to use 0.2.6.

error[E0282]: type annotations needed
   --> /Users/username/.cargo/registry/src/github.com-1ecc6299db9ec823/cocoa-0.13.0/src/appkit.rs:491:9
    |
491 |         msg_send![self, releaseGlobally];
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |         |
    |         cannot infer type
    |         consider giving `result` a type
    |
    = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

My current workaround is to use objc = ">=0.2,<0.2.7" in my cargo.toml. I'm honestly not sure why it fails to build, but I didn't expect that 0.2.6 -> 0.2.7 would be a breaking change. I have an issue open in my project aclysma/skulpin#6 - just wanted to check that this is the best solution?

Thanks for all the work on this project!

@aclysma aclysma changed the title Is 0.2.7 backwards compatible with 0.2.6 Is 0.2.7 backwards compatible with 0.2.6? Nov 18, 2019
@SSheldon
Copy link
Owner

@aclysma sorry for the inconvenience! 0.2.7 has a soundness fix to prevent future undefined behavior. If it's failing to compile, that means your dependencies could exercise undefined behavior in a future version of rust. There's more info on context and reasoning in #62.

I'll see if I can give you any advice on how to fix this over at aclysma/skulpin#6.

@aclysma
Copy link
Author

aclysma commented Nov 19, 2019

Thanks so much for the help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants