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

Unified Diff: sdk/lib/_internal/pub/lib/src/command/uploader.dart

Issue 19810004: pkg/args: expose immutable collections (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: trying again Created 7 years, 5 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
« no previous file with comments | « pkg/args/pubspec.yaml ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sdk/lib/_internal/pub/lib/src/command/uploader.dart
diff --git a/sdk/lib/_internal/pub/lib/src/command/uploader.dart b/sdk/lib/_internal/pub/lib/src/command/uploader.dart
index 86daaa1c711134d327319d2faa14651f810c75da..4ede2c194860c160fc4c48949843cfc2f962823f 100644
--- a/sdk/lib/_internal/pub/lib/src/command/uploader.dart
+++ b/sdk/lib/_internal/pub/lib/src/command/uploader.dart
@@ -44,12 +44,14 @@ class UploaderCommand extends PubCommand {
exit(exit_codes.USAGE);
}
- var command = commandOptions.rest.removeAt(0);
+ var rest = commandOptions.rest.toList();
+
+ var command = rest.removeAt(0);
if (!['add', 'remove'].contains(command)) {
log.error('Unknown uploader command "$command".');
this.printUsage();
exit(exit_codes.USAGE);
- } else if (commandOptions.rest.isEmpty) {
+ } else if (rest.isEmpty) {
log.error('No uploader given for "pub uploader $command".');
this.printUsage();
exit(exit_codes.USAGE);
@@ -60,7 +62,7 @@ class UploaderCommand extends PubCommand {
if (package != null) return package;
return new Entrypoint(path.current, cache).root.name;
}).then((package) {
- var uploader = commandOptions.rest[0];
+ var uploader = rest[0];
return oauth2.withClient(cache, (client) {
if (command == 'add') {
var url = server.resolve("/api/packages/"
« no previous file with comments | « pkg/args/pubspec.yaml ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698