Skip to content

Conversation

@svenevs
Copy link
Contributor

@svenevs svenevs commented Feb 8, 2023

Closes: actions/setup-python#260.

Time consideration: GHA has a lot installed via brew, uninstallation took 4 minutes. Fixing the symlinks and running upgrade (on today) took 3 minutes 11 seconds.

Relates: actions/setup-python#250, actions/setup-python#252, actions/setup-python#257.


This change is Reviewable

Copy link
Contributor Author

@svenevs svenevs left a comment

Choose a reason for hiding this comment

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

+@BetsyMcPhail for review please.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all discussions resolved, LGTM missing from assignee betsymcphail, platform LGTM missing (waiting on @BetsyMcPhail)


.github/workflows/ci.yml line 60 at r1 (raw file):

          sudo rm -rf /Library/Frameworks/Python.framework/
          # Run upgrades now to fail-fast (setup scripts do this anyway).
          brew update && brew upgrade

FYI we are hopeful this will continue to remain functional, we are not sure if the original suggestion of brew install --force python3 && brew unlink python3 && brew unlink python3 && brew link --overwrite python3 will work for us. However, in the future when both 3.10 and 3.11 (for example) update, we may have another hiccup in which case we can try this. It may be that this tells it not to symlink the others?

Copy link
Contributor

@BetsyMcPhail BetsyMcPhail left a comment

Choose a reason for hiding this comment

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

CI is happy for now and this seems like the better option. :lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all discussions resolved, platform LGTM missing (waiting on @svenevs)

Copy link
Contributor Author

@svenevs svenevs left a comment

Choose a reason for hiding this comment

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

+@xuchenhan-tri for platform review per schedule please 🙂 Related slack discussion: https://drakedevelopers.slack.com/archives/C270MN28G/p1675866323159289

Reviewable status: all discussions resolved, LGTM missing from assignee xuchenhan-tri, platform LGTM missing (waiting on @xuchenhan-tri)

Copy link

@xuchen-han xuchen-han left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all discussions resolved, platform LGTM from [xuchenhan-tri] (waiting on @svenevs)

@svenevs svenevs merged commit 3aebbc4 into main Feb 8, 2023
@svenevs svenevs deleted the fix/macos-gha-brew-python-again-again-again branch February 8, 2023 20:03
@svenevs svenevs restored the fix/macos-gha-brew-python-again-again-again branch February 16, 2023 15:42
svenevs added a commit that referenced this pull request Feb 16, 2023
@svenevs svenevs deleted the fix/macos-gha-brew-python-again-again-again branch February 16, 2023 15:43
svenevs added a commit that referenced this pull request Feb 16, 2023
svenevs added a commit that referenced this pull request Feb 16, 2023
svenevs added a commit that referenced this pull request Feb 16, 2023
svenevs added a commit that referenced this pull request Feb 24, 2023
New failures related to `brew upgrade`, unrelated to python.

Partial continuation of: #250, #252, #257, #259, #262.
svenevs added a commit that referenced this pull request Feb 24, 2023
New failures related to `brew upgrade`, unrelated to python.

Partial continuation of: #250, #252, #257, #259, #262.
svenevs added a commit that referenced this pull request Apr 6, 2023
New failures related to brew upgrade on `go`, fix the same way
as #263.  Partial continuation of #250, #252, #257, #259, #262.
svenevs added a commit that referenced this pull request Apr 6, 2023
New failures related to brew upgrade on `go`, fix the same way
as #263.  Partial continuation of #250, #252, #257, #259, #262.
svenevs added a commit that referenced this pull request Apr 6, 2023
New failures related to brew upgrade on `go`, fix the same way
as #263.  Partial continuation of #250, #252, #257, #259, #262.
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.

No module named pip on Python 3.7

4 participants