diff options
| author | Klaus Aehlig <klaus.aehlig@huawei.com> | 2022-04-26 14:58:42 +0200 |
|---|---|---|
| committer | Klaus Aehlig <klaus.aehlig@huawei.com> | 2022-04-26 16:32:44 +0200 |
| commit | 0bf658d28ed15fb7c37d588ad27cdd8a6a481d2e (patch) | |
| tree | 0f80f6732c897a9f1e63f6ddda144bdf98104a7e /src/buildtool/execution_engine/executor/executor.hpp | |
| parent | a495d8b8c8eb23656da03eb152b7b4c25f6703c6 (diff) | |
| download | justbuild-0bf658d28ed15fb7c37d588ad27cdd8a6a481d2e.tar.gz | |
Rebuild: report full command line
With the reformating of the analyse result we accepted longer
output lines anyway. Allowing them in the report of a detected
flaky action immediately gets the log and console output more
useful, as the source can often be identified without looking
at the full definition of the action in the action graph.
Diffstat (limited to 'src/buildtool/execution_engine/executor/executor.hpp')
| -rw-r--r-- | src/buildtool/execution_engine/executor/executor.hpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/buildtool/execution_engine/executor/executor.hpp b/src/buildtool/execution_engine/executor/executor.hpp index 9ef38488..02caf6c5 100644 --- a/src/buildtool/execution_engine/executor/executor.hpp +++ b/src/buildtool/execution_engine/executor/executor.hpp @@ -552,13 +552,11 @@ class Rebuilder { auto const& action_id = action.Id(); if (msg->tellp() <= 0) { bool tainted = action.MayFail() or action.NoCache(); - static constexpr auto kMaxCmdChars = 69; // 80 - (prefix + suffix) auto cmd = GetCmdString(action); (*msg) << "Found flaky " << (tainted ? "tainted " : "") << "action:" << std::endl << " - id: " << action_id << std::endl - << " - cmd: " << cmd.substr(0, kMaxCmdChars) - << (cmd.length() > kMaxCmdChars ? "..." : "") << std::endl; + << " - cmd: " << cmd << std::endl; } (*msg) << " - output '" << path << "' differs:" << std::endl << " - " << rebuilt.ToString() << " (rebuilt)" << std::endl |