Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@chaselatta
Copy link
Contributor

Adds a script to download the Fuchsia SDK from a given GCS bucket.
This script can run as an optional gclient hook to allow for the
Fuchsia architecture to run LSC checks on commits to fuchsia.git.

Copy link
Contributor

@iskakaushik iskakaushik left a comment

Choose a reason for hiding this comment

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

LGTM

@godofredoc
Copy link
Contributor

I believe after this change we'll need to update the repo_utils module to accept properties to be passed to gclient.

@chinmaygarde chinmaygarde requested a review from zanderso January 27, 2022 21:07
@chinmaygarde
Copy link
Member

From Triage: @zanderso would like to take a closer look at the implications of this patch. Please hold for an additional review.

@chaselatta
Copy link
Contributor Author

Will do, for context this is needed to enable fx lsc checks from fuchsia to ensure that SDK changes don't break external repositories. Those artifacts are only uploaded to gcs and not cipd which is why we need the hook instead of just updating the cipd bucket in the DEPS file.

Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

lgtm w/ nit

# Prints to stderr.
def eprint(*args, **kwargs):
print(*args, file=sys.stderr, **kwargs)

Copy link
Member

Choose a reason for hiding this comment

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

nit: two lines between top level methods.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Adds a script to download the Fuchsia SDK from a given GCS bucket.
This script can run as an optional gclient hook to allow for the
Fuchsia architecture to run LSC checks on commits to fuchsia.git.
@chaselatta chaselatta added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Jan 28, 2022
@fluttergithubbot fluttergithubbot merged commit bc51c29 into flutter:main Jan 28, 2022
@chaselatta chaselatta deleted the lsc_script branch January 28, 2022 18:12
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants