From owner-cvs-all Wed Oct 31 11: 3:10 2001 Delivered-To: cvs-all@freebsd.org Received: from whale.sunbay.crimea.ua (whale.sunbay.crimea.ua [212.110.138.65]) by hub.freebsd.org (Postfix) with ESMTP id B107137B403; Wed, 31 Oct 2001 11:02:58 -0800 (PST) Received: (from ru@localhost) by whale.sunbay.crimea.ua (8.11.6/8.11.2) id f9VJ2hu53301; Wed, 31 Oct 2001 21:02:43 +0200 (EET) (envelope-from ru) Date: Wed, 31 Oct 2001 21:02:43 +0200 From: Ruslan Ermilov To: Poul-Henning Kamp Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sbin/natd natd.8 natd.c Message-ID: <20011031210243.N91951@sunbay.com> References: <20011031202106.G91951@sunbay.com> <73893.1004553742@critter.freebsd.dk> <20011031205743.L91951@sunbay.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <20011031205743.L91951@sunbay.com>; from ru@FreeBSD.org on Wed, Oct 31, 2001 at 08:57:43PM +0200 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Grr, this was intended to be sent in private, sorry. :( On Wed, Oct 31, 2001 at 08:57:43PM +0200, Ruslan Ermilov wrote: > On Wed, Oct 31, 2001 at 07:42:22PM +0100, Poul-Henning Kamp wrote: > > In message <20011031202106.G91951@sunbay.com>, Ruslan Ermilov writes: > > > > > > >Please back this change out, indicated by: ``Requested by: ru''. > > > > No. > > > You can't just say "no" here, as "every disputed change should be > backed out if requested by a maintainer", but see below. > > > >I don't agree with this change; > > > > Well, I agree with this change. > > > > >this is not what Flemming submitted. > > > > No, I changed it with Flemmings permission. > > > I wasn't aware, and you didn't mention it in the commitlog. > > > >His patch added the -nolog_ipfw_denied, while you changed this to > > >-log_ipfw_denied, breaking POLA (the current behavior of natd), > > >which is BAD, as this message (usually) indicates a misconfigured > > >firewall. > > > > No, Adding negative options breaks POLA. > > > No, changing the current behavior breaks POLA. > > > >As the maintainer of natd(8), I would appreciate it if you mailed > > >me about your intentions to commit this. > > > > Your totally unflexible behaviour in the discussion with Flemming > > (and me) about this change earns you no credit as maintainer. > > > Unflexible? We were disputing the change, I offered an alternate > solution which was rejected, then I swamped with some non-FreeBSD > related things, and you just committed this without even telling > me about your intentions. I'm sure you won't behav like this if > Flemming wasn't Dutch. > > > >I re-opened the PR in question, and will look into this soon after > > >I return from BSDCon/Euro. > > > > Lets discuss it at BSDcon/Euro then and leave the tree in peace > > until then. > > > Please make this option the default then, and let's discuss it at > the Con. > > > Cheers, > -- > Ruslan Ermilov Oracle Developer/DBA, > ru@sunbay.com Sunbay Software AG, > ru@FreeBSD.org FreeBSD committer, > +380.652.512.251 Simferopol, Ukraine > > http://www.FreeBSD.org The Power To Serve > http://www.oracle.com Enabling The Information Age -- Ruslan Ermilov Oracle Developer/DBA, ru@sunbay.com Sunbay Software AG, ru@FreeBSD.org FreeBSD committer, +380.652.512.251 Simferopol, Ukraine http://www.FreeBSD.org The Power To Serve http://www.oracle.com Enabling The Information Age To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message