Don't use RUSTC_BOOTSTRAP for nightly-only tests

Merge remote-tracking branch 'upstream/master' into z-allowed-features

Auto merge of #9316 - ehuss:fix-semver-1.51, r=alexcrichton

Fix semver docs for 1.51.

1.51 slightly changed the text of a few error messages. This should get CI passing again.

Fix semver docs for 1.51.

Fix after merge

Merge remote-tracking branch 'upstream/master' into z-allowed-features

Future-proof tests

Print features without Debug

Handle allow-features in add()

Sort feature list

Move from match guard to if

Include allowed features in fingerprint

If this weren't the case, a user that compiled, and then re-ran cargo
with a particular set of allowed features would not see any breakage
even if other unstable features were used.

The downside of this is that changing allow-features causes a full
re-compile, but that still _seems_ like the right thing to do.

Forward -Zallow-features inside mod compiler

util: Add iterator join method

Auto merge of #9302 - ehuss:cargo-config, r=alexcrichton

Add `cargo config` subcommand.

This adds an initial version of the `cargo config` command as discussed in #2362.

Closes #2362

Print environment note for json format, too.

Auto merge of #9313 - matklad:feat-doc, r=ehuss

Give one more example for the --featuers CLI

The previous example confused me because I thought that `foo-feat` is
the syntax for `foo/feat`. The "explicit" example should clarify that.

closes #9305

Give one more example for the --featuers CLI

The previous example confused me because I thought that `foo-feat` is
the syntax for `foo/feat`. The "explicit" example should clarify that.

closes #9305

Remove duplicate code.

Auto merge of #9308 - ehuss:version-bump, r=alexcrichton

Bump to 0.54.0, update changelog

Auto merge of #9309 - sciguy16:out_dir_issue_url, r=ehuss

Make the URL to the tracking issue for `--out-dir` into a link

Make the URL to the tracking issue for `--out-dir` into a link

Update changelog for 1.52

Bump to 0.54.0

Auto merge of #9307 - ijackson:exit-code-string, r=joshtriplett

tests: Tolerate "exit status" in error messages

"exit code" is wrong terminology on Unix.  I am trying to fix this in Rust stdlib in https://github.com/rust-lang/rust/pull/83462 but this currently breaks the cargo test suite.

See that MR for full explanation of the change.

tests: Tolerate "exit status" in error messages

"exit code" is wrong terminology on Unix.  I am trying to fix this in
Rust stdlib in
    https://github.com/rust-lang/rust/pull/83462
but this currently breaks the cargo test suite.

See that MR for full explanation of the change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Auto merge of #9298 - alexcrichton:mac-split-debuginfo-default, r=ehuss

Default macOS targets to `unpacked` debuginfo

This commit continues the work from #9112 to enable `unpacked` split
debuginfo on macOS targets by default. This has been discussed on [internals]
for awhile now and no breakage has emerged while significant speedups
have. This is expected to be a compile-time and `target`-directory size
win for almost all macOS Rust projects.

While breakage is possible it's possible to mitigate this with
project-local or global cargo configuration of the `dev` and `test` profiles.

[internals]: https://internals.rust-lang.org/t/help-test-faster-incremental-debug-macos-builds-on-nightly/14016/9

Default macOS targets to `unpacked` debuginfo

This commit continues the work from #9112 to enable `unpacked` split
debuginfo on macOS targets by default. This has been discussed on [internals]
for awhile now and no breakage has emerged while significant speedups
have. This is expected to be a compile-time and `target`-directory size
win for almost all macOS Rust projects.

While breakage is possible it's possible to mitigate this with
project-local or global cargo configuration of the `dev` and `test` profiles.

[internals]: https://internals.rust-lang.org/t/help-test-faster-incremental-debug-macos-builds-on-nightly/14016/9

Add `cargo config` subcommand.

Auto merge of #9300 - alexcrichton:fix-publish-metadata, r=ehuss

Fix publication of packages with metadata and resolver

This commit fixes an issue where packages which specify `resolver = '2'`
cannot be packaged if they also have a `package.metadata` table. The
issue is that the `toml` serialization implementation serializes fields
in order which requires that tables be emitted last.

Move script to bottom of page.

Fix publication of packages with metadata and resolver

This commit fixes an issue where packages which specify `resolver = '2'`
cannot be packaged if they also have a `package.metadata` table. The
issue is that the `toml` serialization implementation serializes fields
in order which requires that tables be emitted last.

Auto merge of #9299 - ehuss:fix-config-include, r=alexcrichton

Fix config includes not working.

The config-include feature wasn't working because the config values were being loaded before the call to `configure`, so the unstable flag was always false.

I had to remove the unstable warning because it was a bit awkward to support, and I figure it's not that important.

Fix config includes not working.

Forward -Zallow-features to rustc

Test that allowed-features in config works

Auto merge of #9263 - Aaron1011:zero-incompat-note, r=ehuss

Emit note when `--future-incompat-report` had nothing to report

Auto merge of #9282 - lf-:remove-author, r=ehuss

RFC 3052: Stop including authors field in manifests made by cargo new

See https://github.com/rust-lang/rust/issues/83227

~~This is definitely a draft as there are a couple of tests I'm still working on fixing.~~ One question I ran into while digging through these test failures is that there is a Cargo config option for author name, `cargo-new.name`. Should we keep supporting that? I feel like perhaps we should as the user has specified explicit intent that they want their name in the authors of a manifest as generated by `cargo new`, but it seems weird to support *just* that case. There is likewise an environment variable `CARGO_NAME` that signals similar intent.

Should we completely drop the feature of putting in author names, and require people to manually put them in if they wish, or allow these limited cases where the user is specifically instructing *cargo* to do it?

Test the generated manifest does not contain authors =

Delete uses of the USER env var from tests

Remove every unnecessary appearance of the authors field in docs

Restore the tests that could be reused

Finish documentation changes

Update docs for default absent authors field

Delete redundant broken tests

RFC 3052: Stop including authors field in manifests made by cargo new

See https://github.com/rust-lang/rust/issues/83227

Auto merge of #9290 - ehuss:non-member-features, r=alexcrichton

Refactor feature handling, and improve error messages.

This changes the way feature strings are handled with an eye towards fixing some improper handling and to improve error messages. The key change is to stop treating all features as free-form strings and instead try to handle them as typed values. This helps avoid needing to deal with parsing different feature syntax (like `dep:` or `foo/bar`) or forgetting to handle it properly.

Overview of refactoring changes:

* `RequestedFeatures` changed to an enum to differentiate between features coming from the command-line, and those that are from a dependency.
* Moved parsing of CLI features to an earlier stage (now stored in `CompileOptions`), and ensures that they are properly handled as `FeatureValue` instead of strings.
* Pushed some feature validation earlier. For example, `DetailedTomlDependency` now validates things so you can see the location for the errant `Cargo.toml` (previously some validation was deep in the resolver, which provided poor errors).

This is a pretty large PR, but at the core it is just changing `RequestedFeatures` and then dealing with the fallout from that. Hopefully this is an improvement overall.

List of user-visible changes:

* Fix handling in resolver V2 of `--features bar?/feat` and `--features dep:bar`
* Better error handling for `bar/feat` and `dep:bar` in dependency declarations.
* Feature keys in the `[features]` table can no longer contain slashes.
* Fixed a minor issue with `cargo tree -e features --all-features -Z namespaced-features`
* Fixed a panic with `cargo tree` involving `-Z weak-dep-features`

I did a small amount of benchmarking, and I wasn't able to record much of a difference.

Auto merge of #9292 - ehuss:cargo-util, r=alexcrichton

Split out cargo-util package for cargo-test-support.

This splits out code from `cargo` that was being used by `cargo-test-support` in order to remove the dev-dependency cycle on the `cargo` crate. The intent here is to improve development build times. On my system, `touch src/cargo/lib.rs ; cargo check --profile=test` goes from 6.4 seconds to 3.5. I suspect more substantial changes (more than just `touch`) should exhibit even more improvements.

This has a substantial downside that it is another package to manage publishing, particularly with making sure the version is updated correctly for crates.io. I'm uncertain if on balance it is worth it, but I lean towards moving forward with it.

Auto merge of #9293 - ehuss:unnecessary-semi-resolver-tests, r=alexcrichton

Fix redundant_semicolons warning in resolver-tests.

Fix redundant_semicolons warning in resolver-tests.

Remove the dev-dependency cycle on cargo.

Remove cargo:: link.

Remove use of CRATES_IO_INDEX in cargo-test-support.

Unfortunate duplication of the string, but I think it is unlikely
to have any consequences.

Remove CargoResult from cargo-test-support.

Remove use of Rustc from cargo-test-support.

cargo-test-support wasn't using any of the caching or other logic from
Rustc, so this just swaps with a very basic implementation in order to
remove the dependency on `cargo`.

Move Sha256 to cargo-util.

Move `paths` to cargo-util.

Move is_ci to cargo-util.

Move ProcessBuilder to cargo-util.

Split util code into a separate package.

Auto merge of #9237 - yaymukund:serde-expecting, r=ehuss

Use serde's error message option to avoid implementing `Deserialize`.

This is a cleanup based on https://github.com/serde-rs/serde/issues/1883, which fell out of https://github.com/rust-lang/cargo/pull/8664

It looks like this changes the resulting error messages. I'll leave it up to you to decide if the tradeoff makes sense here. Maybe the correct answer here is to make `serde`'s error messages include more details (e.g. `invalid type: int`).

Use serde's error message option to avoid implementing `Deserialize`.

Refactor feature handling, and improve error messages.

Import anyhow::bail

Auto merge of #9279 - Revantus:offline_precise, r=alexcrichton

Allow `cargo update` to operate with the --offline flag

Closes #9248

Limiting to `--precise` operations as the package should be cached locally prior to the usage of offline.

Added an additional test to [offline.rs](tests/testsuite/offline.rs).

Fix offline tests. Rename to cached/not_cached.

Remove eager fail for update --offline

Auto merge of #9285 - rusty-snake:patch-1, r=ehuss

Fix typo in faq.md

Fix typo in faq.md

Add -Zallow-features to match rustc's -Z

Auto merge of #9268 - ehuss:2021-fix-edition-v2-warning, r=alexcrichton

Add report if `cargo fix --edition` changes features.

This adds a small report if using `cargo fix --edition` to transition from 2018 to 2021 to determine if the new resolver will result in different features.

There is a gauntlet of checks that must pass before it even tries to show the differences:
* If the resolver is already specified, skip.
* If in a virtual workspace, skip (no way to specify global edition).
* If not migrating the root package, skip. The edition only changes the resolver in the root package.
* If not migrating all targets, skip. It's uncertain if the user intends to set the package edition or not.

Closes #9048

Auto merge of #9275 - ehuss:features-non-member, r=alexcrichton

Fix --feature pkg/feat for V1 resolver for non-member.

#8997 had an unintended regression where `-p foo --feature foo/feat` syntax where `foo` is an **optional non-member** fails with an error that `foo` did not match any packages.  The issue is that the member/feature selection routine needed to slot this into the features for the package in the current working directory (it was incorrectly treating `foo` as a workspace member).

V2 outright does not allow specifying features for non-workspace members.

Fixes #9265

Remove --all-targets restriction for v2 resolver migration check.

Add report if `cargo fix --edition` changes features.

Auto merge of #9276 - ehuss:fix-collision-root-removal, r=alexcrichton

Fix doc duplicate removal of root units.

The doc collision removal code didn't consider the possibility that there was a collision with a root unit.  This caused problems in conjunction with #9142 where cargo would panic because a root unit got removed from the graph because of a filename collision. The solution here is to avoid removing root units during the doc collision sweep.

This has a downside that this reintroduces a filename collision.

An alternate solution would be to make the set of root units mutable, and remove from that list, but I figured this is simpler and more conservative.

Fixes #9274

Fix doc duplicate removal of root units.

Auto merge of #9271 - jonhoo:patch-in-config-in-help, r=ehuss

Add CLI help text for patch-in-config

Tracking issue: #9269

Add CLI help text for patch-in-config

Auto merge of #9270 - jonhoo:document-patch-in-config, r=ehuss

Document `-Zpatch-in-config`

Tracking issue: https://github.com/rust-lang/cargo/issues/9269

Link tracking issue

Use feature name as heading

Document the patch-in-config feature

Auto merge of #9204 - jonhoo:patch-in-config, r=alexcrichton

Support [patch] in .cargo/config files

This patch adds support for `[patch]` sections in `.cargo/config.toml`
files. Patches from config files defer to `[patch]` in `Cargo.toml` if
both provide a patch for the same crate.

The current implementation merge config patches into the workspace
manifest patches. It's unclear if that's the right long-term plan, or
whether these patches should be stored separately (though likely still
in the manifest). Regardless, they _should_ likely continue to be
parsed when the manifest is parsed so that errors and such occur in the
same place regardless of where a patch is specified.

Fixes #5539.

Fixes from review

Auto merge of #9264 - Aaron1011:future-incompat-report-test-cmd, r=alexcrichton

Add `--future-incompat-report` support to `cargo test`

This was missed in the original PR

Emit note when `--future-incompat-report` had nothing to report

Fix --feature pkg/feat for V1 resolver for non-member.

Auto merge of #9262 - Matt-Gleich:master, r=Eh2406

🍱 Crop favicon

Hello! At first glance I found doc.rust-lang.org/cargo's favicon to be a bit small compared to other site's favicons. Looked into it and found that the logo hasn't been cropped. All I did was crop the favicon in this PR. As you can see from the image below (new favicon on the right) it looks much better:


This has been done to crates.io's favicon as well in rust-lang/crates.io#3411. Thanks, you maintainers! :)

Add `--future-incompat-report` support to `cargo test`

This was missed in the original PR

🍱 Crop favicon

