vriska/hydra
0
0
Fork 0
forked from lix-project/hydra
Code Pull requests Activity

Separate the build IDs in the build_finished payload with tabs

Browse source 
hydra-notify splits the payload on tabs so we shouldn't separate the
IDs with spaces.
This commit is contained in:
Bas van Dijk 2020-04-08 12:05:25 +02:00
parent 4cabb37ebd
commit 6e358189ad
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/hydra-queue-runner/hydra-queue-runner.cc
View file

@ -468,9 +468,9 @@ void State::notifyBuildStarted(pqxx::work & txn, BuildID buildId)
void State::notifyBuildFinished(pqxx::work & txn, BuildID buildId,
const std::vector<BuildID> & dependentIds)
{
auto payload = fmt("%d ", buildId);
auto payload = fmt("%d", buildId);
for (auto & d : dependentIds)
payload += fmt("%d ", d);
payload += fmt("\t%d", d);
// FIXME: apparently parameterized() doesn't support NOTIFY.
txn.exec(fmt("notify build_finished, '%s'", payload));
}