Revert "(cleanup) standardize on shouldRunPub and usesPubOptions() in commands (#13427)" (#13430)

This reverts commit 9315a79b11.

There is an unfortunate interaction with verifyThenRunCommand
This commit is contained in:
Kevin Moore
2017-12-07 14:29:12 -08:00
committed by GitHub
parent 9315a79b11
commit 64feb95ada
2 changed files with 8 additions and 5 deletions

View File

@@ -28,7 +28,10 @@ import '../version.dart';
class CreateCommand extends FlutterCommand {
CreateCommand() {
usesPubOption();
argParser.addFlag('pub',
defaultsTo: true,
help: 'Whether to run "flutter packages get" after the project has been created.'
);
argParser.addFlag(
'with-driver-test',
negatable: true,
@@ -158,7 +161,7 @@ class CreateCommand extends FlutterCommand {
templateContext['description'] = description;
generatedCount += _renderTemplate('package', dirPath, templateContext);
if (shouldRunPub)
if (argResults['pub'])
await pubGet(context: PubContext.createPackage, directory: dirPath);
final String relativePath = fs.path.relative(dirPath);
@@ -176,7 +179,7 @@ class CreateCommand extends FlutterCommand {
templateContext['description'] = description;
generatedCount += _renderTemplate('plugin', dirPath, templateContext);
if (shouldRunPub)
if (argResults['pub'])
await pubGet(context: PubContext.createPlugin, directory: dirPath);
if (android_sdk.androidSdk != null)
@@ -214,7 +217,7 @@ class CreateCommand extends FlutterCommand {
previewDart2: false,
);
if (shouldRunPub) {
if (argResults['pub']) {
await pubGet(context: PubContext.create, directory: appPath);
injectPlugins(directory: appPath);
}

View File

@@ -362,7 +362,7 @@ abstract class FlutterCommand extends Command<Null> {
}
// Validate the current package map only if we will not be running "pub get" later.
if (parent?.name != 'packages' && !shouldRunPub) {
if (parent?.name != 'packages' && !(_usesPubOption && argResults['pub'])) {
final String error = new PackageMap(PackageMap.globalPackagesPath).checkValid();
if (error != null)
throw new ToolExit(error);