Replace some assert_that usage in doc comments

Move MatchResult from hamcrest to testsuite/support

Remove hamcrest Matcher

Remove hamcrest has_installed_exe & is_not

Remove hamcrest existing_file()

Remove hamcrest existing_dir()

Remove hamcrest contains()

Auto merge of #5942 - dwijnand:less-assert_that, r=alexcrichton

Further reduce assert_that usage

Refs #5742

Revert back to some .exec_with_output() usage

Avoids difference in indentations across different PATHs.

Enable Drop for Execs

Allow .build_command() to set Execs to ran

Make last two .exec_with_output() usages set Execs to ran

Replace .exec_with_output() usage with .with_stdout_contains

Replace .exec_with_output() usage with .run()

Remove a dead comment in testsuite/support/mod.rs

Update reference to .stream() in ARCHITECTURE.md

Drop reference to the [/] macro in ARCHITECTURE.md

Replace Output hamcrest matchers for Execs

Replace some bare ProcessBuilder usage with Execs

Make Project::process return Execs

Auto merge of #5929 - dwijnand:remove-hamcrest, r=alexcrichton

Replace lots of hamcrest's assert_that with a builder style

Refs #5742

I've been thinking of possible solutions to close out #5742, so I'm submitting this for early feedback.

@alexcrichton what do you think? If you like the look of this I'll apply it across the suite.

r? @alexcrichton

Migrate from tests fom assert_that/execs to .run()

Wrap ProcessBuilder in Execs & make .cargo return that

Make old Execs methods take not consume self

Extract Execs::match_process

Add #[must_use] to Execs

Inline Execs::_with_stderr

Remove an unrun "cargo build" ProcessBuilder

Auto merge of #5939 - ehuss:test-package-docs, r=alexcrichton

Add documentation for creating test dependencies.

Using `Package` has always been a little unclear to me. I think I understand
it now. Maybe this will help.

Add documentation for creating test dependencies.

Using `Package` has always been a little unclear to me. I think I understand
it now. Maybe this will help.

Auto merge of #5935 - alexcrichton:vendor-paths, r=ehuss

Only use non-absolute paths for `path` dependencies

Previously Cargo would use a non-absolute path for any dependency contained
within the workspace root but this switches Cargo to only using relative paths
for `path` dependencies. In practice this shouldn't make much difference, but
for vendored crates and moving around `CARGO_HOME` it can produce more
consistent results when target directories are shared.

Closes #5923

Only use non-absolute paths for `path` dependencies

Previously Cargo would use a non-absolute path for any dependency contained
within the workspace root but this switches Cargo to only using relative paths
for `path` dependencies. In practice this shouldn't make much difference, but
for vendored crates and moving around `CARGO_HOME` it can produce more
consistent results when target directories are shared.

Closes #5923

Auto merge of #5628 - ehuss:metabuild, r=alexcrichton

Metabuild (RFC 2196)

Auto merge of #5937 - ehuss:fix-codemap, r=alexcrichton

Fix test failure on nightly due to `codemap::Span` change.

Update due to rust-lang/rust#52953.

Fix test failure on nightly due to `codemap::Span` change.

Update due to rust-lang/rust#52953.

New metabuild strategy using custom src_path enum.

- Use new enum `TargertSourcePath` for Target::src_path to make it explicit that metabuild has a special path.
- `cargo metadata` now skips the metabuild Target.
- JSON artifacts include the true path to the metabuild source file. This may not be the best solution, but it's unclear what it should be, and I would prefer to avoid breaking the output. Alternatively it could just not emit anything? I'm not completely familiar with the use case of these artifact messages.
- Place the file in `target/.metabuild/metabuild-pkgname-HASH.rs` instead of in the debug/release directory.  Its contents do not depend on the profile.
- Fix bug in write_if_changed.
- More tests.

Remove unnecessary change.

Address review comments.

- Add newline at end of file.
- Remove unnecessary rustfmt changes.
- Move file writing code to `write_if_changed`.

Metabuild (RFC 2196)

Auto merge of #5927 - dwijnand:uninstall-cwd, r=ehuss

Make "cargo uninstall" uninstall the cwd bins

Fixes #5916

Tested with a local build of cargo, using coreutils:

    17:33:57 $ dcargo uninstall
    Removing /Users/dnw/.cargo/bin/uutils

Handle Window's missing file error message

Make "cargo uninstall" uninstall the cwd bins

Auto merge of #5919 - RalfJung:mtime, r=alexcrichton

fix cargo not doing anything when the input and output mtimes are equal

That's a problem as the input may have changed in that same second but after the output got generated!

