Date: Tue, 26 Jan 2016 18:23:12 +0000 From: Alexey Dokuchaev <danfe@FreeBSD.org> To: Jason Helfman <jgh@FreeBSD.org> Cc: Mathieu Arnold <mat@freebsd.org>, "svn-ports-head@freebsd.org" <svn-ports-head@freebsd.org>, "svn-ports-all@freebsd.org" <svn-ports-all@freebsd.org>, Kurt Jaeger <pi@freebsd.org>, "ports-committers@freebsd.org" <ports-committers@freebsd.org> Subject: Re: svn commit: r407237 - head/mail/imaputils Message-ID: <20160126182312.GA54534@FreeBSD.org> In-Reply-To: <CAMuy=%2BgjxtuEviVKavXhp6TqASn3010d88S0YNJSA2E15_-gbw@mail.gmail.com> References: <201601251910.u0PJAdeU003331@repo.freebsd.org> <20160126141038.GA46193@FreeBSD.org> <4C5F6D463ACD91B52D4AC5AF@ogg.in.absolight.net> <20160126153932.GC46193@FreeBSD.org> <CAMuy=%2BgjxtuEviVKavXhp6TqASn3010d88S0YNJSA2E15_-gbw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jan 26, 2016 at 10:08:26AM -0800, Jason Helfman wrote:
> On Tue, Jan 26, 2016 at 7:39 AM, Alexey Dokuchaev wrote:
> > 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[: foo]" line first, and PR line after. [...]
>
> I did put a PR in for this issue, which could possibly be leveraged and
> clear the commit confusion. The PR adds a "Reported by" to the FreeBSD
> template.
I don't see the need to put a reference to PR on a follow-up commit that
just had cleaned things up a bit (avoid needless assignment in this case).
> These are problem reports, and I think it is appropriate to have that
> someone reported a problem report.
Correct, but this is irrelevant to the commit in question (r407237).
> If there is a submission associated with the report, it then is very
> clear that it was reported by one person, and another individual
> submitted a fix.
One of the long-standing traditions of FreeBSD is that we kindly ask folks
not just report the problem ("hey, port foo/bar needs an update!") but also
provide a patch (that is, submission). Hence in general it is *not* clear
that PR was reported by one person, and another individual submitted a fix:
we simply do not (rightfully) encourage this type of workflow, we ask for
patches. :-)
> I put this in awhile ago, and there was some discussion about it, however
> the PR has not yet been committed at this point. Whether or not this is a
> good solution, or there is something better, is another matter entirely.
Sorry, I don't think I quite understand this paragraph (particularly, which
PR?).
./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20160126182312.GA54534>