[U-Boot] [OT] CC:s in commits (was: [PATCH 1/2] socfpga: Adding Scan Manager driver)

Dinh Nguyen dinguyen at altera.com
Mon Jan 13 22:14:58 CET 2014


Hi Wolfgang,

On Mon, 2014-01-13 at 22:08 +0100, ZY - wd wrote:
> Dear Albert,
> 
> In message <20140113173924.684ce548 at lilith> you wrote:
> > 
> > > Actually these Cc will be helpful when using git send-email. It will
> > > auto cc to these mailing list when the patch is send out for review
> > > (instead entering the names manually every time). Thanks
> > 
> > That is useful to the sender, but useless to anyone else.
> 
> This is not correct.  It is also an indicatioin who was explicitly
> addressed when the patch was submitted, so you can later still see
> from the commit message who was invited to commend and refrained from
> doing so.  This may help a lot in discussions like "but I never had a
> chance to see this patch".
> 
> Please keep these.  The are really useful.

I'm glad you weighed in on this. I have done this CC: field many times
in patches sent to kernel.org, and have not received any feedbacks to
remove it. Some patches can include quite a few entries. So I was kinda
surprised to see a comment about it here on the u-boot list.

Dinh
> 
> Best regards,
> 
> Wolfgang Denk
> 





More information about the U-Boot mailing list