From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp1.linuxfoundation.org (smtp1.linux-foundation.org [172.17.192.35]) by mail.linuxfoundation.org (Postfix) with ESMTPS id 2654BDB2 for ; Fri, 23 Aug 2019 19:23:22 +0000 (UTC) Received: from Galois.linutronix.de (Galois.linutronix.de [193.142.43.55]) by smtp1.linuxfoundation.org (Postfix) with ESMTPS id AD35C7FB for ; Fri, 23 Aug 2019 19:23:21 +0000 (UTC) Date: Fri, 23 Aug 2019 21:23:18 +0200 (CEST) From: Thomas Gleixner To: Joel Fernandes In-Reply-To: Message-ID: References: <20190823013619.GA8130@mit.edu> <20190823151843.GH8130@mit.edu> <20190823161947.GA112509@dtor-ws> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Cc: Barret Rhoden , ksummit , Greg Kroah-Hartman , Jonathan Nieder , Tomasz Figa , Han-Wen Nienhuys , Theodore Tso , David Rientjes , Dmitry Torokhov , Dmitry Vyukov Subject: Re: [Ksummit-discuss] Allowing something Change-Id (or something like it) in kernel commits List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, 23 Aug 2019, Joel Fernandes via Ksummit-discuss wrote: > On Fri, Aug 23, 2019 at 3:08 PM Joel Fernandes wrote: > > Another thing that could be done is, have the git hook insert the > > And, here I meant have git-send-email or the patch sending program > insert the unique ID into the discardable part. > > In my flow, I have scripts that remove metadata from my commit message > and insert them into the discardable part of the patch already which I > use for "additional notes". I am not sure what it would take to have > people use such scripts... Well, if some organization wants those IDs or whatever then they will come up with proper tooling which enforces this. We won't ever be able to enforce it on every submitter on the planet. For maintainers we can make the Message-ID link mandatory if the community agrees on it. git has some form of support for that already as I learned in some other thread. And adding it to homebrewn tooling is not rocket science either. Thanks, tglx