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

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Aug 23, 2021

Removes gradle wrapper from Android projects, updates build rules/scripts. This eliminates downloading gradle itself at build time, but does not eliminate other network connections gradle makes to download app dependencies.

I created the cipd package by downloading the gradle distribution, unzipping it, and running:

cipd create -in gradle-7.0.2 -install-mode copy -tag version:7.0.2 -name flutter/gradle 

Part of flutter/flutter#88717

@dnfield dnfield requested review from blasten and zanderso August 23, 2021 17:45
@google-cla google-cla bot added the cla: yes label Aug 23, 2021
@zanderso
Copy link
Member

The license checker has flagged an issue

@dnfield
Copy link
Contributor Author

dnfield commented Aug 23, 2021

Looking into that - it's failing to realize this is apache licensed somehow.

@dnfield
Copy link
Contributor Author

dnfield commented Aug 23, 2021

Ok, telling the tool to ignore //third_party/gradle since we only use it for build. This will likely trigger a different failure but it's tends to work better IME to get the new license tool hash from CI than to try to get it locally.

@dnfield
Copy link
Contributor Author

dnfield commented Aug 23, 2021

Builds should succeed now.


SCRIPT_PATH = os.path.dirname(os.path.realpath(__file__))

BAT = '.bat' if sys.platform.startswith(('cygwin', 'win')) else ''
Copy link

Choose a reason for hiding this comment

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

nit: BAT -> EXT

Copy link

@blasten blasten left a comment

Choose a reason for hiding this comment

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

LGTM

@zanderso
Copy link
Member

Presubmit still looks sad about the license tool signature. Otherwise lgtm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants