Wrap dependencies in DependencyDescriptor objects

Author: rotu

colcon_core/package_identification/python.py

@@ -46,14 +46,19 @@ def identify(self, desc):  # noqa: D102
         desc.name = name
 
         desc.metadata['version'] = config['metadata'].version
-
-        desc.dependencies['build'].update(config['setup_requires'])
-        desc.dependencies['run'].update(config['install_requires'])
-        desc.dependencies['test'].update(config['tests_require'])
+        desc.dependencies = {
+            'build': {create_dependency_descriptor(c) for c in
+                      config['setup_requires']},
+            'run':   {create_dependency_descriptor(c) for c in
+                      config['install_requires']},
+            'test':  {create_dependency_descriptor(c) for c in
+                      config['tests_require']},
+        }
 
         def getter(env):
             nonlocal setup_py
-            return get_setup_result(setup_py, env)
+            d = get_setup_result(setup_py, env)
+            return d
 
         desc.metadata['get_python_setup_options'] = getter
 
@@ -66,11 +71,10 @@ def get_setup_result(setup_py, env=None):
     :param env: environment variables to set before running setup.py
     :return: Dictionary of data describing the package.
     """
-
     conn_recv, conn_send = multiprocessing.Pipe(duplex=False)
     p = multiprocessing.Process(
         target=_get_setup_result_target,
-        args=(setup_py, env, conn_send)
+        args=(setup_py.absolute(), env, conn_send)
     )
     p.start()
     p.join()
@@ -79,7 +83,7 @@ def get_setup_result(setup_py, env=None):
                     setup_py, p.exitcode)
 
     if not conn_recv.poll():
-        return None
+        raise RuntimeError('No results available from Python setup script')
     return conn_recv.recv()