add failing test

Auto merge of #6493 - ehuss:fix-fingerprint-patch, r=alexcrichton

Fix fingerprint calculation for patched deps.

If you have A→B→C where B and C are in a registry, and you `[patch]` C, the fingerprint calculation wasn't working correctly when C changes. The following sequence illustrates the problem:

1. Do a build from scratch.
2. Touch a file in C.
3. Build again. Everything rebuilds as expected.
4. Build again. You would expect this to be all fresh, but it rebuilds A.

The problem is the hash-busting doesn't propagate up to parents from dependencies. Normal targets normally aren't a problem because they have a `LocalFingerprint::MtimeBased` style local value which always recomputes the hash. However, registry dependencies have a `Precalculated` style local value which never recomputes the hash.

The solution here is to always recompute the hash. This shouldn't be too expensive, and is only done when writing the fingerprint, which should only happen when the target is dirty. I'm not entirely certain why the caching logic was added in #4125.

Fixes rust-lang/rust#57142

Auto merge of #6512 - matklad:serde-version, r=dwijnand

serialize version directly

Streamline serialization more, cc #6511

Auto merge of #6355 - MarcusCalhoun-Lopez:dyld_fix, r=alexcrichton

use DYLD_FALLBACK_LIBRARY_PATH for dylib_path_envvar on macOS

See https://users.rust-lang.org/t/subprocess-and-dynamic-library-linking-problem-interaction/7873/10
See https://trac.macports.org/ticket/57692

Auto merge of #6510 - d-e-s-o:add-missing-noun, r=dwijnand

Fix error message when resolving dependencies

The error message:
> Attempting to resolve a with more then one crate with the links=

seems to be missing a noun. It appears that we are resolving a
dependency, so adding that.

serialize version directly

Auto merge of #6511 - matklad:metadata-path, r=dwijnand

use PathBuf in cargo metadata

I don't remember why we've used strings in the first place here, but I
think that historically there just weren't Serialize impls for
`Path`s!

use path when serializing a package

use PathBuf in cargo metadata

I don't remember why we've used strings in the first place here, but I
think that historically there just weren't Serialize impls for
`Path`s!

Fix error message when resolving dependencies

The error message:
> Attempting to resolve a with more then one crate with the links=

seems to be missing a noun. It appears that we are resolving a
dependency, so adding that.

Auto merge of #6506 - DamianX:ContributingTestLink, r=dwijnand

Fixed link to testsuite in CONTRIBUTING.md

Fixed link to testsuite in CONTRIBUTING.md

Auto merge of #6501 - vishallama:vishallama-doc-update-cargo.toml, r=dwijnand

Update display of contents of Cargo.toml

When creating a (binary) program/crate using 'cargo new', with the new 2018 version of Rust the autogenerated Cargo.toml file contains a couple of additional lines. These lines have to do with edition and dependencies.

Auto merge of #6502 - vishallama:vishallama-doc-update-cargo-1.toml, r=dwijnand

Update display of contents of Cargo.toml

The display of the contents of the Cargo.toml file needs to be updated, in keeping with the Rust 2018 edition.

Update display of contents of Cargo.toml

The display of the contents of the Cargo.toml file needs to be updated, in keeping with the Rust 2018 edition.

Update display of contents of Cargo.toml

When creating a (binary) program using 'cargo new', with the new 2018 version of Rust the autogenerated Cargo.toml file contains a couple of additional lines. These lines have to do with edition and dependencies.

Auto merge of #6495 - dwijnand:fixup-cargo-install-help, r=ehuss

Fixup cargo install's help message

None

Auto merge of #6497 - ehuss:test-require-output-check, r=dwijnand

testsuite: Require failing commands to check output.

This requires that commands that expect an error (using `with_status()`) also check some part of the output, to ensure that the test is working correctly. As evidence by the few tests fixed here, it is dangerous to assume that the correct error will trigger.

Some of these check rustc error messages which is generally undesirable since they aren't always stable, but I think in the few cases they shouldn't be much of a problem. If that seems too risky, I can make them more generic.

