cron: support multiple CHILD_TAGs for TERMINATE_CHILDREN

MaxKellermann · MaxKellermann · commit 7d6a968f857d · 2025-10-30T08:37:27.000+01:00
The `tag` field is really a list of strings separated by null bytes.
diff --git a/debian/changelog b/debian/changelog
@@ -1,6 +1,7 @@
 cm4all-workshop (7.2) unstable; urgency=low
 
   * workshop: disable plan UID check for "translate" plans
+  * cron: support multiple CHILD_TAGs for TERMINATE_CHILDREN
 
  --   
 
diff --git a/libcommon b/libcommon
@@ -1 +1 @@
-Subproject commit f474b7aecf529bc14a4f967e69b9e454cc445126
+Subproject commit e9e24e66d03905ead2b05163025782e6cb11bec2
diff --git a/src/cron/Workplace.cxx b/src/cron/Workplace.cxx
@@ -24,6 +24,7 @@
 #include "co/Task.hxx"
 #include "util/DeleteDisposer.hxx"
 #include "util/StringCompare.hxx"
+#include "util/StringList.hxx"
 #include "EmailService.hxx"
 #include "debug.h"
 
@@ -66,7 +67,7 @@ class CronWorkplace::Running final
 	}
 
 	bool IsTag(std::string_view _tag) const noexcept {
-		return tag == _tag;
+		return StringListContains(tag, '\0', _tag);
 	}
 
 	void Start(SocketAddress translation_socket,

Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,7 @@`
`24`	`24`	`#include "co/Task.hxx"`
`25`	`25`	`#include "util/DeleteDisposer.hxx"`
`26`	`26`	`#include "util/StringCompare.hxx"`
	`27`	`+#include "util/StringList.hxx"`
`27`	`28`	`#include "EmailService.hxx"`
`28`	`29`	`#include "debug.h"`
`29`	`30`
`@@ -66,7 +67,7 @@ class CronWorkplace::Running final`
`66`	`67`	`}`
`67`	`68`
`68`	`69`	`bool IsTag(std::string_view _tag) const noexcept {`
`69`		`- return tag == _tag;`
	`70`	`+ return StringListContains(tag, '\0', _tag);`
`70`	`71`	`}`
`71`	`72`
`72`	`73`	`void Start(SocketAddress translation_socket,`