Auto merge of #1482 - DavidAwad:patch-1, r=alexcrichton

Auto merge of #1483 - tamird:rustup, r=alexcrichton

Passes `make test` locally. r? @alexcrichton

Rustup 2015-04-02

Make `print-new-snapshot` fail on 403

Update index.md

Auto merge of #1475 - alexcrichton:update, r=huonw

Along the way, this also removes the ability to specify `[[lib]]` targets while
getting warnings. While we're at it remove nearly all unstable features that
Cargo is depending on as well!

r? @huonw

Auto merge of #1452 - alexcrichton:issue-1355, r=huonw

The hashes would occasionally disappear when the main package registry overwrote
a previous registry source, so this just adds logic to load hashes if they're
missing.

Closes #1355

Update to rust-2015-04-02

Along the way, this also removes the ability to specify `[[lib]]` targets while
getting warnings. While we're at it remove nearly all unstable features that
Cargo is depending on as well!

Auto merge of #1476 - brson:installer, r=alexcrichton

Upgrade rust-installer

Auto merge of #1455 - alexcrichton:issue-1449, r=brson

Previously --extern was passed for *all* upstream dependencies, causing
conflicts if some had duplicate names. Now cargo only passes --extern for
libraries that were built including immediate dependencies. Cargo also
additionally now properly passes `-L dependency=` instead of just a plain `-L`.

Closes #1449

Fix passing --extern and -L to doc tests

Previously --extern was passed for *all* upstream dependencies, causing
conflicts if some had duplicate names. Now cargo only passes --extern for
libraries that were built including immediate dependencies. Cargo also
additionally now properly passes `-L dependency=` instead of just a plain `-L`.

Closes #1449

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

Use Entry::or_insert where possible

Update to rust 2015-03-30

Auto merge of #1467 - alexcrichton:update-flate2, r=alexcrichton

There were a few bugs in creation of gzip'd files which caused package creation
to fail in some circumstances.

Closes #1465

Help diagnose flaky test

Update flate2-rs

There were a few bugs in creation of gzip'd files which caused package creation
to fail in some circumstances.

Closes #1465

Auto merge of #1461 - brson:installer, r=alexcrichton

Update rust-installer. Fixes --help and interop with NixOS

Auto merge of #1405 - alexcrichton:issue-1401, r=alexcrichton

This should inadvertently fix #1401

Use new `kind=` syntax for libs throughout Cargo

This should inadvertently fix #1401

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

Down with as_slice!

Auto merge of #1460 - apasel422:1459, r=alexcrichton

closes #1459

Auto merge of #1457 - japaric:doc-target, r=alexcrichton

closes #1427

r? @alexcrichton
cc @dpc

pass translated crate name to doc tests

closes #1459

Update to rust 2015-03-26

add --target flag support to cargo doc

closes #1427

Auto merge of #1453 - alexcrichton:doc-assert, r=huonw

Closes #1454

Auto merge of #1451 - alexcrichton:issue-1450, r=huonw

Examples, tests, binaries, etc, are all allowed to have hyphens (cargo does some
renaming internally).

Close #1450

Only disallow hyphens in lib target names

Examples, tests, binaries, etc, are all allowed to have hyphens (cargo does some
renaming internally).

Close #1450

Fix an erroneous assertion in `cargo doc`

Lazily load hashes for registry sources

The hashes would occasionally disappear when the main package registry overwrote
a previous registry source, so this just adds logic to load hashes if they're
missing.

Closes #1355

Auto merge of #1443 - alexcrichton:hyphens-to-underscores, r=alexcrichton

This change allows *packages* to have hyphens in them, but they are always
translated to underscores when translated to a crate name. This means that all
crates are compiled with a `--crate-name` that has no hyphens (as well as
`--extern` directives having no hyphens).

Binaries and examples, however, are allowed to contain
hyphens in their name. The "crate name" will still have an underscore, but the
output will be in the same dasherized name.

Explicitly named targets are not allowed to have hyphens in them as well.

Normalize hyphens to underscores in crate names

This change allows *packages* to have hyphens in them, but they are always
translated to underscores when translated to a crate name. This means that all
crates are compiled with a `--crate-name` that has no hyphens (as well as
`--extern` directives having no hyphens).

Binaries, examples, benchmarks, and tests, however, are allowed to contain
hyphens in their name. The "crate name" will still have an underscore, but the
output will be in the same dasherized name.

Explicitly named targets are not allowed to have hyphens in them as well.

Auto merge of #1445 - alexcrichton:issue-727, r=alexcrichton

Closes #727

Add a test for tests in benchmarks

Closes #727

Auto merge of #1444 - alexcrichton:issue-1398, r=huonw

This commit removes the ndebug support from Cargo and also adds a new
configuration option for profiles, `debug-assertions`, which controls whether
debug assertions in the compiler are turned on or not.

Closes #1398

Remove ndebug, add config of debug assertions

This commit removes the ndebug support from Cargo and also adds a new
configuration option for profiles, `debug-assertions`, which controls whether
debug assertions in the compiler are turned on or not.

Closes #1398

Auto merge of #1441 - alexcrichton:issue-1323, r=bson

This commit is a complete overhaul of how Cargo handles compilation profiles
internally. The external interface of Cargo is not affected by this change.

Previously each Target had a Profile embedded within it. Conceptually a Target
is an entry in the manifest (a binary, benchmark, etc) and a Profile controlled
various flags (e.g. --test, -C opt-level, etc). Each Package then contained many
profiles for each possible compilation mode. For example a Package would have
one target for testing a library, benchmarking a library, documenting a library,
etc. When it came to building these targets, Cargo would filter out the targets
listed to determine what needed to be built.

This filtering was largely done based off an "environment" represented as a
string. Each mode of compilation got a separate environment string like `"test"`
or `"bench"`. Altogether, however, this approach had a number of drawbacks:

* Examples in release mode do not currently work. This is due to how examples
  are classified and how release mode is handled (e.g. the "release" environment
  where examples are meant to be built in the "test" environment).
* It is not trivial to implement `cargo test --release` today.
* It is not trivial to implement `cargo build --bin foo` where *only* the binary
  `foo` is built. The same is true for examples.
* It is not trivial to add selective building of a number of
  binaries/examples/etc.
* Filtering the list of targets to build has some pretty hokey logic that
  involves pseudo-environments like "doc-all" vs "doc". This logic is duplicated
  in a few places and in general is quite brittle.
* The TOML parser greatly affects compilation due to the time at which profiles
  are generated, which seems somewhat backwards.
* Profiles must be overridden, but only partially, at compile time becuase only
  the top-level package's profile is applied.

In general, this system just needed an overhaul. This commit made a single
change of separating `Profile` from `Target` and then basically hit `make` until
all the tests passed again. The other large architectural changes are:

* Environment strings are now entirely gone.
* Filters are implemented in a much more robust fashion.
* Release mode is now handled much more gracefully.
* The unit of compilation in the backend is no longer (package, target) but
  rather (package, target, profile). This change had to be propagated many
  location in the `cargo_rustc` module.
* The backend does not filter targets to build *at all*. All filtering now
  happens entirely in the frontend.

I'll test issues after this change lands, but the motivation behind this is to
open the door to quickly fixing a number of outstanding issues against Cargo.
This change itself is not intended to close many bugs.

Fix tests for windows

len() == 0 => is_empty()

Auto merge of #1440 - tamird:rustup, r=alexcrichton

@alexcrichton

Make sure test directory exists

Remove unneeded `allow`s

Update to rust 2015-03-22

Fix printing `Fresh` for crates

Fix trying to document build scripts

Fix documenting binaries with libraries

Fix `cargo doc`

Both tweak how deep documentation is compiled as well as fixing up the
dependencies for a documented target.

Update tests for tweaked error messages

Don't factor bins into fingerprint calculations

The dependency is just there to get them to compile, not to actually induce
recompiles.

Fix fingerprints of build scripts that aren't compiled

Update some error messages in tests

Don't compile build scripts if they aren't run

Fix `cargo run` with no binaries

Update error message for missing examples

Fix test for bins/examples

Binaries are no longer always built when examples are built, so update the test
accordingly.

List examples in binaries built

This is required for `cargo run` to work

Don't fail `cargo run` with bins + examples

If no filter is supplied, then only consider binaries as candidates, not
examples as well.

Fix a subtesting test

Libraries are no longer built if they aren't doctested, so something else needed
to be present to trigger the library being built.

Only doctest libs by default

Fix configuring lib targets with profile options

Fix cargo test --no-run

Fix test for `--test` filtering

The flag now only applies to integration tests, not all targets with the
specified name.

Remove an extraneous is_test condition

If a profile's `test` flag is set to true, then it will always generate a unit
test so the `is_test` check is redundant.

Fix cargo test filtering for binaries

This adds a new `--bin` flag to `cargo test` to specifically say that a binary
should be tested. Additionally the dependencies are tweaked such that binaries
to not depend on themselves being available.

Overhaul how cargo treats profiles

This commit is a complete overhaul of how Cargo handles compilation profiles
internally. The external interface of Cargo is not affected by this change.

Previously each Target had a Profile embedded within it. Conceptually a Target
is an entry in the manifest (a binary, benchmark, etc) and a Profile controlled
various flags (e.g. --test, -C opt-level, etc). Each Package then contained many
profiles for each possible compilation mode. For example a Package would have
one target for testing a library, benchmarking a library, documenting a library,
etc. When it came to building these targets, Cargo would filter out the targets
listed to determine what needed to be built.

This filtering was largely done based off an "environment" represented as a
string. Each mode of compilation got a separate environment string like `"test"`
or `"bench"`. Altogether, however, this approach had a number of drawbacks:

* Examples in release mode do not currently work. This is due to how examples
  are classified and how release mode is handled (e.g. the "release" environment
  where examples are meant to be built in the "test" environment).
* It is not trivial to implement `cargo test --release` today.
* It is not trivial to implement `cargo build --bin foo` where *only* the binary
  `foo` is built. The same is true for examples.
* It is not trivial to add selective building of a number of
  binaries/examples/etc.
* Filtering the list of targets to build has some pretty hokey logic that
  involves pseudo-environments like "doc-all" vs "doc". This logic is duplicated
  in a few places and in general is quite brittle.
* The TOML parser greatly affects compilation due to the time at which profiles
  are generated, which seems somewhat backwards.
* Profiles must be overridden, but only partially, at compile time becuase only
  the top-level package's profile is applied.

In general, this system just needed an overhaul. This commit made a single
change of separating `Profile` from `Target` and then basically hit `make` until
all the tests passed again. The other large architectural changes are:

* Environment strings are now entirely gone.
* Filters are implemented in a much more robust fashion.
* Release mode is now handled much more gracefully.
* The unit of compilation in the backend is no longer (package, target) but
  rather (package, target, profile). This change had to be propagated many
  location in the `cargo_rustc` module.
* The backend does not filter targets to build *at all*. All filtering now
  happens entirely in the frontend.

I'll test issues after this change lands, but the motivation behind this is to
open the door to quickly fixing a number of outstanding issues against Cargo.
This change itself is not intended to close many bugs.

Auto merge of #1433 - alexcrichton:dylib, r=huonw

This may not end up being stable for 1.0, and we don't really need much
functionality from it anyway.

Remove dependency on DynamicLibrary

This may not end up being stable for 1.0, and we don't really need much
functionality from it anyway.

Auto merge of #1431 - IvanUkhov:newlines, r=alexcrichton

Hi,

It seems that people generally prefer not having newlines after section headers in their `Cargo.toml`s. However, when one creates a new package via `cargo new`, a newline gets automatically inserted after `[package]`. Then a typical `Cargo.toml` ends up being some what inconsistent: there is a newline after `[package]`, and there are no newlines after other headers.

On top of that, the examples given on crates.io do not seem to be consistent: [sometimes](http://doc.crates.io/index.html) there is a newline after `[package]`, and [sometimes](http://doc.crates.io/manifest.html) there is none. The same applies to other section names.

I am wondering if there should be some style conventions in this regard.

Regards,
Ivan

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

Make the usage of newlines in Cargo.toml consistent

Auto merge of #1429 - IvanUkhov:build-script, r=alexcrichton

Update to rust master

Fix a typo in the description of the links manifest key

Auto merge of #1425 - alexcrichton:issue-1421, r=huonw

This was accidentally broken in a recent refactoring.

Closes #1421

Re-fix the `help` command's implementation

This was accidentally broken in a recent refactoring.

Closes #1421

Auto merge of #1402 - cmr:issue-1015, r=alexcrichton

Deprecates the rustc-args metadata keyword in favor of rustc-link-lib and
rustc-link-search, which are more precise and allows for easy, correct
handling of spaces in pathnames.

Closes #1015

Add rustc_private feature to tests which need it.

Add dedicated metadata for adding -l/-L flags

Deprecates the rustc-args metadata keyword in favor of rustc-link-lib and
rustc-link-search, which are more precise and allows for easy, correct
handling of spaces in pathnames.

Closes #1015

Auto merge of #1412 - jakub-:rust-nightly, r=alexcrichton

Update to the latest nightly

Auto merge of #1409 - alexcrichton:less-compiles, r=huonw

This was previously required to ensure that all relevant metadata files would
remain in place and not get deleted, but this auto-clean behavior has since been
removed so this should no longer be necessary.

Auto merge of #1406 - alexcrichton:issue-1404, r=wycats

When loading targets to compile, be sure to use the targets from the package
that's being passed down to the compilation step, not the one that was passed in
which is overridden.

Closes #1404

Auto merge of #1410 - johnz133:patch-2, r=huonw

cargo build now outputs to /target/debug, changed doc to reflect that.
steve suggested to add description for cargo build --release
autogenerates to guide.html. Used pre and code tags instead of ``` by following previous examples.

Updated build in guide.md

autogenerates to guide.html

Don't add non-compiled packages to the graph

This was previously required to ensure that all relevant metadata files would
remain in place and not get deleted, but this auto-clean behavior has since been
removed so this should no longer be necessary.

Don't intermingle targets between packages

When loading targets to compile, be sure to use the targets from the package
that's being passed down to the compilation step, not the one that was passed in
which is overridden.

Closes #1404

Auto merge of #1396 - alexcrichton:issue-1390, r=huonw

When calculating the output directory filename, only use the "file name"
component of the target triple specified as otherwise the output could be placed
into an odd location.

Closes #1390

Improve experience with the flexible target specification

When calculating the output directory filename, only use the "file name"
component of the target triple specified as otherwise the output could be placed
into an odd location.

Closes #1390

Auto merge of #1395 - alexcrichton:profile, r=brson

* Don't use `{:?}` in profiling output, it's too noisy
* Allow an integer to be specified with `CARGO_PROFILE` indicating how many
  levels deep should be printed.

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

Continuation of #1384

Fix all tests from the update to rust master

Reduce usage of unstable features

Update to rust master

Merge branch 'master' of https://github.com/Victor-Savu/cargo into update

Improve the profiling output of Cargo

* Don't use `{:?}` in profiling output, it's too noisy
* Allow an integer to be specified with `CARGO_PROFILE` indicating how many
  levels deep should be printed.

Auto merge of #1389 - jimmycuadra:patch-1, r=alexcrichton

Add documentation for the `harness` manifest option, introduced in a6589681351dc4a87b6fc268d47b7d608004b455.

Document the `harness` manifest option.

Add documentation for the `harness` manifest option, introduced in a6589681351dc4a87b6fc268d47b7d608004b455.