use Box::leak and clean up our string leak

use impl Trait

Auto merge of #5503 - ehuss:fix-rustc-test, r=matklad

Fix `cargo rustc` for test with implicit binary.

Fixes #5502

Auto merge of #5507 - ordovicia:no-secs-frac, r=matklad

Does not print seconds fraction with minutes

As discussed in https://github.com/rust-lang/cargo/pull/5456#issuecomment-387731047,
seconds fraction seems unnecessary when the elapsed time is reported in minutes.

Remove unnecessary computation in printing elapsed time

Does not print seconds fraction with minutes

Fix `cargo rustc` for test with implicit binary.

Fixes #5502

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

Bump to 0.29.0

Bump to 0.29.0

Auto merge of #5497 - dwijnand:patch-1, r=alexcrichton

Mention +nightly in ARCHITECTURE.md

See the conversation on Gitter: https://gitter.im/rust-lang/cargo?at=5aee83305cf0b830045d0602.

typo

Mention +nightly in ARCHITECTURE.md

Auto merge of #5493 - ehuss:fix-changing_bin_features_caches_targets, r=matklad

Fix random Windows CI error for changing_bin_features_caches_targets

Fixes #5481.

Fix random Windows CI error for changing_bin_features_caches_targets

Fixes #5481.

Auto merge of #5491 - ehuss:profile-spec, r=alexcrichton

Add package spec support to profile overrides.

Note: It errors out if multiple overrides match the same package.  This could potentially merge the overrides together with a hierarchy similar to CSS specificity.  However, that seems overkill for now.

Add package spec support to profile overrides.

Note: It errors out if multiple overrides match the same package.  This could potentially merge the overrides together with a hierarchy similar to CSS specificity.  However, that seems overkill for now.

Auto merge of #5461 - matklad:meta-rename, r=alexcrichton

Support crate renames in `cargo metadata`

This adds information about (currently unstable) crate renames to metadata. Unfortunately, we already expose dependencies as a list of package ids (which are strings), so we can't easily add a `rename` field easily. For this reason, I've added a parallel `deps` key, which basically deprecates `dependencies` field.

So, the new format looks like this

```JavaScript
{
    "dependencies": [
        "bar 0.2.0 (registry+https://github.com/rust-lang/crates.io-index)",
        "bar 0.1.0 (registry+https://github.com/rust-lang/crates.io-index)"
    ],
    "deps": [
        {
            "extern_crate_name": "baz", // this one is actually renamed
            "id": "bar 0.2.0 (registry+https://github.com/rust-lang/crates.io-index)"
        },
        {
            "extern_crate_name": "bar",
            "id": "bar 0.1.0 (registry+https://github.com/rust-lang/crates.io-index)"
        }
    ],
    "features": [],
    "id": "foo 0.5.0 (path+file://[..])"
},
```

Questions:

* Is there a better name for `extern_crate_name`? This name is precise, but it's longer than I like, and might become opaque in meaning if we remove `extern crate` from the language :)
* Should we feature gate this (i.e, only produce `deps` if we have a feature in `Cargo.toml`)? I think the answer is yes, but that'll require threading `Features` to `Workspace`...

r? @alexcrichton

Expand dependency info of cargo --metadata

The old `dependencies` key in `resolve` is deprecated.

Instead, the new `deps` key introduced which lists dependencies kinds
(dev/build/normal) as well as `extern crate` name for the dep.

Extract Package::lib_target method

Auto merge of #5490 - alexcrichton:fix-thrash, r=matklad

Factor in `used_in_plugin` to target filenames

This prevents thrashing the cache of compiled libraries for when they're used in
a plugin or not.

Factor in `used_in_plugin` to target filenames

This prevents thrashing the cache of compiled libraries for when they're used in
a plugin or not.

Auto merge of #5489 - dwijnand:drop-GitSource-TODO, r=matklad

Drop PathSource TODO in GitSource

This effort was started in de6944798cf2940b91c742b8d6cd4294ea28f76a,
and looks (to me) to be resolved at this point.

Auto merge of #5488 - alexcrichton:Retry, r=matklad

Cop out on fixing a spurious test failure

This commit cops out trying to fix `rename_with_link_search_path` by simply
adding a loop on Windows to retry the operation that looks to need retrying.

Drop PathSource TODO in GitSource

This effort was started in de6944798cf2940b91c742b8d6cd4294ea28f76a,
and looks (to me) to be resolved at this point.

Cop out on fixing a spurious test failure

This commit cops out trying to fix `rename_with_link_search_path` by simply
adding a loop on Windows to retry the operation that looks to need retrying.

Auto merge of #5358 - djc:options-build-config, r=matklad

Let CompileOptions create a BuildConfig directly

This puts input validation in a more central place and prevents copying/moving stuff around as separate values.

Auto merge of #5487 - matklad:mixed-deps, r=alexcrhiton

FIx false positive warning

We warn if a feature was specified corresponding to a dependency which
is not optional. However, a dependency can be both optional and
required, and we shouldn't warn in that case.

cc https://github.com/rust-lang/cargo/pull/5480

r? @alexcrichton

FIx false positive warning

We warn if a feature was specified corresponding to a dependency which
is not optional. However, a dependency can be both optional and
required, and we shouldn't warn in that case.

Auto merge of #5486 - ehuss:deprecate-doc, r=matklad

Deprecate `doc` profile.

Fixes #5434

Deprecate `doc` profile.

Fixes #5434

Auto merge of #5485 - patriksvensson:feature/GH-5474, r=matklad

Do not allow running library examples.

This is my first contribution to anything Rust related that I haven't written myself, and I'm not a very proficient Rust programmer (yet), so would love some feedback on improvements.

I also might have solved this the wrong way... 😄

Closes #5474

(cc @matklad)

Made adjustments based on feedback from review.

Do not allow running library examples.

Closes #5474

Auto merge of #5480 - alexcrichton:fix-regr, r=matklad

Fix optional deps in multiple sections

This commit fixes an issue where an optional dependency was listed multiple
times in a manifest (multiple sections). This regression was introduced by #5415
and happened because in the resolver we didn't record a `Dependency` as it was
accidentally deduplicated too soon.

The fix here was to ensure that all `Dependency` annotations make their way into
`Resolve` now that we rely on the listed `Dependency` values for correctness.

Closes #5475

Fix optional deps in multiple sections

This commit fixes an issue where an optional dependency was listed multiple
times in a manifest (multiple sections). This regression was introduced by #5415
and happened because in the resolver we didn't record a `Dependency` as it was
accidentally deduplicated too soon.

The fix here was to ensure that all `Dependency` annotations make their way into
`Resolve` now that we rely on the listed `Dependency` values for correctness.

Closes #5475

Auto merge of #5483 - ehuss:disable-panic-test, r=matklad

Disable --all-targets panic tests until we can land a fix.

Disable --all-targets panic tests until we can land a fix.

Auto merge of #5479 - matklad:ghost-hunting, r=alexcrichton

Investigating spurious windows errors

r? @alexcrichton

Try to diagnose/fix spurious windows error with better rm -r

Revert "Attmept to fix a spurious failure on AppVeyor"

This reverts commit db3328f8f7b96701faa62756a0d55ed71899d894.

We get spurious errors for thouse tests even with debug info disabled,
so let's reenable it back.

Auto merge of #5476 - dwijnand:Registry-vs-Source-confusion, r=matklad

Try & simplify Registry vs Source confusion

Refs #3006.

I tried to go further and replace references of `Registry` with `PackageRegistry`, but I ran into lifetime issue in `ops::resolve_with_previous` which I think has to do with no longer passing `PackageRegistry` as a `Registry` trait object (with a new and distinct lifetime).

Trash RegistryBuilder, dead cfg(test) code

Re-organise Source impls method definition order

Optimised for minimised PR diff, and thus reviewability

Re-simplify GitSource::query

Move query definitions back into Source impls!

Remove all "impl Registry for XSource" blocks!

Drop unnecessary Registry for Box

Duplicate query in Source, drop Registry parent type

.. and move query definitions to the struct impls.

Start moving methods from Registry to Source

For now just move `supports_checksums` and `requires_precise` methods
down from `Registry` to `Source`, as they're not as entangled as
`query`.

Rename default() method that does not implement Default

Store BuildConfig directly in CompileOptions

Pass full build_config to generate_targets()

Move rustc and target configs from BuildConfig into BuildContext

Pass Workspace directly into Config::rustc()

This means we can store the cache file name in one place.

Proxy host_triple() and target_triple() on BuildContext

Initialize Compilation fields directly from BuildContext

Pass target configs directly to BuildState::new()

Move BuildConfig and dependencies into new module

Move more input validation into BuildConfig::new()

Store compile mode as enum in BuildConfig

Expose CompileMode in command_prelude

Store message format as enum in BuildConfig

Auto merge of #5472 - alexcrichton:rename-rust, r=matklad

Rename the `rust` manifest key to `edition`

This'll hopefully jive better with the terminology of "edition" throughout the
rest of Rust!

Rename the `rust` manifest key to `edition`

This'll hopefully jive better with the terminology of "edition" throughout the
rest of Rust!

Auto merge of #5473 - ehuss:autobin-warning, r=alexcrichton

Address autobins warning.

Removes the large warning displayed if you build Cargo with itself (but adds a warning if you build with an older Cargo).  Not sure which approach you prefer to address it, or if you want to do this later.

Address autobins warning.

Auto merge of #5470 - alexcrichton:hopeful-fix, r=matklad

Attmept to fix a spurious failure on AppVeyor

I *think* the issue is that `link.exe` is generating debuginfo in the background
which keeps the file open and prevents us from deleting it, but hopefully by
disabling debug information we'll either be able to confirm or deny this
hypothesis.

Attmept to fix a spurious failure on AppVeyor

I *think* the issue is that `link.exe` is generating debuginfo in the background
which keeps the file open and prevents us from deleting it, but hopefully by
disabling debug information we'll either be able to confirm or deny this
hypothesis.

Auto merge of #5468 - matklad:aliases, r=alexcrichton

Tweak handling of aliases

Previously, `execute_subcommand` was called recursively, and each call
would `.configure` the `config` again. It worked, but seemed rather
fragile.

This commit handles aliases more explicitly, ensures that `.configure`
is called once and, as a bonus, adds a warning for when an alias is
shadowed by the built in.

Tweak error messages

By convention, all errors should start with a lowercase letter

Make alias handling more straightforward

Previously, `execute_subcommand` was called recursively, and each call
would `.configure` the `config` again. It worked, but seemed rather
fragile.

This commit handles aliases more explicitly, ensures that `.configure`
is called once and, as a bonus, adds a warning for when an alias is
shadowed by the built in.

Auto merge of #5464 - ehuss:restore-example-test, r=alexcrichton

Partially revert change to testing examples.

Fixes #5437

I don't think changing the behavior was quite the correct thing to do.  This new behavior is very similar to the old with a few small differences:

```
cargo test
    ORGINAL: Only builds examples.
    NEW: Builds all examples.  Any example with `test` set is tested.

cargo test --tests
    ORIGINAL: Runs all examples as tests.
    NEW: Only runs examples as tests if `test` is set.

cargo test --examples
    ORIGINAL: Runs all examples as tests.
    NEW: (SAME)

cargo test --example foo
    ORIGINAL: Runs the given example as a test.
    NEW: (SAME)

cargo test --all-targets
    ORIGINAL: Runs all examples as tests.
    NEW: (SAME)
```

Auto merge of #5469 - matklad:fix-link, r=matklad

Fix link in docs

Fix link in docs

Auto merge of #5466 - matklad:pr-5465, r=matklad

Fix target_dir handling for RLS

Supersedes  and closes #5465

Fix the test

Revert part to #5393

Commit 0b530c30867da26a4b59146f490c9f1d5377c20a (which this commit mostly reverts) did some refactoring around the `target_dir` function. However, it introduced a bug because it meant that where `CARGO_TARGET_DIR` was specified but `--target-dir` was not, the value from `CARGO_TARGET_DIR` was ignored.

Partially revert change to testing examples.

Fixes #5437

I don't think changing the behavior was quite the correct thing to do.  This new behavior is very similar to the old with a few small differences:

```
cargo test
    ORGINAL: Only builds examples.
    NEW: Builds all examples.  Any example with `test` set is tested.

cargo test --tests
    ORIGINAL: Runs all examples as tests.
    NEW: Only runs examples as tests if `test` is set.

cargo test --examples
    ORIGINAL: Runs all examples as tests.
    NEW: (SAME)

cargo test --example foo
    ORIGINAL: Runs the given example as a test.
    NEW: (SAME)

cargo test --all-targets
    ORIGINAL: Runs all examples as tests.
    NEW: (SAME)
```

Auto merge of #5462 - matklad:install-releases, r=alexcrichton

Don't install pre-releases by default

Currently, `cargo install` will try to install a pre-release version, if available (try `cargo install rand`). This happens because we use `VersionReq::any`, if no version is specified, and that matches pre-releases. The fix is to use `*`, which is different from `any`.

This needs to be done in `cargo install`, and not directly in `parse_no_deprecated`, the latter would be wrong, as demonstrated by a new test with patch.

Don't install pre-releases by default

Auto merge of #5459 - matklad:more-comments, r=alexcrichton

Explain why we need a `Vec<Dependency>` in `Resolve`.

Looks like everyone, who sees it, asks the same question, so let's add an explanation!

cc https://github.com/rust-lang/cargo/pull/5428#issuecomment-385459845

Auto merge of #5456 - LukasKalbertodt:patch-1, r=matklad

Show elapsed time in minutes if >= 60 secs

In large projects with long compile times, seeing "428.65 secs" isn't as clear to humans as seeing the number of minutes (and seconds).

**Old**:
```
Finished dev [unoptimized + debuginfo] target(s) in 2.23 secs
Finished dev [unoptimized + debuginfo] target(s) in 63.94 secs
Finished dev [unoptimized + debuginfo] target(s) in 428.65 secs
```

**New**:
```
Finished dev [unoptimized + debuginfo] target(s) in 2.23s
Finished dev [unoptimized + debuginfo] target(s) in 1m 3.94s
Finished dev [unoptimized + debuginfo] target(s) in 7m 8.65s
```

Note that I also changed `secs` to `s`, because `7 mins 8.65 secs` and `7m 8.65 secs` both look strange IMO. But if you disagree and you'd prefer `secs`, just tell me and I'll change it.

I *didn't* add a check for `secs >= 3600` to print the time in hours. I *hope* this is not necessary...

Auto merge of #5458 - alexcrichton:track-panic, r=matklad

Track panic mode in fingerprint

Ensure that if we've previously compiled a crate with panic=abort and we later
need it for panic=unwind we correctly recompile it.

Closes #5445

Adjust tests to new time output format

Show elapsed time in minutes if >= 60 secs

In large projects with long compile times, seeing "400.65 secs" isn't as
clear as seeing the number of minutes (and seconds).

Auto merge of #5454 - alexcrichton:fix, r=matklad

Fix optional dependencies and required dev-deps

This fixes an accidental bug introduced in #5300 by ensuring a local map keeps
track of the fact that there can be multiple dependencies for one name

Closes #5453

Explain why we need a `Vec<Dependency>` in `Resolve`.

Auto merge of #5446 - csmoe:new_name, r=alexcrichton

Emit correct project name with --name

Fixes #5440

Track panic mode in fingerprint

Ensure that if we've previously compiled a crate with panic=abort and we later
need it for panic=unwind we correctly recompile it.

Closes #5445

Auto merge of #5432 - djc:split-context, r=alexcrichton

Split Context into two types

@matklad here's the crazy attempt to make the `Context` situation slightly more sane. It basically splits all of the stuff that is immutable over the lifetime of the compilation from the stuff that's initialized empty. The latter is now stored in a type which I have provisionally called `BuildProgress` -- feel free to bikeshed about the name (I came up with `BuildData` as another alternative). It stores a reference to the `Context` to make things easier for now, we might want to change that down the line.

IMO this is a pretty good improvement, which clarifies the design quite a bit.

@alexcrichton you'll probably want to review this, too.

Store unit dependencies directly into Context

Move Links into context module

Extract new build_context module out of context module

Split Context into two types

Fix optional dependencies and required dev-deps

This fixes an accidental bug introduced in #5300 by ensuring a local map keeps
track of the fact that there can be multiple dependencies for one name

Closes #5453

Auto merge of #5451 - DarkDrek:add_doc_clean, r=alexcrichton

Add cargo clean --doc

Implements #5449.
It only removes the doc directory and therefore keeps all other build artifacts intact.
When `--doc` is used all other options are ignored.

My test case is mostly copy&paste from `clean_release` so maybe the `.toml`s can be simplified.

Add cargo clean --doc

Let Context store BuildConfig by reference

Auto merge of #5447 - ehuss:ws-error-message, r=alexcrichton

Better error message for `cargo rustc` in a workspace.

Fixes #5371

Auto merge of #5448 - ehuss:debug_env, r=alexcrichton

Fix "DEBUG" env var in build.rs for debug=0

Fixes #5370