svn commit: r292975 - stable/10/sys/dev/pci

[Oliver Pinter]

On Thursday, December 31, 2015, NGie Cooper <yaneurabeya at gmail.com> wrote:

>
> > On Dec 31, 2015, at 08:42, John Baldwin <john at baldwin.cx <javascript:;>>
> wrote:
> >
> > On Thursday, December 31, 2015 03:55:02 AM Garrett Cooper wrote:
> >> Author: ngie
> >> Date: Thu Dec 31 03:55:02 2015
> >> New Revision: 292975
> >> URL: https://svnweb.freebsd.org/changeset/base/292975
> >>
> >> Log:
> >>  MFC r278860:
> >>  r278860 (by jmg):
> >>
> >>  remove NULL check as M_WAITOK will not return NULL
> >>
> >>  Reviewed by:        jhb
> >>  Sponsored by:       FreeBSD Foundation
> >
> > Including the metadata from the HEAD commit in the MFC can be
> misleading, and
> > I think it is best ommitted.  For example, I did not review this MFC (and
> > reviewing MFC's in their own right matters since they may require
> additional
> > work than commits to HEAD due to ABI concerns), nor did the Foundation
> sponsor
> > the MFC.  Rather, I think MFCs should only specify metadata related to
> the
> > actual merge itself.
>
> Agreed. I’ve changed my mfc_log script to skip over some more unnecessary
> svn log metadata:
> https://github.com/yaneurabeya/scratch/commit/4f897efb96f96f951ff3909857f67e63b24b733a
> .
> Thanks :)!
> -NGie



It would be nice to commit this script under tools directory.

> _______________________________________________
> svn-src-stable-10 at freebsd.org <javascript:;> mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
> To unsubscribe, send any mail to "
> svn-src-stable-10-unsubscribe at freebsd.org <javascript:;>"