Auto merge of #6496 - k-nasa:delete_unnecessary_return, r=ehuss

Delete unnecessary 'return'

Unnecessary return keyword deleted as it is also in the title

testsuite: Require failing commands to check output.

Delete unnecessary space

refactor: delete unnecessary return keyword

Auto merge of #6494 - ehuss:fix-unused-patch-warning, r=dwijnand

Fix new unused patch warning.

The unused `[patch]` warning added in #6470 was misfiring if you build only a subset of a workspace that doesn't include the patch.

I'm about 80% confident this new logic is correct. It addresses the scenario I discovered, and I tested a bunch of commands (generate-lockfile, update, etc.) to make sure they all work. However, the many ways that `resolve_with_previous` is called is tricky to ensure every scenario works correctly.

Fixup cargo install's help message

Fix new unused patch warning.

Fix fingerprint calculation for patched deps.

use DYLD_FALLBACK_LIBRARY_PATH for dylib_path_envvar on macOS

When loading and linking a dynamic library or bundle, dlopen
searches in LD_LIBRARY_PATH, DYLD_LIBRARY_PATH, PWD, and
DYLD_FALLBACK_LIBRARY_PATH.
In the Mach-O format, a dynamic library has an "install path."
Clients linking against the library record this path, and the
dynamic linker, dyld, uses it to locate the library.
dyld searches DYLD_LIBRARY_PATH *before* the install path.
dyld searches DYLD_FALLBACK_LIBRARY_PATH only if it cannot
find the library in the install path.
Setting DYLD_LIBRARY_PATH can easily have unintended
consequences.

See https://users.rust-lang.org/t/subprocess-and-dynamic-library-linking-problem-interaction/7873/10
See https://trac.macports.org/ticket/57692

Auto merge of #6481 - ehuss:man-update, r=alexcrichton

Some minor documentation changes.

A few updates for things that changed recently, some things that were
wrong/missing, and hopefully some clarifications on some matters.

For reviewing, I recommend ignoring the auto-generated .html/.man files and focus on the .md/.adoc files.

Auto merge of #6480 - ehuss:metadata-links, r=alexcrichton

Add `links` to `cargo metadata`.

This is needed for creating registry index entries.

Some minor documentation changes.

A few updates for things that changed recently, some things that were
wrong/missing, and hopefully some clarifications on some matters.

Add `links` to `cargo metadata`.

This is needed for creating registry index entries.

Auto merge of #6476 - dwijnand:salvaged-semver-work, r=Eh2406

Salvaged semver work

None

Replace PackageId::wrap with PackageId::pure

Auto merge of #6470 - ehuss:warn-unused-patch, r=alexcrichton

Warn on unused patches.

Adds a warning when a `[patch]` entry fails to match anything.
I've hit this several times in the past, and it always takes me
5 minutes of confusion to figure out what's wrong.

Fixes #6444

Warn on unused patches.

Auto merge of #6473 - Eh2406:rustc-0, r=alexcrichton

don't write a an incorrect rustc version to the fingerprint file

In making https://github.com/holmgr/cargo-sweep/pull/14 I noted that some fingerprint files related to build scripts report being built with a rustc that hashes to "0". To work around this I just marked them as still being needed, even though no rustc that hashes to "0" is currently installed.

I believe that this PR just filles in the correct info for the build script fingerprints. This makes it possible for outside tools to more reliably clean up after cargo, at basically no cost.
@ehuss Thanks again for the help.

don't write a an incorrect rustc version to the fingerprint file

Auto merge of #6466 - ehuss:registry-cleanup, r=alexcrichton

Rewrite `login` and registry cleanups.

