Add rustfmt.toml with formatting disabled

Currently, Cargo does not use rustfmt for its source code. By adding a
rustfmt.toml that simply disables all formatting rules, editors which
use rustfmt by default for all Rust code will do the right thing and
leave the source code unchanged.

Auto merge of #4683 - djc:requirements, r=alexcrichton

Introduce Requirements struct to clarify code

`cargo::core::resolver::build_requirements()` previously, in this order:

* Defined local variables to track state
* Called a function to mutate the local variables
* Defined the aforementioned function
* Returned two out of three local variables as a tuple

This PR changes the code so that this is a recast as a struct (appropriately called `Requirements`), which is mutated in a more fine-grained way by its methods and acts as the return type for `build_requirements()`. To me, this seems a lot easier to understand.

This work was done in the context of #1286, but I figured it was easier to start landing some of the refactoring to avoid bitrot and get early (well, earlier) feedback.

Auto merge of #4684 - matklad:lazy-update, r=alexcrichton

Don't update lockfiles from previous Cargo versions if `--locked` is passed

Recently, we've stopped outputting `[root]` section to Cargo.lock files. The problem with this is that somebody might want to have a Cargo.lock file with a `[root]` section for compatibility with older Cargos, but at the same time use newer Cargo to build the crate. In this case, newer Cargo would remove `[root]` from the lockfile. Such updating of the lockfiles to the latest versions is a reasonable behavior, but it might be useful to be able to override it with `--locked` flag.

Context: https://github.com/rust-lang/cargo/issues/4563#issuecomment-340238773

r? @alexcrichton

Don't update lockfiles from previous Cargo versions if `--locked` is passed

Auto merge of #4676 - mgeisler:ci-caching, r=alexcrichton

Explain why caching is only done on $HOME/.cargo/bin/ in Travis

After having experimented with the Travis and AppVeyor caches, I concluded that they don't really help here: they're large and take a very long time to both download when the build starts and upload after it is finished.

Explain why we are not using 'cache: cargo' with Travis

Auto merge of #4685 - ehuss:all-targets-no-lib, r=alexcrichton

Fix --all-targets in a crate without a lib.

Fixes #4615.

Fix --all-targets in a crate without a lib.

Fixes #4615.

Auto merge of #4561 - nossralf:uninstall-many, r=matklad

Support uninstallation of multiple packages

This is a WIP pull request with support for uninstalling multiple packages. It mirrors the logic used for `cargo install`.

Fixes #4560

Use specific methods to add requirements

Move logic for walking over dependent features around

Move shared logic into separated methods

Extract method for handling feature requirements

Extract method for recording a requested dependency

Extract function to record if a feature was seen before

Extract method for requiring crate features

Move resolution requirement state into a separate struct

Auto merge of #4607 - sunjay:patcherror, r=alexcrichton

Improving the error message for when a patched dependency does not resolve to anything

Hi,
I just spent a long time debugging what this error message means and so I thought I would try to help improve it. Could someone who knows the codebase well look at this and see if this error message even makes sense here. I know this is the information I would have needed, but the error may occur in other cases where this message doesn't make sense. I'm open to any suggestions that would help make it more clear and maybe point people to where they should look for more information.

