Make windows CI happy

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

print the full destination path when no track duplicates

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Auto merge of #9846 - ehuss:fix-only-edition-lints, r=alexcrichton

Change `cargo fix --edition` to only fix edition lints.

This changes it so that `cargo fix --edition` will only fix edition lints.  The reason for this is that sometimes non-edition lints get in the way, and make suggestions that can cause failures.  An example is a user that only ever runs `cargo test` or `cargo check --profile=test` locally, and doesn't realize there are problems with running without `cfg(test)` such as unused warnings.

This works by using `--cap-lints=allow` along with `--force-warn` which takes precedence over `cap-lints`.

This only works on nightly since `--force-warn` is still unstable.  I will update this as part of #9800.

Closes #5738

Auto merge of #9848 - nipunn1313:desc, r=alexcrichton

Show desc of well known subcommands (fmt, clippy) in cargo --list

Fixes #8680

An approach to #8680 that shows these in `cargo --list` without showing them directly in the `cargo --help`.

```
➜  cargo git:(desc) target/debug/cargo --list | grep clippy
    clippy               Checks a package to catch common mistakes and improve your Rust code.
```

Here's what mine looks like visually now:

Show desc of well known subcommands (fmt, clippy) in cargo --list

Change `cargo fix --edition` to only fix edition lints.

Auto merge of #9843 - ehuss:fix-test-cargo-env, r=alexcrichton

Fix test not to rely on `cargo` in PATH.

This fixes a test that was trying to execute `cargo` from PATH.  This test doesn't work on rust-lang/rust where the rustup installation is removed, and thus there is no `cargo` in PATH.

Fix test not to rely on `cargo` in PATH.

Auto merge of #9827 - weihanglo:issue-6199, r=Eh2406

Improve resolver message to include dependency requirements

Resolves #6199.

Thanks for previous efforts: #5452, #6374, #6665, which are great but somehow outdated, so I tweak them and create this PR. This will also be obsolete if we ship pubgrub-rs with cargo in the future 😃 But before that happens, IMO these changes are still helpful.

---

This PR changes the resolver error message from

https://github.com/rust-lang/cargo/blob/216f915c46b8ada2323423d049314ba18247ef95/tests/testsuite/build.rs#L1104-L1106

to

https://github.com/rust-lang/cargo/blob/0afd40b4de17a5c45145a0762beb4ef001720fe1/tests/testsuite/build.rs#L1104-L1106

Also provide different message for different source kinds, such like:

https://github.com/rust-lang/cargo/blob/0afd40b4de17a5c45145a0762beb4ef001720fe1/tests/testsuite/build.rs#L2810-L2812

## TODO?

From https://github.com/rust-lang/cargo/pull/5452#issuecomment-402832200, there shall be at least one task left behind:

> 3. Special case pind by a lock file and not a `"=1.1.2"` in a dependency. Also add a "note: try cargo update" to the end.

In this PR, `validate_links` also faces this issue that a dependency requirement is locked into a precise version `=0.1.0`.

https://github.com/rust-lang/cargo/blob/a5f8bc94f5d38539dd127f735ea4d3a515c230fd/tests/testsuite/build_script.rs#L1002-L1004

I am uncertain about how to resolve this. Besides  the function`validate_links`, is this problem really a thing that may happen? If not, since `validate_links` only handles old validation logic, it may be ok to drop the commit a5f8bc94f5d38539dd127f735ea4d3a515c230fd and leave it as is.

Auto merge of #9836 - Firstyear:9804-cargo-docs-improve, r=alexcrichton

Add hint for cargo metadata in environment section

Fixes #9804

Add hint for cargo metadata in environment section

Auto merge of #9834 - ehuss:fix-build-std-proc-macro-only, r=alexcrichton

Fix panic with build-std of a proc-macro.

