git_repo: return keep tags on success

... instead of the plain boolean value.

1 files changed, 6 insertions, 4 deletions

diff --git a/src/buildtool/file_system/git_repo.hpp b/src/buildtool/file_system/git_repo.hpp
index 2a4a4238..95fa14a6 100644
--- a/src/buildtool/file_system/git_repo.hpp
+++ b/src/buildtool/file_system/git_repo.hpp
@@ -159,11 +159,12 @@ class GitRepo {
 
     /// \brief Create annotated tag for given commit.
     /// Only possible with real repository and thus non-thread-safe.
-    /// Returns success flag.
+    /// Returns the tag on success.
     /// It guarantees the logger is called exactly once with fatal if failure.
     [[nodiscard]] auto KeepTag(std::string const& commit,
                                std::string const& message,
-                               anon_logger_ptr const& logger) noexcept -> bool;
+                               anon_logger_ptr const& logger) noexcept
+        -> std::optional<std::string>;
 
     /// \brief Retrieves the commit of the HEAD reference.
     /// Only possible with real repository and thus non-thread-safe.
@@ -188,11 +189,12 @@ class GitRepo {
     /// \brief Ensure given tree is kept alive via a tag. It is expected that
     /// the tree is part of the repository already.
     /// Only possible with real repository and thus non-thread-safe.
-    /// Returns success flag.
+    /// Returns the tag on success.
     /// It guarantees the logger is called exactly once with fatal if failure.
     [[nodiscard]] auto KeepTree(std::string const& tree_id,
                                 std::string const& message,
-                                anon_logger_ptr const& logger) noexcept -> bool;
+                                anon_logger_ptr const& logger) noexcept
+        -> std::optional<std::string>;
 
     /// \brief Get the tree id of a subtree given the root commit
     /// Calling it from a fake repository allows thread-safe use.