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

Re: [PATCH v4] gitk: Allow commit editing

From
Jeff King <peff@peff.net>
Date
Sep 7, 2011, 20:10 UTC
Message-ID
<20110907201011.GF13364@sigill.intra.peff.net>
In-Reply-To
<87fwknaveh.fsf@steelpick.2x.cz>
On Sat, Aug 27, 2011 at 02:31:02PM +0200, Michal Sojka wrote:
Show 5 quoted lines
> Here is a new version with the micro-optimization.
> 
> Another minor change is that this patch now applies to gitk repo
> (http://git.kernel.org/pub/scm/gitk/gitk.git) instead of the main git
> repo.

Thanks. This version fixes most of my complaints and looks reasonable sane, as long as you accept the idea that starting an interactive rebase behind the scenes is sane. I'm still not 100% convinced. But then, I am not a big gitk user, either (I like visualizing with it, but I rarely want to do anything interactive). So I'll leave Paul and anyone else to argue or make a decision on that count.

-Peff
Previous: Michal SojkaNext: Paul Mackerras
Message 7 of 10 in “Re: [PATCH RFC] gitk: Allow commit editing”
  1. Michal SojkaAug 19, 2011
  2. gitk: Allow commit editingMichal Sojka, Aug 19, 2011
  3. Jeff KingAug 25, 2011
  4. Michal SojkaAug 25, 2011
  5. Jeff KingAug 25, 2011
  6. gitk: Allow commit editingMichal Sojka, Aug 27, 2011
  7. Jeff KingSep 7, 2011
  8. Paul MackerrasSep 8, 2011
  9. Michal SojkaSep 13, 2011
  10. Jeff KingAug 25, 2011

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.