Common API: correctly report failure

If fetching via the primary API failed and there is no fallback,
we should fail rather than tacitly continuing with the next object
to fetch.

(cherry-picked from 74096dce2ee85ea5df940155fc2717d249e14d80)

1 files changed, 5 insertions, 2 deletions

diff --git a/src/buildtool/execution_api/common/common_api.cpp b/src/buildtool/execution_api/common/common_api.cpp
index c21d14a4..7466abbf 100644
--- a/src/buildtool/execution_api/common/common_api.cpp
+++ b/src/buildtool/execution_api/common/common_api.cpp
@@ -58,8 +58,8 @@ auto CommonRetrieveToFds(
                     fd);
                 // locally we might be able to fallback to Git in native mode
                 try {
-                    if (fallback and not(*fallback)(info, fd)) {
-                        return false;
+                    if (fallback) {
+                        success = (*fallback)(info, fd);
                     }
                 } catch (std::exception const& ex) {
                     Logger::Log(LogLevel::Error,
@@ -71,6 +71,9 @@ auto CommonRetrieveToFds(
                     return false;
                 }
             }
+            if (not success) {
+                return false;
+            }
         }
         else {
             Logger::Log(