Skip to content

Conversation

@onur
Copy link
Contributor

@onur onur commented Oct 24, 2018

This is my solution for default-target. It is using default-target metadata option to build package in first place and placing default build to root of the crate documentation directory.

For example winapi will always be build with x86_64-pc-windows-msvc target first and documentation crated with this target will be available in docs.rs/winapi/$version/winapi/.

Copy link
Contributor

@QuietMisdreavus QuietMisdreavus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good! As you mentioned, by doing the initial build on the default-target, this will ensure that crates that only build on that target (or are practically only available on a certain target even if they build everywhere, like winapi) are checked properly before giving up on building their documentation. I have one comment, but since it matches what the old code does, you can merge this without changing it.

Are you planning on rebuilding crates that already have a default-target set? The reason i went with a redirect for my version was because winapi was already published with that set, so it would work without having to rebuild it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants