rust-lang / crater

Run experiments across parts of the Rust ecosystem!
https://crater.rust-lang.org
642 stars 90 forks source link

Crater shows many crates as "regressed" but there is no error #733

Open RalfJung opened 3 months ago

RalfJung commented 3 months ago

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.

Skgland commented 3 months ago

3 out of the four listed examples appear to have an error in crater/rustwide itself

Skgland commented 3 months ago

The fourth one might be sorted properly with 397f3f0 from #729

RalfJung commented 3 months ago

Some more examples:

These have

[INFO] [stderr] error: could not compile `advent_of_code` (bin "04" test); 1 warning emitted

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.

RalfJung commented 3 months ago

These errors all seem to be spurious, for the errors mentioned in the OP none of them reproduced in a rerun.

Mark-Simulacrum commented 3 months ago

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.