Add ros2plugin (backport #165) - #280
Merged
ahcorde merged 3 commits intoOct 9, 2025
Merged
Conversation
Copy link
Copy Markdown
Sorry, something went wrong.
Signed-off-by: artivis <jeremie.deray@canonical.com> Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com> Co-authored-by: Alejandro Hernandez Cordero <ahcorde@gmail.com> (cherry picked from commit 1a596d6) # Conflicts: # pluginlib/include/pluginlib/class_desc.hpp # pluginlib/include/pluginlib/class_list_macros.hpp # pluginlib/include/pluginlib/impl/split.hpp # pluginlib/src/list_plugins.cpp # pluginlib/test/include/test_base.hpp # pluginlib/test/include/test_plugins.hpp # pluginlib/test/include/visibility_control.hpp # pluginlib/test/test_plugins.cpp
Copy link
Copy Markdown
Author
|
Cherry-pick of 1a596d6 has failed: On branch mergify/bp/jazzy/pr-165 Your branch is up to date with 'origin/jazzy'. You are currently cherry-picking commit 1a596d6. (fix conflicts and run "git cherry-pick --continue") (use "git cherry-pick --skip" to skip this patch) (use "git cherry-pick --abort" to cancel the cherry-pick operation) Changes to be committed: new file: ros2plugin/package.xml new file: ros2plugin/resource/ros2plugin new file: ros2plugin/ros2plugin/__init__.py new file: ros2plugin/ros2plugin/api/__init__.py new file: ros2plugin/ros2plugin/command/__init__.py new file: ros2plugin/ros2plugin/command/plugin.py new file: ros2plugin/ros2plugin/verb/__init__.py new file: ros2plugin/ros2plugin/verb/list.py new file: ros2plugin/setup.py new file: ros2plugin/test/test_copyright.py new file: ros2plugin/test/test_flake8.py new file: ros2plugin/test/test_pep257.py new file: ros2plugin/test/test_xmllint.py Unmerged paths: (use "git add/rm <file>..." as appropriate to mark resolution) both added: pluginlib/include/pluginlib/class_desc.hpp both added: pluginlib/include/pluginlib/class_list_macros.hpp both added: pluginlib/include/pluginlib/impl/split.hpp deleted by us: pluginlib/src/list_plugins.cpp deleted by us: pluginlib/test/include/test_base.hpp deleted by us: pluginlib/test/include/test_plugins.hpp both added: pluginlib/test/include/visibility_control.hpp both added: pluginlib/test/test_plugins.cpp To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
Sorry, something went wrong.
Merged
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Copy link
Copy Markdown
Contributor
|
Pulls: #280
|
Sorry, something went wrong.
ahcorde
approved these changes
Oct 9, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add the plugin command together with the list verb to ros2 CLI.
At the moment it produces the following results:
while passing the flag --packages only list packages that export plugins and the relative path to the XML file(s ?),
I'd be more than open to discuss actual output and/or output layout.
Note this does not exactly mimic ROS 1 in terms of prints layout nor in terms of use (yet),
e.g. ROS 1 --> rospack plugins --attrib=plugin <base_package_name>.
Fix ros2/ros2cli#328
Replace ros2/ros2cli#340
Signed-off-by: artivis jeremie.deray@canonical.com
This is an automatic backport of pull request #165 done by Mergify.