Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 9 Sep 2012 22:41:52 -0400
From:      Eitan Adler <eadler@freebsd.org>
To:        Doug Barton <dougb@freebsd.org>
Cc:        svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Martin Matuska <mm@freebsd.org>, svn-src-stable-9@freebsd.org
Subject:   Re: svn commit: r240225 - stable/9/sys/cddl/compat/opensolaris/sys
Message-ID:  <CAF6rxgnhQB93VoWjai9OZd43C=Gauu0jFF%2BbyRKnoFD37ukMDA@mail.gmail.com>
In-Reply-To: <504D33AA.8010707@FreeBSD.org>
References:  <201209080241.q882foC1005948@svn.freebsd.org> <504B9AC3.2070705@FreeBSD.org> <504B9E4C.6030301@FreeBSD.org> <CAF6rxgmDRZ=ZaDMF6PJCjYdDPVCcw-zaRXff_zEeD8v_Ciw5MQ@mail.gmail.com> <504D33AA.8010707@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 9 September 2012 20:26, Doug Barton <dougb@freebsd.org> wrote:
> While reasonable minds can differ on this topic, I regard that as a
> reason _not_ to include the PR number in merges. IMO it's far better to
> have one line "MFCs complete" than it is to repeat the same commit
> message verbatim in GNATS N times.

It is helpful to me when people forget to close the PR. It is also
helpful when tracking down a bug that was fixed in some versions and
not others. This is especially annoying when a fix was for example,
MFCed to 7 and 8, but not 9. This happens more than you would think.
:)

-- 
Eitan Adler
Source & Ports committer
X11, Bugbusting teams



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAF6rxgnhQB93VoWjai9OZd43C=Gauu0jFF%2BbyRKnoFD37ukMDA>