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`

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

Some minor clippy fixes.

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

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

allow `find_candidate` to backtrack from a `PublicDependency`

change `PublicDependency` to store the `age` for each edge

change `age` to count edges not just activations

A pub dep conflict can be made by connecting two already activated pids

Some cleanups

move `public_dependency`s code to methods for better organization

Note: this commit does not change code, just moves it

named-profiles: add backward compatibility if feature is disabled

The effects over the profile used by targets are made conditional
in this commit, using the old scheme if the `named-profiles` feature
is disabled. This also affects the `profile_targets` tests, which
now have two modes - stable, and nightly with the feature enabled.

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

Bump to 0.41.0, Update changelog

Bump to 0.41.0, Update changelog

Auto merge of #7452 - pyrrho:bug7346/transitive_patches, r=Eh2406

Bug7346/transitive patches

Fixes #7346.

A cursory comparison between current stable and nightly shows that projects with this topology resolve similarly. If there are other behaviors I should test, I'd be happy to expand that section. This is a pretty focused change, though, so I'm not sure what else there is to break.

Sorry about the delay in putting this PR together. Good news is I know more than I did last week.

Auto merge of #7448 - ehuss:gitignore-lockfile, r=alexcrichton

Allow gitignore of Cargo.lock with explicit `include`.

If a package has an `include` list, but `Cargo.lock` is in `.gitignore`, then Cargo would complain that `Cargo.lock` is "dirty".  This changes it so that ignored `Cargo.lock` is allowed, even though it is still packaged.  This is under the presumption that `Cargo.lock` is machine generated, so it is not critical.  This was also an unexpected regression.

If you don't have an `include` list, then there is no complaint about `Cargo.lock` being dirty because Cargo uses git to deduce the file list, and `Cargo.lock` would be skipped for the dirty check (but still included in the package).

Closes #7319

Don't panic on a non-fatal error

Modify Context::flag_activated to return ActivateResult

Add a test to model the bug found in #7346

Allow gitignore of Cargo.lock with explicit `include`.

Auto merge of #7446 - alexcrichton:quiet-output, r=Eh2406

Improve test output with `--quiet`

We had a few locations where the shell was written to raw instead of
through the test harness or through other captured mechanisms. This
updates the test suite so testing Cargo with `--quiet` provides a nice
and clean report of tests executed.

Auto merge of #7445 - alexcrichton:more-stable, r=Eh2406

Allow a number of tests to run on stable

These tests all relied on support which has now ridden to stable at this
point, so let's let them run on stable!

Improve test output with `--quiet`

We had a few locations where the shell was written to raw instead of
through the test harness or through other captured mechanisms. This
updates the test suite so testing Cargo with `--quiet` provides a nice
and clean report of tests executed.

Allow a number of tests to run on stable

These tests all relied on support which has now ridden to stable at this
point, so let's let them run on stable!

Merge remote-tracking branch 'origin/master' into custom-profile-pr-rfc

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

Auto merge of #7425 - alexcrichton:kind-string, r=ehuss

Refactor `Kind` to carry target name in `Target`

This commit is an internal refactoring of Cargo's compilation backend to
eventually support compiling multiple target simultaneously. The
original motivation for this came up in discussion of #7297 and this has
long been something I've intended to update Cargo for. Nothing in the
backend currently exposes the ability to actually build multiple target
simultaneously, but this should have no function change with respect to
all current consumers. Eventually we'll need to refactor APIs of how you
enter the compilation backend to compile for multiple targets.

Auto merge of #7444 - matthiaskrgr:clippy_v8, r=Eh2406

 fix a bunch of clippy warnings

support http.ssl-version.min/max to specify min/max respectively

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)"

fix a bunch of clippy warnings

Address review about doc blocks

Refactor how compile targets are handled

Rename `Kind` to `CompileKind` to reflect that it's intended for
compilation. Additionally change the `Target` variant to have a newtype
`CompileTarget` instead of just being a raw string. This new
`CompileTarget` type has a fallible constructor and handles custom json
target files internally.

Two accessors are available for `CompileTarget`, one is `rustc_target()`
which goes straight to rustc and everything else uses `short_name()`
which is the raw target or file stem for json files. The `short_name` is
used everywhere in Cargo for all purposes like configuration, env vars,
target directory naming, etc.

Refactor `Kind` to carry target name in `Target`

This commit is an internal refactoring of Cargo's compilation backend to
eventually support compiling multiple target simultaneously. The
original motivation for this came up in discussion of #7297 and this has
long been something I've intended to update Cargo for. Nothing in the
backend currently exposes the ability to actually build multiple target
simultaneously, but this should have no function change with respect to
all current consumers. Eventually we'll need to refactor APIs of how you
enter the compilation backend to compile for multiple targets.

remove unsafe sslv2 and sslv3 methods

add an option to specify ssl version

Fixes #6684

Auto merge of #7440 - alexcrichton:always-pipeline, r=ehuss

Remove dynamic detection of pipelining support

Pipelining has made its way to stable so no need to check that any
more.

Auto merge of #7441 - ehuss:fix-zalgo, r=alexcrichton

Fix zalgo formatting.

Now that 1.38 has hit stable, we can finally update this rustfmt change.

Fix zalgo formatting.

Merge remote-tracking branch 'origin/master' into custom-profile-pr-rfc

Auto merge of #7429 - alexcrichton:fix-osx-cpu, r=ehuss

Fix macOS collection of CPU data

There's very little documentation on `host_processor_info` from what I can tell, so I'm just cribbing examples I've found elsewhere on the internet. Turns out two things were wrong:

* One is that `host_processor_info` returns allocated memory we need to deallocate. Who knew!
* Next is that one of the out parameters, `cpu_info_cnt`, is only somehow related to the size of the return, but all example code appears to just read data regardless of what it is.

In any case this commit reads [libuv's implementation](https://github.com/libuv/libuv/blob/040543eebf4983b1459a1e0e0e26dae68b80cc28/src/unix/darwin.c#L174-L225) which if good enough for node.js is probably good enough for us.

Closes #7427

Auto merge of #7428 - alexcrichton:nocapture, r=ehuss

Don't ever capture CPU state if timings are disabled

This should initially help with the panic in #7427 but doesn't solve it.

Auto merge of #7421 - ehuss:build-std-sysroot, r=alexcrichton

Change build-std to use --sysroot

This transitions build-std to use `--sysroot` instead of `--extern`. This is necessary because existing crates have a certain expectation of how standard library crates are exposed. It was intended that explicit dependencies in `Cargo.toml` would solve this problem, but I didn't really consider this would be a backwards-incompatible change, so using `--sysroot` is probably the best way to go, even though it's not ideal.

Closes rust-lang/wg-cargo-std-aware#31
Closes rust-lang/wg-cargo-std-aware#40

Fix a panic collecting cpu data on OSX

There's very little documentation on `host_processor_info` from what I
can tell, so I'm just cribbing examples I've found elsewhere on the
internet. It appears that I've misinterpreted one of the out parameters
of `host_processor_info` as the number of elements of an array, but it
actually is just something forwarded to `vm_deallocate`.

Call `vm_deallocate` on OSX for CPU statistics

Apparently StackOverflow doesn't have all the answers. The examples
there I lifted this from didn't account to call `vm_deallocate` because
it looks like we're handed allocated memory! Indeed running the previous
state capture in a loop it infinitely allocated memory, but now it holds
steady when called in a loop.

Don't ever capture CPU state if timings are disabled

This should initially help with the panic in #7427 but doesn't solve it.

Add comment for unusual placement of add_sysroot_artifact.

Add some comments about Layout naming and .sysroot.

Auto merge of #7411 - Eh2406:mtime, r=alexcrichton

set -Zmtime_on_use from config or ENV

This lets you set the `-Zmtime_on_use` in config, it worked for me with
```toml
[unstable]
mtime_on_use=true
```
I did not find the ENV that would allow work. Suggestions?
Does this need tests?

Closes: #6978

More docs

Add warning when mixing -Zbuild-std with --build-plan.

Don't add sysroot lib path to dylib path for build-std.

use the newer `.get::<Option<bool>>`

Simplify mock rustc wrapper.

Remove dynamic detection of pipelining support

Pipelining has made its way to stable so no need to check that any
more.

Auto merge of #7419 - alexcrichton:fix-features-regression, r=ehuss

Fix interpretation of `--features a b` on the CLI

Fixes an accidental regression from #7084 where `--features a b` was
erroneously mistinterpreted as `--features "a b"`.

Closes #7418

Fix interpretation of `--features a b` on the CLI

Fixes an accidental regression from #7084 where `--features a b` was
erroneously mistinterpreted as `--features "a b"`.

Closes #7418

Auto merge of #7422 - rust-lang:dependabot/cargo/env_logger-0.7.0, r=Eh2406

Update env_logger requirement from 0.6.0 to 0.7.0

Updates the requirements on [env_logger](https://github.com/sebasmagri/env_logger) to permit the latest version.
<details>
<summary>Release notes</summary>

*Sourced from [env_logger's releases](https://github.com/sebasmagri/env_logger/releases).*

> ## 0.7.0
> # Key Changes
>
> - Indent multiline messages by default
> - Support more timestamp precision
> - Update to the 2018 edition
>
> # Changes to minimum Rust
>
> The minimum version of Rust required has been set at `1.31.0`. We may change this in patch versions, but will always flag it in the release notes here.
>
> You can always check the `.travis.yml` file to see the current minimum supported version.
>
> # Contributions
>
> - [@&#8203;95ulisse](https://github.com/95ulisse) [Indentation for multiline log messages](https://github-redirect.dependabot.com/sebasmagri/env_logger/pull/134)
> - [@&#8203;oherrala](https://github.com/oherrala) [Add more timestamp precisions](https://github-redirect.dependabot.com/sebasmagri/env_logger/pull/140)
> - [Update to 2018 edition](https://github-redirect.dependabot.com/sebasmagri/env_logger/pull/142)
</details>
<details>
<summary>Commits</summary>

- [`424f031`](https://github.com/sebasmagri/env_logger/commit/424f03167401b0cff8f3da8424adc3dc1e7683d0) Merge pull request [#143](https://github-redirect.dependabot.com/sebasmagri/env_logger/issues/143) from KodrAus/cargo/0.7.0
- [`058655e`](https://github.com/sebasmagri/env_logger/commit/058655eb6875414debae568c0346038eca83c1fe) prepare for 0.7.0 release
- [`bfac641`](https://github.com/sebasmagri/env_logger/commit/bfac6411adb05b7587b5de4acea0d5c3044ef029) Merge pull request [#142](https://github-redirect.dependabot.com/sebasmagri/env_logger/issues/142) from KodrAus/feat/2018
- [`ef10fa5`](https://github.com/sebasmagri/env_logger/commit/ef10fa512ab253ca86cbb9f8f95731b64aadf323) use specific log version
- [`dc65e48`](https://github.com/sebasmagri/env_logger/commit/dc65e48bae6e13c302451d58e2002f1354d5db01) bump msrv to 1.31.0 for editions
- [`8209524`](https://github.com/sebasmagri/env_logger/commit/8209524a2898b8367b369437e7de2dd48a14edb4) update to 2018 edition
- [`7105e63`](https://github.com/sebasmagri/env_logger/commit/7105e63885e2e325f3eceddcfe0ae6ae95e1ee0f) make breaking changes to timestamp API
- [`75911d7`](https://github.com/sebasmagri/env_logger/commit/75911d7711477b7efd8d0e6eb0886b2c54a17b6f) Merge pull request [#140](https://github-redirect.dependabot.com/sebasmagri/env_logger/issues/140) from oherrala/timestamp-formats
- [`95cd4ed`](https://github.com/sebasmagri/env_logger/commit/95cd4ed266be75d64bdf05c459c44bba5d759e30) Trick the compiler to think we are using timestamp
- [`92a5b2d`](https://github.com/sebasmagri/env_logger/commit/92a5b2dfa56d6926c97f07919aed857c39ac571e) Add more timestamp precisions
- Additional commits viewable in [compare view](https://github.com/sebasmagri/env_logger/compare/v0.6.0...v0.7.0)
</details>
<br />

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
- `@dependabot use these labels` will set the current labels as the default for future PRs for this repo and language
- `@dependabot use these reviewers` will set the current reviewers as the default for future PRs for this repo and language
- `@dependabot use these assignees` will set the current assignees as the default for future PRs for this repo and language
- `@dependabot use this milestone` will set the current milestone as the default for future PRs for this repo and language
- `@dependabot badge me` will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot [dashboard](https://app.dependabot.com):
- Update frequency (including time of day and day of week)
- Pull request limits (per update run and/or open at any time)
- Automerge options (never/patch/minor, and dev/runtime dependencies)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)

Finally, you can contact us by mentioning @dependabot.

</details>

Update env_logger requirement from 0.6.0 to 0.7.0

Updates the requirements on [env_logger](https://github.com/sebasmagri/env_logger) to permit the latest version.
- [Release notes](https://github.com/sebasmagri/env_logger/releases)
- [Changelog](https://github.com/sebasmagri/env_logger/blob/master/CHANGELOG.md)
- [Commits](https://github.com/sebasmagri/env_logger/compare/v0.6.0...v0.7.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Some more --sysroot tests.

Change how standard_lib tests work

* Minimize the sysroot crates in play
* Don't use build scripts to inject args
* Use `RUSTC_WRAPPER` to dynamically switch `--sysroot` depending on
  whether we're building sysroot crates or not.
* Minimize dependency graph in sysroot, only have each crate depend on a
  dummy crates.io crate for testing and otherwise don't depend on
  anything to load the desired sysroot crate directly.