search

ros-infrastructure / catkin_pkg

Standalone Python library for the catkin build system.
https://github.com/ros/catkin
Other
47 stars 91 forks source link

support package format 3 #197

Closed dirk-thomas closed 6 years ago

dirk-thomas commented 6 years ago

As defined in REP 149.

dirk-thomas commented 6 years ago

In the current state it does successfully parse all manifest files of ROS 2, exposes an API for all new information as well as provide an API for condition evaluation.

It doesn't utilize the new information yet in any of the non-parsing API, e.g. consider group dependencies in topological order or passing context for the dependency conditions.

dirk-thomas commented 6 years ago

The patch now does utilize the new information in the topological order as well as uses the context to evaluate dependency conditions.

@ros-infrastructure/ros_team Ready for review.

dirk-thomas commented 6 years ago

I believe the pyparsing dependency needs to be added to the stdeb.cfg file too.

Thank you - good catch. I added it here.

dirk-thomas commented 6 years ago

Waiting for a confirmation from @nuclearsandwich before merging...