ros2 / launch

Tools for launching multiple processes and for writing tests involving multiple processes.
Apache License 2.0
126 stars 144 forks source link

Shutdown On Process Exit #739

Closed DLu closed 9 months ago

DLu commented 1 year ago

Convenience function for the old required functionality in ROS 1.

DLu commented 1 year ago

[note: trying to import the new function in utilities/__init__.py leads to a circular dependency.

DLu commented 1 year ago

Apologies for the repeat pushes. This PR is ready for review now.

sloretz commented 1 year ago

@methylDragon friendly ping 🧇

moriarty commented 1 year ago

Something similar exists for required which was added here: https://github.com/ros2/launch/pull/179

DLu commented 9 months ago

@moriarty Thanks for pointing that out.

Updating my documentation here instead: https://github.com/MetroRobots/rosetta_launch/pull/12