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

Incoming nightly breakage #189

Open
oli-obk opened this issue Feb 22, 2022 · 5 comments
Open

Incoming nightly breakage #189

oli-obk opened this issue Feb 22, 2022 · 5 comments

Comments

@oli-obk
Copy link

oli-obk commented Feb 22, 2022

While your crate does nothing wrong per se, the incoming breakage may be somewhat annoying, and you can get ahead of the breakage.

The PR that will break this crate: rust-lang/rust#94081

The breakage: https://crater-reports.s3.amazonaws.com/pr-94081/try%23e251c42c969d6f8dbaae00e1028489c3a29f8789/reg/noria-common-0.6.0/log.txt

The fix is to move

pub fn rpc<Q: Serialize, R: 'static>(
and
type RpcFuture<A, R> = impl Future<Output = Result<R, failure::Error>>;
into their own module, so that the type alias doesn't try to get inferred at the same time as the impl Trait usage in the return type.

On a positive note: rust-lang/rust#65442 is now very much on my radar, with the PR that breaks your code this becomes actually fixable without a complex hack!

@jonhoo
Copy link
Contributor

jonhoo commented Feb 26, 2022

Thanks for the heads up! This repository is basically archived at this point, and the upstream crate hasn't been updated in a while, so I probably won't go through and fix this, but this should be helpful to anyone who does want to pick up the code. So I'll leave it open :)

@muety
Copy link

muety commented Mar 17, 2022

So the project is not under active development anymore?

@jonhoo
Copy link
Contributor

jonhoo commented Mar 17, 2022

No, I haven't worked on this since I graduated back in November 2020, and I don't think anyone's currently continuing the research in my lab at MIT either. This repository was, and still is, a research prototype :)

@ogbanugot
Copy link

Hi @jonhoo based on the breaking changes coming in can you help state clearly which version of nightly one needs to run this project smoothly. Thanks a lot.

@jonhoo
Copy link
Contributor

jonhoo commented May 7, 2022

Ah, that's a good question. I last worked on this end of November 2020, so a nightly from around there is likely to work. I don't have an exact date though.

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

4 participants