svn commit: r284412 - in vendor-sys/illumos/dist/uts/common/fs/zfs: . sys

Andriy Gapon avg at FreeBSD.org
Mon Jun 15 13:50:00 UTC 2015


On 15/06/2015 16:42, Ed Maste wrote:
> On 15 June 2015 at 09:03, Andriy Gapon <avg at freebsd.org> wrote:
>>
>> I guess I could extend my script to expand 5911 to
>> https://www.illumos.org/issues/5911
> 
> Yeah, I think that would be useful -- although it's probably too long
> to go in the first line, then.
> 
> I guess the problem is that the upstream Illumos commits follow a
> certain convention, and that meaning/context is lost when they come
> into FreeBSD. Linking to the original issue should take care of that.
> 

Yes, indeed.  Regular ZFS/FreeBSD hackers are aware of the conventions and
places to look, but most of committers do not necessarily know of them.

So, this is the format that I will try use (using this commit as an example):
========================================
5911 ZFS "hangs" while deleting file

Reviewed by: Bayard Bell <buffer.g.overflow at gmail.com>
Reviewed by: Alek Pinchuk <alek at nexenta.com>
Reviewed by: Simon Klinkert <simon.klinkert at gmail.com>
Reviewed by: Dan McDonald <danmcd at omniti.com>
Approved by: Richard Lowe <richlowe at richlowe.net>
Author: Matthew Ahrens <mahrens at delphix.com>

illumos/illumos-gate at 46e1baa6cf6d5432f5fd231bb588df8f9570c858
https://www.illumos.org/issues/5911
========================================

-- 
Andriy Gapon


More information about the svn-src-vendor mailing list