Conversation
|
Some changes occurred in src/tools/cargo cc @ehuss |
|
|
@bors r+ rollup=never p=1 |
Update cargo 23 commits in 840b83a10fb0e039a83f4d70ad032892c287570a..9a0712751ca9444775c7352d3c942a155b656437 2025-07-30 13:59:19 +0000 to 2025-08-15 20:30:47 +0000 - Fixes error while running the cargo clippy --all-targets -- -D warning (rust-lang/cargo#15843) - Clarify that `cargo doc --no-deps` is cumulative and won’t delete prev (rust-lang/cargo#15800) - docs: Formatting and cross-linking to build-dir/target-dir docs (rust-lang/cargo#15840) - Stabilize `build.build-dir` (rust-lang/cargo#15833) - make resolve features public for cargo-as-a-library (rust-lang/cargo#15835) - chore(deps): bump slab from 0.4.10 to 0.4.11 (rust-lang/cargo#15832) - chore: remove x86_64-apple-darwin from CI and tests (rust-lang/cargo#15831) - chore(deps): update msrv (3 versions) to v1.87 (rust-lang/cargo#15819) - perf(package): Always reuse the workspace's target-dir (rust-lang/cargo#15783) - More helpful error for invalid cargo-features = [] (rust-lang/cargo#15781) - Add initial integration for `--json=timings` behing `-Zsection-timings` (rust-lang/cargo#15780) - add is_inherited methods to InheritableDependency and InheritableField (rust-lang/cargo#15828) - chore(deps): update compatible (rust-lang/cargo#15804) - docs(unstable): Link out to the Plumbing commands effort (rust-lang/cargo#15821) - chore(deps): update cargo-semver-checks to v0.43.0 (rust-lang/cargo#15825) - test(build-std): relax the thread name assertion (rust-lang/cargo#15822) - chore(deps): update msrv (1 version) to v1.89 (rust-lang/cargo#15815) - Update semver tests for 1.89 (rust-lang/cargo#15816) - Accessing each build script's `OUT_DIR` and in the correct order (rust-lang/cargo#15776) - chore: bump to 0.92.0; update changelog (rust-lang/cargo#15807) - docs: `-Zpackage-workspace` has been stabilized (rust-lang/cargo#15808) - chore(deps): update rust crate cargo_metadata to 0.21.0 (rust-lang/cargo#15795) - docs(build-rs): Fix broken intra-doc links (rust-lang/cargo#15810) r? ghost
This comment has been minimized.
This comment has been minimized.
|
💔 Test failed - checks-actions |
There is incompatibility between rustix@1.0.8 and libc@0.2.175 that leads to compilation errors. See * bytecodealliance/rustix#1496 * rust-lang/rust#145478 (comment)
### What does this PR try to resolve? There is incompatibility between rustix@1.0.8 and libc@0.2.175 that leads to compilation errors. See * bytecodealliance/rustix#1496 * rust-lang/rust#145478 (comment) ### How to test and review this PR? In rust-lang/rust, update the `src/tools/cargo` submodule accordingly, and run ``` cargo run --manifest-path src/ci/citool/Cargo.toml run-local dist-loongarch64-musl ``` With this downgrade it should compile.
fbcca5f to
08263c9
Compare
|
rust-lang/cargo#15851 should unblock loongarch64. @bors retry |
|
@bors r+ rollup=never p=1 |
Update cargo 28 commits in 840b83a10fb0e039a83f4d70ad032892c287570a..71eb84f21aef43c07580c6aed6f806a6299f5042 2025-07-30 13:59:19 +0000 to 2025-08-17 17:18:56 +0000 - update tests to match lint message changes from #140794 (rust-lang/cargo#15849) - chore: downgrade to libc@0.2.174 (rust-lang/cargo#15851) - Reorder `lto` options in profiles.md (rust-lang/cargo#15841) - feat(unstable): add -Zbuild-analysis unstable feature (rust-lang/cargo#15845) - refactor(unstable): group stabilized features (rust-lang/cargo#15846) - Fixes error while running the cargo clippy --all-targets -- -D warning (rust-lang/cargo#15843) - Clarify that `cargo doc --no-deps` is cumulative and won’t delete prev (rust-lang/cargo#15800) - docs: Formatting and cross-linking to build-dir/target-dir docs (rust-lang/cargo#15840) - Stabilize `build.build-dir` (rust-lang/cargo#15833) - make resolve features public for cargo-as-a-library (rust-lang/cargo#15835) - chore(deps): bump slab from 0.4.10 to 0.4.11 (rust-lang/cargo#15832) - chore: remove x86_64-apple-darwin from CI and tests (rust-lang/cargo#15831) - chore(deps): update msrv (3 versions) to v1.87 (rust-lang/cargo#15819) - perf(package): Always reuse the workspace's target-dir (rust-lang/cargo#15783) - More helpful error for invalid cargo-features = [] (rust-lang/cargo#15781) - Add initial integration for `--json=timings` behing `-Zsection-timings` (rust-lang/cargo#15780) - add is_inherited methods to InheritableDependency and InheritableField (rust-lang/cargo#15828) - chore(deps): update compatible (rust-lang/cargo#15804) - docs(unstable): Link out to the Plumbing commands effort (rust-lang/cargo#15821) - chore(deps): update cargo-semver-checks to v0.43.0 (rust-lang/cargo#15825) - test(build-std): relax the thread name assertion (rust-lang/cargo#15822) - chore(deps): update msrv (1 version) to v1.89 (rust-lang/cargo#15815) - Update semver tests for 1.89 (rust-lang/cargo#15816) - Accessing each build script's `OUT_DIR` and in the correct order (rust-lang/cargo#15776) - chore: bump to 0.92.0; update changelog (rust-lang/cargo#15807) - docs: `-Zpackage-workspace` has been stabilized (rust-lang/cargo#15808) - chore(deps): update rust crate cargo_metadata to 0.21.0 (rust-lang/cargo#15795) - docs(build-rs): Fix broken intra-doc links (rust-lang/cargo#15810)
|
💔 Test failed - checks-actions |
|
The (I'm assuming that's more CI/runner flakiness, unrelated to any particular changes.) |
|
Even if this job spontaneously becomes unstuck, it's still not going to finish in under 6 hours. Better to reroll the CI dice now. @bors retry |
|
Failed in rollup: #145552 (comment) |
|
Whoops, didn't mean to do that to this PR. @bors r=weihanglo |
actions/runner#3792 maybe? I don't know how self-hosted runners and Actions talk to each other though |
|
☀️ 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 239e8b1 (parent) -> aaf87a6 (this PR) Test differencesShow 10 test diffsStage 2
Additionally, 3 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 aaf87a6d289053246d19f94fbc9972582816e93d --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 (aaf87a6): comparison URL. Overall result: ❌✅ regressions and improvements - please read the text belowOur benchmarks found a performance regression caused by this PR. Next Steps:
@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 -2.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 2.3%, secondary 4.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (secondary -0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 470.133s -> 472.035s (0.40%) |
|
Hard to be 100% confident given recency, but I'm pretty sure this is just bimodality. Cargo shouldn't have much practical relation to rustc perf anyway, assuming we don't tweak codegen/rustc flags (and I don't see that in the diff above). |
|
I didn't see anything outstanding. Could be rust-lang/cargo#15804 that updating some dependencies (like toml?). syn@2.0.101 does have a 200+ lines of Cargo.toml, though I am not sure if it really caused the regression: https://docs.rs/crate/syn/2.0.101/source/Cargo.toml. |
|
We don't measure Cargo's execution, just rustc, so basically nothing at the Cargo level should get reflected (even if it's terrible :) |
|
@rustbot label: +perf-regression-triaged (to avoid this falling into perf triage again) |
28 commits in 840b83a10fb0e039a83f4d70ad032892c287570a..71eb84f21aef43c07580c6aed6f806a6299f5042
2025-07-30 13:59:19 +0000 to 2025-08-17 17:18:56 +0000
ltooptions in profiles.md (Reorderltooptions in profiles.md cargo#15841)cargo doc --no-depsis cumulative and won’t delete prev (Clarify thatcargo doc --no-depsis cumulative and won’t delete prev cargo#15800)build.build-dir(Stabilizebuild.build-dircargo#15833)--json=timingsbehing-Zsection-timings(Add initial integration for--json=timingsbehing-Zsection-timingscargo#15780)OUT_DIRand in the correct order (Preserve the order of the build scripts cargo#15776)-Zpackage-workspacehas been stabilized (docs:-Zpackage-workspacehas been stabilized cargo#15808)