git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] t0005: skip signal death exit code test on Windows

From
Johannes Sixt <j.sixt@viscovery.net>
Date
Jun 6, 2013, 06:34 UTC
Message-ID
<51B02D81.3000700@viscovery.net>
In-Reply-To
<20130605071206.GC14427@sigill.intra.peff.net>
From: Johannes Sixt <j6t@kdbg.org>

The test case depends on that test-sigchain can commit suicide by a call to raise(SIGTERM) in a way that run-command.c::wait_or_whine() can detect as death through a signal. There are no POSIX signals on Windows, and a sufficiently close emulation is not available in the Microsoft C runtime (and probably not even possible).

The particular deficiency is that when a signal is raise()d whose SIG_DFL action will cause process death (SIGTERM in this case), the implementation of raise() just calls exit(3).

We could check for exit code 3 in addition to 143, but that would miss the point of the test entirely. Hence, just skip it on Windows.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
---
 t/t0005-signals.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/t0005-signals.sh b/t/t0005-signals.sh
index ad9e604..981437b 100755
--- a/t/t0005-signals.sh
+++ b/t/t0005-signals.sh
@@ -20,7 +20,7 @@ test_expect_success 'sigchain works' '
 	test_cmp expect actual
 '
 
-test_expect_success 'signals are propagated using shell convention' '
+test_expect_success !MINGW 'signals are propagated using shell convention' '
 	# we use exec here to avoid any sub-shell interpretation
 	# of the exit code
 	git config alias.sigterm "!exec test-sigchain" &&
-- 
1.8.3.1489.g15123b5
Previous: Jeff KingNext: Jeff King
Message 80 of 104 in “What's cooking in git.git (Jun 2013, #02; Tue, 4)”
  1. Junio C HamanoJun 4, 2013
  2. [Administrivia] On ruby and contrib/Junio C Hamano, Jun 5, 2013
  3. David LangJun 5, 2013
  4. Felipe ContrerasJun 5, 2013
  5. Michael HaggertyJun 5, 2013
  6. Junio C HamanoJun 5, 2013
  7. Matthieu MoyJun 6, 2013
  8. Junio C HamanoJun 7, 2013
  9. Thomas Ferris NicolaisenJun 6, 2013
  10. Felipe ContrerasJun 5, 2013
  11. demerphqJun 6, 2013
  12. Felipe ContrerasJun 6, 2013
  13. Barry FishmanJun 6, 2013
  14. Felipe ContrerasJun 6, 2013
  15. Barry FishmanJun 6, 2013
  16. Felipe ContrerasJun 6, 2013
  17. Barry FishmanJun 6, 2013
  18. Felipe ContrerasJun 6, 2013
  19. Charles McGarveyJun 6, 2013
  20. Greg TroxelJun 6, 2013
  21. Felipe ContrerasJun 6, 2013
  22. David LangJun 6, 2013
  23. Felipe ContrerasJun 6, 2013
  24. Ramkumar RamachandraJun 6, 2013
  25. David LangJun 6, 2013
  26. Ramkumar RamachandraJun 7, 2013
  27. Junio C HamanoJun 7, 2013
  28. Ramkumar RamachandraJun 7, 2013
  29. Felipe ContrerasJun 7, 2013
  30. Ramkumar RamachandraJun 7, 2013
  31. Ramkumar RamachandraJun 7, 2013
  32. Johannes SchindelinJun 9, 2013
  33. Ramkumar RamachandraJun 9, 2013
  34. Junio C HamanoJun 9, 2013
  35. Felipe ContrerasJun 10, 2013
  36. Junio C HamanoJun 7, 2013
  37. Felipe ContrerasJun 7, 2013
  38. Duy NguyenJun 8, 2013
  39. Felipe ContrerasJun 8, 2013
  40. Duy NguyenJun 8, 2013
  41. Felipe ContrerasJun 8, 2013
  42. Felipe ContrerasJun 7, 2013
  43. Greg TroxelJun 6, 2013
  44. Felipe ContrerasJun 6, 2013
  45. Ramkumar RamachandraJun 6, 2013
  46. Dependencies and packaging (Re: [Administrivia] On ruby and contrib/)Jonathan Nieder, Jun 6, 2013
  47. Felipe ContrerasJun 7, 2013
  48. Johannes SchindelinJun 6, 2013
  49. Ramkumar RamachandraJun 6, 2013
  50. Johannes SchindelinJun 7, 2013
  51. Ramkumar RamachandraJun 7, 2013
  52. Matthieu MoyJun 7, 2013
  53. Ramkumar RamachandraJun 7, 2013
  54. Ramkumar RamachandraJun 7, 2013
  55. Matthieu MoyJun 7, 2013
  56. Ramkumar RamachandraJun 7, 2013
  57. Matthieu MoyJun 7, 2013
  58. Felipe ContrerasJun 7, 2013
  59. Jonathan NiederJun 7, 2013
  60. Matthew RuffaloJun 7, 2013
  61. Junio C HamanoJun 7, 2013
  62. Felipe ContrerasJun 7, 2013
  63. Ramkumar RamachandraJun 7, 2013
  64. Johannes SchindelinJun 9, 2013
  65. Duy NguyenJun 8, 2013
  66. Felipe ContrerasJun 8, 2013
  67. Duy NguyenJun 8, 2013
  68. Felipe ContrerasJun 8, 2013
  69. Duy NguyenJun 8, 2013
  70. Felipe ContrerasJun 8, 2013
  71. Jeff KingJun 8, 2013
  72. Felipe ContrerasJun 8, 2013
  73. Jeff KingJun 9, 2013
  74. Felipe ContrerasJun 9, 2013
  75. Jeff KingJun 9, 2013
  76. Felipe ContrerasJun 9, 2013
  77. Johannes SchindelinJun 9, 2013
  78. Johannes SixtJun 5, 2013
  79. Jeff KingJun 5, 2013
  80. t0005: skip signal death exit code test on WindowsJohannes Sixt, Jun 6, 2013
  81. Jeff KingJun 6, 2013
  82. Felipe ContrerasJun 6, 2013
  83. Jeff KingJun 6, 2013
  84. Felipe ContrerasJun 6, 2013
  85. Junio C HamanoJun 6, 2013
  86. Jeff KingJun 6, 2013
  87. Johannes SixtJun 7, 2013
  88. Erik Faye-LundJun 7, 2013
  89. Johannes SixtJun 7, 2013
  90. Erik Faye-LundJun 7, 2013
  91. Johannes SixtJun 7, 2013
  92. Erik Faye-LundJun 7, 2013
  93. Johannes SixtJun 7, 2013
  94. Erik Faye-LundJun 7, 2013
  95. mingw: make mingw_signal return the correct handlerJohannes Sixt, Jun 10, 2013
  96. Erik Faye-LundJun 10, 2013
  97. Junio C HamanoJun 10, 2013
  98. Jeff KingJun 9, 2013
  99. Junio C HamanoJun 9, 2013
  100. Johannes SixtJun 10, 2013
  101. Erik Faye-LundJun 10, 2013
  102. Felipe ContrerasJun 6, 2013
  103. Erik Faye-LundJun 7, 2013
  104. Erik Faye-LundJun 7, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.