diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2022-11-16 07:52:11 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2022-11-16 07:52:11 +0200 |
commit | f80c8ff7ff3b1eef22a3c90943f324d45d855b97 (patch) | |
tree | f44a25aa71de05e11c417dcc01cc3f6b9eca97ee /libbuild2/test/rule.cxx | |
parent | 44b0a5989f76570fc19dc41314f31c4fa9c2039b (diff) |
Initial low verbosity diagnostics rework
Diffstat (limited to 'libbuild2/test/rule.cxx')
-rw-r--r-- | libbuild2/test/rule.cxx | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/libbuild2/test/rule.cxx b/libbuild2/test/rule.cxx index 8a754e5..5cdd8fb 100644 --- a/libbuild2/test/rule.cxx +++ b/libbuild2/test/rule.cxx @@ -540,11 +540,19 @@ namespace build2 if (verb) { - diag_record dr (text); - dr << "test " << ts; - - if (!t.is_a<alias> ()) - dr << ' ' << t; + // If the target is an alias, then testscript itself is the + // target. + // + if (t.is_a<alias> ()) + print_diag ("test", ts); + else + { + // In this case the test is really a combination of the target + // and testscript and using "->" feels off. Also, let's list the + // testscript after the target even though its a source. + // + print_diag ("test", t, ts, "+"); + } } res.push_back (ctx.dry_run @@ -1247,7 +1255,7 @@ namespace build2 if (verb >= 2) print_process (args); // Note: prints the whole pipeline. else if (verb) - text << "test " << tt; + print_diag ("test", tt); if (!ctx.dry_run) { |