Fix maturin sdist --manifest-path <PATH> for workspace project

messense · Mar 12, 2022 · 54973be · 54973be
1 parent a34f76f
commit 54973be
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 * Package license files in `.dist-info/license_files` following PEP 639 in [#837](https://github.com/PyO3/maturin/pull/837)
 * Stop testing Python 3.6 on CI since it's already EOL in [#840](https://github.com/PyO3/maturin/pull/840)
+* Fix `maturin sdist --manifest-path <PATH>` for workspace project in [#843](https://github.com/PyO3/maturin/pull/843)
 
 ## [0.12.10] - 2022-03-09
 

diff --git a/src/source_distribution.rs b/src/source_distribution.rs
@@ -96,9 +96,15 @@ fn add_crate_to_source_distribution(
     known_path_deps: &HashMap<String, PathDependency>,
     root_crate: bool,
 ) -> Result<()> {
+    let crate_dir = manifest_path.as_ref().parent().with_context(|| {
+        format!(
+            "Can't get parent directory of {}",
+            manifest_path.as_ref().display()
+        )
+    })?;
     let output = Command::new("cargo")
-        .args(&["package", "--list", "--allow-dirty", "--manifest-path"])
-        .arg(manifest_path.as_ref())
+        .args(&["package", "--list", "--allow-dirty"])
+        .current_dir(crate_dir)
         .output()
         .context("Failed to run cargo")?;
     if !output.status.success() {