Add support for package.autolib field

LukeMathWalker · Oct 2, 2024 · f349eca · f349eca
1 parent aac767c
commit f349eca
Show file tree

Hide file tree

Showing 19 changed files with 37 additions and 1 deletion.
diff --git a/src/lib.rs b/src/lib.rs
@@ -310,7 +310,7 @@ impl<Metadata: for<'a> Deserialize<'a>> Manifest<Metadata> {
                 // `lib.required-features` has no effect on `[lib]`
                 // (see https://doc.rust-lang.org/cargo/reference/cargo-targets.html#the-required-features-field).
                 lib.required_features.clear();
-            } else if src.contains("lib.rs") {
+            } else if !matches!(package.autolib, Some(false)) && src.contains("lib.rs") {
                 self.lib = Some(Product {
                     name: Some(package.name.replace('-', "_")),
                     path: Some("src/lib.rs".to_string()),
@@ -1001,6 +1001,9 @@ pub struct Package<Metadata = Value> {
     /// The default binary to run by cargo run.
     pub default_run: Option<String>,
 
+    /// Disables library auto discovery.
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub autolib: Option<bool>,
     /// Disables binary auto discovery.
     ///
     /// Use [Manifest::autobins()] to get the effective value.
@@ -1051,6 +1054,7 @@ impl<Metadata> Package<Metadata> {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/autolib.rs b/tests/autolib.rs
@@ -143,3 +143,18 @@ fn test_legacy_lib_with_new_edition() {
     let error = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap_err();
     insta::assert_snapshot!(error, @"can't find library, rename file to `src/lib.rs` or specify lib.path");
 }
+
+#[test]
+fn test_disabled_autolib() {
+    let manifest = r#"
+    [package]
+    name = "auto-lib"
+    version = "0.1.0"
+    edition = "2021"
+    autolib = false
+    "#;
+    let tempdir = utils::prepare(manifest, vec!["src/lib.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
+
+    assert!(m.lib.is_none());
+}
diff --git a/tests/snapshots/autotarget__full_example.snap b/tests/snapshots/autotarget__full_example.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/autotarget__full_example_with_declarations_2015.snap b/tests/snapshots/autotarget__full_example_with_declarations_2015.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/autotarget__full_example_with_declarations_2021.snap b/tests/snapshots/autotarget__full_example_with_declarations_2021.snap
@@ -34,6 +34,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/autotarget__full_example_without_discovery.snap b/tests/snapshots/autotarget__full_example_without_discovery.snap
@@ -34,6 +34,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: Some(
                 false,
             ),

diff --git a/tests/snapshots/parse__autobuild.snap b/tests/snapshots/parse__autobuild.snap
@@ -34,6 +34,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__build_dependencies_casing.snap b/tests/snapshots/parse__build_dependencies_casing.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__default_features_casing.snap b/tests/snapshots/parse__default_features_casing.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__dev_dependencies_casing.snap b/tests/snapshots/parse__dev_dependencies_casing.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__legacy-2.snap b/tests/snapshots/parse__legacy-2.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__legacy.snap b/tests/snapshots/parse__legacy.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__metadata.snap b/tests/snapshots/parse__metadata.snap
@@ -34,6 +34,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__opt_level.snap b/tests/snapshots/parse__opt_level.snap
@@ -79,6 +79,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__opt_version.snap b/tests/snapshots/parse__opt_version.snap
@@ -26,6 +26,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__package_inheritance.snap b/tests/snapshots/parse__package_inheritance.snap
@@ -42,6 +42,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__proc_macro_casing-2.snap b/tests/snapshots/parse__proc_macro_casing-2.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__proc_macro_casing.snap b/tests/snapshots/parse__proc_macro_casing.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,

diff --git a/tests/snapshots/parse__workspace_dependency-2.snap b/tests/snapshots/parse__workspace_dependency-2.snap
@@ -30,6 +30,7 @@ Manifest {
             exclude: None,
             include: None,
             default_run: None,
+            autolib: None,
             autobins: None,
             autoexamples: None,
             autotests: None,