Extract {build,dev}-dependencies (#58)

The build-dependencies are needed by build scripts, which makes it a
sort of "export" or "devel" dependency of the package. Since colcon
doesn't currently disambiguate between this class and "run"
dependencies, we will need to include the dependencies under "run".

The "dev" dependencies however are not needed for dependees of a
package, and only the package itself.

Author: cottsay

colcon_cargo/package_augmentation/cargo.py

@@ -74,9 +74,19 @@ def extract_dependencies(content, path):
         for k, v in content.get('dependencies', {}).items()
         if k != name
     }
+    build_depends = {
+        create_dependency_descriptor(k, v, path)
+        for k, v in content.get('build-dependencies', {}).items()
+        if k != name
+    }
+    dev_depends = {
+        create_dependency_descriptor(k, v, path)
+        for k, v in content.get('dev-dependencies', {}).items()
+        if k != name
+    }
     return {
-        'build': depends,
-        'run': depends,
+        'build': depends | build_depends | dev_depends,
+        'run': depends | build_depends,
     }
 
 

test/rust-sample-package/Cargo.toml

@@ -9,5 +9,8 @@ edition = "2018"
 [dependencies]
 local-rust-pure-library = {package = "rust-pure-library", path = "../rust-pure-library"}
 
+[dev-dependencies]
+tempdir = "0.3"
+
 [target.'cfg(windows)'.dependencies]
 windows-sys = "0.59"

test/spell_check.words

@@ -32,6 +32,7 @@ setuptools
 skipif
 staticmethod
 symlink
+tempdir
 tempfile
 testcase
 testsuite

test/test_build.py

@@ -63,10 +63,13 @@ def test_package_augmentation():
     cpi.identify(desc)
     aug.augment_package(desc)
     print(desc)
-    assert len(desc.dependencies['build']) == 2
+    assert len(desc.dependencies['build']) == 3
     assert PURE_LIBRARY_PACKAGE_NAME in desc.dependencies['build']
     assert 'windows-sys' in desc.dependencies['build']
-    assert desc.dependencies['run'] == desc.dependencies['build']
+    assert 'tempdir' in desc.dependencies['build']
+    assert len(desc.dependencies['run']) == 2
+    assert PURE_LIBRARY_PACKAGE_NAME in desc.dependencies['run']
+    assert 'windows-sys' in desc.dependencies['run']
 
 
 # Ported from Python 3.13 implementation