Commit cbb2761e authored by Devon Carew's avatar Devon Carew

Merge pull request #1560 from devoncarew/target_option

normalize -t --target option
parents 4b7d601e bfb89195
......@@ -159,8 +159,8 @@ class ApkCommand extends FlutterCommand {
help: 'Output APK file.');
argParser.addOption('target',
abbr: 't',
defaultsTo: '',
help: 'Target app path or filename used to build the FLX.');
defaultsTo: flx.defaultMainPath,
help: 'Target app path / main entry-point file.');
argParser.addOption('flx',
abbr: 'f',
defaultsTo: '',
......
......@@ -17,7 +17,13 @@ class BuildCommand extends FlutterCommand {
argParser.addFlag('precompiled', negatable: false);
argParser.addOption('asset-base', defaultsTo: defaultAssetBase);
argParser.addOption('compiler');
argParser.addOption('main', defaultsTo: defaultMainPath);
argParser.addOption('target',
abbr: 't',
defaultsTo: defaultMainPath,
help: 'Target app path / main entry-point file.'
);
// TODO(devoncarew): Remove this once the xcode project is switched over.
argParser.addOption('main', hide: true);
argParser.addOption('manifest', defaultsTo: defaultManifestPath);
argParser.addOption('private-key', defaultsTo: defaultPrivateKeyPath);
argParser.addOption('output-file', abbr: 'o', defaultsTo: defaultFlxOutputPath);
......@@ -37,7 +43,7 @@ class BuildCommand extends FlutterCommand {
return await build(
toolchain,
assetBase: argResults['asset-base'],
mainPath: argResults['main'],
mainPath: argResults.wasParsed('main') ? argResults['main'] : argResults['target'],
manifestPath: argResults['manifest'],
outputPath: outputPath,
snapshotPath: argResults['snapshot'],
......
......@@ -11,6 +11,7 @@ import '../application_package.dart';
import '../base/common.dart';
import '../base/context.dart';
import '../device.dart';
import '../flx.dart';
import '../runner/flutter_command.dart';
import '../toolchain.dart';
import 'install.dart';
......@@ -41,7 +42,8 @@ abstract class StartCommandBase extends FlutterCommand {
help: 'Start tracing during startup.');
argParser.addOption('target',
abbr: 't',
help: 'Target app path or filename to start.');
defaultsTo: defaultMainPath,
help: 'Target app path / main entry-point file.');
argParser.addOption('route',
help: 'Which route to load when starting the app.');
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment