Stabilize namespaced and weak dependency features.

Auto merge of #10214 - weihanglo:revert-10188-issue-9528, r=alexcrichton

Be resilient to most IO error and filesystem loop while walking dirs

Let `PathSource::walk` be resilient to most IO errors and filesystem loop.

This PR also

- Add a test validating the resilience against filesystem loop to prevent regression.
- Emit warning when filesystem loop found while walking the filesystem. This is the only way I can think of now to solve #9528

Fixes #10213.

Auto merge of #10258 - joshtriplett:unconditional-pipelining, r=alexcrichton

Remove the option to disable pipelining

Cargo has had pipelining enabled by default for a long time, without
issue. Remove support for `build.pipelining = false`. (Continue parsing
the option from the config file, but ignore it.)

<!--
Thanks for submitting a pull request 🎉! Here are some tips for you:

* If this is your first contribution, read "Cargo Contribution Guide":
  https://doc.crates.io/contrib/
* Run `cargo fmt --all` to format your code changes.
* Small commits and pull requests are always preferable and easy to review.
* If your idea is large and needs feedback from the community, read how:
  https://doc.crates.io/contrib/process/#working-on-large-features
* Cargo takes care of compatibility. Read our design principles:
  https://doc.crates.io/contrib/design.html
* When changing help text of cargo commands, follow the steps to generate docs:
  https://github.com/rust-lang/cargo/tree/master/src/doc#building-the-man-pages
* If your PR is not finished, set it as "draft" PR or add "WIP" in its title.
* It's ok to use the CI resources to test your PR, but please don't abuse them.

### What does this PR try to resolve?

Explain the motivation behind this change.
A clear overview along with an in-depth explanation are helpful.

You can use `Fixes #<issue number>` to associate this PR to an existing issue.

### How should we test and review this PR?

Demonstrate how you test this change and guide reviewers through your PR.
With a smooth review process, a pull request usually gets reviewed quicker.

If you don't know how to write and run your tests, please read the guide:
https://doc.crates.io/contrib/tests

### Additional information

Other information you want to mention in this PR, such as prior arts,
future extensions, an unresolved problem, or a TODO list.
-->

Remove the option to disable pipelining

Cargo has had pipelining enabled by default for a long time, without
issue. Remove support for `build.pipelining = false`. (Continue parsing
the option from the config file, but ignore it.)

Auto merge of #10255 - joshtriplett:unconditional-artifact-timing, r=alexcrichton

Always ask rustc for messages about artifacts, and always process them

Rather than attempt to determine which compilations require metadata and
only ask rustc to emit messages about artifacts for those compilations,
just unconditionally generate and process such messages for all
compilations.

In addition to simplifying code, this also gives us that information for
timing purposes as well, even when not pipelining.

<!--
Thanks for submitting a pull request 🎉! Here are some tips for you:

* If this is your first contribution, read "Cargo Contribution Guide":
  https://doc.crates.io/contrib/
* Run `cargo fmt --all` to format your code changes.
* Small commits and pull requests are always preferable and easy to review.
* If your idea is large and needs feedback from the community, read how:
  https://doc.crates.io/contrib/process/#working-on-large-features
* Cargo takes care of compatibility. Read our design principles:
  https://doc.crates.io/contrib/design.html
* When changing help text of cargo commands, follow the steps to generate docs:
  https://github.com/rust-lang/cargo/tree/master/src/doc#building-the-man-pages
* If your PR is not finished, set it as "draft" PR or add "WIP" in its title.
* It's ok to use the CI resources to test your PR, but please don't abuse them.

### What does this PR try to resolve?

Explain the motivation behind this change.
A clear overview along with an in-depth explanation are helpful.

You can use `Fixes #<issue number>` to associate this PR to an existing issue.

### How should we test and review this PR?

Demonstrate how you test this change and guide reviewers through your PR.
With a smooth review process, a pull request usually gets reviewed quicker.

If you don't know how to write and run your tests, please read the guide:
https://doc.crates.io/contrib/tests

### Additional information

Other information you want to mention in this PR, such as prior arts,
future extensions, an unresolved problem, or a TODO list.
-->

Test IO error resilience while walking directories

Remove `build_script::build_script_scan_eacces`  test case because cargo
ignores it and returns its path during a `cargo build`. The caller still
has a chance to hit the IO error if they does access it.

Be resilient to most IO error and filesystem loop

Recover from IO errors wile walking directories, only abort when error
from `walkdir` is without an path to recover.

Always ask rustc for messages about artifacts, and always process them

Rather than attempt to determine which compilations require metadata and
only ask rustc to emit messages about artifacts for those compilations,
just unconditionally generate and process such messages for all
compilations.

In addition to simplifying code, this also gives us that information for
timing purposes as well, even when not pipelining.

Auto merge of #10254 - joshtriplett:rmeta-required-no-timings, r=alexcrichton

Make rmeta_required no longer depend on whether timing is enabled

This doesn't appear to affect the quality of the timing information at
all.

If there's additional information we need from rustc about what it's
doing at any given time, we could add mechanisms to retrieve that
information, but enabling timing shouldn't force building more than we
otherwise would have.

Auto merge of #10218 - weihanglo:pr-template, r=alexcrichton

The first version of pull request template

### What this PR tries to resolve?

We're trying to extract information out off the head of contributors.
Hope this help the review process more friendly for everyone.

### How to test and review this PR?

Copy this template and paste to your GitHub Web UI to see if it renders well.

### Additional information

Kubernetes community has one of the best guide of ["Best Practices for Faster Reviews"].
It describes tips for contributors tocreate a high quality PR. We might want
to create our own one and include it in PR template in the future.

In addition, I don't know how much time it take to update the changelog. I saw
some projects let PR author write down the release note of the PR. This sounds
helpful to take some burden off from maintainers, as well as let contributors
feel more involved. Anyway, it's not in this PR but is good to discuss later.

["Best Practices for Faster Reviews"]: https://github.com/kubernetes/community/blob/9b38b270640009f28d7d8710f74d955e0c374cef/contributors/guide/pull-requests.md#best-practices-for-faster-reviews

Make rmeta_required no longer depend on whether timing is enabled

This doesn't appear to affect the quality of the timing information at
all.

If there's additional information we need from rustc about what it's
doing at any given time, we could add mechanisms to retrieve that
information, but enabling timing shouldn't force building more than we
otherwise would have.

Auto merge of #10088 - joshtriplett:stabilize-strip, r=ehuss

Stabilize the `strip` profile option, now that rustc has stable `-C strip`

https://github.com/rust-lang/rust/pull/90058/ stabilized `-Z strip` as `-C strip`. Stabilize the corresponding Cargo option.

Update the documentation to move it from the unstable reference to the profiles documentation.

Update the tests to account for stabilization.

Auto merge of #10248 - ehuss:windows-ssh-agent, r=Eh2406

Update docs for windows ssh-agent.

This updates the documentation for SSH authentication using Windows and ssh-agent.

Per #10237 and https://github.com/rust-lang/rust/pull/92374, Cargo now supports ssh-agent on Windows.

I'm about 80% sure this information is correct based on my reading of the code and testing.  The ssh-agent that is part of mingw (or git-for-windows) uses emulated sockets, which are not compatible with traditional Windows-style named pipes.

Auto merge of #10244 - joshtriplett:tyop, r=ehuss

Fix typo: substract -> subtract

Update docs for windows ssh-agent.

Fix typo: substract -> subtract

Tweak wordings for PR template

Co-authored-by: Ruby Lazuli <general@patchmixolydic.com>

Auto merge of #10239 - Turbo87:patch-1, r=ehuss

timings: Fix tick mark alignment

### Before

<img width="1030" alt="Bildschirmfoto 2021-12-31 um 11 58 53" src="https://user-images.githubusercontent.com/141300/147819686-ce35da70-59d3-4174-b3c9-05e926260a4f.png">

### After

<img width="1032" alt="Bildschirmfoto 2021-12-31 um 11 58 23" src="https://user-images.githubusercontent.com/141300/147819683-a12d4387-dc30-40bc-aedc-c10fdac7e4e6.png">

timings: Fix tick mark alignment

Auto merge of #10238 - Aaron1011:remove-lifetimes, r=joshtriplett

Remove unused lifetimes

This is blocking the Crater run in https://github.com/rust-lang/rust/pull/92413,
since a 'try' build needs to build Cargo.

Remove unused lifetimes

This is blocking the Crater run in https://github.com/rust-lang/rust/pull/92413,
since a 'try' build needs to build Cargo.

Update stabilization date for strip option

Co-authored-by: Eric Huss <eric@huss.org>

Auto merge of #10224 - MidasLamb:find-closest-capitalization, r=joshtriplett

Make levenshtein distance case insensitive.

When typing in a single character shortcut as a capital, it always
returns `b` as the suggestion as every one-letter abbreviation
is a lev distance 1 away from the capitalized one.
By making the levenshtein distance case insensitive, the case-mismatched
one-letter abbriviation (e.g. `C` to `c`) will be suggested, rather
than `b`

Link to rustc documentation for `-C strip`

Stabilize the `strip` profile option, now that rustc has stable `-C strip`

https://github.com/rust-lang/rust/pull/90058/ stabilized `-Z strip` as
`-C strip`. Stabilize the corresponding Cargo option.

Update the documentation to move it from the unstable reference to the
profiles documentation.

Update the tests to account for stabilization, but make them no-ops on
stable/beta for now until rustc 1.58 releases.

Add comment explaining the lowercasing in the levenshtein distance calculation.

Move section titles into comment

Make levenshtein distance case insensitive.

When typing in a single character shortcut as a capital, it always
returns `b` as the suggestion as every one-letter abbreviation
is a lev distance 1 away from the capitalized one.
By making the levenshtein distance case insensitive, the case-mismatched
one-letter abbriviation (e.g. `C` to `c`) will be suggested, rather
than `b`

Test cargo build being resilient to filesystem loop

Auto merge of #10212 - SamMorrowDrums:patch-1, r=alexcrichton

[docs] Adds basic CI yaml for GitHub Actions

Currently there is no documentation for GitHub Actions, so I have attempted to add an Actions Workflow that is equivalent to the other CI snippets in the file. You can view a successful run of this Action in my repo for experimenting with this here: https://github.com/SamMorrowDrums/rust-action-test/actions/runs/1593666172

The Rust code I tested it with is just the boilerplate from `cargo init`.

add pull_request trigger and update add note on fail case

add information on starter workflow

The first version of pull request template

We're trying to extract information out off the head of contributors.
Hope this help the review process more friendly for everyone.

further simplify the workflow

Auto merge of #10209 - jonhoo:pub-manifest-from-str, r=alexcrichton

Add function for parsing already-read manifest

This makes the function formerly known as `do_read_manifest` public
under the name `read_manifest_from_str` so that Cargo-as-a-library users
can parse a manifest without re-reading a `Cargo.toml` file they have
already read from disk themselves.

add in override flag to set default cargo toolchain

Adds basic CI yaml for GitHub Actions

Auto merge of #10210 - nyurik:test-help, r=ehuss

Minor docs change for `cargo test --help`

Per #10208, make `cargo test -- --help` more discoverable.

Minor docs change for `cargo test --help`

Per #10208, make `cargo test -- --help` more discoverable.

Add function for parsing already-read manifest

This makes the function formerly known as `do_read_manifest` public
under the name `read_manifest_from_str` so that Cargo-as-a-library users
can parse a manifest without re-reading a `Cargo.toml` file they have
already read from disk themselves.

Auto merge of #10205 - hi-rustin:rustin-patch-clippy, r=Eh2406

Make clippy happy

Remove needless borrow.

Auto merge of #10202 - weihanglo:github-issue-config, r=alexcrichton

Enhance descriptions of issue templates

## Why

To discuss new features with larger scope, I think it's better discuss with the community, not just people in this repo. Hope these changes can guide people to a more proper place for this kind of feature request.

## Screenshots

#### Add two new types of issue template

- Question
- Inspiring Idea

<img width="500" alt="image" src="https://user-images.githubusercontent.com/14314532/146328466-f234c431-e355-4da1-a34a-b7b4c2490599.png">

#### Mention internal forum and zulip to guide user to discuss

<img width="500" alt="image" src="https://user-images.githubusercontent.com/14314532/146328580-4798983a-58d3-4e5e-9006-bf6d1d089c83.png">

Auto merge of #10196 - charlesroussel:master, r=alexcrichton

Add workaround for sporadic kills when building on Macos

This is the workaround for the issue https://github.com/rust-lang/cargo/issues/10060

Add workaround for sporadic kills when building on Macos

