Skip to content

Respect GZ_PYTHON_INSTALL_PATH if passed as an argument (backport #1586)#1674

Open
mergify[bot] wants to merge 1 commit into
mainfrom
mergify/bp/main/pr-1586
Open

Respect GZ_PYTHON_INSTALL_PATH if passed as an argument (backport #1586)#1674
mergify[bot] wants to merge 1 commit into
mainfrom
mergify/bp/main/pr-1586

Conversation

@mergify

@mergify mergify Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

🦟 Bug fix

Summary

The PR adds the ability to set the GZ_PYTHON_INSTALL_PATH from CMake arguments by no overriding it if it is set. This is useful for paths injected by the different package managers and distributions.

One use case is for the ROS vendor packages where the Python path is controlled by an ament_python helper call.

Easier to see https://github.com/gazebosim/sdformat/pull/1586/files?w=1

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.)

GeNote 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 #1586 done by Mergify.

Signed-off-by: Jose Luis Rivero <jrivero@honurobotics.com>
Co-authored-by: Steve Peters <scpeters@openrobotics.org>
(cherry picked from commit ae7ac38)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

1 participant