summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKlaus Aehlig <klaus.aehlig@huawei.com>2025-04-10 14:56:38 +0200
committerKlaus Aehlig <klaus.aehlig@huawei.com>2025-04-10 14:56:38 +0200
commitaff782ac8e0b7465dd62e4e645cce3cd7f7205af (patch)
tree078e3973927f4d6eb91c938ef71de74c4de9fc31
parentb37d36aaa4104f7e8e3fc5e9a1520656ba4e1839 (diff)
downloadjustbuild-aff782ac8e0b7465dd62e4e645cce3cd7f7205af.tar.gz
graph_traverser: drop logging graph-internal identifiers
Those are the sha256sum of the serialisation of an artifact and that serialisation does not end up in the compatible CAS. In other words, they do not refer to anything the user can access. Therefore, drop this message that is not helpful.
-rw-r--r--src/buildtool/graph_traverser/graph_traverser.cpp3
1 files changed, 0 insertions, 3 deletions
diff --git a/src/buildtool/graph_traverser/graph_traverser.cpp b/src/buildtool/graph_traverser/graph_traverser.cpp
index 0ba2f09a..440a8078 100644
--- a/src/buildtool/graph_traverser/graph_traverser.cpp
+++ b/src/buildtool/graph_traverser/graph_traverser.cpp
@@ -569,7 +569,6 @@ void GraphTraverser::PrintOutputs(
std::vector<std::filesystem::path> const& paths,
std::vector<DependencyGraph::ArtifactNode const*> const& artifact_nodes,
std::map<std::string, ArtifactDescription> const& runfiles) const {
- std::string msg_dbg{"Artifact ids:"};
std::string msg_failed{"Failed artifacts:"};
bool failed{false};
nlohmann::json json{};
@@ -601,7 +600,6 @@ void GraphTraverser::PrintOutputs(
id);
}
}
- msg_dbg += fmt::format("\n {}: {}", path, id);
}
if (not clargs_.build.show_runfiles and not runfiles.empty()) {
@@ -609,7 +607,6 @@ void GraphTraverser::PrintOutputs(
}
Logger::Log(logger_, LogLevel::Info, "{}", message);
- Logger::Log(logger_, LogLevel::Debug, "{}", msg_dbg);
if (failed) {
Logger::Log(logger_, LogLevel::Info, "{}", msg_failed);
}