svn commit: r272467 - in head/sys: cddl/contrib/opensolaris/uts/common/fs/zfs fs/nfs fs/nfsserver

Ian Lepore ian at FreeBSD.org
Fri Oct 3 17:23:28 UTC 2014


On Sat, 2014-10-04 at 01:17 +0800, Marcelo Araujo wrote:
> Hey,
> 
> Yes I have, you could check here: https://reviews.freebsd.org/D798
> 
> I should added the phabric on the commit log. My bad!
> 

IMO, no amount of verbiage in phabricator linked to from a commit
message is a substitute for a proper detailed commit message.

-- Ian

> Best Regards,
> On Oct 4, 2014 1:12 AM, "Bryan Drewery" <bdrewery at freebsd.org> wrote:
> 
> > On 10/2/2014 9:24 PM, Marcelo Araujo wrote:
> > > Author: araujo (ports committer)
> > > Date: Fri Oct  3 02:24:41 2014
> > > New Revision: 272467
> > > URL: https://svnweb.freebsd.org/changeset/base/272467
> > >
> > > Log:
> > >   Fix failures and warnings reported by newpynfs20090424 test tool.
> > >   This fix addresses only issues with the pynfs reports, none of these
> > >   issues are know to create problems for extant real clients.
> >
> > Do you have more details than "fix bugs"? :)
> >
> > Even which failures were fixed would be helpful to document.
> >
> > >
> > >   Submitted by:       Bart Hsiao <bart.hsiao at gmail.com>
> > >   Reworked by:        myself
> > >   Reviewed by:        rmacklem
> > >   Approved by:        rmacklem
> > >   Sponsored by:       QNAP Systems Inc.
> > >
> >
> >
> > --
> > Regards,
> > Bryan Drewery
> >
> >




More information about the svn-src-head mailing list