Uh oh!
There was an error while loading. Please reload this page.
Refactor versions detection in build-manifest - #77145
Merged
bors merged 5 commits intoSep 29, 2020
Merged
Conversation
Mark-Simulacrum
commented
Sep 24, 2020
Member
Hm, so we can just merge this, but I'm thinking it would be good to @bors try and try to release a dev-static nightly with it just to test that it works on RCS as well. Unfortunately our CI here doesn't run build manifest in any capacity :/ |
bors
commented
Sep 24, 2020
Collaborator
bors added a commit
to rust-lang-ci/rust
that referenced
this pull request
Sep 24, 2020
…versions, r=<try> Refactor versions detection in build-manifest This PR refactors how `build-manifest` handles versions, making the following changes: * `build-manifest` now detects the "package releases" on its own, without relying on rustbuild providing them through CLI arguments. This drastically simplifies calling the tool outside of `x.py`, and will allow to ship the prebuilt tool in a tarball in the future, with the goal of stopping to invoke `x.py` during `promote-release`. * The `tar` command is not used to extract the version and the git hash from tarballs anymore. The `flate2` and `tar` crates are used instead. This makes detecting those pieces of data way faster, as the archive is decompressed just once and we stop parsing the archive once all the information is retrieved. * The code to extract the version and the git hash now stores all the collected data dynamically, without requiring to add new fields to the `Builder` struct every time. I tested the changes locally and it should behave the same as before. r? `@Mark-Simulacrum`
bors
commented
Sep 24, 2020
Collaborator
☀️ Try build successful - checks-actions, checks-azure |
emilyalbini
commented
Sep 24, 2020
MemberAuthor
Kicked off promotion. |
emilyalbini
commented
Sep 24, 2020
MemberAuthor
Promotion ended, it seems to work? |
emilyalbiniforce-pushed
the
refactor-build-manifest-versions
branch
from
September 24, 2020 17:26
bb8430e to
73d9c24CompareMark-Simulacrum
commented
Sep 24, 2020
Member
@bors r+ |
bors
commented
Sep 24, 2020
Collaborator
📌 Commit 73d9c24 has been approved by |
emilyalbini
commented
Sep 25, 2020
MemberAuthor
@bors r- Wait, there is actually a bug, this does not find the |
emilyalbini
commented
Sep 25, 2020
MemberAuthor
Addressed. |
emilyalbini
commented
Sep 29, 2020
MemberAuthor
@bors r=Mark-Simulacrum |
bors
commented
Sep 29, 2020
Collaborator
📌 Commit 0749ad0 has been approved by |
bors
commented
Sep 29, 2020
Collaborator
bors
commented
Sep 29, 2020
Collaborator
☀️ Test successful - checks-actions, checks-azure |
ehuss added a commit
to ehuss/rust
that referenced
this pull request
Jan 17, 2022
The version was removed in rust-lang#77145
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR refactors how
build-manifesthandles versions, making the following changes:build-manifestnow detects the "package releases" on its own, without relying on rustbuild providing them through CLI arguments. This drastically simplifies calling the tool outside ofx.py, and will allow to ship the prebuilt tool in a tarball in the future, with the goal of stopping to invokex.pyduringpromote-release.tarcommand is not used to extract the version and the git hash from tarballs anymore. Theflate2andtarcrates are used instead. This makes detecting those pieces of data way faster, as the archive is decompressed just once and we stop parsing the archive once all the information is retrieved.Builderstruct every time.I tested the changes locally and it should behave the same as before.
r? @Mark-Simulacrum