Skip to content

Conversation

@emersonknapp
Copy link
Contributor

@emersonknapp emersonknapp commented Oct 23, 2025

Description

Part of robograph-project/planning#38
Depends on ros2/launch#914

Make FindPackage{X} substitutions a subclass of PathSubstitution to get the path join operator for free.

Lets you change:

PathSubstitution(FindPackageShare('my_pkg')) / 'launch' / 'my_launch.yaml'
# or
PathJoinSubstitution([FindPackageShare('my_pkg'), 'launch', 'my_launch.yaml'])

into

FindPackageShare('my_pkg') / 'launch' / 'my_launch.yaml'

Is this user-facing behavior change?

New user-facing capability, does not affect existing behavior

Did you use Generative AI?

no

Additional Information

@emersonknapp emersonknapp force-pushed the emerson/find-package-path-subs branch from 5e193f0 to f04646a Compare October 23, 2025 09:32
@emersonknapp emersonknapp moved this to In review in Robograph Planning Oct 23, 2025
@ahcorde
Copy link
Contributor

ahcorde commented Oct 24, 2025

Pulls: ros2/launch#914, #494
Gist: https://gist.githubusercontent.com/ahcorde/1d8541d31501406682d13767e15e788e/raw/cb7660853dbaea155933d8576f146a1a0d5953d6/ros2.repos
BUILD args: --packages-above-and-dependencies launch launch_ros test_launch_ros
TEST args: --packages-above launch launch_ros test_launch_ros
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17368

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@emersonknapp
Copy link
Contributor Author

emersonknapp commented Oct 28, 2025

Pulls: ros2/launch#914, #494
Gist: https://gist.githubusercontent.com/emersonknapp/2c725d3e9c99a8e2cd15f567470d1ada/raw/cb7660853dbaea155933d8576f146a1a0d5953d6/ros2.repos
BUILD args: --packages-above-and-dependencies launch launch_ros test_launch_ros
TEST args: --packages-above launch launch_ros test_launch_ros
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17378

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
    • Build Status
  • Windows Build Status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants