From 6e358189ad13131341654459de42301bc1ba162e Mon Sep 17 00:00:00 2001 From: Bas van Dijk Date: Wed, 8 Apr 2020 12:05:25 +0200 Subject: [PATCH] Separate the build IDs in the build_finished payload with tabs hydra-notify splits the payload on tabs so we shouldn't separate the IDs with spaces. --- src/hydra-queue-runner/hydra-queue-runner.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/hydra-queue-runner/hydra-queue-runner.cc b/src/hydra-queue-runner/hydra-queue-runner.cc index c80f8d3e..558005e7 100644 --- a/src/hydra-queue-runner/hydra-queue-runner.cc +++ b/src/hydra-queue-runner/hydra-queue-runner.cc @@ -468,9 +468,9 @@ void State::notifyBuildStarted(pqxx::work & txn, BuildID buildId) void State::notifyBuildFinished(pqxx::work & txn, BuildID buildId, const std::vector & 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)); }