diff options
author | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2024-01-29 18:09:56 +0100 |
---|---|---|
committer | Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com> | 2024-01-31 17:15:46 +0100 |
commit | a9b3d717eac4c4f3b87b2bb7fe9f0961f4bb6fb0 (patch) | |
tree | b5fa92fc826dc7579d42e34be573bd3ee3ceb4bd /src/other_tools/root_maps/distdir_git_map.cpp | |
parent | 66df7f4956c6fe1823eb43d31c3cbac7d8716125 (diff) | |
download | justbuild-a9b3d717eac4c4f3b87b2bb7fe9f0961f4bb6fb0.tar.gz |
just-mr: Serve endpoint always implies a remote-execution endpoint
As the serve service always has an associated remote-execution
endpoint, when just-mr gets passed only a serve endpoint address it
will now default to implying a remote-execution endpoint exists too
at that address. Additionally, now the implementation of the
--backup-to-remote option more clearly shows that it can only be
done in native mode.
Diffstat (limited to 'src/other_tools/root_maps/distdir_git_map.cpp')
-rw-r--r-- | src/other_tools/root_maps/distdir_git_map.cpp | 23 |
1 files changed, 0 insertions, 23 deletions
diff --git a/src/other_tools/root_maps/distdir_git_map.cpp b/src/other_tools/root_maps/distdir_git_map.cpp index 2ee0265f..c2ba2e20 100644 --- a/src/other_tools/root_maps/distdir_git_map.cpp +++ b/src/other_tools/root_maps/distdir_git_map.cpp @@ -254,19 +254,6 @@ auto CreateDistdirGitMap( /*fatal=*/true); return; } - // at this point we cannot continue without - // the remote api - if (not remote_api) { - (*logger)( - fmt::format( - "Missing remote-execution " - "endpoint needed to sync " - "workspace root {} with the " - "serve endpoint.", - distdir_tree_id), - /*fatal=*/true); - return; - } // the tree is known locally, so we upload // it to remote CAS for the serve endpoint // to retrieve it and set up the root @@ -401,16 +388,6 @@ auto CreateDistdirGitMap( /*fatal=*/true); return; } - // at this point we cannot continue without the remote api - if (not remote_api) { - (*logger)( - fmt::format("Missing remote-execution endpoint " - "needed to sync workspace root {} with " - "the serve endpoint.", - tree_id), - /*fatal=*/true); - return; - } // try to supply the serve endpoint with the tree via the // remote CAS if (remote_api.value()->IsAvailable({digest})) { |