[galactic] Support scoping environment variables (#601)

ros2 / launch

Tools for launching multiple processes and for writing tests involving multiple processes.

Apache License 2.0

128 stars 144 forks source link

[galactic] Support scoping environment variables (#601) #631

Closed jacobperron closed 2 years ago

jacobperron commented 2 years ago

Partial backport of #601 to Galactic.

jacobperron commented 2 years ago

@ros-pull-request-builder retest this please

jacobperron commented 2 years ago

Testing with --packages-above launch launch_xml launch_yaml:

Linux
Linux-aarch64
Windows

jacobperron commented 2 years ago

I've also backported https://github.com/ros2/launch/pull/656 as part of this PR (7f62bf7)

jacobperron commented 2 years ago

Linux
Linux-aarch64
Windows

jacobperron commented 2 years ago

Linux
Linux-aarch64
Windows

clalancette commented 2 years ago

We've got the one failure on Linux, which is a flake, and the one failure on Windows (which has nothing to do with this PR). I'm going to go ahead and merge this.