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

cherry-pick depot_tools 6a1d778 to fix macOS Cataliona issues #3175

Merged
merged 1 commit into from
Oct 21, 2019

Conversation

chrmoritz
Copy link
Contributor

@chrmoritz chrmoritz commented Oct 21, 2019

Alternative to #3172, which just cherry-picks the relevant fix from 6a1d778 to depot_tools to fix the Catalina issues, while avoiding having to deal with the Windows virtualenv issues from a full depot_tools update, see #3172.

Goes with denoland/deno_third_party#54.
Verified locally that this is enough for fixing the download_from_google_storage.py on Catalina.

See also: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+/6a1d77869d77b61189247ce8b47ac9d86684a6c2
Chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1012612

Copy link
Member

@piscisaureus piscisaureus left a comment

Choose a reason for hiding this comment

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

LGTM

@piscisaureus piscisaureus merged commit ea8231a into denoland:master Oct 21, 2019
@chrmoritz chrmoritz mentioned this pull request Oct 21, 2019
@chrmoritz chrmoritz deleted the depot_tools2 branch October 21, 2019 22:08
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

2 participants