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

Re: [PATCH 1/2] Update the bash prompt from 'applied' instead of the obsolete 'current'

From
Karl Hasselström <kha@treskal.com>
Date
May 21, 2007, 07:48 UTC
Message-ID
<20070521074838.GA32510@diana.vm.bytemark.co.uk>
In-Reply-To
<20070520211103.17468.21373.stgit@lathund.dewire.com>
On 2007-05-20 23:22:00 +0200, Robin Rosenberg wrote:
Show 13 quoted lines
> söndag 20 maj 2007 skrev Yann Dirson:
>
> > Also I'm not sure it is a good way to look at "applied" file,
> > since Karl IIRC has plans to change this. Better call "stg top"
> > and not touch that again :)
>
> Calling stg is too slow to be be used here. I that command in my
> first draft for this function and people complained (see the thread
> named "Bash snippet to show branch and patch in bash prompt"). It
> takes ~ 0.15s on here which is very noticable, barely below my pain
> threshold.
>
> We'll update the prompt when and if Karl breaks this.

Yes, I can confirm that I'm hard at work breaking this. :-) I'm trying out a way to get around the performance bug Catalin found, but I didn't have time to finish it yesterday.

If that work is included, you could simply find the top patch by doing git-show-ref and figuring out which patch has the same sha1 as HEAD.

But it sucks that stg starts so slowly. It has gotten better, I believe (I think Catalin did some work here?), but 150 ms doesn't really qualify as "instantaneous".

-- 
Karl Hasselström, kha@treskal.com
      www.treskal.com/kalle
Previous: Robin RosenbergNext: Catalin Marinas
Message 5 of 15 in “Bash prompt updates”
  1. 0/2 Bash prompt updatesRobin Rosenberg, May 20, 2007
  2. 1/2 Update the bash prompt from 'applied' instead of the obsolete 'current'Robin Rosenberg, May 20, 2007
  3. Yann DirsonMay 20, 2007
  4. 1/2 Update the bash prompt from 'applied' instead of the obsolete 'current'Robin Rosenberg, May 20, 2007
  5. Karl HasselströmMay 21, 2007
  6. Catalin MarinasMay 21, 2007
  7. Karl HasselströmMay 21, 2007
  8. Karl HasselströmMay 21, 2007
  9. Catalin MarinasMay 21, 2007
  10. Karl HasselströmMay 21, 2007
  11. Catalin MarinasMay 22, 2007
  12. Karl HasselströmMay 22, 2007
  13. Yann DirsonMay 21, 2007
  14. 2/2 Don't use / as separatar since it is common i branch namesRobin Rosenberg, May 20, 2007
  15. 2/2 Don't use / as separatar since it is common i branch namesRobin Rosenberg, May 20, 2007

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.