Auto merge of #7706 - matthiaskrgr:bump_git, r=Eh2406

bump git2 dependencies

This required some manual code changes which dependabot could not perform.

Auto merge of #7738 - JohnTitor:issue-none, r=ehuss

Use issue = "none" instead of "0"

Replace `issue = "0"` with `"none"`

CC: rust-lang/rust#67496

Auto merge of #7737 - Mark-Simulacrum:cache-not-mem, r=ehuss

Read cached output line-by-line

This avoids loading potentially gigabytes of output into memory, which
can cause OOMs.

Fixes #7736.

This does not add a test as I don't really want to generate gigabytes of output (that seems like a bad idea) -- and it's unclear how to test other than by causing OOM on (most) CI systems, and it's unlikely that we'll actually regress here.

Use issue = "none" instead of "0"

Auto merge of #7733 - ehuss:doc-updates, r=Eh2406

Various doc updates

This is a collection of documentation updates that have been in my todo list for a while. Each change is in a separate commit.

The "features" chapter will probably get significant changes in the future, as it is pretty bare right now. Similarly the "workspace" chapter could probably use more examples.

Closes #3062
Closes #3817
Closes #3971
Closes #4212
Closes #4438
Closes #4756
Closes #5822
Closes #6913
Closes #7055

Read cached output line-by-line

This avoids loading potentially gigabytes of output into memory, which
can cause OOMs.

Note that harness=false enables cfg(test).

Add build script examples to reference index.

Move overriding dependencies into a separate chapter.

Auto merge of #7734 - ehuss:version-bump, r=Eh2406

Bump to 0.43.0, Update changelog

Update changelog for 1.41.

Move nightly-only changes to dedicated sections.

Bump to 0.43.0

Add a table of contents for the manifest.

Closes #5822

Move features to a separate chapter.

Move workspaces to a separate chapter.

Move cargo targets to a new chapter.

Closes #6913.

Don't describe the package layout twice.

Try to clarify dev/test profile selection.

Closes #4438

bump git2 dependencies

Document that [replace] should not be used.

cc #7092

Clarify that [replace] requires a version number.

Closes #4212

Try to be clearer about how `--target` is different.

Closes #3817

Try to make it a little clearer that cargo package/publish will rebuild from scratch.

Closes #3062

Show example for 'cfg(feature="foo")' dependency alternative.

Break up the Platform specific dependencies section.

Auto merge of #7721 - ehuss:fix-doc-build-indent, r=Eh2406

Fix indentation mistake in doc comment.

The `build` directory should be under `debug`.

Build script change detection: link to include/exclude

Auto merge of #7724 - ehuss:update-config-tracking-issue, r=alexcrichton

Add config tracking issue links.

Add config tracking issue links.

Fix indentation mistake in doc comment.

The `build` directory should be under `debug`.

Mention `cargo metadata` for custom subcommands.

Describe dep-info files a little better.

Clarify `cargo fix` behavior with features.

Auto merge of #7649 - ehuss:config2, r=alexcrichton

Config enhancements.

This is a collection of changes to config handling. I intended to split this into separate PRs, but they all built on one another so I decided to do it as one. However, I can still split this up if desired.

High level overview:

- Refactorings, mainly to remove `pub` from `Config::get_table` and use serde API instead.
- Add `--config` CLI option.
- Add config `include` to include other files.

This makes some progress on #5416.
Closes #6699.

This makes a minor user-visible change in regards to `StringList` types. If an array is specified in a config as a list, and also as an env var, they will now be merged. Previously the environment variable overrode the file value. But if it is a string, then it won't join (env var takes precedence). I can probably change this, but I'm not sure if the old behavior is desired, or if it should merge all the time?

**Future plans**
This lays the groundwork for some more changes:
- Work on #7253 (`debug-assertions` and `debug` fails in environment vars). I have some ideas to try.
- Consider removing use of `get_list` for `paths`, and use a `Vec<ConfigRelativePath>`. This will require some non-trivial changes to how `ConfigSeqAccess` works. This is one of the last parts that does not use the serde API.
- Possibly change `[source]` to load config values in a lazy fashion. This will unlock the ability to use environment variables with source definitions (like CARGO_SOURCE_CRATES_IO_REPLACE_WITH).
- Possibly change `[profile]` to load config profiles in a lazy fashion. This will make it easier to use environment variables with profiles, particularly with arbitrarily named profiles.
- Possibly remove the case-sensitive environment variables in `-Zadvanced-env`. I think they are just too awkward, and prone to problems. Instead, drive people towards using `--config` instead of env vars.
- Add support for TOML tables in env vars (like `CARGO_PROFILES={my-profile={opt-level=1}})`). I started implementing it, but then looking at the use cases, it didn't seem as useful as I initially thought. However, it's still an option to try.

**Refactoring overview**

- `[source]` table now uses the serde API.
- `[target]` table now uses the serde API. This is complicated since the 'cfg()' entries are different from the triple entries. The 'cfg()' tables are loaded separately, and are accessed from `Config::target_cfgs`. Otherwise, it just uses `config.get` of the specific target.TRIPLE.
    - Moved the target config stuff into `config/target.rs`.
- Various changes to make this work:
    - Added `PathAndArgs` type which replaces `config.get_path_and_args`.
    - Changed `ConfigKey` to track the key parts as a list (instead of a string). This fixes an issue where quoted keys weren't handled properly (like `[foo.'a.b'.bar]`). This also seems to make a little more sense (it was joining parts into a string only to immediately call `split` on it). Changed various APIs to take a `ConfigKey` object instead of a string to avoid that splitting behavior.
    - `ValueDeserializer` now pre-computes the `Definition` so that it can provide a better error message when a value fails to deserialize.

Overall, there shouldn't be significant user-visible changes. Some error messages have changed and warnings have been added for some ignored keys. `-Zadvanced-env` now works for source and target tables, though I'm not really happy with that feature.

Fix for merge conflict.

Adding some config doc comments.

Add some context to a [target] links deserialize error.

Fixes for some test errors on Windows.

Implement config-include.

Add --config CLI option.

Store Definition in ConfigValue.

Add a ConfigBuilder for tests.

Config refactoring.

Auto merge of #7718 - ehuss:fix-vendor-alt-reg, r=alexcrichton

vendor: support alt registries

Adds support for alt registries to `cargo vendor`. It mostly worked before, but panicked when trying to display the `.cargo/config` instructions.

This isn't entirely elegant, as the source replacement looks like this:

```toml
[source.crates-io]
replace-with = "vendored-sources"

[source."file:///Users/eric/Proj/rust/cargo/target/cit/t0/alternative-registry"]
registry = "file:///Users/eric/Proj/rust/cargo/target/cit/t0/alternative-registry"
replace-with = "vendored-sources"

[source."file:///Users/eric/Proj/rust/cargo/target/cit/t0/gitdep"]
git = "file:///Users/eric/Proj/rust/cargo/target/cit/t0/gitdep"
branch = "master"
replace-with = "vendored-sources"

[source.vendored-sources]
directory = "vendor"
```

The duplication of the URLs is a little unfortunate.  It could use the name of the registry, but that is not readily available and is tricky to obtain.  I feel like that is a challenge for another day.

Closes #7674.

Auto merge of #7717 - giraffate:registry_flag_is_already_stabilized, r=ehuss

Update the output of `cargo -Z help`

`--registry` flag is already stabilized at 737382d7e.

Fix some chapter links missing from index pages.

Fix minor `vendor` mistake in glossary.

Be clearer about which fields are optional/required.

This attempts to be clearer about the requirements for crates.io publishing.

This also includes some other clarifications around various fields and specifying dependencies.

Closes #3971
Closes #7055

Simplify some crates.io links.

Fix awkward comma in sentence.

Document the target directory layout.

Closes #4756.

Add links for rustflags/rustdocflags.

Mention that `--features` can be specified multiple times.

Mention --offline in the FAQ entry on offline.

vendor: support alt registries

Update the output of `cargo -Z help`

`--registry` flag is already stabilized at 737382d7e.

Auto merge of #7708 - giraffate:fix_overwriting_credentials, r=ehuss

Fix overwriting alternate registry token

When executing `cargo login`, 2nd alternate registry token overwrites
1st alternate registry token.

Fixes #7701.

Refactoring of creating registry

Refactoring to call new `generate_*` functions instead of duplicating codes

Append `registries` to config

Remove unneeded flags

Auto merge of #7700 - ehuss:proc-macro-extern-prelude, r=alexcrichton

Add proc_macro to the extern prelude.

This makes it so that a proc-macro library can use the `proc_macro` crate without the `extern crate proc_macro;` item on the 2018 edition.  This is the Cargo half of https://github.com/rust-lang/rust/pull/64882.

Auto merge of #7710 - dtolnay:argorder, r=alexcrichton

Pass --edition after --crate-name

This PR swaps the order of the --edition and --crate-name args in the rustc invocation.

- Before: `rustc --edition=2018 --crate-name cargo ...`
- After: `rustc --crate-name cargo --edition=2018 ...`

The crate name is a lot more relevant when looking at processes in `top` for example, and should appear first.

Auto merge of #7631 - jsgf:explicit-version, r=alexcrichton

vendor: implement --versioned-dirs

Implement `--explicit-version` from standalone cargo-vendor. This helps with vendoring
performance as it avoids redundantly deleting and re-copying already vendored packages.

For example, re-vendoring cargo's dependencies it makes a big difference in wallclock
time. For initial vendoring it makes no difference, but re-vendoring (ie, when most or all dependencies haven't changed) without explicit versions is actually slightly slower
(5.8s -> 6s), but with explicit versions it goes from 5.8s -> 1.6s.

Timings:

Without explicit versions, initial vendor
real 0m5.810s
user 0m0.924s
sys 0m2.491s

Re-vendor:
real 0m6.083s
user 0m0.937s
sys 0m2.654s

With explicit versions, initial vendor:
real 0m5.810s
user 0m0.937s
sys 0m2.461s

Re-vendor:
real 0m1.567s
user 0m0.578s
sys 0m0.967s

Its interesting to look at the syscall summary:

Without explicit versions:
```
% time     seconds  usecs/call     calls    errors syscall
------ ----------- ----------- --------- --------- ----------------
 25.17    1.104699          18     59432      1065 openat
 19.86    0.871574          21     41156     13825 unlink
 13.64    0.598739           2    210510           lstat
  9.02    0.395948          29     13208           copy_file_range
  8.00    0.351242          11     30245           read
  6.36    0.279005           3     72487      4476 statx
  5.35    0.235027           6     37219           write
  4.02    0.176267           3     58368           close
```
with explicit versions:
```
 29.38    0.419068          15     27798     13825 unlink
 25.52    0.364021           1    209586           lstat
 20.67    0.294788          16     17967      1032 openat
 10.42    0.148586           4     35646           write
  3.53    0.050350           3     13825           chmod
  3.14    0.044786           2     16701      1622 statx
  2.19    0.031171           1     16936           close
  1.86    0.026538          24      1078           rmdir
```

Specifically, there are a lot fewer opens, copy_file_ranges, and unlinks.

Refactoring

Pass --edition after --crate-name

This PR swaps the order of the --edition and --crate-name args in the
rustc invocation.

    Before: rustc --edition=2018 --crate-name cargo ...
    After: rustc --crate-name cargo --edition=2018 ...

The crate name is a lot more relevant when looking at processes in top
for example, and should appear first.

Fix overwriting alternate registry token

When executing `cargo login`, 2nd alternate registry token overwrites
1st alternate registry token.

Fixes #7701.

Add proc_macro to the extern prelude.

Auto merge of #7698 - dtolnay:description, r=alexcrichton

Remove use of deprecated Error::description method

We are getting ready to deprecate std::error::Error::description: rust-lang/rust#66919. This PR replaces one place that should be using Display instead.

Auto merge of #7695 - giraffate:add_and_update_tests_for_jobs_of_cargo_build, r=ehuss

Add and move tests for jobs of `cargo build`

A test when argument is negative is added. In addition, `default_cargo_config_jobs` and `good_cargo_config_jobs` is moved from `testsuite/bad_config.rs` to `testsuite/build.rs` because these tests are not for `bad config`.

Auto merge of #7680 - stefanhoelzl:issue-7183, r=ehuss

include dotfiles in packages

This PR solves #7183

It changes the behavior of `cargo package` to also include dotfiles by default.

It should be discussed if this is intended or if the implementation should be changed to only include dotfiles which are specified in the `include` section.

From the [existing comment](https://github.com/stefanhoelzl/cargo/blob/40885dfab40a1bf62b22aa03f732ef45163c013f/src/cargo/sources/path.rs#L358) it is a little bit unclear to me, but I supposed it was intended only to exclude directories starting with a dot?

Auto merge of #7699 - ehuss:build-std-no-sysroot, r=alexcrichton

Switch build-std to use --extern

Switch `build-std` to use `--extern` flags instead of `--sysroot`.

This is mostly a revert of #7421. It uses the new extern flag options introduced in https://github.com/rust-lang/rust/pull/67074. It avoids modifying the extern prelude which was the source of the problem of https://github.com/rust-lang/wg-cargo-std-aware/issues/40.

Closes https://github.com/rust-lang/wg-cargo-std-aware/issues/49
Reopens https://github.com/rust-lang/wg-cargo-std-aware/issues/31

Switch build-std to use --extern

Remove use of deprecated Error::description method

Auto merge of #7694 - danielparks:issue-7691-manifest-badges-section, r=ehuss

Clarify manifest format docs for [badges] section

Previously the `[badges]` section documentation was mixed in with the `[package]` section. This extracts it out under its own heading.

This also pulls the `license` and `license-file` field documentation out of the package metadata code block. It sets the example license to “MIT OR Apache-2.0” since that is the typical licensing scheme for many crates.

This includes minor rewrites for clarity and to match the register of the rest of the document.

This resolves issue #7691.

Auto merge of #7696 - matthiaskrgr:typ0s, r=ehuss

fix a couple of typos

fix a couple of typos

Add and move tests for jobs of `cargo build`

A test when argument is negative is added. In addition,
`default_cargo_config_jobs` and `good_cargo_config_jobs` is moved from
`testsuite/bad_config.rs` to `testsuite/build.rs` because these tests
are not for `bad config`.

Clarify manifest format docs for [badges] section

Previously the `[badges]` section documentation was mixed in with the
`[package]` section. This extracts it out under its own heading.

This also pulls the `license` and `license-file` field documentation out
of the package metadata code block. It sets the example license to “MIT
OR Apache-2.0” since that is the typical licensing scheme for many
crates.

This includes minor rewrites for clarity and to match the register of
the rest of the document.

This resolves issue #7691.

deleted unneccessary ignore pattern

Auto merge of #7693 - ehuss:bump-cargo-platform, r=Eh2406

Bump cargo-platform version.

Needed for changes from #7660.

Bump cargo-platform version.

Auto merge of #7690 - giraffate:add_test_for_cargo_locate_project, r=alexcrichton

Add a test for `cargo locate-project`

There seems to be no test for `cargo locate-project`, so I add the simple test case for this command.

Document --version-dirs in cargo-vendor.adoc and re-generate all doc formats.

Add a test for `cargo locate-project`

Auto merge of #7660 - benediktwerner:master, r=alexcrichton

Emit error on [target.'cfg(debug_assertions)'.dependencies] and similar

Closes #7634

Cargo now emits an error if an unsupported cfg key or name is used for specifying a target.

This PR also updates the docs to clarify this behavior.

Auto merge of #7687 - giraffate:Update_the_layout_of_cargo_commands, r=ehuss

Update the layout of `Cargo Commands` in doc

It seems better to improve the follwing two points.
- A subcategory like [`Build Commands`](https://doc.rust-lang.org/stable/cargo/commands/build-commands.html) does not contain any contents.
- [`Cargo Commands`](https://doc.rust-lang.org/stable/cargo/commands/index.html) actually just points to `cargo` command.

So, I fix categories like `Build Commands` to include links to each
commands. In addition, I add `cargo` command to `General Commands`.

I just changed the way it was arranged and nothing about the contents.
The following is an example.

before
<img width="1283" alt="スクリーンショット 2019-12-09 10 01 44" src="https://user-images.githubusercontent.com/17407489/70399888-f2c3df80-1a6a-11ea-813b-4bba47d3b659.png">

after
<img width="1415" alt="スクリーンショット 2019-12-09 10 06 43" src="https://user-images.githubusercontent.com/17407489/70400018-a4631080-1a6b-11ea-8bd4-ab15338ae508.png">

Update the layout of `Cargo Commands` in doc

It seems better to improve the follwing two points.
- A subcategory like `Build Commands` does not contain any contents.
- `Cargo Commands` actually just points to `cargo` command.

So, I fix categories like `Build Commands` to include links to each
commands. In addition, I add `cargo` command to `General Commands`.

Auto merge of #7682 - fintelia:patch-1, r=ehuss

Features and dependencies can't have the same name

The document was describing a Cargo.toml containing
```
[dependencies]
postgres = { version = "*", optional = true }
postgres-macros = { version = "*", optional = true }

[features]
postgres = ["postgres", "postgres-macros"]
```

If you tried doing this you'd get an error:
```
error: failed to parse manifest at `/home/jonathan/tmp/features/Cargo.toml`

Caused by:
  Features and dependencies cannot have the same name: `postgres`
```

variant with old behaviour but dotfiles can be included

test variant to exclude dot-files/dirs by default

added dotfile to include test

got rid of extra test for dotfiles

Features and dependencies can't have the same name

The document was describing a Cargo.toml containing
```
[dependencies]
postgres = { version = "*", optional = true }
postgres-macros = { version = "*", optional = true }

[features]
postgres = ["postgres", "postgres-macros"]
```

If you tried doing this you'd get an error:
```
error: failed to parse manifest at `/home/jonathan/tmp/features/Cargo.toml`

Caused by:
  Features and dependencies cannot have the same name: `postgres`
```

fix format issues

Warn instead of error

include dotfiles

added testcase

Auto merge of #7672 - giraffate:fix_some_typos, r=alexcrichton

Fix some typos in doc

Convert --explicit-version -> --versioned-dirs