Auto merge of #3667 - jbendig:required_features, r=alexcrichton

Added required_features for issue #1570.

Based on PR #2056 by @tsurai and PR #2325 by @JanLikar

I tried to fix most everything that was talked about in the previous pull requests. Docs still need to be updated though.

Auto merge of #3681 - lukaslueg:master, r=alexcrichton

Pinpoint hamcrest-dependency in cargotest to 0.1.1

hamcrest 0.1 currently resolves to 0.1.3, which cargotest can't actually build with. Cargo's own dependency on =0.1.1 used to shadow the problem.

Fixed `cargo install` performing an empty install when all binaries require at least one feature that's not selected.

Changed an .any() call with double negatives into a more readable .all().

Pinpoint hamcrest-dependency in cargotest to 0.1.1

hamcrest 0.1 currently resolves to 0.1.3, which cargotest can't actually build with. Cargo's own dependency on =0.1.1 used to shadow the problem.

Auto merge of #3671 - z1mvader:master, r=alexcrichton

trying to fix unimplemented flock under linux

It fixes https://github.com/rust-lang/cargo/issues/3666

Fixed required-features bench tests failing when compiled on stable rust.

Added documentation for the target field `required-features`.

Auto merge of #3669 - Susurrus:master, r=alexcrichton

Correct capitalization of GitLab

Never realized this is how they spelled it when I added these docs. Equivalent changes are pending for `crates.io` in rust-lang/crates.io#547

Auto merge of #3659 - nathanaeljones:master, r=alexcrichton

Fix #3658 .cargo/config prevents freshness (sort after HashMap)

The many vectors of BuildOutput are populated from a HashMap in cargo_compile... and later these vectors are hashed.

HashMaps are the bane of Cargo's existence.

But for now, we sort.

Correct capitalization of GitLab

Fix #3659 differently; sort HashMap keys before iteration.

Reverts 7a8b25a (and the need for it)

Add test fresh_builds_possible_with_link_libs

Auto merge of #3583 - alexcrichton:fix-tags, r=brson

Handle `rev` being a `tag`

Previously if a rev was specified as a tag then we'd trip an assertion because
resetting to that tag would reset to the tag that the commit pointed to, which
would then cause the head id of the repo to be different than what we thought it
was.

Instead, we handle the case where a `rev` specification is a tag explicitly by
using the tag's target id as the revision that we're going to check out, not the
id of the tag itself.

Closes #3580

Added required_features for issue #1570.

Based on PR #2056 by @tsurai and PR #2325 by @JanLikar

trying to fix unimplemented flock under linux

Auto merge of #3651 - pkgw:pr-issue-3366, r=alexcrichton

Attempt to solve #3366, regarding spurious shared library search paths.

This drops `native_dirs` entries that are not within the target output when modifying the (DY)LD_LIBRARY_PATH environment variable before running programs.

CC #3366

Attempt to fix the run_with_library_paths test for Windows.

I was just pasting the build directories into Rust string literals, so Windows
paths with backslashes were being interpreted as having unknown string
escapes. Raw string guards should fix this for all but the most pathological
of build directories.

Auto merge of #3664 - pwoolcoc:issue-3391, r=alexcrichton

Assume that a `build.rs` file is a build script

If cargo sees a `build.rs` file in the same directory as the current
`Cargo.toml`, it will assume that the `build.rs` file is a build script,
unless there is `build = false` in the  `Cargo.toml` file.

Closes #3391

Auto merge of #3663 - alexcrichton:update, r=alexcrichton

Update dependencies

Another major version of Cargo, another suite of updated dependencies!

Update dependencies

Another major version of Cargo, another suite of updated dependencies!

Assume that a `build.rs` file is a build script

If cargo sees a `build.rs` file in the same directory as the current
`Cargo.toml`, it will assume that the `build.rs` file is a build script,
_unless there is_ `build = false` _in the _ `Cargo.toml` _file_.

Closes #3391

Adjust test overrides_and_links.

While order between rust-c-link-lib and rustc-flags was always undefined
(for TOML configuration), order of link paths/libs within a single
rustc-flags value was maintained. Now we sort them all.

style: reduce line length

Update the 'run_with_library_paths' test to reflect the new filtering.

For the library paths to be passed through, they must reside within the build
output directory.

Make fresh_builds_possible_with_multiple_metadata_overrides faster

Fix #3658 .cargo/config prevents freshness (sort if sourced from HashMap)

Rename test to fresh_builds_possible_with_multiple_metadata_overrides

Add (non-deterministically) failing test for #3658

Auto merge of #3644 - Keruspe:master, r=alexcrichton

ensure libressl compatibility

It looks like master still isn't compatible with libressl.

With this patch it builds and runs fine and all tests pass here.

tests: fix git User-Agent for Windows

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

Auto merge of #3655 - king6cong:rustfmt, r=alexcrichton

rustfmt for bin/cargo.rs

None

rustfmt for bin/cargo.rs

Auto merge of #3632 - Susurrus:master, r=alexcrichton

Document build badge for Gitlab CI

This doesn't make sense to merge until rust-lang/crates.io#539 is merged, but I figured I'd get it all spooled up since that PR is already ready for merging.

Attempt to solve #3366, regarding spurious shared library search paths.

This drops `native_dirs` entries that are not within the target directory when
modifying the (DY)LD_LIBRARY_PATH environment variable before running
programs.

Auto merge of #3650 - bluetech:profile-doc-fix, r=alexcrichton

Fix the documentation for the PROFILE envvar passed to build scripts

Closes #2084 (+ a typo fix).

Fix the documentation for the PROFILE envvar passed to build scripts

Closes #2084.

Fix typo

Auto merge of #3648 - cuviper:powerpc64-gcc4, r=alexcrichton

Update OPENSSL_CC_powerpc64-unknown-linux-gnu

With the new `rust-slave-linux-cross:2017-02-02`, powerpc64 no longer
uses gcc-5.  I updated the `Dockerfile` at the time to reflect this, but
missed `OPENSSL_CC_powerpc64-unknown-linux-gnu` in `Makefile.in`.

Update OPENSSL_CC_powerpc64-unknown-linux-gnu

With the new `rust-slave-linux-cross:2017-02-02`, powerpc64 no longer
uses gcc-5.  I updated the `Dockerfile` at the time to reflect this, but
missed `OPENSSL_CC_powerpc64-unknown-linux-gnu` in `Makefile.in`.

Auto merge of #3647 - alexcrichton:dox, r=alexcrichton

Update API doc URL for Cargo

Closes #3640

Update API doc URL for Cargo

Closes #3640

ensure libressl compatibility

Update libgit2-sys, openssl-sys and openssl
as the versions currently in Cargo.lock are
not compatible with libressl.
Also update tests accordingly

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

Auto merge of #3639 - GuillaumeGomez:fix-doc-tests, r=alexcrichton

Replace remaining _0 doc tests

This time, I `grep`ed all remaining `_0` tests. This update should be the last one.

Auto merge of #3637 - cuviper:custom-powerpc-s390x, r=alexcrichton

Update docker/cross for new IBM toolchains

This will start using the toolchains with older glibc ABI for powerpc,
powerpc64, powerpc64le, and s390x, per rust-lang/rust-buildbot#149.

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

Bump to 0.18.0

Auto merge of #3634 - steveklabnik:bump-semver, r=alexcrichton

bump semver version

Thanks to @raphlinus , semver-parser no longer depends on regex. The code is (probably, I haven't technically tested it but it's hard to imagine it's not) faster, smaller, and has zero dependencies, as well as better errors.

This new version of semver only updates the dependencies involved, no interface changes, so it should be a drop-in replacement.

Auto merge of #3633 - shiver:3617, r=alexcrichton

Add description to metadata output #3617

Just took a quick stab at issue #3617.

Is my interpretation of the requirements correct, and if so, would you prefer new tests or are the current alterations sufficient.

Thanks!

Auto merge of #3628 - zackmdavis:the_maelstrom_of_my_purity_born_of_pain, r=alexcrichton

make build tests not depend on minutiae of rustc output

This little patch arises from the maelstrom of my purity born of pain.

It's the pain of seeing rust-lang/rust#38103 in its perfect crystalline beauty waste away on page four of https://github.com/rust-lang/rust/pulls, waiting, ready, itching to land, dying with anticipation to bring the light of clearer lint group error messages to Rust users of all creeds and nations, only for its promise to be cruelly blocked by the fateful, hateful hand of circular dependency. For it is written in src/tools/cargotest/main.rs that the Cargo tests must pass before the PR can receive Appveyor's blessing, but the Cargo tests could not pass (because they depend on fine details of the output that the PR is meant to change), and the Cargo tests could not be changed (because updating the test expectation to match the proposed new compiler output, would fail with the current compiler).

The Gordian knot is cut in the bowels of cargotest's very notion of comparison (of JSON objects) itself, by means of introducing a magic string literal `"{...}"`, which can server as a wildcard for any JSON sub-object.

And so it will be for the children, and the children's children, and unto the 1.17.0 and 1.18.0 releases, that Cargo's build test expectations will faithfully expect the exact JSON output by Cargo itself, but the string literal `"{...}"` shall be a token upon the JSON output by rustc, and when I see `"{...}"`, I will pass over you, and the failure shall not be upon you.

And this day shall be unto you for a memorial.

supersedes #3513

r? @alexcrichton

Replace remaining _0 doc tests

Update docker/cross for new IBM toolchains

This will start using the toolchains with older glibc ABI for powerpc,
powerpc64, powerpc64le, and s390x, per rust-lang/rust-buildbot#149.

Bump to 0.18.0

bump semver version

Fixed failing read-metadata test due to addition of description.

Metadata response now also contains package description.

Auto merge of #3621 - matklad:fix-job-leakage, r=alexcrichton

Don't leak job

This fixes that problem with diverging `handle_error`. Now the return types of all subcommands match, and we can return `CliResult` up to main.

make build tests not depend on minutiæ of rustc output

This little patch arises from the maelstrom of my purity born of pain.

It's the pain of seeing rust-lang/rust#38103 in its perfect
crystalline beauty waste away on page four of
https://github.com/rust-lang/rust/pulls, waiting, ready, itching to
land, dying with anticipation to bring the light of clearer lint group
error messages to Rust users of all creeds and nations, only for its
promise to be cruelly blocked by the fateful, hateful hand of circular
dependency. For it is written in src/tools/cargotest/main.rs that the
Cargo tests must pass before the PR can receive Appveyor's blessing,
but the Cargo tests could not pass (because they depend on fine
details of the output that the PR is meant to change), and the Cargo
tests could not be changed (because updating the test expectation to
match the proposed new compiler output, would fail with the current
compiler).

The Gordian knot is cut in the bowels of cargotest's very notion of
comparison (of JSON objects) itself, by means of introducing a magic
string literal `"{...}"`, which can server as a wildcard for any JSON
sub-object.

And so it will be for the children, and the children's children, and
unto the 1.17.0 and 1.18.0 releases, that Cargo's build test
expectations will faithfully expect the exact JSON output by Cargo
itself, but the string literal `"{...}"` shall be a token upon the
JSON output by rustc, and when I see `"{...}"`, I will pass over you,
and the failure shall not be upon you.

And this day shall be unto you for a memorial.

Auto merge of #3616 - GuillaumeGomez:new_rustdoc_test, r=alexcrichton

Update to new rustdoc --test output

cc @alexcrichton

Auto merge of #3609 - jmatraszek:build_proper_binary, r=alexcrichton

Fix building multiple binaries that do not have path spacified in Cargo.toml

When multiple binaries are specified in Cargo.toml, the binaries that do not have `path` specified are build from `src/main.rs`. Discovered here: https://github.com/rust-lang-nursery/thanks/pull/40#issuecomment-275493045.

This was caused by setting for a binary a main layout here https://github.com/rust-lang/cargo/blob/master/src/cargo/util/toml.rs#L478, which caused `normalize` to not fallback to default binary path here https://github.com/rust-lang/cargo/blob/master/src/cargo/util/toml.rs#L1149 (as `bin.path` was always `Some("/path/to/main.rs")`.

Added a test and fixed this by not using `layout.main()`, so right now for bins without `path` specified we fallback to default path inferred from bin's name (e.g. `src/bin/foo.rs`), test if the file exists and only if it doesn't -- fallback to `src/main.rs`.

I do not have any knowledge about Cargo's design, so I am not sure if this is the proper place to test for file existence.

Document build badge for Gitlab CI

Auto merge of #3562 - matklad:member-workspace, r=alexcrichton

Find workspace via `workspace_root` link in containing member

This PR proposes to change the logic for determining workspace members. Here are the algorithms we used previously for this:

# [RFC](https://github.com/rust-lang/rfcs/blob/master/text/1525-cargo-workspace.md)  flavor

If `[members]` key present, it compliantly defines the set of members. Otherwise, members are all (transitive) path dependencies.

The problem with this approach is that it violates convention over configuration principle: almost always you want a path dependency to be a member, even if there are some explicit members. Listing **all** path deps is just to much work.

# Original implementation

So, the actual algorithm **unconditionally** included path dependencies as memebers.

This is also problematic, because certain workspace configurations become impossible. In particular, you can't have a path dependency which is not a member of the workspace. This issue was reported in #3192.

# Current implementation

Current implementation (was merged couple of days ago) includes path dependency into the workspace only if is inside the workspace directory. This solves the problem in #3192 perfectly. However, some configuration are still impossible: you can't have a non-member path dependency inside a workspace directory. But the thinking is that if you don't want this path-dep to be a member, just don't put it inside the workspace directory.

There is another problem with current imlementation. Suppose you have an explicit member which lives alongside the workspace. Suppose this member has a path-dep which lives inside the member's folder. Under current implementation, this path-dep won't be a member of the workspace. It seems logical that it should be though (but we haven't received any actual bug reports yet)!

# Implementation in this PR

So, with this PR, the logic is as follows: members are explicit members + all path dependencies which reside under any of the explicit members.

Auto merge of #3004 - ehiggs:master, r=alexcrichton

Resubmission of templating PR (#1747)

This is a manual rebase of the older #1747 PR which was basically unrebasable due to the time it's been dormant.. This implements templating for Cargo and is basically the work of Greg Chapple (@gchp).

I'd love for this feature to move forward since it's really tedious to create all the directories (e.g. `src/bin`) and copy paste docopt examples which I then edit. Then implement the `main` program to delegate to the subcommands in `src/bin`, etc.

Auto merge of #3630 - alexcrichton:empty-version, r=alexcrichton

Fix empty 'version' file

This teaches the makefiles to construct the version string again so
it can be emitted to the 'version' file for packaging.

Fixes failing nightly builds.

Fix empty 'version' file

This teaches the makefiles to construct the version string again so
it can be emitted to the 'version' file for packaging.

Fixes failing nightly builds.

Fix metadata tests

Ignore more output

Don't leak job

Remove unused 'file' function.

Upgrade handlebars to 0.25.

Cargo templating for `new` and `init`

PR #3004 This is a resubmission of the PR #1747 (from scratch) which adds
support for templating in Cargo. The templates are implemented using the
handlebars crate (where the original PR used mustache).

Examples:
cargo new --template https://url/to/template somedir foo
cargo new --template https://url/to/templates --template-subdir somedir foo
cargo new --template ../path/to/template somedir foo

Auto merge of #3614 - matklad:simpler-cli-2, r=alexcrichton

Remove generic infrastrucure for printing json

Only a couple of command really use this features, so it's clearer just to print json right on the spot rather then return it though the call stack.

`CliResult` is now just a `Result<(), CliError>`.

Auto merge of #3618 - sbeckeriv:3473-new-subcommand-doc-update, r=alexcrichton

Update new command help doc

Resolves #3473

Dearest reviewer,

I hope that all is well in rustland. I was reviewing some open issues
and saw this update to the help text for the new command. I have added
the suggested help text from issue #3473. I formated to the message to
match the publish bin's job flag.

Thank you for your time,
Becker

Auto merge of #3606 - Susurrus:no_git_deps2, r=alexcrichton

Clarify error for repo-based dependencies when publishing.

Elaborate on the exact error cause and directions on how to resolve it.

Update new command help doc

Resolves #3473

Dearest reviewer,

I hope that all is well in rustland. I was reviewing some open issues
and saw this update to the help text for the new command. I have added
the suggested help text from issue #3473. I formated to the message to
match the publish bin's job flag.

Thank you for your time,
Becker

Update to new rustdoc --test output

Remove generic infrastrucure for printing json

 Only a couple of command really use this features, so it's clearer just to print json right on the spot rather then return it though the call stack.

Clarify error for repo-based dependencies when publishing.

Specifically add a description of how to resolve the error.

Auto merge of #3608 - matklad:simple-cli, r=alexcrichton

Simplify cli-handling functions

This simplifies somewhat roundabout `main` in `bin/cargo.rs`.

Simplify cli-handling functions

Fix typo

Auto merge of #3604 - froydnj:rich-version-info, r=alexcrichton

implement `cargo --version --verbose`

As suggested in #3584.  This is a bit underwhelming, and I'm unsure if some of the complexity in froydnj/cargo@775c900 is really warranted, but this series gets the job done.  Sample output when building with `configure` and `make`:

```
froydnj@hawkeye:~/src/cargo.git$ target/x86_64-unknown-linux-gnu/release/cargo --version
cargo-0.17.0-dev (ae4a4d8 2017-01-27)
froydnj@hawkeye:~/src/cargo.git$ target/x86_64-unknown-linux-gnu/release/cargo --version --verbose
cargo-0.17.0-dev (ae4a4d8 2017-01-27)
release: 0.17.0
commit-hash: ae4a4d8fc55bf7eca3d974f953dc61729e4a40db
commit-date: 2017-01-27
```

Fix building multiple binaries

Add test

Auto merge of #3593 - Susurrus:master, r=alexcrichton

Improve error for dependencies that don't have the same source paths

I've added an additional test case which is how my project compiled. Part of the issue was that I didn't know that each dependency needs to use the same path for all build targets. The previous error message was unclear both in what was going on and how to resolve it. The new error message should be more clear.

This is tied to #3592.

Auto merge of #3557 - raphlinus:master, r=alexcrichton

Add dep-info generation

Work in progress: add a --dep-info flag to cargo build (and also
rustc) that outputs dependency information in a form compatible with
make and ninja, to a specified file. This will help in integrating
into other build systems.

Auto merge of #3590 - dgellow:fix-3536, r=alexcrichton

When an error occured in a JobQueue print it directly

Fix #3536

directly print error only if there is more than one job in the queue

Add simple tests, cleanup

Tests for existence for dep-info output in simple compilation cases.

Deletes the dep-info file if it fails (for example, if it can't find
one of the dep-info inputs).

make `cargo --version --verbose` print extra information

Fixes #3584.

remove CFG_RELEASE from the Makefile

The previous commit made this unused; much of the logic for assigning to
this variable now lives in Cargo itself.

export rich version information from cargo::version

To support `cargo --version --verbose`, ala rustc, we need more
information to be injected into cargo when it's built from the Makefile,
and a more explicit data structure to be returned from cargo::version.
We implement fmt::Display for our newly-created structure so clients
don't have to bother with the details of interpreting the structure if
all they want is a string.

Auto merge of #3603 - ehuss:patch-1, r=alexcrichton

Fix broken code block

It seems like the Markdown parser requires a blank line before a code block (otherwise it treats it like an inline span).

Fix broken code block

It seems like the Markdown parser requires a blank line before a code block (otherwise it treats it like an inline span).

Auto merge of #3600 - onur:fix-link, r=alexcrichton

Fix crates.io link under specifying dependencies

I just realized when I was replying to #2947, second crates.io link under "Specifying path dependencies" is not working.

And TBH the phrase in parentheses is unclear for me.

cc @carols10cents

Auto merge of #3598 - pornel:proxyconf, r=alexcrichton

Example proxy config

#3596

Auto merge of #3601 - alexcrichton:bump-openssl, r=brson

Update OpenSSL to 1.0.2k

Updates in accordance with the security advisory issued today.

Update OpenSSL to 1.0.2k

Updates in accordance with the security advisory issued today.

Fix crates.io link under specifying dependencies

Example proxy config

#3596

Auto merge of #3597 - Susurrus:badge_docs, r=alexcrichton

Add gitlab to the supported services for AppVeyor badges.

This is undocumented but supported behavior for AppVeyor. I have already done this with a crate I own and the badge works on both the AppVeyor and crates.io end.