Rebuild docs for rephrase --manifest-path.

Reprhase --manifest-path section

Change the order of the words, for readability.

Auto merge of #7536 - ehuss:profile-override-warning-fix, r=alexcrichton

Fix profile override warning in a workspace.

Profile overrides would erroneously warn about unused packages in a workspace if the package was not being built.  The fix here is to retain the `Resolve` for the entire workspace, and use that to determine the valid set of packages.

I think it would be good for a long-term goal to get rid of the second ("targeted") resolve.  I'm still contemplating how a separate features resolver could achieve that, but it seems feasible long-term.

Closes #7378

Auto merge of #7534 - ehuss:fix-windows-nightly, r=alexcrichton

Fix some tests failing on Windows nightly.

- Windows MSVC binaries don't have metadata hash anymore.
- Properly escape backslashes in config TOML.

Auto merge of #7535 - ehuss:better-windows-errors, r=alexcrichton

Show better error message for Windows abnormal termination.

This helps display better error messages when there is an abnormal termination on Windows.

If rustc crashes, there was a slight mistake in #6270, where the error code was actually negative, so the message was getting hidden behind the `--verbose` flag.  This changes it show that the abnormal error is always shown if rustc crashes in an unusual way.

This also changes `cargo test` to display a better message if the test crashes.  Previously:

```
running 1 test
error: test failed, to rerun pass '--bin crash'
```

New:

```
running 1 test
error: test failed, to rerun pass '--bin crash'

Caused by:
  process didn't exit successfully: `D:\Temp\crash\target\debug\deps\crash-b3c2389529da3d3e.exe` (exit code: 0xc0000374, STATUS_HEAP_CORRUPTION)
```

I didn't add any tests because testing on this on Windows seems a little precarious.  AFAIK, the exact error message depends on the processor (like x86 vs i686), and Windows version.  I was concerned about chasing down every little nuance, though I'm willing to try if it seems important.

Fixes rust-lang/rust#65692 (I think).

Auto merge of #7541 - alexcrichton:try-apt-update, r=alexcrichton

Run `apt update` before `apt install`

Trying to fix recent errors on CI...

Run `apt update` before `apt install`

Trying to fix recent errors on CI...

Fix profile override warning in a workspace.

Show better error message for Windows abnormal termination.

Fix some tests failing on Windows nightly.

Auto merge of #7532 - dario23:install-cmd-help-typo, r=alexcrichton

Fix typo in `cargo install --profile` help

Fix typo in `cargo install --profile` help

Auto merge of #7531 - ehuss:z-flag-parsing, r=alexcrichton

Use stricter -Z flag parsing.

This makes sure that toggle flags don't have any arguments (like `-Zno-index-update=foo`).  It also provides slightly better error messages, like indicating which flag failed.

Use stricter -Z flag parsing.

Auto merge of #7523 - bruceg:fix-zero-timestamps, r=alexcrichton

Set timestamp on generated files in archive to now

When generating files (Cargo.lock, Cargo.toml, and
.cargo_vcs_info.json), cargo neglected to set any timestamp on the file
in the archive. This results in them being created on disk with a
timestamp of 0 (Jan 1 1970 GMT) which is confusing another tool I use.

This patch alters the behavior to set the mtime to now.

Signed-off-by: Bruce Guenter <bruce@untroubled.org>

Auto merge of #7460 - alexcrichton:panic-abort-tests, r=ehuss

Support rustc's `-Z panic-abort-tests` in Cargo

Recently added in rust-lang/rust#64158 the `-Z panic-abort-tests` flag
to the compiler itself will activate a mode in the `test` crate which
enables running tests even if they're compiled with `panic=abort`.  It
effectively runs a test-per-process.

This commit brings the same support to Cargo, adding a `-Z
panic-abort-tests` flag to Cargo which allows building tests in
`panic=abort` mode. While I wanted to be sure to add support for this in
Cargo before we stabilize the flag in `rustc`, I don't actually know how
we're going to stabilize this here. Today Cargo will automatically
switch test targets to `panic=unwind`, and so if we actually were to
stabilize this flag then this configuration would break:

    [profile.dev]
    panic = 'abort'

In that case tests would be compiled with `panic=unwind` (due to how
profiles work today) which would clash with crates also being compiled
with `panic=abort`. I'm hopeful though that we can perhaps either figure
out a solution for this and maybe even integrate it with the ongoing
profiles work.

Fix broken tests on nightly

Inherit `panic` settings in test/bench profiles

We've always ignored the `panic` settings configured in test/bench
profiles, so this just continues to ignore them but in a slightly
different way.

Support rustc's `-Z panic-abort-tests` in Cargo

Recently added in rust-lang/rust#64158 the `-Z panic-abort-tests` flag
to the compiler itself will activate a mode in the `test` crate which
enables running tests even if they're compiled with `panic=abort`.  It
effectively runs a test-per-process.

This commit brings the same support to Cargo, adding a `-Z
panic-abort-tests` flag to Cargo which allows building tests in
`panic=abort` mode. While I wanted to be sure to add support for this in
Cargo before we stabilize the flag in `rustc`, I don't actually know how
we're going to stabilize this here. Today Cargo will automatically
switch test targets to `panic=unwind`, and so if we actually were to
stabilize this flag then this configuration would break:

    [profile.dev]
    panic = 'abort'

In that case tests would be compiled with `panic=unwind` (due to how
profiles work today) which would clash with crates also being compiled
with `panic=abort`. I'm hopeful though that we can perhaps either figure
out a solution for this and maybe even integrate it with the ongoing
profiles work.

Auto merge of #7526 - ehuss:rustfmt, r=alexcrichton

rustfmt for nightly changes.

Something in the latest nightly versions of rustfmt has slightly changed the logic here. Fortunately stable ignores the new formatting, and it looks better to me anyways.  I tend to always use nightly, so this helps me with a slight annoyance where this line keeps getting changed.

Wrap some really long lines.

These do not wrap due to https://github.com/rust-lang/rustfmt/issues/3863.

rustfmt for nightly changes.

Auto merge of #7525 - ehuss:allow-all-features-virtual-ws, r=alexcrichton

Allow --all-features in root of virtual workspace.

Accidentally restricted in #7507, pointed out by @hjmallon, `--all-features` flag is allowed in the root of a virtual workspace.  Added a test to demonstrate its strange behavior.

