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

Please update the dependencies for the tiled example #451

Closed
willowell opened this issue Dec 3, 2019 · 1 comment · Fixed by #452
Closed

Please update the dependencies for the tiled example #451

willowell opened this issue Dec 3, 2019 · 1 comment · Fixed by #452
Labels

Comments

@willowell
Copy link

OS: macOS 10.15.1
rustc: 1.38.0

The tiled example fails to build through cargo because the cocoa v0.9.2 dependency fails to compile, producing this error:
error[E0282]: type annotations needed --> /Users/$(name)/.cargo/registry/src/github.com-1ecc6299db9ec823/cocoa-0.9.2/src/appkit.rs:479:9 | 479 | 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)

According to the pull requests for the cocoa crate, this issue was resolved very recently: servo/core-foundation-rs#342

I also tried just bumping up piston, piston_window, and tiled in the Cargo.toml file for the tiled example, but that led to compilation errors in main.rs instead of in the dependencies. For instance, rustc said that the closure on line 41: window.draw_2d(&e, |c, g| { requires three arguments, but it got only two.

@bvssvni bvssvni added the easy label Dec 4, 2019
@bvssvni
Copy link
Member

bvssvni commented Dec 4, 2019

|c, g, _| should work.

bvssvni added a commit to bvssvni/piston-examples that referenced this issue Dec 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants