Set env path

[Tacha-S]

🦟 Bug fix

Fixes #630

Summary

I set the environment variables to correctly load plugins when launching with a composable node.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

[azeey]

Thanks for the contribution!!

This approach works, but only when using the python launch file. It wouldn't work for those using the gzserver action from XML. So, could you move these changes to the action (https://github.com/gazebosim/ros_gz/blob/ros2/ros_gz_sim/ros_gz_sim/actions/gzserver.py)?

[Amronos]

I think this PR fixes #630 instead of #549.

[Tacha-S]

This approach works, but only when using the python launch file. It wouldn't work for those using the gzserver action from XML. So, could you move these changes to the action (ros2/ros_gz_sim/ros_gz_sim/actions/gzserver.py)?

I moved to setting environment variables in ros2/ros_gz_sim/ros_gz_sim/actions/gzserver.py.

I think this PR fixes #630 instead of #549.

Oops, it seems I accidentally selected the one above when choosing an issue.
Thank you. I’ve fixed it.

[ahcorde]

Do you mind to add some documentation about this ?

[azeey]

LGTM!

[ahcorde]

https://github.com/Mergifyio backport jazzy

[mergify]

backport jazzy

✅ Backports have been created

• #661 Set env path (backport #659) has been created for branch jazzy