Auto merge of #5752 - dwijnand:project_at, r=alexcrichton

Remove the argument from the `project` test support function

By rewriting the tests, with rerast (https://github.com/google/rerast), to
use the newly introduced "at" method.

First I added the following temporary function to cargotest::support:

    pub fn project_foo() -> ProjectBuilder {
       project("foo")
    }

Then I defined the following rewrite.rs:

    use cargotest::support::{ project, project_foo };

    fn rule1(a: &'static str) {
       replace!(project("foo") => project_foo());
       replace!(project(a) => project_foo().at(a));
    }

Then I ran rerast:

    cargo +nightly rerast --rules_file=rewrite.rs --force --targets tests --file tests/testsuite/main.rs

Finally I searched and replaced the references to project_foo with
argument-less project (a little awkardly on macOS with a git clean).

    find tests -type f -exec sed -i -e 's/project_foo/project/g' {} +
    git clean -d tests

Fixes #5746

Migrate trailing calls to `project` with an argument

.. in docs, commented code & tests targetting non-macos.

Remove the argument from the `project` test support function

By rewriting the tests, with rerast (https://github.com/google/rerast),
to use the newly introduced "at" method.

First I added the following temporary function to cargotest::support:

    pub fn project_foo() -> ProjectBuilder {
        project("foo")
    }

Then I defined the following rewrite.rs:

    use cargotest::support::{ project, project_foo };

    fn rule1(a: &'static str) {
        replace!(project("foo") => project_foo());
        replace!(project(a) => project_foo().at(a));
    }

Then I ran rerast:

    cargo +nightly rerast --rules_file=rewrite.rs --force --targets tests --file tests/testsuite/main.rs

Finally I searched and replaced the references to project_foo with
argument-less project (a little awkardly on macOS with a git clean).

    find tests -type f -exec sed -i -e 's/project_foo/project/g' {} +
    git clean -d tests

Add tests/ProjectBuilder::at

Auto merge of #5741 - rust-lang:dependabot/cargo/termcolor-1.0, r=alexcrichton

Update termcolor requirement to 1.0

Updates the requirements on [termcolor](https://github.com/BurntSushi/termcolor) to permit the latest version.
<details>
<summary>Commits</summary>

- See full diff in [compare view](https://github.com/BurntSushi/termcolor/commits/wincolor-1.0.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`.

---

<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 merge` will merge this PR after your CI passes on it
- `@dependabot ignore this [patch|minor|major] version` will close this PR and stop Dependabot creating any more for this minor/major 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 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)
- Automerge options (never/patch/minor, and dev/runtime dependencies)
- Pull request limits (per update run and/or open at any time)
- 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 termcolor requirement to 1.0

Updates the requirements on [termcolor](https://github.com/BurntSushi/termcolor) to permit the latest version.
- [Release notes](https://github.com/BurntSushi/termcolor/releases)
- [Commits](https://github.com/BurntSushi/termcolor/commits/wincolor-1.0.0)

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

Auto merge of #5735 - enizor:doc-profile-debug, r=alexcrichton

Doc: manifest profile.X.debug can be either an u32 or a bool

See #5734

Doc: manifest profile.X.debug can be either an u32 or a bool

Fixes #5734

Auto merge of #5723 - alexcrichton:cargo-fix, r=ehuss

Import `cargo fix` directly in to Cargo

This commit imports the `cargo fix` subcommand in rust-lang-nursery/rustfix
directly into Cargo as a subcommand. This should allow us to ease our
distribution story of `cargo fix` as we prepare for the upcoming 2018 edition
release.

It's been attempted here to make the code as idiomatic as possible for Cargo's
own codebase. Additionally all tests from cargo-fix were imported into Cargo's
test suite as well. After this lands and is published in nightly the `cargo-fix`
command in rust-lang-nursery/rustfix will likely be removed.

cc rust-lang/rust#52272

Import `cargo fix` directly in to Cargo

This commit imports the `cargo fix` subcommand in rust-lang-nursery/rustfix
directly into Cargo as a subcommand. This should allow us to ease our
distribution story of `cargo fix` as we prepare for the upcoming 2018 edition
release.

It's been attempted here to make the code as idiomatic as possible for Cargo's
own codebase. Additionally all tests from cargo-fix were imported into Cargo's
test suite as well. After this lands and is published in nightly the `cargo-fix`
command in rust-lang-nursery/rustfix will likely be removed.

cc rust-lang/rust#52272

Auto merge of #5732 - Eh2406:sort_unstable, r=alexcrichton`,

most sorts can be unstable

Inspired by [this](https://github.com/rust-lang/cargo/blob/94f7058a483b05ad742da5efb66dd1c2d4b8619c/src/bin/cargo/main.rs#L112-L122) witch was improved in #5691, I did a quick review of `sort`s in the code. Most can be unstable, some can use a `_key` form, and none had unnecessary allocation.

most sorts can be unstable

Auto merge of #5710 - RalfJung:default-run, r=alexcrichton

implement default-run option to set default binary for cargo run

The implementation is not pretty but as good as I could make it. The fact that all this logic in `cargo_run` is for diagnosis only and essentially just re-implements the filtering done elsewhere really threw me off.

Fixes #2200

document new feature

fix code nits

order of lits of binaries is not stable

feature-gate default-run

implement default-run option to set default binary for cargo run

Auto merge of #5691 - Eh2406:hyphen-underscore, r=alexcrichton

Make index lookup robust to _ vs -, but don't let the user get it wrong.

This does a brute force search thru combinations of hyphen and underscores to allow queries of crates to pass the wrong one.

This is a small first step of fixing #2775

Where is best to add test?

only suggest lev_distance < 4

Auto merge of #5543 - roblabla:doc-private-items, r=alexcrichton

Add document-private-items flag to cargo doc

Add a `--document-private-items` flag to `cargo doc`, that mimics the equivalent `cargo rustdoc -- --document-private-items`. This works by relaying the flag to the underlying rustdoc call.

Verify that private items were actually documented

Add a test for document-private-items

Auto merge of #5726 - ehuss:warn-virtual-keys, r=alexcrichton

Warn about unused virtual manifest keys.

Fixes #4243

Auto merge of #5727 - Eh2406:todos, r=alexcrichton

fix some Todo's

Just grep for todo and fix the simple ones.

`recv_timeout` was stable in 1.12

remove unused Serialize

remove unused Deserialize

remove incorrect todo

I tried repeatedly to remove the Rc with nll on.
`query` works, but it can not be called repeatedly.

Warn about unused virtual manifest keys.

Fixes #4243

oops...

Auto merge of #5715 - euclio:third-party-help, r=alexcrichton

help: display external subcommand help

When invoking `cargo help <subcommand>`, if the subcommand isn't found, but it *is* an external subcommand, call that subcommand with `--help`.

A test should probably be written for this, but I'm not sure how best to mock an external subcommand.

document custom subcommand help

add tests

move the Uncanonicalizing loop to a Iterator

Auto merge of #5711 - alexcrichton:fix-stack-overflow, r=ehuss

Partially revert dep changes in #5651

Some logic which was tweaked around the dependencies of build script targets was
tweaked slightly in a way that causes cargo to stack overflow by accientally
adding a dependency loop. This commit implements one of the strategies discussed
in #5711 to fix this situation.

The problem here is that when calculating the deps of a build script we need the
build scripts of *other* packages, but the exact profile is somewhat difficult
to guess at the moment we're generating our build script unit. To solve this the
dependencies towards other build scripts' executions is added in a different
pass after all other units have been assembled. At this point we should know for
sure that all build script executions are in the dependency graph, and we just
need to add a few more edges.

Closes #5708

help: display external subcommand help

Auto merge of #5700 - alexcrichton:update-width, r=matklad

Update terminal width each time we print a display

This should help accomodate terminals that are resized while a progress bar is
being displayed. Additionally this shouldn't come with too much of a performance
impact because we're already heavily throttling printing of a progress update!

Closes #4732

Partially revert dep changes in #5651

Some logic which was tweaked around the dependencies of build script targets was
tweaked slightly in a way that causes cargo to stack overflow by accientally
adding a dependency loop. This commit implements one of the strategies discussed
in #5711 to fix this situation.

The problem here is that when calculating the deps of a build script we need the
build scripts of *other* packages, but the exact profile is somewhat difficult
to guess at the moment we're generating our build script unit. To solve this the
dependencies towards other build scripts' executions is added in a different
pass after all other units have been assembled. At this point we should know for
sure that all build script executions are in the dependency graph, and we just
need to add a few more edges.

Closes #5708

Auto merge of #5712 - alexcrichton:better-debug, r=ehuss

Improve Debug display for a few types

Try to cut down on the wordiness here and make these implementations a bit more
usable by default.

sort suggestions by distance

Auto merge of #5716 - kennytm:unstable-compile-progress, r=alexcrichton

Reintroduce the compile progress bar as an unstable feature (-Z compile-progress)

This allows us to test the feature on-demand to see if there's any other bugs besides #5695.

Also, fixed the flickering #5697 (this was caused by build script emitting Stdout/Stderr events).

Compile progress: Do not update when a build script prints something.

Fix #5697. There may still be some little flickering, but should be much
less severe than before.

Bring back the compile progress bar as an -Z option.

update test to get ci passing

Auto merge of #5714 - jeremyBanks:docs-project-to-package, r=alexcrichton

Replace remaining uses of `[project]` in documentation with `[package]`

The `[project]` section is not currently referenced anywhere else in the documentation, so the use here confused me, and I spent a while trying to add it to a `Cargo.toml` that already had a `[package]`.

`[project]` seems to be a deprecated alias for `[package]`, which is documented, so let's swap it out.

Auto merge of #5713 - mbrubeck:doc, r=alexcrichton

Clarify FAQ about Cargo.lock for libraries

The current wording in the FAQ implies that checking in a Cargo.lock for a library could cause version conflicts in consumers of the library, which is not true.

The text in the FAQ was based on [this comment][1] (emphasis mine):

> If each dependent library checked in a Cargo.lock, **and Cargo used it,** you would instead get multiple, duplicate copies of those dependencies

but it accidentally dropped the "and Cargo used it" requirement, which I think is the more important part.

[1]: https://github.com/rust-lang/cargo/pull/534#issuecomment-54776481

Replace remaining uses of `[project]` in documentation with `[package]`

The `[project]` section is not currently referenced anywhere else in the documentation, so the use here confused me. It seems to be a deprecated alias for `[package]`, which is documented, so let's swap it.

allow each source to recommend packages that are close to a dependency

Clarify FAQ about Cargo.lock for libraries

The current wording in the FAQ implies that checking in a Cargo.lock for
a library could cause version conflicts in consumers of the library,
which is not true.

The text in the FAQ was based on [this comment][1] (emphasis mine):

> If each dependent library checked in a Cargo.lock, **and Cargo used
> it,** you would instead get multiple, duplicate copies of those
> dependencies

but it accidentally dropped the "and Cargo used it" requirement, which I
think is the more important part.

[1]: https://github.com/rust-lang/cargo/pull/534#issuecomment-54776481

Improve Debug display for a few types

Try to cut down on the wordiness here and make these implementations a bit more
usable by default.

add test for wrong_case in registry

add test for wrong_case in cargo.toml

Update terminal width each time we print a display

This should help accomodate terminals that are resized while a progress bar is
being displayed. Additionally this shouldn't come with too much of a performance
impact because we're already heavily throttling printing of a progress update!

Closes #4732

Auto merge of #5698 - alexcrichton:no-progress-sad, r=kennytm

Disable progress bar for build in Cargo

This is primarily blocked on #5695 which unfortunately doesn't have a great fix
today, so disable it for now while we try to work out a better solution.

Closes #5695
Closes #5697

start with the name as given, and cap the number to be switched

cargo +stable fmt

make index lookup robust to _ vs -

Disable progress bar for build in Cargo

This is primarily blocked on #5695 which unfortunately doesn't have a great fix
today, so disable it for now while we try to work out a better solution.

Closes #5695
Closes #5697

Auto merge of #5694 - Eh2406:more_InternedString, r=alexcrichton

More interned string

This is a series of attempts to remove temporary strings. There is no evidence that this will help with anything, but it just seems cleaner.

Probably easier to read one commit at a time.

cargo fmt and impl Borrow<str> for InternedString

add the Cows back in and add a test to keep them

one more small change

oops...

reduce 2 more allocation of temp strings

reduce InternedString::new from the hot loop

reduce InternedString::new

Auto merge of #5693 - rust-lang:dependabot/cargo/lazycell-1.0, r=alexcrichton

Update lazycell requirement to 1.0

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

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

> ## v1.0.0
> <a name="v1.0.0"></a>
> ## v1.0.0 (2018-06-06)
>
>
> #### Features
>
> *   Add #![no_std] ([e59f6b55](https://github.com/indiv0/lazycell/commit/e59f6b5531e310d3df26b0eb40b1431937f38096))
</details>
<details>
<summary>Changelog</summary>

*Sourced from [lazycell's changelog](https://github.com/indiv0/lazycell/blob/master/CHANGELOG.md).*

> ## v1.0.0 (2018-06-06)
>
>
> #### Features
>
> *   Add #![no_std] ([e59f6b55](https://github.com/indiv0/lazycell/commit/e59f6b5531e310d3df26b0eb40b1431937f38096))
>
>
>
> <a name="0.6.0"></a>
> ## 0.6.0 (2017-11-25)
>
>
> #### Bug Fixes
>
> *   fix soundness hole in borrow_with ([d1f46bef](https://github.com/indiv0/lazycell/commit/d1f46bef9d1397570aa9c3e87e18e0d16e6d1585))
>
> #### Features
>
> *   add Default derives ([71bc5088](https://github.com/indiv0/lazycell/commit/71bc50880cd8e20002038197c9b890f5b76ad096))
> *   add LazyCell::try_borrow_with ([bffa4028](https://github.com/indiv0/lazycell/commit/bffa402896670b5c78a9ec050d82a58ee98de6fb))
> *   add LazyCell::borrow_mut method ([fd419dea](https://github.com/indiv0/lazycell/commit/fd419dea965ff1ad3853f26f37e8d107c6ca096c))
>
> #### Breaking Changes
>
> *   add `T: Send` for `AtomicLazyCell` `Sync` impl ([668bb2fa](https://github.com/indiv0/lazycell/commit/668bb2fa974fd6707c4c7edad292c76a9017d74d), closes [#67](https://github-redirect.dependabot.com/indiv0/lazycell/issues/67))
>
> #### Improvements
>
> *   add `T: Send` for `AtomicLazyCell` `Sync` impl ([668bb2fa](https://github.com/indiv0/lazycell/commit/668bb2fa974fd6707c4c7edad292c76a9017d74d), closes [#67](https://github-redirect.dependabot.com/indiv0/lazycell/issues/67))
>
>
>
> <a name="v0.5.1"></a>
> ## v0.5.1 (2017-03-24)
>
>
> #### Documentation
>
> *   fix missing backticks ([44bafaaf](https://github.com/indiv0/lazycell/commit/44bafaaf93a91641261f58ee38adadcd4af6458e))
>
> #### Improvements
>
> *   derive `Debug` impls ([9da0a5a2](https://github.com/indiv0/lazycell/commit/9da0a5a2ffac1fef03ef02851c2c89d26d67d225))
>
> #### Features
>
> *   Add get method for Copy types ([dc8f8209](https://github.com/indiv0/lazycell/commit/dc8f8209888b6eba6d18717eba6a22614629b997))
>
>
></table> ... (truncated)
</details>
<details>
<summary>Commits</summary>

- See full diff in [compare view](https://github.com/indiv0/lazycell/commits/v1.0.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`.

---

**Note:** This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.

You can always request more updates by clicking `Bump now` in your [Dependabot dashboard](https://app.dependabot.com).

<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 merge` will merge this PR after your CI passes on it
- `@dependabot ignore this [patch|minor|major] version` will close this PR and stop Dependabot creating any more for this minor/major 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 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)
- Automerge options (never/patch/minor, and dev/runtime dependencies)
- Pull request limits (per update run and/or open at any time)
- 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>

reduce more allocation of temp strings

Update lazycell requirement to 1.0

Updates the requirements on [lazycell](https://github.com/indiv0/lazycell) to permit the latest version.
- [Release notes](https://github.com/indiv0/lazycell/releases)
- [Changelog](https://github.com/indiv0/lazycell/blob/master/CHANGELOG.md)
- [Commits](https://github.com/indiv0/lazycell/commits/v1.0.0)

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

reduce allocation of temp strings

Auto merge of #5680 - mikeyhew:fix-shell-quoting, r=alexcrichton

Fix the shell_quoting test

r? @alexcrichton

add-on to #5666
- Added `mod shell_quoting;` to testsuite/main.rs, so it actually runs
- fixed bugs in the test

add `MSYSTEM=1` to shell_quoting test
- this should make it use unix escaping on windows

Auto merge of #5689 - ehuss:doc-opt-level, r=alexcrichton

Document opt-level 's' and 'z' stabilized in 1.28.

Document opt-level 's' and 'z' stabilized in 1.28.

Auto merge of #5688 - nrc:rls-features, r=alexcrichton

Add missing winapi features

To support RLS in Rust repo, features from https://github.com/rust-lang/rust/pull/51677#issuecomment-402781751

r? @alexcrichton

Add missing winapi features

To support RLS in Rust repo, features from https://github.com/rust-lang/rust/pull/51677#issuecomment-402781751

Auto merge of #5571 - pickfire:patch-1, r=alexcrichton

Remove explicit --bin for getting started doc

Auto merge of #5651 - ehuss:panic-rustdoc, r=matklad

Fix doctests linking too many libs.

Fixes #5650.  cc #5435

As part of my recent work on profiles, I introduced some situations where a
library can be compiled multiple times with different settings.  Doctests were
greedily grabbing all dependencies for a package, regardless of which target
is was for.  This can cause doctests to fail if it links multiple copies of
the same library.

One way to trigger this is `cargo test --release` if you have dependencies, a
build script, and `panic="abort"`.  There are other (more obscure) ways to
trigger it with profile overrides.

Auto merge of #5685 - alexcrichton:fix-cargo-install-target-dir, r=matklad

Fix `cargo install` using a workspace target dir

Closes #5662

Fix `cargo install` using a workspace target dir

Closes #5662

get the new test working

- Added `mod shell_quoting` to testsuite/main.rs
- fixed up errors in the test, now that the test suite knows about it

Auto merge of #5666 - mikeyhew:fix-shell-quoting, r=alexcrichton

[needs test] always shell-escape command arguments after failure

cc @joshtriplett
fixes #5665

Removes the `debug_string` method, in favour of always using the
`fmt::Display` impl. `debug_string` didn’t escape the command
arguments, so that’s why we ended up with unescaped arguments after
compilation failed.

Don't merge this yet, because I still want to add a regression test. I don't think there's any tests on shell quoting yet, so I'll probably add a new test file including tests for the "Running <command>" output too. I still have to figure out how to write tests, and don't have the energy to do that tonight 😴

I just wanted to say, by the way, I was pleasantly surprised with how clean the code is in this repo, especially compared to rustc. It made it really easy to find the relevant part of the code and implement these changes.

added a test that `feature=foo` will be quoted

The doesn’t doesn’t seem to run when I run `cargo test` though

Auto merge of #5673 - matklad:no-internal, r=alexcrichton

Show "duplicates in lockfile" without --verbose

It is possible to get duplicate pacakges after a merge, so it make
sense to print a little bit more details by defaults than just
"failed to parse a lockfile".

Show lockfile parsing errors without --verbose

It is possible to get invalid lockfile after an automatic merge, so it
make sense to print a little bit more details by defaults than just
"failed to parse a lockfile".

Auto merge of #5672 - ehuss:cbfct-take3, r=alexcrichton

cbfct: Take 3

Third attempt to fix race condition in changing_bin_features_caches_targets.
Previous fix for linux re-broke Windows
(https://ci.appveyor.com/project/rust-lang-libs/cargo/build/1.0.5004)

cbfct: Take 3

Third attempt to fix race condition in changing_bin_features_caches_targets.
Previous fix for linux re-broke Windows
(https://ci.appveyor.com/project/rust-lang-libs/cargo/build/1.0.5004)

Auto merge of #5652 - Eh2406:masquerade_as_nightly_cargo, r=alexcrichton

skip test if not nightly; fix for #5648

closes #5648
The alternative was to use `set_var` to emulate `masquerade_as_nightly_cargo` but we worried that it would not get unset correctly. Although I think each test is its own process, so it should work.

Auto merge of #5669 - alexcrichton:less-build, r=matklad

Fix avoiding a rebuild when moving around a workspace

There's a case where Cargo will recompile a project even if the fingerprint
looks like it's fresh, when some output files are missing. This was intended to
cover the case where an output file was deleted manually or otherwise messed
with. The check here was a bit too eager, however. It checked not only the
actual output destination of the compiler but *also* the location that we hard
link the output file up to.

Due to recent changes in #5460 we don't always create the hard links for path
dependencies in the top-level dir, and this meant that if the library were
compiled and then tested later on the test may recompile the original library by
accident.

The fix in this commit is to cease looking for the hardlink if it exists or not.
This way we only check for the presence of the output file itself and only
recompile if that file is missing. The reason for this is that we
unconditionally relink files into place whether it's fresh or not, so we'll
always recreate the hard link anyway if it's missing.

cc rust-lang/rust#51717

default tests to stable and override where needed

Fix avoiding a rebuild when moving around a workspace

There's a case where Cargo will recompile a project even if the fingerprint
looks like it's fresh, when some output files are missing. This was intended to
cover the case where an output file was deleted manually or otherwise messed
with. The check here was a bit too eager, however. It checked not only the
actual output destination of the compiler but *also* the location that we hard
link the output file up to.

Due to recent changes in #5460 we don't always create the hard links for path
dependencies in the top-level dir, and this meant that if the library were
compiled and then tested later on the test may recompile the original library by
accident.

The fix in this commit is to cease looking for the hardlink if it exists or not.
This way we only check for the presence of the output file itself and only
recompile if that file is missing. The reason for this is that we
unconditionally relink files into place whether it's fresh or not, so we'll
always recreate the hard link anyway if it's missing.

cc rust-lang/rust#51717

Auto merge of #5620 - kennytm:compile-progress, r=matklad

Displays a one line progress of what crates are currently built.

cc #2536, #3448.

The change is based on #3451, but uses the progress bar introduced in #4646 instead. The percentage is simply the number of crates processed ÷ total crates count, which is inaccurate but better than nothing.

Output looks like:
[](https://asciinema.org/a/YTiBAz4K4vfidNTAnehtyH46l)

More fixes.

Auto merge of #5614 - infinity0:master, r=alexcrichton

Support cross-compile install

Amazingly this Just Works, tested cross-compiling aho-corasick from amd64 to armhf on Debian.

Always use the Display impl for ProcessBuilder output

fixes #5665

remove the `debug_string` method, in favour of always using the
`fat::Display` impl. `debug_string` didn’t escape the command
arguments, so that’s why we ended up with unescaped arguments after
compilation failed

Support cross-compile install

Revert "`cargo install` will ignore the target triple specified in a project directory"

This reverts commit 1bad99180aa71f9e0db1dd1629e53beb58ca65b3.

Addressed comments.

Displays a one line progress of what crates are currently built.

cc #2536, #3448.