fix(bootstrap/dist)!: change dist path for rustc-docs to avoid clashing#150289
fix(bootstrap/dist)!: change dist path for rustc-docs to avoid clashing#150289bors merged 1 commit intorust-lang:mainfrom
rustc-docs to avoid clashing#150289Conversation
|
|
@Kobzol No, that's the rustdoc for nightly |
|
Sorry, posted the wrong link, I meant https://doc.rust-lang.org/nightly/nightly-rustc/rustc_ast/ast/index.html. |
|
Seems okay, but this needs to merge with rust-lang/promote-release#101 in relatively-tight lock-step so we minimize the time where rustc docs isn't available; the promote-release side needs infra-admin coordination. |
|
@jieyouxu The preparation PR rust-lang/promote-release#101 has just been accepted and merged. If I have understood it correctly, it should only take one nightly release to make sure no breakage has been introduced by it and then you can unfreeze this one, many thanks in advance 🙏
|
|
Ok cool, I'll wait until Dec 31 to be safe. |
|
@bors r+ rollup=never |
|
☀️ Test successful - checks-actions |
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 8d670b9 (parent) -> 4862272 (this PR) Test differencesShow 4 test diffs4 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 48622726c4a91c87bf6cd4dbe1000c95df59906e --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (4862272): comparison URL. Overall result: ❌ regressions - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary 3.1%, secondary -2.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 2.3%, secondary -0.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 477.934s -> 477.392s (-0.11%) |
|
It remains to be seen tonight whether the migration (esp. rust-lang/promote-release#101) is working alright. Once it's confirmed to be okay I'll move on to rust-lang/rustup#3717. |
Closes #75833, part of rust-lang/rustup#3717:
The TLDR is that currently the "rustc book"'s path clashes with "rustc APIs" (i.e.
rustc-docs), so we change the path of the latter to make it distributable via rustup.I'm new to
bootstrapso please feel free to point out any inconsistencies or any extra work I'll need to do to fully land this somewhat breaking change.