Commit 533c7b56f2624f1df6684a834f5780a38052bb00 | zig

author	Andrew Kelley <andrew@ziglang.org> 2023-03-01 14:40:14-07:00
committer	Andrew Kelley <andrew@ziglang.org> 2023-03-15 10:48:13-07:00
log	533c7b56f2624f1df6684a834f5780a38052bb00
tree	19aa5a3d57c8baa423075aa68aa3273b1c7f559b
parent	b4997d08902ed312d7504765014c34f4ea862534

build runner: hide repeated steps in the build summary

1 files changed, 53 insertions(+), 37 deletions(-)

lib
- build_runner.zig

lib/build_runner.zig+53-37

 @@ -350,6 +350,8 @@ fn runStepNames(
     var failure_count: usize = 0;
     var pending_count: usize = 0;
     var total_compile_errors: usize = 0;
+    var compile_error_steps: std.ArrayListUnmanaged(*Step) = .{};
+    defer compile_error_steps.deinit(gpa);
     for (step_stack.keys()) |s| {
         switch (s.state) {
 @@ -369,7 +371,11 @@ fn runStepNames(
             .success => success_count += 1,
             .failure => {
                 failure_count += 1;
-                total_compile_errors += s.result_error_bundle.errorMessageCount();
+                const compile_errors_len = s.result_error_bundle.errorMessageCount();
+                if (compile_errors_len > 0) {
+                    total_compile_errors += compile_errors_len;
+                    try compile_error_steps.append(gpa, s);
+                }
             },
         }
     }
 @@ -392,20 +398,22 @@ fn runStepNames(
     var print_node: PrintNode = .{ .parent = null };
     if (step_names.len == 0) {
         print_node.last = true;
-        printTreeStep(b, b.default_step, stderr, ttyconf, &print_node) catch {};
+        printTreeStep(b, b.default_step, stderr, ttyconf, &print_node, &step_stack) catch {};
     } else {
         for (step_names, 0..) |step_name, i| {
             const tls = b.top_level_steps.get(step_name).?;
             print_node.last = i + 1 == b.top_level_steps.count();
-            printTreeStep(b, &tls.step, stderr, ttyconf, &print_node) catch {};
+            printTreeStep(b, &tls.step, stderr, ttyconf, &print_node, &step_stack) catch {};
         }
     }
     if (failure_count == 0) return cleanExit();
     // Finally, render compile errors at the bottom of the terminal.
+    // We use a separate compile_error_steps array list because step_stack is destructively
+    // mutated in printTreeStep above.
     if (total_compile_errors > 0) {
-        for (step_stack.keys()) |s| {
+        for (compile_error_steps.items) |s| {
             if (s.result_error_bundle.errorMessageCount() > 0) {
                 s.result_error_bundle.renderToStdErr(ttyconf);
             }
 @@ -442,7 +450,10 @@ fn printTreeStep(
     stderr: std.fs.File,
     ttyconf: std.debug.TTY.Config,
     parent_node: *PrintNode,
+    step_stack: *std.AutoArrayHashMapUnmanaged(*Step, void),
 ) !void {
+    const first = step_stack.swapRemove(s);
+    if (!first) try ttyconf.setColor(stderr, .Dim);
     try printPrefix(parent_node, stderr);
     if (parent_node.parent != null) {
 @@ -456,43 +467,48 @@ fn printTreeStep(
     // TODO print the dep prefix too?
     try stderr.writeAll(s.name);
-    switch (s.state) {
+    if (first) {
-        .precheck_unstarted => unreachable,
+        switch (s.state) {
-        .precheck_started => unreachable,
+            .precheck_unstarted => unreachable,
-        .precheck_done => unreachable,
+            .precheck_started => unreachable,
-        .running => unreachable,
+            .precheck_done => unreachable,
+            .running => unreachable,
-        .dependency_failure => {
+            .dependency_failure => {
-            try ttyconf.setColor(stderr, .Dim);
+                try ttyconf.setColor(stderr, .Dim);
-            try stderr.writeAll(" transitive failure\n");
+                try stderr.writeAll(" transitive failure\n");
-            try ttyconf.setColor(stderr, .Reset);
+                try ttyconf.setColor(stderr, .Reset);
-        },
+            },
-        .success => {
+            .success => {
-            try ttyconf.setColor(stderr, .Green);
+                try ttyconf.setColor(stderr, .Green);
-            try stderr.writeAll(" success\n");
+                try stderr.writeAll(" success\n");
-            try ttyconf.setColor(stderr, .Reset);
+                try ttyconf.setColor(stderr, .Reset);
-        },
+            },
-        .failure => {
+            .failure => {
-            try ttyconf.setColor(stderr, .Red);
+                try ttyconf.setColor(stderr, .Red);
-            if (s.result_error_bundle.errorMessageCount() > 0) {
+                if (s.result_error_bundle.errorMessageCount() > 0) {
-                try stderr.writer().print(" {d} errors\n", .{
+                    try stderr.writer().print(" {d} errors\n", .{
-                    s.result_error_bundle.errorMessageCount(),
+                        s.result_error_bundle.errorMessageCount(),
-                });
+                    });
-            } else {
+                } else {
-                try stderr.writeAll(" failure\n");
+                    try stderr.writeAll(" failure\n");
-            }
+                }
-            try ttyconf.setColor(stderr, .Reset);
+                try ttyconf.setColor(stderr, .Reset);
-        },
+            },
-    }
+        }
-    for (s.dependencies.items, 0..) |dep, i| {
+        for (s.dependencies.items, 0..) |dep, i| {
-        var print_node: PrintNode = .{
+            var print_node: PrintNode = .{
-            .parent = parent_node,
+                .parent = parent_node,
-            .last = i == s.dependencies.items.len - 1,
+                .last = i == s.dependencies.items.len - 1,
-        };
+            };
-        try printTreeStep(b, dep, stderr, ttyconf, &print_node);
+            try printTreeStep(b, dep, stderr, ttyconf, &print_node, step_stack);
+        }
+    } else {
+        try stderr.writer().print(" ({d} repeated dependencies)\n", .{s.dependencies.items.len});
+        try ttyconf.setColor(stderr, .Reset);
     }
 }