Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file | 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file |
| 2 // for details. All rights reserved. Use of this source code is governed by a | 2 // for details. All rights reserved. Use of this source code is governed by a |
| 3 // BSD-style license that can be found in the LICENSE file. | 3 // BSD-style license that can be found in the LICENSE file. |
| 4 | 4 |
| 5 /** | 5 /** |
| 6 * Handles the `list` pub command. This is mostly just used so we can pull | 6 * Handles the `list` pub command. This is mostly just used so we can pull |
| 7 * some basic data out of pub in the integration tests. Once pub is more | 7 * some basic data out of pub in the integration tests. Once pub is more |
| 8 * full-featured and has other commands that test everything it does, this | 8 * full-featured and has other commands that test everything it does, this |
| 9 * may go away. | 9 * may go away. |
| 10 */ | 10 */ |
| 11 class ListCommand extends PubCommand { | 11 class ListCommand extends PubCommand { |
| 12 String get description() => 'print the contents of repositories'; | 12 String get description() => 'print the contents of repositories'; |
| 13 | 13 |
| 14 void onRun() { | 14 void onRun() { |
| 15 cache.listAll().then((packages) { | 15 printIds(String title, List<PackageId> ids) { |
|
Bob Nystrom
2012/05/03 00:15:49
I would go ahead and make this a method. Part of t
nweiz
2012/05/04 01:03:43
Done.
| |
| 16 packages.sort((a, b) => a.name.compareTo(b.name)); | 16 ids = new List<PackageId>.from(ids); |
|
Bob Nystrom
2012/05/03 00:15:49
How about making PackageId implement Comparable?
nweiz
2012/05/04 01:03:43
Done.
| |
| 17 for (final package in packages) { | 17 ids.sort((a, b) { |
| 18 print(package.name); | 18 var sourceComp = a.source.name.compareTo(b.source.name); |
| 19 if (sourceComp != 0) return sourceComp; | |
| 20 return a.name.compareTo(b.name); | |
| 21 }); | |
| 22 | |
| 23 print('From $title:'); | |
| 24 for (var id in ids) { | |
| 25 print(' $id'); | |
| 19 } | 26 } |
| 20 }); | 27 } |
| 28 | |
| 29 // TODO(nweiz): also list the contents of the application cache when it's | |
| 30 // able to determine the source of its packages (that is, when we have a | |
| 31 // lockfile). | |
| 32 systemCache.listAll().then((ids) => printIds('system cache', ids)); | |
| 21 } | 33 } |
| 22 } | 34 } |
| OLD | NEW |