Python package identification might throw exception on valid setup.py/setup.cfg

[murphm8]

Pip is adding support for PEP 508 URL dependencies pypa/pip#5571. I believe that the code linked below will throw an exception if there is a URL in the dependencies list since it is 'not a valid version'.

colcon-core/colcon_core/package_identification/python.py

Line 125 in 83bf039

requirement = parse_requirement(requirement_string)

[dirk-thomas]

Can you please provide a reproducible example.

[dirk-thomas]

@murphm8 Friendly ping.

[murphm8]

Hey, sorry I haven't followed up. Been super busy. I will get you a reproduction as soon as I am able.

[murphm8]

I ran a quick test. I must've been using an improperly formatted string when I tried this before. I tested building a python package with a dependency of colcon-core@git+https://github.com/colcon/colcon-core.git#egg=colcon-core and colcon-core@git+https://github.com/colcon/colcon-core.git@0.3.12#egg=colcon-core-0.3.12.

The create_dependency_descriptor returned a DependencyDescriptor with name colcon-core. For the dependency with the version specified there is no version metadata, but I think that is expected.

For bundle I am installing dependencies into the bundle based on this metadata. It might be beneficial to put these PEP 508 urls in the metadata so the correct version is installed when the dependency was specified as a PEP 508 url.

Closing this since it doesn't appear that the parse requirement will fail with a correctly formatted PEP 508 url.