Skip to content

Conversation

@shrujanus
Copy link

Refactored the ./controller_manager/test/test_ros2_control_node_launch.py file to use PathSubstitution and removed the unnecessary variables in the import.

@mergify
Copy link
Contributor

mergify bot commented Nov 6, 2025

@shrujanus, all pull requests must be targeted towards the master development branch.
Once merged into master, it is possible to backport to humble, but it must be in master
to have these changes reflected into new distributions.

@shrujanus shrujanus mentioned this pull request Nov 6, 2025
8 tasks
Copy link
Member

@christophfroehlich christophfroehlich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, but please Please open a PR against master branch.

from ament_index_python.packages import get_package_share_directory, get_package_prefix
from launch import LaunchDescription
from launch_testing.actions import ReadyToTest
import launch_testing.markers
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we still need this for @pytest.mark.launch_test?

import pytest
import unittest
import os
import launch_testing
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we use this?

@shrujanus
Copy link
Author

shrujanus commented Nov 7, 2025

Thanks, but please Please open a PR against master branch.

Sorry about that! Should I close this PR and request another one to master? or is there a way to change that right here?

@christophfroehlich
Copy link
Member

As you like. you also can rebase your commit on top of master, force push, and change the base branch of this PR.

@shrujanus
Copy link
Author

As you like. you also can rebase your commit on top of master, force push, and change the base branch of this PR.

The reason I used the humble branch is that I wasn’t able to build the project when I was on master. I’m running ROS 2 Humble on Ubuntu 22.04, so I switched to the humble branch because it matches my ROS distribution and compiles correctly.

Is it common for developers to create branches off their ROS distribution branch (e.g., humble) and later rebase onto master? Or is the expectation that the master branch should compile across all supported ROS distributions?

@christophfroehlich
Copy link
Member

As you like. you also can rebase your commit on top of master, force push, and change the base branch of this PR.

The reason I used the humble branch is that I wasn’t able to build the project when I was on master. I’m running ROS 2 Humble on Ubuntu 22.04, so I switched to the humble branch because it matches my ROS distribution and compiles correctly.

Is it common for developers to create branches off their ROS distribution branch (e.g., humble) and later rebase onto master? Or is the expectation that the master branch should compile across all supported ROS distributions?

ros2_control stack's rolling version is compatible with humble distro, see development version in the docs.
A more elegant developer workflow might be using a docker image, maybe in a vscode devcontainer setup, and develop directly on rolling. You can use the docker files here as a basis, or use one of the several resources for devcontainer setups. I use this one btw.

@shrujanus shrujanus closed this Nov 9, 2025
@shrujanus shrujanus deleted the feat_launch_file branch November 9, 2025 15:27
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.

2 participants