[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