Skip to content

Add in a comment explaining where Python3::Interpreter comes from.#510

Merged
clalancette merged 1 commit intorollingfrom
clalancette/add-ament-python-comment
Feb 16, 2024
Merged

Add in a comment explaining where Python3::Interpreter comes from.#510
clalancette merged 1 commit intorollingfrom
clalancette/add-ament-python-comment

Conversation

@clalancette
Copy link
Copy Markdown
Contributor

This is a compromise between doing the very correct thing (find_package(Python3) in ament_python_install_packge), and not changing things unnecessarily.

This is a compromise between doing the very correct thing
(find_package(Python3) in ament_python_install_packge), and
not changing things unnecessarily.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette clalancette requested a review from sloretz February 16, 2024 19:30
@clalancette
Copy link
Copy Markdown
Contributor Author

Given that this is just an update to a comment, I'm going to go ahead and merge this in with just Rpr job passing.

@clalancette clalancette merged commit dc3df2f into rolling Feb 16, 2024
@delete-merged-branch delete-merged-branch bot deleted the clalancette/add-ament-python-comment branch February 16, 2024 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants