fix bug reading binaries config from Cargo.toml (#377)

Author: PWhiddy

.changes/fix-all-targets-included.md

@@ -0,0 +1,6 @@
+---
+"cargo-packager": patch
+"@crabnebula/packager": patch
+---
+
+Fixed a bug where "binaries" parameter in Cargo.toml would be ignored and all targets would be packaged.

crates/packager/src/cli/config.rs

@@ -200,19 +200,22 @@ fn load_configs_from_cargo_workspace(cli: &super::Cli) -> Result<Vec<(Option<Pat
                     .as_ref()
                     .map(|p| p.as_std_path().to_owned());
             }
-            let targets = package
-                .targets
-                .iter()
-                .filter(|t| t.is_bin())
-                .collect::<Vec<_>>();
-            for target in &targets {
-                config.binaries.push(Binary {
-                    path: target.name.clone().into(),
-                    main: match targets.len() {
-                        1 => true,
-                        _ => target.name == package.name,
-                    },
-                })
+            // Auto-detect binaries if none were explicitly configured
+            if config.binaries.is_empty() {
+                let targets = package
+                    .targets
+                    .iter()
+                    .filter(|t| t.is_bin())
+                    .collect::<Vec<_>>();
+                for target in &targets {
+                    config.binaries.push(Binary {
+                        path: target.name.clone().into(),
+                        main: match targets.len() {
+                            1 => true,
+                            _ => target.name == package.name,
+                        },
+                    })
+                }
             }
             configs.push((
                 Some(package.manifest_path.as_std_path().to_path_buf()),