Update cargo submodule - #160527
Conversation
|
|
@bors r+ rollup=never p=1 |
|
@bors p=9 |
|
@bors treeopen |
|
Tree is now open for merging. |
This comment has been minimized.
This comment has been minimized.
Update cargo submodule 21 commits in 7c83d4cc0953b81d823e47d640c64da9b8bd4fac..c79e8f89441b3e73d6d65d125c0c745792808c74 2026-07-29 21:34:53 +0000 to 2026-08-04 19:17:33 +0000 - fix(diag): Ensure diagnostic titles work without snippets (rust-lang/cargo#17304) - refactor: Remove unnecessary mut in sources (rust-lang/cargo#17305) - feat(trim-paths): emit unremap files for final artifacts (rust-lang/cargo#17303) - fix: prevent panic when `package.build` is empty (rust-lang/cargo#17268) - Add a suggestion when adding `[lints]` to a workspace to use `[workspace.lints]` instead (rust-lang/cargo#17300) - chore(deps): update embarkstudios/cargo-deny-action action to v2.1.1 (rust-lang/cargo#17291) - refactor: move sysroot lookup to GlobalContext (rust-lang/cargo#17276) - fix(trim-paths): unambiguous and reversible remap rules (rust-lang/cargo#17302) - Avoid parsing unchanged lockfiles (rust-lang/cargo#17301) - Remove unnecessary to_path_buf (rust-lang/cargo#17295) - chore(deps): update cargo-semver-checks to v0.50.0 (rust-lang/cargo#17297) - chore(deps): update actions/checkout action to v6.1.0 (rust-lang/cargo#17290) - Remove unnecessary return at end of functions (rust-lang/cargo#17292) - make __CARGO_TEST_FORCE_ARGFILE available in distributed builds (rust-lang/cargo#17293) - Fix manual_readme lint for lower-priority README files (rust-lang/cargo#17208) - fix(git): make checkout names independent of git config (rust-lang/cargo#17289) - fix(diag): Rename redundant_readme to manual_readme (rust-lang/cargo#17288) - Remove redundant double call .to_string() (rust-lang/cargo#17286) - fix(completions): complete paths for cargo run arguments (rust-lang/cargo#17284) - test(git): exercise multi git revision lockfile (rust-lang/cargo#17279) - add context to lints documentation (rust-lang/cargo#17273) r? ghost
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test for 5085881 failed: CI. Failed job:
|
|
Interesting, benchmarks failed. I'll take a look into it. |
|
I think that rust-lang/cargo#17276 broke |
|
rust-lang/rustc-perf#2520 + rustc-perf submodule sync should unblock this PR. |
|
#160574 should unblock this PR (hopefully). |
This comment has been minimized.
This comment has been minimized.
Update cargo submodule try-job: dist-x86_64-linux-quick
|
@bors try parent=5a89b81fc99809e2ae0f6a9dd1d9d71d1db4fa4d Forgot to set the parent. |
This comment has been minimized.
This comment has been minimized.
|
TIL try parent |
|
Nice nice. @Kobzol should we roll these two PR up? |
Update rustc-perf submodule To bring in rust-lang/rustc-perf#2520, which should unblock rust-lang#160527.
|
I'll do the easier thing and throw #160574 into a rollup then this can go afterwards |
This comment has been minimized.
This comment has been minimized.
Update cargo submodule try-job: dist-x86_64-linux
Update rustc-perf submodule To bring in rust-lang/rustc-perf#2520, which should unblock rust-lang/rust#160527.
This comment has been minimized.
This comment has been minimized.
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 b070f45 (parent) -> 887804d (this PR) Test differencesShow 32 test diffsStage 2
Additionally, 4 doctest diffs were found. These are ignored, as they are noisy. Job group index Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 887804d5686c3bf9a090b5bbc23e3dcdfd9e316a --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 (887804d): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)Results (primary 2.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -3.2%, secondary 4.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 459.982s -> 458.35s (-0.35%) |
View all comments
21 commits in 7c83d4cc0953b81d823e47d640c64da9b8bd4fac..c79e8f89441b3e73d6d65d125c0c745792808c74
2026-07-29 21:34:53 +0000 to 2026-08-04 19:17:33 +0000
package.buildis empty (fix: prevent panic whenpackage.buildis empty cargo#17268)[lints]to a workspace to use[workspace.lints]instead (Add a suggestion when adding[lints]to a workspace to use[workspace.lints]instead cargo#17300)r? ghost