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

Fixes compile error #144

Closed
wants to merge 1 commit into from
Closed

Fixes compile error #144

wants to merge 1 commit into from

Conversation

dmfallak
Copy link

@dmfallak dmfallak commented Sep 4, 2024

Problem

Build fails with the following:

Compiling casper-server v0.1.0 (/nail/home/dfallak/.cargo/git/checkouts/casper-b952159e8c9cd1ac/dd54a31/casper-server)
error[E0282]: type annotations needed for `Box<_>`
  --> casper-server/src/lua/uri.rs:32:17
   |
32 |             let mut query_pairs = query
   |                 ^^^^^^^^^^^^^^^
...
38 |             query_pairs.sort_by_key(|x| x[0]);
   |                         ----------- type must be known at this point
   |
help: consider giving `query_pairs` an explicit type, where the placeholders `_` are specified
   |
32 |             let mut query_pairs: Box<_> = query
   |                                ++++++++

For more information about this error, try `rustc --explain E0282`.
error: could not compile `casper-server` (bin "casper-server") due to 1 previous error

Solution

Provide explicit types for the untyped variables.

Verification

Run cargo build in casper-server directory.

@khvzak
Copy link
Member

khvzak commented Sep 5, 2024

@dmfallak the most actual branch is dev , did not merge yet to main, will do today.

@khvzak khvzak changed the base branch from main to dev September 5, 2024 09:07
Base automatically changed from dev to main September 6, 2024 10:25
@khvzak khvzak closed this Sep 26, 2024
@khvzak khvzak deleted the u/dfallak/fix-build-error branch September 26, 2024 11:59
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

Successfully merging this pull request may close these issues.

2 participants