Skip to content

rustbuild: fix local_rebuild - #52969

Merged
bors merged 1 commit into
rust-lang:masterfrom
Keruspe:local_rebuild
Aug 4, 2018
Merged

rustbuild: fix local_rebuild#52969
bors merged 1 commit into
rust-lang:masterfrom
Keruspe:local_rebuild

Conversation

@Keruspe

Copy link
Copy Markdown
Contributor

If we detect a local rebuild (e.g. bootstrap compiler is the same version as target compiler), we set stage to 1.
When trying to build e.g. UnstableBook, we use Mode::ToolBootstrap and stage is 1.
Just allow Mode::ToolBootstrap and stagge != 0 if we are in a local_rebuild

This fixes building current master using current beta (as master hasn't yet been bumped to 1.30).
This should be backported to beta too, as currently we cannot build beta using itself because of that.

r? @alexcrichton

If we detect a local rebuild (e.g. bootstrap compiler is the same version as target compiler), we set stage to 1.
When trying to build e.g. UnstableBook, we use Mode::ToolBootstrap and stage is 1.
Just allow Mode::ToolBootstrap and stagge != 0 if we are in a local_rebuild
Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>
@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 2, 2018
@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+ rollup

@bors

bors commented Aug 2, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 66a4718 has been approved by alexcrichton

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 2, 2018
cramertj added a commit to cramertj/rust that referenced this pull request Aug 2, 2018
rustbuild: fix local_rebuild
If we detect a local rebuild (e.g. bootstrap compiler is the same version as target compiler), we set stage to 1.
When trying to build e.g. UnstableBook, we use Mode::ToolBootstrap and stage is 1.
Just allow Mode::ToolBootstrap and stagge != 0 if we are in a local_rebuild
This fixes building current master using current beta (as master hasn't yet been bumped to 1.30).
This should be backported to beta too, as currently we cannot build beta using itself because of that.
r? @alexcrichton
@cramertjcramertj mentioned this pull request Aug 2, 2018
cramertj added a commit to cramertj/rust that referenced this pull request Aug 3, 2018
rustbuild: fix local_rebuild
If we detect a local rebuild (e.g. bootstrap compiler is the same version as target compiler), we set stage to 1.
When trying to build e.g. UnstableBook, we use Mode::ToolBootstrap and stage is 1.
Just allow Mode::ToolBootstrap and stagge != 0 if we are in a local_rebuild
This fixes building current master using current beta (as master hasn't yet been bumped to 1.30).
This should be backported to beta too, as currently we cannot build beta using itself because of that.
r? @alexcrichton
@cramertjcramertj mentioned this pull request Aug 3, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Aug 4, 2018
rustbuild: fix local_rebuild
If we detect a local rebuild (e.g. bootstrap compiler is the same version as target compiler), we set stage to 1.
When trying to build e.g. UnstableBook, we use Mode::ToolBootstrap and stage is 1.
Just allow Mode::ToolBootstrap and stagge != 0 if we are in a local_rebuild
This fixes building current master using current beta (as master hasn't yet been bumped to 1.30).
This should be backported to beta too, as currently we cannot build beta using itself because of that.
r? @alexcrichton
@kennytmkennytm mentioned this pull request Aug 4, 2018
bors added a commit that referenced this pull request Aug 4, 2018
Rollup of 14 pull requests
Successful merges:
- #51919 (Provide `{to,from}_{ne,le,be}_bytes` functions on integers)
- #52940 (Align 6-week cycle check with beta promotion instead of stable release.)
- #52968 (App-lint-cability)
- #52969 (rustbuild: fix local_rebuild)
- #52995 (Remove unnecessary local in await! generator)
- #52996 (RELEASES.md: fix the `hash_map::Entry::or_default` link)
- #53001 (privacy: Fix an ICE in `path_is_private_type`)
- #53003 (Stabilize --color and --error-format options in rustdoc)
- #53022 (volatile operations docs: clarify that this does not help wrt. concurrency)
- #53024 (Specify reentrancy gurantees of `Once::call_once`)
- #53041 (Fix invalid code css rule)
- #53047 (Make entire row of doc search results clickable)
- #53050 (Make left column of rustdoc search results narrower)
- #53062 (Remove redundant field names in structs)
@bors
bors merged commit 66a4718 into rust-lang:masterAug 4, 2018
@Keruspe

Copy link
Copy Markdown
ContributorAuthor

Is it possible to nominate this for backport to the beta branch?

@cuvipercuviper added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Aug 31, 2018
@cuviper

Copy link
Copy Markdown
Member

Nominated for beta, as requested.

@alexcrichtonalexcrichton added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Aug 31, 2018
@emilyalbiniemilyalbini mentioned this pull request Sep 1, 2018
@emilyalbiniemilyalbini removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Sep 1, 2018
bors added a commit that referenced this pull request Sep 1, 2018
[beta] Rollup backports
Merged and approved:
* #53653: Address two regressions
* #53377: std: Use target_pointer_width for BACKTRACE_ELF_SIZE
* #52969: rustbuild: fix local_rebuild
r? @ghost
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beta-acceptedAccepted for backporting to the compiler in the beta channel.S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Keruspe@alexcrichton@bors@cuviper@emilyalbini@rust-highfive