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

look at output of failure tests #55

Closed
ahl opened this issue Sep 3, 2020 · 1 comment
Closed

look at output of failure tests #55

ahl opened this issue Sep 3, 2020 · 1 comment

Comments

@ahl
Copy link
Collaborator

ahl commented Sep 3, 2020

In particular, this output from bad_endpoint6.stderr seems to have regressed:

error[E0063]: missing fields `x`, `y` in initializer of `Ret`
  --> $DIR/bad_endpoint6.rs:17:1
   |
17 | / #[endpoint {
18 | |     method = GET,
19 | |     path = "/test",
20 | | }]
   | |__^ missing `x`, `y`

error[E0277]: the trait bound `Ret: serde::ser::Serialize` is not satisfied
  --> $DIR/bad_endpoint6.rs:17:1
   |
17 | / #[endpoint {
18 | |     method = GET,
19 | |     path = "/test",
20 | | }]
   | |__^ the trait `serde::ser::Serialize` is not implemented for `Ret`
   |
   = note: required by `dropshot::HttpResponseOk`

error[E0277]: the trait bound `fn(std::sync::Arc<dropshot::RequestContext>) -> impl std::future::Future {<impl std::convert::From<bad_endpoint> for dropshot::ApiEndpoint>::from::bad_endpoint}: dropshot::handler::    HttpHandlerFunc<_, _>` is not satisfied
  --> $DIR/bad_endpoint6.rs:17:1
   |
17 | / #[endpoint {
18 | |     method = GET,
19 | |     path = "/test",
20 | | }]
   | |__^ the trait `dropshot::handler::HttpHandlerFunc<_, _>` is not implemented for `fn(std::sync::Arc<dropshot::RequestContext>) -> impl std::future::Future {<impl std::convert::From<bad_endpoint> for                dropshot::ApiEndpoint>::from::bad_endpoint}`

That should be referencing the offending code, not the macro.

@ahl
Copy link
Collaborator Author

ahl commented Sep 8, 2020

Closing this as it seems to be a problem in rust 1.46.0:
rust-lang/rust#76360

Things should improve in a subsequent rust release.

@ahl ahl closed this as completed Sep 8, 2020
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

1 participant