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

Re: [RFC PATCH] Make :/ accept a regex rather than a fixed pattern

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 18, 2010, 17:03 UTC
Message-ID
<7vhbl0xoh2.fsf@alter.siamese.dyndns.org>
In-Reply-To
<alpine.LFD.2.00.1005131457350.3711@i5.linux-foundation.org>
Linus Torvalds <torvalds@linux-foundation.org> writes:
Show 29 quoted lines
> From: Linus Torvalds <torvalds@linux-foundation.org>
> Date: Fri, 23 Apr 2010 08:20:20 -0700
> Subject: [PATCH] Make :/ accept a regex rather than a fixed pattern
>
> This also makes it trigger anywhere in the commit message, rather than
> just at the beginning. Which tends to be a lot more useful.
>
> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
> ---
>
> I'm re-sending, because after having this in my tree for several weeks 
> now, I actually end up still using it. 
>
> I agree that it's still not a wonderful thing, and it's entirely possible 
> that we should strive to use just HEAD in :/ commit finding rather than 
> all refs we can find, but for at least the kernel, I don't care (since my 
> tree tends to have just one main branch anyway, apart from some random 
> testing stuff I keep around).
>
> But even if we want to limit it to HEAD, that would be a separate patch.
>
> I've personally used it for things like
>
> 	git show :/mqueue	# did I apply that 'mqueue' patch?
> 	git show :/akpm		# what was the last patch I got from Andrew?
>
> and while in all cases I could admittedly have done exactly the same thing 
> with something like "git log -p -1 --grep=mqueue" instead, I've used it as 
> a nice shortcut.

I already took this but I actually started hating it. As I have merge.summary on,

    git show ':/diff: add configuration option'

almost never hits the commit I am looking for---instead it ends up finding a merge that merges it.

I however suspect that I would hate it much less after retraining my fingers to anchor the pattern at the beginning of the line. But not yet...

Previous: Linus Torvalds
Message 2 of 2 in “Make :/ accept a regex rather than a fixed pattern”
  1. Make :/ accept a regex rather than a fixed patternLinus Torvalds, May 13, 2010
  2. Junio C HamanoJun 18, 2010

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.