Fixes https://github.com/rust-lang/cargo/issues/5918

update comment based on further research

Auto merge of #5926 - alexcrichton:more-errors, r=dwijnand

List URL in HTTP download failures

Hopefully will help diagnose issues like #5924

Auto merge of #5925 - dwijnand:improve-install-cwd-deprecation, r=alexcrichton

Improve the `cargo install` deprecation messaging

Fixes #5917

r? @matklad
/cc @brson also as I'd love to hear your feedback too.

List URL in HTTP download failures

Hopefully will help diagnose issues like #5924

Fix compilation error

Apparently the bail! macro doesn't support trailing commas :-/

Improve the `cargo install` deprecation messaging

Auto merge of #5902 - little-arhat:feature-add-required-features-to-metadata-output, r=alexcrichton

Add `required-features` to `SerializedTarget`

This will add it to `cargo metadata` output and will make it
possible to enable features needed to build specific target.

Auto merge of #5914 - alexcrichton:system-git, r=dwijnand

Add a configuration option to fetch with git-the-CLI

Currently Cargo always uses `libgit2` to perform all fetches of git
repositories, but sometimes this is not sufficient. The `libgit2` library
doesn't support all authentication schemes that `git` does and it isn't always
quite at feature parity with `git` itself, especially in terms of network
configuration.