- Login:
    - Change `login` so that it reads the API host from the registry config so it knows the `{api}/me` URL to display.
    - The `--host` flag is deprecated/unused. It wasn't used for much before.
    - `--registry` supports interactive entry of the token (does not require it on the command line).
    - Displays a message after saving the token (fixes #5868).
    - Because `login` now requires an index, some of the tests had to be updated.
- Fix so that if `api` is missing from the config, it will display a nice error message instead of panicking with unwrap.

Rewrite `login` and registry cleanups.

Auto merge of #6462 - pavel-zhigulin:fix_issue_\96\96#6461, r=ehuss

[issue#6461] Fix cargo commands list

Fixes #6461 -  cargo autocompletion script

Remove tail call

Use ToSemver more

Auto merge of #6469 - ehuss:registry-name, r=dwijnand

Restrict registry names to same style as package names.

See https://github.com/rust-lang/cargo/issues/4688#issuecomment-341114657

Restrict registry names to same style as package names.

Auto merge of #6463 - dwijnand:rip-stale-bot, r=alexcrichton

Remove Stale bot's configuration

We couldn't get stale bot to operate as we'd like (poke up N
older-then-X-period tickets a day) and therefore we're tearing it down.

We've chosen to keep the "stale" label, so tickets closed by the bot
may be found with that label.

Closes #6035

Auto merge of #6464 - dwijnand:add-labels-to-issue-templates, r=Eh2406

Add labels to issue templates

Make use of https://blog.github.com/changelog/2018-12-05-issue-template-automation-improvements/

Add labels to issue templates

Make use of https://blog.github.com/changelog/2018-12-05-issue-template-automation-improvements/

Remove Stale bot's configuration

We couldn't get stale bot to operate as we'd like (poke up N
older-then-X-period tickets a day) and therefore we're tearing it down.

We've chosen to keep the "stale" label, so tickets closed by the bot
may be found with that label.

Use awk to parse cargo --list

[issue#6161] Fix cargo commands list

Auto merge of #6459 - ehuss:fix-man-checks, r=alexcrichton

Fix new man page links.

The rustc link checker found some problems:
- The raw generated HTML files were being included when they shouldn't.
  Fixed by moving them out of the mdbook directory.
- The `print.html` file concatenates all the pages together, causing duplicate
  `id`'s.  Fix by giving each page a unique set of IDs.

Fix new man page links.

The rustc link checker found some problems:
- The raw generated HTML files were being included when they shouldn't.
  Fixed by moving them out of the mdbook directory.
- The `print.html` file concatenates all the pages together, causing duplicate
  `id`'s.  Fix by giving each page a unique set of IDs.

Auto merge of #6432 - ehuss:fix-metabuild-json, r=alexcrichton

Fix metabuild compile errors with --message-format=json.

If an error occurs while compiling a metabuild target with `--message-format=json`, it would panic because it was unable to serialize `Target`. This change makes it so that it places a fake "metabuild.rs" string in the `src_path` in this situation.

I'm very unhappy with this solution, but I'm unable to think of something better. Changing `src_path` to an `Option` (or something) would break existing tools (this might break, but maybe not catastrophically?). I tried implementing something that resets the `src_path` to the correct path in the target dir after the workspace is configured, but it felt very brittle – you have to fix up after all dependencies are downloaded, and there's not a good place to ensure that happens correctly. Any alternate ideas?

This adds a `with_json_contains_unordered` to help with tests.

Auto merge of #6456 - ehuss:patch-alt-reg, r=alexcrichton

Support alt-registry names in [patch] table.

Closes #5149

Support alt-registry names in [patch] table.

Update metabuild test to 2018.

Switch SerializedTarget::src_path to None instead of using fake path.

Fix metabuild compile errors with --message-format=json.

If an error occurs while compiling a metabuild target with
`--message-format=json`, it would panic because it was unable to serialize
`Target`. This change makes it so that it places a fake "metabuild.rs" string in
the `src_path` in this situation.

I'm very unhappy with this solution, but I'm unable to think of something
better. Changing `src_path` to an `Option` (or something) would break existing
tools. I tried implementing something that resets the `src_path` to the correct
path in the target dir after the workspace is configured, but it felt very
brittle – you have to fix up after all dependencies are downloaded, and there's
not a good way to ensure that happens correctly.

This adds a `with_json_contains_unordered` to help with tests.

Auto merge of #6455 - wezm:patch-1, r=dwijnand

Update the rustup URL

Yesterday at the Melbourne Rust meetup I helped someone install Rust on their Mac. They were following the instructions from the cargo book and I was surprised to see `rustup` and all the tools being installed globally in `/usr/local/bin`. Eventually we worked out that the URL in the cargo book was different from the one on the Rust website and on rustup.rs. After using the other URL the install proceeded as expected, installing into the user's home directory. It appears the old link is an old version of rustup/rust-init.

This PR updates the install command to match the Rust home page.

Update the rustup URL

Auto merge of #6405 - ehuss:new-man, r=alexcrichton

New man pages.

This is a rewrite and update of the man pages. This also adds them to the website documentation. They are now in Asciidoc format to make it easy to output multiple formats and have decent formatting. There is a Makefile with instructions on how to rebuild the man pages.

Closes #5729.

Auto merge of #6451 - dwijnand:reify-DepFingerprint, r=alexcrichton

Reify the DepFingerprint type

I'm very slightly concerned I might've incompatibly changed the serialise/deserialise code with respect to existing serialised data, so please double-check my work.

Reify the DepFingerprint type

Auto merge of #6449 - dwijnand:extract-Fingerprint-new, r=alexcrichton

Extract Fingerprint::new

None

Auto merge of #6448 - dwijnand:upgrade-the-metabuild-to-rust-2018, r=ehuss

Upgrade the metabuild to Rust 2018

Just happened across the code that @ehuss mentioned in https://github.com/rust-lang/cargo/pull/6423#issuecomment-446807216 and thought of reviving that effort.

@ehuss says there's no reason to.  Let's see if it passes CI and we should land this change or not.

Auto merge of #6450 - dwijnand:make-edition-comparing-code-consistent, r=Eh2406

Make edition comparing code consistent

Rather than sometimes comparing equality, sometimes using ordering and
sometimes pattern matching, consistently compare by equalty to 2015.

Extract Fingerprint::new

Make edition comparing code consistent

Rather than sometimes comparing equality, sometimes using ordering and
sometimes pattern matching, consistently compare by equalty to 2015.

Upgrade the metabuild to Rust 2018

Auto merge of #6447 - ehuss:doc-name-authors, r=alexcrichton

Document `name` and `authors` in [package]

I don't think we need to take a principled stand on the exact meaning of
"authors", so I left it somewhat vague. Also, it was made optional in #3682 so
the existing docs were a little wrong.

Closes #5934

Document `name` and `authors` in [package]

Auto merge of #6443 - ehuss:ci-install-mdbook, r=alexcrichton

Travis: only use mdbook 0.1.7.

The latest mdbook does not build (https://github.com/rust-lang-nursery/mdBook/issues/852). Cargo uses 0.1.7, so force install only that version.

On CI only use mdbook 0.1.7.

Auto merge of #6439 - rust-lang:dependabot/cargo/git2-curl-0.9.0, r=alexcrichton

Update git2-curl requirement from 0.8.1 to 0.9.0

Updates the requirements on [git2-curl](https://github.com/alexcrichton/git2-rs) to permit the latest version.
<details>
<summary>Commits</summary>

- See full diff in [compare view](https://github.com/alexcrichton/git2-rs/commits)
</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 cancel merge` will cancel a previously requested merge
- `@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 git2-curl requirement from 0.8.1 to 0.9.0

Updates the requirements on [git2-curl](https://github.com/alexcrichton/git2-rs) to permit the latest version.
- [Release notes](https://github.com/alexcrichton/git2-rs/releases)
- [Commits](https://github.com/alexcrichton/git2-rs/commits)

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

Auto merge of #6438 - rust-lang:dependabot/cargo/git2-0.8.0, r=alexcrichton

Update git2 requirement from 0.7.5 to 0.8.0

Updates the requirements on [git2](https://github.com/alexcrichton/git2-rs) to permit the latest version.
<details>
<summary>Commits</summary>

- See full diff in [compare view](https://github.com/alexcrichton/git2-rs/commits/git2-curl-0.8.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 cancel merge` will cancel a previously requested merge
- `@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 git2 requirement from 0.7.5 to 0.8.0

Updates the requirements on [git2](https://github.com/alexcrichton/git2-rs) to permit the latest version.
- [Release notes](https://github.com/alexcrichton/git2-rs/releases)
- [Commits](https://github.com/alexcrichton/git2-rs/commits/git2-curl-0.8.0)

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

Auto merge of #6419 - ehuss:fix-show-errors, r=alexcrichton

Display errors when `cargo fix` fails.

It can be difficult to figure out what's wrong when a user reports that `cargo fix` fails. It can be hard to figure out which suggestion caused a compile error, especially if the error is in another file/location.

Auto merge of #6434 - ehuss:fix-fix-concurrent, r=dwijnand

cargo fix: fix targets with shared sources.

If `cargo fix` attempts to fix multiple targets concurrently that have shared
source files, it would apply fixes multiple times causing corruption of the
source code. Fix this by locking on the package path instead of the target
filename, essentially serializing all targets within a package.

Fixes #6415.

cargo fix: fix targets with shared sources.

If `cargo fix` attempts to fix multiple targets concurrently that have shared
source files, it would apply fixes multiple times causing corruption of the
source code. Fix this by locking on the package path instead of the target
filename, essentially serializing all targets within a package.

Update rustfix to 0.4.4.

Display errors when `cargo fix` fails.

It can be difficult to figure out what's wrong when a user reports that
`cargo fix` fails. There's often a large list of warnings, and it can
be hard to figure out which one caused a compile error.

Auto merge of #6431 - ehuss:panic-in-panic, r=nrc

Fix panic-in-panic in tests.

There are some very rare circumstances that can cause a double panic during
development. For example, `.with_json("")` (or any invalid JSON) will panic, and then the drop
will also panic.  This can cause a confusing SIGILL.

Fix panic-in-panic in tests.

There are some very rare circumstances that can cause a double panic during
development. For example, `.with_json("")` will panic, and then the drop
will also panic.

Auto merge of #6422 - dwijnand:more-rust-2018, r=alexcrichton

More Rust 2018 edition cleanups

None

Auto merge of #6429 - dwijnand:cleanup-SourceId-trait-impls, r=alexcrichton

Cleanup some trait impls for SourceId

None

Cleanup some trait impls for SourceId

Remove trailing extern crate usage

Remove trailing cfg_attr feature = "cargo-clippy" usage

Auto merge of #6427 - phansch:remove_nightly_check_in_test, r=dwijnand

Remove a nightly check from doc tests

None

Remove a nightly check from doc tests

Auto merge of #6425 - dwijnand:promote-failure-error, r=alexcrichton

Replace CargoError with failure::Error

None

Auto merge of #6426 - dwijnand:allow-testsuite-warnings-in-dev, r=nrc

Allow testsuite warnings in dev

This makes the deny(warnings) in the testsuite conditional on a new
"deny-warnings" feature, that is then enabled in CI.

Ideally I could use the (reasonably well established) CI env var (like
we do for proptests), but I don't know how to get the attribute to be
defined in terms of an env var.

Allow testsuite warnings in dev

This makes the deny(warnings) in the testsuite conditional on a new
"deny-warnings" feature, that is then enabled in CI.

Ideally I could use the (reasonably well established) CI env var (like
we do for proptests), but I don't know how to get the attribute to be
defined in terms of an env var.

Replace CargoError with failure::Error

Merge pull request #6371 from matklad/update-dry-run

add `--dry-run` option to cargo update

Auto merge of #6416 - alexcrichton:idioms, r=dwijnand

Migrate to some Rust 2018 idioms

Run a few lints, hand-edit a few things, otherwise try to help push Cargo into the 2018 edition!

Reformat after idiom lints

Turn on Rust 2018 idiom warnings unconditionally

It may take us awhile to get used to them, so let's see how this goes!

Run `cargo fix --edition-idioms` and fixup output

This gets Cargo passing the `--edition-idioms` lints and more down the
road of the 2018 edition!