Skip to content
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

Fix hardcoded python call #79520

Merged
merged 1 commit into from Dec 6, 2022
Merged

Fix hardcoded python call #79520

merged 1 commit into from Dec 6, 2022

Conversation

leegarrett
Copy link
Contributor

SUMMARY

Fix the hardcoded call and instead use the variable defined further up. Makes it easier to build on Debian/Ubuntu where python is called as python3.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

docs

ADDITIONAL INFORMATION

@ansibot
Copy link
Contributor

ansibot commented Dec 2, 2022

Thanks for your Ansible docs contribution! We talk about Ansible documentation on matrix at #docs:ansible.im and on libera IRC at #ansible-docs if you ever want to join us and chat about the docs! We meet there on Tuesdays (see the Ansible calendar) and welcome additions to our weekly agenda items - scroll down to find the upcoming agenda and add a comment to put something new on that agenda.

click here for bot help

@ansibot ansibot added affects_2.15 bug This issue/PR relates to a bug. docs This issue/PR relates to or includes documentation. docs_only All changes are to files within the docs/docsite/ directory needs_triage Needs a first human triage before being processed. new_contributor This PR is the first contribution by a new community member. small_patch labels Dec 2, 2022
@mkrizek mkrizek requested a review from samccann December 6, 2022 15:09
@mkrizek mkrizek removed the needs_triage Needs a first human triage before being processed. label Dec 6, 2022
@samccann
Copy link
Contributor

samccann commented Dec 6, 2022

Validated on test that the docs build fine with this change.

@leegarrett
Copy link
Contributor Author

Thanks! 👍

@samccann samccann merged commit 3fd519d into ansible:devel Dec 6, 2022
@samccann
Copy link
Contributor

samccann commented Dec 6, 2022

Thanks @leegarrett for the Ansible fix and welcome to the Ansible project!

samccann pushed a commit to samccann/ansible that referenced this pull request Dec 8, 2022
Co-authored-by: Lee Garrett <lgarrett@rocketjump.eu>
(cherry picked from commit 3fd519d)
@samccann samccann mentioned this pull request Dec 8, 2022
@ansible ansible locked and limited conversation to collaborators Dec 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.15 bug This issue/PR relates to a bug. docs_only All changes are to files within the docs/docsite/ directory docs This issue/PR relates to or includes documentation. new_contributor This PR is the first contribution by a new community member. small_patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants