diff options
| author | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2023-10-31 16:08:45 +0100 |
|---|---|---|
| committer | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2023-11-14 13:35:01 +0100 |
| commit | fca9ce75d5cb6d90edab84aec5fdfea7af7d8b59 (patch) | |
| tree | c709128abc46b8a9fa41e0b05e95c3d212b5f61e /src/other_tools | |
| parent | 17f88a4752243f09ede9571014b8d723aca6ca44 (diff) | |
| download | justbuild-fca9ce75d5cb6d90edab84aec5fdfea7af7d8b59.tar.gz | |
DistdirCheckout: Improve log messages
Diffstat (limited to 'src/other_tools')
| -rw-r--r-- | src/other_tools/repo_map/repos_to_setup_map.cpp | 30 |
1 files changed, 18 insertions, 12 deletions
diff --git a/src/other_tools/repo_map/repos_to_setup_map.cpp b/src/other_tools/repo_map/repos_to_setup_map.cpp index d0396a0c..4c1b80ad 100644 --- a/src/other_tools/repo_map/repos_to_setup_map.cpp +++ b/src/other_tools/repo_map/repos_to_setup_map.cpp @@ -471,30 +471,36 @@ void DistdirCheckout(ExpressionPtr const& repo_desc, // check mandatory fields auto repo_desc_content = (*resolved_repo_desc)->At("content"); if (not repo_desc_content) { - (*logger)( - "DistdirCheckout: Mandatory field \"content\" is " - "missing", - /*fatal=*/true); + (*logger)(fmt::format( + "DistdirCheckout: Mandatory field \"content\" is " + "missing for repository {}", + nlohmann::json(dist_repo_name).dump()), + /*fatal=*/true); return; } if (not repo_desc_content->get()->IsString()) { (*logger)(fmt::format("DistdirCheckout: Unsupported value {} " - "for mandatory field \"content\"", - repo_desc_content->get()->ToString()), + "for mandatory field \"content\" for " + "repository {}", + repo_desc_content->get()->ToString(), + nlohmann::json(dist_repo_name).dump()), /*fatal=*/true); return; } auto repo_desc_fetch = (*resolved_repo_desc)->At("fetch"); if (not repo_desc_fetch) { - (*logger)( - "DistdirCheckout: Mandatory field \"fetch\" is missing", - /*fatal=*/true); + (*logger)(fmt::format("DistdirCheckout: Mandatory field " + "\"fetch\" is missing for repository {}", + nlohmann::json(dist_repo_name).dump()), + /*fatal=*/true); return; } if (not repo_desc_fetch->get()->IsString()) { - (*logger)(fmt::format("DistdirCheckout: Unsupported value {} " - "for mandatory field \"fetch\"", - repo_desc_fetch->get()->ToString()), + (*logger)(fmt::format( + "DistdirCheckout: Unsupported value {} " + "for mandatory field \"fetch\" for repository {}", + repo_desc_fetch->get()->ToString(), + nlohmann::json(dist_repo_name).dump()), /*fatal=*/true); return; } |