Date: Sun, 22 Feb 2015 23:59:52 -0800 From: Garrett Cooper <yaneurabeya@gmail.com> To: John Baldwin <jhb@freebsd.org> Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers <src-committers@freebsd.org>, svn-src-stable-9@freebsd.org, Garrett Cooper <ngie@freebsd.org> Subject: Re: svn commit: r278718 - in stable/9: etc/rc.d sbin share/man/man4 share/mk sys/modules/geom tools/build/mk tools/build/options Message-ID: <C123BC3A-6CC4-4FFA-86B5-AA510061CF75@gmail.com> In-Reply-To: <5328252.MWfFGgsrnY@ralph.baldwin.cx> References: <201502132136.t1DLaHLi008470@svn.freebsd.org> <5328252.MWfFGgsrnY@ralph.baldwin.cx>
next in thread | previous in thread | raw e-mail | index | archive | help
--Apple-Mail=_D41C2C07-DF15-44A9-BEAE-9B6C28587346 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=windows-1252 On Feb 16, 2015, at 8:11, John Baldwin <jhb@freebsd.org> wrote: > On Friday, February 13, 2015 09:36:17 PM Garrett Cooper wrote: >> Author: ngie >> Date: Fri Feb 13 21:36:16 2015 >> New Revision: 278718 >> URL: https://svnweb.freebsd.org/changeset/base/278718 >>=20 >> Log: >> MFC r278717: >>=20 >> r278717: >>=20 >> MFC r277678: >>=20 >> r277678: >>=20 >> Add MK_CCD knob for building and installing ccd(4), ccdconfig, = etc >>=20 >> Sponsored by: EMC / Isilon Storage Division >=20 > I believe you are supposed to merge from HEAD to 9, not from 10 to 9. =20= I try to do that where it makes sense. Merging and redoing some of the = work I did going from head to head-1 increases the likelihood of error = (especially with all of the build system refactoring that took place in = the past year). > But also, I find these log messages quite noisy. I much prefer just: >=20 > MFC <head rev>: > <original log message> >=20 > Where the <original log message> is not extra-indented but is = formatted=20 > similar to a normal commit. I was doing that [subjectively] for readability, but that=92s ok, I=92ll = take out the extra indentation. > On a more general note, if I'm merging a change with several followup = fixes, I=20 > 1) always merge the entire batch of changes so as not to leave stable/ = in a=20 > broken state (most folks also do this), and 2) I don't cut and paste = all N=20 > logs verbatim. This tends to be very hard to read. Instead, I do = 'MFC <list=20 > of head revs>' and then use a brief summary of the change being = merged. Often=20 > this means using the log message of the first change (which introduces = the new=20 > feature and explains it) but omitting short descriptions of specific = bugs=20 > fixed (which aren't very useful to someone reading the stable log = message as=20 > the commit in question is introducing the needed feature in its fixed = state.) >=20 > This does require a bit more effort editorial wise, but I think it = results in=20 > commit logs for stable that are more readable. Sure. I wish that there was a set format for this. All of this is scripted for = me, so merging is a trivial task =97 it=92s just a bit confusing when I = have 5 different people telling me my MFC message is wrong, but oh well=85= it=92s just a script. Lol. Thanks :), --Apple-Mail=_D41C2C07-DF15-44A9-BEAE-9B6C28587346 Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP using GPGMail -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQEcBAEBCgAGBQJU6t34AAoJEMZr5QU6S73eCnQIAKDoJ7PcyrMok1pubDHtVfWb V9F6YcJt0oTrQRFCyMiya7ITTX2bhBmIxO9fu/F+FwqVDTuzmnGlOw7nGGa0HTf/ d85br0KKS4O7cMm1/jGiNLj87AiwXrPK8YE2bYuhYbMzs95Efyessf3Fo2LBWPqh 6PvBiwJbz8mAqG3nZrlUgwLWOkOlunrfWo/oWvlWFucJcISspnawtiZ+2qP4C1j8 vGd1m8YdN7CJtDTrbiYunPLOgDm4IkyKo6EdhbJwteCiL/0qdenvfEv6z7RpgObS JKpzt1wZ1P4f5t7dWOUQ0CY11FI8jE/KAs3plCRDrh+z+jzcnC7PX2SoeskzQoE= =mPjj -----END PGP SIGNATURE----- --Apple-Mail=_D41C2C07-DF15-44A9-BEAE-9B6C28587346--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?C123BC3A-6CC4-4FFA-86B5-AA510061CF75>