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 68A95982 for ; Mon, 13 Jul 2015 14:33:37 +0000 (UTC) Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by smtp1.linuxfoundation.org (Postfix) with ESMTPS id C457A7C for ; Mon, 13 Jul 2015 14:33:36 +0000 (UTC) Date: Mon, 13 Jul 2015 16:33:31 +0200 From: Jan Kara To: Dan Carpenter Message-ID: <20150713143331.GA21079@quack.suse.cz> References: <1481488.5WJFbB0Dlm@vostro.rjw.lan> <1436341028.2136.14.camel@HansenPartnership.com> <20150708080032.CE89E4306F@saturn.retrosnub.co.uk> <20150708145315.29030a75@gandalf.local.home> <20150710181409.GA30145@treble.redhat.com> <1436576450.27924.59.camel@stgolabs.net> <20150712034824.GA4236@treble.hsd1.ky.comcast.net> <20150712052317.GB15346@x> <20150712122823.GA4366@treble.redhat.com> <20150713140113.GB4341@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150713140113.GB4341@mwanda> Cc: James Bottomley , jic23@jic23.retrosnub.co.uk, Jason Cooper , ksummit-discuss@lists.linuxfoundation.org Subject: Re: [Ksummit-discuss] [CORE TOPIC] Recruitment (Reviewers, Testers, Maintainers, Hobbyists) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon 13-07-15 17:20:47, Dan Carpenter wrote: > On Sun, Jul 12, 2015 at 07:28:23AM -0500, Josh Poimboeuf wrote: > > I agree that the Reviewed-by tag doesn't apply here. It communicates > > patch approval, rather than review helpfulness. That's why I proposed a > > new tag from the patch author and/or maintainer. > > I've lobbied for: > > With-Fixes-from: xxx > > It would only be given for actual bug fixes and not for complaining > about the subject line, white space or spelling mistakes. This seems like overengineering to me. I usually give a credit in the changelog if I find someone's contribution big enough. Also a persistent reviewer will eventually get his credit via Reviewed-by :). So I don't think yet another tag is really necessary. Honza -- Jan Kara SUSE Labs, CR