gazebosim / ros_gz

Integration between ROS (1 and 2) and Gazebo simulation
https://gazebosim.org
Apache License 2.0
211 stars 125 forks source link

Should provide option to set `shell=False` in launch files #563

Open azeey opened 1 week ago

azeey commented 1 week ago

Due to https://github.com/ros2/launch/issues/757 Gazebo launched by gz_sim.launch.py does not shut down cleanly. We should provide a way to set shell=False to workaround this issue. Most users probably don't need shell=True, which is mainly used to expand environment variables and file globs [1] since launch provides a way to set environment variables and find absolute paths to files, so we might consider making shell=False, the default.