Remove last TODO comments.

Some test cleanup for profiles.

Add more profile override validation tests.

Add test for profile override on non-dev/release.

Add warning if `panic` is set in test or bench profile.

Add warnings for unknown profile overrides.

Minor tweaks on how examples are handled with tests, and some panic propagation.

Avoid running `build.rs` too often.

Add extra rustc/rustdoc args to the fingerprint and metadata.

Update for some review comments.

Fix a variety of profile bugs.

Move `Profile` back into `Unit` as a copy.

The de-duplication of Units just doesn't work without it.

Minor cleanup.

Profile Overrides (RFC #2282 Part 1)

Auto merge of #5415 - alexcrichton:rename-same-dep, r=matklad

Fix renaming crates as they come from 2 sources

Previously there was a verification in manifest parsing that the same dependency
must come from the same source, but this erroneously triggered an error to get
emitted when the `package` key was used to rename crates. The first change here
was to update that clause to key off the `rename` field rather than the `name`
field.

Afterwards, though, this exposed an existing bug in the implementation. During
compilation we have a `Resolve` which is a graph of crates, but we don't know
*why* each edge in the dependency graph exists. In other words we don't know,
when looking at an edge of the graph, what `Dependency` caused that edge to be
drawn. We need to know this when passing `--extern` flags because the
`Dependency` is what lists what's being renamed.

This commit then primarily refactors `Resolve::deps` from an iterator of package
ids to an iterator of a tuples. The first element is the package id from before
and the second element is a list of `Dependency` directives which caused the
edge to ber driven.

This refactoring cleaned up a few places in the backend where we had to work
around the lack of this knowledge. Namely this also fixes the extra test added
here.

Closes #5413

Fix renaming crates as they come from 2 sources

Previously there was a verification in manifest parsing that the same dependency
must come from the same source, but this erroneously triggered an error to get
emitted when the `package` key was used to rename crates. The first change here
was to update that clause to key off the `rename` field rather than the `name`
field.

Afterwards, though, this exposed an existing bug in the implementation. During
compilation we have a `Resolve` which is a graph of crates, but we don't know
*why* each edge in the dependency graph exists. In other words we don't know,
when looking at an edge of the graph, what `Dependency` caused that edge to be
drawn. We need to know this when passing `--extern` flags because the
`Dependency` is what lists what's being renamed.

This commit then primarily refactors `Resolve::deps` from an iterator of package
ids to an iterator of a tuples. The first element is the package id from before
and the second element is a list of `Dependency` directives which caused the
edge to ber driven.

This refactoring cleaned up a few places in the backend where we had to work
around the lack of this knowledge. Namely this also fixes the extra test added
here.

Closes #5413

Auto merge of #5349 - bmwill:fetch-by-target, r=alexcrichton

cargo-fetch: add option to fetch for a target

Teach cargo-fetch how to optionally fetch dependencies based on a target
platform by specifying the target triple via `--target <TRIPLE>`.

#5216

Auto merge of #5410 - klnusbaum:edition_5406, r=matklad

switch to using the --edition flag

Now that we have an `--edition` flag in the rust compiler, let's switch to using it.

fixes #5406

cargo-fetch: add option to fetch for a target

Teach cargo-fetch how to optionally fetch dependencies based on a target
platform by specifying the target triple via `--target <TRIPLE>`.

Signed-off-by: Brandon Williams <bmwill@google.com>

Auto merge of #5394 - henriklaxhuber:master, r=alexcrichton

Pass linker path to build script

This change adds the environment variable LINKER to pass the path of the linker used by cargo to the build script. This complements the variable RUSTC (the rustc binary used) to give the build script full knowledge of its environment.

A specific usage example would be automatically generating bindings to system headers in cross compilation, e.g. by locating jni.h for android targets.

Auto merge of #5393 - smithsps:master, r=matklad

Add target directory parameter --target-dir

Implements: https://github.com/rust-lang/cargo/issues/5308

Adds a target directory parameter, that acts in the same manner as the environment variable `CARGO_TARGET_DIR`, to the following subcommands:
- `bench`
- `build`
- `check`
- `clean`
- `doc`
- `package`
- `publish`
- `run`
- `rustc`
- `rustdoc`
- `test`

Auto merge of #5398 - dwijnand:drop-legacy-paths, r=matklad

Drop legacy path support under Rust edition 2018 (or later)

builds on #5335

submitted for early feedback: wdyt @matklad? is this what you had in mind? what should change? what should be added? how should we test this? is the current (2015) messaging enough to drop it in 2018?

r? @matklad

<!--{"baseBranch":"rust-lang:cargo:target-autodiscovery"}-->

Add target directory parameter: address suggestions

Moved RUSTC_LINKER test into its own fn

Add target directory parameter --target-dir

Fixed RUSTC_LINKER test for Windows hosts

Fixed build script env var test

Added tests for RUSTC_LINKER buils script env var

Added tests for the environment variable RUSTC_LINKER that is
passed to the build script.
Also slightly improved readability of the code responsible for
passing the env var.

switch to using the --edition flag

Auto merge of #5407 - klausi:fixed-minimal-versions, r=matklad

fix(dependendies): Bump minimal dependency versions so that cargo successfully builds with those

Spin-off from #5275 .

Just bump the minimum dependency versions here, we are developing CI testing in #5275 .

fix(dependendies): Bump minimal dependency versions so that cargo successfully builds with those

Auto merge of #5405 - dwijnand:contributing-lint-overrides, r=matklad

Detail how to override lint warnings when compiling tests

alternative to #5403

Detail how to override lint warnings when compiling tests

Auto merge of #5404 - matklad:remove-useless-nightly, r=matklad

Remove some `is_nightly` checks from tests

Remove some `is_nightly` checks from tests

Auto merge of #5400 - kornelski:master, r=alexcrichton

Docs: rename gcc-rs → cc-rs

Docs: rename gcc-rs → cc-rs

Drop legacy path support under Rust edition 2018 (or later)

Added missing file extension

Changed LINKER env var to RUSTC_LINKER, improved docs

Auto merge of #5335 - dwijnand:target-autodiscovery, r=alexcrichton

Introduce autoXXX keys for target auto-discovery

In Rust 2015 absence of the configuration makes it default to not
include auto-discovered targets (i.e false), with a warnings message.

In Rust 2018 absence makes it default to include auto-discovered
targets (i.e true).

Fixes #5330

---

_original_

Fixes #5330

submitted for early review. feedback very welcome, I'm happy to iterate (and learn).

in particular I require assistance with the borrowing of `warnings` in `clean_benches`.

Auto merge of #5399 - dwijnand:cleanup/drop-an-old-visible-doc-todo, r=matklad

Drop an old visible doc TODO

Looks like this originates from
https://github.com/istankovic/cargo-book/commit/418611304daac22e3590aee9322355ae159802fe

But it's even visible in the render docs:
https://doc.rust-lang.org/cargo/reference/manifest.html#package-selection
:-O

Drop an old visible doc TODO

Looks like this originates from
https://github.com/istankovic/cargo-book/commit/418611304daac22e3590aee9322355ae159802fe

But it's even visible in the render docs:
https://doc.rust-lang.org/cargo/reference/manifest.html#package-selection
:-O

Added documentation for LINKER environment variable

This commit amends the documentation on environment variables to
reflect the proposed LINKER variable.

backtick auto-discovery key usage advice

manually line wrap warnings at 80 characters

prefer PathBuf::display over {:?}

avoid windows path assertion bug

Auto merge of #5396 - dwijnand:arg_targets_all-typos, r=matklad

fix typos in arg_targets_all

None

fix typos in arg_targets_all

Introduce autoXXX keys for target auto-discovery

In Rust 2015 absence of the configuration makes it default to not
include auto-discovered targets (i.e false), with a warnings message.

In Rust 2018 absence makes it default to include auto-discovered
targets (i.e true).

Fixes #5330

Extract toml_targets_and_inferred & inferred_to_toml_targets

Increase scope nesting to borrow

Move edition earlier in TomlManifest::to_real_manifest

and fix a typo in the error message

Auto merge of #5395 - dwijnand:contributing-setup-cross-tests, r=alexcrichton

add contributing notes for cross compile test setup

adding notes from @alexcrichton's help in https://gitter.im/rust-lang/cargo?at=5ada2eae2b9dfdbc3ab01f29.

add contributing notes for cross compile test setup

Pass linker path to build script

This change adds the environment variable LINKER to pass the path
of the linker used by cargo to the build script. This ammends the
variable RUSTC (the rustc binary used) to give the build script
full knowledge of its environment.

A specific usage example would be automatically generating bindings
to system headers in cross compilation, e.g. by locating jni.h for
android targets.

Auto merge of #5391 - varkor:check-message, r=alexcrichton

Print "Checking" for cargo check

I often alternate between `check` and `build` and I'd like to be able to tell which is currently happening (short-term memory being what it is, and "compiling" implying to me that codegen is occurring). Changing the message to "Checking" seemed to be reasonable (and there was precedent for `doc` with "Documenting").

Update tests to check for Checking

Auto merge of #5390 - matklad:new-feature-behavior, r=alexcrichton

Enable new behavior of `--feature`

So far, the feedback on https://internals.rust-lang.org/t/help-us-test-the-breaking-bug-fix-to-cargo-features/7317 has been positive, so here's a PR to try this in nightly.

Note that the logic is slightly tweak for the case `cargo build -p not-a-workspace-member`: we want not only to resolve all ws members in this case, but to enable all of their features as well!

As a sanity check, this seems to be forward compatible with further improvements to features:

1) when we solve the grand bug of features being unified across the whole workspace, `cargo build -p not-a-member` would hopefully just work without additional contortions.
2) we might add a way to specify features per package, like `cargo build -p foo -p bar --features "foo/serde bar/serde"`

