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

[PATCH 1/2] diffcore-pickaxe: make error messages more consistent

From
Ramkumar Ramachandra <artagnon@gmail.com>
Date
May 17, 2013, 12:23 UTC
Message-ID
<1368793403-4642-2-git-send-email-artagnon@gmail.com>
In-Reply-To
<1368793403-4642-1-git-send-email-artagnon@gmail.com>

Currently, diffcore-pickaxe reports two distinct errors for the same user error:

    $ git log --pickaxe-regex -S'\1'
    fatal: invalid pickaxe regex: Invalid back reference
    $ git log -G'\1' # --pickaxe-regex is implied
    fatal: invalid log-grep regex: Invalid back reference

Since the error has nothing to do with "log-grep", change the -G error message to match the -S error message.

Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
---
 diffcore-pickaxe.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/diffcore-pickaxe.c b/diffcore-pickaxe.c
index 63722f8..d69a7a2 100644
--- a/diffcore-pickaxe.c
+++ b/diffcore-pickaxe.c
@@ -122,7 +122,7 @@ static void diffcore_pickaxe_grep(struct diff_options *o)
 		char errbuf[1024];
 		regerror(err, &regex, errbuf, 1024);
 		regfree(&regex);
-		die("invalid log-grep regex: %s", errbuf);
+		die("invalid pickaxe regex: %s", errbuf);
 	}
 
 	pickaxe(&diff_queued_diff, o, &regex, NULL, diff_grep);
-- 
1.8.1.2.432.g070c57d
Previous: Ramkumar RamachandraNext: Junio C Hamano
Message 2 of 10 in “Improve diffcore-pickaxe documentation”
  1. 0/2 Improve diffcore-pickaxe documentationRamkumar Ramachandra, May 17, 2013
  2. 1/2 diffcore-pickaxe: make error messages more consistentRamkumar Ramachandra, May 17, 2013
  3. Junio C HamanoMay 18, 2013
  4. 2/2 diffcore-pickaxe doc: document -S and -G properlyRamkumar Ramachandra, May 17, 2013
  5. Junio C HamanoMay 18, 2013
  6. Junio C HamanoMay 19, 2013
  7. Ramkumar RamachandraMay 24, 2013
  8. Phil HordMay 24, 2013
  9. Ramkumar RamachandraMay 24, 2013
  10. Junio C HamanoMay 24, 2013

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.