Skip to content

Conversation

@fengzhou-msft
Copy link
Member

@fengzhou-msft fengzhou-msft commented Oct 12, 2020

Description

Homebrew/homebrew-core#62253 migrated to Python 3.9 for Azure CLI, but Azure CLI is not compatible with Python 3.9 yet. ADO has not supported Python 3.9 either and it caused failures when testing homebrew formula, which blocked the release of our packages.

This PR updates the homebrew template to sync the changes with Homebrew master branch except that Python is still 3.8 and build the homebrew formula from the template.

Testing Guide

Tested the changes in build_test branch: https://dev.azure.com/azure-sdk/public/_build/results?buildId=570383&view=results

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

Copy link
Contributor

@arrownj arrownj left a comment

Choose a reason for hiding this comment

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

Actually I still doubt that whether homebrew will update the python version even we pin python version in the formula.


livecheck do
url "https://github.com/Azure/azure-cli/releases/latest"
regex(%r{href=.*?/tag/azure-cli[._-]v?(\d+(?:\.\d+)+)["' >]}i)
Copy link
Member

Choose a reason for hiding this comment

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

Can we add some comments about what this regex does?

Copy link
Member Author

Choose a reason for hiding this comment

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

It's added by Homebrew team, seems to parsing the version number from https://github.com/Azure/azure-cli/releases/latest and then do livecheck.

@fengzhou-msft
Copy link
Member Author

Actually I still doubt that whether homebrew will update the python version even we ping python version in the formula.

Valid concern, based on their migration plan: Homebrew/homebrew-core#62201, they only test whether the formula can be installed locally. I added a comment to let them know Azure CLI is not compatible with Python 3.9.

@fengzhou-msft fengzhou-msft requested a review from yungezz October 12, 2020 06:40
Copy link
Member

@yungezz yungezz left a comment

Choose a reason for hiding this comment

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

hi @fengzhou-msft could you pls help to follow up how to get change nofitication on homebrew, we met surprise twice in past releases. It will be helpful if we could get earlier notification.

@fengzhou-msft
Copy link
Member Author

fengzhou-msft commented Oct 12, 2020

hi @fengzhou-msft could you pls help to follow up how to get change nofitication on homebrew, we met surprise twice in past releases. It will be helpful if we could get earlier notification.

The change is made through PRs as an open source project. They don't have CODEOWNERS in their repository, which is an ideal way to get notified for PRs.
There are some third-party tools to get notified when some file changes on Github such as https://app.github-file-watcher.com/

@yungezz
Copy link
Member

yungezz commented Oct 12, 2020

hi @fengzhou-msft could you pls help to follow up how to get change nofitication on homebrew, we met surprise twice in past releases. It will be helpful if we could get earlier notification.

The change is made through PRs as an open source project. They don't have CODEOWNERS in their repository, which is an ideal way to get notified for PRs.
There are some third-party tools to get notified when some file changes on Github such as https://app.github-file-watcher.com/

:( let me comment at the our previous PR to feedback about this.

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.

5 participants