-
Notifications
You must be signed in to change notification settings - Fork 91
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
Crater shows many crates as "regressed" but there is no error #733
Comments
3 out of the four listed examples appear to have an error in crater/rustwide itself
|
Some more examples:
These have
which is very strange -- it seems like rustc says "could not compile" but there has been no error emitted. That's the same as what the first of the above has. |
These errors all seem to be spurious, for the errors mentioned in the OP none of them reproduced in a rerun. |
Interestingly, "could not compile" messaging actually comes from Cargo, not rustc (https://github.com/rust-lang/cargo/blob/3d2cf56d40041b4974bae74e7ad6b293f4f5daad/src/cargo/core/compiler/mod.rs#L426). That same code has a debug assert trying to cover this case. My suspicion is that we're seeing something in exec fail in an unexpected way -- but I'm not sure what it could be. It seems like this is probably a Cargo bug though. |
Log details of failure if no errors were seen ### What does this PR try to resolve? Crater has started (some time ago, though exact point is not known) to see failures in crates without logging an error. We're suspecting that there's something wrong in Cargo or rustc, but it's also possible that Crater itself is at fault. In any case, this should provide more details. cc rust-lang/crater#733 ### How should we test and review this PR? My hope is that this is reasonable to land without additional tests -- as-is, we expect it to never happen (there's even a debug assert later in the same code path) but it seems it *is* happening sometimes in the wild.
Most of the recent crater runs I looked at have a ton of crates that "regressed", but there's no build failure to be seen in the logfile. For instance, in https://crater-reports.s3.amazonaws.com/pr-129199/index.html, all the items I looked at in "build failed (unknown)" fall in that category. Concrete examples:
Also see here for some more examples of this.
The text was updated successfully, but these errors were encountered: