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

Unified Diff: sdk/lib/_internal/pub_generated/lib/src/command/uploader.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/uploader.dart
diff --git a/sdk/lib/_internal/pub_generated/lib/src/command/uploader.dart b/sdk/lib/_internal/pub_generated/lib/src/command/uploader.dart
index 20c8c98a804e5da97421e03594528dc5b1bb90be..e43bbaea5980baa3e8ed0b23dca5e53f56cf9ead 100644
--- a/sdk/lib/_internal/pub_generated/lib/src/command/uploader.dart
+++ b/sdk/lib/_internal/pub_generated/lib/src/command/uploader.dart
@@ -19,34 +19,34 @@ import '../source/hosted.dart';
/// Handles the `uploader` pub command.
class UploaderCommand extends PubCommand {
+ String get name => "uploader";
String get description =>
"Manage uploaders for a package on pub.dartlang.org.";
- String get usage => "pub uploader [options] {add/remove} <email>";
+ String get invocation => "pub uploader [options] {add/remove} <email>";
String get docUrl => "http://dartlang.org/tools/pub/cmd/pub-uploader.html";
- bool get takesArguments => true;
/// The URL of the package hosting server.
- Uri get server => Uri.parse(commandOptions['server']);
+ Uri get server => Uri.parse(argResults['server']);
UploaderCommand() {
- commandParser.addOption(
+ argParser.addOption(
'server',
defaultsTo: HostedSource.defaultUrl,
help: 'The package server on which the package is hosted.');
- commandParser.addOption(
+ argParser.addOption(
'package',
help: 'The package whose uploaders will be modified.\n'
'(defaults to the current package)');
}
- Future onRun() {
- if (commandOptions.rest.isEmpty) {
+ Future run() {
+ if (argResults.rest.isEmpty) {
log.error('No uploader command given.');
this.printUsage();
return flushThenExit(exit_codes.USAGE);
}
- var rest = commandOptions.rest.toList();
+ var rest = argResults.rest.toList();
// TODO(rnystrom): Use subcommands for these.
var command = rest.removeAt(0);
@@ -61,7 +61,7 @@ class UploaderCommand extends PubCommand {
}
return new Future.sync(() {
- var package = commandOptions['package'];
+ var package = argResults['package'];
if (package != null) return package;
return new Entrypoint(path.current, cache).root.name;
}).then((package) {

Powered by Google App Engine
This is Rietveld 408576698