Added missing file extension

Changed LINKER env var to RUSTC_LINKER, improved docs

Added documentation for LINKER environment variable

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

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

Speedup no-op builds by caching rustc invocations

Hide BuildConfig.host_triple behind a getter

Associated Rustc with a build config

We want to create `rustc` only when we already have workspace, so that
we could use the `target` dir to store cached info about the compiler.

Extract mtime calculation

Correctly print fractional part of a second

Auto merge of #5355 - djc:target-info-no-context, r=matklad

Make it possible to construct TargetInfo without Context

This should make stuff like #5349 easier.

Auto merge of #5354 - djc:config-initialization, r=matklad

Move initialization of BuildConfig, TargetConfig into new() methods

Inline creation of TargetInfos in Context::new()

Don't require full Context to build a TargetInfo

Remove target_triple() method from Context

Favor accessing cx.build_config directly.

Remove host_triple() method from Context

Favor accessing cx.build_config directly.

Remove requested_target() method from Context

Favor accessing cx.build_config directly.

Correct formatting with cargo fmt

Auto merge of #5353 - matklad:features, r=alexcrichton

New semantics for `--features` flag

Historically, feature-related flags like `--all-features`,
`--no-default-features` and `--features` operated on the *current*
package. That is, `cargo --package foo --feature feat` would activate
`feat` for the package at the current working directory, and not for the
`foo` package. `-Z package-features` flag implements the more obvious
semantics for this combination of flags. This changes behavior, and that
is why we want to start with an unstable opt-in. The changes are:

* `--feature` flag affects the selected package. It is an error to
  specify `--feature` with more than a single `-p`, or with `-p` outside
  workspace (the latter could work in theory, but would be hard to
  implement).

* `--all-features` and `--no-default-features` affect all selected
  packages, and not the one at cwd.

* The package in `cwd` is not implicitly enabled when doing feature
  selection. That is, `cargo build -Z package-features -p foo` could
  select *less* features for various packages than `cargo build -p foo`.

r? @alexcrichton

New semantics for `--features` flag

Historically, feature-related flags like `--all-features`,
`--no-default-features` and `--features` operated on the *current*
package. That is, `cargo --package foo --feature feat` would activate
`feat` for the package at the current working directory, and not for the
`foo` package. `-Z package-features` flag implements the more obvious
semantics for this combination of flags. This changes behavior, and that
is why we want to start with an unstable opt-in. The changes are:

* `--feature` flag affects the selected package. It is an error to
  specify `--feature` with more than a single `-p`, or with `-p` outside
  workspace (the latter could work in theory, but would be hard to
  implement).

* `--all-features` and `--no-default-features` affect all selected
  packages, and not the one at cwd.

* The package in `cwd` is not implicitly enabled when doing feature
  selection. That is, `cargo build -Z package-features -p foo` could
  select *less* features for various packages than `cargo build -p foo`.

Move BuildConfig initialization into new() method

Move creation of TargetConfig into new() method

Auto merge of #5350 - matklad:simplify-install, r=djc

Simplify install

Just a mechanical simplification.

Auto merge of #5352 - surma-dump:what-is-a-manifest, r=matklad

Mention Cargo.toml in manifest help description

I was looking for a CLI flag to pass a different TOML to cargo, but didn’t realize it’s called “manifest” from reading the help.

Remove parens

Mention Cargo.toml in manifest help description

Auto merge of #5346 - derekdreery:package_id_docs, r=matklad

Add docs

Just adds some docs matching what @matklad told me on irc :)

remove getter/setter docs

Cleanup formatting

Minor simplification

Auto merge of #5348 - djc:resolve-target-specific, r=alexcrichton

Have compilation context info available earlier in the build process

Eventually, I hope this will allow us to ignore platform-specific dependencies when irrelevant for the current build earlier in the process. This should save on extraneous errors. As is, this seems like it already decreases coupling in the code base.

Move compiler driver code from ops into core

Use relative dependencies inside cargo_rustc module

Use Unit type as interface to compilation

Move cargo_rustc::compile_targets() to Context::compile()

Hoist Context creation out of compile_targets()

Deliver units to compilation Context after initialization

Error out early on empty targets

Initialize BuildConfig in a single place

Inline single use variable binding

Process build configuration before resolving dependencies

Correct formatting with cargo fmt

Add docs

Auto merge of #5341 - dwijnand:test-help, r=alexcrichton

tweak test/bench's help around filtering & libtest

Attempt to bring more clarity about test/bench's filtering arg by
detailing the distinction and integration between cargo's test/bench and
libtest.

Also make sure to mention how to see libtest's help/usage message.

Fixes #5337

Feedback welcome.  Also, how do I auto-generate the man pages in
src/etc/man/?  I'm wasn't sure about the escaping to attempt manually
editing.

Auto merge of #5343 - dwijnand:contributing-fmt, r=matklad

add a rustfmt detail to CONTRIBUTING.md

stealing from the info in https://github.com/rust-lang/cargo/pull/5301#issuecomment-380389555

add a rustfmt detail to CONTRIBUTING.md

tweak test/bench's help around filtering & libtest

Attempt to bring more clarity about test/bench's filtering arg by
detailing the distinction and integration between cargo's test/bench and
libtest.

Also make sure to mention how to see libtest's help/usage message.

Fixes #5337

Auto merge of #5333 - dwijnand:warn-install-2018, r=matklad

Warn/error when cargo installing the cwd in 2015/2018

Fixes #5327

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

more cargo install usage adaptations in the same tests

adapt legitimate cargo install usage to avoid the new warning/error

add the inverse `is_not has_installed_exe` assertion

warn in 2015, error in 2018

re-implement by adding a few from_cwd args

found a way to reuse SourceId::for_path in SourceId::from_cwd

document SourceId::from_cwd

enable cargo edition feature & bump to 2018

Warn about cargo installing the cwd in 2018 edition

Fixes #5327

Auto merge of #5331 - dwijnand:contributing-fmt, r=matklad

Tweak rustfmt details in CONTRIBUTING.md

Closes #5329