-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Actix failing build #8734
Comments
Could this sudden failure be due to some "floating" version on a dependency? Ideally all dependencies should be pinned. For instance I could find this "nightly" reference, could it make the build fail at some point: |
There isn't much we could have done about this. Nightly failures happen and libraries that opt in to using those unstable features automatically subscribe to dealing with these kinds of failures at the detriment of transitive dependencies. I trust that they're on it and this will be resolved in a couple of days. |
Do you mean that the |
I haven't looked closely enough at other things other than the message you pasted:
|
I am definitely not blaming Actix, just wanted to understand why the benchmark started failing overnight as we are expected to pin all dependencies to prevent these issues. |
I guess you can only pin as far as the library itself does. Crates in the Rust ecosystem expect semver compatibility within a semver range. In this case, everything works fine for stable users. However, This issue is kinda surprising to me, because I would expect TFB to use a stable compiler. |
@sebastienros Whenever I see a failure for one of the frameworks I follow closely, I open a GitHub issue in that framework's repo directly. I have found that maintainers of frameworks are sometimes not aware for weeks when this kind of failure occurs. I recommend you try this in the actix repo's Issues page (if you have not yet done this). |
@mkvalor there's no need, I'm right here :P |
Oof, I originally suspected so; should've just looked at top contributors to Actix. But the pronoun in the third comment and the reference to the project in the fifth comment tipped my suspicion in a different direction. All good! |
Log:
/cc @ecruz-te @robjtede
The text was updated successfully, but these errors were encountered: