Update cargo

[arlosi]

8 commits in 1b15556767f4b78a64e868eedf4073c423f02b93..7ac9416d82cd4fc5e707c9ec3574d22dff6466e5
2023-07-18 14:44:47 +0000 to 2023-07-24 14:29:38 +0000

• fix(cargo-credential): should enable feature serde/derive (fix(cargo-credential): should enable feature serde/derive cargo#12396)
• fix: encode URL params correctly for SourceId in Cargo.lock (fix: encode URL params correctly for SourceId in Cargo.lock cargo#12280)
• docs: format config override caveat as a note (docs: format config override caveat as a note cargo#12392)
• credential provider implementation (credential provider implementation cargo#12334)
• feat(crates-io): expose HTTP headers and Error type (feat(crates-io): expose HTTP headers and Error type cargo#12310)
• chore: Don't update test data (chore: Don't update test data cargo#12380)
• fix: only skip mtime check on ~/.cargo/{git,registry} (fix: only skip mtime check on ~/.cargo/{git,registry} cargo#12369)
• Update docs for artifact JSON debuginfo levels. (Update docs for artifact JSON debuginfo levels. cargo#12376)

Since rust-lang/cargo#12334 makes built-in credential providers part of the cargo binary, it's no longer needed to build them in bootstrap.

[rustbot]

r? @albertlarsan68

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

[rustbot]

⚠️ Warning ⚠️

• These commits modify submodules.

[rustbot]

Some changes occurred in src/tools/cargo

cc @ehuss

[albertlarsan68]

r=me on the bootstrap changes

[ehuss]

Thanks!

I think the lines at

rust/src/bootstrap/dist.rs

Lines 1088 to 1093 in fc8a3e3

	for dirent in fs::read_dir(cargo.parent().unwrap()).expect("read_dir") {
	let dirent = dirent.expect("read dir entry");
	if dirent.file_name().to_str().expect("utf8").starts_with("cargo-credential-") {
	tarball.add_file(&dirent.path(), "libexec", 0o755);
	}
	}

also need to be removed. It's not critical since it would not generate an error or cause problems since it just skips over the entries, but it would become dead code after this change.

[ehuss]

Thanks!

@bors r+ p=1

[bors]

📌 Commit ecfac68 has been approved by ehuss

It is now in the queue for this repository.

[ehuss]

cc @weihanglo Just verifying you saw this.

[bors]

⌛ Testing commit ecfac68 with merge 864bdf7...

[bors]

☀️ Test successful - checks-actions
Approved by: ehuss
Pushing 864bdf7 to master...

[rust-timer]

Finished benchmarking commit (864bdf7): 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.0%	[1.0%, 1.0%]	1
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-	-	0

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)	-2.5%	[-2.5%, -2.5%]	1
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-2.5%	[-2.5%, -2.5%]	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: 651.189s -> 651.472s (0.04%)