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

ansible@2.8 Revert back to python 3.7 #56148

Closed
wants to merge 1 commit into from

Conversation

laserscout
Copy link
Sponsor Contributor

@laserscout laserscout commented Jun 11, 2020

Hello,

I had issues using ansible 2.8 after upgrading to v2.8.12; the issue that I got is described on ansible/ansible issue 63973. On the issue thread it's mentioned by a member of the ansible group, that the bug is caused by python 3.8 which isn't supported.

For that reason I reverted 3808861e8d * ansible@2.8: migrate to python@3.8 which solved all my issue

Checklist

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

@iMichka
Copy link
Member

iMichka commented Jun 11, 2020

Can you add a comment after the python dependency line with a link to the issue, so that we do not try to update it later on?

@laserscout
Copy link
Sponsor Contributor Author

Can you add a comment after the python dependency line with a link to the issue, so that we do not try to update it later on?

Sounds like a great idea!

@laserscout laserscout force-pushed the ansible@2.8-python3.7 branch 2 times, most recently from c5a2017 to 381b1b2 Compare June 11, 2020 07:52
@laserscout
Copy link
Sponsor Contributor Author

Is there anything I can do to fix the failed CI tests?

They seem to be caused by a runner foo

mkdir bottles
mkdir: bottles: File exists
cat bottles/steps_output.txt
cat: bottles/steps_output.txt: No such file or directory

Copy link
Member

@SMillerDev SMillerDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @laserscout ! Without contributions like yours it'd be impossible to keep homebrew going with the high standards that users have come to expect from the project. You can feel good knowing that you've made the world a tiny bit better for homebrew users around the world! 👍 🎉

@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

@laserscout laserscout deleted the ansible@2.8-python3.7 branch June 15, 2020 14:06
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.

None yet

4 participants