delete unused try macro from read2.rs

Auto merge of #3500 - alexcrichton:appveyor-new-deploy, r=alexcrichton

Try using before_deploy on AppVeyor

Try using before_deploy on AppVeyor

Auto merge of #3468 - nrc:metadata-emit, r=alexcrichton

cargo check: use --emit=metadata rather than --crate-type=metadata

Requires https://github.com/rust-lang/rust/pull/38571 (don't land before that does)

r? @alexcrichton

Fixup tests

Use --emit=metadata rather than --crate-type=metadata

Requires https://github.com/rust-lang/rust/pull/38571

Auto merge of #3485 - sfackler:useragent, r=alexcrichton

Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.

r? @alexcrichton

Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.

Auto merge of #3480 - alexcrichton:one-flaky-test, r=brson

Protect against spurious failure in ctrl_c test

A failure was witnessed in the Rust repository [1] which happened right after
this test and was a problem removing a directory. Local testing confirms that if
you kill Cargo then right afterwards it's very unlikely to be able to remove the
build directory, presumably because the child process is still getting torn down
in the background.

This commit fixes the ctrl_c test itself to wait for itself to release the bulid
directory, at which point the test has definitely passed.

[1]: https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1331/job/xq4ogmglj7sllibw

Protect against spurious failure in ctrl_c test

A failure was witnessed in the Rust repository [1] which happened right after
this test and was a problem removing a directory. Local testing confirms that if
you kill Cargo then right afterwards it's very unlikely to be able to remove the
build directory, presumably because the child process is still getting torn down
in the background.

This commit fixes the ctrl_c test itself to wait for itself to release the bulid
directory, at which point the test has definitely passed.

[1]: https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1331/job/xq4ogmglj7sllibw

Auto merge of #3477 - sfackler:shell-escape, r=alexcrichton

Use externally sourced shell-escape

Closes #3374

r? @alexcrichton

Use externally sourced shell-escape

Closes #3374

Auto merge of #3470 - alexcrichton:no-more-urls, r=steveklabnik

Delete manual installation URLs

These have existed since the beginning of time, but they haven't needed to exist
since long before 1.0. Let's just delete them now that they're out of date
anyway.

Delete manual installation URLs

These have existed since the beginning of time, but they haven't needed to exist
since long before 1.0. Let's just delete them now that they're out of date
anyway.

Auto merge of #3435 - alexcrichton:bump-libssh2, r=brson

Update libssh2 to fix a segfault on Windows

There's some more discussion on #3401, but this essentially is just an inclusion
of libssh2/libssh2#163

Closes #3401

Auto merge of #3417 - alexcrichton:fix-svn, r=brson

Ignore dot dirs in directory sources

Looks like they'll conflict with a VCS, so let's just skip them like we do in
directory traversal elsewhere.

Closes #3414

Auto merge of #3412 - alexcrichton:fix-recursion, r=brson

Fix recursion in build_path_deps

We were slightly too eager to follow pointers!

Closes #3330

Auto merge of #3467 - alexcrichton:verbose, r=alexcrichton

Attempt to debug a problem on AppVeyor

Let's get verbose logs from this step

Attempt to debug a problem on AppVeyor

Let's get verbose logs from this step

Auto merge of #3438 - petrochenkov:bsdoc, r=alexcrichton

Add some more details to build script docs

r? @alexcrichton

Add some more details to build script docs

Auto merge of #3458 - alexcrichton:docs, r=steveklabnik

Update README build instructions

Closes #3437

Auto merge of #3454 - alexcrichton:bump1, r=alexcrichton

Bump crates-io to 0.6.0

Auto merge of #3439 - gentoo90:bashcomp, r=alexcrichton

Bashcomp

Update bash completion

Auto merge of #3426 - nrc:check-fix-2, r=alexcrichton

Test for #3419

Based on top of https://github.com/rust-lang/cargo/pull/3425.

I'm not sure if this is an acceptable test because it downloads rustc-serialize.

r? @alexcrichton

Auto merge of #3436 - matklad:abs-target, r=alexcrichton

Make sure targets always use absolute path

Closes #3416

I'd want to add a test for it later. Seems a bit suspicious that this change has not caused any test to fail.

Update README build instructions

Auto merge of #3457 - alexcrichton:no-check-on-beta, r=alexcrichton

Only run check tests on nightly

Only run check tests on nightly

Bump crates-io to 0.6.0

Test for cargo check with macro and both lib and bin.

Closes #3419

Add a test

Includes adding facility for checking that output does not contain a string.

Update bash completion

Unify indentation in bash completion

Make sure targets always use absolute path

Update libssh2 to fix a segfault on Windows

There's some more discussion on #3401, but this essentially is just an inclusion
of libssh2/libssh2#163

Closes #3401

Ignore dot dirs in directory sources

Looks like they'll conflict with a VCS, so let's just skip them like we do in
directory traversal elsewhere.

Closes #3414

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

Bump to 0.17.0

Bump to 0.17.0

Auto merge of #3424 - matklad:naming-things, r=alexcrichton

Minor, more explicit names

Ultraminor renaming. Have to look at this tests because it fails if you link with older libgit.

Auto merge of #3413 - matklad:ref, r=alexcrichton

Refactor resolve_dependencies

This moves `resolve_dependencies` function from `compile.rs` into the `resolve.rs`, where it more reasonably belongs.

Auto merge of #3410 - fmdkdd:fix-3390-message-format-parse-stderr, r=alexcrichton

Fix `--message-format JSON` when rustc emits non-JSON warnings

The `--message-format JSON` flag parses all the stderr output of rustc to JSON,
but rustc can emit non-JSON lines to stderr (e.g., for warning about the
unstable `-Z` flag on the stable channel), causing cargo to fail reporting
compilation errors when using `--message-format JSON`.

This commit adds a check to look for lines beginning with `{` to only parse
these lines as JSON.  Other lines from rustc are forwarded to the stderr of
cargo.

Fixes #3390.

make sure we use the check profile where possible for lib deps.

Fixes #3418

Minor, more explicit names

Don't carry source ids along with sources

Be more explicit about temporary workspaces

Add source_id method to the Source

Fix recursion in build_path_deps

We were slightly too eager to follow pointers!

Closes #3330

Add a test for `--message-format` parsing the mixed output of rustc

Rename resolve_dependencies -> resolve_ws_precisely

Simplify `resolve_ws` API

Move `resolve_dependencies` into resolve.rs

Auto merge of #3411 - alexcrichton:tweaks, r=alexcrichton

Minor build tweaks

* Clean out OpenSSL when building
* Test 32-bit Cargo on OSX, not the 64-bit version on the 32-bit builder

Minor build tweaks

* Clean out OpenSSL when building
* Test 32-bit Cargo on OSX, not the 64-bit version on the 32-bit builder

Fix `--message-format JSON` when rustc emits non-JSON warnings

The `--message-format JSON` flag parses all the stderr output of rustc to JSON,
but rustc can emit non-JSON lines to stderr (e.g., for warning about the
unstable `-Z` flag on the stable channel), causing cargo to fail reporting
compilation errors when using `--message-format JSON`.

This commit adds a check to look for lines beginning with `{` to only parse
these lines as JSON.  Other lines from rustc are forwarded to the stderr of
cargo.

Fixes #3390.

Auto merge of #3409 - matklad:workspace-manifest-error, r=alexcrichton

Don't ignore errors in workspace manifest

Closes #3407

If we find `Cargo.toml` during an up tree traversal, we'd better not to ignore errors within it!

Don't ignore errors in workspace manifest

Auto merge of #3393 - alexcrichton:fix-libz-linkage, r=brson

Bump libz-sys to 1.0.9

This just switches libz to always link statically instead of relying on the
system zlib. For MinGW it seems that linkage may default pull in a DLL, which is
almost never what we want.

Closes #3384

Auto merge of #3403 - matklad:doc-loc, r=alexcrichton

cargo doc: show where are docs

Let's print the file we are trying to open. It's useful when we fail to detect the browser, because the user would be able to copy-paste url manually.

Bump libz-sys to 1.0.9

This just switches libz to always link statically instead of relying on the
system zlib. For MinGW it seems that linkage may default pull in a DLL, which is
almost never what we want.

Also update curl-sys to fix a build issue on MinGW.

Closes #3384

Auto merge of #3402 - matklad:packages, r=alexcrichton

Refactor packages specification

A small refactoring to avoid returning a triple.

cargo doc: show where are docs

Refactor packages specification

Auto merge of #3396 - pornel:test, r=alexcrichton

Test for issue #2064

I've tried to reproduce the problem from #2064, but to my surprise the test passes reliably :)

Auto merge of #3296 - nrc:check, r=alexcrichton

cargo check

~~This is not finished - the big omission is no tests, and it needs some more testing too. It also requires https://github.com/rust-lang/rust/pull/37681.~~

However, this is my first non-trivial Cargo patch, so I'd like to get some early feedback.

r? @alexcrichton

and cc @rust-lang/tools since this adds a new Cargo sub-command (although we have previously discussed and approved the idea in a tools meeting).

Auto merge of #3398 - alexcrichton:git-rev, r=alexcrichton

Fix git rev showing up in version number

Makefiles didn't support an out-of-tree build, so needed to update them to do
so.

Auto merge of #3363 - alexcrichton:libcmt, r=alexcrichton

Compile statically against the MSVC CRT

This updates our AppVeyor builds to compile with `-Ctarget-feature=+crt-static`
to help Cargo be a bit more portable and not rely on the MSVC redistributable
artifacts. Over time this may even let us converge on only releasing one build
of Cargo and just pairing that with all Windows toolchains...

Auto merge of #3399 - alexcrichton:fix-nightly, r=alexcrichton

Fix compatibility with Rust nightly

These two tests actually shouldn't have ever passed, but nightly Rust is more
principled about "$OUT_DIR" and doesn't leak it where possible, so these two
tests were accidentally compiling due to leaking '$OUT_DIR' for Cargo itself.

Auto merge of #3386 - matklad:warn-replace, r=alexcrichton

Warn if replace is not actually used

closes #3324

I've added warnings to `resolve_dependencies` inside `cargo_compile.rs`. Adding them to `resolve_with_previous` does not work because that method is called several times.

Perhaps the best idea is to refactor resolve facade, such that we have only `ops::resolve`  and not `ops::resolve_ws`, `ops::resolve_with_previous` and `ops::resolve_dependencies`. IIRC, there were some bugs because of logic mismatch between `compile` and `metadata` commands.

Fix compatibility with Rust nightly

These two tests actually shouldn't have ever passed, but nightly Rust is more
principled about "$OUT_DIR" and doesn't leak it where possible, so these two
tests were accidentally compiling due to leaking '$OUT_DIR' for Cargo itself.

Fix git rev showing up in version number

Makefiles didn't support an out-of-tree build, so needed to update them to do
so.

Auto merge of #3385 - matklad:package-test, r=alexcrichton

Add test for --package and virtual manifest

closes #3194

The issue was actually fixed by #3221 (thanks @euclio !), so let's just add a test (a copy of `virtual_works` basically).

Auto merge of #3371 - trafficland:master, r=alexcrichton

Add -vv to the list of options printed by cargo --help.

Compile statically against the MSVC CRT

This updates our AppVeyor builds to compile with `-Ctarget-feature=+crt-static`
to help Cargo be a bit more portable and not rely on the MSVC redistributable
artifacts. Over time this may even let us converge on only releasing one build
of Cargo and just pairing that with all Windows toolchains...

Test for issue #2064

Auto merge of #3395 - alexcrichton:fix-appveyor, r=alexcrichton

Pin to an older version of rustup temporarily

cc #3394

Pin to an older version of rustup temporarily

cc #3394

Fix some rebasing errors

Auto merge of #3392 - matklad:patch-1, r=steveklabnik

Add explanatory comment to header.html

See #3387

Add explanatory comment to header.html

See #3387

Warn if replace is not actually used

closes #3324

Add test for --package and virtual manifest

closes #3194

Added -vv to the list of options printed by cargo and its sub commands.

Be a bit more careful not to crash if the compiler doesn't support --crate-type metadata

tests

cargo check

Adds a new mode - check - which only checks code, rather than generating machine code. It takes into account that proc macros and build scripts will still require generated code.

Implemented by adding a check profile and setting this on each Unit, unless the unit is required to be built (i.e., is a proc macro, build script, or dep of one).

Auto merge of #3221 - euclio:test-all, r=alexcrichton

Add `--all` flag to `cargo test`

Work towards #2878.

This flag allows a user to test all members of a workspace by using `cargo test --all`. The command is also supported when using a virtual workspace manifest.

replace `current_opt` with membership check

fix formatting issues

remove unused argument to `Context::lib_profile`

remove unnecessary usage of `current_opt`

avoid round-tripping through string

remove unnecessary match

show warnings for all members of the workspace

pass `Packages` into resolve dependencies

remove current_package from context

add `--all` flag to `cargo-test`

avoid panic when virtual workspace has no members

Auto merge of #3361 - pwoolcoc:default-build-script, r=alexcrichton

Assume `build.rs` in the same directory as `Cargo.toml` is a build script (unless explicitly told not to)

So, in May I posted a question in the #cargo IRC room: https://botbot.me/mozilla/cargo/2016-05-26/?msg=66770068&page=1

This PR does what was discussed there. If `cargo` sees a `build.rs` in the same directory as the `Cargo.toml`, it will assume `build.rs` is a build script unless `package.build = false`. Just for completeness I also made `build = true` mean the same as `build = "build.rs"` but I'm not sure if that is okay or not.

change this to only emit a warning right now