Skip to content

Remove major version numbers from install paths#1238

Merged
azeey merged 1 commit intogazebosim:mainfrom
azeey:rm_install_path_versions
Mar 11, 2026
Merged

Remove major version numbers from install paths#1238
azeey merged 1 commit intogazebosim:mainfrom
azeey:rm_install_path_versions

Conversation

@azeey
Copy link

@azeey azeey commented Mar 11, 2026

Summary

Follow up to gazebo-tooling/release-tools#1244 and part of gazebo-tooling/release-tools#1446

This removes major version numbers from install paths since side-by-side installation is no longer supported. Removing the versions helps reduce the number of changes required when updating packaging files (e.g. homebrew formulae).

Here's the diff between the build/install_manifest.txt files of this and the main branch:

--- old_install_manifest.txt	2026-03-10 22:40:45
+++ new_install_manifest.txt	2026-03-10 22:41:03
@@ -139,3 +139,3 @@
-~/ws/rotary/install/lib/gz-rendering-10/engine-plugins/libgz-rendering-ogre.10.0.0~pre1.dylib
-~/ws/rotary/install/lib/gz-rendering-10/engine-plugins/libgz-rendering-ogre.10.dylib
-~/ws/rotary/install/lib/gz-rendering-10/engine-plugins/libgz-rendering-ogre.dylib
+~/ws/rotary/install/lib/gz-rendering/engine-plugins/libgz-rendering-ogre.10.0.0~pre1.dylib
+~/ws/rotary/install/lib/gz-rendering/engine-plugins/libgz-rendering-ogre.10.dylib
+~/ws/rotary/install/lib/gz-rendering/engine-plugins/libgz-rendering-ogre.dylib
@@ -274,3 +274,3 @@
-~/ws/rotary/install/lib/gz-rendering-10/engine-plugins/libgz-rendering-ogre2.10.0.0~pre1.dylib
-~/ws/rotary/install/lib/gz-rendering-10/engine-plugins/libgz-rendering-ogre2.10.dylib
-~/ws/rotary/install/lib/gz-rendering-10/engine-plugins/libgz-rendering-ogre2.dylib
+~/ws/rotary/install/lib/gz-rendering/engine-plugins/libgz-rendering-ogre2.10.0.0~pre1.dylib
+~/ws/rotary/install/lib/gz-rendering/engine-plugins/libgz-rendering-ogre2.10.dylib
+~/ws/rotary/install/lib/gz-rendering/engine-plugins/libgz-rendering-ogre2.dylib

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the fix (as needed)
  • 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)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • 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.

Backports: If this is a backport, please use Rebase and Merge instead.

Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Mar 11, 2026
@azeey azeey merged commit 1045e96 into gazebosim:main Mar 11, 2026
17 of 23 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Mar 11, 2026
@azeey azeey deleted the rm_install_path_versions branch March 11, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants