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

[PATCH] cvsimport: have default merge regex also match beginning of commit message

From
Gerrit Pape <pape@smarden.org>
Date
Feb 1, 2008, 13:15 UTC
Message-ID
<20080201131514.1979.qmail@4264665dc6742b.315fe32.mid.smarden.org>

The default value of @mergerx uses \W, which matches a non-word character; this means that commit messages like "Merging FOO" are not matched by default; using \b, which matches a word boundary, instead of \W fixes that.

This change was suggested by Fr�d�ric Bri�re through
 http://bugs.debian.org/463468
Signed-off-by: Gerrit Pape <pape@smarden.org>
---
 git-cvsimport.perl |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/git-cvsimport.perl b/git-cvsimport.perl
index 5694978..9516242 100755
--- a/git-cvsimport.perl
+++ b/git-cvsimport.perl
@@ -164,7 +164,7 @@ if ($#ARGV == 0) {
 
 our @mergerx = ();
 if ($opt_m) {
-	@mergerx = ( qr/\W(?:from|of|merge|merging|merged) (\w+)/i );
+	@mergerx = ( qr/\b(?:from|of|merge|merging|merged) (\w+)/i );
 }
 if ($opt_M) {
 	push (@mergerx, qr/$opt_M/);
-- 
1.5.3.8
Next: Gerrit Pape
Message 1 of 2 in “cvsimport: have default merge regex also match beginning of commit message”
  1. cvsimport: have default merge regex also match beginning of commit messageGerrit Pape, Feb 1, 2008
  2. cvsimport: have default merge regex also match beginning of commit messageGerrit Pape, Feb 7, 2008

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.