rustdoc and reexported types

[stanislav-tkach]

Example (lib.rs):

extern crate serde;

pub mod reexport {
    pub use serde::Serialize;
    pub use serde::ser::Error as SerError;
}

In the generated documentation for the reexport::SerError type there is a reference to the serde::Serialize, but this reference is broken (target/doc/reexport_test/trait.Serialize.html instead of target/doc/reexport_test/reexport/trait.Serialize.html).

[Freyskeyd]

Hey @DarkEld3r

thank's for the report, can you provide a rust version please?

[stanislav-tkach]

I have used rustc 1.20.0 (f3d6973f4 2017-08-27), but on the latest stable (rustc 1.21.0 (3b72af97e 2017-10-09)) there is the same problem.

[Freyskeyd]

Yeah, I see. But I think it's a serde error.

https://docs.serde.rs/src/serde/ser/mod.rs.html#107

Anyway, I don't know if it's a doc generation error.

[stanislav-tkach]

I'm also not sure if it is rustdoc bug: reexporting is specific case, but I prefer to create an issue to make sure if it is a problem or not. 😅

On the other hand, documentation works correctly for serde (https://docs.serde.rs/serde/trait.Serialize.html) itself, so it cannot (shouldn't) be fixed inside this crate.

[ollie27]

#43466 should fix issues like this.

[TimNN]

This is currently expected behaviour since rustdoc doesn't do any kind of link rewriting currently. As @ollie27 noted, there are plans to support this in rustdoc so I'm gonna close this issue.