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

Unified Diff: mojo/public/dart/third_party/barback/lib/src/build_result.dart

Issue 1346773002: Stop running pub get at gclient sync time and fix build bugs (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 3 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: mojo/public/dart/third_party/barback/lib/src/build_result.dart
diff --git a/mojo/public/dart/third_party/barback/lib/src/build_result.dart b/mojo/public/dart/third_party/barback/lib/src/build_result.dart
new file mode 100644
index 0000000000000000000000000000000000000000..faeb2566a5fd746e29bbf630ed03810a7f72cf5e
--- /dev/null
+++ b/mojo/public/dart/third_party/barback/lib/src/build_result.dart
@@ -0,0 +1,61 @@
+// Copyright (c) 2013, 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.
+
+library barback.build_result;
+
+import 'errors.dart';
+import 'utils.dart';
+
+/// An event indicating that the cascade has finished building all assets.
+///
+/// A build can end either in success or failure. If there were no errors during
+/// the build, it's considered to be a success; any errors render it a failure,
+/// although individual assets may still have built successfully.
+class BuildResult {
+ // TODO(rnystrom): Revise how to track error results. Errors can come from
+ // both logs and exceptions. Accumulating them is likely slow and a waste of
+ // memory. If we do want to accumulate them, we should at least unify them
+ // in a single collection (probably of log entries).
+ /// All errors that were thrown during the build.
+ final Set<BarbackException> errors;
+
+ /// `true` if the build succeeded.
+ bool get succeeded => errors.isEmpty;
+
+ BuildResult(Iterable<BarbackException> errors)
+ : errors = flattenAggregateExceptions(errors).toSet();
+
+ /// Creates a build result indicating a successful build.
+ ///
+ /// This equivalent to a build result with no errors.
+ BuildResult.success()
+ : this([]);
+
+ /// Creates a single [BuildResult] that contains all of the errors of
+ /// [results].
+ factory BuildResult.aggregate(Iterable<BuildResult> results) {
+ var errors = unionAll(results.map((result) => result.errors));
+ return new BuildResult(errors);
+ }
+
+ String toString() {
+ if (succeeded) return "success";
+
+ return "errors:\n" + errors.map((error) {
+ var stackTrace = null;
+ if (error is TransformerException || error is AssetLoadException) {
+ stackTrace = error.stackTrace.terse;
+ }
+
+ var msg = new StringBuffer();
+ msg.write(prefixLines(error.toString()));
+ if (stackTrace != null) {
+ msg.write("\n\n");
+ msg.write("Stack chain:\n");
+ msg.write(prefixLines(stackTrace.toString()));
+ }
+ return msg.toString();
+ }).join("\n\n");
+ }
+}
« no previous file with comments | « mojo/public/dart/third_party/barback/lib/src/barback.dart ('k') | mojo/public/dart/third_party/barback/lib/src/errors.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698