Auto merge of #5392 - ehuss:test-does-not-contain, r=alexcrichton

cargotest: Fix `with_*_does_not_contain` to support `[..]` and macro matching.

I changed it so that it is essentially the opposite of `with_*_contains` to keep it symmetric.

Any in-flight PRs using the old style will need to be updated (else they will incorrectly silently pass).  Alternatively, we could rename the method to avoid that.

The following tests contained brackets, so they were not checking what they thought they were checking.  I did a cursory look at them, but perhaps someone else could double-check that they make sense.  Asserting what *doesn't* happen can be tricky since there is an infinite number of things that won't happen.  Preferably a test would assert that it appears in one scenario and not another (like `incremental_profile` does), but some of them don't or can't.

```
build::incremental_profile
build::incremental_config
build::cargo_compile_with_workspace_excluded
build::build_all_exclude
build::targets_selected_default
check::targets_selected_default
check::check_filters
rustc::targets_selected_default
rustc_info_cache::rustc_info_cache
warn_on_failure::no_warning_on_bin_failure
warn_on_failure::warning_on_lib_failure
```

BTW, would you be interested in a PR that adds some documentation to `cargotest`?
I've discovered things I didn't know where there.  I think some docstrings on some of the methods, and a short guide for new contributors would be helpful.

Fix `with_*_does_not_contain` to support `[..]` and macro matching.

Print "Checking" for cargo check

Enable new behavior of `--feature`

Auto merge of #5389 - matklad:one-hard-cs-problem, r=alexcrichton

One hard cs problem

Closes https://github.com/rust-lang/cargo/issues/5313

r? @alexcrichton

Note that, due to the first commit, this still gives us all the benefits of #5249: if RUSTFLAGS is empty, we will run only a single rustc process, even if we can't cache it across different cargo invocations.

Don't try to use the same info if target == host

The info might be different due to RUSTFLAGS, which, in general,
are applied only to target.

rustfmt

Improve rustc cache

Even if we've failed to calculate fingerprint and thus can't persist
cache to disk, it is still valid to cache rustc output within single
process.

Auto merge of #5386 - ysimonson:new-metadata, r=matklad

Added new metadata fields

Addresses #5373.

`cargo metadata` should now include all of the non-optional suggested fields in the [API guidelines](https://rust-lang-nursery.github.io/api-guidelines/documentation.html#cargotoml-includes-all-common-metadata-c-metadata).

Explicitly test for new package metadata fields

Added new metadata fields.

`cargo metadata` should now include all of the non-optional suggested fields in the API guidelines (see section C-METADATA.)

Auto merge of #5379 - alexcrichton:bump-fieltime, r=matklad

Bump version of `filetime` dependency

Auto merge of #5383 - dwijnand:contributing-custom-build, r=alexcrichton

add build-from-source info to Tests architecture docs

it's mentioned in the README, but it's worth mentioning again in the
Tests section of the architecture doc.

Auto merge of #5381 - dwijnand:github-casing, r=alexcrichton

Fix how GitHub is written

just a silly PR - I noticed it in the README, and applied it repo-wide.

Auto merge of #5380 - dwijnand:typo, r=alexcrichton

typo

None

add build-from-source info to Tests architecture docs

it's mentioned in the README, but it's worth mentioning again in the
Tests section of the architecture doc.

Bump version of `filetime` dependency

Auto merge of #5372 - alexcrichton:bump, r=matklad

Bump to 0.28.0

Looks like we've forgotten to do this so far on nightly!

typo

Fix how GitHub is written

Auto merge of #5374 - matklad:simplify, r=alexcrichton

Simplify RegistryPackage deserialization

Just mapping dependencies seems so much simpler than thread locals / deserializers with default. Am I missing something, or this indeed will work? :-)

