Merge pull request #733 from basvandijk/fix-build_finished-notifications

Separate the build IDs in the build_finished payload with tabs
This commit is contained in:
Bas van Dijk 2020-04-08 12:27:58 +02:00 committed by GitHub
commit d816503814
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

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