The specific problem that I ran into occurred when I was trying to upgrade the `rustfmt` submodule in the rust codebase. The `rustfmt-nightly` dependency changed version numbers and this conflicted with what was in the `Cargo.lock` file. After some detective work I was able to find the [documentation on `[patch]`](http://doc.crates.io/manifest.html#the-patch-section) which I had never read before and the following relevant paragraphs from [some other documentation](http://doc.crates.io/specifying-dependencies.html#testing-a-bugfix):

> Next up we need to ensure that our lock file is updated to use this new version of uuid so our project uses the locally checked out copy instead of one from crates.io. The way [patch] works is that it'll load the dependency at ../path/to/uuid and then whenever crates.io is queried for versions of uuid it'll also return the local version.
>
> This means that the version number of the local checkout is significant and will affect whether the patch is used. Our manifest declared uuid = "1.0" which means we'll only resolve to >= 1.0.0, < 2.0.0, and Cargo's greedy resolution algorithm also means that we'll resolve to the maximum version within that range. Typically this doesn't matter as the version of the git repository will already be greater or match the maximum version published on crates.io, but it's important to keep this in mind!

If it was not for the person who wrote those docs, I would not have known what to do here!

New error message with issue link

Improving the error message for when a patched dependency does not resolve to anything

Auto merge of #4592 - ehuss:check_test, r=alexcrichton

Add unit test checking to `cargo check`

This is an extension of PR #4039, fixing #3431, #4003, #4059, #4330.  The fixes for #4059 can potentially be separated into a separate PR, though there may be some overlap.

The general gist of the changes:

- Add `--profile test` flag to `cargo check` that will enable checking of unit tests.
- `--tests` will implicitly enable checking of unit tests within the lib (checks the same targets as `cargo test`).  This affects the `check`, `test`, and `build` commands.
- `--benches` behaves similarly by using the same targets as `cargo bench`.
- Fix erroneously linking tests when run with `--test`.

There is one thing I did not do because I wanted more feedback on what others think the expected behavior should be.  What should the behavior of `--all-targets` be?  This patch does not (yet) make any changes to its behavior.  My initial thinking is that it should *add* a target of `--lib --bins --profile test`, but that essentially means the lib and bin targets will be checked twice (and thus any errors/warnings outside of `#[cfg(test)]` will appear twice, which may be confusing, and generally take twice as long to run).  I can add that, but I think it would require adding a new `All` variant to `CompileFilter` so that the code in `generate_targets` can detect this scenario.  I wanted feedback before making a more extensive change like that.  The downside of not adding it is that `--all-targets` will ignore unit tests (if you don't specify `--profile test`).

Summary of the profiles used with this patch:

Command                         | Lib               | Bin foo     | Test t1 | Example e1 | Bench b1 |
-------                         | ---               | -------     | ------- | ---------- | -------- |
`check`                         | check             | check       | -       | -          | -        |
`check --profile test`          | check_test†       | check_test† | -       | -          | -        |
`check --lib`                   | check             | -           | -       | -          | -        |
`check --lib --profile test`    | check_test†       | -           | -       | -          | -        |
`check --bin foo`               | check             | check       | -       | -          | -        |
`check -–bin foo –profile test` | check_test†       | check_test† | -       | -          | -        |
`check --bins`                  | check             | check       | -       | -          | -        |
`check --test t1`               | check             | check       | check_test   | -          | -        |
`check --tests`                 | check, check_test†  | check, check_test† | check_test | check†, check_test†  | -    |
`check --all-targets`           | check, check_test†  | check, check_test†  | check_test   | check, check_test† | check_test    |

† = different behavior from today

Auto merge of #4675 - mgeisler:align-cmdline-examples, r=alexcrichton

Align command line examples in "cargo help test"

The initial example used an indentation of four spaces, the other
examples used only two spaces. They now all use four spaces.

I checked all other Cargo commands and found no other such problems.

Auto merge of #4677 - integer32llc:unpin-nightly, r=alexcrichton

Revert "Auto merge of #4659 - integer32llc:pin-nightly, r=alexcrichton"

This reverts commit 55442e8922ae7ad51545576a4f1326ab131a333a, reversing
changes made to 92a3a4efa27900e97a8ccb3c5c9b45f06f439454.

Testing to see if there is a fix on the latest nightly as rumored here: https://github.com/rust-lang/rust/issues/45500#issuecomment-340431032

Revert "Auto merge of #4659 - integer32llc:pin-nightly, r=alexcrichton"

This reverts commit 55442e8922ae7ad51545576a4f1326ab131a333a, reversing
changes made to 92a3a4efa27900e97a8ccb3c5c9b45f06f439454.

Auto merge of #4672 - kennytm:fix-4671, r=alexcrichton

When uplifting directories, symlink them instead of hard-link them.

Fixes #4671.

Align command line examples in "cargo help test"

The initial example used an indentation of four spaces, the other
examples used only two spaces. They now all use four spaces.

I checked all other Cargo commands and found no other such problems.

When uplifting directories, symlink them instead of hard-link them.

Fixes #4671.

Test partially successful uninstallation

Test bailout when specifying --bin during uninstall of multiple packages

Use multiple install command in the corresponding test

Bailout if --bin is specified when uninstalling multiple packages

Make it possible to uninstall multiple packages

Auto merge of #4665 - lukaslueg:issue4327, r=matklad

Discover vcs in new-cmd only if --vcs not set

If a commandline-argument is given, it takes precedence
over the config and existing vcs. Avoid trying to discover
existing repositories in that case, saving use from
shelling out to `hg`. Fixes #4327

Discover vcs in new-cmd only if --vcs not set

If a commandline-argument is given, it takes precedence
over the config and existing vcs. Avoid trying to discover
existing repositories in that case, saving use from
shelling out to `hg`. Fixes #4327

Auto merge of #4650 - main--:patch-1, r=matklad

Document that cargo test --release uses profile.bench

It certainly makes sense but it's still surprising behavior when the docs clearly state `cargo bench` = `profile.bench`, `cargo test` = `profile.test`. Had to dive into the code to figure this out.

Document that cargo test --release uses profile.bench

Auto merge of #4634 - kivikakk:cargo-env-prevails, r=matklad

Use argv[0] for cargo_exe so we don't rely on /proc on Linux

This is a proposed solution to #4450. I'm not at all wedded to the idea or the code, though, so feel free to shoot it down with abandon if this isn't something that'd work out or that you like.

In short, we use the existing `CARGO_ENV` (`"CARGO"`) if present, and only if not do we attempt to perform a lookup with `env::current_exe()` ourselves. This means users without access to `current_exe` (such as Linux without `procfs` mounted) can supply the `CARGO` env var themselves for external commands to use.

My concern here is: what if maybe we intentionally switch cargo binaries and didn't intend for this to happen? Could this ever happen outside a test environment? This kind-of-sorta-happened by accident in the test suite, necessitating the explicit removal of `CARGO_ENV` from the subprocess environment, because the actual cargo executing the test suite propagated its own path into the test subprocess!

/cc @alexcrichton as the originator of the idea of `CARGO_ENV`

Try to replicate shell behavior more accurately

Merge remote-tracking branch 'upstream/master' into cargo-env-prevails

Auto merge of #4644 - dethoter:4620, r=matklad

Fix documentation for package.readme option.

Fix for #4620.

Auto merge of #4506 - withoutboats:alt-registries, r=alexcrichton

Alternative registries

An implementation of alt registries. Still needs to get tested more thoroughly but seems to work.

Auto merge of #4627 - integer32llc:document-libs-and-bins, r=matklad

Document the lib if a lib and bin have the same name

Fixes #4341, as discussed in that issue.

- Removes the check that bailed if there was a  bin and lib with the
  same name
- Exclude bins with the same name as libs from the proposed targets to
  build when compiling docs
- Adjust tests to expect this behavior

Add more tests about documenting lib vs bins with the same name

Document the lib if a lib and bin have the same name

Fixes #4341.

- Removes the check that bailed if there was a  bin and lib with the
  same name
- Exclude bins with the same name as libs from the proposed targets to
  build when compiling docs
- Adjust tests to expect this behavior

Auto merge of #4649 - jongiddy:gitignore-newline, r=alexcrichton

Add a newline before appended VCS ignore lines

The `cargo init` command appends to `.gitignore` or `.hgignore` without checking if the last line of the existing file ends with a newline.

This change will insert a newline before the appended lines. If the last line has no newline, this will ensure the lines are kept separated. If the newline is present, this will insert a blank line. Both Git and Mercurial ignore blank lines in these files.

Auto merge of #4654 - rwakulszowa:inferred_test_cleanup, r=matklad

Clean up inferred* tests

Deduplicate, simplify and rename interred_* tests.
Follow up to #4496
@matklad

Auto merge of #4659 - integer32llc:pin-nightly, r=alexcrichton

try pinning to a nightly of two weeks ago

This passed CI [over here](https://github.com/rust-lang/cargo/pull/4658) and should get the 32-bit windows cross compiling tests passing until someone figures out why rustc is broken for that purpose

r? @alexcrichton

cc https://github.com/rust-lang/rust/issues/45500

Update alt-registry tests to the use the new interface.

Merge branch 'master' of github.com:rust-lang/cargo into alt-registries

try pinning to a nightly of two weeks ago

Add tests of transitive dependencies.

Clean up inferred* tests

Adding test for alternate registry dependency, with alternate registry dependency.

Switch to use registry in the dependency so that it matches RFC2141.

Sorting out file permissions.

Add support for other registries in dependencies.

Adding some extra tests.

Document that cargo test --release uses profile.bench

Add a newline before appended VCS ignore lines

Don't change order of tests and benches for cleaner diff.

Fix broken bench test for new --benches behavior.

Rework handling of --tests and --benches with implied targets.

Comments!

Fix documentation for package.readme option.

Auto merge of #4623 - alexcrichton:remove-lock, r=matklad

Delete Cargo.lock from this repo

There's now a lock file upstream in rust-lang/rust so the one here isn't
actually used, and otherwise this crate is used as a dependency so the lock file
isn't respected anyway!

Auto merge of #4637 - kivikakk:vers-version, r=alexcrichton

Allow cargo install --version as well (preferred)

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

The usage looks pretty ugly with this extra line in it, but it's not really avoidable with our docopt use.

We fail if both are provided; accepting both feels like a path to demons.

Try current_exe, then argv0 canonicalize, then PATH

Delete Cargo.lock from this repo

There's now a lock file upstream in rust-lang/rust so the one here isn't
actually used, and otherwise this crate is used as a dependency so the lock file
isn't respected anyway!

Auto merge of #4638 - efyang:docs-cc, r=alexcrichton

Update docs to use cc instead of gcc

Closes #4626.

Auto merge of #4633 - kivikakk:readme-file, r=carols10cents

transmit: send README filename as well as content

This is required to solve https://github.com/rust-lang/crates.io/issues/995; we currently only send the README content, but not the name of the README itself, so it's not possible to determine how we should render it.

I've confirmed the existing crates.io server silently ignores the new field, so this should be safe to roll out whenever.

Update docs to use cc instead of gcc

Let's not accept both, just to be safe

Allow --version as well (preferred)

Auto merge of #4635 - dethoter:4430, r=alexcrichton

Fix a documentation for package.build option.

Fix for #4430.

Fix a documentation for package.build option.

Auto merge of #4630 - ehuss:new-test-temp-dir, r=alexcrichton

Fix `new` tests on Windows inadvertently picking up ~/.cargo/config.

Fixes #4601.  By relying on `__CARGO_TEST_ROOT`, using a temp directory is no
longer necessary.

transmit: send README filename as well as content

Fix `new` tests on Windows inadvertently picking up ~/.cargo/config.

Fixes #4601.  By relying on `__CARGO_TEST_ROOT`, using a temp directory is no
longer necessary.

Auto merge of #4628 - pixelistik:patch-1, r=alexcrichton

Fix typo in docs

Fix typo in docs

Auto merge of #4619 - camjackson:patch-1, r=alexcrichton

Document how to make `build.rs` write to the terminal

Documenting the solution mentioned here: https://github.com/rust-lang/cargo/issues/985

Auto merge of #4624 - alexcrichton:bump, r=alexcrichton

Bump to 0.24.0

It's that time of the year again!

Bump to 0.24.0

It's that time of the year again!

Document how to make `build.rs` write to the terminal

Fix wrong profile selection for `cargo build` and `cargo rustc`.

Fix broken tests due to different profile selection.

Add unit test checking to `cargo check`

- Add `--profile test` flag to `cargo check` that will enable checking of unit tests.
- `--tests` will implicitly enable checking of unit tests within the lib.
- Don't implicitly compile binaries when using just `--test` filters.
- Fix erroneously linking tests when run with `--test`.

Fixes #3431, #4003, #4059, #4330.

Auto merge of #4616 - kennytm:fix-4490, r=alexcrichton

Uplift *.dSYM

Fixed #4490.

The solution is based on #4570. Simply adding `.dSYM` into `add_target_specific_suffixes` will cause cargo trying to actually run that `.dSYM` folder, so I've upgraded the `linkable` boolean into a 3-value enum `TargetFileType`, to tell `cargo run` and `cargo test` to avoid these debug symbol files.

(I haven't checked if it can solve #4056, don't wanna mess with Spotlight 😝)

Auto merge of #4618 - integer32llc:config-in-home, r=alexcrichton

Clarify that $HOME/.cargo is always checked

Clarify that $HOME/.cargo is always checked

Uplift *.dSYM. Fix #4490.

Auto merge of #4610 - hdhoang:patch-1, r=alexcrichton

layout: fix typo, and improve pronoun

"it" is ambiguous between `new` and `at` (the most recent noun).

Auto merge of #4314 - nodakai:phantom-type-in-projectbuilder, r=matklad

cargotest/support: remove internal mutability in favor of switching types

`Cell<bool>` was removed from `ProjectBuilder`. Instead `PhantomData<T>` was added to it (and `RepoBuilder`) to manage state transition in the type level (`T` is either `Configuring` or `Done`.)
`ProjectBuilder::cargo_process()` was removed as its design heavily depended on the internal mutability.

Also added `#[must_use]` to `ProjectBuilder` and `RepoBuilder` to check for call sites of their `build()` method.

layout: fix typo, and improve pronoun

"it" is ambiguous between `new` and `at` (the most recent noun).

Auto merge of #4608 - vmx:patch-1, r=alexcrichton

Remove wrong information about git repository

Dependencies cannot be a git repository when published to crates.io.

Remove wrong information about git repository

Dependencies cannot be a git repository when published to crates.io.

Auto merge of #4594 - behnam:workspace, r=alexcrichton

[core/workspace] Create WorkspaceRootConfig

Create `WorkspaceRootConfig`, which knows its `root_dir` and lists of
`members` and `excludes`, to answer queries on which paths are a member
and which are not.

----

This is the first step of the fix, that's only a codemod to put together the relevant parts : `workspace.members`, `workspace.excludes`, and the `root_dir` (where `members` and `excludes` are relative to). There's no logic change in this PR to keep review easier.

The added tests are commented out, because they fail with the current logic. The next step to get these steps to pass.

Tracker: <https://github.com/rust-lang/cargo/issues/4089>
Old PR: <https://github.com/rust-lang/cargo/pull/4297>

Auto merge of #4581 - tamird:master, r=alexcrichton

cargo_rustc: remove workaround for fixed upstream issue

Fixed in https://github.com/rust-lang/rust/pull/25411. Also, the
removed code is implicated in test failures observed in
https://github.com/rust-lang/rust/pull/44515.

r? @alexcrichton

Auto merge of #4602 - ehuss:fix-new-test-windows, r=alexcrichton

Allow test to pass if user has ~/.gitconfig on Windows.

The `new` test will fail on Windows if you have `~/.gitconfig` configured with a username/email.  This is because libgit2 searches quite a few environment variables hunting for a config file.  Rather than adjusting other environment variables (I think at a minimum it would be `%HOMEDRIVE%%HOMEPATH%` and `%USERPROFILE%`), this just adds an empty config file since libgit2 will stop at the first one it finds.

cargo_rustc: remove workaround for fixed upstream issue

Fixed in https://github.com/rust-lang/rust/pull/25411. Also, the
removed code is implicated in test failures observed in
https://github.com/rust-lang/rust/pull/44515.