ros2 / launch

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

Launch with parsing error should not start anything #627

Open timonegk opened 2 years ago

timonegk commented 2 years ago

Bug report

Required Info:

Steps to reproduce issue

  1. Launch a launch file including other launch files.
  2. Have an invalid launch file included.

Expected behavior

A parser error should be thrown, not executing anything in the launch files.

Actual behavior

Some nodes specified before the parser error was encountered are started. The launch then stops with the parser error, leaving already started nodes orphaned in the background.