Skip to content

Really stop recompressing to-be-pruned artifacts#105

Merged
emilyalbini merged 1 commit intorust-lang:masterfrom
Mark-Simulacrum:really-skip-recompressing
Apr 9, 2026
Merged

Really stop recompressing to-be-pruned artifacts#105
emilyalbini merged 1 commit intorust-lang:masterfrom
Mark-Simulacrum:really-skip-recompressing

Conversation

@Mark-Simulacrum
Copy link
Copy Markdown
Member

This was added as a temporary measure in a226029 because otherwise build-manifest failed to read version information, which resulted in the packages getting dropped. As of the mentioned PR (rust-lang/rust#110436) this is no longer an issue, and that PR has long reached all branches.

Empirically, we see in build metrics that nightly releases take around 1h50m, while beta and stable releases take above 3 hours. This is largely explained by extra recompression done in beta/stable releases today of files that subsequently are deleted in prune_unused_files.

This also avoids having different code paths on different channels, which seems like a good win no matter what.

This was added as a temporary measure in a226029 because otherwise
build-manifest failed to read version information, which resulted in the
packages getting dropped. As of the mentioned PR (rust-lang/rust#110436)
this is no longer an issue, and that PR has long reached all branches.

Empirically, we see in build metrics that nightly releases take around
1h50m, while beta and stable releases take above 3 hours. This is
largely explained by extra recompression done in beta/stable releases
today of files that subsequently are deleted in `prune_unused_files`.

This also avoids having different code paths on different channels,
which seems like a good win no matter what.
@emilyalbini emilyalbini merged commit 9db98eb into rust-lang:master Apr 9, 2026
6 checks passed
@Mark-Simulacrum Mark-Simulacrum deleted the really-skip-recompressing branch April 9, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants