Date: Fri, 30 Aug 2019 17:44:26 -0700 From: Cy Schubert <Cy.Schubert@cschubert.com> To: Ian Lepore <ian@freebsd.org>, Mateusz Guzik <mjguzik@gmail.com>, Oliver Pinter <oliver.pinter@hardenedbsd.org> Cc: "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r351471 - in head/sys: kern sys Message-ID: <EB07A728-199F-4F74-BD32-28CDCC2D4C27@cschubert.com> In-Reply-To: <6035df87490ebb7429f76d57a3a5d33e86c06b56.camel@freebsd.org> References: <201908250511.x7P5BiPl016074@repo.freebsd.org> <CAPQ4fftD57MXX8BpJB2RxNBdR2Ur144mSkB=nMiyFJU4KDs0=w@mail.gmail.com> <CAGudoHG8%2BrnK8j8BwD5HA6CyGoXMyipu0=upXYsuaKJV7-wcXQ@mail.gmail.com> <6035df87490ebb7429f76d57a3a5d33e86c06b56.camel@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On August 30, 2019 3:09:06 PM PDT, Ian Lepore <ian@freebsd=2Eorg> wrote: >On Sat, 2019-08-31 at 00:03 +0200, Mateusz Guzik wrote: >> On 8/25/19, Oliver Pinter <oliver=2Epinter@hardenedbsd=2Eorg> wrote: >> > On Sunday, August 25, 2019, Mateusz Guzik <mjg@freebsd=2Eorg> wrote: >> >=20 >> > > Author: mjg >> > > Date: Sun Aug 25 05:11:43 2019 >> > > New Revision: 351471 >> > > URL: https://svnweb=2Efreebsd=2Eorg/changeset/base/351471 >> > >=20 >> > > Log: >> > > vfs: add vholdnz (for already held vnodes) >> >=20 >> >=20 >> > Why? >> >=20 >> > (Yes, is can read the real reason in phabricator, but the >> > phabricator >> > didn't considered a persistent information in relation to commit >> > message=2E >> > Review helper tool just comes and goes as time goes forward, but >> > commit >> > messages survives the repo conversions to other VCS=2E=2E=2E) >> >=20 >> > And once you have taken the time to write a correct description in >> > phabricator, it would be really really nice and helpful if you >> > would copy >> > them into commit message=2E >> >=20 >>=20 >> I think it's a fair point that if I included this info in the review >> it >> should make it's way into the commit message=2E >>=20 > >When I open a new phab review, I specifically write the one-line >summary and text that follows with the idea that I'm going to cut and >paste that exact text as the commit message (mostly because it's easier >to just write all that stuff once)=2E If there are things I want to say >there that don't make sense for the commit I typically add them as a >separate comment right after I open the initial review=2E > >-- Ian I like to start out with a one line summary if possible, subsequently prov= iding more detail=2E The first paragraph or two of a phab review could/should be written to sim= ply cut and paste that into the commit message=2E I try to aim for this=2E I suppose when a person has been working on some code for a while what app= ears obvious may not be so obvious to others at first=2E It's an easy trap = to fall into=2E Something I catch myself at times=2E --=20 Pardon the typos and autocorrect, small keyboard in use=2E Cheers, Cy Schubert <Cy=2ESchubert@cschubert=2Ecom> FreeBSD UNIX: <cy@FreeBSD=2Eorg> Web: http://www=2EFreeBSD=2Eorg The need of the many outweighs the greed of the few=2E
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?EB07A728-199F-4F74-BD32-28CDCC2D4C27>