-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Enable docs on dist-x86_64-musl #74871
Conversation
Add the rust-docs component to toolchain x86_64-unknown-linux-musl, which allows people using rustup on their musl-based linux distribution to download the rust-docs.
(rust_highfive has picked a reviewer for you, use r? to override) |
musl is at 1h 7m currently, at least based off of one build, so I think it should be fine to merge this since we're at an overall ceiling of around 2 hours. @bors r+ |
📌 Commit b5d143b has been approved by |
⌛ Testing commit b5d143b with merge 022af3a6dc0dc3137483b41e581dbdaeea49fa2a... |
💔 Test failed - checks-actions |
@Etherealist: 🔑 Insufficient privileges: not in try users |
@bors retry Cargo spurious test failure? cc @ehuss
|
Thanks for the ping! That error has been fixed yesterday via #74923 (rust-lang/cargo#8559). |
⌛ Testing commit b5d143b with merge a932dedafa960cf337e78bd4b01e07ada47a3a05... |
☀️ Test successful - checks-actions, checks-azure |
👀 Test was successful, but fast-forwarding failed: 422 Update is not a fast forward |
rust-lang/homu#75 |
…arth Rollup of 9 pull requests Successful merges: - rust-lang#74751 (Clean up E0730 explanation) - rust-lang#74782 (Don't use "weak count" around Weak::from_raw_ptr) - rust-lang#74835 (Clean up E0734 explanation) - rust-lang#74871 (Enable docs on dist-x86_64-musl) - rust-lang#74905 (Avoid bool-like naming) - rust-lang#74907 (Clean up E0740 explanation) - rust-lang#74915 (rustc: Ignore fs::canonicalize errors in metadata) - rust-lang#74934 (Improve diagnostics when constant pattern is too generic) - rust-lang#74951 (Cherry-pick the release notes for 1.45.1) Failed merges: r? @ghost
…imulacrum Enable docs on in the x86_64-unknown-linux-musl manifest Add the rust-docs component to toolchain x86_64-unknown-linux-musl, which allows people using rustup on their musl-based linux distribution to download the rust-docs. Generating and uploading the docs was enabled in b5d143b (rust-lang#74871). In rust-lang#75102 @Mark-Simulacrum found that we are uploading the docs, but the correct manifest is missing. * The relevant call to build-manifest seems to be [in bootstrap](https://github.com/rust-lang/rust/blob/c058a8b8dc5dea0ed9b33e14da9e317e2749fcd7/src/bootstrap/dist.rs#L2334) * The manifest is then used in [promote-release crontab](https://github.com/rust-lang/rust-central-station/blob/master/crontab)
Add the
rust-docs
component to toolchainx86_64-unknown-linux-musl
, which allows people using rustup on their musl-based linux distribution to download the rust-docs.--disable-docs
is based on the assumption thatx86_64-unknown-linux-musl
is only a cross-compile target.I have tested that the docs are built. I assume the build-system will automatically detect the docs and create a
rust-docs
component. I will monitor the components and create a follow-up PR, if the docs aren't published.See also #70619, where we enabled
rust-lld
to enable the wasm-workflow on musl-based linux distributions.