Revert #132772 to fix unknown git commit hash

[jieyouxu]

Reverts #132772 to address #132845, we seem to have unintentionally omitted commit hash.

r? @onur-ozkan

[rustbot]

This PR modifies src/bootstrap/src/core/config.

If appropriate, please update CONFIG_CHANGE_HISTORY in src/bootstrap/src/utils/change_tracker.rs.

This PR modifies src/bootstrap/defaults.

If appropriate, please update CONFIG_CHANGE_HISTORY in src/bootstrap/src/utils/change_tracker.rs.

[RalfJung]

@bors r+ p=10
(I'm not on the team, but this is just a revert, and we want to land it ASAP so it does not become part of the next nightly)

[bors]

📌 Commit 5dbf135 has been approved by RalfJung

It is now in the queue for this repository.

[jieyouxu]

Since queue is very light, I'm going to yield the other PR that just started.

@bors rollup=never

[bors]

⌛ Testing commit 5dbf135 with merge 00c8722...

[jieyouxu]

I'm not on the team, but this is just a revert, and we want to land it ASAP so it does not become part of the next nightly

(I was going to self-approve as per revert policy but was double-checking queue, this works too 😆)

[jieyouxu]

Huh. This failed well because commit hash was unknown. I'll temporarily disable this test to get the revert through.

[bors]

💔 Test failed - checks-actions

[jieyouxu]

I'll babysit this revert to make sure it goes through. Closing the tree temporarily to not have to keep yielding other PRs.

@bors p=50
@bors treeclosed=1000

[jieyouxu]

@bors r+

[bors]

📌 Commit c4c6d2b has been approved by jieyouxu

It is now in the queue for this repository.

[bors]

🌲 The tree is currently closed for pull requests below priority 1000. This pull request will be tested once the tree is reopened.

[jieyouxu]

@bors p=1001

[bors]

⌛ Testing commit c4c6d2b with merge c22887b...

[jieyouxu]

PR to re-enable version-verbose-commit-hash after this revert PR lands: #132848

[RalfJung]

PR to re-enable version-verbose-commit-hash after this revert PR lands: #132846

You meant #132848 I think :)

[RalfJung]

The 2nd commit here can be reverted once #132852 landed.

[bors]

☀️ Test successful - checks-actions
Approved by: jieyouxu
Pushing c22887b to master...

[RalfJung]

@bors treeclosed-

[rust-timer]

Finished benchmarking commit (c22887b): comparison URL.

Overall result: ❌ regressions - no action needed

@rustbot label: -perf-regression

Instruction count

This is the most reliable metric that we have; it was used to determine the overall result at the top of this comment. However, even this metric can sometimes exhibit noise.

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

Max RSS (memory usage)

Results (primary 2.0%, secondary -3.1%)

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)	2.0%	[0.6%, 3.8%]	4
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-3.1%	[-3.7%, -2.5%]	2
All ❌✅ (primary)	2.0%	[0.6%, 3.8%]	4

Cycles

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

Binary size

Results (primary 0.1%, secondary 0.2%)

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.1%	[0.0%, 0.2%]	72
Regressions ❌ (secondary)	0.2%	[0.0%, 0.8%]	65
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	0.1%	[0.0%, 0.2%]	72

Bootstrap: 787.727s -> 787.115s (-0.08%)
Artifact size: 335.29 MiB -> 335.35 MiB (0.02%)