On 15/06/2015 16:42, Ed Maste wrote: > On 15 June 2015 at 09:03, Andriy Gapon <a...@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.overf...@gmail.com> Reviewed by: Alek Pinchuk <a...@nexenta.com> Reviewed by: Simon Klinkert <simon.klink...@gmail.com> Reviewed by: Dan McDonald <dan...@omniti.com> Approved by: Richard Lowe <richl...@richlowe.net> Author: Matthew Ahrens <mahr...@delphix.com> illumos/illumos-gate@46e1baa6cf6d5432f5fd231bb588df8f9570c858 https://www.illumos.org/issues/5911 ======================================== -- Andriy Gapon _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