Auto merge of #10188 - weihanglo:issue-9528, r=alexcrichton

Detect filesystem loop during walking the projects

Resolves #9528

~~This PR also adds a new dependency `same-file` but since it's already a
dependency of `cargo-util`, so nothing added actually.~~

Use `walkdir` to detect filesystem loop and gain performance boost!

Auto merge of #10204 - hi-rustin:rustin-patch-doc, r=alexcrichton

Error about not having any crates with documentation

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

Error about not having any crates with documentation.

Make clippy happy

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

add open_no_doc_crate test

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

Error when the absence of any crates with files

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

Mention forum and zulip in feature-request issue template

Enhance issue template config

Auto merge of #10201 - ehuss:dep-doc-false, r=Eh2406

Don't document libs with doc=false

The `doc=false` setting was not being checked in the `[lib]` table when determining dependencies to document. This changes it so that that setting is honored.

Don't document libs with doc=false

Auto merge of #10198 - pinkforest:master, r=ehuss

Bumps up tar to 0.4.36

Minor changes - All tests pass and some manual tests

Bumps up tar to 0.4.36

Return bool instead of CargoResult for filter

Since `StripPrefixError` returned from [`Path::strip_prefix`] is just an
error with "prefix not found. We can simply return false for it.

Also use `Match::is_ignore` instead of manually matching.

[`Path::strip_prefix`]: https://doc.rust-lang.org/1.57.0/std/path/struct.Path.html#method.strip_prefix

Test filesystem loop during traversal

Use unordered since order of warning differs on each platform.

Use walkdir to walk filesyste and detect loop

Auto merge of #10152 - steven-joruk:quiet-config, r=ehuss

Support `term.quiet` configuration

Fixes #10128

This follows the existing support for `--verbose` and `term.verbose`.

I've renamed the related tests to be a bit clearer now there are more cases, and the existing quiet tests now prove that they hide the cargo log.

I'm unsure whether I'm supposed to regenerate the documentation as part of this?

Auto merge of #10193 - sstangl:help-alias-10138, r=alexcrichton

Display alias target on 'cargo help <alias>`

```
Previously, `cargo help <alias>` resolved the alias and displayed the
help for the targeted subcommand. For example, if `br` were aliased to
`build --release`, `cargo help br` would display the manpage for
cargo-build.

With this patch, it will print "'br' is aliased to 'build --release'".
```

Addresses issue #10138.

This is my first patch to Cargo. I attempted to follow the style of the surrounding code. Please let me know if any changes are required: happy to make them. In particular, I wasn't sure if any tests exist for this path.

Auto merge of #10145 - QiangHeisenberg:deprecated, r=alexcrichton

delete --host command and message

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

The warning for this command has been around for a long time, and it seems safe to delete it now.
According to Alex's words in the issue, I think he supports deletion. I'm sorry if I misunderstood it and feel free to close it.

Auto merge of #10191 - weihanglo:issue-9881, r=alexcrichton

Improve I/O error message for fingerprint of build script

It is a bit rough but I don't think there is a network I/O error
in `pkg_fingerprint`. Checking only `io::Error` type should be fine.

Resolves #9881

Auto merge of #10177 - ehuss:list-alias, r=alexcrichton

Explicitly mark aliases in `cargo list`.

This I think makes it a little clearer that these are aliases, and matches the form used for built-in aliases.

Auto merge of #10171 - ehuss:no-executable-doc, r=alexcrichton

Don't emit "executable" JSON field for non-executables.

The "executable" field of JSON artifact messages was accidentally filled (with the path to `index.html`) when documenting a binary target. This fixes it so that it is null.

Closes #10149

Auto merge of #10166 - ehuss:scrape-examples-unstable-doc, r=alexcrichton

Move scrape-examples docs to correct section.

This was erroneously put in the stabilized section.

Auto merge of #10161 - weihanglo:issue-7473, r=alexcrichton

Do not suggest source config if nothing to vendor

fixes #7473

Also remove the empty vendor dir if we've just created it but didn't vendor anything.

Auto merge of #10155 - ehuss:bump-deps, r=alexcrichton

Bump versions of local deps.

There have only been very minor changes since the last version.

Auto merge of #10154 - ehuss:version-bump, r=alexcrichton

Bump to 0.60.0, update changelog

