Re: [PATCH 01/10] refs: add "for_each_bisect_ref" function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Thu, 26 Mar 2009, Michael J Gruber wrote:

> Christian Couder venit, vidit, dixit 26.03.2009 08:48:
> 
> > Le jeudi 26 mars 2009, Sverre Rabbelier a écrit :
> >
> >> A 10 patches series with no cover letter?
> > 
> > I am not a big fan of cover letters. Usually I prefer adding comments 
> > in the patches.
> 
> I'm sorry I have to say that, but your individual preferences don't 
> matter. Many of us would do things differently, each in their own way, 
> but people adjust to the list's preferences. It's a matter of attitude. 
> So, please...

Actually, a better way to ask for a cover letter would have been to 
convince Christian.  So I'll try that.

>From the patch series' titles (especially when they are cropped due to the 
text window being too small to fit the indented thread), it is not all 
that obvious what you want to achieve with those 10 patches.

>From recent discussions, I seem to remember that you wanted to have some 
cute way to mark commits as non-testable during a bisect, and I further 
seem to remember that Junio said that very method should be usable outside 
of bisect, too.

Unfortunately, that does not reveal to me, quickly, what is the current 
state of affairs, and what you changed since the last time.

In addition, I am very sorry that I cannot review your patches; day job is 
killing me right now.

Ciao,
Dscho

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux