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

Re: [PATCH 0/7] guilt patches, including git 1.8 support

From
JSJosef 'Jeff' Sipek <jeffpc@josefsipek.net>
Date
Jan 20, 2013, 21:24 UTC
Message-ID
<20130120212425.GA101005@meili.valhalla.31bits.net>
In-Reply-To
<50FC5F6B.6070103@opera.com>
On Sun, Jan 20, 2013 at 10:19:39PM +0100, Per Cederqvist wrote:
Show 25 quoted lines
> On 01/16/13 03:26, Jonathan Nieder wrote:
> >Hi Jeff and other guilty parties,
> >
> >I collected all the guilt patches I could find on-list and added one
> >of my own.  Completely untested, except for running the regression
> >tests.  These are also available via git protocol from
> >
> >   git://repo.or.cz/guilt/mob.git mob
> >
> >Thoughts?
> 
> Regarding the "Change git branch when patches are applied" feature,
> it has saved me a number of times since I implemented it. I've been
> using it extensively, and have not had any problems with it.
> 
> Nevertheless, for backwards compatibility with the currently
> released guilt version I think it should be conditional, and
> disabled per default for one full release cycle.  I've implemented
> this. My code is available from git://repo.or.cz/guilt/ceder.git
> 
> The following commits are not included in your list:
> 
> a194375 Added guilt.reusebranch configuration option.
> 1dd97bd Minor testsuite fix.
> 640caee The tests should not fail if guilt.diffstat is set.

At $WORK, we started doing thorough reviews before committing. I'm kinda sold on this feature. I agree with the off-by-default config flag.

> I don't want Guilt to fork. Jeff, it would be really great if
> you could set up a public repo again!

Yep. I'm trying to reclaim control of the repo.or.cz guilt repo... I no longer have access to jsipek@cs.sunysb.edu and I don't remember the password.

Jeff.
-- 
NT is to UNIX what a doughnut is to a particle accelerator.
Previous: Per CederqvistNext: Josef 'Jeff' Sipek
Message 12 of 13 in “guilt patches, including git 1.8 support”
  1. 0/7 guilt patches, including git 1.8 supportJonathan Nieder, Jan 16, 2013
  2. [GUILT] [PATCH 1/7] get rid of "cat: write error: Broken pipe" error messageJonathan Nieder, Jan 16, 2013
  3. [GUILT] [PATCH 2/7] The tests should not fail if log.date or log.decorate are set.Jonathan Nieder, Jan 16, 2013
  4. [GUILT] [PATCH 3/7] Testsuite: get rid of "Broken pipe" errors from yes.Jonathan Nieder, Jan 16, 2013
  5. [GUILT] [PATCH 4/7] Handle empty patches and patches with only a header.Jonathan Nieder, Jan 16, 2013
  6. [GUILT] [PATCH 5/7] Fix fatal "guilt graph" error in sha1sum invocation.Jonathan Nieder, Jan 16, 2013
  7. [GUILT] [PATCH 6/7] Change git branch when patches are applied.Jonathan Nieder, Jan 16, 2013
  8. [GUILT] [PATCH 7/7] Drop unneeded git version check.Jonathan Nieder, Jan 16, 2013
  9. Theodore Ts'oJan 16, 2013
  10. Josef 'Jeff' SipekJan 16, 2013
  11. Per CederqvistJan 20, 2013
  12. Josef 'Jeff' SipekJan 20, 2013
  13. Josef 'Jeff' SipekJan 27, 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.