From owner-svn-src-all@FreeBSD.ORG Mon Feb 16 07:47:01 2015 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 1E6B2B17; Mon, 16 Feb 2015 07:47:01 +0000 (UTC) Received: from cell.glebius.int.ru (glebius.int.ru [81.19.69.10]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "cell.glebius.int.ru", Issuer "cell.glebius.int.ru" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 78F70CF1; Mon, 16 Feb 2015 07:46:59 +0000 (UTC) Received: from cell.glebius.int.ru (localhost [127.0.0.1]) by cell.glebius.int.ru (8.14.9/8.14.9) with ESMTP id t1G7ku98049122 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Mon, 16 Feb 2015 10:46:56 +0300 (MSK) (envelope-from glebius@FreeBSD.org) Received: (from glebius@localhost) by cell.glebius.int.ru (8.14.9/8.14.9/Submit) id t1G7kuJk049121; Mon, 16 Feb 2015 10:46:56 +0300 (MSK) (envelope-from glebius@FreeBSD.org) X-Authentication-Warning: cell.glebius.int.ru: glebius set sender to glebius@FreeBSD.org using -f Date: Mon, 16 Feb 2015 10:46:56 +0300 From: Gleb Smirnoff To: Craig Rodrigues Subject: Re: svn commit: r276747 - head/sys/netpfil/pf Message-ID: <20150216074656.GE15484@FreeBSD.org> References: <201501060903.t06934qp081875@svn.freebsd.org> <20150122012709.GM15484@FreeBSD.org> <54C16715.6060701@gmx.com> <20150122222314.GO15484@FreeBSD.org> <20150215190100.GQ15484@FreeBSD.org> <20150216012528.GA15484@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Cc: "svn-src-head@freebsd.org" , "svn-src-all@freebsd.org" , Nikos Vassiliadis , "src-committers@freebsd.org" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Feb 2015 07:47:01 -0000 On Sun, Feb 15, 2015 at 11:36:17PM -0800, Craig Rodrigues wrote: C> On Sun, Feb 15, 2015 at 5:25 PM, Gleb Smirnoff wrote: C> C> > On Sun, Feb 15, 2015 at 01:33:19PM -0800, Craig Rodrigues wrote: C> > C> By the way, it would be helpful if you could provide feedback in C> > C> Phabricator. When I created those Phabricator reviews, I added C> > C> you as a reviewer to all of them, so you can't say that you didn't see C> > C> the patches. C> > C> You did not provide feedback on any of them: C> > C> C> > C> https://reviews.freebsd.org/D1309 C> > C> https://reviews.freebsd.org/D1312 C> > C> https://reviews.freebsd.org/D1313 C> > C> https://reviews.freebsd.org/D1315 C> > C> C> > C> Please take some time to go and provide feedback in those C> > C> reviews, so that a better patch can be made that makes you happy. C> > C> > Did you address all problems that arised after code was committed? C> > C> > Please do, otherwise my review would require me to cut-n-paste from C> > my own emails. C> > C> C> Yes, please cut and paste from your e-mails, and put in the reviews. No, I will not do this. You know right now that you have problems in the aforementioned phab URLs, and you are asking me to look at patches at to point them out to you. This is your task, not mine. Please address all already known problems and update the phab revisions. C> It's easier to follow in the individual reviews because there are different C> changes C> in each review, rather than one big revert, which is what you did. I'm already starting to repeat myself. I did a big revert, because the first change wasn't compilable, later changes fixed compilation failures, but introduced unacceptable bugs. That's why I was forced to back out all chain. -- Totus tuus, Glebius.