Add topic_name option to info verb

[KeisukeShima]

In some use cases we may want to get a list of topic names. For this reason I have added the quiet option to a info verb.

Signed-off-by: Keisuke Shima [email protected]

[MichaelOrlov]

@KeisukeShima Thanks for the PR.
I think "--quiet" option for showing just topics names is too generic.
It will make more sense to name it more explicitly something like "--topic_name".

[KeisukeShima]

@MichaelOrlov Thanks for review.

I think "--quiet" option for showing just topics names is too generic.
It will make more sense to name it more explicitly something like "--topic_name".

I agree with you. Changed PR title and code.

[MichaelOrlov]

LGTM.

[MichaelOrlov]

Gist: https://gist.githubusercontent.com/MichaelOrlov/bc4ad30bda92639fffd415673159a430/raw/5f2042a177ea3d131e0d11c701c6c9ef51dbac7a/ros2.repos
BUILD args: --packages-above-and-dependencies ros2bag rosbag2_tests
TEST args: --packages-above ros2bag rosbag2_tests
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/11307

• Linux
• Linux-aarch64
• Windows

[MichaelOrlov]

@KeisukeShima Newly added test fails on Windows. Could you please take a look on failure?

[KeisukeShima]

@MichaelOrlov
The cause of the ci failure on Windows is due to a different line break code. This has been corrected by using the normalize_lineseps function for standard output.

[MichaelOrlov]

Re-running Windows CI job

• Windows

[MichaelOrlov]

@KeisukeShima It turns out that now test fails due to different file size
Bag size: 49.2 KiB instead of "Bag size: 49.1 KiB"
Feel free to disable or delete this test_info_with_no_options test.

[MichaelOrlov]

Re-running Windows CI job after removing test

• Windows

[ros-discourse]

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/ros-2-tsc-meeting-minutes-2023-01-19/29423/1