Update cargo submodule#154734
Conversation
|
Some changes occurred in src/tools/cargo cc @ehuss |
|
|
rust-lang/cargo#16745 is huge enough so want to get this to nightly ASAP to get a longer test window before hitting stable. |
|
@bors r+ rollup=never p=1 |
This comment has been minimized.
This comment has been minimized.
Update cargo submodule 12 commits in 888f675344eb1cf2308fd53183e667bdd2c58e51..0c7d80bf5ad3b873a5dbf44e0185152890c6d18a 2026-03-30 16:59:25 +0000 to 2026-04-02 20:48:06 +0000 - refactor(network): use async for registry network operations (rust-lang/cargo#16745) - fix(compile): Don't hide hard warnings with build.warnings=allow (rust-lang/cargo#16827) - chore: Upgrade incompatible dependencies (rust-lang/cargo#16825) - fix(compile): build.warnings=allow should not hide denied diagnostics (rust-lang/cargo#16824) - fix(install): Ignore resolver.lockfile-path (rust-lang/cargo#16823) - chore(deps): update compatible (rust-lang/cargo#16820) - refactor(compile): Consolidate output options (rust-lang/cargo#16821) - refactor: Export public dependencies from internal crates (rust-lang/cargo#16819) - refactor(progress): Make the code more approachable (rust-lang/cargo#16813) - Simplified build script bin names in new layout (rust-lang/cargo#16812) - fix: set CARGO env var during rustc -vV probe (rust-lang/cargo#16811) - Split out `cargo-util-terminal` (rust-lang/cargo#16809) r? ghost
This comment has been minimized.
This comment has been minimized.
|
💔 Test for 9814779 failed: CI. Failed job:
|
This follows upstream's change in madler/zlib@1382e6611 due to EWOULDBLOCK is used since madler/zlib@81cc0bebe and `-D_POSIX_SOURCE` would hide `#define EWOULDBLOCK EAGAIN`. See <https://github.com/freebsd/freebsd-src/blob/3e9f4fd6fc50300d052d5619d82a14d3488483d5/sys/sys/errno.h#L92-L93> Actually I feel like _POSIX_C_SOURCE=200112L maybe redundant though let's just do whatever upstream does This was found in * <rust-lang/rust#154734> * <https://triage.rust-lang.org/gha-logs/rust-lang/rust/69786578177>
This comment has been minimized.
This comment has been minimized.
|
Going to try freebsd with my libz-sys change weihanglo/libz-sys@73817ec. Since it is a relatively domain I am not really familiar with, I'll try more tier 2 with host tools platforms. @bors try jobs=dist-x86_64-freebsd,dist-x86_64-illumos,dist-x86_64-netbsd,dist-x86_64-solaris,dist-x86_64-apple,dist-powerpc-linux-gnu,dist-riscv64-linux |
This comment has been minimized.
This comment has been minimized.
Update cargo submodule try-job: dist-x86_64-freebsd try-job: dist-x86_64-illumos try-job: dist-x86_64-netbsd try-job: dist-x86_64-solaris try-job: dist-x86_64-apple try-job: dist-powerpc-linux-gnu try-job: dist-riscv64-linux
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Oops. #154734 (comment) failed. Try again. @bors try jobs=dist-x86_64-freebsd,dist-x86_64-illumos,dist-x86_64-netbsd,dist-x86_64-solaris,dist-x86_64-apple,dist-powerpc-linux,dist-riscv64-linux |
This comment has been minimized.
This comment has been minimized.
Update cargo submodule try-job: dist-x86_64-freebsd try-job: dist-x86_64-illumos try-job: dist-x86_64-netbsd try-job: dist-x86_64-solaris try-job: dist-x86_64-apple try-job: dist-powerpc-linux try-job: dist-riscv64-linux
Update cargo submodule 15 commits in 888f675344eb1cf2308fd53183e667bdd2c58e51..a357df4c26fc14514e66aae2a269456b5545c7db 2026-03-30 16:59:25 +0000 to 2026-04-03 16:47:15 +0000 - chore: downgrade to libz-sys@1.1.24 (rust-lang/cargo#16833) - feat: add frame-pointers profile option (rust-lang/cargo#16742) - upgrade to gix@0.81.0 (rust-lang/cargo#16828) - refactor(network): use async for registry network operations (rust-lang/cargo#16745) - fix(compile): Don't hide hard warnings with build.warnings=allow (rust-lang/cargo#16827) - chore: Upgrade incompatible dependencies (rust-lang/cargo#16825) - fix(compile): build.warnings=allow should not hide denied diagnostics (rust-lang/cargo#16824) - fix(install): Ignore resolver.lockfile-path (rust-lang/cargo#16823) - chore(deps): update compatible (rust-lang/cargo#16820) - refactor(compile): Consolidate output options (rust-lang/cargo#16821) - refactor: Export public dependencies from internal crates (rust-lang/cargo#16819) - refactor(progress): Make the code more approachable (rust-lang/cargo#16813) - Simplified build script bin names in new layout (rust-lang/cargo#16812) - fix: set CARGO env var during rustc -vV probe (rust-lang/cargo#16811) - Split out `cargo-util-terminal` (rust-lang/cargo#16809)
|
BTW I've tried 20 times with and without the patch,
So thanks Arlo for the fix! |
|
Seems stuck in Last lines of output are: By comparing to a successful run, there are still more files for the LLVM build to install. Since the LLVM build seems unrelated to updating cargo: @bors retry |
|
❗ You can only retry pull requests that are approved and have a previously failed auto build. Hint: There is currently a pending auto build on this PR. To cancel it, run |
|
@bors cancel |
|
Auto build was cancelled. Cancelled workflows: The next pull request likely to be tested is #154734. |
This comment has been minimized.
This comment has been minimized.
Update cargo submodule 15 commits in 888f675344eb1cf2308fd53183e667bdd2c58e51..a357df4c26fc14514e66aae2a269456b5545c7db 2026-03-30 16:59:25 +0000 to 2026-04-03 16:47:15 +0000 - chore: downgrade to libz-sys@1.1.24 (rust-lang/cargo#16833) - feat: add frame-pointers profile option (rust-lang/cargo#16742) - upgrade to gix@0.81.0 (rust-lang/cargo#16828) - refactor(network): use async for registry network operations (rust-lang/cargo#16745) - fix(compile): Don't hide hard warnings with build.warnings=allow (rust-lang/cargo#16827) - chore: Upgrade incompatible dependencies (rust-lang/cargo#16825) - fix(compile): build.warnings=allow should not hide denied diagnostics (rust-lang/cargo#16824) - fix(install): Ignore resolver.lockfile-path (rust-lang/cargo#16823) - chore(deps): update compatible (rust-lang/cargo#16820) - refactor(compile): Consolidate output options (rust-lang/cargo#16821) - refactor: Export public dependencies from internal crates (rust-lang/cargo#16819) - refactor(progress): Make the code more approachable (rust-lang/cargo#16813) - Simplified build script bin names in new layout (rust-lang/cargo#16812) - fix: set CARGO env var during rustc -vV probe (rust-lang/cargo#16811) - Split out `cargo-util-terminal` (rust-lang/cargo#16809)
|
Ah. I thought it told us to cancel 😰. |
|
Cancelling it caused more logs to make it through. Last log before it hung was: Which does seem cargo related. @bors r- |
|
This pull request was unapproved. Auto build was cancelled due to unapproval. Cancelled workflows: |
I'm pretty sure this does not use the in-tree cargo. This is running a benchmark against an old clone of cargo (cargo-0.87.1) compiling that old cargo, using stage2 rustc and stage0 cargo. |
|
Yeah, so let's just try again @bors retry |
|
❗ You can only retry pull requests that are approved and have a previously failed auto build. |
|
@bors r+ rollup=never |
|
@bors p=1 |
|
@bors p=6 |
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 c3bd628 (parent) -> ad4b935 (this PR) Test differencesShow 18 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 ad4b9354009cb6bd5a9ff1b5f5a63a13ec98ebc9 --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 (ad4b935): comparison URL. Overall result: ✅ improvements - 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 4.8%, secondary -5.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -3.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary -0.1%, secondary 0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 487.084s -> 488.293s (0.25%) |
View all comments
15 commits in 888f675344eb1cf2308fd53183e667bdd2c58e51..a357df4c26fc14514e66aae2a269456b5545c7db
2026-03-30 16:59:25 +0000 to 2026-04-03 16:47:15 +0000
cargo-util-terminal(Split outcargo-util-terminalcargo#16809)