summaryrefslogtreecommitdiff
path: root/src/buildtool/execution_api/common
diff options
context:
space:
mode:
Diffstat (limited to 'src/buildtool/execution_api/common')
-rw-r--r--src/buildtool/execution_api/common/TARGETS7
-rw-r--r--src/buildtool/execution_api/common/api_bundle.cpp2
-rw-r--r--src/buildtool/execution_api/common/api_bundle.hpp24
3 files changed, 8 insertions, 25 deletions
diff --git a/src/buildtool/execution_api/common/TARGETS b/src/buildtool/execution_api/common/TARGETS
index f95764ea..a2063a70 100644
--- a/src/buildtool/execution_api/common/TARGETS
+++ b/src/buildtool/execution_api/common/TARGETS
@@ -39,12 +39,15 @@
, "srcs": ["api_bundle.cpp"]
, "stage": ["src", "buildtool", "execution_api", "common"]
, "deps":
- [ ["src/buildtool/common", "config"]
+ [ "common"
+ , ["src/buildtool/common", "config"]
, ["src/buildtool/common/remote", "remote_common"]
+ ]
+ , "private-deps":
+ [ ["src/buildtool/execution_api/bazel_msg", "bazel_msg"]
, ["src/buildtool/execution_api/local", "local"]
, ["src/buildtool/execution_api/remote", "bazel"]
]
- , "private-deps": [["src/buildtool/execution_api/bazel_msg", "bazel_msg"]]
}
, "message_limits":
{ "type": ["@", "rules", "CC", "library"]
diff --git a/src/buildtool/execution_api/common/api_bundle.cpp b/src/buildtool/execution_api/common/api_bundle.cpp
index 06857fc0..85ab49f1 100644
--- a/src/buildtool/execution_api/common/api_bundle.cpp
+++ b/src/buildtool/execution_api/common/api_bundle.cpp
@@ -15,6 +15,8 @@
#include "src/buildtool/execution_api/common/api_bundle.hpp"
#include "src/buildtool/execution_api/bazel_msg/bazel_common.hpp"
+#include "src/buildtool/execution_api/local/local_api.hpp"
+#include "src/buildtool/execution_api/remote/bazel/bazel_api.hpp"
ApiBundle::ApiBundle(
std::optional<gsl::not_null<const RepositoryConfig*>> const& repo_config,
diff --git a/src/buildtool/execution_api/common/api_bundle.hpp b/src/buildtool/execution_api/common/api_bundle.hpp
index 09e06901..2f393dba 100644
--- a/src/buildtool/execution_api/common/api_bundle.hpp
+++ b/src/buildtool/execution_api/common/api_bundle.hpp
@@ -21,29 +21,7 @@
#include "gsl/gsl"
#include "src/buildtool/common/remote/remote_common.hpp"
#include "src/buildtool/common/repository_config.hpp"
-#include "src/buildtool/execution_api/local/local_api.hpp"
-#include "src/buildtool/execution_api/remote/bazel/bazel_api.hpp"
-
-/// \brief Utility function to instantiate either a Local or Bazel Execution
-/// API.
-/// \param address if provided, a BazelApi is instantiated
-/// \param repo_config repository configuration to be used by GitApi calls
-/// \param instance_name only used in the construction of the BazelApi object
-[[nodiscard]] static inline auto CreateExecutionApi(
- std::optional<ServerAddress> const& address,
- std::optional<gsl::not_null<const RepositoryConfig*>> const& repo_config =
- std::nullopt,
- std::string const& instance_name = "remote-execution")
- -> gsl::not_null<IExecutionApi::Ptr> {
- if (address) {
- ExecutionConfiguration config;
- config.skip_cache_lookup = false;
-
- return std::make_unique<BazelApi>(
- instance_name, address->host, address->port, config);
- }
- return std::make_unique<LocalApi>(repo_config);
-}
+#include "src/buildtool/execution_api/common/execution_api.hpp"
/// \brief Utility structure for instantiation of local and remote apis at the
/// same time. If the remote api cannot be instantiated, it falls back to