From owner-svn-ports-head@freebsd.org  Tue Jan 26 15:39:32 2016
Return-Path: <owner-svn-ports-head@freebsd.org>
Delivered-To: svn-ports-head@mailman.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by mailman.ysv.freebsd.org (Postfix) with ESMTP id 68751A6EBC2;
 Tue, 26 Jan 2016 15:39:32 +0000 (UTC)
 (envelope-from danfe@freebsd.org)
Received: from freefall.freebsd.org (freefall.freebsd.org
 [IPv6:2001:1900:2254:206c::16:87])
 by mx1.freebsd.org (Postfix) with ESMTP id 5E5181861;
 Tue, 26 Jan 2016 15:39:32 +0000 (UTC)
 (envelope-from danfe@freebsd.org)
Received: by freefall.freebsd.org (Postfix, from userid 1033)
 id 5D3271F1A; Tue, 26 Jan 2016 15:39:32 +0000 (UTC)
Date: Tue, 26 Jan 2016 15:39:32 +0000
From: Alexey Dokuchaev <danfe@FreeBSD.org>
To: Mathieu Arnold <mat@FreeBSD.org>
Cc: Kurt Jaeger <pi@FreeBSD.org>, ports-committers@freebsd.org,
 svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject: Re: svn commit: r407237 - head/mail/imaputils
Message-ID: <20160126153932.GC46193@FreeBSD.org>
References: <201601251910.u0PJAdeU003331@repo.freebsd.org>
 <20160126141038.GA46193@FreeBSD.org>
 <4C5F6D463ACD91B52D4AC5AF@ogg.in.absolight.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4C5F6D463ACD91B52D4AC5AF@ogg.in.absolight.net>
User-Agent: Mutt/1.5.24 (2015-08-30)
X-BeenThere: svn-ports-head@freebsd.org
X-Mailman-Version: 2.1.20
Precedence: list
List-Id: SVN commit messages for the ports tree for head
 <svn-ports-head.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/svn-ports-head>, 
 <mailto:svn-ports-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-ports-head/>
List-Post: <mailto:svn-ports-head@freebsd.org>
List-Help: <mailto:svn-ports-head-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/svn-ports-head>,
 <mailto:svn-ports-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 26 Jan 2016 15:39:32 -0000

On Tue, Jan 26, 2016 at 04:09:57PM +0100, Mathieu Arnold wrote:
> +--On 26 janvier 2016 14:10:38 +0000 Alexey Dokuchaev <danfe@FreeBSD.org>
> wrote:
> | On Mon, Jan 25, 2016 at 07:10:39PM +0000, Kurt Jaeger wrote:
> |> New Revision: 407237
> |> URL: https://svnweb.freebsd.org/changeset/ports/407237
> |> 
> |> Log:
> |>   mail/imaputils: remove redundant GH_PROJECT entry
> |>   
> |>   PR:		206540
> |>   Submitted by:	mat
> | 
> | This commit has nothing to do with that PR, and that PR was not submitted
> | by mat@.  Please be more careful with attribution, it confuses people when
> | done wrong.
> 
> Well, the "submitted by" is not related to who submitted the PR, it is
> related to who submitted the change.  For example, one person can create a
> PR without a patch and the committer fixes it, then the "submitted by" is
> empty, or someone else adds the patch to the PR, then the "submitted by"
> contains the person that sent the patch.

Right, there are few possible combinations like that; usually I try to give
attribution in a way that makes the most sense if read naturally (top to
bottom).  For example, for the actuall patch submitted by foo, with a
related PR (submitted by someone else) it's probably better to put
"Submitted by" line first, and PR line after.  Or clarify whose contribution
is whose by putting it next to name/PR in parentheses or something.

./danfe