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.

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

Auto merge of #4570 - tatsuya6502:target-specific-artifacts, r=alexcrichton

Handle target specific outputs such as .wasm or .dll.lib

Fixes #4500, #4535

Until now, Cargo does not have any knowledge about target-specific output files. It relies solely on rustc for this sort of information (`rustc --print=file-names ...`).

As a result, Cargo does not place some build artifacts (files) to target/{debug,release} directory. These files include *.wasm for wasm32-unknown-emscripten target and *.dll.lib for *-pc-windows-msvc cdylib target.

This commit will add such knowledge to Cargo so that *.wasm and *.dll.lib will be placed in target/{debug,release} directory.

**EDIT**: I added [a summary of changes](https://github.com/rust-lang/cargo/pull/4570#issuecomment-334433635). Please read it for more details of changes.

**IMPORTANT**
Although I added test cases for both wasm32-unknown-emscripten and *-pc-windows-msvc cdylib targets, ~I did not do manual testing on wasm32-unknown-emscripten target as I do not have an environment with emscripten installed. It will be appreciated if anybody tests this change for wasm32-unknown-emscripten target.~  **EDIT**: Tested for both wasm32-unknown-emscripten and x86_64-pc-windows-msvc. Thanks @Herschel for the help.

Auto merge of #4597 - andreevlex:fix-cargo-doc, r=alexcrichton

Fixed spelling error in manifest.md

Fixed spelling error in manifest.md

Auto merge of #4595 - treiff:update-appveyor-badge-documentation, r=alexcrichton

Update docs for appveyor badge.

Can now optionally specify the appveyor `project_name`.

Related PR: [rust-lang/crates.io#1111](https://github.com/rust-lang/crates.io/pull/1111)
Related Issue: [rust-lang/crates.io#587](https://github.com/rust-lang/crates.io/issues/587)

Update docs for appveyor badge.

[workspace] Add some docs

Plus some little codemod.

[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.

Auto merge of #4578 - derekdreery:refactor/cargo_util_config, r=alexcrichton

Remove some cells from Config

In most cases it makes sense to not use interior mutability in Config, so that mutability is more transparent. This PR removes Cells in these cases. It leaves those cases where parameters are only initialized on first access (LazyCell).

Update docs (replace todos)

Auto merge of #4586 - lukaslueg:issue4534, r=alexcrichton

Report hint if single failure with --no-fail-fast

There are two things going on in #4534:

* The doctest-kind masks the unittest-kind if doctests are executed during `--no-fail-fast`, no doctests fail and tests errors have already piled up. This is bug, fixed.
* Cargo would previously not report any unittest-hints if running with `--no-fail-fast`. This behavior has been changed to report a specific hint iif exactly one error has bubbled up during `--no-fail-fast`. The hint in #4534 is now `...pass '--test integ'`.

Report hint if single failure with --no-fail-fast

If a single unit-test fails, report a rerun-hint for it,
even with --no-fail-fast.

Fixes reporting a doctest-hint if doctest succeeds while
unittest fail with --no-fail-fast.

Fixes #4534

Auto merge of #4583 - alexcrichton:unstable, r=matklad

Fix [patch] causing updates with a virtual manifest

This commit fixes the [patch] implementation to avoid causing spurious updates
of the registry when specified inside of a virtual manifest via a path
dependency that was otherwise outside of the workspace.

Cargo previously attempted to learn about path dependencies through the
configuration of the workspace by looking at members, their path dependencies,
and their replace sections. The replace/patch sections, however, only matter at
the root of the workspace and those weren't looked at! This commit fixes this
problem by explicitly looking at the workspace's replace/patch sections and
avoiding looking at other manifest replace/patch which shouldn't matter.

Closes #4552

Fix [patch] causing updates with a virtual manifest

This commit fixes the [patch] implementation to avoid causing spurious updates
of the registry when specified inside of a virtual manifest via a path
dependency that was otherwise outside of the workspace.

Cargo previously attempted to learn about path dependencies through the
configuration of the workspace by looking at members, their path dependencies,
and their replace sections. The replace/patch sections, however, only matter at
the root of the workspace and those weren't looked at! This commit fixes this
problem by explicitly looking at the workspace's replace/patch sections and
avoiding looking at other manifest replace/patch which shouldn't matter.

Closes #4552

Handle target specific outputs such as .wasm

Fixes #4535

- Do not add metadata to wasm32 bin target because generated .js will
  refer corresponding .wasm.
- Handle different usages of "-" and "_" in .js and .wasm file names.
  (e.g "foo-bar.js" vs. "foo_bar.wasm")
- Change file mode of cargo_rustc/context.rs (100755 -> 100644)

Auto merge of #4580 - djc:cargo-update, r=alexcrichton

Update dependencies (unbreak macOS 10.13 with libgit2-sys-0.6.16)

It would be nice to pull in the fix for https://github.com/alexcrichton/git2-rs/issues/257.

Auto merge of #4579 - mdevlamynck:issue-3388, r=alexcrichton

Fix error message when Cargo.toml misses a [package] section

Fixes #3388.

Update dependencies (unbreak macOS 10.13 with libgit2-sys-0.6.16)

Fix error message when Cargo.toml misses a [package] section, remove an unused import

Auto merge of #4571 - raytung:remove-root-from-cargo-lock, r=matklad

Remove root from cargo lock

Removing [root] section from Cargo.lock for #4563

Tests of interest has been updated accordingly, especially [tests/lockfile-compat.rs#oldest_lockfile_still_works](https://github.com/rust-lang/cargo/compare/master...raytung:remove-root-from-cargo-lock?expand=1#diff-48866d1891f97cb799dd0ca7148d1eefR10)

Remove cells where now unnecessary.

Refactor Config::configure to take &mut.

Auto merge of #4556 - lukaslueg:patch-1, r=carols10cents

Update README.md

Table is outdated and of little use anyway

Auto merge of #4558 - lukaslueg:issue4504, r=alexcrichton

Allow multiple arguments in `cargo bench`

It seems the docopt DSL was simply lacking.

Updated old lockfile tests

Removed root related flags

Revert "Removed root related flags"

This reverts commit 6f03cce1c3e6083f882fc56100487e616f60fa6b.

Updated test to ensure cargo update transforms Cargo.lock

Removed root related flags

Revert the file mode on context.rs (100644 -> 100755)

Updated appropriate docs referencing [root]

Removed extraneous logic that generates [root] in cargo.lock

Handle target specific outputs such as .wasm or .dll.lib

Fixes #4500, #4535

Until now, Cargo does not have any knowledge about target-specific
output files. It relies solely on rustc for this sort of information
(`rustc --print=file-names ...`).

As a result, Cargo does not place some build artifacts (files) to
target/{debug,release} directory. These files include *.wasm
for wasm32-unknown-emscripten target and *.dll.lib for
*-pc-windows-msvc cdylib target.

This commit will add such knowledge to Cargo so that *.wasm and
*.dll.lib will be placed in target/{debug,release} directory.

Auto merge of #4564 - rust-lang:matklad-patch-2, r=alexcrichton

Describe E-mentor label in contributing.md

Auto merge of #4559 - derekdreery:add_docs, r=alexcrichton

Add some more docs

Add some more docs for various parts of cargo, and add/improve a few debug implementations.

Describe E-mentor label in contributing.md

Revert Cargo.lock

Merge remote-tracking branch 'upstream/master' into add_docs

Auto merge of #4562 - SimonSapin:btree-manifest, r=matklad

Make manifest serialization deterministic

Fixes #4326

`cargo package` (and so `cargo publish`) parses a crate’s `Cargo.toml`, makes some modifications, and re-serializes it. Because the `TomlManifest` struct uses `HashMap` with its default `RandomState` hasher, the maps’ iteration order changed on every run.

As a result, when using `cargo vendor`, updating a dependency would generate a diff larger than necessary, with non-significant order-changes obscuring significant changes.

This replaces some uses of `HashMap` with `BTreeMap`, whose iteration order is deterministic (based on `Ord`).

Make manifest serialization deterministic

Fixes #4326

`cargo package` (and so `cargo publish`) parses a crate’s `Cargo.toml`,
makes some modifications, and re-serializes it.
Because the `TomlManifest` struct uses `HashMap`
with its default `RandomState` hasher,
the maps’ iteration order changed on every run.

As a result, when using `cargo vendor`,
updating a dependency would generate a diff larger than necessary,
with non-significant order-changes obscuring significant changes.

This replaces some uses of `HashMap` with `BTreeMap`,
whose iteration order is deterministic (based on `Ord`).

Some moer docs

Add some more docs

Allow multiple arguments in `cargo bench`

Fixes #4504

Auto merge of #4554 - lukaslueg:contributing, r=alexcrichton

Expand contribution guidelines

Expand `contributing.md`, including inspirations from Rust's and some other projects' guides.

Update README.md

Table is outdated and of little use anyway

Auto merge of #4553 - integer32llc:typos, r=alexcrichton

Fix some typos in docs and error messages

Auto merge of #4550 - lukaslueg:issue4539, r=alexcrichton

Improve error ico doc-targets with same name

Replace the `assert!` that was triggered in #4539 with a `bail!`, giving the user a clear error that points to the offending packages. The error message now looks like

> The binary `foo_cli` is specified by packages `foo (file://...)` and `bar (file://...)` but can be documented only once. Consider renaming or marking one of the targets as `doc = false`.

or

> The library `foo_cli` is specified by packages `foo (file://...)` and `bar (file://...)` ...

or

> The target `foo_cli` is specified as a library and as a binary by package `foo (file://...)`. It can be documented only once. Consider...

or

> The target `foo_cli` is specified as a library by package `foo (file://...)` and as a binary by package `bar (file://...)`. It can be ...

Add unit-test for all cases, including one that ensures the advertised mitigation `doc = false` actually works.

Auto merge of #4551 - natboehm:owner-invite-messages, r=alexcrichton

Owner invite messages

This PR addresses issue #4537, the plan for `cargo owner --add` requiring invitations in Cargo and the encompassing issue [#924](https://github.com/rust-lang/crates.io/issues/924), requiring an invite to add someone as an owner in Crates.io.

Regarding the Cargo issue, we went with Option 2, changing the `add_owners` function to decode a struct sent from Crates containing a `boolean` and `String`, the `boolean` being the response status and `String` being the success message. This may sound redundant however we concluded that using both of these fields were necessary to support older versions of Cargo - if we changed Crates.io to only return the `String` message on success this would likely break systems using the older version of `add_owner` expecting a response containing a `boolean`. Matching this schema, `add_owners` on the Crates.io side will soon return a struct containing a `boolean` and `String`, and instead of adding a new crate owner to the database will add a crate owner invite. If successful, `modify_owners` now prints the message sent from Crates.io instead of the old hardcoded message.

Resolves #4537

Expand contribution guidelines

Fix some typos in docs and error messages

Auto merge of #4546 - derekdreery:add_docs, r=matklad

Add some docs

I was reading around the source, trying to understand it, and writing docs to help myself. I thought they may be useful for the main crate.

Improve error ico doc-targets with same name

We can currently only document one doc-target per name.
Replace an assert! with a clear error message in that case.
Fixes #4539.

Auto merge of #4547 - lukaslueg:installlisterror, r=alexcrichton

Improve error message for crippled .crates.toml

In case `.crates.toml` can't be parsed for some reason, the user is left wondering where to look. The problem is exasperated by the fact that `.cargo.toml` is a hidden file. Improve the error message during `install`/`uninstall`/`list` and point to the path of `.crates.toml`. A error message might now look like

> error: failed to parse crate metadata at `/home/foobar/.cargo/.crates.toml`
>
> Caused by:
>   invalid TOML found for metadata
>
> Caused by:
>   ...

A unit-test for a crippled `.cargo.toml` had been missing before, added that.

Auto merge of #4542 - equal-l2:sha256-crypto-hash, r=alexcrichton

Use crypto-hash to calculate SHA256

`crypto-hash` is an abstraction library for native hash libraries.
It uses CryptoAPI on Windows, CommonCrypto on macOS, and OpenSSL on *nix.

This PR will also remove `openssl` and `advapi32-sys` from dependencies since they are only used for calculating SHA256, which is superseded by `crypto-hash`.
(`crypto-hash` itself uses `openssl` and `advapi32-sys` though)

add boolean to struct to support what crates is sending

in case an old version of cargo is being used. the old
version should be able to decode the boolean and ignore
the string.

Improve error message for crippled .crates.toml

Add some docs

Auto merge of #4543 - nerdrew:master, r=matklad

teach zsh completion +{stable,beta,nightly}

e.g.: `cargo +nightly test`

teach zsh completion +{stable,beta,nightly}

e.g.: cargo +nightly test

change add_owners to return a string and transmit the status message passed from the owner_add function in crates.io

Auto merge of #4541 - raytung:update-man-page, r=carols10cents

Recover lost man changes

Recover lost man changes per @carols10cents's comment in #2689

Fix compilation error

Use crypto-hash to calculate SHA256

Recover lost man changes

i don't know if this works but i should commit

Auto merge of #4531 - lukaslueg:springclean, r=alexcrichton

Cleaning lints

More fixes and cleanups. There will be at least two more commits to come.

Reset unneeded lint changes

Auto merge of #4523 - dbrgn:badges-docs, r=matklad

Improve docs for build badges

The `repo` format wasn't clear to me. I also fixed a typo and clarified how the is-it-maintained badges are generated

More lint cleaning

Auto merge of #4530 - djc:update-deps, r=alexcrichton

Update dependencies

It would be nice to use openssl-0.9.19 in particular, since it supports
newer LibreSSL.

Not sure how this usually works, would it be better to only update openssl and related dependencies? I figured doing everything at once couldn't hurt, and it seems to pass tests locally at least.

Also, would it be feasible to backport this to the beta branch?

Auto merge of #4532 - copyninja:master, r=alexcrichton

Fix typo's in the cargo's output messages.

I'm forwarding  the patch we used in Debian to fix the typo's which were found in cargo's output messages.

Fix typo's in the cargo's output messages.

Update dependencies

It would be nice to use openssl-0.9.19 in particular, since it supports
newer LibreSSL.

Auto merge of #4528 - adrian5:patch-1, r=alexcrichton

Fix typo in doc

Auto merge of #4526 - lukaslueg:patch-2, r=matklad

Update CONTRIBUTING.md

Point out that code added to Cargo may raise the minimal version of Rust required. Cargo does not have a grace period where it has to compile on "stable -2" or anything like that.

Auto merge of #4525 - lukaslueg:springclean, r=alexcrichton

Cleaning lints

I've started to clean some minor defects in cargo. This is the first commit of possibly many.

Requesting advice if this is actually wanted; #cargo was positive.

Some things raise the minimum version of rust required to compile cargo. E.g. `assert_ne!(foo, bar)` instead of `assert!(foo != bar)` requires (iirc) rust 1.13. Any advice on that in particular?

Auto merge of #4521 - goffrie:master, r=alexcrichton

Use memoized hashes when hashing Fingerprint.

The recursive hashing of dependencies can cause exponential blowup. We
already have a memoized hash available, so use that, Merkle-tree-style.

Remove Github mention from CI badges docs

Fix typo in doc

Update CONTRIBUTING.md

Point out that code added to Cargo may raise the minimal version of Rust required. Cargo does not have a grace period where it has to compile on "stable -2" or anything like that.

Auto merge of #4481 - behnam:travis, r=alexcrichton

[travis] Enable cache for $HOME/.cargo

This should accelerate the `cargo install mdbook` step under `install`,
which is currently called on all linux+nightly setups.

https://github.com/rust-lang/cargo/blob/36ddeff03a31f710095cfbe49287503bb60e294d/.travis.yml#L24-L25

Cleaning lints

Clarify repository key format for build badges

Clarify how is-it-maintained badges are generated

Also, improve the format by breaking up the text into multiple
bullet-points.

Fix typo in Circle CI badge docs

[travis] Enable cache for $HOME/.cargo/bin/

This should accelerate the `cargo install mdbook` step under `install`,
which is currently called on all linux+nightly setups.

Use memoized hashes when hashing Fingerprint.

The recursive hashing of dependencies can cause exponential blowup. We
already have a memoized hash available, so use that, Merkle-tree-style.

Auto merge of #4519 - leavehouse:patch-1, r=alexcrichton

Clarify dependencies documentation

The snippet:

```
... but not `2.0`. `0.0.x` is not considered ...
```
Looks very similar to `... but not 2.0.0.0.x is not considered ...`, when rendered, which is confusing.

This adds a couple words to clarify.

Auto merge of #4520 - goffrie:master, r=alexcrichton

Don't visit the same unit multiple times.

This fixes some accidentally-exponential behaviour we were seeing in our
fairly large workspace. It brings a no-op cargo run from about 10
seconds to 0.3 seconds on my machine.

Also changes an association list to a hashmap because that was showing
up in profiles too.

Don't visit the same unit multiple times.

This fixes some accidentally-exponential behaviour we were seeing in our
fairly large workspace. It brings a no-op cargo run from about 10
seconds to 0.3 seconds on my machine.

Also changes an association list to a hashmap because that was showing
up in profiles too.

Clarify dependencies documentation

The snippet:

```
... but not `2.0`. `0.0.x` is not considered ...
```
Looks very similar to `... but not 2.0.0.0.x is not considered ...`, when rendered, which is confusing.

This adds a couple words to clarify.

Auto merge of #4515 - lukaslueg:issue1006, r=alexcrichton

Improve message for multiple links to native lib

Proposal for a fix to #1006, as advertised in my comment; as discussed briefly with alexcrichton on IRC.

In case multiple packages link to the same library, the error message is now

> error: More than one package links to native library `a`, which can only be linked once.
>
> Package a-sys v0.5.0 (file:///home/lukas/dev/issue1006test/a) links to native library `a`.
> (Dependency via foo v0.5.0 (file:///home/lukas/dev/issue1006test))
>
> Package b-sys v0.5.0 (file:///home/lukas/dev/issue1006test/a/b) also links to native library `a`.
> (Dependency via a-sys v0.5.0 (file:///home/lukas/dev/issue1006test/a) => foo v0.5.0 (file:///home/lukas/dev/issue1006test))
>
> Try updating or pinning your dependencies to ensure that native library `a` is only linked once.
>

In case the root-package itself is an offender:

> Package foo v0.5.0 (file:///home/lukas/dev/issue1006test) links to native library `a`.
> (This is the root-package)
>

IMHO the wording is much clearer now (the term "native library" and "package" are repeated on purpose); printing the whole dependency-chain from the offending package up to the root allows the user to at least figure out where the native library actually comes in.

Added a unit-test, which all pass. Please scrutinize this carefully, it's my first PR for cargo.

Further improve doubly-linked error-msg

Auto merge of #4517 - durka:patch-7, r=alexcrichton

[patch] slipped to 1.21, mark 2

Same as #4513, apparently it showed up in a few places.

Merge pull request #1 from durka/patch-8

update [patch] disclaimer in book