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

(UPDATE to 1.48 when released) Rust 1.46 breaks the build, while 1.45 works just fine #330

Closed
elpiel opened this issue Sep 2, 2020 · 5 comments
Labels
bug Something isn't working Low
Milestone

Comments

@elpiel
Copy link
Member

elpiel commented Sep 2, 2020

Build Error on Sentry, Rust 1.46:

elpiel@kracho:~/PROJECTS/AdEx/rust-adex-validator-stack$ cargo build -p sentry
   Compiling primitives v0.1.0 (/home/elpiel/PROJECTS/AdEx/rust-adex-validator-stack/primitives)
   Compiling adapter v0.1.0 (/home/elpiel/PROJECTS/AdEx/rust-adex-validator-stack/adapter)
   Compiling sentry v0.1.0 (/home/elpiel/PROJECTS/AdEx/rust-adex-validator-stack/sentry)
error: reached the type-length limit while instantiating `std::future::from_generator::<[s...pter::DummyAdapter>]>>, ()}]>}]>`
  --> /home/elpiel/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libcore/future/mod.rs:59:1
   |
59 | / pub const fn from_generator<T>(gen: T) -> impl Future<Output = T::Return>
60 | | where
61 | |     T: Generator<ResumeTy, Yield = ()>,
62 | | {
...  |
85 | |     GenFuture(gen)
86 | | }
   | |_^
   |
   = note: consider adding a `#![type_length_limit="1269353"]` attribute to your crate

error: aborting due to previous error

error: could not compile `sentry`.

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

✔️ Revert it Rust 1.45

For now the solution is to revert Rust version to 1.45.2 and upgrade later

TODO: Update to 1.48 when released and see this issue is fixed

Related merged PR: rust-lang/rust#78410

Related Rust bugs:

@elpiel elpiel added bug Something isn't working High High priority task labels Sep 2, 2020
elpiel added a commit that referenced this issue Sep 2, 2020
…n-to-1-45

Hotfix for issue #330 - Revert Rust to 1.45.2
@elpiel elpiel changed the title Rust 1.46 breaks the build, while 1.45 works just fine (UPDATE to 1.48 when released) Rust 1.46 breaks the build, while 1.45 works just fine Nov 16, 2020
@elpiel
Copy link
Member Author

elpiel commented Nov 16, 2020

rust-lang/rust#75992 was resolved by reverting some commits in rust-lang/rust#78410

For now we are waiting for the next Rust release, probably 1.48 that will be released this(?) week

@shirshak55
Copy link

@elpiel its released yesterday so should be fixed.

@elpiel
Copy link
Member Author

elpiel commented Nov 20, 2020

Thank you for letting me know! I'm already working on updating to it.

Edit: clippy is panicing on the sentry's chain function that handles middlewares, I've opened an issue in clippy that can be found here: rust-lang/rust-clippy#6350

@shirshak55
Copy link

@elpiel I don't think we should be blocked in clippy. Its just a linting tool.

@elpiel
Copy link
Member Author

elpiel commented Nov 23, 2020

I am looking for s way to simplify our middleware chain function and maybe clippy will be fixed by then.
This is not a high priority actually, since it was a compiler regression in the first place.

If clippy is fixed before the fix, I will also upgrade it

@elpiel elpiel added Low and removed High High priority task labels Nov 23, 2020
@elpiel elpiel closed this as completed Nov 25, 2020
@elpiel elpiel modified the milestones: validator-v0.2.0, sentry-v0.2.0 Jan 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Low
Projects
None yet
Development

No branches or pull requests

2 participants