fixing build script docs mistakes

Auto merge of #3284 - alexcrichton:fix-uploads, r=alexcrichton

Publish builds from the auto-cargo branch

Cargo doesn't use auto, it uses auto-cargo

Publish builds from the auto-cargo branch

Cargo doesn't use auto, it uses auto-cargo

Auto merge of #3280 - alexcrichton:test-features, r=brson

Fix passing --features when testing multiple packages

The wrong method was being passed to resolution accidentally. Features specified
via `--features` and `--no-default-features` are spec'd as only applying to the
*current* package, not all packages.

Closes #3279

Fix passing --features when testing multiple packages

The wrong method was being passed to resolution accidentally. Features specified
via `--features` and `--no-default-features` are spec'd as only applying to the
*current* package, not all packages.

Closes #3279

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

Bump to 0.15.0

Bump to 0.15.0

Auto merge of #3274 - alexcrichton:more-travis-again, r=alexcrichton

More travis again

Landing this again hopefully with the doc fix

Don't use travis-cargo to upload docs

Revert "Revert "Continuously publish Cargo builds""

This reverts commit b008422e92295c2e4de5b7de2284d963b55a6f8c.

Auto merge of #3259 - alexcrichton:more-ro, r=brson

Open crate files readonly first

This allows Cargo to work with read-only `CARGO_HOME` directories where the
cache was prepared ahead of time.

Closes #3256

Auto merge of #3270 - brson:cargo-docs, r=alexcrichton

Revert "Continuously publish Cargo builds"

This reverts commit 7799014dc7330445f62e2f925573ca15dd8e8e03.

Fixes https://github.com/rust-lang/cargo/issues/3269.

The reason doc.crates.io doesn't display correctly is that travis-cargo writes its own index.html that is just a meta-redirect to the api docs, overwriting cargo's own docs.

r? @alexcrichton

Revert "Continuously publish Cargo builds"

This reverts commit 7799014dc7330445f62e2f925573ca15dd8e8e03.

Auto merge of #3239 - alexcrichton:travis, r=brson

Continuously publish Cargo builds

This commit tweaks Cargo's automation to continuously publish builds on Travis
an AppVeyor. Once this is merged we can hopefully turn off all buildbot
automation related to Cargo and purely rely on Travis and AppVeyor for this
repository. All CI matrices are ported over to Travis and AppVeyor and a new
musl build of Cargo is even added just to test out adding that for a spin.

Currently Cargo will upload the final artifact for each target to a directory
keyed by the commit hash to a new bucket on S3, rust-lang-cargo-dev. Once we're
happy with the builds then we can change this to `rust-lang-cargo` or anything
else at that point.

Continuously publish Cargo builds

This commit tweaks Cargo's automation to continuously publish builds on Travis
an AppVeyor. Once this is merged we can hopefully turn off all buildbot
automation related to Cargo and purely rely on Travis and AppVeyor for this
repository. All CI matrices are ported over to Travis and AppVeyor and a new
musl build of Cargo is even added just to test out adding that for a spin.

Currently Cargo will upload the final artifact for each target to a directory
keyed by the commit hash to a new bucket on S3, rust-lang-cargo-dev. Once we're
happy with the builds then we can change this to `rust-lang-cargo` or anything
else at that point.

Auto merge of #3257 - nipunn1313:slash, r=alexcrichton

Add a [/] macro expansion for tests

Converts to backslash on windows. Currently, we're using [..],
so this will tighten up the tests.

Revert tests/read-manifest.rs

Open crate files readonly first

This allows Cargo to work with read-only `CARGO_HOME` directories where the
cache was prepared ahead of time.

Closes #3256

Replace instances of src[..]*.rs with [/]

Replace {sep} with [/] as well

Fix error in build-script

Add a [/] macro expansion for tests

Converts to backslash on windows. Currently, we're using [..],
so this will tighten up the tests.

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

Update makefile to 0.14.0

Update makefile to 0.14.0

Auto merge of #3254 - alexcrichton:unignore, r=alexcrichton

Un-ignore proc-macro tests

Auto merge of #3249 - matklad:workspace-profiles, r=alexcrichton

Use a single profile set per workspace

This aims to close #3206.

I have not figured out how to do this 100% backward compatibly, that's why I want to discuss this separately, although a related PR (#3221) is already in flight.

The problem is this: suppose that you have a workspace with two members, A and B and that A includes a profiles section and B does not. Now, mentally `cd` inside B and run `cargo build`. Today, Cargo will use a default profile. We want it to use a profile from A. So here the silent behavior switch will inevitably occur :( Looks like we can't detect this situation.

So this PR just switches the behavior to always use root profiles, and to print a warning if a non-root package specifies profiles. Feel free to reuse it in any form for #3221 if that's convenient!

Un-ignore proc-macro tests

Auto merge of #3253 - nipunn1313:exe, r=alexcrichton

Add an [EXE] hamcrest substitution for file extension

This should make the testing a bit more precise as to whether
we expect targets of the form foo.exe vs foo-abc123.exe. I
was also considering adding a [/] substitution for the fwd
slash vs backslash [..], but thought I would leave that to another PR
if we thought that was a good idea.

Backticks to tests/build-script.rs

Build scripts don't have .exe appended

Fix bench_dylib for win

Add an [EXE] hamcrest substitution for file extension

This should make the testing a bit more precise as to whether
we expect targets of the form foo.exe vs foo-abc123.exe. I
was also considering adding a [/] substitution for the fwd
slash vs backslash [..], but thought I would leave that to another PR
if we thought that was a good idea.

Auto merge of #3248 - alexcrichton:bad2, r=brson

Default urls in [replace] to crates.io

The intention was to do this, and it mistakenly didn't happen!

Closes #3235

Default urls in [replace] to crates.io

The intention was to do this, and it mistakenly didn't happen!

Closes #3235

Use a single profile set per workspace

Auto merge of #3243 - alexcrichton:target-feature, r=brson

Expose rustc cfg values to build scripts

This commit is Cargo's portion of the implementation of [RFC 1721] where it will
expose values printed by `rustc --print cfg` to build scripts.

[RFC 1721]: https://github.com/rust-lang/rfcs/blob/master/text/1721-crt-static.md

This will in turn be used to communicate features like `-C
target-feature=+crt-static` which can be used to compile objects for statically
linking against the msvcrt on MSVC.

Auto merge of #3220 - alexcrichton:bad, r=brson

Load [replace] sections from lock files

This commit fixes a bug in Cargo where path-based [replace] dependencies were
accidentally not loaded from lock files. This meant that even with a lock
file some compilations could accidentally become nondeterministic. The fix here
is to just look at all path dependencies, even those specified through [replace]

Closes #3216

Auto merge of #3175 - alexcrichton:bench-abort, r=brson

Ignore `panic` configuration for test/bench profiles

Both of these profiles link to libtest, so it's invalid to configure them with
`panic="abort"`. To prevent confusing errors just ignore the configuration for
now.

Closes #3166

Expose rustc cfg values to build scripts

This commit is Cargo's portion of the implementation of [RFC 1721] where it will
expose values printed by `rustc --print cfg` to build scripts.

[RFC 1721]: https://github.com/rust-lang/rfcs/blob/master/text/1721-crt-static.md

This will in turn be used to communicate features like `-C
target-feature=+crt-static` which can be used to compile objects for statically
linking against the msvcrt on MSVC.

Auto merge of #3246 - alexcrichton:mtimes-in-the-past, r=alexcrichton

Continuation of #2880

Just applied a patch to help fix the tests.

Auto merge of #3247 - alexcrichton:doc-plugin, r=alexcrichton

Continuation of #3114

Added a test

Add a test for the previous PR

Merge branch 'issue3107-doc-and-targets' of https://github.com/Phaiax/cargo into doc-plugin

Fix some tests from the previous merge

Merge branch 'mtimes-in-the-past' of https://github.com/integer32llc/cargo into mtimes-in-the-past

Ignore `panic` configuration for test/bench profiles

Both of these profiles link to libtest, so it's invalid to configure them with
`panic="abort"`. To prevent confusing errors just ignore the configuration for
now.

Closes #3166

Auto merge of #3242 - jhbabon:cfgs-flags-per-package, r=alexcrichton

FIX: Call rustdoc test with the correct cfg flags of a package.

There was a situation in which if you you had a lib that depends on a package with features, whenever you ran the tests for the package the `rustdoc test` call was failing because it was called with the root `cfg` flags, not the package `cfg` flags.

This fix solves the issue by keeping track of the `cfg` flags per package, so the `rustdoc` command will be generated with the correct `cfg` flags.

Closes #3172

Add text language to code bloks in machine-readable-output.md

Travis fails when running `make doc` because of this file.
This should fix the issue.

Relax conditions in rustdoc test cfg flags tests.

In Windows the rustdoc test output sets more double quotes, so
the test doesn't pass. We need to relax the test so it pass in
*NIX and Windows environments.

Use better iter() semantics to collect cfg flags.

FIX: Call rustdoc test with the correct cfg flags of a package.

There was a situation in which if you you had a lib that depends
on a package with features, whenever you ran the tests for the
package the `rustdoc test` call was failing because rustdoc
was called with the root cfg flags, not the package cfg flags.

This fix solves the issue by keeping track of the cfg flags
per package, so the rustdoc command will be generated with
the correct cfg flags.

Auto merge of #3241 - alexcrichton:curl-update, r=alexcrichton

Update curl and curl-sys

Pick up a few security related fixes in libcurl

Update curl and curl-sys

Pick up a few security related fixes in libcurl

Auto merge of #3213 - matklad:simler-locks, r=alexcrichton

Append only git checkouts

Ok, so this should fix #3209 by using a full git revision as the name of the checkout path.

I don't fully understand the code here, and I haven't thought about backwards/forwards compatibility hard, but the approach should be viable, and maybe even the implementation here is OK?

Auto merge of #3238 - alexcrichton:fix-dox, r=alexcrichton

Render machine-readable-output docs

Closes #3237

Render machine-readable-output docs

Closes #3237

Auto merge of #3231 - TimNN:patch-1, r=alexcrichton

Fix rust-lang/rust#35203 warning/error

rust-lang/rust#35203 made patterns in functions without body into a warn by default lint (which is being `deny`ed here).

cc rust-lang/rust#37378, rust-lang/rust#37416

Auto merge of #3179 - alexcrichton:fix-features-mulitple, r=matklad

Parse --features in `cargo metadata` the same

This accidentally didn't accept space-separated features passed through
`--features` as the logic for doing the splitting wasn't shared. Let's share it!

Fix rust-lang/rust#35203 warning/error

rust-lang/rust#35203 made patterns in functions without body into a warn by default lint (which is being `deny`ed here).

Auto merge of #3230 - alexcrichton:better-download, r=alexcrichton

Download mingw from more reliable location

Use the same "mirror" the rust repo uses

Download mingw from more reliable location

Use the same "mirror" the rust repo uses

Auto merge of #3217 - alexcrichton:fix-regression, r=brson

Ignore summaries in downloaded crates

Unfortunately historical Cargo bugs have made it such that the index sometimes
differs from the actual crate we download. Let's respect the index, however,
which should be our source of truth.

Closes #3214

Ignore summaries in downloaded crates

Unfortunately historical Cargo bugs have made it such that the index sometimes
differs from the actual crate we download. Let's respect the index, however,
which should be our source of truth.

Closes #3214

Auto merge of #3229 - martinhath:master, r=alexcrichton

Add fix and test for #3224

This commit fixes #3224, and adds a test for it.
First time contributor to Cargo here, so I'm sorry if I put stuff the wrong places :)

Add fix and test for #3224

Auto merge of #3226 - alexcrichton:more-cross, r=alexcrichton

Download more cross builds

Download more cross builds

Load [replace] sections from lock files

This commit fixes a bug in Cargo where path-based [replace] dependencies were
accidentally not loaded from lock files. This meant that even with a lock
file some compilations could accidentally become nondeterministic. The fix here
is to just look at all path dependencies, even those specified through [replace]

Closes #3216

Auto merge of #2818 - robinst:use-exec-for-cargo-run, r=alexcrichton

Use CommandExt::exec for `cargo run` on Unix (#2343)

Before, we would spawn a child process for the program. One of the
problems with that is when killing the cargo process, the program
continues running.

With this change, the cargo process is replaced by the program, and
killing it works.

Before (`cargo run` is the parent):

> 502  7615  7614   0  2:26PM ttys012    0:00.12 /Users/rstocker/.multirust/toolchains/stable-x86_64-apple-darwin/bin/cargo run
> 502  7620  7615   0  2:26PM ttys012    0:00.01 target/debug/program

After (the shell is the parent):

> 502 81649 81648   0  5:27PM ttys012    0:00.69 -zsh
> 502  7739 81649   0  2:26PM ttys012    0:01.27 target/debug/program

Parse --features in `cargo metadata` the same

This accidentally didn't accept space-separated features passed through
`--features` as the logic for doing the splitting wasn't shared. Let's share it!

Use CommandExt::exec for `cargo run` on Unix (#2343)

Before, we would spawn a child process for the program. One of the
problems with that is when killing the cargo process, the program
continues running.

With this change, the cargo process is replaced by the program, and
killing it works.

Append only git checkouts

Auto merge of #3149 - cardoe:fix-pkgid-cmd, r=alexcrichton

pkgid cmd: fix help msg with multiple packages

When there are multiple versions of a package that match a given spec
the command tells you to re-run it with the `-p` argument which does not
exist. The command appears to work without it properly.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>

pkgid: add the -p flag as the output suggests

When you use this command and there are multiple packages that match
this spec, the help instructions tell you to rerun it with the -p
argument which was previously not present. Since the code paths are
shared with 'cargo build' and 'cargo update', this adds the -p argument
to 'cargo pkgid' to make things more consistent.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>

Auto merge of #3207 - steveklabnik:gh3202, r=alexcrichton

bump semver to 0.5.1

Fixes #3202

I've verified locally that this works, and have a test for it within semver. Two, actually.

@censoredusername if you could give this a shot to double check that it fixes your exact bug, that'd be cool.

bump semver to 0.5.1

Fixes #3202

Auto merge of #3205 - matklad:fix-rustdoc-ld-path, r=alexcrichton

Pass target environment for rustdoc

This should fix #3200, but I am not sure that this is a correct fix, and I need some input to figure this out.

`rustdoc` is invoked in two places, in `cargo_test.rs` and in `cargo_rustc/mod.rs`. Before the refactoring PR, these invocations used different LD_LIBRARY paths. [The one in cargo_rustc](https://github.com/rust-lang/cargo/pull/3198/files#diff-59acd1a3101aebbb591ac7ab51c19d9eR427) used "host" version, while [the one in cargo_test](https://github.com/rust-lang/cargo/blob/a8baa5b8f36e88170c8c56523b6eb72efc2cc55e/src/cargo/ops/cargo_test.rs#L131) used "target" version.

The original PR changed both to "host", this PR switches both to "target". Is this correct, or should we stick with different environments for building documentation and doctests?

Pass target environment for rustdoc

Auto merge of #3201 - alexcrichton:more-cross, r=alexcrichton

Prepare for more cross-compiled Cargos

Update deps, add support, ensure everything compiles right.

Prepare for more cross-compiled Cargos

Update deps, add support, ensure everything compiles right.

Auto merge of #3198 - matklad:kill-command-type, r=alexcrichton

Remove CommandType struct

This removes `CommandType` struct as well as `cargo_rustc::process` function. So now all process creation goes thorough methods of `Compilation`.

This does change search path order from `util::dylib_path(), host_dylib_path()` to `host_dylib_path(), util::dylib_path()`, but I hope this is not a problem.

This also uncovers the fact that `rustdoc` is run sometimes with and sometimes without `host_dylib_path`. Is this intentional?

Store host dylib path in compilation

Remove CommandType struct

Auto merge of #3193 - matklad:kill-command-proto, r=alexcrichton

Remove command prototype

A followup of #3177 . I am not sure, but perhaps we can remove/refactor `CommandType` as well: for each command variant, `Compilation` as a public dedicated method, but it also has a generic one (`process`) (haha: https://github.com/rust-lang/cargo/pull/1107/files#r22429844).

Remove command prototype

Auto merge of #3187 - edunham:appveyor-gnu, r=alexcrichton

Add GNU triples for #3186

Resources used to take an educated guess at this:
* Servo's appveyor.yml
* Rust's Buildbot config
* Alex
* Some random appveyor config that Alex had sitting around

Let's see if it works!

Use a MinGW that doesn't break pthreads

Also correct the bits to triples mapping, because tyops

Expand matrix for #3186

Auto merge of #3189 - edunham:fix-empty-path-bug, r=alexcrichton

Make it possible for windows-gnu tests to pass

Skip test that uses empty path on Windows, because Windows doesn't do the empty path thing.

empty path is not a thing on Windows

Auto merge of #3185 - semarie:bump-hamcrest, r=alexcrichton

bump hamcrest to 0.1.1

update hamcrest to 0.1.1 and its dependencies.

Closes: #3181

bump hamcrest to 0.1.1

Auto merge of #3183 - rjgoldsborough:broken-docs-links, r=alexcrichton

removing return false causing links to break

Gah! Sorry about that.

fixes #3182

I added a `return false` to the function above to prevent it from toggling itself back off but for some reason put one here as well which stopped the event before the link could fire.

removing return false causing links to break

Auto merge of #3160 - alexcrichton:rustc-to-proc, r=alexcrichton

Blanket rename rustc-macro to proc-macro

Auto merge of #3154 - steveklabnik:gh3124, r=alexcrichton

upgrade semver

This is a spike at fixing #3124.

@alexcrichton I'm not sure how to actually emit the error message here. In the TOML example you linked me: https://github.com/rust-lang/cargo/blob/5593045ddef2744c1042dee0c0037c2ebcc1834e/src/cargo/util/toml.rs#L166

That takes a Config as an argument. This function does not. What's the right approach here?

Also, this code is messy. I am 99% sure I can write it nicer with combinators. This is just to get the discussion going.

Bump to semver@0.5.0

Add a test for deprecation warnings

Tweak message in Dependency::parse

Provide some contextual information about why a dependency failed to parse.

Auto merge of #3177 - matklad:kill-exec-engine, r=alexcrichton

Remove ExecEngine abstraction

Hi! Not sure what was the idea behind exec engine (perhaps to allow swapping it out during tests?), but looks like it does absolutely nothing at the moment, and can be removed.