Skip to content

Conversation

jyn514
Copy link
Member

@jyn514 jyn514 commented May 31, 2023

This makes it more clear that we're using stage 0 to document rustc, not that we're documenting stage0 rustc itself.

It also fixes a bug in msg_sysroot_tool that would print Docing, and removes the Debug impl for Kind to make sure it doesn't happen again.

Before:

Documenting stage0 compiler {rustc-main} (aarch64-apple-darwin)

After:

Documenting compiler {rustc-main} (stage0 -> stage1, aarch64-apple-darwin)

thanks @BoxyUwU for catching this!

@rustbot
Copy link
Collaborator

rustbot commented May 31, 2023

r? @clubby789

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels May 31, 2023
This makes it more clear that we're using stage 0 *to document* rustc, not that we're documenting
stage0 rustc itself.

It also fixes a bug in `msg_sysroot_tool` that would print `Docing`, and removes the `Debug` impl
for `Kind` to make sure it doesn't happen again.

Before:
```
Documenting stage0 compiler {rustc-main} (aarch64-apple-darwin)
```

After:
```
Documenting compiler {rustc-main} (stage0 -> stage1, aarch64-apple-darwin)
```
@clubby789
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented May 31, 2023

📌 Commit 38c0ba7 has been approved by clubby789

It is now in the queue for this repository.

@bors bors 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 May 31, 2023
@bors
Copy link
Collaborator

bors commented Jun 2, 2023

⌛ Testing commit 38c0ba7 with merge a9baa16...

@bors
Copy link
Collaborator

bors commented Jun 2, 2023

☀️ Test successful - checks-actions
Approved by: clubby789
Pushing a9baa16 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jun 2, 2023
@bors bors merged commit a9baa16 into rust-lang:master Jun 2, 2023
@rustbot rustbot added this to the 1.72.0 milestone Jun 2, 2023
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (a9baa16): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
-0.1% [-0.1%, -0.1%] 1
Improvements ✅
(secondary)
-2.3% [-2.3%, -2.3%] 1
All ❌✅ (primary) -0.1% [-0.1%, -0.1%] 1

Cycles

This benchmark run did not return any relevant results for this metric.

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 647.393s -> 644.81s (-0.40%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants