don't use internal throwOnWarnings option

pkg/front_end/tool/bazel/worker.dart

 // Copyright (c) 2017, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 import 'dart:async';
 import 'dart:io';
 
 import 'package:args/args.dart';
 import 'package:bazel_worker/bazel_worker.dart';
 import 'package:front_end/front_end.dart' hide FileSystemException;
 import 'package:front_end/src/fasta/command_line_reporting.dart';
@@ skipping 14 matching lines @@
 }
 
 /// A bazel worker loop that can compute summaries.
 class SummaryWorker extends AsyncWorkerLoop {
   Future<WorkResponse> performRequest(WorkRequest request) async {
     var outputBuffer = new StringBuffer();
     var response = new WorkResponse()..exitCode = 0;
     try {
       await computeSummary(request.arguments,
           isWorker: true, outputBuffer: outputBuffer);
-    } catch (_, s) {
+    } catch (e, s) {
+      outputBuffer.writeln(e);
       outputBuffer.writeln(s);
       response.exitCode = 15;
     }
     response.output = outputBuffer.toString();
     return response;
   }
 }
 
 /// If the last arg starts with `@`, this reads the file it points to and treats
 /// each line as an additional arg.
@@ skipping 36 matching lines @@
 Future computeSummary(List<String> args,
     {bool isWorker: false, StringBuffer outputBuffer}) async {
   var parsedArgs = summaryArgsParser.parse(args);
   var options = new CompilerOptions()
     ..packagesFileUri = Uri.parse(parsedArgs['packages-file'])
     ..inputSummaries = parsedArgs['input-summary'].map(Uri.parse).toList()
     ..sdkSummary = Uri.parse(parsedArgs['dart-sdk-summary'])
     ..multiRoots = parsedArgs['multi-root'].map(Uri.parse).toList()
     ..target = new NoneTarget(new TargetFlags());
 
-  if (outputBuffer != null) {
-    options.onError = (CompilationMessage error) {
-      var severityString = severityName(error.severity, capitalized: true);
+  options.onError = (CompilationMessage error) {
+    var severityString = severityName(error.severity, capitalized: true);
+    if (outputBuffer != null) {
       outputBuffer.writeln('$severityString: ${error.message}');
-      if (error.severity != Severity.nit) {
-        throw error;
-      }
-    };
-  } else {
-    options.throwOnWarningsForDebugging = true;
-  }
+    } else {
+      print('$severityString: ${error.message}');
+    }
+    if (error.severity != Severity.nit) {
+      throw error;

[Siggi Cherem (dart-lang), 2017/09/01 17:19:08]

Instead of throwing here, consider setting a boolean that indicates that errors
were seen. Then throw right after `summaryFor`?

That way you can report all errors that come up, and not just the first one.

+    }
+  };
 
   var sources = parsedArgs['source'].map(Uri.parse).toList();
   var program = await summaryFor(sources, options);
 
   var outputFile = new File(parsedArgs['output']);
   outputFile.createSync(recursive: true);
   outputFile.writeAsBytesSync(program);
 }