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

[PATCH 0/9] Resend of gitster/pb/bisect

From
Pranit Bauva <pranit.bauva@gmail.com>
Date
Jul 12, 2016, 22:35 UTC
Message-ID
<20160712223538.28508-1-pranit.bauva@gmail.com>
Hey Junio,

A small mistake got unnoticed by me which Lars recently pointed out. The naming convention is "git_path_<name_of_file>" and underscore instead of spaces.

Thanks!
The interdiff is:
diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c
index c2f3cee..88a1df8 100644
--- a/builtin/bisect--helper.c
+++ b/builtin/bisect--helper.c
@@ -7,7 +7,7 @@
 #include "argv-array.h"
 #include "run-command.h"
 
-static GIT_PATH_FUNC(git_path_bisect_write_terms, "BISECT_TERMS")
+static GIT_PATH_FUNC(git_path_bisect_terms, "BISECT_TERMS")
 static GIT_PATH_FUNC(git_path_bisect_expected_rev, "BISECT_EXPECTED_REV")
 static GIT_PATH_FUNC(git_path_bisect_ancestors_ok, "BISECT_ANCESTORS_OK")
 static GIT_PATH_FUNC(git_path_bisect_log, "BISECT_LOG")
@@ -100,7 +100,7 @@ static int write_terms(const char *bad, const char *good)
 	if (check_term_format(bad, "bad") || check_term_format(good, "good"))
 		return -1;
 
-	fp = fopen(git_path_bisect_write_terms(), "w");
+	fp = fopen(git_path_bisect_terms(), "w");
 	if (!fp)
 		return error_errno(_("could not open the file BISECT_TERMS"));
 
@@ -134,7 +134,7 @@ static int bisect_clean_state(void)
 	remove_path(git_path_bisect_log());
 	remove_path(git_path_bisect_names());
 	remove_path(git_path_bisect_run());
-	remove_path(git_path_bisect_write_terms());
+	remove_path(git_path_bisect_terms());
 	/* Cleanup head-name if it got left by an old version of git-bisect */
 	remove_path(git_path_head_name());
 	/*


Pranit Bauva (9):
  bisect--helper: use OPT_CMDMODE instead of OPT_BOOL
  bisect: rewrite `check_term_format` shell function in C
  bisect--helper: `write_terms` shell function in C
  bisect--helper: `bisect_clean_state` shell function in C
  t6030: explicitly test for bisection cleanup
  wrapper: move is_empty_file() and rename it as
    is_empty_or_missing_file()
  bisect--helper: `bisect_reset` shell function in C
  bisect--helper: `is_expected_rev` & `check_expected_revs` shell
    function in C
  bisect--helper: `bisect_write` shell function in C

 builtin/am.c                |  20 +--
 builtin/bisect--helper.c    | 310 +++++++++++++++++++++++++++++++++++++++++++-
 cache.h                     |   3 +
 git-bisect.sh               | 146 +++------------------
 t/t6030-bisect-porcelain.sh |  17 +++
 wrapper.c                   |  13 ++
 6 files changed, 355 insertions(+), 154 deletions(-)
-- 
2.9.0
Next: Pranit Bauva
Message 1 of 12 in “Resend of gitster/pb/bisect”
  1. 0/9 Resend of gitster/pb/bisectPranit Bauva, Jul 12, 2016
  2. 2/9 bisect: rewrite `check_term_format` shell function in CPranit Bauva, Jul 12, 2016
  3. 3/9 bisect--helper: `write_terms` shell function in CPranit Bauva, Jul 12, 2016
  4. 1/9 bisect--helper: use OPT_CMDMODE instead of OPT_BOOLPranit Bauva, Jul 12, 2016
  5. 6/9 wrapper: move is_empty_file() and rename it as is_empty_or_missing_file()Pranit Bauva, Jul 12, 2016
  6. 4/9 bisect--helper: `bisect_clean_state` shell function in CPranit Bauva, Jul 12, 2016
  7. 5/9 t6030: explicitly test for bisection cleanupPranit Bauva, Jul 12, 2016
  8. 9/9 bisect--helper: `bisect_write` shell function in CPranit Bauva, Jul 12, 2016
  9. 7/9 bisect--helper: `bisect_reset` shell function in CPranit Bauva, Jul 12, 2016
  10. 8/9 bisect--helper: `is_expected_rev` & `check_expected_revs` shell function in CPranit Bauva, Jul 12, 2016
  11. Christian CouderJul 13, 2016
  12. Pranit BauvaJul 20, 2016

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.