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

Compilation failure of rustc-serialize breaks CI #551

Closed
raoulstrackx opened this issue Jan 5, 2024 · 1 comment · Fixed by #552
Closed

Compilation failure of rustc-serialize breaks CI #551

raoulstrackx opened this issue Jan 5, 2024 · 1 comment · Fixed by #552

Comments

@raoulstrackx
Copy link
Contributor

Compilation of em-app fails with:

em-app$ cargo +nightly-2024-01-05 build --target x86_64-fortanix-unknown-sgx
...
error[E0310]: the parameter type `T` may not live long enough
    --> /home/raoul/.cargo/git/checkouts/rustc-serialize-43e3299849baf4f9/9800ee0/src/serialize.rs:1155:5
     |
1155 |     fn decode<D: Decoder>(d: &mut D) -> Result<Cow<'static, T>, D::Error> {
     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
     |     |
     |     the parameter type `T` must be valid for the static lifetime...
     |     ...so that the type `T` will meet its required lifetime bounds...
     |
note: ...that is required by this bound
    --> /rustc/f688dd684faca5b31b156fac2c6e0ae81fc9bc90/library/alloc/src/borrow.rs:180:30
help: consider adding an explicit lifetime bound
     |
1151 | impl<'a, T: ?Sized + 'static> Decodable for Cow<'a, T>
     |                    +++++++++

For more information about this error, try `rustc --explain E0310`.
error: could not compile `rustc-serialize` (lib) due to 1 previous error

Earlier nightlies compiler do not result in this failure. A patch of rustc-serialize is required.

@raoulstrackx
Copy link
Contributor Author

A PR for rustc-serialized has been filed

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 a pull request may close this issue.

1 participant