Bump cc and run cargo update for bootstrap

[tgross35]

Bump cc to 1.1.22, which includes a caching fix. Also run cargo update which does a minor increment on a few dependencies.

[rustbot]

r? @Kobzol

rustbot has assigned @Kobzol.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[rust-log-analyzer]

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

COPY host-x86_64/mingw-check/validate-toolstate.sh /scripts/
COPY host-x86_64/mingw-check/validate-error-codes.sh /scripts/

# NOTE: intentionally uses python2 for x.py so we can test it still works.
# validate-toolstate only runs in our CI, so it's ok for it to only support python3.
ENV SCRIPT TIDY_PRINT_DIFF=1 python2.7 ../x.py test \
           --stage 0 src/tools/tidy tidyselftest --extra-checks=py:lint,cpp:fmt
# This file is autogenerated by pip-compile with Python 3.10
# by the following command:
#
#    pip-compile --allow-unsafe --generate-hashes reuse-requirements.in

[tgross35]

The differences in cc are minor enough that I doubt anything needs to change in bootstrap (responding to the comment). We specifically want rust-lang/cc-rs#1215, which seems like it has a chance in improving the MSVC issue. More context on that starting around https://rust-lang.zulipchat.com/#narrow/stream/242791-t-infra/topic/Spurious.20CI.20errors.20on.20x86_64-msvc-ext/near/472962033.

[rust-log-analyzer]

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

[Kobzol]

Looks like the dependency count stayed the same. The comment in the Cargo.toml file about cc updates sounded a bit worrying though :)

[tgross35]

Hm, not sure why it looks like these are getting the network errors.

Looks like the dependency count stayed the same. The comment in the Cargo.toml file about cc updates sounded a bit worrying though :)

Yes, I thought so too :) I made this just to see what the changes looked like after the automation PR. But with cc we specifically want rust-lang/cc-rs#1215 since it might help the MSVC error. The only other changes are rust-lang/cc-rs#1209 and rust-lang/cc-rs#1212 which I don't think would affect anything in bootstrap so that doesn't seem too concerning

[tgross35]

cc also @ChrisDenton since you've been doing some of the cc updates

[rust-log-analyzer]

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

[rust-log-analyzer]

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

[tgross35]

Cc @onur-ozkan since you authored the comment at

rust/src/bootstrap/Cargo.toml

Lines 37 to 39 in a1fd235

	# Most of the time updating these dependencies requires modifications to the
	# bootstrap codebase(e.g., https://github.com/rust-lang/rust/issues/124565);
	# otherwise, some targets will fail. That's why these dependencies are explicitly pinned.

, is there anything that would need to change with this cc update?

[onur-ozkan]

Cc @onur-ozkan since you authored the comment at

rust/src/bootstrap/Cargo.toml

Lines 37 to 39 in a1fd235

	# Most of the time updating these dependencies requires modifications to the
	# bootstrap codebase(e.g., https://github.com/rust-lang/rust/issues/124565);
	# otherwise, some targets will fail. That's why these dependencies are explicitly pinned.

, is there anything that would need to change with this cc update?

It's never clear tbh. We need to try and see if we need to make a change on bootstrap :)

[Kobzol]

So let's just merge and see what breaks? Do you want to wait for some confirmation from soneone @tgross35?

[tgross35]

Oh no I just wanted to make sure I wasn't missing something, this is ready whenever you are :)

[Kobzol]

Ok, let's try.

@bors r+ rollup=never

[bors]

📌 Commit 3fb21f3 has been approved by Kobzol

It is now in the queue for this repository.

[bors]

⌛ Testing commit 3fb21f3 with merge 63a0bdd...

[bors]

☀️ Test successful - checks-actions
Approved by: Kobzol
Pushing 63a0bdd to master...

[rust-timer]

Finished benchmarking commit (63a0bdd): comparison URL.

Overall result: ❌ regressions - no action needed

@rustbot label: -perf-regression

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

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

Max RSS (memory usage)

Results (primary 1.0%, secondary -3.6%)

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)	1.0%	[1.0%, 1.0%]	1
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-3.6%	[-3.6%, -3.6%]	1
All ❌✅ (primary)	1.0%	[1.0%, 1.0%]	1

Cycles

Results (secondary -1.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
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-1.2%	[-1.2%, -1.2%]	1
All ❌✅ (primary)	-	-	0

Binary size

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

Bootstrap: 768.953s -> 768.228s (-0.09%)
Artifact size: 341.42 MiB -> 341.42 MiB (-0.00%)