If you try to run `cargo build -Zbuild-std` in a proc-macro project, cargo would panic in [`check_collisions`](https://github.com/rust-lang/cargo/blob/835d5576e1f0b917a42ba18a9556242204377cd1/src/cargo/core/compiler/context/mod.rs#L427). This is because it iterates over every Unit in the build graph checking the `outputs` for filenames.  However, [`outputs`](https://github.com/rust-lang/cargo/blob/835d5576e1f0b917a42ba18a9556242204377cd1/src/cargo/core/compiler/context/compilation_files.rs#L109-L110) was missing the outputs for standard library units. That is because `outputs` is computed by walking the graph starting from the roots.

The bug here is that `attach_std_deps` was adding the standard library units to graph, even though they aren't reachable from the roots, thus creating orphans.

The solution is to avoid adding the standard library units if they are not needed (as is the case when building just a proc-macro).

Fixes #9828

Auto merge of #9821 - steffahn:a_an, r=Eh2406

Fix typos “a”→“an”

See rust-lang/rust#88230

Auto merge of #9832 - Tindarid:patch-1, r=ehuss

Fix typo in git-authentication.md

Fix panic with build-std of a proc-macro.

Auto merge of #9831 - ehuss:fix-debug, r=Eh2406

Add some debug logging for `cargo fix`

This adds some debug logging to the `cargo fix` command to give better insight into what it is actually doing. I've needed to try to debug some things recently where this would have been useful, since cargo runs rustc many times with different arguments.  I think this will be useful if other people report problems, we can ask them to run with `CARGO_LOG=cargo::ops::fix=trace` to gather more information.

Auto merge of #9830 - ehuss:document-third-party-registries, r=alexcrichton

Add documentation about third-party registries.

This adds a link to the wiki where a list of third-party registries can be maintained.

Auto merge of #9818 - hi-rustin:rustin-patch-fix, r=alexcrichton

unset the FIX_ENV when executing the real rustc

close https://github.com/rust-lang/cargo/issues/9706

Improve resolver message for validate_links

Update tests to display dep-req info for dep-chain

Display dep requirement info for cyclic dependencies

Add dep requirement info for dep-chain display

Auto merge of #9801 - arlosi:checksum, r=alexcrichton

Allow crate download by checksum

The `dl` key in `config.json` currently allows the following substitutions: {crate}, {version}, {prefix}, {lowerprefix}.

This change adds a {checksum} placeholder for the crate's sha256 checksum. Does not change any existing behavior.

Allowing downloads by checksum makes it possible for crate files to be placed in a content addressable store.

Auto merge of #9792 - weihanglo:issue-9782, r=ehuss

Emit warning for migrating to unstable edition in stable channel

Resolves #9782

Also alter the original error message slightly.

Fix typo in git-authentication.md

Auto merge of #9771 - hi-rustin:rustin-patch-dep, r=ehuss

Warning for no lib dependencies

close https://github.com/rust-lang/cargo/issues/6702

Add some debug logging for `cargo fix`

Auto merge of #9824 - ehuss:disable-std_docs, r=alexcrichton

Temporarily disable extern-html-root-url test.

A change in https://github.com/rust-lang/rust/pull/82776 broke this test,
so disabling for now until it is figured out how things are going to work.

Add documentation about third-party registries.

Temporarily disable extern-html-root-url test.

A change in https://github.com/rust-lang/rust/pull/82776 broke this test,
so disabling for now until it is figured out how things are going to work.

Fix more typos, “an”→“a” and “an”→“and”

Fix another typo “a”→“an”

Fix typos “a”→“an”

Better fmt for file body

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

unset the FIX_ENV when executing the real rustc

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Auto merge of #9814 - ehuss:move-tmp, r=alexcrichton

Move `tmp` test directory.

The `tmp` directory added in #9375 was placed within the profile directory (such as `target/debug/tmp` or `target/release/tmp`).  This causes problems for any cargo target (binary, test, etc.) with the name `tmp` as there is a name collision.  This PR attempts to address that by moving the `tmp` directory to the root of the target directory (`target/tmp`), and reserving the profile name "tmp".

Fixes #9783

Auto merge of #9813 - ehuss:fix-license-file, r=alexcrichton

Fix test incorrectly validating CARGO_PKG_LICENSE_FILE.

This test had two flaws. One is that it used the wrong key name for `license-file`, and the second was that it wasn't checking the value was set.

Fix test incorrectly validating CARGO_PKG_LICENSE_FILE.

Move `tmp` test directory.

Auto merge of #9774 - Aaron1011:future-incompat-config, r=ehuss

Implement `[future-incompat-report]` config section

Currently, I've just implemented the `always` and `never` frequencies
from the RFC, which don't require tracking any additional state.

cc `@ehuss`

Auto merge of #9809 - ehuss:bump-curl, r=Eh2406

Bump curl.

This updates to the latest version of curl (7.78).  7.77 which is used in rust-lang/rust had an issue where it was failing in Windows 8.  I have confirmed that 7.78 works correctly.

Fixes #9788

Bump curl.

Implement `[future-incompat-report]` config section

Currently, I've just implemented the `always` and `never` frequencies
from the RFC, which don't require tracking any additional state.

Auto merge of #9793 - nipunn1313:install_parallel2, r=alexcrichton

Determine packages to install prior to installing

Old logic (pseudocode)
```
for krate in to_install {
    pkg = determine_pkg(krate);
    install_pkg(pkg);
}
```
New logic
```
let pkgs = to_install.into_iter(|krate| determine_pkg(krate));
pkgs.into_iter(|pkg| install_pkg(pkg));
```

This has the short term benefit of dumping most error messages out earlier in the process (eg a typo in the second package name).

Longer term, it might help with #9741 - as only the second loop would be parallelized. First loop shouldn't be parallelized because it would lead to redundant registry/git updates.

Factor into struct InstallablePackage

Auto merge of #9803 - ehuss:fix-edition-resolve-differences-dev-deps, r=alexcrichton

Show feature resolver differences for dev-dependencies.

During the crater run for 2021, there was a package that failed to update in a confusing way. The issue is that a feature was removed in the new resolver, but only for a dev-dependency. The report displayed with `cargo fix --edition` did not say anything about that, so it took me a bit to figure it out.  This changes it so that the report also includes changes to features of dev-dependencies.  I honestly don't remember my thought process behind the original code.

For example, the offending package now says:

```
When building the following dependencies, the given features will no longer be used:

  log v0.4.8 removed features: std
  syn v0.15.44 (as host dependency) removed features: extra-traits, visit

The following differences only apply when building with dev-dependencies:

  phf_shared v0.7.24 (as host dependency) removed features: unicase
```

And the error that happens after updating to 2021 is:

```
error[E0277]: the trait bound `UniCase<&str>: phf_shared::PhfHash` is not satisfied
  --> /Users/eric/.cargo/registry/src/github.com-1ecc6299db9ec823/mime_guess-1.8.7/build.rs:37:21
   |
37 |         forward_map.entry(UniCase(key), &format!("{:?}", val));
   |                     ^^^^^ the trait `phf_shared::PhfHash` is not implemented for `UniCase<&str>`
```

Hopefully developers will be able to see the note about the feature `unicase` being removed from `phf_shared`, and the error message about `UniCase` not implementing `PhfHash`, and connect the two together.  Previously, the upgrade report didn't mention anything about `phf_shared`, and thus no clues on what went wrong.

Show feature resolver differences for dev-dependencies.

Auto merge of #9802 - joshtriplett:triagebot-windows, r=alexcrichton

Support using rustbot to ping the Windows group

Support using rustbot to ping the Windows group

Allow crate download by checksum

The `dl` key in `config.json` currently allows the following substitutions:
{crate}, {version}, {prefix}, {lowerprefix}.

This change adds a {sha256-checksum} placeholder for the crate's sha256 checksum.

Allowing download by checksum makes it easier for crate files to be placed
in a content addressable store.

Auto merge of #9799 - ehuss:fix-abnormal-error, r=alexcrichton

Show information about abnormal `fix` errors.

During a recent crater run, we ran into a few circumstances where `cargo fix` failed unexpectedly, and we can't reproduce the errors locally.  The sequence was:

1. Cargo ran `rustc` and collected the diagnostics to apply, and modified the files.
2. Cargo ran `rustc` again to verify the fixes. This step failed, but only emitted warnings.
3. Cargo ran `rustc` again to show the original diagnostics, and this exited normally with warnings.

We don't know why the second step failed.  This change makes it so that cargo will collect any non-diagnostic messages (like ICEs), and will also display the exit code if it is abnormal.

Show information about abnormal `fix` errors.

Auto merge of #9798 - ehuss:bump-jobserver, r=alexcrichton

Bump jobserver.

Pulls in https://github.com/alexcrichton/jobserver-rs/pull/35 to fix https://github.com/rust-lang/rust/issues/88091.

Bump jobserver.

Auto merge of #9795 - johnthagen:patch-2, r=ehuss

Render build-std web links as hyperlinks

Several hyperlinks in the unstable docs page are not rendered properly as links when the documentation is built. This PR addresses this.

<img width="622" alt="Screenshot 2021-08-16 201532" src="https://user-images.githubusercontent.com/10340167/129644467-2c8ec535-3d44-497e-8fba-d8364b794faf.png">

Fix an additional link

Render build-std web links as hyperlinks

Refactor additional validation steps to determine_package

Refactor install_one into two parts

Auto merge of #9791 - nipunn1313:cargo_crash_alias_loop, r=alexcrichton

Teach cargo to failfast on recursive/corecursive aliases

Eg.
[alias]
test-1 = test-2
test-2 = test-3
test-3 = test-1

Previously it would stack overflow
Pulled out non controversial bits from from #9768

Auto merge of #9789 - ehuss:fix-value-after-table-profile, r=alexcrichton

Fix value-after-table error with profiles.

If the `strip`, `dir-name`, or `inherits` fields were included along with a profile override, then `cargo publish` would fail with a `ValueAfterTable` error because the fields were listed in the wrong order in the struct definition.

Fixes #9787

Refine warnings and better code

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Teach cargo to failfast on recursive/corecursive aliases

Eg.
[alias]
test-1 = test-2
test-2 = test-3
test-3 = test-1

Previously it would stack overflow

It pulls out non controversial bits from from #9768

Test for warning of migrating to unstable edition

Emit warning for migrating to unstable edition in stable channel

Refactor echo cargo subcommand test helper into cargo-test-support

Auto merge of #9790 - ehuss:fix-plugin-registrar, r=alexcrichton

Fix plugin registrar change.

The plugin tests were broken due to a change in https://github.com/rust-lang/rust/pull/85296 which removed the `plugin_registrar` attribute.

Fix plugin registrar change.

Fix value-after-table error with profiles.

Auto merge of #9627 - whereistejas:many-bin-names, r=ehuss

Ability to specify the output name for a bin target different from the crate name

Hi,

I have opened this to close the following issue #1706 .

I have decided to start by writing a test to outline what behavior is expected from `Cargo`.
As of now, this test fails (for obvious reasons). I will now start writing the code needed to pass this test.

This is my first time contributing to cargo. Please, feel free to let me know if there are any protocols/processes that need to be followed. I'm a newbie at this.

Closes issue #1706

Ability to specify the output name for a bin target different from the crate name

Separate warning and with root_id

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Auto merge of #9773 - nipunn1313:cargo_list_dupe, r=alexcrichton

Deduplicate entries in cargo --list

Fixes #6088

Deduplicate entries in cargo --list

Fixes #6088

Auto merge of #9764 - djmitche:issue8486, r=alexcrichton

Include aliases with other commands

The principal result is that they are now automatically included in the
`cargo --list` output.  Fixes #8486.

Just return a vec

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Include aliases with other commands

The principal result is that they are now automatically included in the
`cargo --list` output.  Fixes #8486.

Use batch warnings and only warn about used dependency packages

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Warning for no lib dependencies

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Auto merge of #9118 - bjorn3:profile_codegen_backend_option, r=joshtriplett

Add a profile option to select the codegen backend

This makes it possible to compile only specific crates with a certain codegen backend.

I have tested this by compiling Bevy with cg_llvm, but one of the examples using cg_clif.

By the way I noticed that many unstable profile options are not checked when using profile overrides.

Fix compile error

Auto merge of #9617 - danieleades:refactor/useless-conversions, r=ehuss

remove useless conversions

Auto merge of #9613 - danieleades:refactor/collapse-if-blocks, r=ehuss

collapse nested if blocks

Auto merge of #9767 - nipunn1313:refactor, r=alexcrichton

Refactor fake_file() away from cargo_command tests

There are already similar preexisting test helpers which
also work on windows. Port over the executable-creation
helper into the file() helper and things appear to pass.

Was inspired to do this when trying to add tests to solve a separate issue.

Refactor fake_file() away from cargo_command tests

There are already similar preexisting test helpers which
also work on windows. Port over the executable-creation
helper into the file() helper and things appear to pass.

Fix test

Update error message

Co-authored-by: Josh Triplett <josh@joshtriplett.org>

Only allow internal backends

Auto merge of #9762 - erickt:bump, r=ehuss

Update cargo-platform to 0.1.2

This preps cargo-platform for a release. The only substantial change is that this should include the license files into the archive that is uploaded to crates.io.

Closes #9758

Auto merge of #9760 - alexcrichton:bump-jobserver, r=Eh2406

Bump to the latest jobserver dependency

This should help mitigate #9739 at least with a better error message as
opposed to deadlocking.

Auto merge of #9763 - erickt:fix-semver-check, r=ehuss

Fix semver check for rust 1.54.0

This updates semver.md to use the new error message for reporting missing generic arguments.

Fix semver check for rust 1.54.0

This updates semver.md to use the new error message for reporting missing
generic arguments.

Update cargo-platform to 0.1.2

This preps cargo-platform for a release. The only substantial change is that
this should include the license files into the archive that is uploaded to
crates.io.

Closes #9758

Update test

Add a profile option to select the codegen backend

This makes it possible to compile only specific crates with a certain
codegen backend.

Bump to the latest jobserver dependency

This should help mitigate #9739 at least with a better error message as
opposed to deadlocking.

Auto merge of #9732 - djc:rust-version-docs, r=ehuss

Stabilize the rust-version field

I've tried to make the documentation here fairly comprehensive. I've also updated the first version for the 2021 edition, which should now be stable pending substantial unforeseen changes.

See #8072.

Add a CHANGELOG entry

Stabilize rust-version

Document the rust-version field