mvukov / rules_ros2

Build ROS 2 with Bazel
Apache License 2.0
81 stars 45 forks source link

Use `exec` in launch wrapper #169

Closed ahans closed 1 year ago

ahans commented 1 year ago

This adds exec to the call of the actual binary in the launch script template. This way, no new process is created for the actual binary, but it takes the place of the wrapper script (including its PID). Without this, ROS 2 launch cannot properly communicate with the binary, as it sends signals to the wrapper script, which by default doesn't forward signals to a child process (the binary in this case).