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

[PATCH v4 06/12] bisect--helper: return error codes from `cmd_bisect__helper()`

From
Miriam Rubio <mirucam@gmail.com>
Date
Feb 17, 2020, 08:40 UTC
Message-ID
<20200217084039.78215-7-mirucam@gmail.com>
In-Reply-To
<20200217084039.78215-1-mirucam@gmail.com>
From: Pranit Bauva <pranit.bauva@gmail.com>

Since we want to get rid of git-bisect.sh, it would be necessary to convert bisect.c exit() calls to return statements so that errors can be reported. Let's prepare for that by making it possible to return different error codes than just 0 or 1.

Different error codes might enable a bisecting script calling the bisect command that uses this function to do different things depending on the exit status of the bisect command.

Mentored-by: Christian Couder <chriscool@tuxfamily.org>
Mentored-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Tanushree Tumane <tanushreetumane@gmail.com>
Signed-off-by: Miriam Rubio <mirucam@gmail.com>
---
 builtin/bisect--helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c
index e21d3d1a4c..e6bd4d6645 100644
--- a/builtin/bisect--helper.c
+++ b/builtin/bisect--helper.c
@@ -713,5 +713,5 @@ int cmd_bisect__helper(int argc, const char **argv, const char *prefix)
 		return error("BUG: unknown subcommand '%d'", cmdmode);
 	}
 	free_terms(&terms);
-	return !!res;
+	return abs(res);
 }
-- 
2.25.0
Previous: Miriam RubioNext: Miriam Rubio
Message 7 of 14 in “Finish converting git bisect to C part 1”
  1. Miriam RubioFeb 17, 2020
  2. 01/12 bisect--helper: convert `vocab_*` char pointers to char arraysMiriam Rubio, Feb 17, 2020
  3. 03/12 bisect: use the standard 'if (!var)' way to check for 0Miriam Rubio, Feb 17, 2020
  4. 02/12 bisect--helper: change `retval` to `res`Miriam Rubio, Feb 17, 2020
  5. 05/12 bisect: add enum to represent bisect returning codesMiriam Rubio, Feb 17, 2020
  6. 04/12 bisect--helper: introduce new `decide_next()` functionMiriam Rubio, Feb 17, 2020
  7. 06/12 bisect--helper: return error codes from `cmd_bisect__helper()`Miriam Rubio, Feb 17, 2020
  8. 08/12 bisect: libify `bisect_checkout`Miriam Rubio, Feb 17, 2020
  9. 07/12 bisect: libify `exit_if_skipped_commits` to `error_if_skipped*` and its dependentsMiriam Rubio, Feb 17, 2020
  10. 10/12 bisect: libify `check_good_are_ancestors_of_bad` and its dependentsMiriam Rubio, Feb 17, 2020
  11. 09/12 bisect: libify `check_merge_bases` and its dependentsMiriam Rubio, Feb 17, 2020
  12. 11/12 bisect: libify `handle_bad_merge_base` and its dependentsMiriam Rubio, Feb 17, 2020
  13. 12/12 bisect: libify `bisect_next_all`Miriam Rubio, Feb 17, 2020
  14. Junio C HamanoFeb 19, 2020

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.