about summary refs log tree commit diff
path: root/Test
diff options
context:
space:
mode:
authorChris Down <chris@chrisdown.name>2020-02-18 13:54:37 -0500
committerDaniel Shahaf <danielsh@apache.org>2020-02-18 19:35:18 +0000
commit69c247de2ff3ca41be93cd90af539926418d023d (patch)
treeda163ec7302eceaf937c7b7fec4328abe572b6ac /Test
parent5c55b3fb50bbfe602fcfa55fa6258e398ecc6b20 (diff)
downloadzsh-69c247de2ff3ca41be93cd90af539926418d023d.tar.gz
zsh-69c247de2ff3ca41be93cd90af539926418d023d.tar.xz
zsh-69c247de2ff3ca41be93cd90af539926418d023d.zip
45463: test: kill: Document why we use SIGURG
See discussion in workers/45460.
Diffstat (limited to 'Test')
-rw-r--r--Test/B11kill.ztst9
1 files changed, 9 insertions, 0 deletions
diff --git a/Test/B11kill.ztst b/Test/B11kill.ztst
index 40dd99cd3..dc6bf9b89 100644
--- a/Test/B11kill.ztst
+++ b/Test/B11kill.ztst
@@ -60,6 +60,15 @@
 ?(eval):kill:2: not enough arguments
 
 # Regression tests: `kill ''` should not result in `kill 0`.
+#
+# We use SIGURG where an explicit sigspec can be provided as:
+#
+# 1. By default it's non-terminal, so even if we regress, we won't kill the
+#    test runner and other processes in the process group since we'll stop
+#    running this test before we get to the plain kill (and thus SIGTERM)
+#    cases;
+# 2. It's also unlikely to be sent for any other reason during the process
+#    lifetime, so the test shouldn't be flaky.
 
   (
     trap 'exit 11' URG