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

[PATCHv2 1/2] git-bisect.txt: streamline run presentation

From
Michael J Gruber <git@drmicha.warpmail.net>
Date
Mar 15, 2011, 21:24 UTC
Message-ID
<57605e2f4533a3c656d24f3a7ec6264e0bd949fe.1300224056.git.git@drmicha.warpmail.net>
In-Reply-To
<7vy64hehbh.fsf@alter.siamese.dyndns.org>

Streamline the presentation of "bisect run" by removing one example which does not introduce new concepts.

Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net>
---
 Documentation/git-bisect.txt |   34 ++++++++--------------------------
 1 files changed, 8 insertions(+), 26 deletions(-)
diff --git a/Documentation/git-bisect.txt b/Documentation/git-bisect.txt
index c39d957..47e8b1e 100644
--- a/Documentation/git-bisect.txt
+++ b/Documentation/git-bisect.txt
@@ -274,53 +274,35 @@ $ git bisect start HEAD origin --    # HEAD is bad, origin is good
 $ git bisect run make test           # "make test" builds and tests
 ------------
 
-* Automatically bisect a broken test suite:
-+
-------------
-$ cat ~/test.sh
-#!/bin/sh
-make || exit 125                   # this skips broken builds
-make test                          # "make test" runs the test suite
-$ git bisect start v1.3 v1.1 --    # v1.3 is bad, v1.1 is good
-$ git bisect run ~/test.sh
-------------
-+
-Here we use a "test.sh" custom script. In this script, if "make"
-fails, we skip the current commit.
-+
-It is safer to use a custom script outside the repository to prevent
-interactions between the bisect, make and test processes and the
-script.
-+
-"make test" should "exit 0", if the test suite passes, and
-"exit 1" otherwise.
-
 * Automatically bisect a broken test case:
 +
 ------------
 $ cat ~/test.sh
 #!/bin/sh
 make || exit 125                     # this skips broken builds
-~/check_test_case.sh                 # does the test case passes ?
+~/check_test_case.sh                 # does the test case pass?
 $ git bisect start HEAD HEAD~10 --   # culprit is among the last 10
 $ git bisect run ~/test.sh
 ------------
 +
-Here "check_test_case.sh" should "exit 0" if the test case passes,
+Here we use a "test.sh" custom script. In this script, if "make"
+fails, we skip the current commit.
+"check_test_case.sh" should "exit 0" if the test case passes,
 and "exit 1" otherwise.
 +
-It is safer if both "test.sh" and "check_test_case.sh" scripts are
+It is safer if both "test.sh" and "check_test_case.sh" are
 outside the repository to prevent interactions between the bisect,
 make and test processes and the scripts.
 
-* Automatically bisect a broken test suite:
+* Automatically bisect a broken test case:
 +
 ------------
 $ git bisect start HEAD HEAD~10 --   # culprit is among the last 10
 $ git bisect run sh -c "make || exit 125; ~/check_test_case.sh"
 ------------
 +
-Does the same as the previous example, but on a single line.
+This shows that you can do without a run script if you write the test
+on a single line.
 
 Author
 ------
-- 
1.7.4.1.404.g62d316
Previous: Junio C HamanoNext: Michael J Gruber
Message 4 of 10 in “git bisect plus fixes (was: PATCH: Add --size-check=[error|warning])”
  1. Ralf WildenhuesMar 14, 2011
  2. git-bisect.txt: example for bisecting with hotfixMichael J Gruber, Mar 14, 2011
  3. Junio C HamanoMar 14, 2011
  4. 1/2 git-bisect.txt: streamline run presentationMichael J Gruber, Mar 15, 2011
  5. 2/2 git-bisect.txt: example for bisecting with hot-fixMichael J Gruber, Mar 15, 2011
  6. Document 'git bisect fix'.Ralf Wildenhues, Mar 14, 2011
  7. Yann DirsonMar 15, 2011
  8. Christian CouderMar 16, 2011
  9. Michael J GruberMar 16, 2011
  10. Junio C HamanoMar 16, 2011

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.