-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Missing documentation for apple and microsoft target on docs.rs #3289
Comments
I believe it's an issue of docs.rs? We surely added all the supported targets in #3279. cc @GuillaumeGomez |
Taking a look! |
docs.rs by default limits all crates to max 10 targets. i don't mind raising the limit as long as we decrease the max build time per target. |
Thanks for clarifying, I see! If it doesn't have much negative impact on other crates and docs.rs, that sounds great to me (to clarify, we still can host some targets, e.g. tier 2 or below, on our Pages, but I guess it's easier to use for these docs to be available on docs.rs). And I noticed #3279 doesn't contain any macOS targets, does it disable builds for macOS (i.e. |
I am not sure what order we build the targets in - which targets we select if there's too many over the limit is an implementation detail I've never tested. Let me change the limits and rebuild 0.2.147 and we can go from there :) |
Thanks @jyn514 ! |
most of the targets seem to be working. here are the failures i saw:
the build is still running so this list is not exhaustive. |
at this point i think i would suggest running docs.rs locally so you can confirm you've properly fixed all the errors; it will be too painful to publish a release every time to test and i'm nearly out of time for rust for the day anyway. as a heuristic |
i'm also a little worried about how long this takes to run ... i started the build 50 minutes ago and it shows no sign of stopping soon. maybe we can change the docs.rs configuration somehow so that build-std is only used for tier 3 targets? we'd have to add a new metadata field, i'd want to confirm first that actually makes it faster. |
I'm surprised it takes this long. We need to check if we link libraries and in such case, not do it if a feature is passed. To be investigated. |
@GuillaumeGomez the problem is that the standard library is built from source - if you could use |
This comment was marked as resolved.
This comment was marked as resolved.
I opened #3292 for that. Quite curious to see the result. |
Use `build-std=core` instead of `build-std` alone Should help with #3289.
The microsoft targets now show up, but the macos (darwin) ones are still missing. This is a bit problematic because it results in broken links in our crate's documentation (here, clicking on |
I added them in #3309. So next release they should appear. |
The latest version of libc only has documentation for the following targets on docs.rs:
This is missing any documentation at least for the windows and macos versions (which were there in 0.2.146 and below, see this, and it doesn't seem to be in the build queue anymore.
The text was updated successfully, but these errors were encountered: