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

Replaced #!/usr/bin/env python with #!/usr/bin/python #509

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AllRWeak
Copy link

Staring from Ansible 2.17.1 modules in this collection are unable the find the Python interpreter. According to This issue ansible/ansible#83603 and Ansible development guide https://docs.ansible.com/ansible/latest/dev_guide/developing_modules_documenting.html#python-shebang-utf-8-coding all modules should start with #!/usr/bin/python

This PR replaces #!/usr/bin/env python with #!/usr/bin/python in all modules to make sure they match with Ansible module format and fixes #508

This fix has also been tested on Ansible 2.17.0 to make sure it's also backwards compatible.

@vmwclabot
Copy link
Member

@AllRWeak, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

@vmwclabot
Copy link
Member

@AllRWeak, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

…sible module format

Signed-off-by: Allar Viik <allar@clarifiedsecurity.com>
@rutgerblom
Copy link

When will this PR get merged?

@laidbackware
Copy link
Contributor

@ggverma when will this be merged?

@luischanu
Copy link

When will this be merged?

@helarnl
Copy link

helarnl commented Nov 2, 2024

This issue did take some time to figure out what problem was. When will this be merged?

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.

ansible-for-nsxt don't work on ansible-core: 2.17.1 or above
6 participants