Skip to content

Comments

Parse the version correctly when using gz log subcommands (backport #750)#751

Merged
scpeters merged 2 commits intomainfrom
mergify/bp/main/pr-750
Dec 11, 2025
Merged

Parse the version correctly when using gz log subcommands (backport #750)#751
scpeters merged 2 commits intomainfrom
mergify/bp/main/pr-750

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Dec 10, 2025

🦟 Bug fix

Fixes #742

Summary

It looks like the stand-alone gz-transport-log-main executable wasn't properly parsing the version number.

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
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

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


This is an automatic backport of pull request #750 done by [Mergify](https://mergify.com).

Signed-off-by: Carlos Agüero <caguero@osrfoundation.org>
(cherry picked from commit 49f125e)
@scpeters
Copy link
Member

I believe the Ubuntu CI is failing since we disabled nightly repos for gz-transport15 in gazebo-tooling/gzdev#100 (comment)

I think this fix is to bump the main branch major version (gazebo-tooling/release-tools#1403 (comment))

@scpeters
Copy link
Member

I believe the Ubuntu CI is failing since we disabled nightly repos for gz-transport15 in gazebo-tooling/gzdev#100 (comment)

I think this fix is to bump the main branch major version (gazebo-tooling/release-tools#1403 (comment))

never mind; I think #752 is the needed fix

@scpeters scpeters mentioned this pull request Dec 11, 2025
9 tasks
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Dec 11, 2025
@scpeters scpeters merged commit dfe2152 into main Dec 11, 2025
14 checks passed
@scpeters scpeters deleted the mergify/bp/main/pr-750 branch December 11, 2025 22:57
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants