Date: Thu, 22 Apr 2021 10:43:46 +0000 From: Alexey Dokuchaev <danfe@freebsd.org> To: Mathieu Arnold <mat@freebsd.org> Cc: Michael Gmelin <grembo@freebsd.org>, ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: 5b7c17856512 - main - git hooks: Rework authorship instructions. Message-ID: <YIFTYpPMe43y6tU9@FreeBSD.org> In-Reply-To: <20210422101616.c3i7dn6gfm6hhner@aching.in.mat.cc> References: <YIFK/tIpcfqWTWVh@FreeBSD.org> <FB32E75F-9AF0-41C2-9EE1-B10D931DECD0@freebsd.org> <20210422101616.c3i7dn6gfm6hhner@aching.in.mat.cc>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Apr 22, 2021 at 12:16:16PM +0200, Mathieu Arnold wrote: > On Thu, Apr 22, 2021 at 12:10:26PM +0200, Michael Gmelin wrote: > > ... > > I agree that author should only be used if a submission is applied > > unaltered (pull request/git patch). > > Submitted by has always been for submission that was unaltered (or only > slightly altered) if it was only the basis of the work, then "submitted > by" is not to be used, in those cases, reported by is the correct > choice. Not really, no. "PR:/Submitted by:" is canonical combination, yet most of us modify submitted patches because they almost always need more work. ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YIFTYpPMe43y6tU9>