Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

forbidding 2018 idioms Incompatible with pin_project_lite 0.1.9 #244

Closed
rbtcollins opened this issue Sep 29, 2020 · 3 comments · Fixed by #245
Closed

forbidding 2018 idioms Incompatible with pin_project_lite 0.1.9 #244

rbtcollins opened this issue Sep 29, 2020 · 3 comments · Fixed by #245

Comments

@rbtcollins
Copy link

Updating pin-project-lite v0.1.8 -> v0.1.9

lead to:

error[E0453]: allow(explicit_outlives_requirements) overruled by outer forbid(rust_2018_idioms)
  --> /home/robertc/.cargo/registry/src/github.heygears.com-1ecc6299db9ec823/surf-2.0.0-alpha.6/src/response.rs:17:1
   |
17 | / pin_project_lite::pin_project! {
18 | |     /// An HTTP response, returned by `Request`.
19 | |     pub struct Response {
20 | |         #[pin]
21 | |         res: http_client::Response,
22 | |     }
23 | | }
   | |_^ overruled by previous forbid
   | 
  ::: /home/robertc/.cargo/registry/src/github.heygears.com-1ecc6299db9ec823/surf-2.0.0-alpha.6/src/lib.rs:73:11
   |
73 |   #![forbid(rust_2018_idioms)]
   |             ---------------- `forbid` level set here
   |
   = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

error: aborting due to previous error

For more information about this error, try `rustc --explain E0453`.
error: could not compile `surf`.

To learn more, run the command again with --verbose.

pinning pin-project-lite back to 0.1.8 allows everything to build again.

@Licenser
Copy link
Contributor

Could we get an alpha.7 with this merged?

@jbr
Copy link
Member

jbr commented Sep 29, 2020

@Licenser 👍 alpha.7 released

@Licenser
Copy link
Contributor

<3 thank you @jbr !

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

Successfully merging a pull request may close this issue.

3 participants