cc @Mark-Simulacrum

Auto merge of #5361 - derekdreery:issue_5345, r=matklad

Fix issue 5345

Fixes #5345.

Also adds some docs and Debug impls.

Add fix and some docs

Simplify RegistryPackage deserialization

Bump to 0.28.0

Looks like we've forgotten to do this so far on nightly!

Auto merge of #5272 - Phlosioneer:5211-missing-size-hints, r=matklad

Implement size_hint for some iterators

This PR implements size_hints for `Deps`, `DepsNotReplaced`, and `Members`.
These size_hints are used extensively by cargo to allocate Vecs.

`Deps`, `DepsNotReplaced`, and `RcVecIter` also now implement `ExactSizeIterator`.

Closes #5211

Add compile-time check for ExactSizeIterator

Auto merge of #5359 - matklad:rustc-cache, r=alexcrichton

Rustc cache

This implements rustc caching, to speed-up no-op builds.

The cache is per-project, and stored in `target` directory. To implement this, I had to move `rustc` from `Config` down to `BuildConfig`.

closes https://github.com/rust-lang/cargo/issues/5315

rustc is rustc.exe on windows

Simplify

Make rustc_info a hidden file

Handle non-rustup cargo + rustup rustc in rustc cache

Auto merge of #5360 - euclio:metadata, r=alexcrichton

include package metadata in `cargo metadata`

Fixes #4819.

All-in-all a pretty small change. I'm a little concerned about threading the `toml::Value` all the way through however, should I make another type? Also, cloning the metadata value initially is maybe not ideal, since it's technically accessible through the original manifest. I figured this way was cleaner.

Remove `internal` from error handling

Auto merge of #5357 - matklad:even-rust-has-format-string-bugs, r=alexcrichton

Correctly print fractional part of a second

So my experiments with `rustc -vV` caching bring no-op build times to around 50ms, and this actually exposes a bug in our time-printing code :-)

Better test description

[Wip] fix issue 5345

include package metadata in `cargo metadata`

Fixes #4819.

Correctly resolve executables in paths on windows

Add env-var to suppress rustc caching