diff options
| author | Brad King <brad.king@kitware.com> | 2022-10-31 13:46:38 +0000 |
|---|---|---|
| committer | Kitware Robot <kwrobot@kitware.com> | 2022-10-31 09:47:19 -0400 |
| commit | 447921ef44eb5e6c26dd122e643b7ac095244dd9 (patch) | |
| tree | b15cea49454ecaca276f6d0c251a4b7bbe163722 /Source | |
| parent | c7e06d0229cf330e375929c363e5c4becaa1d743 (diff) | |
| parent | d78671879b8779ef73b7b256f6d0eeb874ce6687 (diff) | |
| download | cmake-447921ef44eb5e6c26dd122e643b7ac095244dd9.tar.gz | |
Merge topic 'file-download-log-with-hash'
d78671879b file(DOWNLOAD): Fix LOG with EXPECTED_HASH on download failure
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !7848
Diffstat (limited to 'Source')
| -rw-r--r-- | Source/cmFileCommand.cxx | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/Source/cmFileCommand.cxx b/Source/cmFileCommand.cxx index 421ff12557..fe38db53e8 100644 --- a/Source/cmFileCommand.cxx +++ b/Source/cmFileCommand.cxx @@ -2090,6 +2090,13 @@ bool HandleDownloadCommand(std::vector<std::string> const& args, ::curl_global_cleanup(); + // Ensure requested curl logs are returned (especially in case of failure) + // + if (!logVar.empty()) { + chunkDebug.push_back(0); + status.GetMakefile().AddDefinition(logVar, chunkDebug.data()); + } + // Explicitly flush/close so we can measure the md5 accurately. // if (!file.empty()) { @@ -2132,11 +2139,6 @@ bool HandleDownloadCommand(std::vector<std::string> const& args, } } - if (!logVar.empty()) { - chunkDebug.push_back(0); - status.GetMakefile().AddDefinition(logVar, chunkDebug.data()); - } - return true; #else status.SetError("DOWNLOAD not supported by bootstrap cmake."); |