Report all transformer load errors before exiting.

Report all transformer load errors before exiting.

Previously, pub would exit immediately after the first load error was
reported. Now it will wait to see if any other transformers have
errors and report those as well.

R=rnystrom@google.com
BUG=14552
Committed: https://code.google.com/p/dart/source/detail?r=37770

[Bob Nystrom, 2014-06-26 22:24:45]

https://codereview.chromium.org/354053002/diff/1/sdk/lib/_internal/pub/lib/sr...
File sdk/lib/_internal/pub/lib/src/utils.dart (right):

https://codereview.chromium.org/354053002/diff/1/sdk/lib/_internal/pub/lib/sr...
sdk/lib/_internal/pub/lib/src/utils.dart:21: import 'log.dart' as log;
Put this above export.

https://codereview.chromium.org/354053002/diff/1/sdk/lib/_internal/pub/test/t...
File
sdk/lib/_internal/pub/test/transformer/multiple_transformers_reject_their_config_test.dart
(right):

https://codereview.chromium.org/354053002/diff/1/sdk/lib/_internal/pub/test/t...
sdk/lib/_internal/pub/test/transformer/multiple_transformers_reject_their_config_test.dart:54:
pub.stderr.expect(endsWith('Error loading transformer: I hate these '
May as well use a loop here.

[Bob Nystrom, 2014-06-26 22:24:51]

lgtm

[nweiz, 2014-06-26 23:01:04]

https://codereview.chromium.org/354053002/diff/1/sdk/lib/_internal/pub/lib/sr...
File sdk/lib/_internal/pub/lib/src/utils.dart (right):

https://codereview.chromium.org/354053002/diff/1/sdk/lib/_internal/pub/lib/sr...
sdk/lib/_internal/pub/lib/src/utils.dart:21: import 'log.dart' as log;
On 2014/06/26 22:24:45, Bob Nystrom wrote:
> Put this above export.

Done.

https://codereview.chromium.org/354053002/diff/1/sdk/lib/_internal/pub/test/t...
File
sdk/lib/_internal/pub/test/transformer/multiple_transformers_reject_their_config_test.dart
(right):

https://codereview.chromium.org/354053002/diff/1/sdk/lib/_internal/pub/test/t...
sdk/lib/_internal/pub/test/transformer/multiple_transformers_reject_their_config_test.dart:54:
pub.stderr.expect(endsWith('Error loading transformer: I hate these '
On 2014/06/26 22:24:45, Bob Nystrom wrote:
> May as well use a loop here.

Done.

[nweiz, 2014-06-26 23:02:41]

Committed patchset #2 manually as r37770 (presubmit successful).