For anyone that is curious, [this line](https://github.com/rust-lang/cargo/blob/3a9abe3f065554a7fbc59f440df2baba4a6e47ee/src/cargo/ops/resolve.rs#L302) is the code responsible for this behavior.

Allow --all-features in root of virtual workspace.

Refactor the common code for generating an archive member

Signed-off-by: Bruce Guenter <bruce@untroubled.org>

Set timestamp on generated files in archive to now

When generating files (Cargo.lock, Cargo.toml, and
.cargo_vcs_info.json), cargo neglected to set any timestamp on the file
in the archive. This results in them being created on disk with a
timestamp of 0 (Jan 1 1970 GMT) which is confusing another tool I use.

This patch alters the behavior to set the mtime to now.

Signed-off-by: Bruce Guenter <bruce@untroubled.org>

Auto merge of #7512 - Xyene:patch-1, r=alexcrichton

Fix typo in git index initialization error path

This PR proposes a fix for a small typo I ran into while using `cargo`.

Auto merge of #7507 - ehuss:virtual-ws-features, r=alexcrichton

Reject feature flags in a virtual workspace.

This generates an error if feature flags are used in the root of a virtual workspace. Previously these flags were completely ignored.  In the interest of avoiding confusion, I think it would be good to be explicit that these don't currently work.  This could alternatively be a warning, but I think it is better to reject it outright.

cc #4753, #3620, #5015, #6195, etc.

Auto merge of #7504 - ehuss:rename-package-overrides, r=alexcrichton

Rename `overrides` to `package` in profiles.

Renaming per the discussion at https://github.com/rust-lang/rust/issues/48683#issuecomment-488842694.

I left backwards-compatibility in case anyone is using it, since it required very little effort.

cc @da-x, FYI.

Auto merge of #7333 - ehuss:allow-dev-dep-path, r=alexcrichton

Allow publishing with dev-dependencies without a version.

This change allows dev-dependencies without a `version` key to be published.  If a dev-dependency is missing the `version`, it will be stripped from the packaged manifest.

Auto merge of #7450 - ehuss:stabilize-cache-messages, r=alexcrichton

Stabilize cache-messages

This stabilizes the -Zcache-messages feature, making it always enabled.

## What is stabilized?

This feature is intended to redisplay previous warnings on a "fresh" build instead of displaying no output.

Users have occasionally indicated frustration when they know there are warnings, but no output is displayed when the build is fresh. This also improves the interaction between `cargo check` and `cargo clippy-preview`. This also simplifies the code, and opens more possibilities for `rustc` to send side-channel messages to Cargo.

Cargo will now use JSON output from `rustc` and `rustdoc` 100% of the time (`rustdoc --test` does not use JSON). Previously Cargo would only use JSON for pipelined crates.

Cargo will save the JSON output into a file named `output` in the `.fingerprint` directory. This file is only created when the compiler outputs a diagnostic message.

If a crate is being recompiled, and Cargo considers it to be "fresh", it will replay the output file to the console.

## Notable changes in this PR

- Fixed a bug where replays were erroneously including pipeline rmeta artifact json messages.
- clippy-preview is now included in the metadata hash, to force its artifacts to be separate from `cargo check`.
- clippy-preview is no longer force-enabled, under the assumption that caching and fingerprinting is accurate, and the cached messages will be replayed.
- clippy-preview's arguments are included in the fingerprint now that it is not force-enabled.
- Rustdoc colors and short messages were fixed when pipelining was stabilized, so updated tests.

Closes #6986
Closes #6848
Closes #6664
Closes #2458

## Concerns

The only notable issue with this is that switching between short and long human messages only replays the format from the first invocation.  That is, if you do `cargo build` and it generates warnings, then running again with `--message-format=short` will still show the full length human messages. I'm personally fine with that behavior, even though it is not ideal. I think this feature overall improves the situation (where before *no* output was displayed). Being able to re-render between short/long is a very difficult problem, and unlikely to be fixable in the foreseeable future.

There was some concern expressed about being able to disable this. I think that would only be necessary if a severe bug is discovered. I do not feel that this change is risky enough to warrant a configurable option. If it does cause a problem, it can be quickly reverted with a one-line change to set `OutputOptions::cache_cell` to `None`. Since pipelining has been using JSON output for a while now without complaints, I feel pretty confident in it.

Fix typo in git index initialization error path

Reject feature flags in a virtual workspace.

Rename `overrides` to `package` in profiles.

Auto merge of #7502 - matthiaskrgr:dont_lock_target_dir_7490, r=ehuss

don't lock the package cache when cleaning target dir.

Fixes #7490

don't lock the package cache when cleaning target dir.

Fixes #7490

Auto merge of #7499 - orium:document-rustc-wrapper, r=Eh2406

Document rustc wrapper

Document the `rustc-wrapper` configuration.

`Config::get_tool()` now also accepts the configuration path in idiomatic kebab-case.

Auto merge of #7456 - alexcrichton:config-only-serde, r=ehuss

Migrate towards exclusively using serde for `Config`

This series of commits was spawned off a thought I had while reading https://github.com/rust-lang/cargo/issues/7253#issuecomment-535656059, although it ended up not really touching on that at all. I was a little unsettled about how unstructured the config accesses are throughout Cargo and we had sort of two systems (one serde which is nice, one which is more manual) for reading config values.

This PR converts everything to run through serde for deserializing values, except for `get_list` which is funky. There's only one usage of that with the `paths` key though and we can probably fix this soon-ish.

In any case, the highlights of this PR are:

* This PR is surprisingly large. I did a lot of movement in `config.rs` to try to make the file smaller and more understandable.
* The `Value` type which retains information about where it was deserialized from is very special, and has special treatment with serde's data model. That's what allows us to use that and serde at the same time.
* The `ConfigRelativePath` and `ConfigKey` structures have been revamped internally, but morally serve the same purposes as before.
* Cargo now has structured `struct` access for a bunch of its configuration (`net`, `http`, `build`, etc). I would ideally like to move toward a world where this is the *only* way to read configuration, or at least everything conventionally runs through those paths.
* Functionally, this PR should have no difference other than tweaks to error messages here and there, and perhaps more strict validation on commands where we validate more configuration on each run than we previously did.
* This isn't a 100% transition for Cargo yet, but I figured it would be a good idea to post this and get some feedback first.
* In the long run I want to remove `get_env`, `get_cv`, and `get_*_priv` from `Config` as internal details. I'd like to move this all to `de.rs` and have it walk down the tree of configuration as we deserialize a value. For now though these all remain in place and that refactoring is left to a future PR.

More review comments

Auto merge of #7492 - ehuss:enable-msvc-tests, r=alexcrichton

Re-enable some MSVC tests.

These tests were disabled for MSVC in #7400, but I would prefer to keep them running.

Auto merge of #7489 - da-x:fix-7488, r=ehuss

when -Z unstable-options not specified, don't validate --profile

This fixes a regression caused by 8b0561d68f12 - Closes #7488.

Re-enable some MSVC tests.

Rustfmt lint

tests: cover issue #7488

Auto merge of #7470 - alexcrichton:cyclic-error, r=Eh2406

Improve error message for cyclic dependencies

First reported in rust-lang/rust#65014 it looks like our error message
on cyclic dependencies may be confusing at times. It looks like this is
an issue because there are multiple paths through a graph for a
dependency, so using the generic `path_to_top` function isn't producing
the most useful path for this purpose.

We're already walking the graph though, so this commit adds an extra
parameter which collects the list of packages we've visited so far to
produce a hopefully always-accurate error message showing the chain of
dependencies end-to-end for what depends on what.

Add comments of `Value<T>`'s deserialization protocol

Document the new `ConfigKey` implementation

Remove unnecessary clone

Add documentation for `StringList`

Use `#[serde(rename_all)]` instead of listing them all

Run rustfmt

Handle rebase conflicts

Simplify `cargo-new` configuration reading

No need for lots of extra helpers/parsing when using serde!

Consolidate `build` key configuration

Add a typed structure which lists all `build` key configuration
throughout Cargo.

Remove dead `Config::get_i64` method

Remove no-longer-needed `Config::get_bool`

Consolidate `net` configuration into a typed structure

Less need for `get_bool` and friends!

Access `term` config through a deserialized type

Going through and removing users of raw `get_*` functions!

Centralize HTTP configuration in one struct

Gives us one nice place to access and document all HTTP-related configuration

Remove a usage of `get_list`

This callsite doesn't need the full power of `get_list`, knowing the
definition path of each element along the list.

Finish implementing `Value`, use it in helpers

Rewrite helpers like `get_bool` to use `get::<Option<Value<bool>>>`
instead of duplicating the logic that's already with the typed access of
configuration. This is more along the effort to centralize all
deserialization of configuration into typed values instead of using
ad-hoc accessors in a number of locations.

Refactor `ConfigKey` to its own file

Also make it a little less allocation-heavy by tweaking the API to
encourage incremental building of the key and incremental destruction as
we walk throughout the configuration tree.

Extra serde config support to a separate file

Move `config.rs` to `config/mod.rs`

Improve error message for cyclic dependencies

First reported in rust-lang/rust#65014 it looks like our error message
on cyclic dependencies may be confusing at times. It looks like this is
an issue because there are multiple paths through a graph for a
dependency, so using the generic `path_to_top` function isn't producing
the most useful path for this purpose.

We're already walking the graph though, so this commit adds an extra
parameter which collects the list of packages we've visited so far to
produce a hopefully always-accurate error message showing the chain of
dependencies end-to-end for what depends on what.

Auto merge of #7484 - ehuss:clippy-fixes, r=alexcrichton

Some minor clippy fixes.

named-profiles: when -Z unstable-options not specified, don't validate --profile

Fixes #7488.

Some minor clippy fixes.

Auto merge of #7481 - Lythenas:doc-spdx-list-3.6, r=ehuss

Update SPDX list to 3.6

Fixes #7477

Auto merge of #7482 - alexcrichton:fix-bin, r=ehuss

Fix wrong directories in PATH on Windows

This fixes an accidental regression from #7425 where `PATH` was being
augmented on Windows with the wrong search path for target/host
libraries. This commit fixes the issue by simply always calculating the
host/target library paths for `TargetInfo`, and then we explicitly use
the same `TargetInfo` for filling out information in `Compilation`.

Closes #7475

Fix wrong directories in PATH on Windows

This fixes an accidental regression from #7425 where `PATH` was being
augmented on Windows with the wrong search path for target/host
libraries. This commit fixes the issue by simply always calculating the
host/target library paths for `TargetInfo`, and then we explicitly use
the same `TargetInfo` for filling out information in `Compilation`.

Closes #7475

Auto merge of #7476 - tlively:emscripten-wasm-aux, r=alexcrichton

Mark Emscripten's .wasm files auxiliary

This fixes #7471 and fixes #7255 by preventing the .wasm file from
being treated as an executable binary, so `cargo test` and `cargo run`
will no longer try to execute it directly. This change is only made
for Emscripten, which outputs a .js file as the primary executable
entry point, as opposed to other WebAssembly targets for which the
.wasm file is the only output.

Update SPDX list to 3.6

Drop unnecessary target check

rustfmt, even if I disagree

Mark Emscripten's .wasm files auxiliary

This fixes #7471 and fixes #7255 by preventing the .wasm file from
being treated as an executable binary, so `cargo test` and `cargo run`
will no longer try to execute it directly. This change is only made
for emscripten, which outputs a .js file as the primary executable
entry point, as opposed to other WebAssembly targets for which the
.wasm file is the only output.

Auto merge of #7464 - alexcrichton:update-curl-sys, r=ehuss

Update `curl-sys` dependency requirement

Pulls in alexcrichton/curl-rust#304 which fixes a bug from the last curl
update in #7308. This bug was not introduced by the Cargo PR itself but
rather by updating the `curl` submodule in the `curl-sys` crate. Without
this bugfix all downloads of a crate will make a new connection to
crates.io, which drastically increases download time since setting up a
connection takes so long.

Auto merge of #7443 - guanqun:patch-1, r=ehuss

add dependencies for `pkg-config`

Otherwise, it complains:

    run pkg_config fail: "Failed to run `\"pkg-config\" \"--libs\" \"--cflags\" \"openssl\"`: No such file or directory (os error 2)"

Auto merge of #7400 - snf:fix_7358, r=alexcrichton

Removing hash from output files when using MSVC

This is the fix for https://github.com/rust-lang/cargo/issues/7358 suggested by @alexcrichton . Tested and working.

I see a few tests failling but seem unrelated. I'll investigate them tomorrow.

Auto merge of #7465 - alexcrichton:no-preserve, r=Eh2406

Disable preserving mtimes on archives

These are just wasted syscalls for our purposes, no need to issue
updates to the modification/creation/access times of files we unpack!

removing hash from output files when using MSVC

Disable preserving mtimes on archives

These are just wasted syscalls for our purposes, no need to issue
updates to the modification/creation/access times of files we unpack!

Update `curl-sys` dependency requirement

Pulls in alexcrichton/curl-rust#304 which fixes a bug from the last curl
update in #7308. This bug was not introduced by the Cargo PR itself but
rather by updating the `curl` submodule in the `curl-sys` crate. Without
this bugfix all downloads of a crate will make a new connection to
crates.io, which drastically increases download time since setting up a
connection takes so long.

Auto merge of #7462 - hbina:redundant_borrow, r=ehuss

Removed redundant borrow

As described here https://rust-lang.github.io/rust-clippy/master/#needless_borrow.
rust-clippy is complaining that the related borrows are unnecessary.

Auto merge of #7361 - Eh2406:public_dependency-as-type_4, r=alexcrichton

Public dependency refactor and re-allow backjumping

There were **three** attempts at vanquishing exponential time spent in Public dependency resolution. All failures. All three started with some refactoring that seams worth saving. Specifically the data structure `public_dependency` that is used to test for Public dependency conflicts is large, tricky, and modified in line. So lets make it a type with a name and move the interactions into methods.

Next each attempt needed to know how far back to jump to undo any given dependency edge. I am fairly confident that any full solution will need this functionality. I also think any solution will need a way to represent richer conflicts than the existing "is this pid active". So let's keep the `still_applies` structure from the last attempt.

Last each attempt needs to pick a way to represent a Public dependency conflict. The last attempt used three facts about a situation.

- `a1`: `PublicDependency(p)` witch can be read as the package `p` can see the package `a1`
- `b`: `PublicDependency(p)` witch can be read as the package `p` can see the package `b`
- `a2`: `PubliclyExports(b)` witch can be read as the package `b` has the package `a2` in its publick interface.

This representation is good enough to allow for `backjumping`. I.E. `find_candidate` can go back several frames until the `age` when the Public dependency conflict was introduced. This optimization, added for normal dependencies in #4834, saves the most time in practice. So having it for Public dependency conflicts is important for allowing real world experimentation of the Public dependencies feature.  We will have to alter/improve/replace this representation to unlock all of the important optimizations. But I don't know one that will work for all of them and this is a major step forward.

Can be read one commit at a time.

Removed redundant borrow

As described here https://rust-lang.github.io/rust-clippy/master/#needless_borrow.
rust-clippy is complaining that the related borrows are unnecessary.

Auto merge of #7461 - guanqun:unify-quotes-in-cargo-toml, r=alexcrichton

unify the quote in Cargo.toml

A bit nit-picked...

It tries to use double-quotes in Cargo.toml instead of single quote, that would be more uniform in this file.

unify the quote in Cargo.toml

Stabilize cache-messages

Auto merge of #6989 - da-x:custom-profile-pr-rfc, r=ehuss

Support for named profiles (RFC 2678)

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

Implementation according to the [RFC](https://github.com/rust-lang/rfcs/blob/master/text/2678-named-custom-cargo-profiles.md).

Auto merge of #7308 - guanqun:add-ssl-version, r=alexcrichton

add an option to specify ssl version

Fixes #6684

add a bit more docs

run cargo fmt to pass the CI build

Auto merge of #7417 - alexcrichton:less-hashing, r=Eh2406

Go back to not hashing `RUSTFLAGS` in `-Cmetadata`

This is a moral revert of #6503 but not a literal code revert. This
switches Cargo's behavior to avoid hashing compiler flags into
`-Cmetadata` since we've now had multiple requests of excluding flags
from the `-Cmetadata` hash: usage of `--remap-path-prefix` and PGO
options. These options should only affect how the compiler is
invoked/compiled and not radical changes such as symbol names, but
symbol names are changed based on `-Cmetadata`. Instead Cargo will still
track these flags internally, but only for reinvoking rustc, and not for
caching separately based on rustc flags.

Closes #7416

fixes

modify to enum/struct and add test for SslVersionConfig

named-profiles: fix backward compatibility for `rustc` with `--profile=check`

And a small cleanup in `base_profile` to make the logic parallel to the
one in `get_profile`.

add a test to show the new performance