From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Fri, 29 Jan 2016 06:14:49 +0100 Subject: [U-Boot] Include patchwork patch ID in commit message? In-Reply-To: References: <20160127222209.GS426@bill-the-cat> <20160127234521.GW426@bill-the-cat> <56A9B7BA.8040001@denx.de> Message-ID: <56AAF549.4020705@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Joe, Am 28.01.2016 um 16:15 schrieb Joe Hershberger: > Hi Heiko, > > On Thu, Jan 28, 2016 at 12:39 AM, Heiko Schocher wrote: >> Hello Tom, >> >> Am 28.01.2016 um 00:45 schrieb Tom Rini: >>> >>> On Wed, Jan 27, 2016 at 05:15:17PM -0600, Joe Hershberger wrote: >>>> >>>> On Wed, Jan 27, 2016 at 4:22 PM, Tom Rini wrote: >>>>> >>>>> On Wed, Jan 27, 2016 at 03:08:09PM -0600, Joe Hershberger wrote: >>>>>> >>>>>> Hi Tom, >>>>>> >>>>>> I'm playing with the idea of including the patchwork patch ID in the >>>>>> commit message of each commit that I apply to provide better >>>>>> cross-reference ability. >>>>>> >>>>>> * Access to comments on patches >>>>>> * Clarity on exactly which version of a patch was applied >>>>>> * No need to search by patch subject >>>>>> >>>>>> Here is an example in a working branch: >>>>>> >>>>>> >>>>>> http://git.denx.de/?p=u-boot/u-boot-net.git;a=commit;h=48f9a0c786d0a3cbfdf45846567deaebe27a334a >>>>> >>>>> >>>>> I'd prfer Patchwork or Patchwork-ID or something not just Patch. >>>> >>>> >>>> Would it be more or less compelling if it had a format similar this? >>>> >>>> Patchwork: https://patchwork.ozlabs.org/patch/571773/ >>> >>> >>> Yes. >> >> >> Sorry, for dummy question ... what should I see in the above link? > > The link is the think to see, not what it points to. The idea is that > instead of just the patch number, include the patch number in a full > URL for even easier access to the patch. Uh, ah, got it ... I think the patchworks patchnumber would be enough. I like Bins proposal, that patchwork add it automatically ... bye, Heiko -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany