Date: Wed, 27 Jan 2021 14:09:44 -0600 From: Kyle Evans <kevans@freebsd.org> To: Jessica Clarke <jrtc27@freebsd.org> Cc: John Baldwin <jhb@freebsd.org>, Lutz Donnerhacke <lutz@donnerhacke.de>, Lutz Donnerhacke <donner@freebsd.org>, src-committers <src-committers@freebsd.org>, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 5fe433a6e4d8 - main - netgraph/ng_nat: Add RFC 6598/Carrier Grade NAT support Message-ID: <CACNAnaGzjUjZms2YqdrziH1iJdBp9MNrjk45kxo%2BerJBn_p1pA@mail.gmail.com> In-Reply-To: <76B5E43D-EB93-4739-BDD1-BECA4340CCDC@freebsd.org> References: <202101241941.10OJfCGA013081@gitrepo.freebsd.org> <20210124194640.GA6527@belenus.iks-jena.de> <CACNAnaF_SjVZ0BhBQBRdZgQrJ28kzNCQHO%2B5V=1UJQceXvDY6w@mail.gmail.com> <d3bae427-3235-3e91-be23-17aa839f81fa@FreeBSD.org> <76B5E43D-EB93-4739-BDD1-BECA4340CCDC@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jan 27, 2021 at 1:59 PM Jessica Clarke <jrtc27@freebsd.org> wrote: > > On 27 Jan 2021, at 19:55, John Baldwin <jhb@freebsd.org> wrote: > > On 1/24/21 11:51 AM, Kyle Evans wrote: > >> On Sun, Jan 24, 2021 at 1:47 PM Lutz Donnerhacke <lutz@donnerhacke.de> wrote: > >> We switched up the recommendations[0][1] to advocate for what you've > >> done here ~2 weeks ago -- so yes, this is good. :-) > > > > Note that "Submitted by" may still be needed when there are multiple > > authors who contribute to a patch. (And I do kind of think pulling > > it from the template was a bit hasty as it didn't take those cases > > into account.) I do think we want Author to reflect the primary > > author of the change though. > > The git world (or at least the GitHub world) seems to be settling in > Co-authored-by: for that, which is more accurate. > Indeed, whoops- I hadn't thought of specifically noting something for co-authors. =-( I agree with switching to Co-authored-by; some tooling (e.g. GitHub, GitLab) will recognize this and do something appropriate with it if it can identify the coauthors. IMO "Submitted by" is an awkward verbiage for this, as "submitted" would (to me) typically imply the very specific action of presenting the patch and working it through the system (i.e. reviews), and these sets of people don't always match. That's getting pretty pedantic, but I suspect I'm not the only one that makes this kind of association and "co-authors" more clearly spells out how these individuals may be related w.r.t. any given patch in a more objective fashion. Thanks, Kyle Evans
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACNAnaGzjUjZms2YqdrziH1iJdBp9MNrjk45kxo%2BerJBn_p1pA>