From 811e9be4cc9604dba3d768639444c0d9f849a3c7 Mon Sep 17 00:00:00 2001 From: Paul Cristian Sarbu Date: Mon, 26 May 2025 14:46:25 +0200 Subject: RepositoryConfig: Give access to a persistent storage config... ...whenever it is given access to a Git repository. The referenced storage config needs to outlive the repository config instance. --- src/buildtool/tree_structure/tree_structure_utils.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'src/buildtool/tree_structure/tree_structure_utils.cpp') diff --git a/src/buildtool/tree_structure/tree_structure_utils.cpp b/src/buildtool/tree_structure/tree_structure_utils.cpp index f09d627b..758e47fc 100644 --- a/src/buildtool/tree_structure/tree_structure_utils.cpp +++ b/src/buildtool/tree_structure/tree_structure_utils.cpp @@ -208,6 +208,7 @@ auto TreeStructureUtils::ImportToGit( auto TreeStructureUtils::ExportFromGit( ArtifactDigest const& tree, std::vector const& source_repos, + StorageConfig const& storage_config, IExecutionApi const& target_api) noexcept -> expected { if (not tree.IsTree() or not ProtocolTraits::IsNative(tree.GetHashType())) { return unexpected{fmt::format("Not a git tree: {}", tree.hash())}; @@ -231,7 +232,7 @@ auto TreeStructureUtils::ExportFromGit( } RepositoryConfig repo_config{}; - if (not repo_config.SetGitCAS(*repo)) { + if (not repo_config.SetGitCAS(*repo, &storage_config)) { return unexpected{ fmt::format("Failed to set git cas at {}", repo->string())}; } @@ -276,7 +277,8 @@ auto TreeStructureUtils::ComputeStructureLocally( // If the tree is not in the storage, it must be present in git: if (not storage.CAS().TreePath(tree).has_value()) { - auto in_cas = ExportFromGit(tree, known_repositories, local_api); + auto in_cas = + ExportFromGit(tree, known_repositories, storage_config, local_api); if (not in_cas.has_value()) { return unexpected{ fmt::format("While exporting {} from git to CAS:\n{}", -- cgit v1.2.3