-
Notifications
You must be signed in to change notification settings - Fork 27k
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
Add a Flutter command that builds an APK using a local build of the e… #437
Conversation
@@ -113,6 +113,16 @@ class FlutterCommandRunner extends CommandRunner { | |||
} | |||
List<BuildConfiguration> _buildConfigurations; | |||
|
|||
String get enginePath { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it'd be better to use the BuildConfiguration logic, and loop through and build the APK for all the relevant builds, the same way that "flutter test --debug --release" tests both debug and release builds. In particular, how do you know whether to use debug or release builds this way?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed this and decided on a policy of using the first BuildConfiguration having Android as the target platform.
60969b1
to
408c345
Compare
LGTM. Note: Travis is mad. |
} | ||
|
||
BuildConfiguration config = buildConfigurations.firstWhere( | ||
(BuildConfiguration bc) => bc.targetPlatform == TargetPlatform.android); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: ) on next line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
LGTM, just some minor nits. |
408c345
to
83540c0
Compare
…ngine Example: cd flutter/examples/stocks flutter --engine-src-path /path/to/engine/src apk -o Stocks.apk -m apk/AndroidManifest.xml
83540c0
to
0755642
Compare
Add a Flutter command that builds an APK using a local build of the e…
…ngine
Example:
cd flutter/examples/stocks
flutter --engine-src-path /path/to/engine/src apk -o Stocks.apk -m apk/AndroidManifest.xml