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

[PATCH v2 1/2] tests: let --stress-limit=<N> imply --stress

From
Johannes Schindelin via GitGitGadget <gitgitgadget@gmail.com>
Date
Mar 3, 2019, 14:44 UTC
Message-ID
<fbe773c22a4ccea3e4eca8c1bd0206941d23f590.1551624293.git.gitgitgadget@gmail.com>
In-Reply-To
<pull.155.v2.git.gitgitgadget@gmail.com>
From: Johannes Schindelin <johannes.schindelin@gmx.de>

It does not make much sense that running a test with --stress-limit=<N> seemingly ignores that option because it does not stress test at all.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
 t/README      | 2 +-
 t/test-lib.sh | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/t/README b/t/README
index 3aed321248..b61bc930c4 100644
--- a/t/README
+++ b/t/README
@@ -205,7 +205,7 @@ appropriately before running "make".
 --stress-limit=<N>::
 	When combined with --stress run the test script repeatedly
 	this many times in each of the parallel jobs or until one of
-	them fails, whichever comes first.
+	them fails, whichever comes first. Implies `--stress`.
 
 You can also set the GIT_TEST_INSTALLED environment variable to
 the bindir of an existing git installation to test that installation.
diff --git a/t/test-lib.sh b/t/test-lib.sh
index 4e7cb52b57..ab7f27ec6a 100644
--- a/t/test-lib.sh
+++ b/t/test-lib.sh
@@ -153,6 +153,7 @@ do
 		esac
 		;;
 	--stress-limit=*)
+		stress=t;
 		stress_limit=${opt#--*=}
 		case "$stress_limit" in
 		*[!0-9]*|0*|"")
-- 
gitgitgadget
Previous: Johannes Schindelin via GitGitGadgetNext: Johannes Schindelin via GitGitGadget
Message 10 of 15 in “tests: some touchups related to the --stress feature”
  1. 0/2 tests: some touchups related to the --stress featureJohannes Schindelin via GitGitGadget, Mar 2, 2019
  2. 2/2 tests: introduce --stress-jobs=<N>Johannes Schindelin via GitGitGadget, Mar 2, 2019
  3. Eric SunshineMar 3, 2019
  4. Junio C HamanoMar 3, 2019
  5. SZEDER GáborMar 3, 2019
  6. Johannes SchindelinMar 3, 2019
  7. 1/2 tests: let --stress-limit=<N> imply --stressJohannes Schindelin via GitGitGadget, Mar 2, 2019
  8. Junio C HamanoMar 3, 2019
  9. 0/2 tests: some touchups related to the --stress featureJohannes Schindelin via GitGitGadget, Mar 3, 2019
  10. 1/2 tests: let --stress-limit=<N> imply --stressJohannes Schindelin via GitGitGadget, Mar 3, 2019
  11. 2/2 tests: introduce --stress-jobs=<N>Johannes Schindelin via GitGitGadget, Mar 3, 2019
  12. Eric SunshineMar 3, 2019
  13. Junio C HamanoMar 4, 2019
  14. Eric SunshineMar 4, 2019
  15. Jeff KingMar 3, 2019

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.