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

[PATCH 1/1] rebase: default to using the builtin rebase

From
Pratik Karki <predatoramigo@gmail.com>
Date
Aug 8, 2018, 15:53 UTC
Message-ID
<20180808155302.23814-2-predatoramigo@gmail.com>
In-Reply-To
<20180808155302.23814-1-predatoramigo@gmail.com>

Now that the builtin rebase is feature-complete, we should use it by default. Let's keep the legacy scripted version around for the time being; Once the builtin rebase is well-tested enough, we can remove `git-legacy-rebase.sh`.

Signed-off-by: Pratik Karki <predatoramigo@gmail.com>
---
 builtin/rebase.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/builtin/rebase.c b/builtin/rebase.c
index 4e69458161..c8d632b6f4 100644
--- a/builtin/rebase.c
+++ b/builtin/rebase.c
@@ -54,7 +54,7 @@ static int use_builtin_rebase(void)
 	cp.git_cmd = 1;
 	if (capture_command(&cp, &out, 6)) {
 		strbuf_release(&out);
-		return 0;
+		return 1;
 	}
 
 	strbuf_trim(&out);
-- 
2.18.0
Previous: Pratik Karki
Message 2 of 2 in “default to builtin rebase”
  1. Pratik KarkiAug 8, 2018
  2. 1/1 rebase: default to using the builtin rebasePratik Karki, Aug 8, 2018

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.