Re: [Ksummit-discuss] Allowing something Change-Id (or something like it) in kernel commits

Laurent Pinchart
2019-08-23T19:38:42Z

Hi Thomas,

On Fri, Aug 23, 2019 at 09:17:04PM +0200, Thomas Gleixner wrote:
> On Fri, 23 Aug 2019, Dmitry Torokhov wrote:
> > On Fri, Aug 23, 2019 at 12:35:03PM -0400, Joel Fernandes wrote:
> >> On Fri, Aug 23, 2019 at 12:19 PM Dmitry Torokhov wrote:
> >>> On Fri, Aug 23, 2019 at 05:48:55PM +0200, Thomas Gleixner wrote:
> >>>>
> >>>> Yes, it's work for the submitter, but it's always work if the submitter
> >>>> wants to have a proper trace.
> >>>
> >>> Here is where I disagree with you. As a patch submitter, I frankly could
> >>> not care less about proper trace, history, etc. I might be putting cover
> >>
> >> But that is exactly what the problem statement is. Doug does care
> >> about tracing/history and wants that to be more robust etc.
> >
> > Doug here is not a submitter ;)
>
> Well, if he wants the changeids there then submitters need to insert them,
> right? So it's work no matter what unless it can be automated with tooling.
>
> Guess what, how I inject the Link to the coverletter of the previous
> version of a patch series? Definitely not manualy.

Would you be able to share your method for automating this ? I'm sure
many kernel developers could benefit from such automation (both those
who insert links manually now, and those who don't insert links at all
because doing it manually is too tedious).

> So yes, if you want proper traceability then all involved people have to do
> something. If it can be done with tooling fully automated, fine. If not,
> it's work whatever method you chose.
>
> We cannot enforce the changeid thing in the community, but Google can
> inforce it internally. So we give them a way to be traceable w/o plastering
> the kernel logs with potentially useless information.
>
> That said, I'm fine with a Link as well, as long as it is public
> accessible. If not, then the coverletter/discard area is the place to use.

--
Regards,

Laurent Pinchart