Skip to content

Install executable to unversioned folder #518

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 29, 2025
Merged

Conversation

scpeters
Copy link
Member

@scpeters scpeters commented May 29, 2025

🎉 New feature

Follow up to gazebo-tooling/release-tools#1244.

Summary

We removed version numbers from package names in many places as part of gazebo-tooling/release-tools#1244, prioritizing the places that most affect users. While reviewing gazebo-release/sdformat16-release#5, I realized that we can install the executable commands to unversioned folders as well (see gazebosim/sdformat#1569), which should make our packaging metadata easier to maintain.

Test it

I will open a PR to the release repository and build a nightly deb from this branch for testing. Tested with gazebo-release/gz-msgs12-release#3

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • 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.)

Generated-by: Remove this if GenAI was not used.

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.

@github-actions github-actions bot added the 🪵 jetty Gazebo Jetty label May 29, 2025
scpeters added a commit to gazebo-release/gz-msgs12-release that referenced this pull request May 29, 2025
@scpeters scpeters marked this pull request as ready for review May 29, 2025 20:39
@scpeters scpeters requested a review from caguero as a code owner May 29, 2025 20:39
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development May 29, 2025
@scpeters scpeters merged commit 12065f6 into main May 29, 2025
12 of 13 checks passed
@scpeters scpeters deleted the scpeters/install_unversioned branch May 29, 2025 21:40
@github-project-automation github-project-automation bot moved this from In review to Done in Core development May 29, 2025
scpeters added a commit to gazebo-release/gz-msgs12-release that referenced this pull request May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪵 jetty Gazebo Jetty
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants