lix-project/lix
20
57
Fork 31
Code Issues 347 Code Review (Gerrit) Projects 3 Releases Wiki Activity

libstore: extract eager transfers into new method

Browse source 
we'll need this shared code for kjified transfers that don't use
sources. it's a while out, but we can clean this up now already.

Change-Id: Ife8c160e6ab379761362d6c54aba05093deee99e
This commit is contained in:
eldritch horrors 2024-11-09 01:17:28 +01:00
parent 12156d3beb
commit fb85228755
1 changed files with 19 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
src/libstore/filetransfer.cc
View file

@ -720,10 +720,10 @@ struct curlFileTransfer : public FileTransfer
enqueueFileTransfer(uri, headers, std::move(data), false);
}
std::pair<FileTransferResult, box_ptr<Source>> enqueueFileTransfer(
std::optional<std::pair<FileTransferResult, box_ptr<Source>>> tryEagerTransfers(
const std::string & uri,
const Headers & headers,
std::optional<std::string> data,
const std::optional<std::string> & data,
bool noBody
)
{
@ -767,14 +767,14 @@ struct curlFileTransfer : public FileTransfer
);
}
if (S_ISDIR(st.st_mode)) {
return {std::move(metadata), make_box_ptr<StringSource>("")};
return {{std::move(metadata), make_box_ptr<StringSource>("")}};
}
struct OwningFdSource : FdSource
{
AutoCloseFD fd;
OwningFdSource(AutoCloseFD fd) : FdSource(fd.get()), fd(std::move(fd)) {}
};
return {std::move(metadata), make_box_ptr<OwningFdSource>(std::move(fd))};
return {{std::move(metadata), make_box_ptr<OwningFdSource>(std::move(fd))}};
}
}
@ -796,7 +796,7 @@ struct curlFileTransfer : public FileTransfer
FileTransferResult res;
if (!s3Res.data)
throw FileTransferError(NotFound, "S3 object '%s' does not exist", uri);
return {res, make_box_ptr<StringSource>(std::move(*s3Res.data))};
return {{res, make_box_ptr<StringSource>(std::move(*s3Res.data))}};
#else
throw nix::Error(
"cannot download '%s' because Lix is not built with S3 support", uri
@ -804,6 +804,20 @@ struct curlFileTransfer : public FileTransfer
#endif
}
return std::nullopt;
}
std::pair<FileTransferResult, box_ptr<Source>> enqueueFileTransfer(
const std::string & uri,
const Headers & headers,
std::optional<std::string> data,
bool noBody
)
{
if (auto eager = tryEagerTransfers(uri, headers, data, noBody)) {
return std::move(*eager);
}
struct State {
bool done = false, failed = false;
std::exception_ptr exc;