From 65d35006de65e540355833a56bcaa7659f6f1afd Mon Sep 17 00:00:00 2001
From: Oliver Reiche <oliver.reiche@huawei.com>
Date: Tue, 12 Dec 2023 11:09:36 +0100
Subject: Filesystem: Fix copy overwrite of symlink with file

... and improve log messages in case of failure.
---
 src/buildtool/execution_api/git/git_api.hpp            | 3 +++
 src/buildtool/execution_api/local/local_api.hpp        | 3 +++
 src/buildtool/execution_api/remote/bazel/bazel_api.cpp | 3 +++
 3 files changed, 9 insertions(+)

(limited to 'src/buildtool/execution_api')

diff --git a/src/buildtool/execution_api/git/git_api.hpp b/src/buildtool/execution_api/git/git_api.hpp
index 52c21627..dcab8335 100644
--- a/src/buildtool/execution_api/git/git_api.hpp
+++ b/src/buildtool/execution_api/git/git_api.hpp
@@ -83,6 +83,9 @@ class GitApi final : public IExecutionApi {
                     not FileSystemManager::WriteFileAs</*kSetEpochTime=*/true,
                                                        /*kSetWritable=*/true>(
                         *blob, output_paths[i], info.type)) {
+                    Logger::Log(LogLevel::Error,
+                                "staging to output path {} failed.",
+                                output_paths[i].string());
                     return false;
                 }
             }
diff --git a/src/buildtool/execution_api/local/local_api.hpp b/src/buildtool/execution_api/local/local_api.hpp
index d46be8ed..a5596350 100644
--- a/src/buildtool/execution_api/local/local_api.hpp
+++ b/src/buildtool/execution_api/local/local_api.hpp
@@ -114,6 +114,9 @@ class LocalApi final : public IExecutionApi {
                              /*kSetEpochTime=*/true,
                              /*kSetWritable=*/true>(
                              *blob_path, output_paths[i], info.type)) {
+                    Logger::Log(LogLevel::Error,
+                                "staging to output path {} failed.",
+                                output_paths[i].string());
                     return false;
                 }
             }
diff --git a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp
index 3d1cf40e..7539771e 100644
--- a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp
+++ b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp
@@ -263,6 +263,9 @@ auto BazelApi::CreateAction(
             if (not FileSystemManager::WriteFileAs</*kSetEpochTime=*/true,
                                                    /*kSetWritable=*/true>(
                     blobs[pos].data, output_paths[gpos], type)) {
+                Logger::Log(LogLevel::Error,
+                            "staging to output path {} failed.",
+                            output_paths[gpos].string());
                 return false;
             }
         }
-- 
cgit v1.2.3