This commit adds a configuration option to Cargo for fetching git repositories
with the `git` CLI tool rather than the internal `libgit2`. While this exposes
us to changes over time in the `git` CLI it's hopefully a very targeted use case
that doesn't change much. The internal `libgit2` library should be sufficient
for all other forms of git repository management. (and using `git` for only
fetches shouldn't slow us down much)

The new configuration option in `.cargo/config` is:

    [net]
    git-fetch-with-cli = true

which can also be specified with `CARGO_NET_GIT_FETCH_WITH_CLI=true` via an
environment variable.

Closes #5903

attempt to fix test suite race on mac

Add a configuration option to fetch with git-the-CLI

Currently Cargo always uses `libgit2` to perform all fetches of git
repositories, but sometimes this is not sufficient. The `libgit2` library
doesn't support all authentication schemes that `git` does and it isn't always
quite at feature parity with `git` itself, especially in terms of network
configuration.

This commit adds a configuration option to Cargo for fetching git repositories
with the `git` CLI tool rather than the internal `libgit2`. While this exposes
us to changes over time in the `git` CLI it's hopefully a very targeted use case
that doesn't change much. The internal `libgit2` library should be sufficient
for all other forms of git repository management. (and using `git` for only
fetches shouldn't slow us down much)

The new configuration option in `.cargo/config` is:

    [net]
    git-fetch-with-cli = true

which can also be specified with `CARGO_NET_GIT_FETCH_WITH_CLI=true` via an
environment variable.

Closes #5903

fix cargo not doing anything when the input and output mtimes are equal

That's a problem as the input may have changed in that same second but after the output got generated!

Auto merge of #5886 - dekellum:record-package-git-head-3, r=alexcrichton

Generate .cargo_vcs_info.json and include in `cargo package` (take 2)

Implements #5629 and supersedes #5786, with the following improvements:

* With an upstream git2-rs fix (tracked #5823, validated and min version update in: #5858), no longer requires windows/unix split tests.

* Per review in #5786, drops file system output and locks for .cargo_vcs_info.json.

* Now uses serde `json!` macro for json production with appropriate escaping.

* Now includes a test of the output json format.

Possible followup:

* Per discussion in #5786, we could improve reliability of both the VCS dirty check and the git head sha1 recording by preferring (and/or warning otherwise) the local repository bytes of each source file, at the same head commit. This makes the process more appropriately like an atomic snapshot, with no sentry file or other locking required.  However given my lack of a window's license and dev setup, as exhibited by troubles of #5823, this feel intuitively like too much to attempt to change in one iteration here.  I accept the "best effort" concept of this feature as suggested in #5786, and think it acceptable progress if merged in this form.

@alexcrichton r?
@joshtriplett cc

Serialize required_dependencies as 'require-dependencies'

Auto merge of #5912 - rust-lang:dependabot/cargo/opener-0.3.0, r=alexcrichton

Update opener requirement from 0.2.0 to 0.3.0

Updates the requirements on [opener](https://github.com/Seeker14491/opener) to permit the latest version.
<details>
<summary>Changelog</summary>

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

> ## [0.3.0] - 2018-08-18
> ### Added
> - `stderr` field to `OpenError::ExitStatus` variant, which captures anything the failed process wrote to stderr.
>
> ## [0.2.0] - 2018-08-08
> ### Removed
> - The `open_sys` function, which was erroneously pub on non-Windows builds.
>
> ## [0.1.0] - 2018-08-08
> - Initial release.
</details>
<details>
<summary>Commits</summary>

- See full diff in [compare view](https://github.com/Seeker14491/opener/commits/v0.3.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 reopen` will reopen this PR if it is closed
- `@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 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)
- 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 opener requirement from 0.2.0 to 0.3.0

Updates the requirements on [opener](https://github.com/Seeker14491/opener) to permit the latest version.
- [Release notes](https://github.com/Seeker14491/opener/releases)
- [Changelog](https://github.com/Seeker14491/opener/blob/master/CHANGELOG.md)
- [Commits](https://github.com/Seeker14491/opener/commits/v0.3.0)

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

Auto merge of #5908 - orium:fix-package-cargo-toml, r=alexcrichton

Fix serialization bug in `edition` field of `TomlProject`.

Fixes #5906.

Auto merge of #5904 - dwijnand:backslash, r=alexcrichton

Remove backslash management in 2 tool_paths tests

With #5851 this is no longer necessary

Auto merge of #5907 - dwijnand:collapse-test-cmd-args, r=matklad

Collapse ProcessBuilder::arg calls in tests

Refs #5742
More of #5854

Fix 3 corner tests where whitespaces are important

Fix serialization bug in `edition` field of `TomlProject`.

Fixes #5906.

Move .env/.masquerade_as_nightly_cargo to collapse some more p.cargo calls

Collapse multiline ProcessBuilder::arg calls in tests

.. by calling this a bunch of times:

    fastmod --multiline '\.cargo\("([^"]+)"\).[ ]+\.arg\("([^"]+)"\)' '.cargo("${1} ${2}")' tests/testsuite/

Collapse ProcessBuilder::arg calls in tests

.. with mutliple calls of:

    fastmod --accept-all '\.cargo\("([^"]+)"\)\.arg\("([^"]+)"\)' '.cargo("${1} ${2}")' tests/testsuite/

until no changes are left.

Single slash root on Windows

Remove backslash management in 2 tool_paths tests

Add `required-features` to `SerializedTarget`

This will add it to `cargo metadata` output and will make it
possible to enable features needed to build specific target.

Auto merge of #5901 - dwijnand:1.28-win, r=alexcrichton

Align AppVeyor to use 1.28 for minimal versions

.. like Travis CI's setup.

Align AppVeyor to use 1.28 for minimal versions

.. like Travis CI's setup.

Auto merge of #5900 - orium:fix-broken-url, r=alexcrichton

Fix broken url in diagnostic.

Fix broken url in diagnostic.

Fix a clippy lint

Simplify json production and append to archive

Per github review.

Auto merge of #5856 - Eh2406:min-test, r=alexcrichton

run some tests with minimal-versions on CI

In #5757 we discovered that sum test don't pass with minimal-versions, and so only added CI for `cargo check`. This PR is to see if that is still needed, and if it is then which test rely on upstream bugfix.

Auto merge of #5888 - Seeker14491:opener, r=alexcrichton

Handle opening browser with `opener` crate

Fixes #5701

A note about behavior on Linux:
When looking for a browser to open the docs with, Cargo currently tries the `$BROWSER` environment variable, `xdg-open`, `gnome-open`, and finally `kde-open`, in that order. With this commit, this behavior changes; the `opener` crate always uses the `xdg-open` script, which is included in the `opener` crate. The `xdg-open` script takes care of finding a suitable browser.

stop test on appveyor as requested

bump min rust, and remove nightly check

DISABLE_CROSS_TESTS for minimal-versions

fix short_message_format vs nightly

Try serde_json

get `features_are_quoted` working with minimal-versions on CI

try running tests with minimal-versions on CI

Auto merge of #5889 - light4:patch-1, r=alexcrichton

Update transitioning url

Update transitioning url

Handle opening browser with `opener` crate

Fixes #5701

Auto merge of #5884 - dwijnand:some-clippings, r=alexcrichton

Resolve some clippy lint warnings

None

Auto merge of #5887 - Seeker14491:job-fix, r=alexcrichton

 Don't kill child processes on normal exit on Windows

Fix for #5598

Remove unused imports

Merge remote-tracking branch 'origin/master' into job-fix

Don't kill child processes on normal exit on Windows

Fixes #5598

Fix typo

Add test package::vcs_file_collision

Add generated .cargo_vcs_info.json (git hash) file to `cargo package`

Collapse let bindings

Resolve a drop_copy lint warning

Resolve a identity_conversion lint warning

Resolve a single_match lint warning

Resolve a const_static_lifetime lint warning