Auto merge of #10153 - ehuss:fix-profile-docs, r=alexcrichton

Fix some profile documentation.

The `test` and `bench` sections had wrong information about which profile is used.  This also has some changes to try to format and clarify selection a little.

Auto merge of #10172 - ehuss:doc-lib-before-bin, r=alexcrichton

Document lib before bin.

This changes it so that documenting a library is required to finish before documenting a binary. The issue is that the binary may have intra-doc links to the library. If they are documented concurrently, then the links will sometimes fail (since it is a race).   Or, if doing `cargo doc --bins`, then the library docs wouldn't exist at all.

Note that in the tests this introduces some more name collisions if you just run `cargo doc --bins` and there is a colliding library/binary name. There is some risk that someone might be trying to run the commands separately to get around the collision error, but I think it is unlikely.

Auto merge of #10178 - ehuss:version-from-rustc, r=alexcrichton

Sync cargo-the-cli version with rustc.

People occasionally get confused when cargo's version does not match the version of rustc. This happens in a variety of scenarios:
* Point releases.
* Beta releases (cargo is missing the .1 .2, etc.)
* Nightly releases when cargo's version has not yet been bumped.

This changes it so that cargo-the-cli will always report the same version as rustc (assuming they were built with rustbuild). The git information remains the same (reports cargo's last commit sha).

Closes #10122

fix panic if an alias is defined to ""

Update behavior to pass-through simple aliases

This changes the behavior so that simple aliases that directly alias a
subcommand (with no arguments) pass-through to that subcommand, while
complex aliases (with arguments) show the alias.

So for example, `cargo help b` will show the manpage for `cargo-build`,
while `cargo help my-alias`, aliased to `build --release`, will show
"`my-alias` is aliased to `build --release`".

Simplify check_alias() using flatten()

Co-authored-by: Weihang Lo <weihanglo@users.noreply.github.com>

Update src/bin/cargo/commands/help.rs

Co-authored-by: Weihang Lo <weihanglo@users.noreply.github.com>

Update help::help_alias() to assert the new behavior

Display alias target on 'cargo help <alias>`. Closes #10138

Previously, `cargo help <alias>` resolved the alias and displayed the
help for the targeted subcommand. For example, if `br` were aliased to
`build --release`, `cargo help br` would display the manpage for
cargo-build.

With this patch, it will print "'br' is aliased to 'build --release'".

Improve I/O error message for fingerprint of build script

It is a bit rough but I don't think there is a network I/O error
in `pkg_fingerprint`. Checking only `io::Error` type should be fine.

Auto merge of #10185 - Aaron1011:future-incompat-cleanup, r=ehuss

Remove `-Z future-incompat-report` from message displayed to user

This was missed during stabilization

Remove `-Z future-incompat-report` from message displayed to user

This was missed during stabilization

Fix link to config.md#termquiet

Update docs

Code review fixes

Use version from rustbuild.

Remove pre_release in version info.

This field is no longer being used, as pre-release versions have
not been used in cargo for quite some time, and I don't anticipate
them ever returning.

Move cargo's self-version code to a dedicated module.

Explicitly mark aliases in `cargo list`.

Auto merge of #10168 - hi-rustin:rustin-patch-flag, r=ehuss

Unify the description of quiet flag

cc: #8928
I seem to have missed it.

Move scrape-examples docs to correct section.

This was erroneously put in the stabilized section.

Auto merge of #10165 - Aaron1011:stabilize-future-incompat, r=ehuss

Stabilize future-incompat-report

Depends on https://github.com/rust-lang/rust/pull/91535

Add CARGO_FUTURE_INCOMPAT_REPORT_FREQUENCY to environment variables

Adjust `[future-incompat-report]` docs

Improve documentation and add [future-incompat-report] config section

Fix detection of `--json future-incompat`

Add arg_quiet

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

Unify the description of quiet flag

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

Don't emit "executable" JSON field for non-executables.

Document lib before bin.

Update docs

Auto merge of #10133 - joshtriplett:release-short, r=ehuss

Support abbreviating `--release` as `-r`

Of the options people regularly pass to cargo, `--release` seems by far
the most common. Yet even on the command line, we expect people to type
out `--release`.

Add a short version `-r`, and add some tests in the testsuite that
confirm it works.