Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(406)

Unified Diff: sdk/lib/_internal/pub_generated/lib/src/command/global_activate.dart

Issue 805393002: Update pub to use the new command runner API in args. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Code review changes Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sdk/lib/_internal/pub_generated/lib/src/command/global_activate.dart
diff --git a/sdk/lib/_internal/pub_generated/lib/src/command/global_activate.dart b/sdk/lib/_internal/pub_generated/lib/src/command/global_activate.dart
index 2e51a90ece18176e08804cf945ec32497c49ca09..f567f4bf04b6918ff4b89e9753ad975936ce18bb 100644
--- a/sdk/lib/_internal/pub_generated/lib/src/command/global_activate.dart
+++ b/sdk/lib/_internal/pub_generated/lib/src/command/global_activate.dart
@@ -13,54 +13,54 @@ import '../utils.dart';
/// Handles the `global activate` pub command.
class GlobalActivateCommand extends PubCommand {
+ String get name => "activate";
String get description => "Make a package's executables globally available.";
- String get usage => "pub global activate <package...>";
- bool get takesArguments => true;
+ String get invocation => "pub global activate <package...>";
GlobalActivateCommand() {
- commandParser.addOption(
+ argParser.addOption(
"source",
abbr: "s",
help: "The source used to find the package.",
allowed: ["git", "hosted", "path"],
defaultsTo: "hosted");
- commandParser.addFlag(
+ argParser.addFlag(
"no-executables",
negatable: false,
help: "Do not put executables on PATH.");
- commandParser.addOption(
+ argParser.addOption(
"executable",
abbr: "x",
help: "Executable(s) to place on PATH.",
allowMultiple: true);
- commandParser.addFlag(
+ argParser.addFlag(
"overwrite",
negatable: false,
help: "Overwrite executables from other packages with the same name.");
}
- Future onRun() {
+ Future run() {
// Default to `null`, which means all executables.
var executables;
- if (commandOptions.wasParsed("executable")) {
- if (commandOptions.wasParsed("no-executables")) {
- usageError("Cannot pass both --no-executables and --executable.");
+ if (argResults.wasParsed("executable")) {
+ if (argResults.wasParsed("no-executables")) {
+ usageException("Cannot pass both --no-executables and --executable.");
}
- executables = commandOptions["executable"];
- } else if (commandOptions["no-executables"]) {
+ executables = argResults["executable"];
+ } else if (argResults["no-executables"]) {
// An empty list means no executables.
executables = [];
}
- var overwrite = commandOptions["overwrite"];
- var args = commandOptions.rest;
+ var overwrite = argResults["overwrite"];
+ var args = argResults.rest;
readArg([String error]) {
- if (args.isEmpty) usageError(error);
+ if (args.isEmpty) usageException(error);
var arg = args.first;
args = args.skip(1);
return arg;
@@ -70,10 +70,10 @@ class GlobalActivateCommand extends PubCommand {
if (args.isEmpty) return;
var unexpected = args.map((arg) => '"$arg"');
var arguments = pluralize("argument", unexpected.length);
- usageError("Unexpected $arguments ${toSentence(unexpected)}.");
+ usageException("Unexpected $arguments ${toSentence(unexpected)}.");
}
- switch (commandOptions["source"]) {
+ switch (argResults["source"]) {
case "git":
var repo = readArg("No Git repository given.");
// TODO(rnystrom): Allow passing in a Git ref too.
@@ -92,7 +92,7 @@ class GlobalActivateCommand extends PubCommand {
try {
constraint = new VersionConstraint.parse(readArg());
} on FormatException catch (error) {
- usageError(error.message);
+ usageException(error.message);
}
}

Powered by Google App Engine
This is Rietveld 408576698