Some hints about feature lifecycle

Clarify the meaning of feature-related labels

Auto merge of #5563 - ehuss:cargotest-doc, r=alexcrichton

cargotest: Add some more docs

Hopefully this isn't too wordy.  This isn't intended for rustdoc output, but generally to be read in-editor.

cargotest: Add some more docs

Auto merge of #5559 - alexcrichton:update-core-foundation, r=alexcrichton

Update version of core-foundation crate

Update version of core-foundation crate

Auto merge of #5557 - ehuss:linux-cbfct, r=alexcrichton

Fix test failure in changing_bin_features_caches_targets for Linux.

Fixes rust-lang/rust#50962.

My theory is that while copying the binary, another thread forked with the fd
open.  The copy finishes and attempts to exec before the other child execs (and
closes the writeable fd).

I was able to easily repro this on linux.  I ran some stress tests of this fix
locally on linux and on appveyor, and was unable to trigger it again.

Fix test failure in changing_bin_features_caches_targets for Linux.

Fixes rust-lang/rust#50962.

My theory is that while copying the binary, another thread forked with the fd
open.  The copy finishes and attempts to exec before the other child execs (and
closes the writeable fd).

I was able to easily repro this on linux.  I ran some stress tests of this fix
locally on linux and on appveyor, and was unable to trigger it again.

Auto merge of #5541 - ehuss:fix-rustdoc-edition, r=alexcrichton

Fix passing --edition to rustdoc during doctests.

Fixes #5538

Fix passing --edition to rustdoc during doctests.

Fixes #5538

Auto merge of #5540 - ehuss:fix-build-plan-tests, r=alexcrichton

Fix tests when CARGO_TARGET_DIR is set.

Fixes #5536

Fix tests when CARGO_TARGET_DIR is set.

Fixes #5536

Auto merge of #5537 - eddyb:no-trans, r=matklad

Remove -Zno-trans test.

Needed for https://github.com/rust-lang/rust/pull/50615 (where the flag is now `-Zno-codegen`) - which revealed that the test shouldn't even exist anymore, and was accidentally added back (https://github.com/rust-lang/rust/pull/50615#issuecomment-388968326).

Remove -Zno-trans test.

Auto merge of #5535 - ehuss:ignore-test-tab, r=alexcrichton

Ignore <tab> in libtest output.

rust-lang/rust#50387 is changing the output from libtest which affected a few Cargo tests.

Ignore <tab> in libtest output.

rust-lang/rust#50387 is changing the output from libtest which affected a few Cargo tests.

Auto merge of #5533 - dwijnand:patch-1, r=matklad

Detail how to run locally-built nightly cargo

Documenting from https://gitter.im/rust-lang/cargo?at=5af6d407862c5e33e92bf2ca.

Apologies if this is documenting too much what is effectively rustup behaviour.

Let me know if it's time to replace this with links to rustup docs instead.

Detail how to run locally-built nightly cargo

Documenting from https://gitter.im/rust-lang/cargo?at=5af6d407862c5e33e92bf2ca.

Auto merge of #5527 - tcr:patch-1, r=alexcrichton

Point Source Replacement to the Overriding Dependencies section

This would've saved me a minute. This mirrors a similarly helpful caveat in [the [patch] section](https://doc.rust-lang.org/cargo/reference/manifest.html#the-patch-section).

Point Source Replacement to the Overriding Dependencies section

This would've saved me a minute. This mirrors a similarly helpful caveat in [the [patch] section](https://doc.rust-lang.org/cargo/reference/manifest.html#the-patch-section).

Auto merge of #5522 - DarkDrek:user_agent_option, r=alexcrichton

Add option to set user-agent

Fixes #5494

Add option to set user-agent

Fixes #5494

Auto merge of #5520 - alexcrichton:check-with-full-proc-macro, r=matklad

Fix mode generated in `maybe_lib`

The new `mode` for the library dependency is dependent on the library target
rather than the target which is the reason for the dependency on the library!

Closes rust-lang/rust#50640

Auto merge of #5519 - matthiaskrgr:typos, r=alexcrichton

NFC: fix a couple of typos, found by codespell.

Fix mode generated in `maybe_lib`

The new `mode` for the library dependency is dependent on the library target
rather than the target which is the reason for the dependency on the library!

Closes rust-lang/rust#50640

NFC: fix a couple of typos, found by codespell.

Auto merge of #5513 - ehuss:test-binary-panic, r=alexcrichton

Fix `panic` for binaries built during tests.

Fixes #5435

Also re-enable tests now that #5460 has landed.

Fix `panic` for binaries built during tests.

Fixes #5435

Also re-enable tests now that #5460 has landed.

Auto merge of #5512 - Eh2406:simplify_FeatureValue, r=alexcrichton

simplify build_requirements

use InternedString's ability to be a &'static str to appease the borrow checker

This "slight workaround" was added in #5270. So @djc does this still look correct and an improvement?

simplify build_requirements

use InternedString's ability to be a &'static str to appease the borrow checker

This "slight workaround" was added in #5270. So @djc dose this still look correct?

Auto merge of #5301 - mshal:build-plan, r=matklad

Add --build-plan for 'cargo build'

With 'cargo build --build-plan', cargo does not actually run any
commands, but instead prints out what it would have done in the form of
a JSON data structure.

Fixes #3815

Fixup some minor review comments

Auto merge of #5460 - ehuss:conservative-link, r=alexcrichton

Be more conservative about which files are linked to the output dir.

This changes it so that only top-level targets requested on the command-line will be included in the output directory.  Dependencies are no longer included.

Fixes #5444.

Auto merge of #5509 - Eh2406:use_the_new, r=alexcrichton

Use the new stable

The new rust release has stabilized `Box::leak` and `impl Iterator` lets use them!

use Box::leak and clean up our string leak

use impl Trait

Address review comment, no need to check export_dir.

Update due to #5503.

Fix broken test.

Be more conservative about which files are linked to the output dir.

This changes it so that only top-level targets requested on the command-line will be included in the output directory.  Dependencies are no longer included.

Fixes #5444.

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

Add --build-plan for 'cargo build'

With 'cargo build --build-plan', cargo does not actually run any
commands, but instead prints out what it would have done in the form of
a JSON data structure.

Fixes #3815

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