On Monday, December 01, 2014 04:15:49 PM Hans Petter Selasky wrote:
> Hi John,
> 
> These patches were sent to the -current mailing list for review for 12
> days. No big issues were mentioned.
> 
> Also a review has been made at "reviews.freebsd.org":
> https://reviews.freebsd.org/D1243
> 
> No response yet.
> 
> If I missed someone for the review process please let me know.

Even if no one comments, please include the review URL in the "Differential 
Review" field.  Also, I think you could probably have put 'Discussed with:  
adrian, kmacy' in the commit.  Arguably from the followups in that thread you 
might have been able to put "Reviewed by: adrian" as well if this is just the 
cleanup patches that he said were ok.

-- 
John Baldwin
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to