Closed
Description
Discovered in #56693.
Currently, if a crate re-exports an item from another crate with resolution failures, then those resolution failures get reported for the original crate. Since the downstream crate has no way to fix the resolution failure in the upstream crate, it shouldn't be reported, much like capping lints.
As of this writing, std::io::Error
has an intra-link resolution failure in one of its impls. So, the following code triggers the issue.
use std::io::Error;
A fix for this issue should take care to still report failures for re-exports within the same crate.
cc #43466
Metadata
Metadata
Assignees
Labels
Type
Projects
Milestone
Relationships
Development
No branches or pull requests
Activity
Rollup merge of rust-lang#56941 - euclio:deny-libstd-resolution-failu…
Rollup merge of rust-lang#56941 - euclio:deny-libstd-resolution-failu…
Rollup merge of rust-lang#56941 - euclio:deny-libstd-resolution-failu…
Rollup merge of rust-lang#60511 - taiki-e:libstd-intra-doc, r=Dylan-DPC
Assaf1978-boop commentedon Feb 16, 2020
jyn514 commentedon Jun 12, 2020
Interesting, I was about to implement the opposite in #73101. I assume you'd still want the warning for additional documentation on the re-export?
euclio commentedon Jun 23, 2020
Yes, that seems reasonable to me. The goal here is to avoid reporting warnings to downstream crates that have no way to fix them.
13 remaining items