From 8c3cf2716a952f8d35f10113bbd6a41729e70c26 Mon Sep 17 00:00:00 2001
From: Aurelien Regat-Barrel <arb@cyberkarma.net>
Date: Wed, 9 Nov 2022 12:03:11 +0100
Subject: file(DOWNLOAD EXPECTED_HASH): better error message when download
 failed

---
 Source/cmFileCommand.cxx                              | 14 +++++++++-----
 Tests/RunCMake/file/DOWNLOAD-hash-mismatch-stderr.txt |  1 -
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/Source/cmFileCommand.cxx b/Source/cmFileCommand.cxx
index fe38db5..cc9b256 100644
--- a/Source/cmFileCommand.cxx
+++ b/Source/cmFileCommand.cxx
@@ -2107,6 +2107,14 @@ bool HandleDownloadCommand(std::vector<std::string> const& args,
   // Verify MD5 sum if requested:
   //
   if (hash) {
+    if (res != CURLE_OK) {
+      status.SetError(cmStrCat(
+        "DOWNLOAD cannot compute hash on failed download\n"
+        "  status: [",
+        static_cast<int>(res), ";\"", ::curl_easy_strerror(res), "\"]"));
+      return false;
+    }
+
     std::string actualHash = hash->HashFile(file);
     if (actualHash.empty()) {
       status.SetError("DOWNLOAD cannot compute hash on downloaded file");
@@ -2130,11 +2138,7 @@ bool HandleDownloadCommand(std::vector<std::string> const& args,
                                expectedHash,
                                "]\n"
                                "      actual hash: [",
-                               actualHash,
-                               "]\n"
-                               "           status: [",
-                               static_cast<int>(res), ";\"",
-                               ::curl_easy_strerror(res), "\"]\n"));
+                               actualHash, "]\n"));
       return false;
     }
   }
diff --git a/Tests/RunCMake/file/DOWNLOAD-hash-mismatch-stderr.txt b/Tests/RunCMake/file/DOWNLOAD-hash-mismatch-stderr.txt
index 406e315..7a356c1 100644
--- a/Tests/RunCMake/file/DOWNLOAD-hash-mismatch-stderr.txt
+++ b/Tests/RunCMake/file/DOWNLOAD-hash-mismatch-stderr.txt
@@ -4,7 +4,6 @@
     for file: \[.*/Tests/RunCMake/file/DOWNLOAD-hash-mismatch-build/hash-mismatch.txt\]
       expected hash: \[0123456789abcdef0123456789abcdef01234567\]
         actual hash: \[da39a3ee5e6b4b0d3255bfef95601890afd80709\]
-             status: \[0;"No error"\]
 
 Call Stack \(most recent call first\):
   CMakeLists.txt:[0-9]+ \(include\)
-- 
cgit v0.12