Signed-off-by: Matthew Gleich <git@mattglei.ch>

Switching precise check from match to `is_none()`

Allowing update --offline with precise flags

Adding additional test for updating against changed offline
dependancies.

rust-lang/cargo#9248

Auto merge of #9252 - ehuss:prefer-dynamic-ws, r=alexcrichton

Fix logic for determining prefer-dynamic for a dylib.

The logic for determining if a dylib should use `prefer-dynamic` used to be something like "do not use prefer-dynamic if it is the current package".

The current logic has a strange behavior where it works as intended if there is only one package in the workspace, but a workspace with multiple packages will always use `prefer-dynamic`.

Instead of using `current_opt`, which isn't a good concept to use in a workspace, I switched this to be "primary" (a package selected on the command-line).

**History**
* 9879a0a5f621a4ae44b2d9dc62a0175404c05f8f Initial prefer-dynamic behavior.
* #3221 changed to the faulty logic (see comments at https://github.com/rust-lang/cargo/pull/3221#discussion_r88535221). I think there was some confusion there.
* #3478 fixed the logic for one of the changed conditions, but not the one for prefer-dynamic

Auto merge of #9255 - ehuss:fix-deps-filtering, r=Eh2406

Fix issue with filtering exclusive target dependencies.

#8777 incorrectly changed the filtering logic for dependencies.  Essentially it split `filter(any(A && B && C && D))` into two parts `filter(any(A && B)).filter(any(C && D))` which doesn't have the same meaning. The solution here is to pass a closure so that the conditions are joined again.

Fixes #9216

Fix issue with filtering exclusive target dependencies.

Fix logic for determining prefer-dynamic for a dylib.

Auto merge of #9249 - ehuss:doc-pkgid-spec, r=alexcrichton

Update pkgid-spec docs.

The old docs for the pkgid spec weren't correct. Specs never supported abbreviated URLs or aliases for the index (like `crates.io`).  This tries to correct that and add a little more detail.

Part of #7725.

Update pkgid-spec docs.

Auto merge of #9233 - alexcrichton:reword-edition, r=ehuss

Wordsmith the edition documentation a bit more

This hopes to clear up a few recent PRs about the `edition` key in the
manifest. It shuffles things around so the defaults are talked about in
two different paragraphs. The first one talks about how `cargo new` uses
the 2018 edition and is the only paragraph to use the word "default".
The second section talks about the usage of 2015 with no `edition` field
present.

This didn't end up being quite as clear as I hoped it might be, but I'm
hoping that this provides a bit more emphasis or otherwise just
restructures things a bit to be more clear.

Auto merge of #9188 - weihanglo:issue-9041, r=ehuss

Package ID specification urls must contain a host

Resolves https://github.com/rust-lang/cargo/issues/9041

Not sure which commit breaks this. Cargo shipped with rust 1.46 didn't unwrap on a `None` but 1.47 did. Even checkouted to 149022b1d8f382e69c1616f6a46b69ebf59e2dea (cargo of rust 1.46), it still unwrap unexpectedly.

So I ended up following the [Specification grammer](https://doc.rust-lang.org/cargo/reference/pkgid-spec.html#specification-grammar) to make sure there is a `host` in the pkgid urls.

<details>
<summary>See console output</summary>

cargo of rust 1.46
```console
$ cargo +1.46 -vV
cargo 1.46.0 (149022b1d 2020-07-17)
release: 1.46.0
commit-hash: 149022b1d8f382e69c1616f6a46b69ebf59e2dea
commit-date: 2020-07-17

$ cargo +1.46 pkgid /path
error: package ID specification `/path` matched no packages
```

cargo of rust 1.47

```console
$ cargo +1.47 -vV
cargo 1.47.0 (f3c7e066a 2020-08-28)
release: 1.47.0
commit-hash: f3c7e066ad66e05439cf8eab165a2de580b41aaf
commit-date: 2020-08-28

$ cargo +1.47 pkgid /path
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', src/tools/cargo/src/cargo/core/package_id_spec.rs:234:50
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
```

cargo on commit 149022b1d8f382e69c1616f6a46b69ebf59e2dea

```console
$ git checkout 149022b1d8f382e69c1616f6a46b69ebf59e2dea
$ cargo run -- pkgid /path
   Compiling cargo-platform v0.1.1 ([..]/cargo/crates/cargo-platform)
   Compiling crates-io v0.31.1 ([..]/cargo/crates/crates-io)
   Compiling cargo v0.47.0 ([..]/cargo)
    Finished dev [unoptimized + debuginfo] target(s) in 22.90s
     Running `target/debug/cargo pkgid /path`
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', src/cargo/core/package_id_spec.rs:234:50
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
```
</details>