Add "-Zpublic-dependency" for public-dependency feature

Author: linyihai

src/cargo/core/compiler/mod.rs

@@ -89,7 +89,7 @@ use crate::core::compiler::future_incompat::FutureIncompatReport;
 pub use crate::core::compiler::unit::{Unit, UnitInterner};
 use crate::core::manifest::TargetSourcePath;
 use crate::core::profiles::{PanicStrategy, Profile, StripInner};
-use crate::core::{Feature, PackageId, Target, Verbosity};
+use crate::core::{PackageId, Target, Verbosity};
 use crate::util::errors::{CargoResult, VerboseError};
 use crate::util::interning::InternedString;
 use crate::util::machine_message::{self, Message};
@@ -1437,14 +1437,7 @@ pub fn extern_args(
         |dep: &UnitDep, extern_crate_name: InternedString, noprelude: bool| -> CargoResult<()> {
             let mut value = OsString::new();
             let mut opts = Vec::new();
-            if unit
-                .pkg
-                .manifest()
-                .unstable_features()
-                .require(Feature::public_dependency())
-                .is_ok()
-                && !dep.public
-                && unit.target.is_lib()
+            if !dep.public && unit.target.is_lib() && cx.bcx.config.cli_unstable().public_dependency
             {
                 opts.push("priv");
                 *unstable_opts = true;

src/cargo/core/features.rs

@@ -764,6 +764,7 @@ unstable_cli_options!(
     panic_abort_tests: bool = ("Enable support to run tests with -Cpanic=abort"),
     precise_pre_release: bool = ("Enable pre-release versions to be selected with `update --precise`"),
     profile_rustflags: bool = ("Enable the `rustflags` option in profiles in .cargo/config.toml file"),
+    public_dependency: bool = ("Respect a dependency's `public` field in Cargo.toml to control public/private dependencies"),
     publish_timeout: bool = ("Enable the `publish.timeout` key in .cargo/config.toml file"),
     rustdoc_map: bool = ("Allow passing external documentation mappings to rustdoc"),
     rustdoc_scrape_examples: bool = ("Allows Rustdoc to scrape code examples from reverse-dependencies"),
@@ -1140,6 +1141,7 @@ impl CliUnstable {
             "mtime-on-use" => self.mtime_on_use = parse_empty(k, v)?,
             "no-index-update" => self.no_index_update = parse_empty(k, v)?,
             "panic-abort-tests" => self.panic_abort_tests = parse_empty(k, v)?,
+            "public-dependency" => self.public_dependency = parse_empty(k, v)?,
             "profile-rustflags" => self.profile_rustflags = parse_empty(k, v)?,
             "precise-pre-release" => self.precise_pre_release = parse_empty(k, v)?,
             "trim-paths" => self.trim_paths = parse_empty(k, v)?,

src/cargo/core/workspace.rs

@@ -449,7 +449,6 @@ impl<'cfg> Workspace<'cfg> {
                             // NOTE: Since we use ConfigRelativePath, this root isn't used as
                             // any relative paths are resolved before they'd be joined with root.
                             Path::new("unused-relative-path"),
-                            self.unstable_features(),
                             /* kind */ None,
                         )
                     })

src/cargo/ops/cargo_package.rs

@@ -10,8 +10,8 @@ use crate::core::compiler::{BuildConfig, CompileMode, DefaultExecutor, Executor}
 use crate::core::manifest::Target;
 use crate::core::resolver::CliFeatures;
 use crate::core::{registry::PackageRegistry, resolver::HasDevUnits};
-use crate::core::{Feature, Shell, Verbosity, Workspace};
 use crate::core::{Package, PackageId, PackageSet, Resolve, SourceId};
+use crate::core::{Shell, Verbosity, Workspace};
 use crate::sources::PathSource;
 use crate::util::cache_lock::CacheLockMode;
 use crate::util::config::JobsConfig;
@@ -911,12 +911,7 @@ fn run_verify(
     let pkg_fingerprint = hash_all(&dst)?;
     let ws = Workspace::ephemeral(new_pkg, config, None, true)?;
 
-    let rustc_args = if pkg
-        .manifest()
-        .unstable_features()
-        .require(Feature::public_dependency())
-        .is_ok()
-    {
+    let rustc_args = if ws.config().cli_unstable().public_dependency {
         // FIXME: Turn this on at some point in the future
         //Some(vec!["-D exported_private_dependencies".to_string()])
         Some(vec![])

src/cargo/util/toml/mod.rs

@@ -678,7 +678,6 @@ pub fn to_real_manifest(
         nested_paths: &mut nested_paths,
         config,
         warnings: &mut warnings,
-        features: &features,
         platform: None,
         root: package_root,
     };
@@ -1153,7 +1152,6 @@ fn to_virtual_manifest(
             config,
             warnings: &mut warnings,
             platform: None,
-            features: &features,
             root,
         };
         (replace(&me, &mut cx)?, patch(&me, &mut cx)?)
@@ -1302,7 +1300,6 @@ struct Context<'a, 'b> {
     warnings: &'a mut Vec<String>,
     platform: Option<Platform>,
     root: &'a Path,
-    features: &'a Features,
 }
 
 fn verify_lints(lints: Option<manifest::TomlLints>) -> CargoResult<Option<manifest::TomlLints>> {
@@ -1709,7 +1706,6 @@ pub(crate) fn to_dependency<P: ResolveToPath + Clone>(
     warnings: &mut Vec<String>,
     platform: Option<Platform>,
     root: &Path,
-    features: &Features,
     kind: Option<DepKind>,
 ) -> CargoResult<Dependency> {
     dep_to_dependency(
@@ -1723,7 +1719,6 @@ pub(crate) fn to_dependency<P: ResolveToPath + Clone>(
             warnings,
             platform,
             root,
-            features,
         },
         kind,
     )
@@ -1944,13 +1939,18 @@ fn detailed_dep_to_dependency<P: ResolveToPath + Clone>(
     }
 
     if let Some(p) = orig.public {
-        cx.features.require(Feature::public_dependency())?;
-
         if dep.kind() != DepKind::Normal {
             bail!("'public' specifier can only be used on regular dependencies, not {:?} dependencies", dep.kind());
         }
 
-        dep.set_public(p);
+        if !cx.config.cli_unstable().public_dependency {
+            cx.warnings.push(format!(
+                "{name} uses 'public' specifier, pass `-Zpublic-dependency` to enable support for it",
+                name = dep.name_in_toml(),
+            ))
+        } else {
+            dep.set_public(p);
+        }
     }
 
     if let (Some(artifact), is_lib, target) = (

src/doc/src/reference/unstable.md

@@ -296,11 +296,9 @@ The 'public-dependency' feature allows marking dependencies as 'public'
 or 'private'. When this feature is enabled, additional information is passed to rustc to allow
 the 'exported_private_dependencies' lint to function properly.
 
-This requires the appropriate key to be set in `cargo-features`:
+Pass `-Zpublic-dependency` to cargo to enable support for it.
 
 ```toml
-cargo-features = ["public-dependency"]
-
 [dependencies]
 my_dep = { version = "1.2.3", public = true }
 private_dep = "2.0.0" # Will be 'private' by default

tests/testsuite/cargo/z_help/stdout.log

@@ -25,6 +25,7 @@ Available unstable (nightly-only) flags:
     -Z panic-abort-tests        Enable support to run tests with -Cpanic=abort
     -Z precise-pre-release      Enable pre-release versions to be selected with `update --precise`
     -Z profile-rustflags        Enable the `rustflags` option in profiles in .cargo/config.toml file
+    -Z public-dependency        Respect a dependency's `public` field in Cargo.toml to control public/private dependencies
     -Z publish-timeout          Enable the `publish.timeout` key in .cargo/config.toml file
     -Z rustdoc-map              Allow passing external documentation mappings to rustdoc
     -Z rustdoc-scrape-examples  Allows Rustdoc to scrape code examples from reverse-dependencies

tests/testsuite/cargo_add/detect_workspace_inherit_public/in/primary/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [package]
 name = "bar"
 version = "0.0.0"

tests/testsuite/cargo_add/detect_workspace_inherit_public/out/primary/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [package]
 name = "bar"
 version = "0.0.0"

tests/testsuite/cargo_add/no_public/in/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/no_public/out/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/overwrite_no_public/in/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/overwrite_no_public/out/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/overwrite_no_public_with_public/in/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/overwrite_no_public_with_public/out/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/overwrite_public/in/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/overwrite_public/out/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/overwrite_public_with_no_public/in/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/overwrite_public_with_no_public/out/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/public/in/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]

tests/testsuite/cargo_add/public/out/Cargo.toml

@@ -1,4 +1,3 @@
-cargo-features = ["public-dependency"]
 [workspace]
 
 [package]