Bump Cargo to 0.30.0

Looks like we accidentally forgot to do this!

Auto merge of #5989 - alexcrichton:change-default, r=Mark-Simulacrum

Change the default edition for `cargo new` to 2018

As it says on the tin! Some tests were updated to explicitly pass 2015 so they
can continue to work on stable, and otherwise `cargo new` should now by default
generate a 2018 project.

Change the default edition for `cargo new` to 2018

As it says on the tin! Some tests were updated to explicitly pass 2015 so they
can continue to work on stable, and otherwise `cargo new` should now by default
generate a 2018 project.

Auto merge of #5984 - alexcrichton:stabilize-edition, r=ehuss

Stabilize `edition` key and add `cargo new --edition`

This commit stabilizes the `edition` key in `Cargo.toml`, both in the
`[package]` section and inside subtargets. Additionally the `cargo new` and
`cargo init` subcommands have been enhanced with a `--edition` flag to allow
explicitly specifying the edition to be generated.

This commit does not yet change the default edition that's generated.

Closes #5980

Stabilize `edition` key and add `cargo new --edition`

This commit stabilizes the `edition` key in `Cargo.toml`, both in the
`[package]` section and inside subtargets. Additionally the `cargo new` and
`cargo init` subcommands have been enhanced with a `--edition` flag to allow
explicitly specifying the edition to be generated.

This commit does not yet change the default edition that's generated.

Closes #5980

Auto merge of #5987 - dwijnand:remove-cwd-root, r=alexcrichton

Remove/rework some redundant .cwd(p.root()) in tests

Just a little trim to the tests.

Auto merge of #5985 - dwijnand:fix-rustc-wrapper-unset, r=alexcrichton

Don't use an empty RUSTC_WRAPPER

Fixes #5588

Don't use an empty RUSTC_WRAPPER

Simplify test::install_ignores_cargo_config

Fix change in install::install_ignores_cargo_config

support::git::Repository doesn't support .cargo (yet?)

Auto merge of #5983 - alexcrichton:fix-rustc-wrapper, r=dwijnand

fix: Fix unusual errors with `RUSTC_WRAPPER`

This commit fixes the interaction of `cargo fix` and `RUSTC_WRAPPER`, ensuring
that Cargo at least doesn't die internally. For now `RUSTC_WRAPPER` is
overridden for normal execution but we can eventually one day probably support
`RUSTC_WRAPPER`!

Closes #5981

Remove/reworksome redundant .cwd(p.root()) in tests

fix: Fix unusual errors with `RUSTC_WRAPPER`

This commit fixes the interaction of `cargo fix` and `RUSTC_WRAPPER`, ensuring
that Cargo at least doesn't die internally. For now `RUSTC_WRAPPER` is
overridden for normal execution but we can eventually one day probably support
`RUSTC_WRAPPER`!

Closes #5981

Auto merge of #5977 - ehuss:timeout-doc, r=alexcrichton

Fix http.timeout documentation.

This was changed in #2770, but the docs were never updated.

Fix http.timeout documentation.

This was changed in #2770, but the docs were never updated.

Auto merge of #5976 - ehuss:git-cli-test-win, r=alexcrichton

Add a way to skip the `use_the_cli` test for rust repo.

Add a way to skip the `use_the_cli` test for rust repo.

Auto merge of #5973 - lostiniceland:5717-update-docs, r=alexcrichton

Do not use a disabled-option in the docs

Updates rust-lang/cargo#5717

Since people tend to copy/paste, the docs should not use the option which will disable curls low-speed-limit. Rather use a value lower than the default of 10

Also reverts removal of default low-speed-time (30s) in case no timeout is configured

Revert removal of default-value for curl low-speed-time

In case no timeout is specified a default value is needed.

Do not use a disabled-option in the docs

Updates rust-lang/cargo#5717

Since people tend to copy/paste, the docs should not use the option which will disable curls low-speed-limit. Rather use a value lower than the default of 10

Auto merge of #5970 - tbu-:pr_doc_fixlink, r=alexcrichton

Fix a doc link

Auto merge of #5971 - dwijnand:one-architecture, r=alexcrichton

Migrate some architecture notes from the wiki

Merge https://github.com/rust-lang/cargo/wiki/Cargo's-architecture into
the ARCHITECTURE.md in the repo.

Fixes #5528

Migrate some architecture notes from the wiki

Merge https://github.com/rust-lang/cargo/wiki/Cargo's-architecture into
the ARCHITECTURE.md in the repo.

Fix a doc link

Auto merge of #5957 - lostiniceland:5717-curl-low_speed, r=alexcrichton

Provide http-config option for curl low-speed-limit/time

Fixes rust-lang/cargo#5717

Provides new override options for curl in Cargo http config-section.

```
[http]
timeout = 2
low-speed-limit = 5
```

Test with the following during crate-download
`sudo tc qdisc add dev eth0 handle 1: root htb default 11 && sudo tc class add dev eth0 parent 1: classid 1:1 htb rate 8bps && sudo tc class add dev eth0 parent 1:1 classid 1:11 htb rate 8bps`

Clear with
`sudo tc qdisc del dev eth0 root`

Auto merge of #5966 - matthiaskrgr:clippy_3, r=dwijnand

clippy: resolve all warnings about useless format!()

Provide http-config option for curl low-speed-limit

Fixes rust-lang/cargo#5717

low-speed-time was removed because the corresponding curl-option is already set via http.timeout.

removed unnecessary Option.

clippy: resolve all warnings about useless format!()

Auto merge of #5874 - japaric:revert-revert-gh5606, r=alexcrichton

make `cargo install` ignore .cargo/config

closes #5850

r? @alexcrichton

Provide http-config option for curl low-speed-limit/time

Fixes rust-lang/cargo#5717

fix unit test

drop CompileMode::Install; override target in `exec`

add Install to all_modes

fix test source

add a note to the reference

make `cargo install` ignore .cargo/config

closes #5850

Auto merge of #5944 - dwijnand:fix-force-rebuild, r=alexcrichton

Force `cargo fix` to rebuild

Fixes #5736

This is a resubmit of @killercup's #5750, rebased on current master.

@alexcrichton From browsing the code I feel like `-p` would still restrict the packages to rebuild, despite the rebuild flag added. But I might be misreading or not-fully-reading the code. Could you give me some mentoring instructions for the test cases you're concerned with?

Auto merge of #5952 - dwijnand:fix-path-deep_dependencies_trigger_rebuild, r=alexcrichton

Fix path::deep_dependencies_trigger_rebuild often failing in CI

A shallow fix for `path::deep_dependencies_trigger_rebuild` in particular as it's failed my PRs often.

See https://github.com/rust-lang/cargo/pull/5935#issuecomment-417148372, and #5940 for the bigger picture.

Auto merge of #5950 - dwijnand:cwd, r=alexcrichton

Introduce the CWD macro in test output asserting

Avoids dealing with things like CWD changing.

Mentioned in #5742

Introduce the CWD macro in test output asserting

Avoids dealing with things like CWD changing.

Fix path::deep_dependencies_trigger_rebuild often failing in CI

Assert fix::doesnt_rebuild_dependencies stderr fully

Gives more confidence.

Only force rebuild of top-level units, not of unit dependencies

Auto merge of #5949 - dwijnand:extract-FixArgs-prepare_for_edition_resolve, r=alexcrichton

Extract FixArgs::prepare_for_edition_resolve

A little code dedup from working on #5948.

Test that cargo fix doesn't aggressively rebuild

Extract FixArgs::prepare_for_edition_resolve

Auto merge of #5943 - dwijnand:fix-allow-staged, r=alexcrichton

Add `--allow-staged` to `cargo fix`

Fixes #5737

This is @jljusten's branch, adapted to the testsuite changes in master.
Submitted as an alternative to #5910 to expedite #5737 resolution in time for Edition RC 1.

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

Remove Cargo's internal `hamcrest` module

Fixes #5742

Actually delete the hamcrest module..

Remove further references to hamcrest in the docs

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.

Always rebuild targets when using cargo-fix

Fix typo in `cargo fix --help`

Merge branch 'master' into fix-allow-staged

* master: (25 commits)
  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
  Add documentation for creating test dependencies.
  Only use non-absolute paths for `path` dependencies
  Fix test failure on nightly due to `codemap::Span` change.
  New metabuild strategy using custom src_path enum.
  Remove unnecessary change.
  Address review comments.
  Metabuild (RFC 2196)
  Handle Window's missing file error message
  Make "cargo uninstall" uninstall the cwd bins
  update comment based on further research
  List URL in HTTP download failures
  Fix compilation error
  Improve the `cargo install` deprecation messaging
  ...

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.