-
Notifications
You must be signed in to change notification settings - Fork 203
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
several Bevy crates failed to build with yanked dependencies error but could not find which #2556
Comments
@TrialDragon compiled a list of affected crates on Discord:
|
just a rebuild for |
( I'll need to figure out why the error was incomplete in the database, the feature to actually see manifest / cargo errors is relatively recent ) |
@mockersf can you confirm that the docs are fine now? Should we rebuild more releases? |
seems to be good, thanks! |
@syphar looks like the exact same thing is happening with
|
For now I queued rebuilds for all of these. ( making the error visible is on my todo list, also adding more loggin) |
@syphar all the reported docs work now, thanks :) |
@syphar we published a new patch version of Bevy yesterday, and again some crates didn't build with the same error about a yanked dependency. https://docs.rs/crate/bevy_gltf/0.14.2/builds/1350490 for an example error Here is the list:
Anything we could do to help investigate? |
with some quick code reading, it seems the "yanked" flag comes from https://github.com/rust-lang/rustwide/blob/0ff427b7d1ed032734254e73c0dbedde785d4782/src/prepare.rs#L119-L120 Is there a cache somewhere of published crates that isn't up to date? As we publish crates that depends on each other, that could explain. And with the "recent" (and awesome!) speedup of docs.rs to build the queue that would explain why we're seeing this now |
So, we don't have a cache here. Generally our retry mechanism should help with these kind of errors, by just retrying at another later time. |
@mockersf do you have an example for a failed release because of this? I want to check some details around our retry mechanism. |
I think I found out what's happening |
this is the second regression coming from #2467 For normal build errors (= the build was executed, but failed) we never retried. This logic broke with the PR above, which is why we don't re-attempt builds right now. I'll work on a fix, and then next release it should be fine again. |
I just deployed the fix #2605 . When you do your next bulk release, everything should start working again as it did for a long time, feel free to reopen this issue / ping me if not, then I'll recheck. |
Crate name
bevy_state
Build failure link
https://docs.rs/crate/bevy_state/0.14.0/builds/1281410
Additional details
This is complaining about a yanked dependency, but we could not find which
Also happened to
Would it be possible to have more info about which dependency was yanked?
The text was updated successfully, but these errors were encountered: